Add Smartbill API exception handling and invoice functionality

Expanded Smartbill API exception to include status code handling and introduced creation of new API responses for invoices. Changes were also made in Invoice, InvoiceEndpoint, ProformaEndpoint and SmartBillCloudRestClient classes to improve exception handling and data return types. A new file, ProformaInvoice, was also introduced. These enhancements improve code reliability and functionality.

.gitignore

@@ -9,4 +9,4 @@ coverage
 .php_cs.cache
 .phpunit.result.cache
 .php-cs-fixer.cache
-
+/.phpactor.json

src/ApiResponse/CreateInvoiceResponse.php

@@ -0,0 +1,65 @@
+<?php
+
+namespace Ag84ark\SmartBill\ApiResponse;
+
+use Illuminate\Contracts\Support\Arrayable;
+
+class CreateInvoiceResponse implements Arrayable
+{
+    private string $errorText = '';
+    private string $message = '';
+
+    private string $number = '';
+
+    private string $series = '';
+
+    private string $url = '';
+
+    public static function fromArray(array $data): CreateInvoiceResponse
+    {
+        $invoiceResponse = new CreateInvoiceResponse();
+        $invoiceResponse->errorText = $data['errorText'];
+        $invoiceResponse->message = $data['message'];
+        $invoiceResponse->number = $data['number'];
+        $invoiceResponse->series = $data['series'];
+        $invoiceResponse->url = $data['url'];
+
+        return $invoiceResponse;
+    }
+
+    public function getErrorText(): string
+    {
+        return $this->errorText;
+    }
+
+    public function getMessage(): string
+    {
+        return $this->message;
+    }
+
+    public function getNumber(): string
+    {
+        return $this->number;
+    }
+
+    public function getSeries(): string
+    {
+        return $this->series;
+    }
+
+    public function getUrl(): string
+    {
+        return $this->url;
+    }
+
+    public function toArray(): array
+    {
+        return [
+            'errorText' => $this->errorText,
+            'message' => $this->message,
+            'number' => $this->number,
+            'series' => $this->series,
+            'url' => $this->url,
+        ];
+    }
+}

src/Endpoints/InvoiceEndpoint.php

@@ -2,6 +2,7 @@
 
 namespace Ag84ark\SmartBill\Endpoints;
 
+use Ag84ark\SmartBill\ApiResponse\CreateInvoiceResponse;
 use Ag84ark\SmartBill\Resources\Invoice;
 use Ag84ark\SmartBill\Resources\ReverseInvoices;
 use Ag84ark\SmartBill\SmartBillCloudRestClient;
@@ -26,7 +27,7 @@ public function getPDFInvoice($number)
         return $this->rest_read($url,  'Accept: application/octet-stream');
     }
 
-    public function createInvoice(Invoice $invoice)
+    public function createInvoice(Invoice $invoice): CreateInvoiceResponse
     {
         return $this->createInvoiceFromArray($invoice->toArray());
     }
@@ -64,9 +65,9 @@ public function getStatusInvoicePayments($number)
         return $this->rest_read($url);
     }
 
-    public function createInvoiceFromArray(array $data)
+    public function createInvoiceFromArray(array $data): CreateInvoiceResponse
     {
-        return $this->rest_create(self::INVOICE_URL, $data);
+        return CreateInvoiceResponse::fromArray($this->rest_create(self::INVOICE_URL, $data));
     }
 
     public function createReverseInvoiceFromArray(array $data)

src/Endpoints/ProformaEndpoint.php

@@ -2,7 +2,7 @@
 
 namespace Ag84ark\SmartBill\Endpoints;
 
-use Ag84ark\SmartBill\Resources\Invoice;
+use Ag84ark\SmartBill\Resources\ProformaInvoice;
 use Ag84ark\SmartBill\SmartBillCloudRestClient;
 
 class ProformaEndpoint extends BaseEndpoint
@@ -25,7 +25,7 @@ public function getPDFProforma($number)
         return $this->rest_read($url, 'Accept: application/octet-stream');
     }
 
-    public function createProforma(Invoice $invoice)
+    public function createProforma(ProformaInvoice $invoice)
     {
         return $this->createProformaFromArray($invoice->toArray());
     }

src/Exceptions/SmartbillApiException.php

@@ -4,4 +4,17 @@
 
 class SmartbillApiException extends \Exception
 {
+    private int $statusCode;
+
+    public function __construct($message = "", $statusCode = 0, \Throwable $previous = null)
+    {
+        $message = $message ?: 'Smartbill API Exception';
+        $this->statusCode = $statusCode;
+        parent::__construct($message, $statusCode, $previous);
+    }
+
+    public function getStatusCode(): int
+    {
+        return $this->statusCode;
+    }
 }

src/Resources/Invoice.php

@@ -59,6 +59,8 @@ class Invoice
      */
     private bool $useEstimateDetails = false;
 
+    private ?array $estimate = null;
+
     /**
      * true - se foloseste tva la incasare;
      * false  - nu se foloseste tva la incasare
@@ -79,9 +81,6 @@ class Invoice
     /** se completeaza doar pentru afisarea linkului de plata pe pdf */
     private ?string $paymentUrl = null;
 
-    /** numarul proformei */
-    private ?string $number = null;
-
     /** Trimiterea facturii clientului la emitere utilizand $email data de mai jos */
     private bool $sendEmail = false;
 
@@ -108,11 +107,15 @@ public static function make(): self
         return new self();
     }
 
-    public static function makeProforma(string $number): self
+    public static function makeFromProforma(string $proformaSeries, string $number): self
     {
         $invoice = new self();
         $invoice->seriesName = config('smartbill.proformaSeries');
-        $invoice->number = $number;
+        $invoice->useEstimateDetails = true;
+        $invoice->estimate = [
+            'number' => $number,
+            'seriesName' => $proformaSeries,
+        ];
 
         return $invoice;
     }
@@ -161,12 +164,14 @@ public function toArray(): array
             'colectedTax' => $this->colectedTax,
             'paymentTotal' => $this->paymentTotal,
             'paymentUrl' => $this->paymentUrl,
-            'number' => $this->number,
             'sendEmail' => $this->sendEmail,
             'email' => $this->email ? $this->email->toArray() : null,
-            'products' => $this->products->map(function (InvoiceProduct $product) {
-                return $product->toArray();
-            })->toArray(),
+            'estimate' => $this->estimate,
+            'products' => $this->products->count()
+                ? $this->products->map(function (InvoiceProduct $product) {
+                    return $product->toArray();
+                })->toArray()
+                : null,
         ])->filter(function ($value) {
             return ! is_null($value);
         })->toArray();
@@ -524,18 +529,6 @@ public function setPaymentUrl(?string $paymentUrl): Invoice
         return $this;
     }
 
-    public function getNumber(): ?string
-    {
-        return $this->number;
-    }
-
-    public function setNumber(?string $number): Invoice
-    {
-        $this->number = $number;
-
-        return $this;
-    }
-
     public function isSendEmail(): bool
     {
         return $this->sendEmail;
@@ -571,4 +564,19 @@ public function setProducts(Collection $products): Invoice
 
         return $this;
     }
+
+    public function getEstimate(): ?array
+    {
+        return $this->estimate;
+    }
+
+    public function setEstimate(array $estimate): Invoice
+    {
+        $this->estimate = [
+            'number' => $estimate['number'],
+            'seriesName' => $estimate['seriesName'],
+        ];
+
+        return $this;
+    }
 }