cfdi40 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f610f343a775755d3ec5e603588f8693ac502c576c56a32bf6dda422a7eebebe
-  data.tar.gz: 0e72a90973ca6f28b42b0c793951400a0d5ae8532bbb7c2d750c917cbae41b38
+  metadata.gz: cad6896abeef260eaf8d4f9e659da6b6572fd693e51713261c982addf854373a
+  data.tar.gz: c68c19bb783b5864c81103899d5ab46b1281d6eccaecbd43efee35cf927777d6
 SHA512:
-  metadata.gz: 1cfcc729870cb04928ad9534068843df9ea61a9a6f9b0e994500f0b94b16f01659427984a05e05cc991900055bd4ac5bf8456d2ddabe0ffd7570f8e229693042
-  data.tar.gz: cfd0ca0717d596c1347e42d6ff680c39c13c57c0cf83c5ad9005ca96c9efcc5f9e1bd55ed706a086b99657bba0ff244b5eba8abcfabe50980f1481651429a87d
+  metadata.gz: 0ffc2e71680140e910f44dc35c5201ad28747454d4167290c57f52836bbd75dcae1aaa6768735d7435b2510cd96b6d9be6d271dddddce59e598c86ae71a502f1
+  data.tar.gz: 8649c8ade0dbbbee0caf4e9a31692d30874eb86ee5712ae3144501cd41614e6739c3d7d90b9e2584d561d7856345ad6b37063921671d50daa76c60b95d907d62

data/lib/cfdi40/comprobante.rb

@@ -2,6 +2,7 @@
 
 # Create and Read XML documents
 module Cfdi40
+  # root node
   class Comprobante < Node
     define_namespace "xsi", "http://www.w3.org/2001/XMLSchema-instance"
     define_namespace "cfdi", "http://www.sat.gob.mx/cfd/4"
@@ -13,7 +14,7 @@ module Cfdi40
     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 :fecha, xml_attribute: "Fecha", format: :t_FechaH
     define_attribute :sello, xml_attribute: "Sello", readonly: true
     define_attribute :forma_pago, xml_attribute: "FormaPago"
     define_attribute :no_certificado, xml_attribute: "NoCertificado"
@@ -32,6 +33,7 @@ module Cfdi40
 
     attr_reader :emisor, :receptor, :conceptos, :private_key, :sat_csd, :errors
     attr_writer :key_data, :key_pass
+    attr_accessor :loaded_xml
 
     def initialize
       super
@@ -43,7 +45,7 @@ module Cfdi40
       @receptor = Receptor.new
       @receptor.parent_node = self
       @sat_csd = SatCsd.new
-      @fecha ||= Time.now.strftime("%Y-%m-%dT%H:%M:%S")
+      @fecha ||= Time.now
       @children_nodes = [@emisor, @receptor, @conceptos]
       set_defaults
     end
@@ -91,6 +93,7 @@ module Cfdi40
 
       digest = @sat_csd.private_key.sign(OpenSSL::Digest.new("SHA256"), original_content)
       @sello = Base64.strict_encode64 digest
+      lock
       @docxml = nil
     end
 
@@ -192,7 +195,9 @@ module Cfdi40
     end
 
     def to_xml
-      sign
+      return loaded_xml if !loaded_xml.nil? && signed?
+
+      sign unless signed?
       docxml.to_xml
     end
 
@@ -204,17 +209,25 @@ module Cfdi40
       @errors.empty?
     end
 
+    def valid_signature?
+      return false unless signed?
+
+      signature_validator = SignatureValidator.new(to_xml)
+      signature_validator.valid?
+    end
+
+    def signed?
+      !docxml.root.attributes["Sello"].nil?
+    end
+
     def cadena_original
       original_content
     end
 
     def original_content
-      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
+      xml_string = loaded_xml.nil? ? docxml.to_s : loaded_xml
+
+      Cfdi40::OriginalContent.generate(xml_string)
     end
 
     # Shortcut to attribute TotalImpuestosTrasladados of impuestos node
