{
"tool_response": {
"filePath": "\/var\/www\/dev.campus.systemische-tools.de\/src\/Domain\/DTO\/ContentOrderDTO.php",
"oldString": " \/**\n * Parse comma-separated collections or JSON array.\n *\n * @param string|array<string> $collections\n * @return array<string>\n *\/\n private static function parseCollections(mixed $collections): array\n {\n if (is_array($collections)) {\n return array_map('strval', $collections);\n }\n\n $collectionsStr = is_string($collections) ? $collections : '';\n\n if ($collectionsStr === '') {\n return [];\n }",
"newString": " \/**\n * Parse comma-separated collections or JSON array.\n *\n * @return array<string>\n *\/\n private static function parseCollections(mixed $collections): array\n {\n if (is_array($collections)) {\n \/** @var array<mixed> $collections *\/\n return array_values(array_map(fn(mixed $v): string => (string) $v, $collections));\n }\n\n if (!is_string($collections) || $collections === '') {\n return [];\n }",
"originalFile": "<?php\n\ndeclare(strict_types=1);\n\nnamespace Domain\\DTO;\n\nuse Domain\\ValueObject\\GenerationStatus;\nuse Domain\\ValueObject\\CritiqueStatus;\n\n\/\/ @responsibility: DTO für Content-Order-Daten aus Repository\n\nfinal readonly class ContentOrderDTO\n{\n \/**\n * @param array<string> $collections\n *\/\n public function __construct(\n public int $id,\n public string $title,\n public string $briefing,\n public string $model,\n public array $collections,\n public int $contextLimit,\n public GenerationStatus $generationStatus,\n public CritiqueStatus $critiqueStatus,\n public ?int $authorProfileId,\n public ?int $contractId,\n public ?int $structureId,\n public \\DateTimeImmutable $createdAt,\n public ?\\DateTimeImmutable $updatedAt,\n public ?int $latestVersionId = null,\n public ?string $authorProfileName = null,\n public ?string $contractName = null,\n public ?string $structureName = null,\n ) {}\n\n \/**\n * Create from database row.\n *\n * @param array<string, mixed> $row\n *\/\n public static function fromDatabaseRow(array $row): self\n {\n $id = isset($row['id']) ? (int) $row['id'] : 0;\n $title = isset($row['title']) ? (string) $row['title'] : '';\n $briefing = isset($row['briefing']) ? (string) $row['briefing'] : '';\n $model = isset($row['model']) ? (string) $row['model'] : '';\n $collections = self::parseCollections($row['collections'] ?? '');\n $contextLimit = isset($row['context_limit']) ? (int) $row['context_limit'] : 3;\n $generationStatusStr = isset($row['generation_status']) ? (string) $row['generation_status'] : 'idle';\n $critiqueStatusStr = isset($row['critique_status']) ? (string) $row['critique_status'] : 'idle';\n $createdAtStr = isset($row['created_at']) ? (string) $row['created_at'] : 'now';\n $updatedAtStr = isset($row['updated_at']) ? (string) $row['updated_at'] : null;\n\n return new self(\n id: $id,\n title: $title,\n briefing: $briefing,\n model: $model,\n collections: $collections,\n contextLimit: $contextLimit,\n generationStatus: GenerationStatus::from($generationStatusStr),\n critiqueStatus: CritiqueStatus::from($critiqueStatusStr),\n authorProfileId: isset($row['author_profile_id']) ? (int) $row['author_profile_id'] : null,\n contractId: isset($row['contract_id']) ? (int) $row['contract_id'] : null,\n structureId: isset($row['structure_id']) ? (int) $row['structure_id'] : null,\n createdAt: new \\DateTimeImmutable($createdAtStr),\n updatedAt: $updatedAtStr !== null ? new \\DateTimeImmutable($updatedAtStr) : null,\n latestVersionId: isset($row['latest_version_id']) ? (int) $row['latest_version_id'] : null,\n authorProfileName: isset($row['author_profile_name']) ? (string) $row['author_profile_name'] : null,\n contractName: isset($row['contract_name']) ? (string) $row['contract_name'] : null,\n structureName: isset($row['structure_name']) ? (string) $row['structure_name'] : null,\n );\n }\n\n \/**\n * Parse comma-separated collections or JSON array.\n *\n * @param string|array<string> $collections\n * @return array<string>\n *\/\n private static function parseCollections(mixed $collections): array\n {\n if (is_array($collections)) {\n return array_map('strval', $collections);\n }\n\n $collectionsStr = is_string($collections) ? $collections : '';\n\n if ($collectionsStr === '') {\n return [];\n }\n\n \/\/ Try JSON first\n $decoded = json_decode($collectionsStr, true);\n if (is_array($decoded)) {\n return array_map('strval', $decoded);\n }\n\n \/\/ Fall back to comma-separated\n return array_values(array_filter(array_map('trim', explode(',', $collectionsStr))));\n }\n\n \/**\n * Check if generation can be started.\n *\/\n public function canStartGeneration(): bool\n {\n return $this->generationStatus->canStart();\n }\n\n \/**\n * Check if critique can be started.\n *\/\n public function canStartCritique(): bool\n {\n return $this->critiqueStatus->canStart() && $this->latestVersionId !== null;\n }\n}\n",
"structuredPatch": [
{
"oldStart": 76,
"oldLines": 18,
"newStart": 76,
"newLines": 16,
"lines": [
" \/**",
" * Parse comma-separated collections or JSON array.",
" *",
"- * @param string|array<string> $collections",
" * @return array<string>",
" *\/",
" private static function parseCollections(mixed $collections): array",
" {",
" if (is_array($collections)) {",
"- return array_map('strval', $collections);",
"+ \/** @var array<mixed> $collections *\/",
"+ return array_values(array_map(fn(mixed $v): string => (string) $v, $collections));",
" }",
" ",
"- $collectionsStr = is_string($collections) ? $collections : '';",
"-",
"- if ($collectionsStr === '') {",
"+ if (!is_string($collections) || $collections === '') {",
" return [];",
" }",
" "
]
}
],
"userModified": false,
"replaceAll": false
}
}