Merge branch 'main' into feat/system-logs

Reviewed-on: #25

assets/vue/app/domain/setting/application/store/contentSourceStore.js

@@ -28,6 +28,10 @@ export const useContentSourceStore = defineStore('contentSource', {
         // Health check state
         checkingHealth: false,
         checkHealthError: null,
+
+        // Delete state
+        deleting: false,
+        deleteError: null,
     }),
 
     getters: {
@@ -172,6 +176,28 @@ export const useContentSourceStore = defineStore('contentSource', {
             }
         },
 
+        // Delete a source
+        async deleteSource(id) {
+            if (this.deleting) return;
+
+            this.deleting = true;
+            this.deleteError = null;
+
+            try {
+                await contentSourceRepository.delete(id);
+                this.sources = this.sources.filter(source => source.id !== id);
+                if (this.currentSource && this.currentSource.id === id) {
+                    this.currentSource = null;
+                }
+            } catch (error) {
+                this.deleteError = error.message;
+                console.error('Erreur lors de la suppression de la source:', error);
+                throw error;
+            } finally {
+                this.deleting = false;
+            }
+        },
+
         // Clear current source
         clearCurrentSource() {
             this.currentSource = null;

assets/vue/app/domain/setting/infrastructure/api/apiContentSourceRepository.js

@@ -93,6 +93,17 @@ export class ApiContentSourceRepository {
         }
     }
 
+    /**
+     * Supprime une source de contenu
+     */
+    async delete(id) {
+        try {
+            await this.apiClient.delete(`/content-sources/${id}`);
+        } catch (error) {
+            throw new Error(error.response?.data?.message || 'Erreur lors de la suppression de la source');
+        }
+    }
+
     /**
      * Teste une configuration de scraper
      */

assets/vue/app/domain/setting/presentation/components/ContentSourceDeleteModal.vue

@@ -0,0 +1,123 @@
+<template>
+    <TransitionRoot as="template" :show="isOpen">
+        <Dialog as="div" class="relative z-50" @close="closeModal">
+            <TransitionChild
+                as="template"
+                enter="ease-out duration-300"
+                enter-from="opacity-0"
+                enter-to="opacity-100"
+                leave="ease-in duration-200"
+                leave-from="opacity-100"
+                leave-to="opacity-0"
+            >
+                <div class="fixed inset-0 bg-gray-500 dark:bg-gray-900 bg-opacity-75 dark:bg-opacity-80 transition-opacity" />
+            </TransitionChild>
+
+            <div class="fixed inset-0 z-10 overflow-y-auto">
+                <div class="flex min-h-full items-end justify-center p-4 text-center sm:items-center sm:p-0">
+                    <TransitionChild
+                        as="template"
+                        enter="ease-out duration-300"
+                        enter-from="opacity-0 translate-y-4 sm:translate-y-0 sm:scale-95"
+                        enter-to="opacity-100 translate-y-0 sm:scale-100"
+                        leave="ease-in duration-200"
+                        leave-from="opacity-100 translate-y-0 sm:scale-100"
+                        leave-to="opacity-0 translate-y-4 sm:translate-y-0 sm:scale-95"
+                    >
+                        <DialogPanel class="relative transform overflow-hidden rounded-lg bg-white dark:bg-gray-800 px-6 pb-6 pt-6 text-left shadow-xl transition-all sm:my-8 sm:w-full sm:max-w-lg">
+                            <div class="mb-6">
+                                <DialogTitle as="h3" class="text-lg font-semibold leading-6 text-gray-900 dark:text-gray-100">
+                                    Supprimer la source de contenu
+                                </DialogTitle>
+                            </div>
+
+                            <!-- Error state -->
+                            <div v-if="error" class="mb-6 bg-red-100 dark:bg-red-900/20 border border-red-400 dark:border-red-700 text-red-700 dark:text-red-400 px-4 py-3 rounded">
+                                {{ error }}
+                            </div>
+
+                            <!-- Warning message -->
+                            <div class="mb-6">
+                                <div class="flex items-center mb-4">
+                                    <ExclamationTriangleIcon class="h-6 w-6 text-red-500 mr-3" />
+                                    <span class="text-sm font-medium text-gray-900 dark:text-gray-100">Action irréversible</span>
+                                </div>
+                                <p class="text-sm text-gray-600 dark:text-gray-400 mb-4">
+                                    Êtes-vous sûr de vouloir supprimer la source <strong>{{ source?.baseUrl }}</strong> ?
+                                </p>
+                                <div class="bg-yellow-50 dark:bg-yellow-900/20 border border-yellow-200 dark:border-yellow-700 rounded-md p-4">
+                                    <div class="flex">
+                                        <ExclamationTriangleIcon class="h-5 w-5 text-yellow-400" />
+                                        <div class="ml-3">
+                                            <h3 class="text-sm font-medium text-yellow-800 dark:text-yellow-300">
+                                                Attention
+                                            </h3>
+                                            <div class="mt-2 text-sm text-yellow-700 dark:text-yellow-400">
+                                                <p>Cette source ne pourra plus être utilisée pour le scraping des chapitres.</p>
+                                            </div>
+                                        </div>
+                                    </div>
+                                </div>
+                            </div>
+
+                            <!-- Action buttons -->
+                            <div class="mt-6 flex justify-end space-x-3">
+                                <button
+                                    type="button"
+                                    class="inline-flex justify-center rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 px-4 py-2 text-sm font-medium text-gray-700 dark:text-gray-300 shadow-sm hover:bg-gray-50 dark:hover:bg-gray-600 focus:outline-none focus:ring-2 focus:ring-blue-500 focus:ring-offset-2"
+                                    @click="closeModal"
+                                    :disabled="isLoading"
+                                >
+                                    Annuler
+                                </button>
+                                <button
+                                    type="button"
+                                    class="inline-flex justify-center rounded-md border border-transparent bg-red-600 px-4 py-2 text-sm font-medium text-white shadow-sm hover:bg-red-700 focus:outline-none focus:ring-2 focus:ring-red-500 focus:ring-offset-2 disabled:opacity-50 disabled:cursor-not-allowed"
+                                    @click="confirmDelete"
+                                    :disabled="isLoading"
+                                >
+                                    <ArrowPathIcon v-if="isLoading" class="h-4 w-4 mr-2 animate-spin" />
+                                    {{ isLoading ? 'Suppression...' : 'Supprimer définitivement' }}
+                                </button>
+                            </div>
+                        </DialogPanel>
+                    </TransitionChild>
+                </div>
+            </div>
+        </Dialog>
+    </TransitionRoot>
+</template>
+
+<script setup>
+import { Dialog, DialogPanel, DialogTitle, TransitionChild, TransitionRoot } from '@headlessui/vue';
+import { ArrowPathIcon, ExclamationTriangleIcon } from '@heroicons/vue/24/outline';
+
+const props = defineProps({
+    isOpen: {
+        type: Boolean,
+        required: true
+    },
+    source: {
+        type: Object,
+        default: null
+    },
+    isLoading: {
+        type: Boolean,
+        default: false
+    },
+    error: {
+        type: String,
+        default: null
+    }
+});
+
+const emit = defineEmits(['close', 'confirm']);
+
+const closeModal = () => {
+    emit('close');
+};
+
+const confirmDelete = () => {
+    emit('confirm');
+};
+</script>

assets/vue/app/domain/setting/presentation/pages/ScrapperEdit.vue

@@ -176,6 +176,15 @@
             </div>
         </div>
         </div>
+
+        <!-- Delete Modal -->
+        <ContentSourceDeleteModal
+            :is-open="isDeleteModalOpen"
+            :source="currentSource"
+            :is-loading="isDeleting"
+            :error="deleteError"
+            @close="isDeleteModalOpen = false"
+            @confirm="confirmDeleteSource" />
     </div>
 </template>
 
@@ -185,6 +194,7 @@ import {
     CheckCircleIcon,
     ExclamationTriangleIcon,
     PencilSquareIcon,
+    TrashIcon,
     XCircleIcon,
     XMarkIcon
 } from '@heroicons/vue/24/outline';
@@ -194,6 +204,7 @@ import { useRoute, useRouter } from 'vue-router';
 import Toolbar from '../../../../shared/components/ui/Toolbar.vue';
 import { useContentSourceStore } from '../../application/store/contentSourceStore';
 import { ApiContentSourceRepository } from '../../infrastructure/api/apiContentSourceRepository';
+import ContentSourceDeleteModal from '../components/ContentSourceDeleteModal.vue';
 import ContentSourceForm from '../components/ContentSourceForm.vue';
 
 const route = useRoute();
@@ -217,6 +228,9 @@ const showTestResults = ref(false);
 const showSuccessMessage = ref(false);
 const testResults = ref({});
 const testingConfiguration = ref(false);
+const isDeleteModalOpen = ref(false);
+const isDeleting = ref(false);
+const deleteError = ref(null);
 
 const isEditing = computed(() => !!route.params.id);
 
@@ -238,6 +252,7 @@ const toolbarConfig = computed(() => ({
         { type: 'label', text: isEditing.value ? 'Modifier la configuration' : 'Nouvelle configuration', class: 'text-sm font-medium' },
     ],
     rightSection: [
+        ...(isEditing.value ? [{ type: 'button', icon: TrashIcon, label: 'Supprimer', onClick: () => { isDeleteModalOpen.value = true; }, class: 'text-red-600 hover:text-red-700' }, { type: 'divider' }] : []),
         { type: 'button', icon: PencilSquareIcon, label: isEditing.value ? 'Mettre à jour' : 'Créer', onClick: () => formRef.value?.submitForm(), disabled: saving.value },
     ],
 }));
