Protokoll #4847

{
    "event": "PreToolUse",
    "tool_name": "Edit",
    "tool_input": {
        "file_path": "\/var\/www\/dev.campus.systemische-tools.de\/src\/Controller\/CriticsController.php",
        "old_string": "    public function update(string $id): void\n    {\n        $this->requireCsrf();\n\n        $result = $this->criticsUseCase->update(\n            id: (int) $id,\n            name: $_POST['name'] ?? '',\n            fokusText: $_POST['fokus'] ?? '',\n            promptId: !empty($_POST['prompt_id']) ? (int) $_POST['prompt_id'] : null,\n            sortOrder: (int) ($_POST['sort_order'] ?? 0),\n            isActive: isset($_POST['is_active'])\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $id);\n        exit;\n    }",
        "new_string": "    public function update(string $id): void\n    {\n        $this->requireCsrf();\n\n        $command = UpdateCriticCommand::fromRequest((int) $id, $_POST);\n        $errors = $command->validate();\n\n        if ($errors !== []) {\n            $_SESSION['error'] = implode(' ', $errors);\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $result = $this->criticsUseCase->update(\n            id: $command->id,\n            name: $command->name,\n            fokusText: $command->fokusText,\n            promptId: $command->promptId,\n            sortOrder: $command->sortOrder,\n            isActive: $command->isActive\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $id);\n        exit;\n    }"
    }
}

