Rollback calcular traslados sobre totales

satcfdi · satcfdi · commit 0afd08241f06 · 2025-10-15T14:47:25.000-06:00
diff --git a/satcfdi/create/cfd/cfdi40.py b/satcfdi/create/cfd/cfdi40.py
@@ -424,8 +424,7 @@ def __init__(
             cfdi_relacionados: CfdiRelacionados | Sequence[CfdiRelacionados | dict] = None,
             complemento: CFDI | Sequence[CFDI] = None,
             addenda: CFDI | Sequence[CFDI] = None,
-            fecha: datetime = None,
-            _traslados_sobre_totales: bool = False
+            fecha: datetime = None
     ):
         super().__init__({
             'Version': self.version,
@@ -456,14 +455,14 @@ def __init__(
             'NoCertificado': '',
             'Certificado': '',
         })
-        self.compute(traslados_sobre_totales=_traslados_sobre_totales)
+        self.compute()
 
-    def compute(self, traslados_sobre_totales=False):
+    def compute(self):
         self["Conceptos"] = conceptos = _make_conceptos(self["Conceptos"], rnd_fn=rounder(self["Moneda"]))
         self["SubTotal"] = sub_total = sum(c['Importe'] for c in conceptos)
         descuento = sum(c.get('Descuento') or 0 for c in conceptos)
         self['Descuento'] = descuento or None
-        self['Impuestos'] = impuestos = make_impuestos(conceptos, rnd_fn=rounder(self["Moneda"]), traslados_sobre_totales=traslados_sobre_totales)
+        self['Impuestos'] = impuestos = make_impuestos(conceptos)
         total = sub_total - descuento
         if impuestos:
             total += impuestos.get('TotalImpuestosTrasladados') or 0
diff --git a/satcfdi/create/compute.py b/satcfdi/create/compute.py
@@ -101,7 +101,7 @@ def group_impuestos(elements, pfx="", ofx=""):
     return impuestos
 
 
-def make_impuestos(conceptos, rnd_fn, traslados_sobre_totales):
+def make_impuestos(conceptos):
     impuestos = group_impuestos(conceptos)
 
     if retenciones := impuestos.get('Retenciones'):
@@ -111,10 +111,6 @@ def make_impuestos(conceptos, rnd_fn, traslados_sobre_totales):
             impuestos['TotalImpuestosRetenidos'] = sum(imp)
 
     if traslados := impuestos.get('Traslados'):
-        if traslados_sobre_totales:
-            for i in traslados:
-                if i["TipoFactor"] == 'Tasa':
-                    i["Importe"] = rnd_fn(i["Base"] * i["TasaOCuota"])
         impuestos['Traslados'] = traslados
         imp = list(i["Importe"] for i in traslados if i["Importe"] is not None)
         if imp:
diff --git a/tests/test_create_cfdi40.py b/tests/test_create_cfdi40.py
@@ -476,8 +476,7 @@ def test_suma_conceptos_iva():
                     )
                 ),
             )
-        ],
-        _traslados_sobre_totales=True
+        ]
     )
     invoice.sign(signer)
 
diff --git a/tests/test_create_cfdi40/iva_concepto_suma.html b/tests/test_create_cfdi40/iva_concepto_suma.html
@@ -350,15 +350,15 @@ <h5>Conceptos</h5>
 		</td>
 		<td>002</td>
 		<td>16.0000%</td>
-		<td>32.01</td>
+		<td>32.00</td>
 	</tr>
 	<tr class="r h">
 		<td>
 			<b>Total</b>
 		</td>
 		<td></td>
 		<td></td>
-		<td>232.07</td>
+		<td>232.06</td>
 	</tr>
 </table><div><br></div>
 </body>
diff --git a/tests/test_create_cfdi40/iva_concepto_suma.pretty.py b/tests/test_create_cfdi40/iva_concepto_suma.pretty.py
@@ -55,9 +55,9 @@
  'Fecha': datetime.datetime(2020, 1, 1, 22, 40, 38),
  'Folio': '123456',
  'FormaPago': None,
