adjust version information

Application/Controller/orderOverviewPdfGenerator.php

@@ -11,6 +11,7 @@
 namespace D3\PdfDocuments\Application\Controller;
 
 use D3\PdfDocuments\Application\Model\Exceptions\noPdfHandlerFoundException;
+use D3\PdfDocuments\Application\Model\Exceptions\wrongPdfGeneratorInterface;
 use D3\PdfDocuments\Application\Model\Interfaces\pdfdocumentsOrderInterface;
 use D3\PdfDocuments\Application\Model\Registries\registryOrderoverview;
 use OxidEsales\Eshop\Application\Model\Order;
@@ -22,8 +23,9 @@ class orderOverviewPdfGenerator
      * @param Order $order
      * @param int $iSelLang
      * @throws noPdfHandlerFoundException
+     * @throws wrongPdfGeneratorInterface
      */
-    public function generatePdf(Order $order, $iSelLang = 0)
+    public function generatePdf(Order $order, int $iSelLang = 0)
     {
         $Pdf= $this->getPdfClass();
 
@@ -34,6 +36,7 @@ class orderOverviewPdfGenerator
     /**
      * @return pdfdocumentsOrderInterface
      * @throws noPdfHandlerFoundException
+     * @throws wrongPdfGeneratorInterface
      */
     public function getPdfClass()
     {

Application/Model/AbstractClasses/pdfdocumentsGeneric.php

@@ -12,22 +12,18 @@ namespace D3\PdfDocuments\Application\Model\AbstractClasses;
 
 use Assert\InvalidArgumentException;
 use D3\PdfDocuments\Application\Model\Constants;
-use D3\PdfDocuments\Application\Model\Exceptions\pdfGeneratorExceptionAbstract;
 use D3\PdfDocuments\Application\Model\Interfaces\pdfdocumentsGenericInterface as genericInterface;
 use OxidEsales\Eshop\Core\Base;
 use OxidEsales\Eshop\Core\Exception\StandardException;
 use OxidEsales\Eshop\Core\Registry;
 use OxidEsales\Eshop\Core\UtilsView;
-use OxidEsales\EshopCommunity\Core\Di\ContainerFacade;
 use OxidEsales\EshopCommunity\Internal\Container\ContainerFactory;
 use OxidEsales\EshopCommunity\Internal\Framework\Module\Facade\ModuleSettingServiceInterface;
-use OxidEsales\EshopCommunity\Internal\Framework\Templating\TemplateEngineInterface;
 use OxidEsales\EshopCommunity\Internal\Framework\Templating\TemplateRenderer;
 use OxidEsales\EshopCommunity\Internal\Framework\Templating\TemplateRendererBridgeInterface;
 use OxidEsales\Twig\Resolver\TemplateChain\TemplateNotInChainException;
 use Psr\Container\ContainerExceptionInterface;
 use Psr\Container\NotFoundExceptionInterface;
-use Smarty;
 use Spipu\Html2Pdf\Exception\Html2PdfException;
 use Spipu\Html2Pdf\Html2Pdf;
 use Twig\Error\Error;
@@ -46,38 +42,9 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
 
     public $filenameExtension = 'pdf';
 
-    /** @var Smarty  */
-    public $oTemplateEngine;
-
     /** @var string */
     public $filename;
 
-    /**
-     * pdfDocumentsGeneric constructor.
-     */
-    public function __construct()
-    {
-        parent::__construct();
-	    
-	    $this->oTemplateEngine =  $this->d3GetTemplateEngine();
-    }
-
-	public function d3GetTemplateEngine() :TemplateEngineInterface
-	{
-		$renderer = $this->d3GetTemplateRendererBridge()->getTemplateRenderer();
-		return $renderer->getTemplateEngine();
-	}
-	
-	/**
-	 * @throws ContainerExceptionInterface
-	 * @throws NotFoundExceptionInterface
-	 */
-	public function d3GetTemplateRendererBridge(): TemplateRendererBridgeInterface
-	{
-		return ContainerFactory::getInstance()->getContainer()
-			->get(TemplateRendererBridgeInterface::class);
-	}
-	
     public function runPreAction()
     {
     }
@@ -87,11 +54,14 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
     }
 
     /**
-     * @param $sFilename
+     * @param string $sFilename
      * @param int $iSelLang
      * @param string $target
-     * @return mixed|string|null
+     *
+     * @return string|null
+     * @throws ContainerExceptionInterface
      * @throws Html2PdfException
+     * @throws NotFoundExceptionInterface
      */
     public function genPdf($sFilename, $iSelLang = 0, $target = self::PDF_DESTINATION_STDOUT)
     {
@@ -99,10 +69,10 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
         $oPdf->setTestIsImage(false);
         $htmlContent = $this->getHTMLContent($iSelLang);
         $oPdf->writeHTML($htmlContent);
-        $oPdf->pdf->SetAuthor(Registry::getConfig()->getActiveShop()->getFieldData('oxname'));
-        $oPdf->pdf->SetTitle(Registry::getLang()->translateString($this->getTitleIdent()));
-        $oPdf->pdf->SetCreator('D³ PDF Documents for OXID eShop');
-        $oPdf->pdf->SetSubject(NULL);
+        $oPdf->pdf->setAuthor( Registry::getConfig()->getActiveShop()->getFieldData( 'oxname'));
+        $oPdf->pdf->setTitle( Registry::getLang()->translateString( $this->getTitleIdent()));
+        $oPdf->pdf->setCreator( 'D³ PDF Documents for OXID eShop');
+        $oPdf->pdf->setSubject( NULL);
         return $this->output($oPdf, $sFilename, $target, $htmlContent);
     }
 
@@ -110,6 +80,15 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
      * @param int $iLanguage
      * @throws Html2PdfException
      */
+
+    /**
+     * @param $iLanguage
+     *
+     * @return void
+     * @throws ContainerExceptionInterface
+     * @throws Html2PdfException
+     * @throws NotFoundExceptionInterface
+     */
     public function downloadPdf($iLanguage = 0)
     {
         try {
@@ -118,20 +97,20 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
             $this->genPdf($sFilename, $iLanguage, self::PDF_DESTINATION_DOWNLOAD);
             $this->runPostAction();
             Registry::getUtils()->showMessageAndExit('');
-        } catch (pdfGeneratorExceptionAbstract $e) {
-            Registry::get(UtilsView::class)->addErrorToDisplay($e);
-            Registry::getLogger()->error($e);
         } catch (InvalidArgumentException $e) {
-            Registry::get(UtilsView::class)->addErrorToDisplay($e);
             Registry::getLogger()->error($e);
+            Registry::get(UtilsView::class)->addErrorToDisplay($e);
         }
     }
 
     /**
      * @param string $path
-     * @param int    $iLanguage
+     * @param int $iLanguage
      *
+     * @return void
+     * @throws ContainerExceptionInterface
      * @throws Html2PdfException
+     * @throws NotFoundExceptionInterface
      */
     public function savePdfFile($path, $iLanguage = 0)
     {
@@ -144,9 +123,6 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
                 self::PDF_DESTINATION_FILE
             );
             $this->runPostAction();
-        } catch (pdfGeneratorExceptionAbstract $e) {
-            Registry::get(UtilsView::class)->addErrorToDisplay($e);
-            Registry::getLogger()->error($e);
         } catch (InvalidArgumentException $e) {
             Registry::get(UtilsView::class)->addErrorToDisplay($e);
             Registry::getLogger()->error($e);
@@ -156,8 +132,10 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
     /**
      * @param int $iLanguage
      *
-     * @return null|string
+     * @return string|null
+     * @throws ContainerExceptionInterface
      * @throws Html2PdfException
+     * @throws NotFoundExceptionInterface
      */
     public function getPdfContent($iLanguage = 0)
     {
@@ -167,9 +145,6 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
             $ret = $this->genPdf( $sFilename, $iLanguage, self::PDF_DESTINATION_STRING );
             $this->runPostAction();
             return $ret;
-        } catch (pdfGeneratorExceptionAbstract $e) {
-            Registry::get(UtilsView::class)->addErrorToDisplay($e);
-            Registry::getLogger()->error($e);
         } catch (InvalidArgumentException $e) {
             Registry::get(UtilsView::class)->addErrorToDisplay($e);
             Registry::getLogger()->error($e);
@@ -178,18 +153,17 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
         return null;
     }
 
-    /**
-     * @param int $iSelLang
-     */
-    public function setTemplateEngineVars($iSelLang)
+    public function getTemplateEngineVars(int $iSelLang): array
     {
         unset($iSelLang);
-		
-        $this->oTemplateEngine->addGlobal('config', Registry::getConfig());
-        $this->oTemplateEngine->addGlobal('oViewConf', Registry::getConfig()->getActiveView()->getViewConfig());
-        $this->oTemplateEngine->addGlobal('shop', Registry::getConfig()->getActiveShop());
-        $this->oTemplateEngine->addGlobal('lang', Registry::getLang());
-        $this->oTemplateEngine->addGlobal('document', $this);
+
+        return [
+            'config' => Registry::getConfig(),
+            'oViewConf' => Registry::getConfig()->getActiveView()->getViewConfig(),
+            'shop'  => Registry::getConfig()->getActiveShop(),
+            'lang'  => Registry::getLang(),
+            'document' => $this
+        ];
     }
 	
     /**
@@ -198,7 +172,7 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
      * @return mixed
      * @throws InvalidArgumentException
      */
-    public function getHTMLContent($iSelLang = 0)
+    public function getHTMLContent(int $iSelLang = 0)
     {
         $blCurrentRenderFromAdmin = self::$_blIsAdmin;
         self::$_blIsAdmin = $this->renderTemplateFromAdmin();
@@ -208,16 +182,16 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
         $currTplLang = $lang->getTplLanguage();
         $lang->setTplLanguage($iSelLang);
 
-        $this->setTemplateEngineVars($iSelLang);
-		
 	    try {
-		    $content = $this->oTemplateEngine->render($this->getTemplate(), $this->d3GetTemplateEngine()->getGlobals());
+            $content = $this->getTemplateRenderer()->renderTemplate(
+                $this->getTemplate(),
+                $this->getTemplateEngineVars($iSelLang)
+            );
 	    } catch (Error|TemplateNotInChainException $error) {
 		    
 		    //Registry::getLogger()->error(dumpVar(__METHOD__." ".__LINE__), [$error->getFile()]);
 			
-		    $error = oxNew(StandardException::class, $error->getMessage());
-		    throw $error;
+		    throw oxNew(StandardException::class, $error->getMessage());
 	    }
 	    
 	    $lang->setTplLanguage($currTplLang);
@@ -227,6 +201,13 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
         return $content;
     }
 
+    protected function getTemplateRenderer(): TemplateRenderer
+    {
+        return ContainerFactory::getInstance()->getContainer()
+            ->get(TemplateRendererBridgeInterface::class)
+            ->getTemplateRenderer();
+    }
+
     /**
      * arguments for Html2Pdf class constructor
      * - $orientation = 'P',
@@ -324,14 +305,12 @@ abstract class pdfdocumentsGeneric extends Base implements genericInterface
 
         // maximize filename length to 255 bytes
         $ext = pathinfo($filename, PATHINFO_EXTENSION);
-        $filename = mb_strcut(
+        return mb_strcut(
                         pathinfo($filename, PATHINFO_FILENAME),
                         0,
                         255 - ($ext ? strlen($ext) + 1 : 0),
                         mb_detect_encoding($filename)
                     ) . ($ext ? '.' . $ext : '');
-
-        return $filename;
     }
 
     public function beautifyFilename($filename)

Application/Model/AbstractClasses/pdfdocumentsOrder.php

@@ -57,19 +57,22 @@ abstract class pdfdocumentsOrder extends pdfdocumentsGeneric implements orderInt
      * @param int $iSelLang
      * @throws InvalidArgumentException
      */
-    public function setTemplateEngineVars($iSelLang)
+    public function getTemplateEngineVars($iSelLang): array
     {
-        parent::setTemplateEngineVars($iSelLang);
-
-        $this->oTemplateEngine->addGlobal('order', $this->getOrder());
-
         $oUser = oxNew(User::Class);
         $oUser->load($this->getOrder()->getFieldData('oxuserid'));
-        $this->oTemplateEngine->addGlobal('user', $oUser);
 
         $oPayment = oxNew(Payment::class);
         $oPayment->loadInLang($iSelLang, $this->getOrder()->getFieldData('oxpaymenttype'));
-        $this->oTemplateEngine->addGlobal('payment', $oPayment);
+
+        return array_merge(
+            parent::getTemplateEngineVars($iSelLang),
+            [
+                'order'    => $this->getOrder(),
+                'user'    => $oUser,
+                'payment'   => $oPayment
+            ]
+        );
     }
 
     /**
@@ -116,7 +119,7 @@ abstract class pdfdocumentsOrder extends pdfdocumentsGeneric implements orderInt
     }
 
     /**
-     * @return false|string
+     * @return false|int
      * @throws InvalidArgumentException
      */
     public function getPayableUntilDate()

Application/Model/Documents/invoicewithoutlogoPdf.php

@@ -39,7 +39,8 @@ class invoicewithoutlogoPdf extends invoicePdf
     /**
      * @return string
      */
-    public function getTemplate(){
+    public function getTemplate()
+    {
         return '@d3PdfDocuments/documents/invoice/invoiceNoLogo';
     }
 }

Application/Model/Interfaces/pdfdocumentsGenericInterface.php

@@ -35,29 +35,29 @@ interface pdfdocumentsGenericInterface
     /**
      * @param int $iLanguage
      */
-    public function downloadPdf($iLanguage = 0);
+    public function downloadPdf(int $iLanguage = 0);
 
     /**
      * @param int $iLanguage
      *
      * @return string|null
      */
-    public function getPdfContent($iLanguage = 0);
+    public function getPdfContent(int $iLanguage = 0);
 
     /**
      * @param string $path
      * @param int $iLanguage
      */
-    public function savePdfFile($path, $iLanguage = 0);
+    public function savePdfFile(string $path, int $iLanguage = 0);
 
     /**
-     * @param        $sFilename
+     * @param string $sFilename
      * @param int    $iSelLang
      * @param string $target
      *
      * @return mixed
      */
-    public function genPdf($sFilename, $iSelLang = 0, $target = 'I');
+    public function genPdf(string $sFilename, int $iSelLang = 0, string $target = 'I');
 
     /**
      * @param string $filename

Application/Model/Registries/registryAbstract.php

@@ -28,10 +28,12 @@ abstract class registryAbstract implements registryGenericInterface
 
     /**
      * @param $className
+     *
+     * @throws wrongPdfGeneratorInterface
      */
     public function addGenerator($className)
     {
-        if (false == $this->hasGenerator($className)) {
+        if ( ! $this->hasGenerator( $className ) ) {
             /** @var pdfdocumentsGenericInterface $generator */
             $generator = oxNew( $className );
 
@@ -44,12 +46,13 @@ abstract class registryAbstract implements registryGenericInterface
     /**
      * @param $className * generator fully qualified class name
      * @param pdfdocumentsGenericInterface $item
+     * @throws wrongPdfGeneratorInterface
      */
     protected function addItem($className, pdfdocumentsGenericInterface $item)
     {
         $requiredInterface = $this->getRequiredGeneratorInterfaceClassName();
 
-        if (false == $item instanceof $requiredInterface) {
+        if ( ! $item instanceof $requiredInterface ) {
             throw oxNew(wrongPdfGeneratorInterface::class, $requiredInterface);
         }
 

Application/Model/Registries/registryOrdermanagerActions.php

@@ -14,10 +14,14 @@ use D3\PdfDocuments\Application\Model\Documents\deliverynotePdf;
 use D3\PdfDocuments\Application\Model\Documents\deliverynotewithoutlogoPdf;
 use D3\PdfDocuments\Application\Model\Documents\invoicePdf;
 use D3\PdfDocuments\Application\Model\Documents\invoicewithoutlogoPdf;
+use D3\PdfDocuments\Application\Model\Exceptions\wrongPdfGeneratorInterface;
 use D3\PdfDocuments\Application\Model\Interfaces\pdfdocumentsOrderInterface;
 
 class registryOrdermanagerActions extends registryAbstract implements registryOrdermanagerActionsInterface
 {
+    /**
+     * @throws wrongPdfGeneratorInterface
+     */
     public function __construct()
     {
         $this->addGenerator(invoicePdf::class);

Application/Model/Registries/registryOrderoverview.php

@@ -14,10 +14,14 @@ use D3\PdfDocuments\Application\Model\Documents\deliverynotePdf;
 use D3\PdfDocuments\Application\Model\Documents\deliverynotewithoutlogoPdf;
 use D3\PdfDocuments\Application\Model\Documents\invoicePdf;
 use D3\PdfDocuments\Application\Model\Documents\invoicewithoutlogoPdf;
+use D3\PdfDocuments\Application\Model\Exceptions\wrongPdfGeneratorInterface;
 use D3\PdfDocuments\Application\Model\Interfaces\pdfdocumentsOrderInterface;
 
 class registryOrderoverview extends registryAbstract implements registryOrderoverviewInterface
 {
+    /**
+     * @throws wrongPdfGeneratorInterface
+     */
     public function __construct()
     {
         $this->addGenerator(invoicePdf::class);

Modules/Application/Controller/d3_overview_controller_pdfdocuments.php

@@ -13,38 +13,59 @@ namespace D3\PdfDocuments\Modules\Application\Controller;
 use D3\PdfDocuments\Application\Controller\orderOverviewPdfGenerator;
 use D3\PdfDocuments\Application\Model\Exceptions\noPdfHandlerFoundException;
 use D3\PdfDocuments\Application\Model\Exceptions\pdfGeneratorExceptionAbstract;
+use D3\PdfDocuments\Application\Model\Exceptions\wrongPdfGeneratorInterface;
 use D3\PdfDocuments\Application\Model\Registries\registryOrderoverview;
-use OxidEsales\Eshop\Application\Controller\Admin\OrderOverview;
+use Doctrine\DBAL\Driver\Exception;
+use Doctrine\DBAL\Exception as DBALException;
+use Doctrine\DBAL\ParameterType;
+use Doctrine\DBAL\Query\QueryBuilder;
 use OxidEsales\Eshop\Application\Model\Order;
 use OxidEsales\Eshop\Core\DatabaseProvider;
 use OxidEsales\Eshop\Core\Exception\DatabaseConnectionException;
 use OxidEsales\Eshop\Core\Registry;
 use OxidEsales\Eshop\Core\TableViewNameGenerator;
+use OxidEsales\EshopCommunity\Internal\Container\ContainerFactory;
+use OxidEsales\EshopCommunity\Internal\Framework\Database\QueryBuilderFactoryInterface;
+use Psr\Container\ContainerExceptionInterface;
+use Psr\Container\NotFoundExceptionInterface;
 
 class d3_overview_controller_pdfdocuments extends d3_overview_controller_pdfdocuments_parent
 {
-	/**
+    /**
      * @return bool
-     * @throws DatabaseConnectionException
+     * @throws Exception
      */
-    public function d3CanExport()
+    public function d3CanExport(): bool
     {
-        // We force reading from master to prevent issues with slow replications or open transactions (see ESDEV-3804).
-        $masterDb = DatabaseProvider::getMaster();
-        $sOrderId = $this->getEditObjectId();
+        try {
+            $sOrderId = $this->getEditObjectId();
 
-        $viewNameGenerator = Registry::get(TableViewNameGenerator::class);
-        $sTable = $viewNameGenerator->getViewName("oxorderarticles");
+            $viewNameGenerator = Registry::get( TableViewNameGenerator::class );
+            $sTable            = $viewNameGenerator->getViewName( "oxorderarticles" );
 
-        $sQ = "select count(oxid) from $sTable where oxorderid = " . $masterDb->quote($sOrderId) . " and oxstorno = 0";
-        return (bool) $masterDb->getOne($sQ);
+            /** @var QueryBuilder $queryBuilder */
+            $queryBuilder = ContainerFactory::getInstance()->getContainer()->get( QueryBuilderFactoryInterface::class )->create();
+            $queryBuilder
+                ->select( 'oxid' )
+                ->from( $sTable )
+                ->where(
+                    $queryBuilder->expr()->and(
+                        $queryBuilder->expr()->eq( 'oxorderid', $queryBuilder->createNamedParameter( $sOrderId ) ),
+                        $queryBuilder->expr()->eq( 'oxstorno', $queryBuilder->createNamedParameter( 0, ParameterType::INTEGER ) )
+                    )
+                );
+
+            return $queryBuilder->execute()->fetchOne();
+        } catch (NotFoundExceptionInterface|ContainerExceptionInterface|DBALException) {
+            return false;
+        }
     }
 
     /**
      * @throws noPdfHandlerFoundException
      * @throws pdfGeneratorExceptionAbstract
      */
-    public function d3CreatePDF()
+    public function d3CreatePDF(): void
     {
         $soxId = $this->getEditObjectId();
         if ($soxId != "-1" && isset($soxId)) {
@@ -58,9 +79,10 @@ class d3_overview_controller_pdfdocuments extends d3_overview_controller_pdfdocu
     }
 
     /**
-    * @return registryOrderoverview
-    */
-    public function d3getGeneratorList()
+     * @return registryOrderoverview
+     * @throws wrongPdfGeneratorInterface
+     */
+    public function d3getGeneratorList(): registryOrderoverview
     {
         return oxNew(registryOrderoverview::class);
     }

README.en.md

@@ -1,10 +1,15 @@
-> [german version](README.md)
+![stability mature](https://img.shields.io/badge/stability-mature-008000.svg)
+[![latest tag](https://img.shields.io/packagist/v/d3/pdfdocuments?label=release)](https://packagist.org/packages/d3/pdfdocuments)
+![License](https://img.shields.io/packagist/l/d3/pdfdocuments)
+
+[![deutsche Version](https://logos.oxidmodule.com/de2_xs.svg)](README.md)
+[![english version](https://logos.oxidmodule.com/en2_xs.svg)](README.en.md)
 
 # PDF Documents
 
 PDF document generator for OXID eShop
 
-Create a wide variety of static or dynamic PDF documents at the touch of a button. The document content is created from Smarty templates.
+Create a wide variety of static or dynamic PDF documents at the touch of a button. The document content is created from templates (Smarty or Twig).
 
 At the orders of your OXID shop you have the option of creating an invoice and delivery note.
 
@@ -12,7 +17,7 @@ The module can be easily extended to adapt existing documents or add new ones. E
 
 ## System requirements:
 
-- installed OXID eShop version from 7.0 - 7.1
+- installed OXID eShop version 7.x
 - PHP version for which installation packages are available (PHP 8)
 - Installation via Composer
 

README.md

@@ -1,10 +1,15 @@
-> [english version](README.en.md)
+![stability mature](https://img.shields.io/badge/stability-mature-008000.svg)
+[![latest tag](https://img.shields.io/packagist/v/d3/pdfdocuments?label=release)](https://packagist.org/packages/d3/pdfdocuments)
+![License](https://img.shields.io/packagist/l/d3/pdfdocuments)
+
+[![deutsche Version](https://logos.oxidmodule.com/de2_xs.svg)](README.md)
+[![english version](https://logos.oxidmodule.com/en2_xs.svg)](README.en.md)
 
 # PDF-Dokumente
 
 PDF-Dokumentgenerator für OXID eShop
 
-Erstellen Sie unterschiedliche statische oder dynamische PDF-Dokumente auf Kopfdruck. Der Dokumentinhalt wird aus Smartytemplates erstellt.
+Erstellen Sie unterschiedliche statische oder dynamische PDF-Dokumente auf Knopfdruck. Der Dokumentinhalt wird aus Templates (Smarty bzw. Twig) erstellt.
 
 An den Bestellungen Ihres OXID-Shops steht Ihnen die Erstellung von Rechnung und Lieferschein zur Verfügung.
 
@@ -12,7 +17,7 @@ Das Modul kann einfach erweitert werden, um bestehende Dokumente anzupassen oder
 
 ## Systemanforderungen:
 
-- installierter OXID eShop in Version ab 7.0 - 7.1
+- installierter OXID eShop in Version 7.x
 - PHP-Version, für die Installationspakete verfügbar sind (PHP 8)
 - Installation via Composer
 

composer.json

@@ -7,13 +7,18 @@
     "modules",
     "eShop",
     "d3",
-    "PDF"
+    "PDF",
+    "documents",
+    "invoice",
+    "delivery note",
+    "d3dependencycheck",
+    "id_d3PdfDocuments"
   ],
   "authors": [
     {
         "name": "D3 Data Development (Inh. Thomas Dartsch)",
         "email": "info@shopmodule.com",
-        "homepage": "http://www.d3data.de",
+        "homepage": "https://www.d3data.de",
         "role": "Owner"
     }
   ],
@@ -21,17 +26,9 @@
   "license": [
     "GPL-3.0-or-later"
   ],
-  "extra": {
-    "oxideshop": {
-      "blacklist-filter": [
-        "composer.json",
-        "docs/**/*.*"
-      ]
-    }
-  },
   "require": {
     "php": "^8.0",
-    "oxid-esales/oxideshop-ce": "7.0 - 7.1",
+    "oxid-esales/oxideshop-ce": "7.0 - 7.2",
     "spipu/html2pdf": "~5.2.8",
     "beberlei/assert": "^3.3.2"
   },

docs/daux/changelog/010_Changelog.md

@@ -7,11 +7,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://git.d3data.de/D3Public/pdfdokumente/compare/2.0.0...rel_2.x)
+## [Unreleased](https://git.d3data.de/D3Public/pdfdokumente/compare/2.0.2.0...rel_2.x)
 
-## [2.0.0](https://git.d3data.de/D3Public/pdfdokumente/compare/1.0.4.0...2.0.0) - 2024-08-XX
+## [2.0.2.0](https://git.d3data.de/D3Public/pdfdokumente/compare/2.0.1.1...2.0.2.0) - 2025-01-21
+### Added
+- installierbar in OXID 7.2
+### Fixed
+- Syntaxfehler in Twig Templates
+- Wiederverwendung von Captures in unterschiedlichen Templates der selben Rendererinstanz in Smarty behoben
+### Changed
+- Template Renderer Verwendung
+
+## [2.0.1.1](https://git.d3data.de/D3Public/pdfdokumente/compare/2.0.1.0...2.0.1.1) - 2024-10-04
+### Fixed
+- Syntax der Steuerangaben in Twig-Templates
+
+## [2.0.1.0](https://git.d3data.de/D3Public/pdfdokumente/compare/2.0.0.0...2.0.1.0) - 2024-10-01
+### Added
+- Entwicklungshandbuch
+- Überladungsblock für Unternehmenslogo
+### Fixed
+- CSS wird nicht als Referenz sondern inline eingebunden
+- Entwicklermodus kann konfiguriert werden
+- Smarty Templates
+### Removed
+- Modul Connector Bedingung
+
+## [2.0.0.0](https://git.d3data.de/D3Public/pdfdokumente/compare/1.0.4.0...2.0.0.0) - 2024-09-12
 ### Added
 - installierbar in OXID 7.0 && 7.1 (CE 7.0.x - 7.1.x)
+- Support für Smarty- und Twig-Templates
+### Removed
+- Support für OXID < 7.0
 
 ## [1.0.4.0](https://git.d3data.de/D3Public/pdfdokumente/compare/1.0.3.1...1.0.4.0) - 2023-12-22
 ### Added

docs/daux/config.json

@@ -1,9 +1,9 @@
 {
     "title": "<i class='fab fa-d3 d3fa-color-blue'></i> PDF Dokumente",
-    "moduleversion": "2.0.0",
+    "moduleversion": "2.0.2.0",
     "titledesc": "für den Oxid eShop",
     "author": "D³ Data Development",
-    "moduledate": "12.09.2024",
+    "moduledate": "21.01.2025",
     "editors": "",
     "tagline": "",
     "image": "",

docs/daux/installation/010_Anforderungen/010_Systemanforderungen.md

@@ -8,10 +8,10 @@ title: Systemanforderungen
     * OXID eShop Community Edition (CE), Professional Edition (PE) oder Enterprise Edition (EE) in Compilation Version
         * 7.0.x
         * 7.1.x
+        * 7.2.x
 * Themes
     * Diese Modul bindet sich nicht in Frontendthemes ein und ist daher davon unabhängig
-* (D3) Modul-Connector ([kostenfrei bei D3 erhältlich](https://www.oxidmodule.com/connector/)) ab Version 7.1
 * Template-Engine
-    * Smarty ( Ungetestet! )
+    * Smarty
     * Twig
 * Installation via [Composer](https://getcomposer.org)

docs/daux/installation/010_Anforderungen/030_Kompatibilitaet_mit_anderen_Modulen.md

@@ -2,4 +2,4 @@
 title: Kompatibilität mit anderen Modulen
 ---
 
-Das PDF Dokumente-Modul integriert sich auch in die Erweiterung "Auftragsmanager" von (D3) Data Development, sofern dieses im Shop installiert wurde. Diese Integration wurde für die Version 4.0.0.0 des "Auftragsmanager"-Moduls entwickelt. 
+Das PDF Dokumente-Modul integriert sich auch in die Erweiterung "Auftragsmanager" von (D3) Data Development, sofern dieses im Shop installiert wurde. Diese Integration wurde für die Version 6.0.0.0 des "Auftragsmanager"-Moduls entwickelt. 

docs/daux/installation/060_Deinstallation/040_Dateien_loeschen.md

@@ -2,8 +2,7 @@
 title: Dateien löschen
 ---
   
-Löschen Sie den Ordner `{$modulefolder}` und seine enthaltenen Elemente aus dem Verzeichnis `source/modules/d3` Ihres Shops. 
-Bestätigen Sie anschließend im Shopadmin unter [ Erweiterungen ] -> [ Module ] -> [ Installierte Shop-Module ] die Nachricht:
+Sofern nötig, bestätigen Sie im Shopadmin unter [ Erweiterungen ] -> [ Module ] -> [ Installierte Shop-Module ] die Nachricht:
 
 > Es gibt registrierte Erweiterungen im eShop, deren Verzeichnis nicht vorhanden ist.    
 > Möchten Sie alle Modulinformationen einschließlich Konfigurationseinstellungen und Blockinformationen für Templates löschen?

docs/daux/manual/010_Funktionen/010_Funktionen.md

@@ -2,8 +2,8 @@
 title: Funktionen
 ---
 
-Erstellen Sie unterschiedlichste statische oder dynamische PDF-Dokument auf Kopfdruck. Der Dokumentinhalt wird aus Twigtemplates erstellt.
+Erstellen Sie unterschiedlichste statische oder dynamische PDF-Dokument auf Kopfdruck. Der Dokumentinhalt wird aus Templates (Smarty bzw. Twig) erstellt.
 
 An den Bestellungen Ihres OXID-Shops steht Ihnen die Erstellung von Rechnung und Lieferschein zur Verfügung.
 
-Das Modul kann einfach erweitert werden, um bestehende Dokumente anzupassen oder Neue hinzuzufügen.  
+Das Modul kann einfach erweitert werden, um bestehende Dokumente anzupassen oder Neue hinzuzufügen.

docs/daux/manual/020_Programmierung/010_Anpassungen.md

@@ -2,7 +2,7 @@
 title: Anpassungen an bestehenden Dokumenten
 ---
 
-Die Dokumente werden aus Twigtemplates erstellt, die Sie im `views/twig/documents` finden. Die entsprechenden Smarty Pendants sind unter `views/smarty/documents`.
+Die Dokumente werden aus Templates (Smarty bzw. Twig) erstellt, die Sie im Ordner `views/smarty/documents` bzw. `views/twig/documents` finden.
 
 Für Änderungen einzelner Dokumentbereiche können Sie die darin notierten Templateblöcke in einem eigenen Modul überladen und deren Inhalt so verändern oder ergänzen. So müssen Sie das Originalmodul nicht verändern.
 

docs/daux/manual/020_Programmierung/020_Templatedesign.md

@@ -14,10 +14,9 @@ Layouts können mit inline CSS-Styles definiert werden. Die Einbindung externer
 
 Beachten Sie bitte, dass auch nur ein CSS-Subset unterstützt wird. Mit etwas CSS-Kreativität sollten sich die üblichen Formatierungen dennoch deutlich einfacher umsetzen lassen, als dies direkt in PHP-Programmierung möglich wäre.
 
-## Twig
+## Templates
 
-Innerhalb der Templates steht Ihnen die komplette Twig-Funktionalität zur Verfügung.
-Smarty wird perspektivisch noch nachgereich.
+Innerhalb der Templates steht Ihnen die komplette Twig- bzw. Smarty-Funktionalität zur Verfügung.
 
 ## Debug
 

metadata.php

@@ -12,21 +12,15 @@ use D3\PdfDocuments\Application\Model\Constants as Constants;
 use D3\PdfDocuments\Modules\Application\Controller\d3_overview_controller_pdfdocuments;
 use OxidEsales\Eshop\Application\Controller\Admin\OrderOverview;
 
-/**
- * Metadata version
- */
 $sMetadataVersion = '2.1';
 
-/**
- * Module information
- */
 $aModule = [
     'id'            => Constants::OXID_MODULE_ID,
     'title'         => [
         'de'        => '(D3) PDF-Dokumente',
         'en'        => '(D3) PDF documents',
     ],
-    'version'       => '2.0.0',
+    'version'       => '2.0.2.0',
     'author'        => 'D3 Data Development (Inh.: Thomas Dartsch)',
     'email'         => 'support@shopmodule.com',
     'url'           => 'https://www.oxidmodule.com/',

views/smarty/documents/deliverynote/deliverynote.tpl

@@ -1,19 +1,19 @@
 [{assign var="pagePadding"   value=","|explode:"45,15,30,25"}] [{* top, right, bottom, left *}]
 [{assign var="showLogo"      value=$showLogo|default:true}]
 
-[{capture append="pdfBlock_style"}]
+[{capture assign="pdfBlock_style"}]
     [{block name="pdfStyles"}]
         [{include file="@d3PdfDocuments/assets/d3pdfstyles.css.tpl"}]
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_header"}]
+[{capture assign="pdfBlock_header"}]
     [{block name="pdfHeader"}]
         [{include file="@d3PdfDocuments/documents/inc/page/header.tpl" showLogo=$showLogo}]
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_content"}]
+[{capture assign="pdfBlock_content"}]
     [{include file="@d3PdfDocuments/documents/inc/elements/foldmarks.tpl" pagePadding=$pagePadding}]
 
     [{block name="pdfAddressArea"}]
@@ -37,10 +37,10 @@
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_footer"}]
+[{capture assign="pdfBlock_footer"}]
     [{block name="pdfFooter"}]
         [{include file="@d3PdfDocuments/documents/inc/page/footer.tpl" pagePadding=$pagePadding}]
     [{/block}]
 [{/capture}]
 
-[{include file="@d3PdfDocuments/documents/inc/page/base.tpl" pagePadding=$pagePadding}]
+[{include file="@d3PdfDocuments/documents/inc/page/base.tpl" pagePadding=$pagePadding}]

views/smarty/documents/invoice/invoice.tpl

@@ -1,19 +1,19 @@
 [{assign var="pagePadding"   value=","|explode:"45,15,30,25"}] [{* top, right, bottom, left *}]
 [{assign var="showLogo"      value=$showLogo|default:true}]
 
-[{capture append="pdfBlock_style"}]
+[{capture assign="pdfBlock_style"}]
     [{block name="pdfStyles"}]
         [{include file="@d3PdfDocuments/assets/d3pdfstyles.css.tpl"}]
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_header"}]
+[{capture assign="pdfBlock_header"}]
     [{block name="pdfHeader"}]
         [{include file="@d3PdfDocuments/documents/inc/page/header.tpl" showLogo=$showLogo}]
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_content"}]
+[{capture assign="pdfBlock_content"}]
     [{* include file="@d3PdfDocuments/documents/inc/helper/rulers" pagePadding=$pagePadding *}]
     [{include file="@d3PdfDocuments/documents/inc/elements/foldmarks.tpl" pagePadding=$pagePadding}]
 
@@ -42,10 +42,10 @@
     [{/block}]
 [{/capture}]
 
-[{capture append="pdfBlock_footer"}]
+[{capture assign="pdfBlock_footer"}]
     [{block name="pdfFooter"}]
         [{include file="@d3PdfDocuments/documents/inc/page/footer.tpl" pagePadding=$pagePadding}]
     [{/block}]
 [{/capture}]
 
-[{include file="@d3PdfDocuments/documents/inc/page/base.tpl" pagePadding=$pagePadding}]
+[{include file="@d3PdfDocuments/documents/inc/page/base.tpl" pagePadding=$pagePadding}]

views/twig/documents/deliverynote/inc/informations.html.twig

@@ -3,7 +3,7 @@
         {% block pdfDocumentInformations %}
             <div>
                 {% set dateFormat = 'D3_PDFDOCUMENTS_DATE_FORMAT'|translate %}
-                {{ translate({ ident: "D3_PDFDOCUMENTS_DATE", suffix: "COLON" }) }} {{ smarty.now|date_format(dateFormat) }}
+                {{ translate({ ident: "D3_PDFDOCUMENTS_DATE", suffix: "COLON" }) }} {{ "now"|date_format(dateFormat) }}
             </div>
         {% endblock %}
     </div>

views/twig/documents/inc/elements/articlecostssummary.html.twig

@@ -32,7 +32,7 @@
             <tr class="producttax">
                 <td class="indent"></td>
                 <td class="description">
-                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", args: VatKey, suffix: "COLON" }) }}
+                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", suffix: "COLON" }) | format( VatKey ) }}
                 </td>
                 <td class="values">
                     {{ lang.formatCurrency(oVat, currency) }} {{ currency.name }}
@@ -104,7 +104,7 @@
             <td class="indent"></td>
             <td class="description">
                 {% if config.getConfigParam('sAdditionalServVATCalcMethod') != 'proportional' %}
-                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", args: order.getFieldData('oxdelvat'), suffix: "COLON" }) }}
+                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", suffix: "COLON" }) | format( order.getFieldData('oxdelvat') ) }}
                 {% else %}
                     {{ translate({ ident: "D3_PDFDOCUMENTS_PROPORTIONAL_TAX", suffix: "COLON" }) }}
                 {% endif %}
@@ -142,7 +142,7 @@
             <td class="indent"></td>
             <td class="description">
                 {% if config.getConfigParam('sAdditionalServVATCalcMethod') != 'proportional' %}
-                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", args: order.getFieldData('oxpayvat'), suffix: "COLON" }) }}
+                    {{ translate({ ident: "D3_PDFDOCUMENTS_TAX", suffix: "COLON" }) | format( order.getFieldData('oxpayvat') ) }}
                 {% else %}
                     {{ translate({ ident: "D3_PDFDOCUMENTS_PROPORTIONAL_TAX", suffix: "COLON" }) }}
                 {% endif %}

views/twig/documents/inc/helper/rulers.html.twig

@@ -22,15 +22,15 @@
 </style>
 
 {# horizontal #}
-{% for rulerItemsHorizontal in 10..600 %}
-    <div class="rulerItemHorizontal" style="left: {{ smarty.section.rulerItemsHorizontal.index - pagePadding.3 }}mm">
+{% for rulerItemsHorizontal in range(10, 600, 10) %}
+    <div class="rulerItemHorizontal" style="left: {{ loop.index0 - pagePadding.3 }}mm">
         {{ loop.index0 }}
     </div>
 {% endfor %}
 
 {# vertical #}
-{% for rulerItemsVertical in 0..600 %}
-    <div class="rulerItemVertical" style="top: {{ smarty.section.rulerItemsVertical.index - pagePadding.0 }}mm">
+{% for rulerItemsVertical in range(0, 600, 10) %}
+    <div class="rulerItemVertical" style="top: {{ loop.index0 - pagePadding.0 }}mm">
         {{ loop.index0 }}
     </div>
 {% endfor %}

views/twig/documents/invoice/invoice.html.twig

@@ -14,6 +14,7 @@
 {% endset %}
 
 {% set pdfBlock_content %}
+{#    {% include "@d3PdfDocuments/documents/inc/helper/rulers.html.twig" with {pagePadding: pagePadding} %}#}
     {% include "@d3PdfDocuments/documents/inc/elements/foldmarks.html.twig" with {pagePadding: pagePadding} %}
 
     {% block pdfAddressArea %}