{
    "tool_response": {
        "filePath": "\/var\/www\/dev.campus.systemische-tools.de\/src\/Controller\/CriticsController.php",
        "oldString": "    public function update(string $id): void\n    {\n        $this->requireCsrf();\n\n        $result = $this->criticsUseCase->update(\n            id: (int) $id,\n            name: $_POST['name'] ?? '',\n            fokusText: $_POST['fokus'] ?? '',\n            promptId: !empty($_POST['prompt_id']) ? (int) $_POST['prompt_id'] : null,\n            sortOrder: (int) ($_POST['sort_order'] ?? 0),\n            isActive: isset($_POST['is_active'])\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $id);\n        exit;\n    }",
        "newString": "    public function update(string $id): void\n    {\n        $this->requireCsrf();\n\n        $command = UpdateCriticCommand::fromRequest((int) $id, $_POST);\n        $errors = $command->validate();\n\n        if ($errors !== []) {\n            $_SESSION['error'] = implode(' ', $errors);\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $result = $this->criticsUseCase->update(\n            id: $command->id,\n            name: $command->name,\n            fokusText: $command->fokusText,\n            promptId: $command->promptId,\n            sortOrder: $command->sortOrder,\n            isActive: $command->isActive\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $id);\n        exit;\n    }",
        "originalFile": "<?php\n\nnamespace Controller;\n\nuse Framework\\Controller;\nuse Infrastructure\\Persistence\\PromptsRepository;\nuse UseCases\\Command\\CreateCriticCommand;\nuse UseCases\\Command\\UpdateCriticCommand;\nuse UseCases\\Critics\\ManageCriticsUseCase;\n\nclass CriticsController extends Controller\n{\n    private ManageCriticsUseCase $criticsUseCase;\n    private PromptsRepository $promptsRepo;\n\n    public function __construct(\n        ?ManageCriticsUseCase $criticsUseCase = null,\n        ?PromptsRepository $promptsRepo = null\n    ) {\n        $this->criticsUseCase = $criticsUseCase ?? new ManageCriticsUseCase();\n        $this->promptsRepo = $promptsRepo ?? new PromptsRepository();\n    }\n\n    public function index(): void\n    {\n        $this->view('critics.index', [\n            'title' => 'Critics verwalten',\n            'critics' => array_map(fn ($dto) => $dto->toArray(), $this->criticsUseCase->getAll()),\n            'stats' => $this->criticsUseCase->getStatistics(),\n        ]);\n    }\n\n    public function criticsNew(): void\n    {\n        $this->view('critics.form', [\n            'title' => 'Neuer Critic',\n            'critic' => null,\n            'prompts' => $this->promptsRepo->findActivePrompts(),\n            'isEdit' => false,\n        ]);\n    }\n\n    public function store(): void\n    {\n        $this->requireCsrf();\n\n        $command = CreateCriticCommand::fromRequest($_POST);\n        $errors = $command->validate();\n\n        if ($errors !== []) {\n            $_SESSION['error'] = implode(' ', $errors);\n            header('Location: \/critics\/new');\n            exit;\n        }\n\n        $result = $this->criticsUseCase->create(\n            name: $command->name,\n            fokusText: $command->fokusText,\n            promptId: $command->promptId,\n            sortOrder: $command->sortOrder,\n            isActive: $command->isActive\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/new');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $result->id);\n        exit;\n    }\n\n    public function show(string $id): void\n    {\n        $critic = $this->criticsUseCase->getById((int) $id);\n        if ($critic === null) {\n            $this->notFound('Critic nicht gefunden');\n        }\n\n        $prompt = null;\n        if ($critic->promptId !== null) {\n            $prompt = $this->promptsRepo->findById($critic->promptId);\n        }\n\n        $this->view('critics.show', [\n            'title' => $critic->name,\n            'critic' => $critic->toArray(),\n            'prompt' => $prompt,\n            'recentCritiques' => $this->criticsUseCase->getRecentCritiques((int) $id, 10),\n        ]);\n    }\n\n    public function edit(string $id): void\n    {\n        $critic = $this->criticsUseCase->getById((int) $id);\n        if ($critic === null) {\n            $this->notFound('Critic nicht gefunden');\n        }\n\n        $this->view('critics.form', [\n            'title' => 'Bearbeiten: ' . $critic->name,\n            'critic' => $critic->toArray() + ['fokus_text' => $critic->fokusAsText()],\n            'prompts' => $this->promptsRepo->findActivePrompts(),\n            'isEdit' => true,\n        ]);\n    }\n\n    public function update(string $id): void\n    {\n        $this->requireCsrf();\n\n        $result = $this->criticsUseCase->update(\n            id: (int) $id,\n            name: $_POST['name'] ?? '',\n            fokusText: $_POST['fokus'] ?? '',\n            promptId: !empty($_POST['prompt_id']) ? (int) $_POST['prompt_id'] : null,\n            sortOrder: (int) ($_POST['sort_order'] ?? 0),\n            isActive: isset($_POST['is_active'])\n        );\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id . '\/edit');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics\/' . $id);\n        exit;\n    }\n\n    public function delete(string $id): void\n    {\n        $this->requireCsrf();\n\n        $result = $this->criticsUseCase->delete((int) $id);\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics\/' . $id);\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics');\n        exit;\n    }\n\n    public function toggle(string $id): void\n    {\n        $this->requireCsrf();\n\n        $result = $this->criticsUseCase->toggleActive((int) $id);\n\n        if (!$result->success) {\n            $_SESSION['error'] = $result->message;\n            header('Location: \/critics');\n            exit;\n        }\n\n        $_SESSION['success'] = $result->message;\n        header('Location: \/critics');\n        exit;\n    }\n}\n",
        "structuredPatch": [
            {
                "oldStart": 111,
                "oldLines": 13,
                "newStart": 111,
                "newLines": 22,
                "lines": [
                    "     {",
                    "         $this->requireCsrf();",
                    " ",
                    "+        $command = UpdateCriticCommand::fromRequest((int) $id, $_POST);",
                    "+        $errors = $command->validate();",
                    "+",
                    "+        if ($errors !== []) {",
                    "+            $_SESSION['error'] = implode(' ', $errors);",
                    "+            header('Location: \/critics\/' . $id . '\/edit');",
                    "+            exit;",
                    "+        }",
                    "+",
                    "         $result = $this->criticsUseCase->update(",
                    "-            id: (int) $id,",
                    "-            name: $_POST['name'] ?? '',",
                    "-            fokusText: $_POST['fokus'] ?? '',",
                    "-            promptId: !empty($_POST['prompt_id']) ? (int) $_POST['prompt_id'] : null,",
                    "-            sortOrder: (int) ($_POST['sort_order'] ?? 0),",
                    "-            isActive: isset($_POST['is_active'])",
                    "+            id: $command->id,",
                    "+            name: $command->name,",
                    "+            fokusText: $command->fokusText,",
                    "+            promptId: $command->promptId,",
                    "+            sortOrder: $command->sortOrder,",
                    "+            isActive: $command->isActive",
                    "         );",
                    " ",
                    "         if (!$result->success) {"
                ]
            }
        ],
        "userModified": false,
        "replaceAll": false
    }
}