Protokoll #4888

{
    "event": "PreToolUse",
    "tool_name": "Edit",
    "tool_input": {
        "file_path": "\/var\/www\/dev.campus.systemische-tools.de\/src\/Framework\/Router.php",
        "old_string": "    \/**\n     * Resolve a controller using the container or direct instantiation.\n     *\n     * @param class-string $controllerClass\n     *\/\n    protected function resolveController(string $controllerClass): Controller\n    {\n        \/\/ Use container if available (with autowiring)\n        if ($this->container !== null) {\n            return $this->container->get($controllerClass);\n        }\n\n        \/\/ Fallback to direct instantiation\n        return new $controllerClass();\n    }",
        "new_string": "    \/**\n     * Resolve a controller using the container or direct instantiation.\n     *\n     * @param class-string<Controller> $controllerClass\n     *\/\n    protected function resolveController(string $controllerClass): Controller\n    {\n        \/\/ Use container if available (with autowiring)\n        if ($this->container !== null) {\n            \/** @var Controller *\/\n            return $this->container->get($controllerClass);\n        }\n\n        \/\/ Fallback to direct instantiation\n        return new $controllerClass();\n    }"
    }
}

{
    "tool_response": {
        "filePath": "\/var\/www\/dev.campus.systemische-tools.de\/src\/Framework\/Router.php",
        "oldString": "    \/**\n     * Resolve a controller using the container or direct instantiation.\n     *\n     * @param class-string $controllerClass\n     *\/\n    protected function resolveController(string $controllerClass): Controller\n    {\n        \/\/ Use container if available (with autowiring)\n        if ($this->container !== null) {\n            return $this->container->get($controllerClass);\n        }\n\n        \/\/ Fallback to direct instantiation\n        return new $controllerClass();\n    }",
        "newString": "    \/**\n     * Resolve a controller using the container or direct instantiation.\n     *\n     * @param class-string<Controller> $controllerClass\n     *\/\n    protected function resolveController(string $controllerClass): Controller\n    {\n        \/\/ Use container if available (with autowiring)\n        if ($this->container !== null) {\n            \/** @var Controller *\/\n            return $this->container->get($controllerClass);\n        }\n\n        \/\/ Fallback to direct instantiation\n        return new $controllerClass();\n    }",
        "originalFile": "<?php\n\nnamespace Framework;\n\nclass Router\n{\n    protected array $routes = [];\n    protected ?Container $container = null;\n\n    \/**\n     * Set the DI container for controller resolution.\n     *\/\n    public function setContainer(Container $container): void\n    {\n        $this->container = $container;\n    }\n\n    public function get(string $path, array|callable $handler): self\n    {\n        return $this->add('GET', $path, $handler);\n    }\n\n    public function post(string $path, array|callable $handler): self\n    {\n        return $this->add('POST', $path, $handler);\n    }\n\n    public function put(string $path, array|callable $handler): self\n    {\n        return $this->add('PUT', $path, $handler);\n    }\n\n    public function delete(string $path, array|callable $handler): self\n    {\n        return $this->add('DELETE', $path, $handler);\n    }\n\n    public function add(string $method, string $path, array|callable $handler): self\n    {\n        $this->routes[] = [\n            'method' => $method,\n            'path' => $path,\n            'handler' => $handler,\n        ];\n\n        return $this;\n    }\n\n    public function dispatch(string $uri, string $method): void\n    {\n        foreach ($this->routes as $route) {\n            if ($route['method'] !== $method) {\n                continue;\n            }\n\n            $pattern = $this->convertToRegex($route['path']);\n\n            if (preg_match($pattern, $uri, $matches)) {\n                array_shift($matches);\n                $this->handle($route['handler'], $matches);\n\n                return;\n            }\n        }\n\n        http_response_code(404);\n        echo '404 Not Found';\n    }\n\n    protected function convertToRegex(string $path): string\n    {\n        \/\/ Support {name:pattern} syntax for custom patterns (e.g., {path:.*})\n        $pattern = preg_replace_callback('\/\\{([a-zA-Z]+):([^}]+)\\}\/', function ($matches) {\n            return '(' . $matches[2] . ')';\n        }, $path);\n\n        \/\/ Support simple {name} syntax\n        $pattern = preg_replace('\/\\{([a-zA-Z]+)\\}\/', '([^\/]+)', $pattern);\n\n        return '#^' . $pattern . '$#';\n    }\n\n    protected function handle(array|callable $handler, array $params): void\n    {\n        if (is_callable($handler)) {\n            call_user_func_array($handler, $params);\n\n            return;\n        }\n\n        [$controllerClass, $method] = $handler;\n        $controller = $this->resolveController($controllerClass);\n        \/** @var callable $callback *\/\n        $callback = [$controller, $method];\n        call_user_func_array($callback, $params);\n    }\n\n    \/**\n     * Resolve a controller using the container or direct instantiation.\n     *\n     * @param class-string $controllerClass\n     *\/\n    protected function resolveController(string $controllerClass): Controller\n    {\n        \/\/ Use container if available (with autowiring)\n        if ($this->container !== null) {\n            return $this->container->get($controllerClass);\n        }\n\n        \/\/ Fallback to direct instantiation\n        return new $controllerClass();\n    }\n}\n",
        "structuredPatch": [
            {
                "oldStart": 98,
                "oldLines": 12,
                "newStart": 98,
                "newLines": 13,
                "lines": [
                    "     \/**",
                    "      * Resolve a controller using the container or direct instantiation.",
                    "      *",
                    "-     * @param class-string $controllerClass",
                    "+     * @param class-string<Controller> $controllerClass",
                    "      *\/",
                    "     protected function resolveController(string $controllerClass): Controller",
                    "     {",
                    "         \/\/ Use container if available (with autowiring)",
                    "         if ($this->container !== null) {",
                    "+            \/** @var Controller *\/",
                    "             return $this->container->get($controllerClass);",
                    "         }",
                    " "
                ]
            }
        ],
        "userModified": false,
        "replaceAll": false
    }
}