cfdi40 0.0.3 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb9a12ec72926735a554f302b13993914393c626cc4dcd797791dfcf0ed8191a
-  data.tar.gz: 4bc4708eb4c2c0671fa11c81dbcb40a8a8c9a585d2fa02531137efeee942a9d0
+  metadata.gz: d48bd150dec201d3a2b0812d49080313c5e4aec6efd836e5606e9e2edabd822f
+  data.tar.gz: 3f437736ebc91684c7652f217f64a0e04c9e8b18519a18a02397ab4f95ca8db9
 SHA512:
-  metadata.gz: f41d5ee33e1e705d2b12b2c623cd6b197d2c93c1a9e5d994ffc38630954fc4cb6ed6a1c2ae084513bfa6991f7c1bf67b4cc609ebbca3c2d1c99d74167897f470
-  data.tar.gz: 9f55e7a93233c1c8121d26424c2a7604d36b58f6695b104dc64ea39091e16bb45f3da10231ff4367e75c56d843fd724284b2adae053617244a2295227d3bc41d
+  metadata.gz: 4491ff366eeb72c4c6d1792227750e5c3f333d6df52f7d974c05eff8d4d94f30a44e98dfefaf03b1aaa41a8d1407e2765a3ef026617faa071fb86049cbbec9f9
+  data.tar.gz: 77c1ee1687c4b13af13768afa7a5b3ee0c7ab2071a4a3ddbf45d3bedcf4a18bb2a8e56b07bdf1762d6e833c052cf77c1b3272b04a75cb3f28d2c0494b76d66af

data/CHANGELOG.md

@@ -1,5 +1,7 @@
 # Cfdi40
 
+Please see `README_es-MX.md`
+
 ## [Unreleased]
 
 ### Added

data/Gemfile.lock

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

data/README.md

@@ -12,6 +12,8 @@ TODO: Document, document, document
 ## Features
 
 * XML generation and sign.
+* Node 'iedu'
+* Concept is calculated from gross price or net price
 
 ## Future features
 

data/README_es-MX.md

@@ -14,7 +14,7 @@ que esta herramienta:
   información esencial para la elaboración del CFDi.
 * Realice los cálculos complementarios como impuestos,
   totales, etcétera.
-* Valide el CFDi contra los CSD
+* Valide el CFDi contra los XSD
 * Selle el CFDi.
 
 # Uso
@@ -55,6 +55,23 @@ que esta herramienta:
 
 # Cambios
 
+# 0.0.5
+
+* IVA tasa 0 e IVA excento. Los conceptos con iva tasa 0 deben llevar el
+  nodo de traslados de impuestos. Los conceptos excentos de iva no
+  llevan el nodo de traslado. Al agregar el concepto usar `nil` para el
+  IVA cuando sea excento.
+* Nuevo cálculo de impuestos e importes. El PAC solo acepta 2 decimales
+  en los nodos de traslado. Esto puede generar discrepancias al sumar
+  los conceptos si no se maneja el mismo número de decimales. Solamente
+  el valor unitario se manejará con 6 decimales.
+
+# 0.0.4
+
+* Atributos 'Total' y 'Subtotal' van con 2 decimales. Aunque el anexo 20
+  indica que el tipo es `t_Importe` (6 decimales) el PAC acepta solo 2
+  decimales si el CFDi está en pesos mexicanos.
+
 # 0.0.3
 
 * Lee RFC en certificados de personas morales. Los certificados de

data/lib/cfdi40/comprobante.rb

@@ -19,11 +19,11 @@ module Cfdi40
     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_Importe
-    define_attribute :descuento, xml_attribute: 'Descuento', format: :t_Importe
+    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_Importe
+    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'
@@ -60,7 +60,7 @@ module Cfdi40
       @sat_csd ||= SatCsd.new
       @sat_csd.cert_der = cert_data
       emisor.rfc = @sat_csd.rfc
-      emisor.nombre = @sat_csd.name
+      emisor.nombre ||= @sat_csd.name
       @no_certificado = @sat_csd.no_certificado
       @certificado = @sat_csd.cert64
       true
@@ -82,18 +82,42 @@ module Cfdi40
       @docxml = nil
     end
 
-    # clave_prod_serv
-    # no_identificacion
-    # cantidad
-    # clave_unidad
-    # unidad
-    # descripcion
-    # valor_unitario
-    # importe
-    # descuento
-    # objeto_imp
+    # ## Required attributes
+    #
+    # +clave_prod_serv+:: From SAT catalogue
+    # +clave_unidad+:: From SAT catalogue
+    # +cantidad+:: Must be greather than 0
+    # +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.
+    #                  All quantities are calculated based on this price and taxes rate.
+    # +precio_neto+:: Precio after taxes or net price. All quantities are calculated from this prices.
+    #                 When both, +precio_neto+ and +precio_bruto+ exist, +precio_neto+ is used
+    #
+    # The most common usage requires only the net price (+precio_neto+).
+    #
+    # ## Optional attributes:
+    # +no_identificacion+::
+    # +unidad+::
+    # +descuento+:: PENDING
+    #
+    # ## Special attributes
+    # 
+    # ### IEDU attributes
+    #
+    # IEDU node (path: cfdi:Comprobante/cfdi:Conceptos/cfdi:Concepto/cfdi:ComplementoConcepto/iedu:instEducativas) is
+    # generated when one of +iedu_nombre_alumno+, +iedu_curp+, +iedu_nivel_educativo+ exist.
+    #
+    # +iedu_nombre_alumno+::
+    # +iedu_curp+::
+    # +iedu_nivel_educativo+::
+    # +iedu_aut_rvoe+::
+    # +iedu_rfc_pago+::
     #