- 'Impuestos': {'TotalImpuestosTrasladados': Decimal('32.01'),
+ 'Impuestos': {'TotalImpuestosTrasladados': Decimal('32.00'),
                'Traslados': [{'Base': Decimal('200.06'),
-                              'Importe': Decimal('32.01'),
+                              'Importe': Decimal('32.00'),
                               'Impuesto': <Impuesto.IVA: '002'>,
                               'TasaOCuota': Decimal('0.160000'),
                               'TipoFactor': <TipoFactor.TASA: 'Tasa'>}]},
@@ -73,10 +73,10 @@
               'ResidenciaFiscal': None,
               'Rfc': 'KIJ0906199R1',
               'UsoCFDI': 'G03'},
- 'Sello': 'T59GluNZjmodDh0AJMV8LTvuw82HfKjtUVUfFjsahXdI/TBG4pVTsqT5gd5vRzV7/CxqVzjrIq2593sKOxS73OQWamwFx7dIXHwNKfQDVNdgBsi9uvwozrf3Bb0HUMk/1Rx8NyA3LKlm4PrdpG0bOvcPzAK6nsaKkxjDzVv4+cweGhaqKyna0YNaczYrvgCfMamKQd3T7mspbq7Ene0jmYK1WQLCEjBGnZqYmTJcJmrpxen9FZoseVD4dxYyAxRLmqAowWaZQJYfQSoVxltUrZVcdcqAP6W/awaxagpZHZ2clPJSQJper7tnK83t6Rc26TN9F4XoKqsz+b/UR0cw6g==',
+ 'Sello': 'NPiFQDW7WZP47PcvulN8YPtLzT2e5s74nzBtnl6bgx/tMKEam3NcYzjIuF09QGAWjiwUK8mR+fNWrMZJ6kgKAu6r2RfyPSL2GAOtFYnG5pz4uZBzG3lREBLY/tVcK4ID/SSzzRMZ38jk/dXZeMxy90Tu+kpAb6zG5xVaWhYpF9Ud40ukE345GloVDrQj1FwzW8MJBnKdKewRi3gstwvQLKkQHX1EPa0Ghs/crVTFpepHLHqdH62droEzl1YpPgwdAXBCR5XlWbrPCaTvw2VXB1eQG16WJ9baKn4udndig3doFElZd23fCnuutl3a8tfudlvabISWV+Z9iROQr350mw==',
  'Serie': 'A',
  'SubTotal': Decimal('200.06'),
  'TipoCambio': None,
  'TipoDeComprobante': 'I',
- 'Total': Decimal('232.07'),
+ 'Total': Decimal('232.06'),
  'Version': '4.0'}
diff --git a/tests/test_create_cfdi40/iva_concepto_suma.xml b/tests/test_create_cfdi40/iva_concepto_suma.xml
@@ -1,5 +1,5 @@
 <?xml version='1.0' encoding='UTF-8'?>