@@ -328,6 +343,21 @@ const handleImageLoad = (event) => {
     event.target.style.display = 'block';
 };
 
+const confirmDeleteSource = async () => {
+    isDeleting.value = true;
+    deleteError.value = null;
+
+    try {
+        await contentSourceStore.deleteSource(route.params.id);
+        isDeleteModalOpen.value = false;
+        await router.push({ name: 'scrapper-configurations' });
+    } catch (error) {
+        deleteError.value = error.message;
+    } finally {
+        isDeleting.value = false;
+    }
+};
+
 const formatErrorType = (type) => {
     const typeMap = {
         'selector_error': 'Erreur sélecteur',

src/Domain/Setting/Application/Command/DeleteContentSourceCommand.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace App\Domain\Setting\Application\Command;
+
+readonly class DeleteContentSourceCommand
+{
+    public function __construct(
+        public int $id
+    ) {
+    }
+}

src/Domain/Setting/Application/CommandHandler/DeleteContentSourceCommandHandler.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace App\Domain\Setting\Application\CommandHandler;
+
+use App\Domain\Setting\Application\Command\DeleteContentSourceCommand;
+use App\Domain\Setting\Domain\Contract\Repository\ContentSourceRepositoryInterface;
+use App\Domain\Setting\Domain\Exception\ContentSourceNotFoundException;
+
+readonly class DeleteContentSourceCommandHandler
+{
+    public function __construct(
+        private ContentSourceRepositoryInterface $contentSourceRepository
+    ) {
+    }
+
+    public function handle(DeleteContentSourceCommand $command): void
+    {
+        $contentSource = $this->contentSourceRepository->findById($command->id);
+
+        if (!$contentSource) {
+            throw new ContentSourceNotFoundException($command->id);
+        }
+
+        $this->contentSourceRepository->delete($contentSource);
+    }
+}

src/Domain/Setting/Infrastructure/ApiPlatform/Resource/DeleteContentSourceResource.php

@@ -0,0 +1,50 @@
+<?php
+
+namespace App\Domain\Setting\Infrastructure\ApiPlatform\Resource;
+
+use ApiPlatform\Metadata\ApiResource;
+use ApiPlatform\Metadata\Delete;
+use App\Domain\Setting\Infrastructure\ApiPlatform\State\Processor\DeleteContentSourceStateProcessor;
+use App\Domain\Setting\Infrastructure\ApiPlatform\State\Provider\DeleteContentSourceStateProvider;
+
+#[ApiResource(
+    shortName: 'ContentSource',
+    operations: [
+        new Delete(
+            uriTemplate: '/content-sources/{id}',
+            provider: DeleteContentSourceStateProvider::class,
+            processor: DeleteContentSourceStateProcessor::class,
+            name: 'delete_content_source',
+            openapiContext: [
+                'summary' => 'Delete a content source',
+                'description' => 'Permanently deletes a content source',
+                'parameters' => [
+                    [
+                        'name' => 'id',
+                        'in' => 'path',
+                        'required' => true,
+                        'schema' => [
+                            'type' => 'integer'
+                        ],
+                        'description' => 'The content source ID'
+                    ]
+                ],
+                'responses' => [
+                    '204' => [
+                        'description' => 'Content source successfully deleted'
+                    ],
+                    '404' => [
+                        'description' => 'Content source not found'
+                    ]
+                ]
+            ]
+        )
+    ]
+)]
+class DeleteContentSourceResource
+{
+    public function __construct(
+        public int $id
+    ) {
+    }
+}

