fix: entity repository

config/services.yml

@@ -51,9 +51,6 @@ services:
   PrestaShop\Module\Psgdpr\Repository\ConsentRepository:
     class: 'PrestaShop\Module\Psgdpr\Repository\ConsentRepository'
 
-  PrestaShop\Module\Psgdpr\Repository\ConsentLangRepository:
-    class: 'PrestaShop\Module\Psgdpr\Repository\ConsentLangRepository'
-
   ## CONTROLLERS ##
   PrestaShop\Module\Psgdpr\Controller\Admin\CustomerController:
     class: 'PrestaShop\Module\Psgdpr\Controller\Admin\CustomerController'

src/Entity/PsgdprConsentLang.php

@@ -25,7 +25,7 @@
 
 /**
  * @ORM\Table()
- * @ORM\Entity(repositoryClass="PrestaShop\Module\Psgdpr\Repository\ConsentLangRepository")
+ * @ORM\Entity()
  */
 class PsgdprConsentLang
 {

src/Repository/ConsentRepository.php

@@ -37,9 +37,9 @@ public function __construct(ManagerRegistry $registry)
      *
      * @param PsgdprConsent $psgdprConsent
      *
-     * @return bool
+     * @return void
      */
-    public function createOrUpdateConsent(PsgdprConsent $psgdprConsent): bool
+    public function createOrUpdateConsent(PsgdprConsent $psgdprConsent): void
     {
         /** @var PsgdprConsent|null $consent */
         $consent = $this->findConsentByModuleId($psgdprConsent->getModuleId());
@@ -64,8 +64,6 @@ public function createOrUpdateConsent(PsgdprConsent $psgdprConsent): bool
 
         $this->getEntityManager()->persist($consent);
         $this->getEntityManager()->flush();
-
-        return true;
     }
 
     /**

src/Repository/LoggerRepository.php

@@ -36,14 +36,12 @@ public function __construct(ManagerRegistry $registry)
      *
      * @param PsgdprLog $log
      *
-     * @return bool
+     * @return void
      */
-    public function add(PsgdprLog $log): bool
+    public function add(PsgdprLog $log)
     {
         $this->getEntityManager()->persist($log);
         $this->getEntityManager()->flush();
-
-        return true;
     }
 
     /**

src/Service/ExportService.php

@@ -34,9 +34,7 @@
 use Language;
 use Module;
 use Order;
-use PDF;
 use PDFGenerator;
-use PdfGeneratorService;
 use PrestaShop\PrestaShop\Adapter\Entity\CustomerThread;
 use PrestaShop\PrestaShop\Core\Domain\Customer\ValueObject\CustomerId;
 use PrestaShopBundle\Translation\TranslatorInterface;
@@ -250,7 +248,14 @@ private function exportCustomerToPdf(CustomerId $customerId, array $customerData
         $this->context->smarty->escape_html = false;
 
         $pdfGenerator = new PDFGenerator(false, 'P');
-        $template = new PdfGeneratorService($customerData, $this->context->smarty);
+
+        $smarty = $this->context->smarty;
+
+        if ($smarty === null) {
+            throw new PrestaShopException('Smarty not initialized');
+        }
+
+        $template = new PdfGeneratorService($customerData, $smarty);
 
         $pdfGenerator->setFontForLang($this->context->language->iso_code);
         $pdfGenerator->startPageGroup();

src/Service/PdfGeneratorService.php

@@ -20,6 +20,15 @@
 
 // require _PS_MODULE_DIR_.'psgdpr/psgdpr.php';
 
+namespace PrestaShop\Module\Psgdpr\Service;
+
+use Configuration;
+use Context;
+use HTMLTemplate;
+use Shop;
+use Smarty;
+use Tools;
+
 class PdfGeneratorService extends HTMLTemplate
 {
     /**
@@ -40,8 +49,6 @@ class PdfGeneratorService extends HTMLTemplate
     /**
      * @param array $customerData
      * @param Smarty $smarty
-     *
-     * @throws PrestaShopException
      */
     public function __construct($customerData, Smarty $smarty)
     {
@@ -61,8 +68,6 @@ public function __construct($customerData, Smarty $smarty)
      * Returns the template's HTML footer
      *
      * @return string HTML footer
-     *
-     * @throws SmartyException
      */
     public function getFooter()
     {
@@ -83,8 +88,6 @@ public function getFooter()
      * Returns the template's HTML content
      *
      * @return string HTML content
-     *
-     * @throws SmartyException
      */
     public function getContent()
     {
@@ -113,7 +116,6 @@ public function getContent()
             }
         }
 
-        // Generate smarty data
         $this->smarty->assign([
             'customerInfo' => [
                 'headers' => $this->customerData['personalinformations']['headers'],