Merge pull request #63 from eclipxe13/development

Validations on `MetodoPago` on `N - Nómina`

Author: eclipxe13

docs/CHANGELOG.md

@@ -28,20 +28,26 @@
 - Change visibility of `CfdiUtils\Cleaner\Cleaner#removeIncompleteSchemaLocation()` to private.
 
 
-## UNRELEASED
+## Version 2.14.1 2021-03-14
 
-### 2020-10-08
+### `MetodoPago` on `N - Nómina`
+
+- Remove redundant valiations `METPAG01` and `METPAG02`.
+- Validation `TIPOCOMP04` does not apply on documents type `N - Nómina`.
+
+### Download certificates (unreleased 2020-10-08)
 
 - Looks like web service at `https://rdc.sat.gob.mx/` is having issues (again). This is breaking testing.
   To mitigate the problem, a new testing class `CertificateDownloaderHelper` has been created to retry the
   download if it fails, for a maximum of 5 attempts. This change does not create a new release version.
 
-### 2020-12-03
+### Travis-CI build (unreleased 2020-12-03)
 
 - Add `XDEBUG_MODE=coverage` on Travis-CI configuration file since it upgrade to `XDebug v3.0.0` and it contains
   a bug that makes `PHPUnit` break with the message: *Use of undefined constant XDEBUG_CC_UNUSED - assumed 'XDEBUG_CC_UNUSED'*.
   This will be resolved on <https://github.com/xdebug/xdebug/pull/699> but not released (yet).
 
+
 ## Version 2.14.0 2020-10-01
 
 - Add `Retenciones` reader to work with *CFDI de retenciones e información de pagos*.

docs/validar/validaciones-estandar.md

@@ -43,14 +43,6 @@ Valida el nodo impuestos del comprobante
 - COMPIMPUESTOSC03: Si existe al menos una retención entonces debe existir el total de retenciones
 
 
-## ComprobanteMetodoPago
-
-Validaciones específicas relacionadas con el método de pago
-
-- METPAG01: Si el tipo de documento es T, P ó N, entonces el metodo de pago no debe existir (CFDI33123, CFDI33124)
-- METPAG02: Si el tipo de documento es I ó E, entonces el metodo de pago debe ser "PUE" o "PPD" (CFDI33121, CFDI33122)
-
-
 ## ComprobanteTipoCambio
 
 - TIPOCAMBIO01: La moneda exista y no tenga un valor vacío
@@ -67,7 +59,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
-- TIPOCOMP04: Si el tipo de comprobante es T, P ó N, entonces no debe existir el método de pago (CFDI33123)
+- 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)
 - TIPOCOMP07: Si el tipo de comprobante es T ó P, entonces el subtotal debe ser cero (CFDI33108)

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

@@ -14,7 +14,7 @@
  * - 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
- * - TIPOCOMP04: Si el tipo de comprobante es T, P ó N, entonces no debe existir el método de pago (CFDI33123)
+ * - 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)
  * - TIPOCOMP07: Si el tipo de comprobante es T ó P, entonces el subtotal debe ser cero (CFDI33108)
@@ -33,9 +33,9 @@ private function registerAsserts(Asserts $asserts)
             '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',
-            'TIPOCOMP04' => 'Si el tipo de comprobante es T, P ó N,'
-                         . ' entonces no debe existir el método de pago (CFDI33123)',
 
+            '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,'
@@ -72,13 +72,13 @@ public function validate(NodeInterface $comprobante, Asserts $asserts)
                 'TIPOCOMP03',
                 Status::when(! $comprobante->offsetExists('FormaPago'))
             );
+        }
+
+        if ('T' === $tipoComprobante || 'P' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP04',
                 Status::when(! $comprobante->offsetExists('MetodoPago'))
             );
-        }
-
-        if ('T' === $tipoComprobante || 'P' === $tipoComprobante) {
             $asserts->putStatus(
                 'TIPOCOMP05',
                 Status::when(! $comprobante->offsetExists('Descuento'))

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

@@ -36,7 +36,6 @@ public function testValidTPN($tipoDeComprobante)
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP01');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP02');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP03');
-        $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP04');
     }
 
     /**
@@ -56,7 +55,6 @@ public function testInvalidTPN($tipoDeComprobante)
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP01');
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP02');
         $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP03');
-        $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP04');
     }
 
     public function providerTP()
@@ -76,12 +74,27 @@ public function testValidTP($tipoDeComprobante)
             'Total' => '0.00',
         ]);
         $this->runValidate();
+        $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP04');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP05');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP06');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP07');
         $this->assertStatusEqualsCode(Status::ok(), 'TIPOCOMP08');
     }
 
+    /**
+     * @param string $tipoDeComprobante
+     * @dataProvider providerTP
+     */
+    public function testInvalidTPMetodoPago($tipoDeComprobante)
+    {
+        $this->comprobante->addAttributes([
+            'TipoDeComprobante' => $tipoDeComprobante,
+            'MetodoPago' => '',
+        ]);
+        $this->runValidate();
+        $this->assertStatusEqualsCode(Status::error(), 'TIPOCOMP04');
+    }
+
     /**
      * @param string $tipoDeComprobante
      * @dataProvider providerTP