cfdi40 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7901aac708d9a35630373ea7d4f38a43fdddd801defb286068fe7b96350db91c
-  data.tar.gz: 34ffabb2c0a58898656c5a652afdbc2e84d04bc0848c5927b7f673a0ebd5875e
+  metadata.gz: ce28369c399d94875c10fa980ead1eb1cd75e0300035ffeb61c7bc310d87b327
+  data.tar.gz: 8d03b666fb2fac0b592032891fadfec3643f64938eb46b2e301bf4a98e68014a
 SHA512:
-  metadata.gz: 9e04325fa2210081dc36519e8d9773110f386964d45e541b8273fbd7c22112d258ba249f0f04ad68cd8f12039648d466228c1759aad8b1b84a9eb69e23edd3a1
-  data.tar.gz: c1f87bc293e895fc4df9d3440413477f1d540142e879b85e3d13e534a69c1f9376080f41212bf26e3bc161726883c238f625121198e2de7db98aec99b0122bd7
+  metadata.gz: 1246bce889af62b67256d2c1278201bfaee2c6ae5693048b7a00b8ec424830192610a8d98df294f5227b47ed3f19a0556f1f07146b057a0b8f65e3c95a8874f0
+  data.tar.gz: d3142214a86217a324b8539bc3455695a257505265e76dc746fa0999d2a2a1ed3486abd8a0e58ef31f4fdd46d828ae9529adcf7d271ce77499657124c855c395

data/.rubocop.yml

@@ -16,3 +16,6 @@ Style/StringLiteralsInInterpolation:
 
 Layout/LineLength:
   Max: 120
+
+Metrics/MethodLength:
+  Max: 20

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    cfdi40 (0.0.7)
+    cfdi40 (0.0.8)
       nokogiri (>= 1.10.10)
 
 GEM
@@ -10,12 +10,12 @@ GEM
     ast (2.4.2)
     json (2.3.1)
     minitest (5.15.0)
-    nokogiri (1.14.2-x86_64-linux)
+    nokogiri (1.15.7-x86_64-linux)
       racc (~> 1.4)
     parallel (1.20.1)
     parser (3.1.2.1)
       ast (~> 2.4.1)
-    racc (1.6.2)
+    racc (1.8.1)
     rainbow (3.0.0)
     rake (13.0.6)
     regexp_parser (1.8.2)

data/README.md

