I removed all namespaces and fix extractor structure

onefacture · Aug 30, 2021 · 9546e6d · 9546e6d
1 parent db9062f
commit 9546e6d
Show file tree

Hide file tree

Showing 22 changed files with 379 additions and 672 deletions.
diff --git a/.gitignore b/.gitignore
@@ -3,3 +3,4 @@ dist
 *cache
 *log
 *.DS_Store
+test.js
diff --git a/.npmignore b/.npmignore
@@ -3,3 +3,4 @@ node_modules
 *log
 *.DS_Store
 src
+test.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "cfdi-to-json",
-  "version": "1.2.5",
+  "version": "1.3.1",
   "description": "",
   "main": "dist/index.js",
   "scripts": {
@@ -19,8 +19,8 @@
   "homepage": "https://github.com/onefacture/cfdi-to-json#readme",
   "dependencies": {
     "he": "1.2.0",
-    "xmldom": "0.1.27",
-    "xpath": "0.0.27"
+    "xmldom": "0.6.0",
+    "xpath": "0.0.32"
   },
   "devDependencies": {
     "@types/he": "1.1.0",

diff --git a/src/CfdiExtractData/__test__/index-spec.ts b/src/CfdiExtractData/__test__/index-spec.ts
@@ -23,14 +23,14 @@ describe('CfdiExtractData', () => {
                 Version="3.3"
                 xmlns:cfdi="http://www.sat.gob.mx/cfd/3">
                     <cfdi:Emisor Rfc="RFC_EMISOR" Nombre="NOMBRE_DEL_EMISOR" RegimenFiscal="612"></cfdi:Emisor>
-                    <cfdi:Receptor Rfc="RFC_RECEPTOR" Nombre="NOMBRE_DEL_RECEPTOR" UsoCFDI="G03"></cfdi:Receptor>
+                    <Receptor Rfc="RFC_RECEPTOR" Nombre="NOMBRE_DEL_RECEPTOR" UsoCFDI="G03"></Receptor>
                     <cfdi:Conceptos>
                         <cfdi:Concepto ClaveProdServ="43231601" Cantidad="1" ClaveUnidad="E48" Unidad="Unidad de servicio" Descripcion="Licencia anual onefacture cfdi" ValorUnitario="430.17" Importe="430.17">
-                            <cfdi:Impuestos>
+                            <Impuestos>
                                 <cfdi:Traslados>
                                     <cfdi:Traslado Base="430.17" Impuesto="002" TipoFactor="Tasa" TasaOCuota="0.160000" Importe="68.83"></cfdi:Traslado>
                                 </cfdi:Traslados>
-                            </cfdi:Impuestos>
+                            </Impuestos>
                         </cfdi:Concepto>
                     </cfdi:Conceptos>
                     <cfdi:Impuestos TotalImpuestosTrasladados="68.83">
@@ -92,9 +92,9 @@ describe('CfdiExtractData', () => {
             "fecha": "2019-04-02T10:41:20",
             "formaPago": "04",
             "impuestos":  {
-                "totalImpuestosTrasladados": "68.83",
+                "totalImpuestosTrasladados": 68.83,
                 "traslados": [{
-                    "importe": "68.83",
+                    "importe": 68.83,
                     "impuesto": "002",
                     "tasaOCuota": "0.160000",
                     "tipoFactor": "Tasa",
@@ -207,9 +207,9 @@ describe('CfdiExtractData', () => {
             "fecha": "2019-04-02T10:41:20",
             "formaPago": "04",
             "impuestos":  {
-                "totalImpuestosTrasladados": "68.83",
+                "totalImpuestosTrasladados": 68.83,
                 "traslados": [{
-                    "importe": "68.83",
+                    "importe": 68.83,
                     "impuesto": "002",
                     "tasaOCuota": "0.160000",
                     "tipoFactor": "Tasa",
@@ -258,6 +258,32 @@ describe('CfdiExtractData', () => {
         });
     });
 
+    it('Basic data without namespace cfdi in root tag', () => {
+        expect(CfdiExtractData.extractGeneralData({
+            contentXML: `
+                <?xml version="1.0" encoding="utf-8"?>
+                <Comprobante
+                    xmlns:cfdi="http://www.sat.gob.mx/cfd/3"
+                    Version="3.3"
+                >
+                    <cfdi:Emisor
+                        Rfc="RFC_EMISOR"
+                        Nombre="NOMBRE_DEL_EMISOR"
+                        RegimenFiscal="612"
+                    ></cfdi:Emisor>
+                </Comprobante>
+            `
+        }))
+        .toEqual({
+            "version": "3.3",
+            "emisor":  {
+                "nombre": "NOMBRE_DEL_EMISOR",
+                "regimenFiscal": "612",
+                "rfc": "RFC_EMISOR"
+            }
+        });
+    });
+
     it('Receptor', () => {
         expect(CfdiExtractData.extractGeneralData({
             contentXML: `
@@ -361,9 +387,9 @@ describe('CfdiExtractData', () => {
         .toEqual({
             "version": "3.3",
             "impuestos":  {
-                "totalImpuestosTrasladados": "68.83",
+                "totalImpuestosTrasladados": 68.83,
                 "traslados": [{
-                    "importe": "68.83",
+                    "importe": 68.83,
                     "impuesto": "002",
                     "tasaOCuota": "0.160000",
                     "tipoFactor": "Tasa"

diff --git a/src/CfdiExtractData/index.d.ts b/src/CfdiExtractData/index.d.ts
@@ -18,7 +18,3 @@ export interface tImpuesto {
 export interface tMinimalData {
 	minimalData: Boolean;
 }
-
-export interface tNamespace {
-    namespace?: any;
-}
-Original file line number
+Diff line change
@@ Expand Up / @@ -3,3 +3,4 @@ dist @@
     *cache
     *log
     *.DS_Store
+    test.js