cfdi40 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c93f265a23f6c466d800ce90e2258525844b3f15451971b0140f8c2ee826d2c5
-  data.tar.gz: 1fcd506dfa5eefbffa1fec8ad3e98543cbc0045936859ca171b42caf46ff6649
+  metadata.gz: d48bd150dec201d3a2b0812d49080313c5e4aec6efd836e5606e9e2edabd822f
+  data.tar.gz: 3f437736ebc91684c7652f217f64a0e04c9e8b18519a18a02397ab4f95ca8db9
 SHA512:
-  metadata.gz: 8a13afec5754aeb24cf6168f66b23b658e79cb8d8cbf93c74a11bc592ae5913f77b3d6d5da45e19b69384c94a9221bb4f7b9897e0bc6672f7360b71a0538d759
-  data.tar.gz: 473a8ce2434b628e1fb61e8342cbdcdc1972fb00350a382e5b2b113c0d16018b804cd6d05438a2b26ddb54ba121491f9df94a75f976247eea682d7fcd7e461f2
+  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.4)
+    cfdi40 (0.0.5)
       nokogiri (>= 1.10.10)
 
 GEM

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

@@ -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

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/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.4"
+  VERSION = "0.0.5"
 end