src/Domain/Setting/Infrastructure/ApiPlatform/State/Processor/DeleteContentSourceStateProcessor.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace App\Domain\Setting\Infrastructure\ApiPlatform\State\Processor;
+
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\State\ProcessorInterface;
+use App\Domain\Setting\Application\Command\DeleteContentSourceCommand;
+use App\Domain\Setting\Application\CommandHandler\DeleteContentSourceCommandHandler;
+use App\Domain\Setting\Domain\Exception\ContentSourceNotFoundException;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
+
+readonly class DeleteContentSourceStateProcessor implements ProcessorInterface
+{
+    public function __construct(
+        private DeleteContentSourceCommandHandler $handler
+    ) {
+    }
+
+    public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): int
+    {
+        if (!isset($uriVariables['id'])) {
+            throw new \InvalidArgumentException('Content source ID is required');
+        }
+
+        try {
+            $command = new DeleteContentSourceCommand((int) $uriVariables['id']);
+            $this->handler->handle($command);
+
+            return Response::HTTP_NO_CONTENT;
+        } catch (ContentSourceNotFoundException $e) {
+            throw new NotFoundHttpException($e->getMessage());
+        }
+    }
+}

src/Domain/Setting/Infrastructure/ApiPlatform/State/Provider/DeleteContentSourceStateProvider.php