@@ -225,10 +238,13 @@ module Cfdi40
     end
 
     def calculate!
+      return false if readonly
+
       @docxml = nil
       @subtotal = @conceptos.children_nodes.map(&:importe).map(&:to_f).sum
       @total = @conceptos.children_nodes.map(&:importe_neto).map(&:to_f).sum
       add_traslados_summary_node
+      true
     end
 
     def concepto_nodes

data/lib/cfdi40/concepto.rb

@@ -9,7 +9,7 @@ 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 :cantidad, xml_attribute: "Cantidad", default: 1, format: :decimal
     define_attribute :clave_unidad, xml_attribute: "ClaveUnidad"
     define_attribute :unidad, xml_attribute: "Unidad"
     define_attribute :descripcion, xml_attribute: "Descripcion"
@@ -75,7 +75,7 @@ module Cfdi40
     end
 
     def traslado_iva_node
-      return nil unless impuestos_node.is_a?(Impuestos)
+      return nil unless impuestos_node.is_a?(ConceptoImpuestos)
 
       impuestos_node.traslado_iva
     end
@@ -140,10 +140,10 @@ module Cfdi40
       return @impuestos_node if defined?(@impuestos_node)
       return nil unless objeto_impuestos?
 
-      @impuestos_node = children_nodes.select { |child| child.is_a?(Impuestos) }.first
+      @impuestos_node = children_nodes.select { |child| child.is_a?(ConceptoImpuestos) }.first
       return if @impuestos_node
 
-      @impuestos_node = Impuestos.new
+      @impuestos_node = ConceptoImpuestos.new
       @impuestos_node.parent_node = self
       children_nodes << @impuestos_node
       @impuestos_node

data/lib/cfdi40/concepto_impuestos.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+# /cfdi:Comprobante/cfdi:Conceptos/cfdiConcepto/cfdi:Impuestos
+module Cfdi40
+  class ConceptoImpuestos < Node
+    define_element_name "Impuestos"
+
+    def traslados
+      return @traslados if defined?(@traslados)
+
+      @traslados = Traslados.new
+      @traslados.parent_node = self
+      children_nodes << @traslados
+      @traslados
+    end
+
+    def traslados_node
+      children_nodes.select { |n| n.is_a?(Traslados) }.first
+    end
+
+    def traslado_nodes
+      return [] if traslados_node.nil?
+
+      traslados_node.traslado_nodes
+    end
+
+    def traslado_iva
+      traslados.traslado_iva
+    end
+  end
+end

data/lib/cfdi40/{totales.rb → cp_totales.rb}

@@ -1,17 +1,18 @@
 # 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"
+  class CpTotales < Node
+    define_element_name 'Totales'
+    define_attribute :ret_iva, xml_attribute: "TotalRetencionesIVA", format: :t_ImporteMXN
+    define_attribute :ret_isr, xml_attribute: "TotalRetencionesISR", format: :t_ImporteMXN
+    define_attribute :ret_ieps, xml_attribute: "TotalRetencionesIEPS", format: :t_ImporteMXN
+    define_attribute :base_iva16, xml_attribute: "TotalTrasladosBaseIVA16", format: :t_ImporteMXN
+    define_attribute :importe_iva16, xml_attribute: "TotalTrasladosImpuestoIVA16", format: :t_ImporteMXN
+    define_attribute :base_iva8, xml_attribute: "TotalTrasladosBaseIVA8", format: :t_ImporteMXN
+    define_attribute :importe_iva8, xml_attribute: "TotalTrasladosImpuestoIVA8", format: :t_ImporteMXN
+    define_attribute :base_iva0, xml_attribute: "TotalTrasladosBaseIVA0", format: :t_ImporteMXN
+    define_attribute :importe_iva0, xml_attribute: "TotalTrasladosImpuestoIVA0", format: :t_ImporteMXN
+    define_attribute :base_iva_excento, xml_attribute: "TotalTrasladosBaseIVAExento", format: :t_ImporteMXN
+    define_attribute :monto_total, xml_attribute: "MontoTotalPagos", format: :t_ImporteMXN
   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"
