From 66458a03a31f1253d3942de2b5f95135e0bcaeec Mon Sep 17 00:00:00 2001 From: David Date: Mon, 27 Nov 2017 14:25:36 -0600 Subject: [PATCH 1/4] Improved description for messages and fixed typo --- .../Cfdi33/Standard/ComprobanteMetodoPago.php | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php index 5ac49f1d..29ffbfd8 100644 --- a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php +++ b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php @@ -10,18 +10,20 @@ * ComprobanteMetodoPago * * Valida que: - * - METPAG01: Si el tipo de documento es T, P ó N, el metodo de pago no debe existir(CFDI33123, CFDI33124) - * - METPAG02: Si el tipo de documento es I ó E, el metodo de pago debe ser "PUE" o "PPD (CFDI33121, CFDI33122) + * - 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) */ class ComprobanteMetodoPago extends AbstractDiscoverableVersion33 { private function registerAsserts(Asserts $asserts) { $assertDescriptions = [ - 'METPAG01' => 'Si el tipo de documento es T, P ó N, el metodo 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, el metodo de pago' - . 'debe ser "PUE" o "PPD" (CFDI33121, CFDI33122)', + 'METPAG02' => 'Si el tipo de documento es I ó E, entonces el metodo de pago' + . ' debe ser "PUE" o "PPD" (CFDI33121, CFDI33122)', ]; foreach ($assertDescriptions as $code => $title) { $asserts->put($code, $title); From 69151f12ab53190695910e5773091c8c5c87c9da Mon Sep 17 00:00:00 2001 From: David Date: Mon, 27 Nov 2017 14:31:25 -0600 Subject: [PATCH 2/4] Added missing space --- .../Validate/Cfdi33/Standard/ComprobanteMetodoPago.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php index 29ffbfd8..3a16d204 100644 --- a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php +++ b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteMetodoPago.php @@ -23,7 +23,7 @@ private function registerAsserts(Asserts $asserts) '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)', + . ' debe ser "PUE" o "PPD" (CFDI33121, CFDI33122)', ]; foreach ($assertDescriptions as $code => $title) { $asserts->put($code, $title); From bacfbfddc0ab187a3ee79100f7f86eba6f5fd443 Mon Sep 17 00:00:00 2001 From: David Date: Mon, 27 Nov 2017 14:38:24 -0600 Subject: [PATCH 3/4] Fixed typos and improved descriptions and indents --- .../Cfdi33/Standard/ComprobanteTipoCambio.php | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoCambio.php b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoCambio.php index 38f0ec5f..36c27c6f 100644 --- a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoCambio.php +++ b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoCambio.php @@ -11,9 +11,9 @@ * * Valida que: * - TIPOCAMBIO01: La moneda exista y no tenga un valor vacío - * - TIPOCAMBIO02: Si la moneda es MXN entonces el tipo de cambio debe ser "1" o no debe existir (CFDI33113) - * - TIPOCAMBIO03: Si la moneda es XXX entonces el tipo de cambio no debe existir (CFDI33115) - * - TIPOCAMBIO04: Si la moneda no es MXN ni XXX entonces el tipo de cambio entonces + * - TIPOCAMBIO02: Si la moneda es "MXN", entonces el tipo de cambio debe ser "1" o no debe existir (CFDI33113) + * - TIPOCAMBIO03: Si la moneda es "XXX", entonces el tipo de cambio no debe existir (CFDI33115) + * - TIPOCAMBIO04: Si la moneda no es "MXN" ni "XXX", entonces el tipo de cambio entonces * el tipo de cambio debe seguir el patrón [0-9]{1,18}(.[0-9]{1,6})? (CFDI33114, CFDI33117) */ class ComprobanteTipoCambio extends AbstractDiscoverableVersion33 @@ -22,11 +22,11 @@ private function registerAsserts(Asserts $asserts) { $assertDescriptions = [ 'TIPOCAMBIO01' => 'La moneda exista y no tenga un valor vacío', - 'TIPOCAMBIO02' => 'Si la moneda es MXN entonces el tipo de cambio debe ser "1"' - . ' o no debe existir (CFDI33113)', - 'TIPOCAMBIO03' => 'Si la moneda es XXX entonces el tipo de cambio no debe existir (CFDI33115)', - 'TIPOCAMBIO04' => 'Si la moneda no es MXN ni XXX entonces el tipo de cambio' - . ' debe seguir el patrón [0-9]{1,18}(.[0-9]{1,6}?) (CFDI33114, CFDI33117)', + 'TIPOCAMBIO02' => 'Si la moneda es "MXN", entonces el tipo de cambio debe ser "1"' + . ' o no debe existir (CFDI33113)', + 'TIPOCAMBIO03' => 'Si la moneda es "XXX", entonces el tipo de cambio no debe existir (CFDI33115)', + 'TIPOCAMBIO04' => 'Si la moneda no es "MXN" ni "XXX", entonces el tipo de cambio' + . ' debe seguir el patrón [0-9]{1,18}(.[0-9]{1,6}?) (CFDI33114, CFDI33117)', ]; foreach ($assertDescriptions as $code => $title) { $asserts->put($code, $title); From fbcd0254ee8635dfe9caa71aa5a5918465324c0b Mon Sep 17 00:00:00 2001 From: David Date: Mon, 27 Nov 2017 15:16:50 -0600 Subject: [PATCH 4/4] Added CFDI codes, minor changes on message descriptions --- .../Standard/ComprobanteTipoDeComprobante.php | 45 ++++++++++--------- 1 file changed, 25 insertions(+), 20 deletions(-) diff --git a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobante.php b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobante.php index f18ab41d..bef854b5 100644 --- a/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobante.php +++ b/src/CfdiUtils/Validate/Cfdi33/Standard/ComprobanteTipoDeComprobante.php @@ -10,37 +10,42 @@ * ConceptoValorUnitario.php * * 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 - * - 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 - * - TIPOCOMP05: Si el tipo de comprobante es T ó P entonces no debe existir el descuento del comprobante - * - TIPOCOMP06: Si el tipo de comprobante es T ó P entonces no debe existir el descuento de los conceptos - * - TIPOCOMP07: Si el tipo de comprobante es T ó P entonces el subtotal debe ser cero - * - TIPOCOMP08: Si el tipo de comprobante es T ó P entonces el total debe ser cero - * - TIPOCOMP09: Si el tipo de comprobante es I, E ó N entonces el valor unitario de todos los conceptos + * - 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) + * - 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) + * - TIPOCOMP08: Si el tipo de comprobante es T ó P, entonces el total debe ser cero + * - TIPOCOMP09: Si el tipo de comprobante es I, E ó N, entonces el valor unitario de todos los conceptos * debe ser mayor que cero - * - TIPOCOMP010: Si el tipo de comprobante es N entonces la moneda debe ser MXN + * - TIPOCOMP010: Si el tipo de comprobante es N, entonces la moneda debe ser MXN */ class ComprobanteTipoDeComprobante extends AbstractDiscoverableVersion33 { private function registerAsserts(Asserts $asserts) { $assertsDescriptions = [ - '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', - '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', + '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)', - 'TIPOCOMP05' => 'Si el tipo de comprobante es T ó P entonces no debe existir el descuento del comprobante', - 'TIPOCOMP06' => 'Si el tipo de comprobante es T ó P entonces no debe existir el descuento de los conceptos', - 'TIPOCOMP07' => 'Si el tipo de comprobante es T ó P entonces el subtotal debe ser cero', + '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)', 'TIPOCOMP08' => 'Si el tipo de comprobante es T ó P entonces el total debe ser cero', - 'TIPOCOMP09' => 'Si el tipo de comprobante es I, E ó N' - . ' entonces el valor unitario de todos los conceptos debe ser mayor que cero', + 'TIPOCOMP09' => 'Si el tipo de comprobante es I, E ó N,' + . ' entonces el valor unitario de todos los conceptos debe ser mayor que cero', - 'TIPOCOMP10' => 'Si el tipo de comprobante es N entonces la moneda debe ser MXN', + 'TIPOCOMP10' => 'Si el tipo de comprobante es N entonces, la moneda debe ser MXN', ]; foreach ($assertsDescriptions as $code => $title) { $asserts->put($code, $title);