type fixes

application/Espo/Tools/Pdf/ContentsString.php

@@ -33,9 +33,11 @@ use Psr\Http\Message\StreamInterface;
 
 use GuzzleHttp\Psr7\Stream;
 
+use RuntimeException;
+
 class ContentsString implements Contents
 {
-    private $contents;
+    private string $contents;
 
     private function __construct(string $contents)
     {
@@ -46,6 +48,10 @@ class ContentsString implements Contents
     {
         $resource = fopen('php://temp', 'r+');
 
+        if ($resource === false) {
+            throw new RuntimeException("Could not open temp.");
+        }
+
         fwrite($resource, $this->getString());
         rewind($resource);
 

application/Espo/Tools/Pdf/Data/DataLoaderManager.php

@@ -59,7 +59,7 @@ class DataLoaderManager
             $data = Data::create();
         }
 
-        /** @var class-string[] */
+        /** @var class-string<DataLoader>[] */
         $classNameList = $this->metadata->get(['pdfDefs', $entity->getEntityType(), 'dataLoaderClassNameList']) ?? [];
 
         foreach ($classNameList as $className) {
@@ -74,7 +74,7 @@ class DataLoaderManager
     }
 
     /**
-     * @param class-string $className
+     * @param class-string<DataLoader> $className
      */
     private function createLoader(string $className): DataLoader
     {

application/Espo/Tools/Pdf/PrinterController.php

@@ -84,7 +84,7 @@ class PrinterController
 
     private function createEntityPrinter(): EntityPrinter
     {
-        /** @var ?class-string */
+        /** @var ?class-string<EntityPrinter> */
         $className = $this->metadata
             ->get(['app', 'pdfEngines', $this->engine, 'implementationClassNameMap', 'entity']) ?? null;
 
@@ -97,7 +97,7 @@ class PrinterController
 
     private function createCollectionPrinter(): CollectionPrinter
     {
-        /** @var ?class-string */
+        /** @var ?class-string<CollectionPrinter> */
         $className = $this->metadata
             ->get(['app', 'pdfEngines', $this->engine, 'implementationClassNameMap', 'collection']) ?? null;
 

application/Espo/Tools/Pdf/Tcpdf/EntityProcessor.php

@@ -30,6 +30,7 @@
 namespace Espo\Tools\Pdf\Tcpdf;
 
 use Espo\Core\Utils\Config;
+use Espo\Core\Utils\Json;
 use Espo\Core\Htmlizer\TemplateRendererFactory;
 use Espo\Core\Htmlizer\TemplateRenderer;
 
@@ -225,7 +226,7 @@ class EntityProcessor
             ];
         }
 
-        $paramsString = urlencode(json_encode($params));
+        $paramsString = urlencode(Json::encode($params));
 
         return "<tcpdf method=\"{$function}\" params=\"{$paramsString}\" />";
     }

application/Espo/Tools/Pdf/Tcpdf/Tcpdf.php

@@ -35,6 +35,8 @@ use Espo\Core\Utils\Util;
 
 use TCPDF_STATIC;
 
+use Espo\Core\Utils\Json;
+
 class Tcpdf extends TcpdfOriginal
 {
     /**
@@ -64,7 +66,7 @@ class Tcpdf extends TcpdfOriginal
 
     public function serializeTCPDFtagParameters($data) /** @phpstan-ignore-line */
     {
-        return urlencode(json_encode($data));
+        return urlencode(Json::encode($data));
     }
 
     protected function unserializeTCPDFtagParameters($data) /** @phpstan-ignore-line */

application/Espo/Tools/Pdf/Tcpdf/TcpdfContents.php

@@ -36,6 +36,8 @@ use Psr\Http\Message\StreamInterface;
 
 use GuzzleHttp\Psr7\Stream;
 
+use RuntimeException;
+
 class TcpdfContents implements Contents
 {
     private $pdf;
@@ -49,6 +51,10 @@ class TcpdfContents implements Contents
     {
         $resource = fopen('php://temp', 'r+');
 
+        if ($resource === false) {
+            throw new RuntimeException("Could not open temp.");
+        }
+
         fwrite($resource, $this->getString());
         rewind($resource);