[12.0][IMP] l10n_es_aeat_mod303: update 2024-10 #3779
Conversation
| aeat_mod303_202107_map_line_120,aeat_mod303_202107_map,120,Operaciones no sujetas por reglas de localización (excepto las incluidas en la casilla 123),0,all,base,both,0,l10n_es.account_tax_template_s_iva_e | ||
| aeat_mod303_202107_map_line_122,aeat_mod303_202107_map,122,Operaciones sujetas con inversión del sujeto pasivo,0,all,base,both,0,l10n_es.account_tax_template_s_iva0_isp | ||
| aeat_mod303_202107_map_line_125,aeat_mod303_202107_map,125,Operaciones sujetas con inversión del sujeto pasivo,0,all,base,both,0,l10n_es.account_tax_template_s_iva0_isp | ||
| id,map_parent_id:id,field_number,name,to_regularize,move_type,field_type,sum_type,inverse,account_id:id,tax_ids/id |
There was a problem hiding this comment.
2024-10-16 17:43:01,305 1464 INFO odoo odoo.modules.loading: loading l10n_es_aeat_mod303/data/2021-07/l10n.es.aeat.map.tax.line.csv
...
KeyError: 'account_id'
| aeat_mod303_2023_map_line_152,aeat_mod303_2023_map,152,Régimen General - Cuota 0%,1,regular,amount,both,0, | ||
| aeat_mod303_2023_map_line_153,aeat_mod303_2023_map,153,Régimen General - Base imponible 5%,0,regular,base,both,0, | ||
| aeat_mod303_2023_map_line_155,aeat_mod303_2023_map,155,Régimen General - Cuota 5%,1,regular,amount,both,0, | ||
| id,map_parent_id:id,field_number,name,to_regularize,move_type,field_type,sum_type,inverse,account_id:id,tax_ids/id |
There was a problem hiding this comment.
Saldrá el mismo problema que en 2021.
| @@ -0,0 +1,63 @@ | |||
| id,map_parent_id:id,field_number,name,to_regularize,move_type,field_type,sum_type,inverse,account_id:id,tax_ids/id | |||
There was a problem hiding this comment.
Saldrá el mismo problema que en 2021.
|
El error viene dado por que en la migración de 12 a 13 del l10n_es_aeat se aplico un cambio importante en el propio commit de migración que provoca esta diferencia. |
|
¿Estáis en proceso de corregirlo? |
Yo estoy con la 11.0. |
|
@etobella ¿tienes en mente modificarlo? |
etobella
force-pushed
the
12.0-303
branch
6 times, most recently
from
7d4b5fd to
8fdd714
Compare
|
La verdad es que realmente no tenia la necesidad de hacerlo, pero como pequeño favor he aproximado la solución |
|
Muchas gracias @etobella. Realmente, te preguntaba para saber si estabas con ello o nos poníamos nosotros. |
|
Pues entonces mejor si lo acabais vosotros, yo no tengo instancias en 12 para instalarlo |
|
@rlizana Al rescate. Me iba a poner a ello y estaba dudando entre seguir con esto o mirar el PR de la versión 11.0. Gracias Roberto!. Probado y por mi parte OK |
|
Nosotros estamos haciendo pruebas y de momento nos falla en v11, tampoco sé cómo de relacionado está este PR con #3785 |
|
@etobella @ljsalvatierra-factorlibre Se ha mergeado ya? |
|
@josepmy falta que la gente haga pruebas para poder proceder al merge |
Nosotros aún seguimos con las pruebas, hemos encontrado varios errores y tenemos pensado solucionarlos a lo largo de esta semana. |
|
Buenas, ¿ se sabe cuando va a estar disponible? |
|
@ljsalvatierra-factorlibre podemos ayudar en algo para avanzar? |
|
Solo falta probarlo, yo he presentado un par de 303 y sin problemas, pero es necesario que alguien más lo valide. |
Hola @josepmy nosotros hemos realizado el backport a v11 aquí #3785 |
|
@pedrobaeza a que te refieres con "lo nuevo" ? |
|
Los mapeos de los impuestos con children_tax_ids están puestos en el nuevo mapeo al padre, en vez de a los hijos. P.ej. IVA 21% Adquisición Intracomunitaria. Bienes corrientes = account_tax_template_p_iva21_ic_bc. Esto hace que aparezcan bases (porque se lleva el padre también a tax_ids en account_move_line), pero no cuotas (porque sólo es un m2o, el tax_line_id). En el PR de la 11 #3785 está correcto |
|
He hecho un fork en https://github.com/Digital5-Odoo/l10n-spain/tree/12.0-fix-l10n_es_aeat_mod303-PR3779 |
|
Basado en el e @jonerikceberio añadidos unos pequeños ajustes en https://github.com/Comunitea/l10n-spain/tree/12.0-fixes-new-l10n_es_aeat_mod303-PR3779 |
|
He probado el fork y es funcional. Podéis tirarlo adelante y zanjamos esto? |
|
@cubells está pendiente de runboat |
|
@etobella puedes hacer push para relanzar el runboat para probarlo. Gracias! |
|
El error es del 390.... |
@pedrobaeza Teniendo en cuenta este comentario #3761 (comment) , podemos dejarlo así? Piensa que no deja de ser el mismo cambio @ljsalvatierra-factorlibre ¿Puedes revisar ya que requeriste cambios y todos se han aplicado? |
|
Pero eso vale para la 14, pero aquí en esta versión tan antigua no deberíamos cambiar las dependencias, y más porque ya existen esos módulos creados. No hay que crearlos desde 0. |
|
En la 14 se hizo exactamente eso, se pidió generar el módulo (por que existe el extra_data y se cambió la dependencia) y luego no |
|
y, de hecho, es el PR del que viene este backport... |
Correcto, se indicó que si al principio y luego se decidió de que como eran ramas tan antiguas y ya no se iban a seguir que se permitía no hacer el módulo extra e integrarlo directamente |
|
Realmente es diferente: en 14.0, no existe Pero bueno, si todos los que seguís estancados en esta versión estáis de acuerdo, OK al cambio. Yo ya migré hace tiempo. @rlizana ¿estás tú también de acuerdo? |
|
Ok por nuestra parte también. |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 790dee5. Thanks a lot for contributing to OCA. ❤️ |
Backport de #3776