Invices and Results are typed objects now (#46)

Author: pablorsk

composer.json

@@ -91,7 +91,7 @@
                 "global": { "lines": 40, "functions": 29 },
                 "/src/Models": { "functions": 25 },
                 "/src/WSAA": { "functions": 0 },
-                "/src/WSFE": { "functions": 56 },
+                "/src/WSFE": { "functions": 46 },
                 "/src/WSMTXCA": { "functions": 0 },
                 "/src/WSPN3": { "functions": 81 }
             }

composer.lock

@@ -14,4 +14,3 @@ parameters:
         - '#Access to an undefined property .*\\Wsaa::\$#'
         - '#Call to an undefined method .*\\Wsaa::#'
         - '#Parameter \#1 ...\$rule of static method Respect\\Validation\\Validator\:\:allOf\(\) expects#'
-        - '#Cannot call method getMessage\(\) on stdClass.#'

resources/rules/phpstan.src.neon

@@ -13,6 +13,9 @@
 use Multinexo\AfipValues\IvaConditionCodes;
 use Multinexo\AfipValues\UnitSizeCodes;
 
+/**
+ * @deprecated
+ */
 class AfipDetail
 {
     /**

src/AfipInvoices/AfipDetail.php

@@ -16,6 +16,9 @@
 use Multinexo\WSMTXCA\AfipResult;
 use Multinexo\WSMTXCA\Wsmtxca;
 
+/**
+ * @deprecated
+ */
 class AfipInvoice
 {
     /** @var int */

src/AfipInvoices/AfipInvoice.php

@@ -79,7 +79,7 @@ private function getDataArray(): array
             'fechaServicioDesde' => null,
             'fechaServicioHasta' => null,
             'fechaVencimientoPago' => null,
-            'arrayComprobantesAsociados' => null,
+            'comprobantesAsociados' => null,
         ];
     }
 

src/AfipInvoices/AfipInvoiceTranslator.php

@@ -30,25 +30,32 @@ public function obtenerObservaciones(stdClass $path, string $name): ?array
     }
 
     // Recupera información de errores detectados lanzandolo en una excepción.
