Revert "Merge pull request 'KAN-11 Move remaining project metadata ownership to templates' (#94) from koda/kan-11-restliche-projekt-konfiguration-auf-templates-umstellen into epic/kan-3"

This reverts commit d39d21d303, reversing
changes made to a8952d7867.

app/Http/Requests/Api/MoveProjectTicketRequest.php

@@ -24,17 +24,12 @@ class MoveProjectTicketRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'ticket_status_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_statuses', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_statuses', 'id')->where('project_id', $project->id),
             ],
         ];
     }

app/Http/Requests/DestroyProjectTicketTypeRequest.php

@@ -29,17 +29,12 @@ class DestroyProjectTicketTypeRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'replacement_ticket_type_id' => [
                 'nullable',
                 'integer',
-                Rule::exists('ticket_types', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_types', 'id')->where('project_id', $project->id),
             ],
         ];
     }

app/Http/Requests/MoveProjectTicketRequest.php

@@ -29,17 +29,12 @@ class MoveProjectTicketRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'ticket_status_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_statuses', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_statuses', 'id')->where('project_id', $project->id),
             ],
             'redirect_to' => ['required', 'string', Rule::in(['backlog', 'board'])],
         ];

app/Http/Requests/StoreProjectTicketBranchRequest.php

@@ -2,10 +2,8 @@
 
 namespace App\Http\Requests;
 
-use App\Models\Project;
 use Illuminate\Contracts\Validation\ValidationRule;
 use Illuminate\Foundation\Http\FormRequest;
-use Illuminate\Validation\Rule;
 
 class StoreProjectTicketBranchRequest extends FormRequest
 {
@@ -24,21 +22,9 @@ class StoreProjectTicketBranchRequest extends FormRequest
      */
     public function rules(): array
     {
-        /** @var Project $project */
-        $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
-
         return [
             'branch_name' => ['required', 'string', 'max:255'],
-            'repository_connection_id' => [
-                'required',
-                'integer',
-                Rule::exists('repository_connections', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
-            ],
+            'repository_connection_id' => ['required', 'integer', 'exists:repository_connections,id'],
         ];
     }
 }

app/Http/Requests/StoreProjectTicketRequest.php

@@ -48,9 +48,6 @@ class StoreProjectTicketRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'acceptance_criteria' => ['nullable', 'array'],
@@ -69,23 +66,17 @@ class StoreProjectTicketRequest extends FormRequest
             'ticket_priority_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_priorities', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_priorities', 'id')->where('project_id', $project->id),
             ],
             'ticket_status_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_statuses', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_statuses', 'id')->where('project_id', $project->id),
             ],
             'ticket_type_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_types', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_types', 'id')->where('project_id', $project->id),
             ],
             'story_points' => ['nullable', 'integer', 'min:0'],
             'title' => ['required', 'string', 'max:255'],

app/Http/Requests/StoreTicketLinkRequest.php

@@ -28,9 +28,6 @@ class StoreTicketLinkRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'linked_ticket_id' => [
@@ -41,9 +38,7 @@ class StoreTicketLinkRequest extends FormRequest
             'ticket_link_type_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_link_types', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_link_types', 'id')->where('project_id', $project->id),
             ],
         ];
     }

app/Http/Requests/StoreTicketStatusRequest.php

@@ -26,17 +26,12 @@ class StoreTicketStatusRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'board_column_id' => [
                 'nullable',
                 'integer',
-                Rule::exists('board_columns', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('board_columns', 'id')->where('project_id', $project->id),
             ],
             'name' => ['required', 'string', 'max:255'],
         ];

app/Http/Requests/UpdateProjectTicketRequest.php

@@ -51,9 +51,6 @@ class UpdateProjectTicketRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'acceptance_criteria' => ['nullable', 'array'],
@@ -71,9 +68,7 @@ class UpdateProjectTicketRequest extends FormRequest
             'field_values.*.ticket_field_definition_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_field_definitions', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_field_definitions', 'id')->where('project_id', $project->id),
             ],
             'field_values.*.value_number' => ['nullable', 'numeric'],
             'field_values.*.value_user_id' => ['nullable', 'integer', Rule::exists('users', 'id')],
@@ -86,16 +81,12 @@ class UpdateProjectTicketRequest extends FormRequest
             'ticket_priority_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_priorities', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_priorities', 'id')->where('project_id', $project->id),
             ],
             'ticket_type_id' => [
                 'required',
                 'integer',
-                Rule::exists('ticket_types', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_types', 'id')->where('project_id', $project->id),
             ],
             'title' => ['required', 'string', 'max:255'],
             'story_points' => ['nullable', 'integer', 'min:0'],
@@ -150,7 +141,7 @@ class UpdateProjectTicketRequest extends FormRequest
 
                 $requestedTicketTypeId = Arr::get($validated, 'ticket_type_id');
                 $requestedTicketType = TicketType::query()
-                    ->where($project->project_template_id === null ? 'project_id' : 'project_template_id', $project->project_template_id ?? $project->id)
+                    ->where('project_id', $project->id)
                     ->find($requestedTicketTypeId);
 
                 if (

app/Http/Requests/UpdateTicketStatusTransitionsRequest.php

@@ -29,17 +29,12 @@ class UpdateTicketStatusTransitionsRequest extends FormRequest
     {
         /** @var Project $project */
         $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
 
         return [
             'allowed_ticket_status_ids' => ['nullable', 'array'],
             'allowed_ticket_status_ids.*' => [
                 'integer',
-                Rule::exists('ticket_statuses', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
+                Rule::exists('ticket_statuses', 'id')->where('project_id', $project->id),
             ],
         ];
     }

app/Http/Requests/ValidateProjectTicketBranchRequest.php

@@ -2,10 +2,8 @@
 
 namespace App\Http\Requests;
 
-use App\Models\Project;
 use Illuminate\Contracts\Validation\ValidationRule;
 use Illuminate\Foundation\Http\FormRequest;
-use Illuminate\Validation\Rule;
 
 class ValidateProjectTicketBranchRequest extends FormRequest
 {
@@ -24,21 +22,9 @@ class ValidateProjectTicketBranchRequest extends FormRequest
      */
     public function rules(): array
     {
-        /** @var Project $project */
-        $project = $this->route('project');
-        $projectTemplateId = $project->project_template_id;
-        $ownershipColumn = $projectTemplateId === null ? 'project_id' : 'project_template_id';
-        $ownershipValue = $projectTemplateId ?? $project->id;
-
         return [
             'branch_name' => ['required', 'string', 'max:255'],
-            'repository_connection_id' => [
-                'required',
-                'integer',
-                Rule::exists('repository_connections', 'id')->where(
-                    fn ($query) => $query->where($ownershipColumn, $ownershipValue),
-                ),
-            ],
+            'repository_connection_id' => ['required', 'integer', 'exists:repository_connections,id'],
         ];
     }
 }