+    define_attribute :equivalencia_dr, xml_attribute: "EquivalenciaDR", default: "1", format: :decimal
     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.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
+# /cfdi:Comprobante/cfdi:Impuestos
 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_Importe
+    define_attribute :total_impuestos_trasladados, xml_attribute: "TotalImpuestosTrasladados", format: :t_Importe
 
     def traslados
       return @traslados if defined?(@traslados)

data/lib/cfdi40/node.rb

@@ -1,13 +1,19 @@
 # frozen_string_literal: true
 
 module Cfdi40
+  # Main class for build CFDi.
+  #
+  # Keeps definitions (names, accessors and formats) and
+  # relations (parent & children) between nodes
   class Node
     # Nokigiri XML Document for the xml_node
     attr_accessor :xml_document, :xml_parent, :children_nodes, :parent_node
     attr_writer :element_name
+    attr_reader :readonly
 
     def initialize
       self.class.verify_class_variables
+      @readonly = readonly
       @children_nodes = []
       set_defaults
     end
@@ -29,16 +35,78 @@ module Cfdi40
 
     def self.define_attribute(accessor, xml_attribute:, default: nil, format: nil, readonly: false)
       verify_class_variables
-      if readonly
-        attr_reader accessor.to_sym
-      else
-        attr_accessor accessor.to_sym
-      end
+      define_reader(accessor, format)
+      define_writer(accessor, readonly, format)
+
       @@attributes[name][accessor.to_sym] = xml_attribute
       @@default_values[name][accessor.to_sym] = default if default
       return unless format
 
-      @@formats[name][accessor.to_sym] = format
+      @@formats[name][accessor.to_sym] = format.to_sym
+    end
+
+    def self.define_reader(accessor, format)
+      case format.to_s
+      when 't_Importe', 'decimal'
+        define_method("#{accessor}".to_sym) do
+          value = instance_variable_defined?("@#{accessor}".to_sym) ? instance_variable_get("@#{accessor}".to_sym) : 0
+          value.to_f.round(6)
+        end
+      when 't_ImporteMXN'
+        define_method("#{accessor}".to_sym) do
+          value = instance_variable_defined?("@#{accessor}".to_sym) ? instance_variable_get("@#{accessor}".to_sym) : 0
+          value.to_f.round(2)
+        end
+      when 't_FechaH', 't_FechaHora'
+        define_method("#{accessor}".to_sym) do
+          value = instance_variable_defined?("@#{accessor}".to_sym) ? instance_variable_get("@#{accessor}".to_sym) : nil
+          return nil unless value.is_a?(Time)
+
+          value
+        end
+      else
+        define_method("#{accessor}".to_sym) do
+          value = instance_variable_defined?("@#{accessor}".to_sym) ? instance_variable_get("@#{accessor}".to_sym) : nil
+          return nil if value.nil?
+
+          value.to_s
+        end
+      end
+    end
+
+    def self.define_writer(accessor, readonly_attribute, format)
+      if readonly_attribute
+        define_method "#{accessor}=".to_sym do |value|
+          raise Cfdi40::Error, "attribute '#{accessor}' can not be modified"
+        end
+      else
+        case format.to_s
+        when 't_FechaH', 't_FechaHora'
+          define_method "#{accessor}=".to_sym do |value|
+            raise Cfdi40::Error, "CFDI is read only" if self.readonly
+
+            clean_cached_xml
+            if value.nil? || value.is_a?(Time)
+              instance_variable_set("@#{accessor}".to_sym, value)
+              return
+            end
+
+            begin
+              parsed_time = Time.strptime(value.to_s, "%Y-%m-%dT%H:%M:%S")
+              instance_variable_set("@#{accessor}".to_sym, parsed_time)
+            rescue
+              raise Cfdi40::Error, "#{value} must have format 'yyyy-mm-ddTHH:MM:SS'"
+            end
+          end
+        else
+          define_method "#{accessor}=".to_sym do |value|
+            raise Cfdi40::Error, "CFDI loaded in read only mode" if self.readonly
+
+            clean_cached_xml
+            instance_variable_set("@#{accessor}".to_sym, value)
+          end
+        end
+      end
     end
 
     def self.define_namespace(namespace, value)
@@ -95,6 +163,14 @@ module Cfdi40
       @children_nodes << child_node
     end
 
+    # Locks for readonly this node and children
+    def lock
+      @readonly = true
+      @children_nodes.each(&:lock)
+
+      true
+    end
+
     def current_namespace
       return unless self.class.respond_to?(:namespaces)
 
@@ -116,8 +192,6 @@ module Cfdi40
     end
 
     def create_xml_node
-      # TODO: Quitar la siguiente linea (set_defaults) si funciona poniendo los defaults en initialize
-      # set_defaults
       before_add if respond_to?(:before_add, true)
       xml_node = xml_document.create_element(expanded_element_name)
       add_namespaces_to(xml_node)
@@ -145,12 +219,14 @@ module Cfdi40
       end
     end
 
+    # Add defined attributes. Skip unused attributes
     def add_attributes_to(node)
       self.class.attributes.each do |object_accessor, xml_attribute|
         next unless respond_to?(object_accessor)
-        next if public_send(object_accessor).nil?
+        next unless instance_variable_defined?("@#{object_accessor}".to_sym)
+        next if instance_variable_get("@#{object_accessor}".to_sym).nil?
 
-        node[xml_attribute] = formated_value(object_accessor)
+        node[xml_attribute] = formatted_value(object_accessor)
       end
     end
 
@@ -162,15 +238,28 @@ module Cfdi40
       end
     end
 
-    def formated_value(accessor)
+    def formatted_value(accessor)
       case self.class.formats[accessor]
-      when :t_Importe
+      when :t_Importe, :decimal
         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" : format("%0.2f", public_send(accessor).to_f)
+      when :t_FechaH, :t_FechaHora
+        value = public_send(accessor)
+        value.is_a?(Time) ? value.strftime("%Y-%m-%dT%H:%M:%S") : ''
       else
         public_send(accessor)
       end
     end
+
+    # Cleans a Nokogiri object and/or loaded_xml string if exists
+    def clean_cached_xml
+      @docxml = nil
+      @loaded_xml = nil
+
+      parent_node&.clean_cached_xml
+    end
+
+
   end
 end

data/lib/cfdi40/original_content.rb

@@ -0,0 +1,23 @@
+# Extracts the "cadena original" from a XML
+module Cfdi40
+  class OriginalContent
+    LOCAL_XSLT_PATH = File.join(File.dirname(__FILE__), "..", "..", "lib/xslt/cadenaoriginal_local.xslt")
+
+    def self.generate(xml_string)
+      generator = new(xml_string)
+      generator.original_content
+    end
+
+    def initialize(xml)
+      @xml_doc = Nokogiri::XML(xml)
+    end
+
+    def original_content
+      xslt = Nokogiri::XSLT(File.open(LOCAL_XSLT_PATH))
+      transformed = xslt.transform(@xml_doc)
+      # The ampersand (&) char must be used in original content
+      # even though the documentation indicates otherwise
+      transformed.children.to_s.gsub("&", "&").strip
+    end
+  end
+end

data/lib/cfdi40/pago.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
+# /cfdi:Comprobante/cfdi:Complemento/pago20:Pagos/pago20:Pago
 module Cfdi40
   class Pago < Node
-    define_attribute :monto, xml_attribute: "Monto"
-    define_attribute :fecha_pago, xml_attribute: "FechaPago"
+    define_attribute :monto, xml_attribute: "Monto", format: :t_Importe
+    define_attribute :fecha_pago, xml_attribute: "FechaPago", format: :t_FechaH
     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"

