cfdi40 0.4.3 → 0.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 463e9e9afbe82401d3664da4e110e8c3be599db170884715b0b2a43762aad5b8
-  data.tar.gz: c688efebca0b62e00c85d0edc2d24be6d3b5add127aa8ee1f9217dd9234be17d
+  metadata.gz: c5d902e16b11421579d59b2cf22aac71d40602829d952e43825c38f448689b67
+  data.tar.gz: 5f1a7d20d12018956c459cce25ea08701778c87a5ac5b665da7760dff72699b4
 SHA512:
-  metadata.gz: aac1fa7a86a1a8fcffe9b435b718f3ce665777da9a27f747e2a170d4d28b675b37e7e9656d2a522558926674dcaaafc08776448ad6652be5f3ab678d58d83d50
-  data.tar.gz: abedeb3fad410bd0e338b91279ceec3652b4dbf182acd884da53c8fdeee88e1259e698e798b3c264d430ea315dcfd8d184d13a02965494658350794cd191ddba
+  metadata.gz: 77abcb157daa8154ec16bd2c1fc847715034f78905f44a7dd94a29c5274121d38d781bcf5ec504b7dbf0661ecd8964773260be691db742158d2eeef34b3e80fc
+  data.tar.gz: '0638c1cbaeb96408dac0f3822a8f198d32452d99fa6af66b8a28fc1fcd16ad95ee274221aa8b2069b40ffaf15bd23223cd0fbe0752553f63767b5ea21659e05a'

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    cfdi40 (0.4.3)
+    cfdi40 (0.4.4)
       nokogiri (>= 1.10.10)
 
 GEM

data/lib/cfdi40/complemento.rb

@@ -32,11 +32,11 @@ module Cfdi40
       @pagos = Cfdi40::Pagos.new
       @pagos.load_from_ng_node(pagos_node)
       @pagos.parent_node = self
+      totales_node = pagos_node.xpath("//pago20:Totales", Cfdi40::Pagos::NG_NAMESPACE).first
+      @pagos.load_totales(totales_node) unless totales_node.nil?
       pagos_node.xpath("//pago20:Pago", Cfdi40::Pagos::NG_NAMESPACE).each do |pago_node|
         @pagos.load_pago(pago_node)
       end
-      totales_node = pagos_node.xpath("//pago20:Totales", Cfdi40::Pagos::NG_NAMESPACE).first
-      @pagos.load_totales(totales_node) unless totales_node.nil?
       @children_nodes << @pagos
       @pagos
     end

data/lib/cfdi40/comprobante.rb

@@ -32,7 +32,7 @@ module Cfdi40
     define_attribute :confirmacion, xml_attribute: "Confirmacion"
 
     attr_reader :emisor, :receptor, :conceptos, :private_key, :sat_csd, :errors, :cadena_original
-    attr_writer :key_data, :key_pass
+    attr_writer :key_data, :key_pass, :namespace_pagos_on_root
     attr_accessor :loaded_xml
 
     def initialize
@@ -47,6 +47,7 @@ module Cfdi40
       @sat_csd = SatCsd.new
       @fecha ||= Time.now
       @children_nodes = [@emisor, @receptor, @conceptos]
+      @namespace_pagos_on_root = false
       set_defaults
     end
 
@@ -214,6 +215,8 @@ module Cfdi40
       return loaded_xml if !loaded_xml.nil? && signed?
 
       sign unless signed?
+      return xml_string_ns_pagos_on_root if @namespace_pagos_on_root && pago_nodes.count > 0
+
       docxml.to_xml
     end
 
@@ -300,6 +303,14 @@ module Cfdi40
       complemento.timbre
     end
 
+    # Some PACs require that the namespace pago20 be placed in root node
+    def add_namespace_pagos_to_root
+      self.class.define_namespace "pago20", "http://www.sat.gob.mx/Pagos20"
+      @schema_location += " http://www.sat.gob.mx/Pagos20 " \
+                           "http://www.sat.gob.mx/sitio_internet/cfd/Pagos/Pagos20.xsd"
+      true
+    end
+
     private
 
     def add_node_concepto_actividad_pago
@@ -399,5 +410,16 @@ module Cfdi40
       @children_nodes << @complemento
       @complemento
     end
