KAN-4 Duplicate Project #86
@@ -7,24 +7,39 @@ use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
|
|||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
use App\Models\Project;
|
use App\Models\Project;
|
||||||
use App\Models\User;
|
use App\Models\User;
|
||||||
|
use Illuminate\Support\Arr;
|
||||||
|
use Illuminate\Support\Facades\DB;
|
||||||
|
|
||||||
class CreateProject
|
class CreateProject
|
||||||
{
|
{
|
||||||
public function __construct(
|
public function __construct(
|
||||||
|
private readonly DuplicateProjectConfiguration $duplicateProjectConfiguration,
|
||||||
private readonly InitializeProjectTicketMetadata $initializeProjectTicketMetadata,
|
private readonly InitializeProjectTicketMetadata $initializeProjectTicketMetadata,
|
||||||
private readonly PermissionService $permissionService,
|
private readonly PermissionService $permissionService,
|
||||||
) {}
|
) {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param array{key: string, name: string} $attributes
|
* @param array{key: string, name: string, creation_mode?: string, source_project_id?: int|string|null, copy_areas?: array<int, string>|null} $attributes
|
||||||
*/
|
*/
|
||||||
public function handle(array $attributes, User $actor): Project
|
public function handle(array $attributes, User $actor): Project
|
||||||
{
|
{
|
||||||
$project = Project::query()->create($attributes);
|
return DB::transaction(function () use ($attributes, $actor): Project {
|
||||||
|
$project = Project::query()->create(Arr::only($attributes, ['key', 'name']));
|
||||||
|
|
||||||
$this->initializeProjectTicketMetadata->handle($project);
|
$this->initializeProjectTicketMetadata->handle($project);
|
||||||
$this->permissionService->grant($actor, Permission::ProjectAdmin, $project);
|
$this->permissionService->grant($actor, Permission::ProjectAdmin, $project);
|
||||||
|
|
||||||
return $project;
|
if (($attributes['creation_mode'] ?? 'scratch') === 'duplicate' && isset($attributes['source_project_id'])) {
|
||||||
|
$sourceProject = Project::query()->findOrFail((int) $attributes['source_project_id']);
|
||||||
|
|
||||||
|
$this->duplicateProjectConfiguration->handle(
|
||||||
|
$sourceProject,
|
||||||
|
$project,
|
||||||
|
array_values($attributes['copy_areas'] ?? []),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
return $project->fresh() ?? $project;
|
||||||
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,250 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Actions\Projects;
|
||||||
|
|
||||||
|
use App\Models\Project;
|
||||||
|
|
||||||
|
class DuplicateProjectConfiguration
|
||||||
|
{
|
||||||
|
public const COPY_PROJECT_SETTINGS = 'project_settings';
|
||||||
|
|
||||||
|
public const COPY_BOARD_WORKFLOW = 'board_workflow';
|
||||||
|
|
||||||
|
public const COPY_TICKET_TYPES = 'ticket_types';
|
||||||
|
|
||||||
|
public const COPY_TICKET_PRIORITIES = 'ticket_priorities';
|
||||||
|
|
||||||
|
public const COPY_TICKET_FIELDS = 'ticket_fields';
|
||||||
|
|
||||||
|
public const COPY_TICKET_LINK_TYPES = 'ticket_link_types';
|
||||||
|
|
||||||
|
public const COPY_REPOSITORY_CONNECTIONS = 'repository_connections';
|
||||||
|
|
||||||
|
public const COPY_NOTIFICATION_ACTIONS = 'notification_actions';
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @return list<string>
|
||||||
|
*/
|
||||||
|
public static function copyableAreas(): array
|
||||||
|
{
|
||||||
|
return [
|
||||||
|
self::COPY_PROJECT_SETTINGS,
|
||||||
|
self::COPY_BOARD_WORKFLOW,
|
||||||
|
self::COPY_TICKET_TYPES,
|
||||||
|
self::COPY_TICKET_PRIORITIES,
|
||||||
|
self::COPY_TICKET_FIELDS,
|
||||||
|
self::COPY_TICKET_LINK_TYPES,
|
||||||
|
self::COPY_REPOSITORY_CONNECTIONS,
|
||||||
|
self::COPY_NOTIFICATION_ACTIONS,
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @return list<array{value: string, label: string, description: string}>
|
||||||
|
*/
|
||||||
|
public static function copyAreaOptions(): array
|
||||||
|
{
|
||||||
|
return [
|
||||||
|
[
|
||||||
|
'value' => self::COPY_PROJECT_SETTINGS,
|
||||||
|
'label' => 'Project settings',
|
||||||
|
'description' => 'Copy general project preferences like story points and author visibility.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_BOARD_WORKFLOW,
|
||||||
|
'label' => 'Board workflow',
|
||||||
|
'description' => 'Copy board columns, ticket statuses, and allowed status transitions.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_TICKET_TYPES,
|
||||||
|
'label' => 'Ticket types',
|
||||||
|
'description' => 'Copy available ticket types, including parent-ticket types.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_TICKET_PRIORITIES,
|
||||||
|
'label' => 'Ticket priorities',
|
||||||
|
'description' => 'Copy the project-specific ticket priorities.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_TICKET_FIELDS,
|
||||||
|
'label' => 'Custom ticket fields',
|
||||||
|
'description' => 'Copy custom ticket field definitions without any ticket values.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_TICKET_LINK_TYPES,
|
||||||
|
'label' => 'Ticket link types',
|
||||||
|
'description' => 'Copy available relationship labels between tickets.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_REPOSITORY_CONNECTIONS,
|
||||||
|
'label' => 'Repository connections',
|
||||||
|
'description' => 'Copy linked repositories without changing the new project identity.',
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'value' => self::COPY_NOTIFICATION_ACTIONS,
|
||||||
|
'label' => 'Notification actions',
|
||||||
|
'description' => 'Copy notification rules and delivery settings.',
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param list<string> $copyAreas
|
||||||
|
*/
|
||||||
|
public function handle(Project $sourceProject, Project $targetProject, array $copyAreas): void
|
||||||
|
{
|
||||||
|
$selectedAreas = array_values(array_intersect(self::copyableAreas(), array_unique($copyAreas)));
|
||||||
|
|
||||||
|
if ($selectedAreas === []) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$sourceProject->loadMissing([
|
||||||
|
'boardColumns',
|
||||||
|
'notificationActions',
|
||||||
|
'repositoryConnections',
|
||||||
|
'ticketFieldDefinitions',
|
||||||
|
'ticketLinkTypes',
|
||||||
|
'ticketPriorities',
|
||||||
|
'ticketStatuses.allowedTargetStatuses',
|
||||||
|
'ticketTypes',
|
||||||
|
]);
|
||||||
|
|
||||||
|
if (in_array(self::COPY_PROJECT_SETTINGS, $selectedAreas, true)) {
|
||||||
|
$targetProject->forceFill([
|
||||||
|
'show_author' => $sourceProject->show_author,
|
||||||
|
'show_story_points' => $sourceProject->show_story_points,
|
||||||
|
])->save();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_BOARD_WORKFLOW, $selectedAreas, true)) {
|
||||||
|
$targetProject->ticketStatuses()->delete();
|
||||||
|
$targetProject->boardColumns()->delete();
|
||||||
|
|
||||||
|
$boardColumnIdMap = [];
|
||||||
|
|
||||||
|
foreach ($sourceProject->boardColumns as $boardColumn) {
|
||||||
|
$newBoardColumn = $targetProject->boardColumns()->create([
|
||||||
|
'name' => $boardColumn->name,
|
||||||
|
'position' => $boardColumn->position,
|
||||||
|
]);
|
||||||
|
|
||||||
|
$boardColumnIdMap[$boardColumn->id] = $newBoardColumn->id;
|
||||||
|
}
|
||||||
|
|
||||||
|
$ticketStatusIdMap = [];
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketStatuses as $ticketStatus) {
|
||||||
|
$newTicketStatus = $targetProject->ticketStatuses()->create([
|
||||||
|
'board_column_id' => $ticketStatus->board_column_id === null
|
||||||
|
? null
|
||||||
|
: ($boardColumnIdMap[$ticketStatus->board_column_id] ?? null),
|
||||||
|
'name' => $ticketStatus->name,
|
||||||
|
'position' => $ticketStatus->position,
|
||||||
|
]);
|
||||||
|
|
||||||
|
$ticketStatusIdMap[$ticketStatus->id] = $newTicketStatus->id;
|
||||||
|
}
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketStatuses as $ticketStatus) {
|
||||||
|
$newTicketStatusId = $ticketStatusIdMap[$ticketStatus->id] ?? null;
|
||||||
|
|
||||||
|
if ($newTicketStatusId === null) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
$targetProject->ticketStatuses()
|
||||||
|
->findOrFail($newTicketStatusId)
|
||||||
|
->allowedTargetStatuses()
|
||||||
|
->sync(
|
||||||
|
$ticketStatus->allowedTargetStatuses
|
||||||
|
->pluck('id')
|
||||||
|
->map(fn (int $statusId): ?int => $ticketStatusIdMap[$statusId] ?? null)
|
||||||
|
->filter()
|
||||||
|
->values()
|
||||||
|
->all(),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_TICKET_TYPES, $selectedAreas, true)) {
|
||||||
|
$targetProject->ticketTypes()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketTypes as $ticketType) {
|
||||||
|
$targetProject->ticketTypes()->create([
|
||||||
|
'name' => $ticketType->name,
|
||||||
|
'position' => $ticketType->position,
|
||||||
|
'is_parent' => $ticketType->is_parent,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_TICKET_PRIORITIES, $selectedAreas, true)) {
|
||||||
|
$targetProject->ticketPriorities()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketPriorities as $ticketPriority) {
|
||||||
|
$targetProject->ticketPriorities()->create([
|
||||||
|
'name' => $ticketPriority->name,
|
||||||
|
'position' => $ticketPriority->position,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_TICKET_FIELDS, $selectedAreas, true)) {
|
||||||
|
$targetProject->ticketFieldDefinitions()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketFieldDefinitions as $ticketFieldDefinition) {
|
||||||
|
$targetProject->ticketFieldDefinitions()->create([
|
||||||
|
'field_type' => $ticketFieldDefinition->field_type,
|
||||||
|
'label' => $ticketFieldDefinition->label,
|
||||||
|
'position' => $ticketFieldDefinition->position,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_TICKET_LINK_TYPES, $selectedAreas, true)) {
|
||||||
|
$targetProject->ticketLinkTypes()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->ticketLinkTypes as $ticketLinkType) {
|
||||||
|
$targetProject->ticketLinkTypes()->create([
|
||||||
|
'label' => $ticketLinkType->label,
|
||||||
|
'counterpart_label' => $ticketLinkType->counterpart_label,
|
||||||
|
'position' => $ticketLinkType->position,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_REPOSITORY_CONNECTIONS, $selectedAreas, true)) {
|
||||||
|
$targetProject->repositoryConnections()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->repositoryConnections as $repositoryConnection) {
|
||||||
|
$targetProject->repositoryConnections()->create([
|
||||||
|
'provider' => $repositoryConnection->provider,
|
||||||
|
'base_url' => $repositoryConnection->base_url,
|
||||||
|
'provider_repository_id' => $repositoryConnection->provider_repository_id,
|
||||||
|
'owner_name' => $repositoryConnection->owner_name,
|
||||||
|
'repository_name' => $repositoryConnection->repository_name,
|
||||||
|
'repository_full_name' => $repositoryConnection->repository_full_name,
|
||||||
|
'repository_url' => $repositoryConnection->repository_url,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (in_array(self::COPY_NOTIFICATION_ACTIONS, $selectedAreas, true)) {
|
||||||
|
$targetProject->notificationActions()->delete();
|
||||||
|
|
||||||
|
foreach ($sourceProject->notificationActions as $notificationAction) {
|
||||||
|
$targetProject->notificationActions()->create([
|
||||||
|
'name' => $notificationAction->name,
|
||||||
|
'enabled' => $notificationAction->enabled,
|
||||||
|
'event_type' => $notificationAction->event_type,
|
||||||
|
'channel' => $notificationAction->channel,
|
||||||
|
'channel_configuration' => $notificationAction->channel_configuration,
|
||||||
|
'message_template' => $notificationAction->message_template,
|
||||||
|
'delivery_url' => $notificationAction->delivery_url,
|
||||||
|
'filters' => $notificationAction->filters,
|
||||||
|
]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -6,6 +6,7 @@ use App\Actions\Permissions\PermissionService;
|
|||||||
use App\Actions\Projects\BuildProjectBacklogPayload;
|
use App\Actions\Projects\BuildProjectBacklogPayload;
|
||||||
use App\Actions\Projects\CreateProject;
|
use App\Actions\Projects\CreateProject;
|
||||||
use App\Actions\Projects\DeleteProject;
|
use App\Actions\Projects\DeleteProject;
|
||||||
|
use App\Actions\Projects\DuplicateProjectConfiguration;
|
||||||
use App\Actions\Projects\ListVisibleProjects;
|
use App\Actions\Projects\ListVisibleProjects;
|
||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
use App\Http\Requests\DeleteProjectRequest;
|
use App\Http\Requests\DeleteProjectRequest;
|
||||||
@@ -51,7 +52,12 @@ class ProjectController extends Controller
|
|||||||
403,
|
403,
|
||||||
);
|
);
|
||||||
|
|
||||||
return Inertia::render('projects/Create');
|
return Inertia::render('projects/Create', [
|
||||||
|
'copyAreaOptions' => DuplicateProjectConfiguration::copyAreaOptions(),
|
||||||
|
'sourceProjects' => ProjectResource::collection(
|
||||||
|
$this->listVisibleProjects->handle($request->user()),
|
||||||
|
)->resolve(),
|
||||||
|
|
|||||||
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function store(StoreProjectRequest $request): RedirectResponse
|
public function store(StoreProjectRequest $request): RedirectResponse
|
||||||
|
|||||||
@@ -2,6 +2,10 @@
|
|||||||
|
|
||||||
namespace App\Http\Requests;
|
namespace App\Http\Requests;
|
||||||
|
|
||||||
|
use App\Actions\Permissions\PermissionService;
|
||||||
|
use App\Actions\Projects\DuplicateProjectConfiguration;
|
||||||
|
use App\Enums\Permission;
|
||||||
|
use App\Models\Project;
|
||||||
use Illuminate\Contracts\Validation\ValidationRule;
|
use Illuminate\Contracts\Validation\ValidationRule;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
use Illuminate\Support\Arr;
|
use Illuminate\Support\Arr;
|
||||||
@@ -20,12 +24,21 @@ class StoreProjectRequest extends FormRequest
|
|||||||
protected function prepareForValidation(): void
|
protected function prepareForValidation(): void
|
||||||
{
|
{
|
||||||
$key = Arr::get($this->all(), 'key');
|
$key = Arr::get($this->all(), 'key');
|
||||||
|
$creationMode = Arr::get($this->all(), 'creation_mode', 'scratch');
|
||||||
|
$copyAreas = Arr::get($this->all(), 'copy_areas', []);
|
||||||
|
|
||||||
if ($key !== null) {
|
if ($key !== null) {
|
||||||
$this->merge([
|
$this->merge([
|
||||||
'key' => strtoupper(trim((string) $key)),
|
'key' => strtoupper(trim((string) $key)),
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$this->merge([
|
||||||
|
'creation_mode' => is_string($creationMode) ? trim($creationMode) : 'scratch',
|
||||||
|
'copy_areas' => is_array($copyAreas)
|
||||||
|
? array_values($copyAreas)
|
||||||
|
: ($copyAreas === null || $copyAreas === '' ? [] : [(string) $copyAreas]),
|
||||||
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -36,6 +49,7 @@ class StoreProjectRequest extends FormRequest
|
|||||||
public function rules(): array
|
public function rules(): array
|
||||||
{
|
{
|
||||||
return [
|
return [
|
||||||
|
'creation_mode' => ['required', 'string', Rule::in(['scratch', 'duplicate'])],
|
||||||
'key' => [
|
'key' => [
|
||||||
'required',
|
'required',
|
||||||
'string',
|
'string',
|
||||||
@@ -44,6 +58,44 @@ class StoreProjectRequest extends FormRequest
|
|||||||
Rule::unique('projects', 'key'),
|
Rule::unique('projects', 'key'),
|
||||||
],
|
],
|
||||||
'name' => ['required', 'string', 'max:255'],
|
'name' => ['required', 'string', 'max:255'],
|
||||||
|
'source_project_id' => [
|
||||||
|
Rule::requiredIf(fn (): bool => $this->input('creation_mode') === 'duplicate'),
|
||||||
|
'nullable',
|
||||||
|
'integer',
|
||||||
|
Rule::exists('projects', 'id'),
|
||||||
|
function (string $attribute, mixed $value, \Closure $fail): void {
|
||||||
|
if ($value === null || $value === '') {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$project = Project::query()->find($value);
|
||||||
|
|
||||||
|
if ($project === null) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$permissionService = app(PermissionService::class);
|
||||||
|
|
||||||
|
if (! $permissionService->allows($this->user(), Permission::ProjectRead, $project)) {
|
||||||
|
$fail('Select a project you can access as the duplication source.');
|
||||||
|
}
|
||||||
|
},
|
||||||
|
],
|
||||||
|
'copy_areas' => ['nullable', 'array'],
|
||||||
|
'copy_areas.*' => [
|
||||||
|
'string',
|
||||||
|
Rule::in(DuplicateProjectConfiguration::copyableAreas()),
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @return array<string, string>
|
||||||
|
*/
|
||||||
|
public function messages(): array
|
||||||
|
{
|
||||||
|
return [
|
||||||
|
'source_project_id.required' => 'Select a source project to duplicate from.',
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,5 +1,6 @@
|
|||||||
<script setup lang="ts">
|
<script setup lang="ts">
|
||||||
import { Form, Head, Link } from '@inertiajs/vue3';
|
import { Head, Link, useForm } from '@inertiajs/vue3';
|
||||||
|
import { computed } from 'vue';
|
||||||
import ProjectController from '@/actions/App/Http/Controllers/ProjectController';
|
import ProjectController from '@/actions/App/Http/Controllers/ProjectController';
|
||||||
import Heading from '@/components/Heading.vue';
|
import Heading from '@/components/Heading.vue';
|
||||||
import InputError from '@/components/InputError.vue';
|
import InputError from '@/components/InputError.vue';
|
||||||
@@ -15,7 +16,9 @@ import { Input } from '@/components/ui/input';
|
|||||||
import { Label } from '@/components/ui/label';
|
import { Label } from '@/components/ui/label';
|
||||||
import AppLayout from '@/layouts/AppLayout.vue';
|
import AppLayout from '@/layouts/AppLayout.vue';
|
||||||
import { create, index } from '@/routes/projects/index';
|
import { create, index } from '@/routes/projects/index';
|
||||||
import type { BreadcrumbItem } from '@/types';
|
import type { BreadcrumbItem, ProjectsCreatePageProps } from '@/types';
|
||||||
|
|
||||||
|
const props = defineProps<ProjectsCreatePageProps>();
|
||||||
|
|
||||||
const breadcrumbs: BreadcrumbItem[] = [
|
const breadcrumbs: BreadcrumbItem[] = [
|
||||||
{
|
{
|
||||||
@@ -27,6 +30,35 @@ const breadcrumbs: BreadcrumbItem[] = [
|
|||||||
href: create(),
|
href: create(),
|
||||||
},
|
},
|
||||||
];
|
];
|
||||||
|
|
||||||
|
const form = useForm({
|
||||||
|
copy_areas: [] as string[],
|
||||||
|
creation_mode: 'scratch',
|
||||||
|
key: '',
|
||||||
|
name: '',
|
||||||
|
source_project_id: props.sourceProjects[0]?.id ? String(props.sourceProjects[0].id) : '',
|
||||||
|
});
|
||||||
|
|
||||||
|
const hasSourceProjects = computed(() => props.sourceProjects.length > 0);
|
||||||
|
|
||||||
|
const copyAreasError = computed(() => {
|
||||||
|
return (
|
||||||
|
form.errors.copy_areas ??
|
||||||
|
Object.entries(form.errors).find(([key]) => key.startsWith('copy_areas.'))?.[1] ??
|
||||||
|
''
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
const submit = (): void => {
|
||||||
|
if (form.creation_mode !== 'duplicate') {
|
||||||
|
form.source_project_id = '';
|
||||||
|
form.copy_areas = [];
|
||||||
|
}
|
||||||
|
|
||||||
|
form.post(ProjectController.store(), {
|
||||||
|
preserveScroll: true,
|
||||||
|
});
|
||||||
|
};
|
||||||
</script>
|
</script>
|
||||||
|
|
||||||
<template>
|
<template>
|
||||||
@@ -36,7 +68,7 @@ const breadcrumbs: BreadcrumbItem[] = [
|
|||||||
<div class="mx-auto flex w-full max-w-3xl flex-1 flex-col gap-6 p-4">
|
<div class="mx-auto flex w-full max-w-3xl flex-1 flex-col gap-6 p-4">
|
||||||
<Heading
|
<Heading
|
||||||
title="Create project"
|
title="Create project"
|
||||||
description="Create a project with a stable uppercase key that will prefix all future ticket keys."
|
description="Create a fresh project or start from an existing project's settings without copying any tickets."
|
||||||
/>
|
/>
|
||||||
|
|
||||||
<Card>
|
<Card>
|
||||||
@@ -49,28 +81,63 @@ const breadcrumbs: BreadcrumbItem[] = [
|
|||||||
</CardHeader>
|
</CardHeader>
|
||||||
|
|
||||||
<CardContent>
|
<CardContent>
|
||||||
<Form
|
<form class="space-y-6" @submit.prevent="submit">
|
||||||
v-bind="ProjectController.store.form()"
|
<div class="grid gap-3">
|
||||||
class="space-y-6"
|
<Label>How do you want to start?</Label>
|
||||||
v-slot="{ errors, processing }"
|
<div class="grid gap-3 sm:grid-cols-2">
|
||||||
>
|
<Label class="flex cursor-pointer items-start gap-3 rounded-xl border p-4">
|
||||||
|
<input
|
||||||
|
v-model="form.creation_mode"
|
||||||
|
type="radio"
|
||||||
|
value="scratch"
|
||||||
|
class="mt-1"
|
||||||
|
/>
|
||||||
|
<div class="space-y-1">
|
||||||
|
<div class="font-medium">Create from scratch</div>
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
Start with the default project setup.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</Label>
|
||||||
|
|
||||||
|
<Label
|
||||||
|
class="flex items-start gap-3 rounded-xl border p-4"
|
||||||
|
:class="hasSourceProjects ? 'cursor-pointer' : 'cursor-not-allowed opacity-60'"
|
||||||
|
>
|
||||||
|
<input
|
||||||
|
v-model="form.creation_mode"
|
||||||
|
type="radio"
|
||||||
|
value="duplicate"
|
||||||
|
class="mt-1"
|
||||||
|
:disabled="!hasSourceProjects"
|
||||||
|
/>
|
||||||
|
<div class="space-y-1">
|
||||||
|
<div class="font-medium">Create from existing project</div>
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
Copy only the settings you explicitly select. Tickets are never duplicated.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</Label>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
||||||
<div class="grid gap-2">
|
<div class="grid gap-2">
|
||||||
<Label for="name">Project name</Label>
|
<Label for="name">Project name</Label>
|
||||||
<Input
|
<Input
|
||||||
id="name"
|
id="name"
|
||||||
name="name"
|
v-model="form.name"
|
||||||
required
|
required
|
||||||
autofocus
|
autofocus
|
||||||
placeholder="Kanban foundation"
|
placeholder="Kanban foundation"
|
||||||
/>
|
/>
|
||||||
<InputError :message="errors.name" />
|
<InputError :message="form.errors.name" />
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="grid gap-2">
|
<div class="grid gap-2">
|
||||||
<Label for="key">Project key</Label>
|
<Label for="key">Project key</Label>
|
||||||
<Input
|
<Input
|
||||||
id="key"
|
id="key"
|
||||||
name="key"
|
v-model="form.key"
|
||||||
required
|
required
|
||||||
maxlength="5"
|
maxlength="5"
|
||||||
placeholder="CORE"
|
placeholder="CORE"
|
||||||
@@ -78,15 +145,80 @@ const breadcrumbs: BreadcrumbItem[] = [
|
|||||||
/>
|
/>
|
||||||
<p class="text-muted-foreground text-sm">
|
<p class="text-muted-foreground text-sm">
|
||||||
Example ticket keys will look like
|
Example ticket keys will look like
|
||||||
<span class="text-foreground font-medium"
|
<span class="text-foreground font-medium">CORE-1</span>.
|
||||||
>CORE-1</span
|
|
||||||
>.
|
|
||||||
</p>
|
</p>
|
||||||
<InputError :message="errors.key" />
|
<InputError :message="form.errors.key" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div
|
||||||
|
v-if="form.creation_mode === 'duplicate'"
|
||||||
|
class="space-y-6 rounded-xl border border-dashed p-4"
|
||||||
|
>
|
||||||
|
<div class="grid gap-2">
|
||||||
|
<Label for="source_project_id">Source project</Label>
|
||||||
|
<select
|
||||||
|
id="source_project_id"
|
||||||
|
v-model="form.source_project_id"
|
||||||
|
class="border-input bg-background ring-offset-background placeholder:text-muted-foreground focus-visible:ring-ring flex h-10 w-full rounded-md border px-3 py-2 text-sm focus-visible:ring-2 focus-visible:ring-offset-2 focus-visible:outline-none disabled:cursor-not-allowed disabled:opacity-50"
|
||||||
|
:disabled="!hasSourceProjects"
|
||||||
|
>
|
||||||
|
<option value="">Select a source project</option>
|
||||||
|
<option
|
||||||
|
v-for="sourceProject in sourceProjects"
|
||||||
|
:key="sourceProject.id"
|
||||||
|
:value="String(sourceProject.id)"
|
||||||
|
>
|
||||||
|
{{ sourceProject.name }} ({{ sourceProject.key }})
|
||||||
|
</option>
|
||||||
|
</select>
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
The new project keeps its own identity, key, and future ticket numbering.
|
||||||
|
</p>
|
||||||
|
<InputError :message="form.errors.source_project_id" />
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="grid gap-3">
|
||||||
|
<div>
|
||||||
|
<Label>Copy options</Label>
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
Choose which settings should be copied into the new project.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div class="grid gap-3">
|
||||||
|
<Label
|
||||||
|
v-for="copyAreaOption in copyAreaOptions"
|
||||||
|
:key="copyAreaOption.value"
|
||||||
|
class="flex cursor-pointer items-start gap-3 rounded-xl border p-4"
|
||||||
|
>
|
||||||
|
<input
|
||||||
|
v-model="form.copy_areas"
|
||||||
|
type="checkbox"
|
||||||
|
:value="copyAreaOption.value"
|
||||||
|
class="mt-1"
|
||||||
|
/>
|
||||||
|
<div class="space-y-1">
|
||||||
|
<div class="font-medium">{{ copyAreaOption.label }}</div>
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
{{ copyAreaOption.description }}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</Label>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<p class="text-muted-foreground text-sm">
|
||||||
|
Operational ticket data is always left behind so the new project starts clean.
|
||||||
|
</p>
|
||||||
|
<InputError :message="copyAreasError" />
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div v-else-if="!hasSourceProjects" class="text-muted-foreground text-sm">
|
||||||
|
No accessible source projects are available for duplication yet.
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="flex items-center gap-3">
|
<div class="flex items-center gap-3">
|
||||||
<Button type="submit" :disabled="processing">
|
<Button type="submit" :disabled="form.processing">
|
||||||
Create project
|
Create project
|
||||||
</Button>
|
</Button>
|
||||||
|
|
||||||
@@ -94,7 +226,7 @@ const breadcrumbs: BreadcrumbItem[] = [
|
|||||||
<Link :href="index()">Cancel</Link>
|
<Link :href="index()">Cancel</Link>
|
||||||
</Button>
|
</Button>
|
||||||
</div>
|
</div>
|
||||||
</Form>
|
</form>
|
||||||
</CardContent>
|
</CardContent>
|
||||||
</Card>
|
</Card>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
@@ -32,11 +32,22 @@ export type AdminUserEditPageProps = {
|
|||||||
user: UserModel;
|
user: UserModel;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
export type ProjectDuplicateCopyAreaOption = {
|
||||||
|
description: string;
|
||||||
|
label: string;
|
||||||
|
value: string;
|
||||||
|
};
|
||||||
|
|
||||||
export type ProjectsIndexPageProps = {
|
export type ProjectsIndexPageProps = {
|
||||||
projects: ProjectModel[];
|
projects: ProjectModel[];
|
||||||
status?: string | null;
|
status?: string | null;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
export type ProjectsCreatePageProps = {
|
||||||
|
copyAreaOptions: ProjectDuplicateCopyAreaOption[];
|
||||||
|
sourceProjects: ProjectModel[];
|
||||||
|
};
|
||||||
|
|
||||||
export type ProjectShowPageProps = {
|
export type ProjectShowPageProps = {
|
||||||
canManageAccess: boolean;
|
canManageAccess: boolean;
|
||||||
project: ProjectModel;
|
project: ProjectModel;
|
||||||
|
|||||||
@@ -3,7 +3,11 @@
|
|||||||
namespace Tests\Feature\Api;
|
namespace Tests\Feature\Api;
|
||||||
|
|
||||||
use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
|
use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
|
||||||
|
use App\Actions\Projects\DuplicateProjectConfiguration;
|
||||||
|
use App\Models\BoardColumn;
|
||||||
use App\Models\Project;
|
use App\Models\Project;
|
||||||
|
use App\Models\TicketStatus;
|
||||||
|
use App\Models\TicketType;
|
||||||
use App\Models\User;
|
use App\Models\User;
|
||||||
use Illuminate\Foundation\Testing\RefreshDatabase;
|
use Illuminate\Foundation\Testing\RefreshDatabase;
|
||||||
use Tests\TestCase;
|
use Tests\TestCase;
|
||||||
@@ -51,6 +55,86 @@ class ProjectApiTest extends TestCase
|
|||||||
$this->assertSame(1, $project->ticketStatuses()->count());
|
$this->assertSame(1, $project->ticketStatuses()->count());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function test_users_can_duplicate_projects_via_api(): void
|
||||||
|
{
|
||||||
|
$user = User::factory()->create();
|
||||||
|
$this->syncGlobalPermissions($user, ['create']);
|
||||||
|
|
||||||
|
$sourceProject = Project::factory()->create([
|
||||||
|
'key' => 'SRC1',
|
||||||
|
'show_author' => false,
|
||||||
|
]);
|
||||||
|
$this->syncProjectPermissions($sourceProject, $user, ['view']);
|
||||||
|
|
||||||
|
$sourceProject->ticketTypes()->delete();
|
||||||
|
$sourceProject->ticketStatuses()->delete();
|
||||||
|
$sourceProject->boardColumns()->delete();
|
||||||
|
|
||||||
|
$column = BoardColumn::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'In Progress',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$backlogStatus = TicketStatus::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Queued',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$activeStatus = TicketStatus::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'board_column_id' => $column->id,
|
||||||
|
'name' => 'Active',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$backlogStatus->allowedTargetStatuses()->sync([$activeStatus->id]);
|
||||||
|
TicketType::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Spike',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
|
||||||
|
$this->withApplicationKey($user)
|
||||||
|
->postJson(route('api.projects.store'), [
|
||||||
|
'key' => 'dup2',
|
||||||
|
'name' => 'Duplicated API Project',
|
||||||
|
'creation_mode' => 'duplicate',
|
||||||
|
'source_project_id' => $sourceProject->id,
|
||||||
|
'copy_areas' => [
|
||||||
|
DuplicateProjectConfiguration::COPY_PROJECT_SETTINGS,
|
||||||
|
DuplicateProjectConfiguration::COPY_BOARD_WORKFLOW,
|
||||||
|
DuplicateProjectConfiguration::COPY_TICKET_TYPES,
|
||||||
|
],
|
||||||
|
])
|
||||||
|
->assertCreated()
|
||||||
|
->assertJsonPath('data.key', 'DUP2')
|
||||||
|
->assertJsonPath('data.name', 'Duplicated API Project');
|
||||||
|
|
||||||
|
$project = Project::query()->where('key', 'DUP2')->firstOrFail();
|
||||||
|
|
||||||
|
$this->assertFalse($project->show_author);
|
||||||
|
$this->assertSame(['In Progress'], $project->boardColumns()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Queued', 'Active'], $project->ticketStatuses()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Spike'], $project->ticketTypes()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Active'], $project->ticketStatuses()->where('name', 'Queued')->firstOrFail()->allowedTargetStatuses()->pluck('name')->all());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_project_duplication_via_api_requires_visible_source_project(): void
|
||||||
|
{
|
||||||
|
$user = User::factory()->create();
|
||||||
|
$this->syncGlobalPermissions($user, ['create']);
|
||||||
|
$sourceProject = Project::factory()->create();
|
||||||
|
|
||||||
|
$this->withApplicationKey($user)
|
||||||
|
->postJson(route('api.projects.store'), [
|
||||||
|
'key' => 'nope',
|
||||||
|
'name' => 'No Access',
|
||||||
|
'creation_mode' => 'duplicate',
|
||||||
|
'source_project_id' => $sourceProject->id,
|
||||||
|
])
|
||||||
|
->assertStatus(422)
|
||||||
|
->assertJsonValidationErrors(['source_project_id']);
|
||||||
|
}
|
||||||
|
|
||||||
public function test_project_show_and_delete_enforce_permissions(): void
|
public function test_project_show_and_delete_enforce_permissions(): void
|
||||||
{
|
{
|
||||||
$user = User::factory()->create();
|
$user = User::factory()->create();
|
||||||
|
|||||||
@@ -3,11 +3,18 @@
|
|||||||
namespace Tests\Feature\Projects;
|
namespace Tests\Feature\Projects;
|
||||||
|
|
||||||
use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
|
use App\Actions\ProjectTicketMetadata\InitializeProjectTicketMetadata;
|
||||||
|
use App\Actions\Projects\DuplicateProjectConfiguration;
|
||||||
use App\Models\BoardColumn;
|
use App\Models\BoardColumn;
|
||||||
|
use App\Models\NotificationAction;
|
||||||
use App\Models\Project;
|
use App\Models\Project;
|
||||||
|
use App\Models\RepositoryConnection;
|
||||||
use App\Models\Team;
|
use App\Models\Team;
|
||||||
use App\Models\Ticket;
|
use App\Models\Ticket;
|
||||||
|
use App\Models\TicketFieldDefinition;
|
||||||
|
use App\Models\TicketLinkType;
|
||||||
|
use App\Models\TicketPriority;
|
||||||
use App\Models\TicketStatus;
|
use App\Models\TicketStatus;
|
||||||
|
use App\Models\TicketType;
|
||||||
use App\Models\User;
|
use App\Models\User;
|
||||||
use Illuminate\Foundation\Testing\RefreshDatabase;
|
use Illuminate\Foundation\Testing\RefreshDatabase;
|
||||||
use Inertia\Testing\AssertableInertia as Assert;
|
use Inertia\Testing\AssertableInertia as Assert;
|
||||||
@@ -100,12 +107,25 @@ class ProjectControllerTest extends TestCase
|
|||||||
{
|
{
|
||||||
$user = User::factory()->create();
|
$user = User::factory()->create();
|
||||||
$this->syncGlobalPermissions($user, ['project:create']);
|
$this->syncGlobalPermissions($user, ['project:create']);
|
||||||
|
$visibleProject = Project::factory()->create([
|
||||||
|
'name' => 'Visible Source',
|
||||||
|
'key' => 'SRC1',
|
||||||
|
]);
|
||||||
|
Project::factory()->create([
|
||||||
|
'name' => 'Hidden Source',
|
||||||
|
'key' => 'SRC2',
|
||||||
|
]);
|
||||||
|
$this->syncProjectPermissions($visibleProject, $user, ['view']);
|
||||||
|
|
||||||
$this->actingAs($user)
|
$this->actingAs($user)
|
||||||
->get(route('projects.create'))
|
->get(route('projects.create'))
|
||||||
->assertOk()
|
->assertOk()
|
||||||
->assertInertia(fn (Assert $page) => $page
|
->assertInertia(fn (Assert $page) => $page
|
||||||
->component('projects/Create'),
|
->component('projects/Create')
|
||||||
|
->has('copyAreaOptions', count(DuplicateProjectConfiguration::copyableAreas()))
|
||||||
|
->has('sourceProjects', 1)
|
||||||
|
->where('sourceProjects.0.id', $visibleProject->id)
|
||||||
|
->where('copyAreaOptions.0.value', DuplicateProjectConfiguration::COPY_PROJECT_SETTINGS),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -187,6 +207,152 @@ class ProjectControllerTest extends TestCase
|
|||||||
->assertRedirect(route('projects.create'));
|
->assertRedirect(route('projects.create'));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function test_authenticated_users_can_duplicate_projects_with_selected_settings_only(): void
|
||||||
|
{
|
||||||
|
$user = User::factory()->create();
|
||||||
|
$this->syncGlobalPermissions($user, ['project:create']);
|
||||||
|
|
||||||
|
$sourceProject = Project::factory()->create([
|
||||||
|
'key' => 'SRC1',
|
||||||
|
'name' => 'Source Project',
|
||||||
|
'show_author' => false,
|
||||||
|
'show_story_points' => false,
|
||||||
|
]);
|
||||||
|
$this->syncProjectPermissions($sourceProject, $user, ['view']);
|
||||||
|
|
||||||
|
$sourceProject->ticketTypes()->delete();
|
||||||
|
$sourceProject->ticketPriorities()->delete();
|
||||||
|
$sourceProject->ticketStatuses()->delete();
|
||||||
|
$sourceProject->boardColumns()->delete();
|
||||||
|
|
||||||
|
$todoColumn = BoardColumn::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Todo',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$doingColumn = BoardColumn::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Doing',
|
||||||
|
'position' => 2,
|
||||||
|
]);
|
||||||
|
|
||||||
|
$backlogStatus = TicketStatus::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'board_column_id' => null,
|
||||||
|
'name' => 'Planned',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$todoStatus = TicketStatus::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'board_column_id' => $todoColumn->id,
|
||||||
|
'name' => 'Todo',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$doingStatus = TicketStatus::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'board_column_id' => $doingColumn->id,
|
||||||
|
'name' => 'Doing',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
$todoStatus->allowedTargetStatuses()->sync([$doingStatus->id]);
|
||||||
|
$backlogStatus->allowedTargetStatuses()->sync([$todoStatus->id]);
|
||||||
|
|
||||||
|
TicketType::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Bug',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
TicketType::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Epic',
|
||||||
|
'position' => 2,
|
||||||
|
'is_parent' => true,
|
||||||
|
]);
|
||||||
|
TicketPriority::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Urgent',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
TicketFieldDefinition::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'field_type' => 'number',
|
||||||
|
'label' => 'Story estimate',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
TicketLinkType::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'label' => 'Blocks',
|
||||||
|
'counterpart_label' => 'Blocked by',
|
||||||
|
'position' => 1,
|
||||||
|
]);
|
||||||
|
RepositoryConnection::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'repository_full_name' => 'OpenSource/source-project',
|
||||||
|
]);
|
||||||
|
NotificationAction::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'name' => 'Notify board changes',
|
||||||
|
]);
|
||||||
|
Ticket::factory()->create([
|
||||||
|
'project_id' => $sourceProject->id,
|
||||||
|
'sequence_number' => 42,
|
||||||
|
'title' => 'Existing ticket',
|
||||||
|
]);
|
||||||
|
|
||||||
|
$response = $this->actingAs($user)
|
||||||
|
->post(route('projects.store'), [
|
||||||
|
'name' => 'Duplicated Project',
|
||||||
|
'key' => 'dupe',
|
||||||
|
'creation_mode' => 'duplicate',
|
||||||
|
'source_project_id' => $sourceProject->id,
|
||||||
|
'copy_areas' => [
|
||||||
|
DuplicateProjectConfiguration::COPY_PROJECT_SETTINGS,
|
||||||
|
DuplicateProjectConfiguration::COPY_BOARD_WORKFLOW,
|
||||||
|
DuplicateProjectConfiguration::COPY_TICKET_TYPES,
|
||||||
|
DuplicateProjectConfiguration::COPY_TICKET_FIELDS,
|
||||||
|
],
|
||||||
|
]);
|
||||||
|
|
||||||
|
$project = Project::query()->where('key', 'DUPE')->firstOrFail();
|
||||||
|
|
||||||
|
$response->assertRedirect(route('projects.show', $project));
|
||||||
|
|
||||||
|
$this->assertSame('Duplicated Project', $project->name);
|
||||||
|
$this->assertFalse($project->show_author);
|
||||||
|
$this->assertFalse($project->show_story_points);
|
||||||
|
$this->assertSame(0, $project->tickets()->count());
|
||||||
|
|
||||||
|
$this->assertSame(['Todo', 'Doing'], $project->boardColumns()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Planned', 'Todo', 'Doing'], $project->ticketStatuses()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Bug', 'Epic'], $project->ticketTypes()->pluck('name')->all());
|
||||||
|
$this->assertSame(['Story estimate'], $project->ticketFieldDefinitions()->pluck('label')->all());
|
||||||
|
$this->assertSame(['Medium'], $project->ticketPriorities()->pluck('name')->all());
|
||||||
|
$this->assertCount(0, $project->ticketLinkTypes);
|
||||||
|
$this->assertCount(0, $project->repositoryConnections);
|
||||||
|
$this->assertCount(0, $project->notificationActions);
|
||||||
|
|
||||||
|
$duplicatedTodoStatus = $project->ticketStatuses()->where('name', 'Todo')->firstOrFail();
|
||||||
|
$this->assertSame(['Doing'], $duplicatedTodoStatus->allowedTargetStatuses()->pluck('name')->all());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_project_duplication_requires_accessible_source_projects(): void
|
||||||
|
{
|
||||||
|
$user = User::factory()->create();
|
||||||
|
$this->syncGlobalPermissions($user, ['project:create']);
|
||||||
|
$sourceProject = Project::factory()->create();
|
||||||
|
|
||||||
|
$this->actingAs($user)
|
||||||
|
->from(route('projects.create'))
|
||||||
|
->post(route('projects.store'), [
|
||||||
|
'name' => 'Denied Duplicate',
|
||||||
|
'key' => 'deny',
|
||||||
|
'creation_mode' => 'duplicate',
|
||||||
|
'source_project_id' => $sourceProject->id,
|
||||||
|
])
|
||||||
|
->assertSessionHasErrors('source_project_id')
|
||||||
|
->assertRedirect(route('projects.create'));
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Ensure project keys cannot exceed five characters.
|
* Ensure project keys cannot exceed five characters.
|
||||||
*/
|
*/
|
||||||
|
|||||||
Reference in New Issue
Block a user
Nur klein: vorher einmal user setzen mit $request->user ? und dann nur noch user nutzen anstatt zweimal auf $request->user zu nutzen?
Nein, würde mehr zeilen/code hinzufügen als zu kürzen,
wär jetzt $request->xxx->yyy->zzz->user() oder so oder öfter als 2x wär das was anders, dann zwischenspeichern