Refonte - Trésorerie > Journal > Modifier colonne

Author: vgreb

app/config/routing/admin_accounting/journal.yml

@@ -33,3 +33,11 @@ admin_accounting_journal_export:
   path: /export
   defaults:
     _controller: AppBundle\Controller\Admin\Accounting\Journal\ExportAction
+
+
+admin_accounting_journal_update_info:
+  path: /update-info/{id}/{modification}
+  defaults:
+    _controller: AppBundle\Controller\Admin\Accounting\Journal\UpdateInfoAction
+  requirements:
+    id: \d+

htdocs/templates/administration/compta_journal.html

@@ -96,7 +96,7 @@ <h2>Journal</h2>
             <td>
             {if $ecriture.evenement=='A déterminer'}
                 <div class="ui form">
-                    <select class=" js-select-change" data-post-url="index.php?page=compta_journal&amp;action=modifier_colonne&amp;column=evenement&amp;id={$ecriture.idtmp}" style="min-width:130px">
+                    <select class=" js-select-change" data-post-url="/admin/accounting/journal/update-info/{$ecriture.idtmp}/event" style="min-width:130px">
                         {foreach from=$events item=event_name key=event_id}
                         <option value="{$event_id}" {if $event_name == "A déterminer"}selected="selected"{/if}>{$event_name}</option>
                         {/foreach}
@@ -109,7 +109,7 @@ <h2>Journal</h2>
              <td>
                 {if $ecriture.categorie=='A déterminer'}
                  <div class="ui form">
-                    <select class="js-select-change" data-post-url="index.php?page=compta_journal&amp;action=modifier_colonne&amp;column=categorie&amp;id={$ecriture.idtmp}" style="min-width:130px">
+                    <select class="js-select-change" data-post-url="/admin/accounting/journal/update-info/{$ecriture.idtmp}/category" style="min-width:130px">
                         {foreach from=$categories item=cat_name key=cat_id}
                         <option value="{$cat_id}" {if $cat_name == "A déterminer"}selected="selected"{/if}>{$cat_name}</option>
                         {/foreach}
@@ -133,7 +133,7 @@ <h2>Journal</h2>
             <td>
             {if $ecriture.reglement=='A déterminer'}
                 <div class="ui form">
-                    <select class="js-select-change" data-post-url="index.php?page=compta_journal&amp;action=modifier_colonne&amp;column=reglement&amp;id={$ecriture.idtmp}" style="min-width:130px">
+                    <select class="js-select-change" data-post-url="/admin/accounting/journal/update-info/{$ecriture.idtmp}/paymentType" style="min-width:130px">
                         {foreach from=$payment_methods item=method_name key=method_id}
                         <option value="{$method_id}" {if $method_name == "A déterminer"}selected="selected"{/if}>{$method_name}</option>
                         {/foreach}

sources/AppBundle/Accounting/TransactionModification.php

@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AppBundle\Accounting;
+
+enum TransactionModification: string
+{
+    case Category = 'category';
+    case PaymentType = 'paymentType';
+    case Event = 'event';
+    case Comment = 'comment';
+    case RequiredAttachment = 'RequiredAttachment';
+
+    public function getPropertyName(): string
+    {
+        return match ($this) {
+            self::Category => 'categoryId',
+            self::PaymentType => 'paymentTypeId',
+            self::Event => 'eventId',
+            self::Comment => 'comment',
+            self::RequiredAttachment => 'attachmentRequired',
+        };
+    }
+
+    public function convertValue(mixed $value): int|string|bool
+    {
+        return match ($this) {
+            self::Category, self::PaymentType, self::Event => (int) $value,
+            self::Comment => (string) $value,
+            self::RequiredAttachment => (bool) $value,
+        };
+    }
+
+    public function allowsEmpty(): bool
+    {
+        return match ($this) {
+            self::Comment, self::RequiredAttachment => true,
+            default => false,
+        };
+    }
+}

sources/AppBundle/Controller/Admin/Accounting/Journal/UpdateInfoAction.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AppBundle\Controller\Admin\Accounting\Journal;
+
+use AppBundle\Accounting\Model\Repository\TransactionRepository;
+use AppBundle\Accounting\Model\Transaction;
+use AppBundle\Accounting\TransactionModification;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\JsonResponse;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
+
+class UpdateInfoAction extends AbstractController
+{
+    public function __construct(
+        private readonly TransactionRepository $transactionRepository,
+    ) {}
+
+    public function __invoke(
+        Request $request,
+        int $id,
+        TransactionModification $modification,
+    ): JsonResponse {
+        $transaction = $this->transactionRepository->get($id);
+        if (!$transaction instanceof Transaction) {
+            throw $this->createNotFoundException();
+        }
+
+        try {
+            $value = $modification->convertValue($request->getPayload()->get('val'));
+
+            if (!$modification->allowsEmpty() && empty($value)) {
+                throw new \InvalidArgumentException(sprintf('Value cannot be empty for "%s"', $value));
+            }
+
+            $setter = 'set' . ucfirst($modification->getPropertyName());
+            $transaction->{$setter}($value);
+            $this->transactionRepository->save($transaction);
+        } catch (\InvalidArgumentException $e) {
+            return new JsonResponse(['error' => $e->getMessage()], Response::HTTP_BAD_REQUEST);
+        }
+
+        return new JsonResponse(['success' => true]);
+    }
+}