+
+    # Creates a new XML an change and puts the namespace pagos on root element.
+    def xml_string_ns_pagos_on_root
+      tmp_nkgdoc = Nokogiri::XML(docxml.to_xml)
+      tmp_nkgdoc.root["xmlns:pago20"] = "http://www.sat.gob.mx/Pagos20"
+      unless tmp_nkgdoc.root["xsi:schemaLocation"].include?("Pagos20")
+        tmp_nkgdoc.root["xsi:schemaLocation"] += " http://www.sat.gob.mx/Pagos20" \
+                                                 " http://www.sat.gob.mx/sitio_internet/cfd/Pagos/Pagos20.xsd"
+      end
+      tmp_nkgdoc.to_xml
+    end
   end
 end

data/lib/cfdi40/docto_relacionado.rb

@@ -6,7 +6,7 @@ module Cfdi40
     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", format: :decimal
+    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

data/lib/cfdi40/impuestos_p.rb

@@ -9,5 +9,19 @@ module Cfdi40
       add_child_node @traslados_p
       @traslados_p
     end
+
+    def traslados_p_node
+      @children_nodes.select {|child| child.instance_of?(Cfdi40::TrasladosP) }.first
+    end
+
+    def load_traslados_p_node(ng_node)
+      @traslados_p = TrasladosP.new
+      @traslados_p.load_from_ng_node(ng_node)
+      add_child_node @traslados_p
+      ng_node.xpath("//pago20:TrasladoP", { "pago20" => "http://www.sat.gob.mx/Pagos20" }).each do |trsld_p_node|
+        @traslados_p.load_traslado_p(trsld_p_node)
+      end
+      @traslados_p
+    end
   end
 end

data/lib/cfdi40/pago.rb

@@ -38,7 +38,7 @@ module Cfdi40
 
       @docto_relacionados =
         @children_nodes.select do |child|
-          child if child.instance_of?(::Cfdi40::DoctoRelacionado)
+          child.instance_of?(::Cfdi40::DoctoRelacionado)
         end
     end
 
@@ -65,5 +65,18 @@ module Cfdi40
       dr.load_impuestos_dr(imp_dr_node) unless imp_dr_node.nil?
       dr
     end
+
+    def impuestos_p_node
+      @children_nodes.select { |child| child.instance_of?(Cfdi40::ImpuestosP) }.first
+    end
+
+    def load_impuestos_p(ng_node)
+      imp_p = ImpuestosP.new
+      imp_p.load_from_ng_node(ng_node)
+      add_child_node imp_p
+      trns_p_node = ng_node.xpath("//pago20:TrasladosP", { "pago20" => "http://www.sat.gob.mx/Pagos20" }).first
+      imp_p.load_traslados_p_node(trns_p_node)
+      imp_p
+    end
   end
 end

data/lib/cfdi40/pagos.rb

@@ -103,11 +103,12 @@ module Cfdi40
     def load_pago(pago_node)
       pago = Pago.new
       pago.load_from_ng_node(pago_node)
-      pago.parent_node = self
-      @children_nodes << pago
+      add_child_node pago
       pago_node.xpath("//pago20:DoctoRelacionado", { "pago20" => "http://www.sat.gob.mx/Pagos20" }).each do |dr_node|
         pago.load_dr(dr_node)
       end
+      imp_p_node = pago_node.xpath("//pago20:ImpuestosP", { "pago20" => "http://www.sat.gob.mx/Pagos20" }).first
+      pago.load_impuestos_p(imp_p_node) if imp_p_node
       pago
     end
 

data/lib/cfdi40/traslados_p.rb

@@ -14,5 +14,16 @@ module Cfdi40
         add_child_node traslado_p
       end
     end
+
+    def traslado_p_nodes
+      @children_nodes.select { |child| child.instance_of?(Cfdi40::TrasladoP) }
+    end
+
+    def load_traslado_p(ng_node)
+      traslado_p = TrasladoP.new
+      traslado_p.load_from_ng_node(ng_node)
+      add_child_node traslado_p
+      traslado_p
+    end
   end
 end

data/lib/cfdi40/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cfdi40
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.4.4
 platform: ruby
 authors:
 - Israel Benítez
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2025-12-05 00:00:00.000000000 Z
+date: 2025-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri