fix optional parameter deprecation notice

Author: fd6130

src/Service/Billplz.php

@@ -11,13 +11,12 @@ class Billplz implements BillplzInterface
 
     private array $config;
 
-    public function __construct
-    (
+    public function __construct(
         array $config
     )
     {
-        $this->client = $config['enable_sandbox'] == false ? 
-            Client::make($config['live']['api_key'], $config['live']['signature_key']) : 
+        $this->client = $config['enable_sandbox'] == false ?
+            Client::make($config['live']['api_key'], $config['live']['signature_key']) :
             Client::make($config['sandbox']['api_key'], $config['sandbox']['signature_key'])->useSandbox();
         $this->config = $config;
     }
@@ -47,32 +46,36 @@ public function getBillPaymentUrl(string $billId): string
         return $this->isSandbox() == true ? sprintf('https://www.billplz-sandbox.com/bills/%s', $billId) : sprintf('https://www.billplz.com/bills/%s', $billId);
     }
 
-    public function createBill
-    (
+    public function createBill(
         string $collectionKey,
         ?string $email = null,
         ?string $mobile = null,
-        string $name,
-        int|string $amount,
-        string $callbackUrl,
-        string $description,
+        string $name = null,
+        int|string $amount = null,
+        string $callbackUrl = null,
+        string $description = null,
         array $optional = []
-    ) : Response
+    ): Response
     {
         $collection = $this->getCurrentConfiguration()['collection'];
 
         $index = array_search($collectionKey, array_column($collection, 'name'));
-        if($index === false)
+        if ($index === false)
         {
             throw new \LogicException(sprintf("Unable to find '%s' from the collection.", $collectionKey));
         }
 
         return $this->client->bill()
-            ->create
-            (
-                $collection[$index]['id'], $email, $mobile, $name, \Duit\MYR::given($amount * 100), $callbackUrl, $description, $optional
-            )
-        ;
+            ->create(
+                $collection[$index]['id'],
+                $email,
+                $mobile,
+                $name,
+                \Duit\MYR::given($amount * 100),
+                $callbackUrl,
+                $description,
+                $optional
+            );
     }
 
     public function computeRedirectSignature(array $params)
@@ -83,7 +86,7 @@ public function computeRedirectSignature(array $params)
             'billplzpaid' . $params['billplz[paid]']
         ];
 
-        return hash_hmac(algo: "sha256", data: implode('|', $data) , key: $this->client->getSignatureKey());
+        return hash_hmac(algo: "sha256", data: implode('|', $data), key: $this->client->getSignatureKey());
     }
 
     public function computeCallbackSignature(array $params)
@@ -103,6 +106,6 @@ public function computeCallbackSignature(array $params)
             'url' . $params['url'],
         ];
 
-        return hash_hmac(algo: "sha256", data: implode('|', $data) , key: $this->client->getSignatureKey());
+        return hash_hmac(algo: "sha256", data: implode('|', $data), key: $this->client->getSignatureKey());
     }
-}
+}

src/Service/BillplzInterface.php

@@ -14,5 +14,5 @@ public function getAllConfiguration(): array;
     public function getCurrentConfiguration(): array;
     public function computeRedirectSignature(array $params);
     public function computeCallbackSignature(array $params);
-    public function createBill(string $collectionKey, ?string $email = null, ?string $mobile = null, string $name, int|string $amount, string $callbackUrl, string $description, array $optional = []): Response;
-}
+    public function createBill(string $collectionKey, ?string $email = null, ?string $mobile = null, string $name = null, int|string $amount = null, string $callbackUrl = null, string $description = null, array $optional = []): Response;
+}