-    # TODO: Document accepted attributes and its use
     def add_concepto(attributes = {})
       concepto = Concepto.new
       concepto.parent_node = @conceptos
@@ -133,7 +157,8 @@ module Cfdi40
     end
 
     def original_content
-      xslt = Nokogiri::XSLT(File.open('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
@@ -226,10 +251,10 @@ module Cfdi40
     end
 
     def load_private_key
-      return unless defined?(@key_data) && defined?(@key_pass)
+      return unless defined?(@key_data)
       
       @sat_csd ||= SatCsd.new
-      @sat_csd.set_private_key(@key_data, @key_pass)
+      @sat_csd.set_private_key(@key_data, (defined?(@key_pass) ? @key_pass : nil))
     end
   end
 end

data/lib/cfdi40/concepto.rb

@@ -1,6 +1,6 @@
 # Represents node 'concepto'
 #
-# * Attribute +Importe+ represente gross amount. Gross amount id before taxes and the result of multiply
+# * Attribute +Importe+ represente gross amount. Gross amount is before taxes and the result of multiply
 #   +ValorUnitario+ by +Cantidad+
 #   
 module Cfdi40
@@ -24,7 +24,7 @@ module Cfdi40
 
     def initialize
       @tasa_iva = 0.16
-      @tasa_ieps = 0
+      @tasa_ieps = nil
       super
     end
 
@@ -33,6 +33,7 @@ module Cfdi40
     def calculate!
       set_defaults
       assign_objeto_imp
+      # TODO: accept discount
       if defined?(@precio_neto) && !@precio_neto.nil?
         calculate_from_net_price
       elsif defined?(@precio_bruto) && !@precio_bruto.nil?
@@ -61,33 +62,26 @@ module Cfdi40
 
     def calculate_from_net_price
       set_defaults
-      @importe_neto = precio_neto * cantidad
-      breakdown_taxes
+      @precio_neto = @precio_neto.round(2)
+      @precio_bruto = (@precio_neto / ((1 + tasa_iva.to_f) * (1 + tasa_ieps.to_f))).round(6)
+      calculate_taxes
       update_xml_attributes
     end
 
-    def breakdown_taxes
-      @base_iva = @importe_neto / (1 + tasa_iva)
-      @iva = @importe_neto - @base_iva
-      @base_ieps = @base_iva / (1 + tasa_ieps)
-      @ieps = @base_iva - @base_ieps 
-      @importe_bruto = @base_ieps
-      @precio_bruto = @importe_bruto / @cantidad
-    end
-
     def calculate_from_gross_price
-      @importe_bruto = @precio_bruto * cantidad
-      add_taxes
+      @precio_bruto = @precio_bruto.round(6)
+      calculate_taxes
+      @precio_neto = (@importe_neto / cantidad).round(2)
       update_xml_attributes
     end
 
-    def add_taxes
-      @base_ieps = @importe_bruto
-      @ieps = @base_ieps * tasa_ieps
-      @base_iva = @base_ieps + @ieps
-      @iva = @base_iva * tasa_iva
-      @importe_neto = @base_iva + @iva
-      @precio_neto = @importe_neto / cantidad
+    def calculate_taxes
+      @base_ieps = (@precio_bruto * cantidad).round(2)
+      @ieps = (@base_ieps * tasa_ieps.to_f).round(2)
+      @base_iva = (@base_ieps + @ieps).round(2)
+      @iva = (@base_iva * tasa_iva.to_f).round(2)
+      @importe_bruto = @base_ieps
+      @importe_neto = (@base_iva + @iva).round(2)
     end
 
     def update_xml_attributes
@@ -96,7 +90,7 @@ module Cfdi40
     end
 
     def add_info_to_traslado_iva
-      return unless @iva > 0
+      return if @tasa_iva.nil?
 
       traslado_iva_node.importe = @iva
       traslado_iva_node.base = @base_iva
@@ -104,9 +98,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.
+
       return if objeto_impuestos == '03'
 
-      self.objeto_impuestos = (@tasa_iva > 0 || @tasa_ieps > 0 ? '02' : '01')
+      self.objeto_impuestos = (!@tasa_iva.nil? || !@tasa_ieps.nil? ? '02' : '01')
     end
 
     def impuestos_node

data/lib/cfdi40/impuestos.rb

@@ -1,7 +1,7 @@
 module Cfdi40
   class Impuestos < Node
-    define_attribute :total_impuestos_retenidos, xml_attribute: 'TotalImpuestosRetenidos', format: :t_Importe
-    define_attribute :total_impuestos_trasladados, xml_attribute: 'TotalImpuestosTrasladados', format: :t_Importe
+    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)

data/lib/cfdi40/node.rb

@@ -147,6 +147,8 @@ module Cfdi40
       case self.class.formats[accessor]
       when :t_Importe
         sprintf("%0.6f", public_send(accessor).to_f)
+      when :t_ImporteMXN
+        sprintf("%0.2f", public_send(accessor).to_f)
       else
         public_send(accessor)
       end

data/lib/cfdi40/traslado.rb

@@ -1,9 +1,9 @@
 module Cfdi40
   class Traslado < Node
-    define_attribute :base, xml_attribute: 'Base', format: :t_Importe
+    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_Importe
+    define_attribute :importe, xml_attribute: 'Importe', format: :t_ImporteMXN
   end
 end

data/lib/cfdi40/version.rb

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