-<cfdi:Comprobante xmlns:cfdi="http://www.sat.gob.mx/cfd/4" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" Version="4.0" Serie="A" Folio="123456" Fecha="2020-01-01T22:40:38" Sello="T59GluNZjmodDh0AJMV8LTvuw82HfKjtUVUfFjsahXdI/TBG4pVTsqT5gd5vRzV7/CxqVzjrIq2593sKOxS73OQWamwFx7dIXHwNKfQDVNdgBsi9uvwozrf3Bb0HUMk/1Rx8NyA3LKlm4PrdpG0bOvcPzAK6nsaKkxjDzVv4+cweGhaqKyna0YNaczYrvgCfMamKQd3T7mspbq7Ene0jmYK1WQLCEjBGnZqYmTJcJmrpxen9FZoseVD4dxYyAxRLmqAowWaZQJYfQSoVxltUrZVcdcqAP6W/awaxagpZHZ2clPJSQJper7tnK83t6Rc26TN9F4XoKqsz+b/UR0cw6g==" NoCertificado="30001000000400002311" Certificado="MIIF1DCCA7ygAwIBAgIUMzAwMDEwMDAwMDA0MDAwMDIzMTEwDQYJKoZIhvcNAQELBQAwggErMQ8wDQYDVQQDDAZBQyBVQVQxLjAsBgNVBAoMJVNFUlZJQ0lPIERFIEFETUlOSVNUUkFDSU9OIFRSSUJVVEFSSUExGjAYBgNVBAsMEVNBVC1JRVMgQXV0aG9yaXR5MSgwJgYJKoZIhvcNAQkBFhlvc2Nhci5tYXJ0aW5lekBzYXQuZ29iLm14MR0wGwYDVQQJDBQzcmEgY2VycmFkYSBkZSBjYWRpejEOMAwGA1UEEQwFMDYzNzAxCzAJBgNVBAYTAk1YMRkwFwYDVQQIDBBDSVVEQUQgREUgTUVYSUNPMREwDwYDVQQHDAhDT1lPQUNBTjERMA8GA1UELRMIMi41LjQuNDUxJTAjBgkqhkiG9w0BCQITFnJlc3BvbnNhYmxlOiBBQ0RNQS1TQVQwHhcNMTkwNTI4MjE1ODQyWhcNMjMwNTI3MjE1ODQyWjCByTEeMBwGA1UEAxMVQkVSRU5JQ0UgWElNTyBRVUVaQURBMR4wHAYDVQQpExVCRVJFTklDRSBYSU1PIFFVRVpBREExHjAcBgNVBAoTFUJFUkVOSUNFIFhJTU8gUVVFWkFEQTELMAkGA1UEBhMCTVgxJTAjBgkqhkiG9w0BCQEWFnBydWViYXNAcHJ1ZWJhcy5nb2IubXgxFjAUBgNVBC0TDVhJUUI4OTExMTZRRTQxGzAZBgNVBAUTElhJUUI4OTExMTZNR1JNWlIwNTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBAJJLU6Iu3cOyJ14hkh9mjCy3XN5i1M36+60vnwTap8Uv1vQQxJqIB4WK85CSJxujhZE0XY2OT6QPJMQ3kqcuMk8Yz+KptHq51Uhs4jiShI0GGoVUqZ/8qSmY+DnYM/WGvSzuaMMm+cMwgMVYusKHE5FI+K39ht9aSP045KIR84ImnDuVp6DJeUaHEtlqspnQfMvZHfmT71IT7niBJOWYThjmCKMX6Y5tnlng8pOs9hjwHzlpUhVlGRjjUSJjeIZZY66kgbQcpOFCUsoyR2SbOZDMWSqlsXNsRZ5sjICxFrMqZ6GoCzobx7tn6WYisTVvMIvkNHzi8a8idyAMVrDhNb0CAwEAAaNPME0wDAYDVR0TAQH/BAIwADALBgNVHQ8EBAMCA9gwEQYJYIZIAYb4QgEBBAQDAgWgMB0GA1UdJQQWMBQGCCsGAQUFBwMEBggrBgEFBQcDAjANBgkqhkiG9w0BAQsFAAOCAgEAFFyKKTbclgM9SellL9BCWusArCHwtERUXnaFzQzVbcXzZ9rrXoiXLo8OCCkE5fT4ig8FcWiH1bkh6ZABQSHKcEEY9ewJWZpD3bA8E7DLHfDHmG1EqRKStajIG1aq3GSvFAetMvJE2epFsWvkO7CkN2hNT9SwsFkBYl+aiCP27LzQ4uD+mXJ6Pg79nD/ayuQDa7RGJ6u5KHY1I66xcpVxOegQr+zolYQ6RfAye6Fk6JxUnYBVXtnsC3GLpB0fMb+53NZGzARVD9/762A3dy3GHlMWs9FVoRN7DBJyNsVSlAodi9ClETC3Tr7WuXIzPZTbu17JEcqgL6Ovrk6L/SLULvhDe89eBmZHGIm5Vaye2X1OeY+CmpKZ0nMzu16+hOhE3ecRTmxHFUCWzyHuH8qyBetiJY64vyHMwgbyqwM3Lk7lEQowhYb4s8vGyJgb0qQoyt0lACbtkM73CwzasPTtxnHZFKzt0C9aYIZvSVQidmK5EbGNh5YWL8tk352Sqjk4yxlZRqH8SkKHoMjPOrTcBJJwzWWQtz9hOl78kLqcYi+TK4ZWvunGXtZqDyQ2omhZBokSAHqUDjcRmOnuMpazF68j8U73Bz2voQtimMJlB/yyT6luFlzUbEK3ckUYBkk0PKxDe/6T7NXj+H4UWhpTivKnrqNWL7qAHLSP1tnmG8s=" SubTotal="200.06" Moneda="MXN" Total="232.07" TipoDeComprobante="I" Exportacion="01" MetodoPago="PPD" LugarExpedicion="56820" xsi:schemaLocation="http://www.sat.gob.mx/cfd/4 http://www.sat.gob.mx/sitio_internet/cfd/4/cfdv40.xsd">
+<cfdi:Comprobante xmlns:cfdi="http://www.sat.gob.mx/cfd/4" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" Version="4.0" Serie="A" Folio="123456" Fecha="2020-01-01T22:40:38" Sello="NPiFQDW7WZP47PcvulN8YPtLzT2e5s74nzBtnl6bgx/tMKEam3NcYzjIuF09QGAWjiwUK8mR+fNWrMZJ6kgKAu6r2RfyPSL2GAOtFYnG5pz4uZBzG3lREBLY/tVcK4ID/SSzzRMZ38jk/dXZeMxy90Tu+kpAb6zG5xVaWhYpF9Ud40ukE345GloVDrQj1FwzW8MJBnKdKewRi3gstwvQLKkQHX1EPa0Ghs/crVTFpepHLHqdH62droEzl1YpPgwdAXBCR5XlWbrPCaTvw2VXB1eQG16WJ9baKn4udndig3doFElZd23fCnuutl3a8tfudlvabISWV+Z9iROQr350mw==" NoCertificado="30001000000400002311" Certificado="MIIF1DCCA7ygAwIBAgIUMzAwMDEwMDAwMDA0MDAwMDIzMTEwDQYJKoZIhvcNAQELBQAwggErMQ8wDQYDVQQDDAZBQyBVQVQxLjAsBgNVBAoMJVNFUlZJQ0lPIERFIEFETUlOSVNUUkFDSU9OIFRSSUJVVEFSSUExGjAYBgNVBAsMEVNBVC1JRVMgQXV0aG9yaXR5MSgwJgYJKoZIhvcNAQkBFhlvc2Nhci5tYXJ0aW5lekBzYXQuZ29iLm14MR0wGwYDVQQJDBQzcmEgY2VycmFkYSBkZSBjYWRpejEOMAwGA1UEEQwFMDYzNzAxCzAJBgNVBAYTAk1YMRkwFwYDVQQIDBBDSVVEQUQgREUgTUVYSUNPMREwDwYDVQQHDAhDT1lPQUNBTjERMA8GA1UELRMIMi41LjQuNDUxJTAjBgkqhkiG9w0BCQITFnJlc3BvbnNhYmxlOiBBQ0RNQS1TQVQwHhcNMTkwNTI4MjE1ODQyWhcNMjMwNTI3MjE1ODQyWjCByTEeMBwGA1UEAxMVQkVSRU5JQ0UgWElNTyBRVUVaQURBMR4wHAYDVQQpExVCRVJFTklDRSBYSU1PIFFVRVpBREExHjAcBgNVBAoTFUJFUkVOSUNFIFhJTU8gUVVFWkFEQTELMAkGA1UEBhMCTVgxJTAjBgkqhkiG9w0BCQEWFnBydWViYXNAcHJ1ZWJhcy5nb2IubXgxFjAUBgNVBC0TDVhJUUI4OTExMTZRRTQxGzAZBgNVBAUTElhJUUI4OTExMTZNR1JNWlIwNTCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBAJJLU6Iu3cOyJ14hkh9mjCy3XN5i1M36+60vnwTap8Uv1vQQxJqIB4WK85CSJxujhZE0XY2OT6QPJMQ3kqcuMk8Yz+KptHq51Uhs4jiShI0GGoVUqZ/8qSmY+DnYM/WGvSzuaMMm+cMwgMVYusKHE5FI+K39ht9aSP045KIR84ImnDuVp6DJeUaHEtlqspnQfMvZHfmT71IT7niBJOWYThjmCKMX6Y5tnlng8pOs9hjwHzlpUhVlGRjjUSJjeIZZY66kgbQcpOFCUsoyR2SbOZDMWSqlsXNsRZ5sjICxFrMqZ6GoCzobx7tn6WYisTVvMIvkNHzi8a8idyAMVrDhNb0CAwEAAaNPME0wDAYDVR0TAQH/BAIwADALBgNVHQ8EBAMCA9gwEQYJYIZIAYb4QgEBBAQDAgWgMB0GA1UdJQQWMBQGCCsGAQUFBwMEBggrBgEFBQcDAjANBgkqhkiG9w0BAQsFAAOCAgEAFFyKKTbclgM9SellL9BCWusArCHwtERUXnaFzQzVbcXzZ9rrXoiXLo8OCCkE5fT4ig8FcWiH1bkh6ZABQSHKcEEY9ewJWZpD3bA8E7DLHfDHmG1EqRKStajIG1aq3GSvFAetMvJE2epFsWvkO7CkN2hNT9SwsFkBYl+aiCP27LzQ4uD+mXJ6Pg79nD/ayuQDa7RGJ6u5KHY1I66xcpVxOegQr+zolYQ6RfAye6Fk6JxUnYBVXtnsC3GLpB0fMb+53NZGzARVD9/762A3dy3GHlMWs9FVoRN7DBJyNsVSlAodi9ClETC3Tr7WuXIzPZTbu17JEcqgL6Ovrk6L/SLULvhDe89eBmZHGIm5Vaye2X1OeY+CmpKZ0nMzu16+hOhE3ecRTmxHFUCWzyHuH8qyBetiJY64vyHMwgbyqwM3Lk7lEQowhYb4s8vGyJgb0qQoyt0lACbtkM73CwzasPTtxnHZFKzt0C9aYIZvSVQidmK5EbGNh5YWL8tk352Sqjk4yxlZRqH8SkKHoMjPOrTcBJJwzWWQtz9hOl78kLqcYi+TK4ZWvunGXtZqDyQ2omhZBokSAHqUDjcRmOnuMpazF68j8U73Bz2voQtimMJlB/yyT6luFlzUbEK3ckUYBkk0PKxDe/6T7NXj+H4UWhpTivKnrqNWL7qAHLSP1tnmG8s=" SubTotal="200.06" Moneda="MXN" Total="232.06" TipoDeComprobante="I" Exportacion="01" MetodoPago="PPD" LugarExpedicion="56820" xsi:schemaLocation="http://www.sat.gob.mx/cfd/4 http://www.sat.gob.mx/sitio_internet/cfd/4/cfdv40.xsd">
   <cfdi:Emisor Rfc="XIQB891116QE4" Nombre="BERENICE XIMO QUEZADA" RegimenFiscal="601"/>
   <cfdi:Receptor Rfc="KIJ0906199R1" Nombre="KIJ, S.A DE C.V." DomicilioFiscalReceptor="59820" RegimenFiscalReceptor="601" UsoCFDI="G03"/>
   <cfdi:Conceptos>
@@ -18,9 +18,9 @@
       </cfdi:Impuestos>
     </cfdi:Concepto>
   </cfdi:Conceptos>
-  <cfdi:Impuestos TotalImpuestosTrasladados="32.01">
+  <cfdi:Impuestos TotalImpuestosTrasladados="32.00">
     <cfdi:Traslados>
-      <cfdi:Traslado Base="200.06" Impuesto="002" TipoFactor="Tasa" TasaOCuota="0.160000" Importe="32.01"/>
+      <cfdi:Traslado Base="200.06" Impuesto="002" TipoFactor="Tasa" TasaOCuota="0.160000" Importe="32.00"/>
     </cfdi:Traslados>
   </cfdi:Impuestos>
 </cfdi:Comprobante>

Original file line number	Diff line number	Diff line change
`@@ -476,8 +476,7 @@ def test_suma_conceptos_iva():`
`476`	`476`	`)`
`477`	`477`	`),`
`478`	`478`	`)`
`479`		`- ],`
`480`		`- _traslados_sobre_totales=True`
	`479`	`+ ]`
`481`	`480`	`)`
`482`	`481`	`invoice.sign(signer)`
`483`	`482`