chore: set default values

Signed-off-by: Vitor Mattos <[email protected]>

Author: vitormattos

example/index.php

@@ -27,14 +27,6 @@
 
 $param = JSignParam::instance();
 
-$param->setJavaVersion('openjdk version "21.0.8" 2025-07-15 LTS');
-$param->setJavaDownloadUrl('https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.8%2B9/OpenJDK21U-jre_x64_linux_hotspot_21.0.8_9.tar.gz');
-$param->setJavaPath(__DIR__ . '/../tmp/java/bin/java');
-
-$param->setJsignPdfVersion('asdfasd');
-$param->setJSignPdfDownloadUrl('https://github.com/intoolswetrust/jsignpdf/releases/download/JSignPdf_2_3_0/jsignpdf-2.3.0.zip');
-$param->setjSignPdfJarPath(__DIR__ . '/../tmp/jsignpdf/JSignPdf.jar');
-
 $param->setCertificate($pfxCertificateContent);
 $param->setPdf(file_get_contents(__DIR__ . '/../tests/resources/pdf-test.pdf'));
 $param->setPassword($password);

src/Runtime/JSignPdfRuntimeService.php

@@ -6,10 +6,7 @@
 
 use InvalidArgumentException;
 use Jeidison\JSignPDF\Sign\JSignParam;
-use PharData;
-use PharException;
 use RuntimeException;
-use UnexpectedValueException;
 use ZipArchive;
 
 class JSignPdfRuntimeService
@@ -29,9 +26,14 @@ public function getPath(JSignParam $params): string
         if ($downloadUrl && $jsignPdfPath) {
             $baseDir = preg_replace('/\/JSignPdf.jar$/', '', $jsignPdfPath);
             if (!is_dir($baseDir)) {
-                throw new InvalidArgumentException('The JSignPdf base dir is not a real directory: '. $baseDir);
+                $ok = mkdir($baseDir, 0755, true);
+                if ($ok === false) {
+                    throw new InvalidArgumentException('The JSignPdf base dir cannot be created: '. $baseDir);
+                }
+            }
+            if (!file_exists($jsignPdfPath)) {
+                self::downloadAndExtract($params);
             }
-            self::downloadAndExtract($params);
             return $jsignPdfPath;
         }
 

src/Runtime/JavaRuntimeService.php

@@ -32,7 +32,10 @@ public function getPath(JSignParam $params): string
         if ($downloadUrl && $javaPath) {
             $baseDir = preg_replace('/\/bin\/java$/', '', $javaPath);
             if (!is_dir($baseDir)) {
-                throw new InvalidArgumentException('The java base dir is not a real directory. Create this directory first: '. $baseDir);
+                $ok = mkdir($baseDir, 0755, true);
+                if ($ok === false) {
+                    throw new InvalidArgumentException('The java base dir is not a real directory. Create this directory first: '. $baseDir);
+                }
             }
             try {
                 self::validateVersion($params);

src/Sign/JSignParam.php

@@ -13,20 +13,22 @@ class JSignParam
     private $pathPdfSigned;
     private $JSignParameters = "-a -kst PKCS12";
     private $isUseJavaInstalled = false;
-    private $javaPath = '';
+    private string $javaPath = '';
     private $tempPath;
     private $tempName;
     private $isOutputTypeBase64 = false;
-    private $jSignPdfJarPath;
-    private ?string $javaVersion = null;
-    private ?string $javaDownloadUrl = null;
-    private ?string $jSignPdfDownloadUrl = null;
+    private string $jSignPdfJarPath;
+    private string $javaVersion = 'openjdk version "21.0.8" 2025-07-15 LTS';
+    private string $javaDownloadUrl = 'https://github.com/adoptium/temurin21-binaries/releases/download/jdk-21.0.8%2B9/OpenJDK21U-jre_x64_linux_hotspot_21.0.8_9.tar.gz';
+    private string $jSignPdfDownloadUrl = 'https://github.com/intoolswetrust/jsignpdf/releases/download/JSignPdf_2_3_0/jsignpdf-2.3.0.zip';
     private ?string $jsignPdfVersion = null;
 
     public function __construct()
     {
         $this->tempName = md5(time() . uniqid() . mt_rand());
         $this->tempPath = __DIR__ . DIRECTORY_SEPARATOR . '..'  . DIRECTORY_SEPARATOR . '..'  . DIRECTORY_SEPARATOR . 'tmp' . DIRECTORY_SEPARATOR;
+        $this->javaPath = $this->tempPath . 'java'  . DIRECTORY_SEPARATOR . 'bin'  . DIRECTORY_SEPARATOR . 'java';
+        $this->jSignPdfJarPath = $this->tempPath . 'jsignpdf'  . DIRECTORY_SEPARATOR . 'JSignPdf.jar';
     }
 
     public static function instance()
@@ -179,7 +181,7 @@ public function setJavaDownloadUrl(string $url): self
         return $this;
     }
 
-    public function getJavaDownloadUrl(): ?string
+    public function getJavaDownloadUrl(): string
     {
         return $this->javaDownloadUrl;
     }
@@ -190,7 +192,7 @@ public function setJSignPdfDownloadUrl(string $url): self
         return $this;
     }
 
-    public function getJSignPdfDownloadUrl(): ?string
+    public function getJSignPdfDownloadUrl(): string
     {
         return $this->jSignPdfDownloadUrl;
     }

tests/JSignPDFTest.php

@@ -24,7 +24,7 @@ function exec(string $command, ?array &$output = null, ?int &$return_var = null)
  */
 class JSignPDFTest extends TestCase
 {
-    private $service;
+    private JSignService $service;
 
     protected function setUp(): void
     {
@@ -171,7 +171,9 @@ public function testSignWhenPasswordIsInvalid()
     public function testJSignPDFNotFound()
     {
         $this->expectExceptionMessageMatches('/JSignPDF not found/');
-        $params = JSignParamBuilder::instance()->withDefault()->setjSignPdfJarPath('invalid_path');
+        $params = JSignParamBuilder::instance()->withDefault();
+        $params->setJSignPdfDownloadUrl('');
+        $params->setjSignPdfJarPath('invalid_path');
         $params->setCertificate($this->getNewCert($params->getPassword()));
         $params->setIsUseJavaInstalled(true);
         $this->service->getVersion($params);