@@ -0,0 +1,39 @@
+<?php
+
+namespace App\Domain\Setting\Infrastructure\ApiPlatform\State\Provider;
+
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\State\ProviderInterface;
+use App\Domain\Setting\Domain\Contract\Repository\ContentSourceRepositoryInterface;
+use App\Domain\Setting\Domain\Exception\ContentSourceNotFoundException;
+use App\Domain\Setting\Infrastructure\ApiPlatform\Resource\DeleteContentSourceResource;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
+
+readonly class DeleteContentSourceStateProvider implements ProviderInterface
+{
+    public function __construct(
+        private ContentSourceRepositoryInterface $contentSourceRepository
+    ) {
+    }
+
+    public function provide(Operation $operation, array $uriVariables = [], array $context = []): DeleteContentSourceResource
+    {
+        if (!isset($uriVariables['id'])) {
+            throw new NotFoundHttpException('Content source ID is required');
+        }
+
+        $id = (int) $uriVariables['id'];
+
+        try {
+            $contentSource = $this->contentSourceRepository->findById($id);
+
+            if (!$contentSource) {
+                throw new ContentSourceNotFoundException($id);
+            }
+
+            return new DeleteContentSourceResource($id);
+        } catch (ContentSourceNotFoundException $e) {
+            throw new NotFoundHttpException($e->getMessage());
+        }
+    }
+}

tests/Feature/Setting/DeleteContentSourceTest.php

@@ -0,0 +1,50 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Tests\Feature\Setting;
+
+use App\Entity\ContentSource;
+use App\Tests\Feature\AbstractApiTestCase;
+use Symfony\Component\HttpFoundation\Response;
+use Zenstruck\Foundry\Test\ResetDatabase;
+
+final class DeleteContentSourceTest extends AbstractApiTestCase
+{
+    use ResetDatabase;
+
+    private int $sourceId;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $source = new ContentSource();
+        $source->setBaseUrl('https://mangadex.org')
+            ->setChapterUrlFormat('https://mangadex.org/chapter/{id}')
+            ->setScrapingType('html');
+
+        $this->entityManager->persist($source);
+        $this->entityManager->flush();
+
+        $this->sourceId = $source->getId();
+    }
+
+    public function testItDeletesSourceSuccessfully(): void
+    {
+        static::createClient()->request('DELETE', "/api/content-sources/{$this->sourceId}");
+
+        $this->assertResponseStatusCodeSame(Response::HTTP_NO_CONTENT);
+
+        $this->entityManager->clear();
+        $deletedSource = $this->entityManager->find(ContentSource::class, $this->sourceId);
+        $this->assertNull($deletedSource);
+    }
+
+    public function testItReturnsNotFoundWhenSourceDoesNotExist(): void
+    {
+        static::createClient()->request('DELETE', '/api/content-sources/999999');
+
+        $this->assertResponseStatusCodeSame(Response::HTTP_NOT_FOUND);
+    }
+}