Refonte - Trésorerie > Comptes banques > Télécharger les justificatifs #2046

app/config/routing/admin_accounting_bank_accounts.yml

@@ -2,3 +2,9 @@ admin_accounting_bank_accounts_list:
   path: /list
   defaults:
     _controller: AppBundle\Controller\Admin\Accounting\BankAccounts\ListStatementAction
+
+
+admin_accounting_bank_accounts_download:
+  path: /download-statements
+  defaults:
+    _controller: AppBundle\Controller\Admin\Accounting\BankAccounts\DownloadStatementAction

composer.json

@@ -11,6 +11,7 @@
     "ext-libxml": "*",
     "ext-openssl": "*",
     "ext-pdo": "*",
+    "ext-zip": "*",
     "algolia/algoliasearch-client-php": "^3.4",
     "beberlei/assert": "^2.9",
     "captioning/captioning": "^2.6",

htdocs/templates/administration/compta_journal.html

@@ -59,7 +59,7 @@ <h2>Journal</h2>
             <i class="icon file"></i>
             Exporter la période en CSV
         </a>
-        <a href="index.php?page=compta_banque&amp;action=download_attachments&amp;id_periode={$smarty.get.id_periode|default:''}" class="item">
+        <a href="/admin/accounting/bank-accounts/download-statements?periodId={$smarty.get.id_periode|default:''}" class="item">
             <i class="icon file zip"></i>
             Télécharger les justificatifs groupés par mois
         </a>

sources/AppBundle/Controller/Admin/Accounting/BankAccounts/DownloadStatementAction.php

@@ -0,0 +1,64 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AppBundle\Controller\Admin\Accounting\BankAccounts;
+
+use AppBundle\Accounting\Model\Repository\InvoicingPeriodRepository;
+use DateInterval;
+use DatePeriod;
+use RuntimeException;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\BinaryFileResponse;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
+use ZipArchive;
+
+class DownloadStatementAction extends AbstractController
+{
+    public function __construct(private readonly InvoicingPeriodRepository $invoicingPeriodRepository) {}
+
+    public function __invoke(Request $request): Response
+    {
+        $periodId = $request->query->has('periodId') && $request->query->get('periodId') ? (int) $request->query->get('periodId') : null;
+        $period = $this->invoicingPeriodRepository->getCurrentPeriod($periodId);
+        try {
+            $year = $period->getStartDate()->format('Y');
+
+            // Create the zip
+            $zipFilename = sys_get_temp_dir() . DIRECTORY_SEPARATOR . 'afup_justificatifs-' . $year . '.zip';
+            $zip = new ZipArchive();
+            $state = $zip->open($zipFilename, ZipArchive::CREATE);
+            if ($state !== true) {
+                throw new RuntimeException("Impossible to open the Zip archive.");
+            } else {
+                $datePeriod = new DatePeriod($period->getStartDate(), new DateInterval('P1M'), $period->getEndDate());
+                /** @var \DateTime $month */
+                foreach ($datePeriod as $month) {
+                    $searchDir = $month->format('Ym');
+                    $zipDir = $month->format('Ym');
+                    $options = [
+                        'add_path' => 'afup_justificatifs-' . $year . '/' . $zipDir . '/',
+                        'remove_all_path' => true,
+                    ];
+                    $zip->addGlob(AFUP_CHEMIN_RACINE . '/uploads/' . $searchDir . '/*.*', 0, $options);
+                }
+                $zip->close();
+
+                $response = new BinaryFileResponse($zipFilename, Response::HTTP_OK, [
+                    'Content-Type' => 'application/zip',
+                    'Content-Transfer-Encoding' => 'Binary',
+                    'Content-Disposition' => 'attachment; filename="' . basename($zipFilename) . '"',
+                    'Cache-Control' => 'max-age=0',
+                ], false);
+                $response->deleteFileAfterSend(true);
+
+                return $response;
+            }
+        } catch (\Exception $exception) {
+            return new Response(null, Response::HTTP_BAD_REQUEST, [
+                'X-Info' => $exception->getMessage(),
+            ]);
+        }
+    }
+}

tests/behat/features/Admin/Tresorerie/Journal.feature

@@ -89,4 +89,4 @@ Feature: Administration - Trésorerie - Journal
     Given I am logged in as admin and on the Administration
     When I follow "Journal"
     And I follow "Télécharger les justificatifs groupés par mois"
-    Then the response header "Content-disposition" should match '#filename="afup_justificatifs-(.*).zip"#'
+    Then the response header "Content-Disposition" should match '#filename="afup_justificatifs-(.*).zip"#'