data/lib/cfdi40/pagos.rb

@@ -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]
@@ -61,7 +61,7 @@ module Cfdi40
     def totales_node
       return @totales_node if defined?(@totales_node)
 
-      @totales_node = Totales.new
+      @totales_node = CpTotales.new
       add_child_node @totales_node
       @totales_node
     end

data/lib/cfdi40/sat_csd.rb

@@ -13,6 +13,12 @@ module Cfdi40
       @x509_cert = OpenSSL::X509::Certificate.new(data)
     end
 
+    # Loads certficate encoded in Base64.
+    # Certs with Base64 encoding are used in CFDIs
+    def cert64=(data)
+      self.cert_der = Base64.decode64(data)
+    end
+
     def load_private_key(key_path, key_pass)
       key_pem = key_to_pem(File.read(key_path))
       @private_key = OpenSSL::PKey::RSA.new(key_pem, key_pass)
@@ -70,7 +76,14 @@ module Cfdi40
     def subject_data
       return unless x509_cert
 
-      x509_cert.subject.to_a
+      # The #to_a method doesn't convert to_utf8
+      # https://ruby-doc.org/core-3.1.2/OpenSSL/X509/Name.html
+      #
+      # A simpliest
+      #     x509_cert.subject.to_a
+      # could't be used so, we need first convert to UTF-8
+
+      x509_cert.subject.to_utf8.split(',').map { |str| str.split '='  }
     end
 
     def key_to_pem(key_der)

data/lib/cfdi40/signature_validator.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+# Verfies signature
+module Cfdi40
+  class SignatureValidator
+    def initialize(xml)
+      @xml_doc = Nokogiri::XML(xml)
+
+    end
+
+    def valid?
+      original_content = OriginalContent.generate(@xml_doc.to_s)
+      cert.public_key.verify(OpenSSL::Digest.new('SHA256'), sign, original_content)
+    end
+
+    def sign
+      sign = @xml_doc.root.attributes["Sello"].to_s
+      raise Cfdi40::Error, 'CFDI is not signed' if sign == ''
+
+      Base64.decode64 sign
+    end
+
+    def cert
+      return @cert if defined?(@cert)
+
+      cert_string = @xml_doc.root.attributes["Certificado"].to_s
+      raise Cfdi40::Error, 'Certificate is not included in XML' if cert_string == ''
+      sat_csd = SatCsd.new
+      sat_csd.cert64 = cert_string
+      sat_csd.x509_cert.serial
+
+      if @xml_doc.root.attributes["NoCertificado"].to_s != sat_csd.no_certificado
+        raise Cfdi40::Error, 'Certificate number in XML does not correspond to the included certificate'
+      end
+
+      @cert = sat_csd.x509_cert
+    end
+  end
+end

data/lib/cfdi40/traslado.rb

@@ -1,11 +1,12 @@
 # frozen_string_literal: true
 
+# /cfdi:Comprobante/cfdi:Impuestos/cfdi:Traslados/cfdi:Traslado
 module Cfdi40
   class Traslado < Node
-    define_attribute :base, xml_attribute: "Base", format: :t_ImporteMXN
+    define_attribute :base, xml_attribute: "Base", format: :t_Importe
     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 :importe, xml_attribute: "Importe", format: :t_Importe
   end
 end

data/lib/cfdi40/traslados.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+# /cfdi:Comprobante/cfdi:Impuestos/cfdi:Traslados
 module Cfdi40
   class Traslados < Node
     def traslado_iva

data/lib/cfdi40/version.rb

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

data/lib/cfdi40/xml_loader.rb

@@ -4,10 +4,11 @@
 # from an XML (string)
 module Cfdi40
   class XmlLoader
-    attr_reader :cfdi, :xml_doc
+    attr_reader :cfdi, :xml_doc, :mode
 
-    def initialize(xml_string)
+    def initialize(xml_string, mode)
       @xml_doc = Nokogiri::XML(xml_string)