-    public function procesar(stdClass $resultado): void
+
+    /**
+     * @param \SoapFault|stdClass $resultado
+     *
+     * @throws WsException
+     */
+    public function procesar($resultado): void
     {
+        $errores = '';
         if (isset($resultado->Errors)) {
             $errores = reset($resultado->Errors)->Msg;
-        } else {
-            //Porque el error viene de otra forma si existe message
-            if (!property_exists($resultado, 'message')) {
-                $errores = isset($resultado->arrayErrores) ?
-                    (isset($resultado->arrayErrores->codigoDescripcion) ?
-                        $resultado->arrayErrores->codigoDescripcion->descripcion
-                        : $resultado->arrayErrores)
-                    : null;
-            } else {
-                $errores = $resultado->getMessage();
+        } elseif (isset($resultado->arrayErrores)) {
+            if (isset($resultado->arrayErrores->codigoDescripcion->descripcion)) {
+                $resultado->arrayErrores->codigoDescripcion = [$resultado->arrayErrores->codigoDescripcion];
+            }
+            foreach ($resultado->arrayErrores->codigoDescripcion as $err) {
+                $errores .= $err->descripcion . ' (' . $err->codigo . ') ';
             }
+        } elseif ($resultado instanceof \SoapFault) {
+            $errores = $resultado->getMessage();
         }
 
-        if (!empty($errores)) {
-            throw new WsException($errores);
+        if (empty($errores)) {
+            return;
         }
+
+        throw new WsException($errores);
     }
 }

src/Exceptions/ManejadorResultados.php

@@ -10,15 +10,16 @@
 
 namespace Multinexo\Models;
 
+use Multinexo\Objects\InvoiceResultObject;
 use stdClass;
 
 interface InvoiceInterface
 {
-    public function createInvoice(): stdClass;
+    public function createInvoice(): InvoiceResultObject;
 
-    public function getCAEA(): stdClass;
+    public function getCAEA(stdClass $data): stdClass;
 
-    public function requestCAEA(): stdClass;
+    public function requestCAEA(stdClass $datos): stdClass;
 
     public function getInvoice(): stdClass;
 }

src/Models/InvoiceInterface.php

@@ -15,7 +15,7 @@
 use Multinexo\Exceptions\ManejadorResultados;
 use Multinexo\Objects\InvoiceObject;
 
-abstract class Invoice implements InvoiceInterface, AfipWebServiceInterface
+abstract class InvoiceWebService implements InvoiceInterface, AfipWebServiceInterface
 {
     /** @var Authentication */
     public $service;

src/Models/Invoice.php renamed to src/Models/InvoiceWebService.php

@@ -11,6 +11,8 @@
 namespace Multinexo\Models;
 
 use Multinexo\Exceptions\ValidationException;
+use Multinexo\Objects\InvoiceObject;
+use Multinexo\Objects\WebServiceEnum;
 use Multinexo\WSMTXCA\Wsmtxca;
 use Multinexo\WSMTXCA\WsParametros;
 use Respect\Validation\Exceptions\NestedValidationException;
@@ -104,15 +106,13 @@ private function getRulesForElectronicInvoice(): array
     {
         $wsReglas = [];
         $codComprobantes = $this->codComprobantes();
-        $puntosVenta = $this->getAvailablePosNumbers();
         $codDocumento = $this->codDocumento();
         $codMonedas = $this->codMonedas();
 
         $reglasFeGenerales = [
             'periodo' => v::notEmpty()->date('Ym'),
             'orden' => v::notEmpty()->intVal()->between(1, 2)->length(1, 1),
             'codigoComprobante' => v::in($codComprobantes),
-            'puntoVenta' => v::in($puntosVenta), // fe s/item?
             'cantidadRegistros' => v::notEmpty()->intVal()->between(1, 9999),
             'codigoConcepto' => v::in(['1', '2', '3']),
             'codigoDocumento' => v::in($codDocumento),
@@ -125,43 +125,35 @@ private function getRulesForElectronicInvoice(): array
             'importeIVA' => v::floatVal()->between(0, 9999999999999.99),
             'importeTotal' => v::floatVal()->between(0, 9999999999999.99),
             'caea' => v::intVal()->length(14, 14),
+            'cotizacionMoneda' => v::notEmpty()->between(0, 9999999999.999999),
+            'numeroComprobante' => v::optional(v::notEmpty()->intVal()->length(1, 8)),
+            'fechaEmision' => v::optional(v::date('Y-m-d')),
+            'fechaServicioDesde' => v::optional(v::date('Y-m-d')),
+            'fechaServicioHasta' => v::optional(v::date('Y-m-d')),
+            'fechaVencimientoPago' => v::optional(v::date('Y-m-d')),
+            'comprobantesAsociados' => v::optional(v::arrayType()),
+            'importeOtrosTributos' => v::optional(v::floatVal()->between(0, 9999999999999.99)),
+            'arrayOtrosTributos' => v::optional(v::arrayType()),
+            'arraySubtotalesIVA' => v::optional(v::objectType()),
         ];
-
-        if ($this->ws === 'wsfe') {
-            $wsReglas = [
-                'cotizacionMoneda' => v::notEmpty()->between(0, 9999999999.999999),
-                'numeroComprobante' => v::optional(v::notEmpty()->intVal()->length(1, 8)),
-                'fechaEmision' => v::optional(v::date('Ymd')),
-                'fechaServicioDesde' => v::optional(v::date('Ymd')),
-                'fechaServicioHasta' => v::optional(v::date('Ymd')),
-                'fechaVencimientoPago' => v::optional(v::date('Ymd')),
-                'arrayComprobantesAsociados' => v::optional(v::objectType()),
-                'arrayOtrosTributos' => v::optional(v::objectType()),
-                'arraySubtotalesIVA' => v::optional(v::objectType()),
+        if ($this->ws === WebServiceEnum::WSFE) {
+            $wsReglasPlus = [
+                'puntoVenta' => v::in($this->getAvailablePosNumbers()),
                 'arrayOpcionales' => v::optional(v::objectType()),
-                'importeOtrosTributos' => v::optional(v::floatVal()->between(0, 9999999999999.99)),
             ];
-        } elseif ($this->ws === 'wsmtxca') {
-            $wsReglas = [
-                'cotizacionMoneda' => v::notEmpty()->between(0, 9999.999999),
-                'numeroComprobante' => v::optional(v::notEmpty()->intVal()->length(1, 8)),
-                'fechaEmision' => v::optional(v::date('Y-m-d')),
-                'fechaServicioDesde' => v::optional(v::date('Y-m-d')),
-                'fechaServicioHasta' => v::optional(v::date('Y-m-d')),
-                'fechaVencimientoPago' => v::optional(v::date('Y-m-d')),
+        } elseif ($this->ws === WebServiceEnum::WSMTXCA) {
+            $wsReglasPlus = [
+                // 'puntoVenta' => v::in($this->getAvailablePosNumbers()), not working on testing environment
+                'puntoVenta' => v::notEmpty()->intVal()->between(1, 9999)->length(1, 4),
                 'codigoTipoAutorizacion' => v::optional(v::in(['A', 'E'])),
                 'observaciones' => v::optional(v::stringType()->length(0, 2000)),
-                'importeOtrosTributos' => v::optional(v::floatVal()->between(0, 9999999999999.99)),
                 'arrayItems' => v::notEmpty()->objectType(),
-                'arraySubtotalesIVA' => v::optional(v::objectType()),
-                'arrayComprobantesAsociados' => v::optional(v::objectType()),
-                'arrayOtrosTributos' => v::optional(v::objectType()),
                 'fechaDesde' => v::optional(v::date('Y-m-d')),
                 'fechaHasta' => v::optional(v::date('Y-m-d')),
             ];
         }
 
-        return array_merge($reglasFeGenerales, $wsReglas);
+        return array_merge($reglasFeGenerales, $wsReglas, $wsReglasPlus ?? []);
     }
 
     // Devuelve mensajes de error personalizados.
@@ -194,12 +186,8 @@ private function getErrorMessages(): array
     // Valida los datos de un array.
     private function validarDatosArray(array $array, string $regla): void
     {
-        if (empty($array)) {
-            return;
-        }
-
         // TODO: Los items tienen que ir si o si en el caso de fe wsmtxca
-        foreach (reset($array) as $dato) {
+        foreach ($array as $dato) {
             $this->validarDatos($dato, $this->getRules($regla));
         }
     }
@@ -209,30 +197,30 @@ public function validateDataInvoice(): void
     {
         $this->validarDatos($this->datos, $this->getRules('fe'));
 
-        if (property_exists($this->datos, 'arrayOtrosTributos')) {
-            $this->validarDatosArray((array) $this->datos->arrayOtrosTributos, 'tributos');
-        }
+        $this->validarDatosArray($this->datos->arrayOtrosTributos, 'tributos');
 
         /** @phpstan-ignore-next-line  */
         if (property_exists($this->datos, 'arraySubtotalesIVA')) {
             $this->validarDatosArray((array) $this->datos->arraySubtotalesIVA, 'iva');
         }
 
-        if (property_exists($this->datos, 'arrayComprobantesAsociados')) {
-            $this->validarDatosArray((array) $this->datos->arrayComprobantesAsociados, 'comprobantesAsociados');
-        }
+        $this->validarDatosArray($this->datos->comprobantesAsociados, 'comprobantesAsociados');
 
         if ($this->ws === 'wsfe') {
             if (property_exists($this->datos, 'arrayOpcionales')) {
                 $this->validarDatosArray((array) $this->datos->arrayOpcionales, 'opcionales');
             }
         } elseif ($this->ws === 'wsmtxca') {
-            $this->validarDatosArray((array) $this->datos->arrayItems, 'items');
+            $this->validarDatosArray($this->datos->items, 'items');
         }
     }
 
     // Valida que los datos ingresados cumplan con determinadas reglas.
-    public function validarDatos(stdClass $datos, stdClass $reglas): void
+
+    /**
+     * @param InvoiceObject|stdClass $datos
+     */
+    public function validarDatos($datos, stdClass $reglas): void
     {
         $validaciones = [];