Merge pull request #65 from TheSpectroMx/master

Actualización en el MetadataExtractor
phpcfdi · Aug 6, 2022 · e628dda · e628dda
2 parents 0f94918 + d6c42b2
commit e628dda
Show file tree

Hide file tree

Showing 9 changed files with 59 additions and 0 deletions.
diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -8,6 +8,13 @@ Usamos [Versionado Semántico 2.0.0](SEMVER.md) por lo que puedes usar esta libr
 
 No hay cambios no liberados, si existen, deben aparecer aquí.
 
+## Versión 3.2.0
+
+### Agregar *Motivo de cancelación* y *Folio de sustitución*
+
+Se agrega la lectura de *Motivo de cancelación* (`motivoCancelacion`) y *Folio de sustitución* (`folioSustitucion`) a `Metadata`. Así como la extracción de estos datos en `MetadataExtractor`.
+Gracias `@TheSpectroMx`.
+
 ## Versión 3.1.2
 
 ### Filtrado de recursos incorrecto

diff --git a/src/Internal/MetadataExtractor.php b/src/Internal/MetadataExtractor.php
@@ -82,6 +82,8 @@ public function defaultFieldsCaptions(): array
             'estatusProcesoCancelacion' => 'Estatus de Proceso de Cancelación',
             'fechaProcesoCancelacion' => 'Fecha de Proceso de Cancelación',
             'rfcACuentaTerceros' => 'RFC a cuenta de terceros',
+            'motivoCancelacion' => 'Motivo',
+            'folioSustitucion' => 'Folio de Sustitución',
         ];
     }
 

diff --git a/src/Metadata.php b/src/Metadata.php
@@ -33,6 +33,8 @@
  * @property-read string $estatusProcesoCancelacion Estatus de Proceso de Cancelación
  * @property-read string $fechaProcesoCancelacion Fecha de Proceso de Cancelación
  * @property-read string $rfcACuentaTerceros RFC a cuenta de terceros
+ * @property-read string $motivoCancelacion Motivo de cancelación
+ * @property-read string $folioSustitucion UUID del CFDI con el que es sustituido
  *
  * @see MetadataExtractor::defaultFieldsCaptions()
  * @implements IteratorAggregate<string, string>

diff --git a/tests/Unit/Internal/MetadataExtractorTest.php b/tests/Unit/Internal/MetadataExtractorTest.php
@@ -183,6 +183,8 @@ public function testUsingFakeInput(): void
             'estatusProcesoCancelacion' => '',
             'fechaProcesoCancelacion' => '',
             'rfcACuentaTerceros' => 'AAA991231AAA',
+            'motivoCancelacion' => '',
+            'folioSustitucion' => '',
             'urlXml' => '', // the data is not included on test file
             'urlPdf' => '',
             'urlCancelRequest' => '',
@@ -220,6 +222,8 @@ public function testExtractUsingSampleWithOneUuid(): void
                 'estatusProcesoCancelacion' => '',
                 'fechaProcesoCancelacion' => '',
                 'rfcACuentaTerceros' => 'ABC010101AAA',
+                'motivoCancelacion' => '',
+                'folioSustitucion' => '',
             ],
         ];
 

diff --git a/tests/_files/fake-to-extract-metadata-missing-uuid.html b/tests/_files/fake-to-extract-metadata-missing-uuid.html
@@ -18,6 +18,8 @@
                 <th><span>Estatus de Proceso de Cancelación</span></th>
                 <th><span>Fecha de Proceso de Cancelación</span></th>
                 <th><span>RFC a cuenta de terceros</span></th>
+                <th><span>Motivo</span></th>
+                <th><span>Folio de Sustitución</span></th>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -36,6 +38,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -54,6 +58,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -72,6 +78,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
         </table>
     </body>

diff --git a/tests/_files/fake-to-extract-metadata-one-cfdi.html b/tests/_files/fake-to-extract-metadata-one-cfdi.html
@@ -18,6 +18,8 @@
                 <th><span>Estatus de Proceso de Cancelación</span></th>
                 <th><span>Fecha de Proceso de Cancelación</span></th>
                 <th><span>RFC a cuenta de terceros</span></th>
+                <th><span>Motivo</span></th>
+                <th><span>Folio de Sustitución</span></th>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -36,6 +38,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;">AAA991231AAA</span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
         </table>
     </body>

diff --git a/tests/_files/fake-to-extract-metadata-ten-cfdi.html b/tests/_files/fake-to-extract-metadata-ten-cfdi.html
@@ -18,6 +18,8 @@
                 <th><span>Estatus de Proceso de Cancelación</span></th>
                 <th><span>Fecha de Proceso de Cancelación</span></th>
                 <th><span>RFC a cuenta de terceros</span></th>
+                <th><span>Motivo</span></th>
+                <th><span>Folio de Sustitución</span></th>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -36,6 +38,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -54,6 +58,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -72,6 +78,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -90,6 +98,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -108,6 +118,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -126,6 +138,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -144,6 +158,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -161,6 +177,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;">Vigente</span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -179,6 +197,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
             <tr>
                 <td style="width:14.2%;">...</td>
@@ -197,6 +217,8 @@
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
                 <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+                <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
             </tr>
         </table>
     </body>

diff --git a/tests/_files/fake-to-extract-metadata-zero-cfdi.html b/tests/_files/fake-to-extract-metadata-zero-cfdi.html
@@ -18,6 +18,8 @@
                 <th><span>Estatus de Proceso de Cancelación</span></th>
                 <th><span>Fecha de Proceso de Cancelación</span></th>
                 <th><span>RFC a cuenta de terceros</span></th>
+                <th><span>Motivo</span></th>
+                <th><span>Folio de Sustitución</span></th>
             </tr>
         </table>
     </body>

diff --git a/tests/_files/sample-to-extract-metadata-one-cfdi.html b/tests/_files/sample-to-extract-metadata-one-cfdi.html
@@ -758,6 +758,12 @@
 			<th>
                                         <span style="width: 200px;">RFC a cuenta de terceros</span>
                                     </th>
+			<th>
+                                        <span style="width: 200px;">Motivo</span>
+                                    </th>
+			<th>
+                                        <span style="width: 200px;">Folio de Sustitución</span>
+                                    </th>
 		</tr>
 		<tr>
 			<td><div style="width:150px; display:block;text-align:center;vertical-align:middle; position: relative;"><table>
@@ -790,6 +796,8 @@
 			<td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
 			<td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:200px;"> </span></td>
             <td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;">ABC010101AAA</span></td>
+			<td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
+			<td style="WORD-BREAK:BREAK-ALL;"><span style="display:inline-block;width:240px;"></span></td>
         </tr>
         <tr>
             <td>