@@ -49,6 +49,16 @@ install`. To release a new version, update the version number in
 a git tag for the version, push git commits and the created tag, and
 push the `.gem` file to [rubygems.org](https://rubygems.org).
 
+## Testing
+
+Run all test
+
+    bundle exec rake test
+
+Run all test in a file
+
+    bundle exec ruby -Ilib:test test/test_cfdi40.rb
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at

data/README_es-MX.md

@@ -99,6 +99,14 @@ Hasta ahora:
     # Genera CFDI firmado
     xml_string = cfdi.to_xml
 
+# Cargar un CFDI a partir de un XML:
+
+    # cfdi.xml es un archivo con un CFDi versión 4.0
+    xml_string = File.read('cfdi.xml')
+    cfdi = Cfdi40.open(xml_string)
+
+Una vez cargado el xml se pueden leer los atributos
+y/o hacer modificaciones
 
 # Lo que sigue
 
@@ -108,6 +116,9 @@ Hasta ahora:
 
 # Cambios
 
+# 0.0.8
+* Carga básica de un CFDI desde XML.
+
 # 0.0.7
 * Ajustes al CFDi con complemento de pagos por validaciones del PAC.
 

data/lib/cfdi40/complemento.rb

@@ -1,14 +1,16 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Complemento < Node
     # See Comprobante#add_pago
-    def add_pago(attributes={})
+    def add_pago(attributes = {})
       pagos.totales_node
       pagos.add_pago(attributes)
     end
 
     def pagos
       return @pagos if defined?(@pagos)
-      
+
       @pagos = Pagos.new
       @pagos.parent_node = self
       @children_nodes << @pagos

data/lib/cfdi40/complemento_concepto.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class ComplementoConcepto < Node
     def inst_educativas_node
@@ -5,7 +7,7 @@ module Cfdi40
 
       @inst_educativas_node = InstEducativas.new
       @inst_educativas_node.parent_node = self
-      self.children_nodes << @inst_educativas_node
+      children_nodes << @inst_educativas_node
       @inst_educativas_node
     end
   end

data/lib/cfdi40/comprobante.rb

@@ -6,32 +6,31 @@ module Cfdi40
     define_namespace "xsi", "http://www.w3.org/2001/XMLSchema-instance"
     define_namespace "cfdi", "http://www.sat.gob.mx/cfd/4"
     define_attribute :schema_location,
-                     xml_attribute: 'xsi:schemaLocation',
+                     xml_attribute: "xsi:schemaLocation",
                      readonly: true,
                      default: "http://www.sat.gob.mx/cfd/4 " \
                               "http://www.sat.gob.mx/sitio_internet/cfd/4/cfdv40.xsd"
-    define_attribute :version, xml_attribute: 'Version', readonly: true, default: '4.0'
-    define_attribute :serie, xml_attribute: 'Serie'
-    define_attribute :folio, xml_attribute: 'Folio'
-    define_attribute :fecha, xml_attribute: 'Fecha'
-    define_attribute :sello, xml_attribute: 'Sello', readonly: true
-    define_attribute :forma_pago, xml_attribute: 'FormaPago'
-    define_attribute :no_certificado, xml_attribute: 'NoCertificado'
-    define_attribute :certificado, xml_attribute: 'Certificado'
-    define_attribute :condiciones_de_pago, xml_attribute: 'CondicionesDePago'
-    define_attribute :subtotal, xml_attribute: 'SubTotal', format: :t_ImporteMXN
-    define_attribute :descuento, xml_attribute: 'Descuento', format: :t_ImporteMXN
-    define_attribute :moneda, xml_attribute: 'Moneda', default: 'MXN'
-    define_attribute :tipo_cambio, xml_attribute: 'TipoCambio'
-    define_attribute :total, xml_attribute: 'Total', format: :t_ImporteMXN
-    define_attribute :tipo_de_comprobante, xml_attribute: 'TipoDeComprobante', default: 'I'
-    define_attribute :exportacion, xml_attribute: 'Exportacion', default: '01'
-    define_attribute :metodo_pago, xml_attribute: 'MetodoPago'
-    define_attribute :lugar_expedicion, xml_attribute: 'LugarExpedicion'
-    define_attribute :confirmacion, xml_attribute: 'Confirmacion'
-
-    attr_reader :emisor, :receptor, :x509_cert, :conceptos, :private_key
-    attr_reader :errors
+    define_attribute :version, xml_attribute: "Version", readonly: true, default: "4.0"
+    define_attribute :serie, xml_attribute: "Serie"
+    define_attribute :folio, xml_attribute: "Folio"
+    define_attribute :fecha, xml_attribute: "Fecha"
+    define_attribute :sello, xml_attribute: "Sello", readonly: true
+    define_attribute :forma_pago, xml_attribute: "FormaPago"
+    define_attribute :no_certificado, xml_attribute: "NoCertificado"
+    define_attribute :certificado, xml_attribute: "Certificado"
+    define_attribute :condiciones_de_pago, xml_attribute: "CondicionesDePago"
+    define_attribute :subtotal, xml_attribute: "SubTotal", format: :t_ImporteMXN
+    define_attribute :descuento, xml_attribute: "Descuento", format: :t_ImporteMXN
+    define_attribute :moneda, xml_attribute: "Moneda", default: "MXN"
+    define_attribute :tipo_cambio, xml_attribute: "TipoCambio"
+    define_attribute :total, xml_attribute: "Total", format: :t_ImporteMXN
+    define_attribute :tipo_de_comprobante, xml_attribute: "TipoDeComprobante", default: "I"
+    define_attribute :exportacion, xml_attribute: "Exportacion", default: "01"
+    define_attribute :metodo_pago, xml_attribute: "MetodoPago"
+    define_attribute :lugar_expedicion, xml_attribute: "LugarExpedicion"
+    define_attribute :confirmacion, xml_attribute: "Confirmacion"
+
+    attr_reader :emisor, :receptor, :conceptos, :private_key, :sat_csd, :errors
     attr_writer :key_data, :key_pass
 
     def initialize
@@ -70,14 +69,27 @@ module Cfdi40
       @key_data = File.read(path)
     end
 
+    # Load from attribute 'Certificado' when the CFDi is
+    # loaded from a string
+    def load_cert
+      return if @sat_csd&.cert64
+
+      @sat_csd ||= SatCsd.new
+      @sat_csd.cert_der = OpenSSL::X509::Certificate.new(Base64.decode64(certificado))
+      true
+    rescue StandardError
+      # puts "Waring; Unable to load certificate from XML string"
+      false
+    end
+
     def sign
       @sat_csd ||= SatCsd.new
       load_private_key if @sat_csd.private_key.nil?
       return unless @sat_csd.private_key
 
-      raise Error, 'Key and certificate not match' unless @sat_csd.valid_pair?
+      raise Error, "Key and certificate not match" unless @sat_csd.valid_pair?
 
-      digest = @sat_csd.private_key.sign(OpenSSL::Digest.new('SHA256'), original_content)
+      digest = @sat_csd.private_key.sign(OpenSSL::Digest.new("SHA256"), original_content)
       @sello = Base64.strict_encode64 digest
       @docxml = nil
     end
@@ -90,7 +102,7 @@ module Cfdi40
     # +descripcion+:: Product or service description
     #
     # ### Price and Taxes attributes
-    # 
+    #
     # +tasa_iva+:: Decimal between 0 and 1. Nil means exempt. Default value is 0.16
     # +tasa_ieps+:: Decimal between 0 and 1. Nil means exempt. Default value is null
     # +precio_bruto+:: Price before apply taxes or gross price.
@@ -106,7 +118,7 @@ module Cfdi40
     # +descuento+:: PENDING
     #
     # ## Special attributes
-    # 
+    #
     # ### IEDU attributes
     #
     # IEDU node (path: cfdi:Comprobante/cfdi:Conceptos/cfdi:Concepto/cfdi:ComplementoConcepto/iedu:instEducativas) is
@@ -119,17 +131,15 @@ module Cfdi40
     # +iedu_rfc_pago+::
     #
     def add_concepto(attributes = {})
-      raise Error, 'CFDi tipo pago no acepta conceptos' if tipo_de_comprobante == 'P'
+      raise Error, "CFDi tipo pago no acepta conceptos" if tipo_de_comprobante == "P"
 
       concepto = Concepto.new
       concepto.parent_node = @conceptos
       attributes.each do |key, value|
         method_name = "#{key}=".to_sym
-        if concepto.respond_to?(method_name)
-          concepto.public_send(method_name, value)
-        else
-          raise Error, ":#{key} no se puede asignar al concepto"
-        end
+        raise Error, ":#{key} no se puede asignar al concepto" unless concepto.respond_to?(method_name)
+
+        concepto.public_send(method_name, value)
       end
       concepto.calculate!
       @conceptos.children_nodes << concepto
@@ -137,6 +147,28 @@ module Cfdi40
       concepto
     end
 
+    # Load node 'Concepto' from a Nokogiri::XML::Element
+    def load_concepto(ng_node)
+      concepto = Concepto.new
+      concepto.parent_node = @conceptos
+      concepto.load_from_ng_node(ng_node)
+      @conceptos.children_nodes << concepto
+      concepto
+    end
+
+    # Load node cfdi:Comprobante/cfdi:Impuestos
+    #
+    # Normally this node is calculated but must be read from the
+    # XML when a CFDi is loaded
+    def load_impuestos(ng_node)
+      impuestos.load_from_ng_node(ng_node)
+      ng_iva_node = ng_node.xpath("cfdi:Traslados/cfdi:Traslado[@Impuesto='002']").first
+      return true if ng_iva_node.nil?
+
+      impuestos.traslado_iva.load_from_ng_node(ng_iva_node)
+      true
+    end
+
     # TODO: Doc params add_pago
     # monto
     # uuid
@@ -148,7 +180,7 @@ module Cfdi40
     # importe_saldo_anterior
     # objeto_impuestos
     def add_pago(attributes = {})
-      raise Error, "CFDi debe ser tipo 'P'" unless tipo_de_comprobante == 'P'
+      raise Error, "CFDi debe ser tipo 'P'" unless tipo_de_comprobante == "P"
 
       add_node_concepto_actividad_pago
       complemento.add_pago(attributes)
@@ -176,28 +208,41 @@ module Cfdi40
     end
 
     def original_content
-      xslt_path = File.join(File.dirname(__FILE__), '..', '..', 'lib/xslt/cadenaoriginal_local.xslt')
+      xslt_path = File.join(File.dirname(__FILE__), "..", "..", "lib/xslt/cadenaoriginal_local.xslt")
       xslt = Nokogiri::XSLT(File.open(xslt_path))
       transformed = xslt.transform(docxml)
       # The ampersand (&) char must be used in original content
       # even though the documentation indicates otherwise
-      transformed.children.to_s.gsub('&amp;', '&').strip
+      transformed.children.to_s.gsub("&amp;", "&").strip
+    end
+
+    # Shortcut to attribute TotalImpuestosTrasladados of impuestos node
+    def total_impuestos_trasladados
+      return nil unless impuestos_node
+
+      impuestos_node.total_impuestos_trasladados
+    end
+
+    def total_iva_node
+      return nil unless impuestos_node
+
+      impuestos_node.traslado_iva
     end
 
     private
 
     def add_node_concepto_actividad_pago
       return if defined?(@concepto_actividad)
-      
-      @receptor.uso_cfdi = 'CP01'
+
+      @receptor.uso_cfdi = "CP01"
       @concepto_actividad = Concepto.new
-      @concepto_actividad.clave_prod_serv = '84111506'
+      @concepto_actividad.clave_prod_serv = "84111506"
       @concepto_actividad.cantidad = 1
-      @concepto_actividad.clave_unidad = 'ACT'
-      @concepto_actividad.descripcion = 'Pago'
+      @concepto_actividad.clave_unidad = "ACT"
+      @concepto_actividad.descripcion = "Pago"
       @concepto_actividad.precio_bruto = 0
       @concepto_actividad.tasa_iva = nil
-      @concepto_actividad.objeto_impuestos = '01'
+      @concepto_actividad.objeto_impuestos = "01"
       @concepto_actividad.calculate!
       @conceptos.add_child_node @concepto_actividad
       calculate!
@@ -230,7 +275,6 @@ module Cfdi40
       impuestos.total_impuestos_trasladados = 0
       traslados.children_nodes = []
       traslados_summary.each do |key, value|
-        #TODO: Sumar los impuestos y agregarlos a los nodos globales de traslados
         traslado = Traslado.new
         traslado.parent_node = impuestos
         traslado.impuesto, traslado.tasa_o_cuota, traslado.tipo_factor = key
@@ -259,9 +303,7 @@ module Cfdi40
       summary
     end
 
-    # TODO: Este método tiene que ser 'impuestos'
-    #       si nos atenemos a que los que acaban con _node buscan en los hijos
-    #       y los que no terminan con _node crean el nodo
+    # Returns a Cfdi40::Node for 'Impuestos'
     def impuestos
       return @impuestos if defined?(@impuestos)
 
@@ -272,7 +314,7 @@ module Cfdi40
     end
 
     def impuestos_node
-      children_nodes.select { |n| n.is_a?(Impuestos)}.first
+      children_nodes.select { |n| n.is_a?(Impuestos) }.first
     end
 
     def traslados
@@ -288,7 +330,7 @@ module Cfdi40
 
     def load_private_key
       return unless defined?(@key_data)
-      
+
       @sat_csd ||= SatCsd.new
       @sat_csd.set_private_key(@key_data, (defined?(@key_pass) ? @key_pass : nil))
     end

data/lib/cfdi40/concepto.rb

@@ -1,20 +1,22 @@
+# frozen_string_literal: true
+
 # Represents node 'concepto'
 #
 # * Attribute +Importe+ represente gross amount. Gross amount is before taxes and the result of multiply
 #   +ValorUnitario+ by +Cantidad+
-#   
+#
 module Cfdi40
   class Concepto < Node
-    define_attribute :clave_prod_serv, xml_attribute: 'ClaveProdServ'
-    define_attribute :no_identificacion,xml_attribute: 'NoIdentificacion'
-    define_attribute :cantidad, xml_attribute: 'Cantidad', default: 1
-    define_attribute :clave_unidad, xml_attribute: 'ClaveUnidad'
-    define_attribute :unidad, xml_attribute: 'Unidad'
-    define_attribute :descripcion, xml_attribute: 'Descripcion'
-    define_attribute :valor_unitario, xml_attribute: 'ValorUnitario', format: :t_Importe
-    define_attribute :importe, xml_attribute: 'Importe', format: :t_Importe
-    define_attribute :descuento, xml_attribute: 'Descuento', format: :t_Importe
-    define_attribute :objeto_impuestos, xml_attribute: 'ObjetoImp', default: '01'
+    define_attribute :clave_prod_serv, xml_attribute: "ClaveProdServ"
+    define_attribute :no_identificacion, xml_attribute: "NoIdentificacion"
+    define_attribute :cantidad, xml_attribute: "Cantidad", default: 1
+    define_attribute :clave_unidad, xml_attribute: "ClaveUnidad"
+    define_attribute :unidad, xml_attribute: "Unidad"
+    define_attribute :descripcion, xml_attribute: "Descripcion"
+    define_attribute :valor_unitario, xml_attribute: "ValorUnitario", format: :t_Importe
+    define_attribute :importe, xml_attribute: "Importe", format: :t_Importe
+    define_attribute :descuento, xml_attribute: "Descuento", format: :t_Importe
+    define_attribute :objeto_impuestos, xml_attribute: "ObjetoImp", default: "01"
 
     attr_accessor :tasa_iva, :tasa_ieps, :precio_neto, :precio_bruto
     attr_reader :iva, :ieps, :base_iva, :importe_neto, :importe_bruto
@@ -38,7 +40,7 @@ module Cfdi40
         calculate_from_net_price
       elsif defined?(@precio_bruto) && !@precio_bruto.nil?
         calculate_from_gross_price
-      elsif !self.valor_unitario.nil?
+      elsif !valor_unitario.nil?
         @precio_bruto = valor_unitario
         calculate_from_gross_price
       end
@@ -49,7 +51,7 @@ module Cfdi40
     end
 
     def objeto_impuestos?
-      objeto_impuestos == '02'
+      objeto_impuestos == "02"
     end
 
     def traslado_nodes
@@ -58,6 +60,16 @@ module Cfdi40
       impuestos_node.traslado_nodes
     end
 
+    def traslado_iva_node
+      return nil unless impuestos_node.is_a?(Impuestos)
+
+      impuestos_node.traslado_iva
+    end
+
+    def load_traslado_iva(ng_node)
+      traslado_iva_node.load_from_ng_node(ng_node)
+    end
+
     private
 
     def calculate_from_net_price
@@ -100,13 +112,13 @@ module Cfdi40
     end
 
     def assign_objeto_imp
-      #01 No objeto de impuesto.
-      #02  Sí objeto de impuesto.
-      #03  Sí objeto del impuesto y no obligado al desglose.
+      # 01 No objeto de impuesto.
+      # 02 Sí objeto de impuesto.
+      # 03 Sí objeto del impuesto y no obligado al desglose.
 
-      return if objeto_impuestos == '03'
+      return if objeto_impuestos == "03"
 
-      self.objeto_impuestos = (!@tasa_iva.nil? || !@tasa_ieps.nil? ? '02' : '01')
+      self.objeto_impuestos = (!@tasa_iva.nil? || !@tasa_ieps.nil? ? "02" : "01")
     end
 
     def impuestos_node
@@ -118,16 +130,10 @@ module Cfdi40
 
       @impuestos_node = Impuestos.new
       @impuestos_node.parent_node = self
-      self.children_nodes << @impuestos_node
+      children_nodes << @impuestos_node
       @impuestos_node
     end
 
-    def traslado_iva_node
-      return nil unless impuestos_node.is_a?(Impuestos)
-
-      impuestos_node.traslado_iva
-    end
-
     def add_inst_educativas
       return unless inst_educativas_present?
 
@@ -151,7 +157,7 @@ module Cfdi40
 
       @complemento_concepto_node = ComplementoConcepto.new
       @complemento_concepto_node.parent_node = self
-      self.children_nodes << @complemento_concepto_node
+      children_nodes << @complemento_concepto_node
       @complemento_concepto_node
     end
 

data/lib/cfdi40/conceptos.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Conceptos < Node
   end

data/lib/cfdi40/docto_relacionado.rb

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class DoctoRelacionado < Node
-    define_attribute :id_documento, xml_attribute: 'IdDocumento'
-    define_attribute :serie, xml_attribute: 'Serie'
-    define_attribute :folio, xml_attribute: 'Folio'
-    define_attribute :moneda_dr, xml_attribute: 'MonedaDR', default: 'MXN'
-    define_attribute :equivalencia_dr, xml_attribute: 'EquivalenciaDR', default: '1'
-    define_attribute :num_parcialidad, xml_attribute: 'NumParcialidad'
-    define_attribute :imp_saldo_ant, xml_attribute: 'ImpSaldoAnt', format: :t_ImporteMXN
-    define_attribute :imp_pagado, xml_attribute: 'ImpPagado', format: :t_ImporteMXN
-    define_attribute :imp_saldo_insoluto, xml_attribute: 'ImpSaldoInsoluto', format: :t_ImporteMXN
-    define_attribute :objeto_imp_dr, xml_attribute: 'ObjetoImpDR', default: '02'
+    define_attribute :id_documento, xml_attribute: "IdDocumento"
+    define_attribute :serie, xml_attribute: "Serie"
+    define_attribute :folio, xml_attribute: "Folio"
+    define_attribute :moneda_dr, xml_attribute: "MonedaDR", default: "MXN"
+    define_attribute :equivalencia_dr, xml_attribute: "EquivalenciaDR", default: "1"
+    define_attribute :num_parcialidad, xml_attribute: "NumParcialidad"
+    define_attribute :imp_saldo_ant, xml_attribute: "ImpSaldoAnt", format: :t_ImporteMXN
+    define_attribute :imp_pagado, xml_attribute: "ImpPagado", format: :t_ImporteMXN
+    define_attribute :imp_saldo_insoluto, xml_attribute: "ImpSaldoInsoluto", format: :t_ImporteMXN
+    define_attribute :objeto_imp_dr, xml_attribute: "ObjetoImpDR", default: "02"
 
     def calculate!
       self.imp_saldo_insoluto = (imp_saldo_ant - imp_pagado).round(2)
@@ -18,13 +20,13 @@ module Cfdi40
 
     # Add nodes for 'traslado_dr' and/or 'retencion_dr' and intermediate nodes
     def add_impuestos
-      add_traslado if objeto_imp_dr == '02'
+      add_traslado if objeto_imp_dr == "02"
     end
 
     def add_traslado
-      return unless objeto_imp_dr == '02'
+      return unless objeto_imp_dr == "02"
 
-      # Taxes values for IVA rate 0.16 assumming that all 'conceptos' in the realted document 
+      # Taxes values for IVA rate 0.16 assumming that all 'conceptos' in the realted document
       # has the same tax rate. This could not be true but the is the most common case.
       traslado_dr = TrasladoDR.new
       traslado_dr.monto_pago = imp_pagado.round(2)
@@ -56,7 +58,7 @@ module Cfdi40
 
     def impuestos_dr
       return @impuestos_dr if defined?(@impuestos_dr)
-      
+
       @impuestos_dr = ImpuestosDR.new
       @impuestos_dr.parent_node = self
       @children_nodes << @impuestos_dr

data/lib/cfdi40/emisor.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Emisor < Node
-    define_attribute :rfc, xml_attribute: 'Rfc'
-    define_attribute :nombre, xml_attribute: 'Nombre'
-    define_attribute :regimen_fiscal, xml_attribute: 'RegimenFiscal'
+    define_attribute :rfc, xml_attribute: "Rfc"
+    define_attribute :nombre, xml_attribute: "Nombre"
+    define_attribute :regimen_fiscal, xml_attribute: "RegimenFiscal"
   end
 end

data/lib/cfdi40/impuestos.rb

@@ -1,19 +1,21 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Impuestos < Node
-    define_attribute :total_impuestos_retenidos, xml_attribute: 'TotalImpuestosRetenidos', format: :t_ImporteMXN
-    define_attribute :total_impuestos_trasladados, xml_attribute: 'TotalImpuestosTrasladados', format: :t_ImporteMXN
+    define_attribute :total_impuestos_retenidos, xml_attribute: "TotalImpuestosRetenidos", format: :t_ImporteMXN
+    define_attribute :total_impuestos_trasladados, xml_attribute: "TotalImpuestosTrasladados", format: :t_ImporteMXN
 
     def traslados
       return @traslados if defined?(@traslados)
 
       @traslados = Traslados.new
       @traslados.parent_node = self
-      self.children_nodes << @traslados
+      children_nodes << @traslados
       @traslados
     end
 
     def traslados_node
-      children_nodes.select { |n| n.is_a?(Traslados)}.first
+      children_nodes.select { |n| n.is_a?(Traslados) }.first
     end
 
     def traslado_nodes

data/lib/cfdi40/impuestos_dr.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class ImpuestosDR < Node
     def traslados_dr

data/lib/cfdi40/impuestos_p.rb

@@ -1,6 +1,7 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class ImpuestosP < Node
-
     def traslados_p
       return @traslados_p if defined?(@traslados_p)
 

data/lib/cfdi40/inst_educativas.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class InstEducativas < Node
-    define_element_name 'instEducativas'
+    define_element_name "instEducativas"
     define_namespace "xsi", "http://www.w3.org/2001/XMLSchema-instance"
     define_namespace "iedu", "http://www.sat.gob.mx/iedu"
     define_attribute :schema_location,
-                     xml_attribute: 'xsi:schemaLocation',
+                     xml_attribute: "xsi:schemaLocation",
                      readonly: true,
                      default: "http://www.sat.gob.mx/iedu http://www.sat.gob.mx/sitio_internet/cfd/iedu/iedu.xsd"
-    define_attribute :version, xml_attribute: 'version', readonly: true, default: '1.0'
-    define_attribute :nombre_alumno, xml_attribute: 'nombreAlumno'
-    define_attribute :curp, xml_attribute: 'CURP'
-    define_attribute :nivel_educativo, xml_attribute: 'nivelEducativo'
-    define_attribute :aut_rvoe, xml_attribute: 'autRVOE'
-    define_attribute :rfc_pago, xml_attribute: 'rfcPago'
+    define_attribute :version, xml_attribute: "version", readonly: true, default: "1.0"
+    define_attribute :nombre_alumno, xml_attribute: "nombreAlumno"
+    define_attribute :curp, xml_attribute: "CURP"
+    define_attribute :nivel_educativo, xml_attribute: "nivelEducativo"
+    define_attribute :aut_rvoe, xml_attribute: "autRVOE"
+    define_attribute :rfc_pago, xml_attribute: "rfcPago"
   end
 end

data/lib/cfdi40/node.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Node
-    # Nokigir XML Document for the xml_node
+    # Nokigiri XML Document for the xml_node
     attr_accessor :xml_document, :xml_parent, :children_nodes, :parent_node
     attr_writer :element_name
 
@@ -33,12 +35,10 @@ module Cfdi40
         attr_accessor accessor.to_sym
       end
       @@attributes[name][accessor.to_sym] = xml_attribute
-      if default
-        @@default_values[name][accessor.to_sym] = default
-      end
-      if format
-        @@formats[name][accessor.to_sym] = format
-      end
+      @@default_values[name][accessor.to_sym] = default if default
+      return unless format
+
+      @@formats[name][accessor.to_sym] = format
     end
 
     def self.define_namespace(namespace, value)
@@ -89,7 +89,7 @@ module Cfdi40
     end
 
     def add_child_node(child_node)
-      raise Error, 'child_node must be a Node object' unless child_node.is_a?(Node)
+      raise Error, "child_node must be a Node object" unless child_node.is_a?(Node)
 
       child_node.parent_node = self
       @children_nodes << child_node
@@ -97,19 +97,28 @@ module Cfdi40
 
     def current_namespace
       return unless self.class.respond_to?(:namespaces)
-      if self.class.namespaces.empty?
-        return parent_node.current_namespace unless parent_node.nil?
-      end
+
+      return parent_node.current_namespace if self.class.namespaces.empty? && !parent_node.nil?
 
       self.class.namespaces.keys.last
     end
 
+    # Load attributes from a Nokogiri::XML::Element.
+    # Attributes are loaded directly to the instance variable
+    def load_from_ng_node(ng_node)
+      # TODO: Se puede cargar el certificado
+      # x509_cert = OpenSSL::X509::Certificate.new(Base64.decode64(<valor del atributo>))
+      self.class.attributes.each do |variable_name, attr_name|
+        next if ng_node.attributes[attr_name].nil?
+
+        instance_variable_set("@#{variable_name}".to_sym, ng_node.attributes[attr_name].value)
+      end
+    end
+
     def create_xml_node
       # TODO: Quitar la siguiente linea (set_defaults) si funciona poniendo los defaults en initialize
       # set_defaults
-      if self.respond_to?(:before_add, true)
-        self.before_add
-      end
+      before_add if respond_to?(:before_add, true)
       xml_node = xml_document.create_element(expanded_element_name)
       add_namespaces_to(xml_node)
       add_attributes_to(xml_node)
@@ -119,13 +128,14 @@ module Cfdi40
 
     # Returns setted @element_name or use class_name
     def element_name
-      return self.class.element_name unless self.class.element_name.nil? || self.class.element_name == ''
+      return self.class.element_name unless self.class.element_name.nil? || self.class.element_name == ""
 
-      self.class.name.split('::').last
+      self.class.name.split("::").last
     end
 
     def expanded_element_name
       return element_name unless current_namespace
+
       "#{current_namespace}:#{element_name}"
     end
 
@@ -155,9 +165,9 @@ module Cfdi40
     def formated_value(accessor)
       case self.class.formats[accessor]
       when :t_Importe
-        public_send(accessor).to_f == 0.0 ? '0' : sprintf("%0.6f", public_send(accessor).to_f)
+        public_send(accessor).to_f == 0.0 ? "0" : format("%0.6f", public_send(accessor).to_f)
       when :t_ImporteMXN
-        public_send(accessor).to_f == 0.0 ? '0' : sprintf("%0.2f", public_send(accessor).to_f)
+        public_send(accessor).to_f == 0.0 ? "0" : format("%0.2f", public_send(accessor).to_f)
       else
         public_send(accessor)
       end

data/lib/cfdi40/pago.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Pago < Node
-    define_attribute :monto, xml_attribute: 'Monto'
-    define_attribute :fecha_pago, xml_attribute: 'FechaPago'
-    define_attribute :forma_pago, xml_attribute: 'FormaDePagoP'
-    define_attribute :moneda, xml_attribute: 'MonedaP', readonly: true, default: 'MXN'
-    define_attribute :tipo_cambio, xml_attribute: 'TipoCambioP', readonly: true, default: '1'
+    define_attribute :monto, xml_attribute: "Monto"
+    define_attribute :fecha_pago, xml_attribute: "FechaPago"
+    define_attribute :forma_pago, xml_attribute: "FormaDePagoP"
+    define_attribute :moneda, xml_attribute: "MonedaP", readonly: true, default: "MXN"
+    define_attribute :tipo_cambio, xml_attribute: "TipoCambioP", readonly: true, default: "1"
 
     attr_accessor :uuid, :serie, :folio, :num_parcialidad, :importe_saldo_anterior, :objeto_impuestos
 
@@ -35,7 +37,7 @@ module Cfdi40
 
       @docto_relacionados =
         @children_nodes.select do |child|
-          child if child.class.name == 'Cfdi40::DoctoRelacionado'
+          child if child.instance_of?(::Cfdi40::DoctoRelacionado)
         end
     end
 
@@ -45,7 +47,7 @@ module Cfdi40
       summary = {}
       docto_relacionados.each do |docto|
         docto.traslados_summary.each do |key, data|
-          summary[key] ||= {base: 0, importe: 0}
+          summary[key] ||= { base: 0, importe: 0 }
           summary[key][:base] += data[:base]
           summary[key][:importe] += data[:importe]
         end

data/lib/cfdi40/pagos.rb

@@ -1,23 +1,23 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Pagos < Node
     define_namespace "pago20", "http://www.sat.gob.mx/Pagos20"
     define_attribute :schema_location,
-                     xml_attribute: 'xsi:schemaLocation',
+                     xml_attribute: "xsi:schemaLocation",
                      readonly: true,
                      default: "http://www.sat.gob.mx/Pagos20 " \
                               "http://www.sat.gob.mx/sitio_internet/cfd/Pagos/Pagos20.xsd"
-    define_attribute :version, xml_attribute: 'Version', readonly: true, default: '2.0'
+    define_attribute :version, xml_attribute: "Version", readonly: true, default: "2.0"
 
-    def add_pago(attributes={})
+    def add_pago(attributes = {})
       pago = Pago.new
       pago.parent_node = self
       attributes.each do |key, value|
         method_name = "#{key}=".to_sym
-        if pago.respond_to?(method_name)
-          pago.public_send(method_name, value)
-        else
-          raise Error, ":#{key} no se puede asignar al nodo Pago"
-        end
+        raise Error, ":#{key} no se puede asignar al nodo Pago" unless pago.respond_to?(method_name)
+
+        pago.public_send(method_name, value)
       end
       pago.monto = pago.monto.round(2)
       pago.add_docto_relacionado
@@ -33,7 +33,7 @@ module Cfdi40
     end
 
     def update_totales_traslado_iva16
-      key = ['002', 'Tasa', '0.160000']
+      key = ["002", "Tasa", "0.160000"]
       return if traslados_summary[key].nil?
 
       totales_node.base_iva16 = traslados_summary[key][:base]
@@ -67,7 +67,7 @@ module Cfdi40
     end
 
     def pago_nodes
-      @children_nodes.select { |node| node.class.name == 'Cfdi40::Pago' }
+      @children_nodes.select { |node| node.instance_of?(::Cfdi40::Pago) }
     end
   end
 end

data/lib/cfdi40/receptor.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Receptor < Node
-    define_attribute :rfc, xml_attribute: 'Rfc'
-    define_attribute :nombre, xml_attribute: 'Nombre'
-    define_attribute :domicilio_fiscal, xml_attribute: 'DomicilioFiscalReceptor'
-    define_attribute :residencia_fiscal, xml_attribute: 'ResidenciaFiscal'
-    define_attribute :num_reg_id_trib, xml_attribute: 'NumRegIdTrib' 
-    define_attribute :regimen_fiscal, xml_attribute: 'RegimenFiscalReceptor'
-    define_attribute :uso_cfdi, xml_attribute: 'UsoCFDI'
+    define_attribute :rfc, xml_attribute: "Rfc"
+    define_attribute :nombre, xml_attribute: "Nombre"
+    define_attribute :domicilio_fiscal, xml_attribute: "DomicilioFiscalReceptor"
+    define_attribute :residencia_fiscal, xml_attribute: "ResidenciaFiscal"
+    define_attribute :num_reg_id_trib, xml_attribute: "NumRegIdTrib"
+    define_attribute :regimen_fiscal, xml_attribute: "RegimenFiscalReceptor"
+    define_attribute :uso_cfdi, xml_attribute: "UsoCFDI"
   end
 end

data/lib/cfdi40/sat_csd.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class SatCsd
-    attr_reader :x509_cert
-    attr_reader :private_key
+    attr_reader :x509_cert, :private_key
 
     def cert_path=(path)
       @cert_path = path
@@ -43,8 +44,8 @@ module Cfdi40
     def no_certificado
       return unless x509_cert
 
-      s = ''
-      x509_cert.serial.to_s(16).split('').each_with_index do |c, i|
+      s = ""
+      x509_cert.serial.to_s(16).chars.each_with_index do |c, i|
         next if i.even?
 
         s += c
@@ -74,9 +75,9 @@ module Cfdi40
 
     def key_to_pem(key_der)
       array_key_pem = []
-      array_key_pem << '-----BEGIN ENCRYPTED PRIVATE KEY-----'
+      array_key_pem << "-----BEGIN ENCRYPTED PRIVATE KEY-----"
       array_key_pem += Base64.strict_encode64(key_der).scan(/.{1,64}/)
-      array_key_pem << '-----END ENCRYPTED PRIVATE KEY-----'
+      array_key_pem << "-----END ENCRYPTED PRIVATE KEY-----"
       array_key_pem.join("\n")
     end
 

data/lib/cfdi40/schema_validator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Validates Schema Using xsd files
 module Cfdi40
   class SchemaValidator
@@ -6,7 +8,8 @@ module Cfdi40
     # schema = Nokogiri::XML::Schema(File.open("/home/israel/git/cfdi40/lib/xsd/cfdv40.xsd"), options)
 
     attr_reader :errors
-    LOCAL_XSD_PATH = File.join(File.dirname(__FILE__), '..', 'xsd', 'cfdv40.xsd')
+
+    LOCAL_XSD_PATH = File.join(File.dirname(__FILE__), "..", "xsd", "cfdv40.xsd")
 
     # Param xml is xml string
     def initialize(xml)

data/lib/cfdi40/totales.rb

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Totales < Node
-    define_attribute :ret_iva, xml_attribute: 'TotalRetencionesIVA'
-    define_attribute :ret_isr, xml_attribute: 'TotalRetencionesISR'
-    define_attribute :ret_ieps, xml_attribute: 'TotalRetencionesIEPS'
-    define_attribute :base_iva16, xml_attribute: 'TotalTrasladosBaseIVA16'
-    define_attribute :importe_iva16, xml_attribute: 'TotalTrasladosImpuestoIVA16'
-    define_attribute :base_iva8, xml_attribute: 'TotalTrasladosBaseIVA8'
-    define_attribute :importe_iva8, xml_attribute: 'TotalTrasladosImpuestoIVA8'
-    define_attribute :base_iva0, xml_attribute: 'TotalTrasladosBaseIVA0'
-    define_attribute :importe_iva0, xml_attribute: 'TotalTrasladosImpuestoIVA0'
-    define_attribute :base_iva_excento, xml_attribute: 'TotalTrasladosBaseIVAExento'
-    define_attribute :monto_total, xml_attribute: 'MontoTotalPagos'
+    define_attribute :ret_iva, xml_attribute: "TotalRetencionesIVA"
+    define_attribute :ret_isr, xml_attribute: "TotalRetencionesISR"
+    define_attribute :ret_ieps, xml_attribute: "TotalRetencionesIEPS"
+    define_attribute :base_iva16, xml_attribute: "TotalTrasladosBaseIVA16"
+    define_attribute :importe_iva16, xml_attribute: "TotalTrasladosImpuestoIVA16"
+    define_attribute :base_iva8, xml_attribute: "TotalTrasladosBaseIVA8"
+    define_attribute :importe_iva8, xml_attribute: "TotalTrasladosImpuestoIVA8"
+    define_attribute :base_iva0, xml_attribute: "TotalTrasladosBaseIVA0"
+    define_attribute :importe_iva0, xml_attribute: "TotalTrasladosImpuestoIVA0"
+    define_attribute :base_iva_excento, xml_attribute: "TotalTrasladosBaseIVAExento"
+    define_attribute :monto_total, xml_attribute: "MontoTotalPagos"
   end
 end

data/lib/cfdi40/traslado.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Traslado < Node
-    define_attribute :base, xml_attribute: 'Base', format: :t_ImporteMXN
-    define_attribute :impuesto, xml_attribute: 'Impuesto'
-    define_attribute :tipo_factor, xml_attribute: 'TipoFactor', default: 'Tasa'
-    define_attribute :tasa_o_cuota, xml_attribute: 'TasaOCuota', format: :t_Importe
-    define_attribute :importe, xml_attribute: 'Importe', format: :t_ImporteMXN
+    define_attribute :base, xml_attribute: "Base", format: :t_ImporteMXN
+    define_attribute :impuesto, xml_attribute: "Impuesto"
+    define_attribute :tipo_factor, xml_attribute: "TipoFactor", default: "Tasa"
+    define_attribute :tasa_o_cuota, xml_attribute: "TasaOCuota", format: :t_Importe
+    define_attribute :importe, xml_attribute: "Importe", format: :t_ImporteMXN
   end
 end

data/lib/cfdi40/traslado_dr.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class TrasladoDR < Node
-    define_attribute :base_dr, xml_attribute: 'BaseDR', format: :t_ImporteMXN
-    define_attribute :impuesto_dr, xml_attribute: 'ImpuestoDR', default: '002'
-    define_attribute :tipo_factor_dr, xml_attribute: 'TipoFactorDR', default: 'Tasa'
-    define_attribute :tasa_o_cuota_dr, xml_attribute: 'TasaOCuotaDR', default: '0.160000', format: :t_Importe
-    define_attribute :importe_dr, xml_attribute: 'ImporteDR', format: :t_ImporteMXN
+    define_attribute :base_dr, xml_attribute: "BaseDR", format: :t_ImporteMXN
+    define_attribute :impuesto_dr, xml_attribute: "ImpuestoDR", default: "002"
+    define_attribute :tipo_factor_dr, xml_attribute: "TipoFactorDR", default: "Tasa"
+    define_attribute :tasa_o_cuota_dr, xml_attribute: "TasaOCuotaDR", default: "0.160000", format: :t_Importe
+    define_attribute :importe_dr, xml_attribute: "ImporteDR", format: :t_ImporteMXN
 
     attr_accessor :monto_pago
 
@@ -14,5 +16,3 @@ module Cfdi40
     end
   end
 end
-
-

data/lib/cfdi40/traslado_p.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class TrasladoP < Node
-    define_attribute :base_p, xml_attribute: 'BaseP', format: :t_ImporteMXN
-    define_attribute :impuesto_p, xml_attribute: 'ImpuestoP', default: '002'
-    define_attribute :tipo_factor_p, xml_attribute: 'TipoFactorP', default: 'Tasa'
-    define_attribute :tasa_o_cuota_p, xml_attribute: 'TasaOCuotaP', default: '0.160000', format: :t_Importe
-    define_attribute :importe_p, xml_attribute: 'ImporteP', format: :t_ImporteMXN
+    define_attribute :base_p, xml_attribute: "BaseP", format: :t_ImporteMXN
+    define_attribute :impuesto_p, xml_attribute: "ImpuestoP", default: "002"
+    define_attribute :tipo_factor_p, xml_attribute: "TipoFactorP", default: "Tasa"
+    define_attribute :tasa_o_cuota_p, xml_attribute: "TasaOCuotaP", default: "0.160000", format: :t_Importe
+    define_attribute :importe_p, xml_attribute: "ImporteP", format: :t_ImporteMXN
   end
 end

data/lib/cfdi40/traslados.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class Traslados < Node
     def traslado_iva
       return @traslado_iva if defined?(@traslado_iva)
-      
+
       @traslado_iva = Traslado.new
       # TODO: FIX magic number
-      @traslado_iva.impuesto = '002'
+      @traslado_iva.impuesto = "002"
       @traslado_iva.parent_node = self
-      self.children_nodes << @traslado_iva
+      children_nodes << @traslado_iva
       @traslado_iva
     end
 

data/lib/cfdi40/traslados_dr.rb

@@ -1,5 +1,6 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class TrasladosDR < Node
   end
 end
-

data/lib/cfdi40/traslados_p.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Cfdi40
   class TrasladosP < Node
     def add_traslado_p_nodes

data/lib/cfdi40/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Cfdi40
-  VERSION = "0.0.7"
+  VERSION = "0.0.8"
 end

data/lib/cfdi40/xml_loader.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+# Methods related to create a new Comprobante
+# from an XML (string)
+module Cfdi40
+  class XmlLoader
+    attr_reader :cfdi, :xml_doc
+
+    def initialize(xml_string)
+      @xml_doc = Nokogiri::XML(xml_string)
+      # TODO. validar versión del CFDI definido en xml_doc
+      @cfdi = Cfdi40::Comprobante.new
+      @cfdi.load_from_ng_node(xml_doc.root)
+      @cfdi.load_cert
+      load_emisor
+      load_receptor
+      load_conceptos
+      load_impuestos
+      @cfdi
+    end
+
+    private
+
+    def load_conceptos
+      n_concepto = 0
+      xml_doc.xpath("//cfdi:Concepto").each do |node|
+        n_concepto += 1
+        concepto = @cfdi.load_concepto(node)
+        iva_path = "//cfdi:Concepto[#{n_concepto}]/cfdi:Impuestos[1]/cfdi:Traslados[1]/cfdi:Traslado[@Impuesto='002']"
+        iva_node = xml_doc.xpath(iva_path).first
+        concepto.load_traslado_iva(iva_node) if iva_node
+      end
+    end
+
+    def load_emisor
+      ng_emisor_node = xml_doc.xpath("//cfdi:Emisor").first
+      return if ng_emisor_node.nil?
+
+      @cfdi.emisor.load_from_ng_node(ng_emisor_node)
+    end
+
+    def load_receptor
+      ng_receptor_node = xml_doc.xpath("//cfdi:Receptor").first
+      return if ng_receptor_node.nil?
+
+      @cfdi.receptor.load_from_ng_node(ng_receptor_node)
+    end
+
+    def load_impuestos
+      impuestos_node = xml_doc.xpath("cfdi:Comprobante/cfdi:Impuestos").first
+      return if impuestos_node.nil?
+
+      @cfdi.load_impuestos(impuestos_node)
+    end
+  end
+end

data/lib/cfdi40.rb

@@ -27,7 +27,8 @@ require_relative "cfdi40/traslado_dr"
 require_relative "cfdi40/impuestos_p"
 require_relative "cfdi40/traslados_p"
 require_relative "cfdi40/traslado_p"
-require_relative "cfdi40/totales.rb"
+require_relative "cfdi40/totales"
+require_relative "cfdi40/xml_loader"
 
 # Leading module and entry point for all features and classes
 #
@@ -40,4 +41,9 @@ module Cfdi40
   def self.new
     Comprobante.new
   end
+
+  def self.open(xml_string, mode: :readwrite)
+    loader = XmlLoader.new(xml_string)
+    loader.cfdi
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cfdi40
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - Israel Benítez
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-04-22 00:00:00.000000000 Z
+date: 2025-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -68,6 +68,7 @@ files:
 - lib/cfdi40/traslados_dr.rb
 - lib/cfdi40/traslados_p.rb
 - lib/cfdi40/version.rb
+- lib/cfdi40/xml_loader.rb
 - lib/xsd/Pagos20.xsd
 - lib/xsd/README.md
 - lib/xsd/TimbreFiscalDigitalv11.xsd