Merge pull request #64 from ircsasw/FormaPago_Nomina

FormaPago para Nómina

Author: eclipxe13

docs/CHANGELOG.md

@@ -28,6 +28,13 @@
 - Change visibility of `CfdiUtils\Cleaner\Cleaner#removeIncompleteSchemaLocation()` to private.
 
 
+## Version 2.14.2 2021-03-16
+
+### `FormaPago` on `N - Nómina`
+
+- Validation `TIPOCOMP03` does not apply on documents type `N - Nómina`.
+
+
 ## Version 2.14.1 2021-03-14
 
 ### `MetodoPago` on `N - Nómina`

docs/validar/validaciones-estandar.md

@@ -58,7 +58,7 @@ Realiza diferentes validaciones relacionadas con el tipo de comprobante:
 
 - TIPOCOMP01: Si el tipo de comprobante es T, P ó N, entonces no debe existir las condiciones de pago
 - TIPOCOMP02: Si el tipo de comprobante es T, P ó N, entonces no debe existir la definición de impuestos (CFDI33179)
-- TIPOCOMP03: Si el tipo de comprobante es T, P ó N, entonces no debe existir la forma de pago
+- TIPOCOMP03: Si el tipo de comprobante es T ó P, entonces no debe existir la forma de pago
 - TIPOCOMP04: Si el tipo de comprobante es T ó P, entonces no debe existir el método de pago (CFDI33123)
 - TIPOCOMP05: Si el tipo de comprobante es T ó P, entonces no debe existir el descuento del comprobante (CFDI33110)
 - TIPOCOMP06: Si el tipo de comprobante es T ó P, entonces no debe existir el descuento de los conceptos (CFDI33179)

src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobante.php

@@ -13,7 +13,7 @@
  * Valida que:
  * - TIPOCOMP01: Si el tipo de comprobante es T, P ó N, entonces no debe existir las condiciones de pago
  * - TIPOCOMP02: Si el tipo de comprobante es T, P ó N, entonces no debe existir la definición de impuestos (CFDI33179)
- * - TIPOCOMP03: Si el tipo de comprobante es T, P ó N, entonces no debe existir la forma de pago
+ * - TIPOCOMP03: Si el tipo de comprobante es T ó P, entonces no debe existir la forma de pago
  * - TIPOCOMP04: Si el tipo de comprobante es T ó P, entonces no debe existir el método de pago (CFDI33123)
  * - TIPOCOMP05: Si el tipo de comprobante es T ó P, entonces no debe existir el descuento del comprobante (CFDI33110)
  * - TIPOCOMP06: Si el tipo de comprobante es T ó P, entonces no debe existir el descuento de los conceptos (CFDI33179)
@@ -32,7 +32,7 @@ private function registerAsserts(Asserts $asserts)
                          . ' entonces no debe existir las condiciones de pago',
             'TIPOCOMP02' => 'Si el tipo de comprobante es T, P ó N,'
                          . ' entonces no debe existir la definición de impuestos (CFDI33179)',
-            'TIPOCOMP03' => 'Si el tipo de comprobante es T, P ó N, entonces no debe existir la forma de pago',
+            'TIPOCOMP03' => 'Si el tipo de comprobante es T ó P, entonces no debe existir la forma de pago',
 
             'TIPOCOMP04' => 'Si el tipo de comprobante es T ó P,'
                          . ' entonces no debe existir el método de pago (CFDI33123)',
@@ -68,13 +68,13 @@ public function validate(NodeInterface $comprobante, Asserts $asserts)
                 'TIPOCOMP02',
                 Status::when(null === $comprobante->searchNode('cfdi:Impuestos'))
             );
+        }
+
+        if ('T' === $tipoComprobante || 'P' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP03',
                 Status::when(! $comprobante->offsetExists('FormaPago'))
             );
-        }
-
-        if ('T' === $tipoComprobante || 'P' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP04',
                 Status::when(! $comprobante->offsetExists('MetodoPago'))
@@ -96,12 +96,14 @@ public function validate(NodeInterface $comprobante, Asserts $asserts)
                 Status::when($this->isZero($comprobante['Total']))
             );
         }
+
         if ('I' === $tipoComprobante || 'E' === $tipoComprobante || 'N' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP09',
                 Status::when($this->checkConceptosValorUnitarioIsGreaterThanZero($comprobante))
             );
         }
+
         if ('N' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP10',

tests/CfdiUtilsTests/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobanteTest.php

@@ -35,7 +35,6 @@ public function testValidTPN($tipoDeComprobante)
         $this->runValidate();
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP01');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP02');
-        $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP03');
     }
 
     /**
@@ -54,7 +53,6 @@ public function testInvalidTPN($tipoDeComprobante)
         $this->runValidate();
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP01');
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP02');
-        $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP03');
     }
 
     public function providerTP()
@@ -70,10 +68,13 @@ public function testValidTP($tipoDeComprobante)
     {
         $this->comprobante->addAttributes([
             'TipoDeComprobante' => $tipoDeComprobante,
+            'FormaPago' => null, // set to null to make clear that it must not exists
+            'MetodoPago' => null, // set to null to make clear that it must not exists
             'SubTotal' => '0',
             'Total' => '0.00',
         ]);
         $this->runValidate();
+        $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP03');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP04');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP05');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP06');
@@ -85,13 +86,15 @@ public function testValidTP($tipoDeComprobante)
      * @param string $tipoDeComprobante
      * @dataProvider providerTP
      */
-    public function testInvalidTPMetodoPago($tipoDeComprobante)
+    public function testInvalidTP($tipoDeComprobante)
     {
         $this->comprobante->addAttributes([
             'TipoDeComprobante' => $tipoDeComprobante,
+            'FormaPago' => '',
             'MetodoPago' => '',
         ]);
         $this->runValidate();
+        $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP03');
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP04');
     }