+      @mode = mode
       # TODO. validar versión del CFDI definido en xml_doc
       @cfdi = Cfdi40::Comprobante.new
       @cfdi.load_from_ng_node(xml_doc.root)
@@ -16,11 +17,20 @@ module Cfdi40
       load_receptor
       load_conceptos
       load_impuestos
+
+      @cfdi.lock if readonly?
+      @cfdi.loaded_xml = xml_string
       @cfdi
     end
 
     private
 
+    def readonly?
+      return true if mode == 'ro'
+
+      @xml_doc.root.attributes["Sello"].to_s != ''
+    end
+
     def load_conceptos
       n_concepto = 0
       xml_doc.xpath("//cfdi:Concepto").each do |node|

data/lib/cfdi40.rb

@@ -3,8 +3,11 @@
 require "nokogiri"
 require "base64"
 require "openssl"
+require "time"
 require_relative "cfdi40/version"
 require_relative "cfdi40/schema_validator"
+require_relative "cfdi40/signature_validator"
+require_relative "cfdi40/original_content"
 require_relative "cfdi40/sat_csd"
 require_relative "cfdi40/node"
 require_relative "cfdi40/comprobante"
@@ -12,6 +15,7 @@ require_relative "cfdi40/emisor"
 require_relative "cfdi40/receptor"
 require_relative "cfdi40/conceptos"
 require_relative "cfdi40/concepto"
+require_relative "cfdi40/concepto_impuestos"
 require_relative "cfdi40/impuestos"
 require_relative "cfdi40/traslados"
 require_relative "cfdi40/traslado"
@@ -27,7 +31,7 @@ require_relative "cfdi40/traslado_dr"
 require_relative "cfdi40/impuestos_p"
 require_relative "cfdi40/traslados_p"
 require_relative "cfdi40/traslado_p"
-require_relative "cfdi40/totales"
+require_relative "cfdi40/cp_totales"
 require_relative "cfdi40/xml_loader"
 
 # Leading module and entry point for all features and classes
@@ -42,8 +46,15 @@ module Cfdi40
     Comprobante.new
   end
 
-  def self.open(xml_string, mode: :readwrite)
-    loader = XmlLoader.new(xml_string)
+  # Modes:
+  # * 'rw' read and write
+  # * 'ro' read only
+  def self.open(xml_string, mode: 'rw')
+    unless mode.nil? || %w[rw ro].include?(mode)
+      STDERR.puts "Unknow mode '#{mode}' ignored. Valid modes are 'ro', 'rw'"
+      mode = nil
+    end
+    loader = XmlLoader.new(xml_string, mode)
     loader.cfdi
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cfdi40
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Israel Benítez
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2025-06-19 00:00:00.000000000 Z
+date: 2025-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -47,7 +47,9 @@ files:
 - lib/cfdi40/complemento_concepto.rb
 - lib/cfdi40/comprobante.rb
 - lib/cfdi40/concepto.rb
+- lib/cfdi40/concepto_impuestos.rb
 - lib/cfdi40/conceptos.rb
+- lib/cfdi40/cp_totales.rb
 - lib/cfdi40/docto_relacionado.rb
 - lib/cfdi40/emisor.rb
 - lib/cfdi40/impuestos.rb
@@ -55,12 +57,13 @@ files:
 - lib/cfdi40/impuestos_p.rb
 - lib/cfdi40/inst_educativas.rb
 - lib/cfdi40/node.rb
+- lib/cfdi40/original_content.rb
 - lib/cfdi40/pago.rb
 - lib/cfdi40/pagos.rb
 - lib/cfdi40/receptor.rb
 - lib/cfdi40/sat_csd.rb
 - lib/cfdi40/schema_validator.rb
-- lib/cfdi40/totales.rb
+- lib/cfdi40/signature_validator.rb
 - lib/cfdi40/traslado.rb
 - lib/cfdi40/traslado_dr.rb
 - lib/cfdi40/traslado_p.rb