[17.0][MIG] l10n_es_aeat_mod349: Migration to version 17.0 #3503

manuelregidor · 2024-03-27T09:17:33Z

No description provided.

…c module for aeat models, 347 module was portedto v6.0 and adds new module to print AEAT model 349.

…ueñas mejoras generales (vistas, traducciones, código) y corrección pequeño bug por un olvido en la adaptación de la v5 a la v6

* Only use the first four digits of fiscal year's code to determine fiscal year's number and also give a decent message to the user if it's not a number instead of simply crashing. * Hacer opcional el campo Operation Key en las facturas, permitiendo que esté en blanco. * Ser un poco menos rigurosos con que el año esté en el código del año fiscal. Atención: deberíamos añadir un campo con este valor en lugar de hacer este hack.

* FIX: Corregimos una serie de errores que había en las rectificaciones a la hora de obtener las declaraciones originales, si había alguna que cumplía los requisitos pero sin pertenecer a un modelo. Quizás se solucionara mejor con un on_delete='cascade'. * RFR: Cambio de licencia de GPL3 a AGPL3 en el resto de módulos de localización española, con el consentimiento de todos los autores expresado vía lista de correo del equipo de localización openerp-spain@googlegroups.com

* Only use the first four digits of fiscal year's code to determine fiscal year's number and also give a decent message to the user if it's not a number instead of simply crashing. * Resolve a problem with python isinstance which considers 'False' is an integer and which makes OpenERP create operation_key field as integer instead of char. * Convert a return None to return False, because the XML-RPC protocol doesn't support returning None.

…or: global name 'pooler' is not defined" al iniciar el calculo.

… sólo uno y arrelgar un bug en la generación del modelo cuando el partner tiene el campo title lleno.

…y H que son respectivamente Entregas intracomunitarias con exención de tasas y Entregas intracomunitarias con exención de tasas por un representante fiscal oficial fueron publicadas el 26/11/2011 en el boe. Se puede ver el artículo en: http://planartus.blogspot.es/1309888740/

…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True

…n os import de objetos de OpenERP, el namespace openerp, refactorizado a PEP-8 y adaptacion vistas a formato 7

…n_es_aeat. * Traducciones. * Vista del wizard BOE * Vistas de facturas mejoradas * Icono * Asignar una clave a las facturas ya existentes.

…49 y l10n_es_payment_order

…turas rectificativas a declarar.

…e o2m [FIX] l10n_es_aeat_mod349: Añadida restricción que impide eliminar facturas que aparezcan en registros de detalle

* IMP: Fiscal position onchange in new API + speed imp + refactoring * FIX: l10n_es_aeat_mod349: Remove check of names There's no need of a check about the name validity, because l10n_es_aeat already transform all the exported strings to valid ASCII characters (though unidecode), so I remove it entirely this check. * FIX: Use first period year in export Relying in fiscal year code for getting the fiscal year is not safe, because you can use that code for defining your journal sequences (for example, with the module account_auto_fy_sequence). We can't use also the year of the fiscal year, because you can have split fiscal years (i. e. like the academic courses). What we uses then is the year of the first period of the declaration, as there's no legal possibility of having mixed periods from different years.

…xport

…er calcular el 349 en períodos distintos de los fiscales

l10n_es_aeat_mod349/models/account_move.py

l10n_es_aeat_mod349/models/account_tax.py

pedrobaeza · 2024-04-25T06:58:43Z

l10n_es_aeat_mod349/models/res_company.py

+class ResCompany(models.Model):
+    _inherit = "res.company"
+
+    def _get_tax_ids_from_xmlids(self, tax_templates):


Sí, esto debe estar en el mapeado de la línea 349 como lo está en el mapeado regular para los otros modelos:

l10n-spain/l10n_es_aeat/models/l10n_es_aeat_map_tax_line.py

Lines 56 to 64 in af67194

def get_taxes_for_company(self, company):

"""Obtain the taxes corresponding to this line according the given company."""

self.ensure_one()

tax_ids = set()

for tax_xmlid in self.tax_xmlid_ids:

tax_id = company._get_tax_id_from_xmlid(tax_xmlid.name)

if tax_id:

tax_ids.add(tax_id)

return self.env["account.tax"].browse(list(tax_ids))

l10n_es_aeat_mod349/security/ir.model.access.csv

HaraldPanten

@manuelregidor Creo que no se han tenido en cuenta los commits mencionados en la conversación, puedes añadirlos?

Por otra parte, en las facturas de cliente es visible la columna l10n_es_aeat_349_operation_key . Es necesario?

manuelregidor · 2024-04-29T06:04:18Z

@HaraldPanten Los commits adicionales no se han tenido todavía en cuenta.

Con respecto a la visibilidad de la columna l10n_es_aeat_349_operation_key, hay cambios en el commit #3540, que todavía no se ha añadido.

HaraldPanten · 2024-04-29T10:28:04Z

@manuelregidor

Podemos ir añadiendo, entonces:

#3331 (ya se fusionó)
#3540 (ya se fusionó)

Y quedamos a la espera de #3544 (que también deberemos añadir). A este creo que no le queda mucho, solo que los tests parece que están un poco perezosos, esta mañana...

Gracias.

pedrobaeza · 2024-04-29T10:35:07Z

Parece que las GitHub actions no están arrancando en todo OCA. Pero https://www.githubstatus.com/ no reporta ningún fallo.

Regresión introducida en OCA@22d4155#diff-cf203442de5cb2bdf3aa371f214d1651256a58baa63ac5659c844287d53b5121. Fix OCA#3539. @moduon MT-5826

manuelregidor · 2024-05-14T14:03:42Z

@HaraldPanten @pedrobaeza @ValentinVinagre Ya están incluidos los 3 PRs que quedaban pendientes

HaraldPanten

Revisión funcional OK.

Probado que los errores añadidos aquí #3331 también sean visibles en la nueva versión.

@pedrobaeza

pedrobaeza

/ocabot merge nobump

OCA-git-bot · 2024-05-15T10:21:57Z

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-3503-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2024-05-15T10:30:08Z

Congratulations, your PR was merged at 32f3906. Thanks a lot for contributing to OCA. ❤️

…ubl_cii Related to OCA#3503 The peppol_eas field exists only if account_edi_ubl_cii is installed, if we don't have this module installed (because we don't have it in the scaffolding) the test should not fail.

Related to OCA#3503

Omar (pexego) and others added 30 commits March 27, 2024 10:12

[ADD] l10n_es_aeat, l10n_es_aeat_347, l10n_es_aeat_349: Adds a generi…

794365b

…c module for aeat models, 347 module was portedto v6.0 and adds new module to print AEAT model 349.

[IMP+FIX] l10n_es_aeat, l10n_es_aeat_mod347, l10n_es_aeat_mod349: Peq…

f3f15ff

…ueñas mejoras generales (vistas, traducciones, código) y corrección pequeño bug por un olvido en la adaptación de la v5 a la v6

Corrección bug 1021631 por el cual la aplicación da el error "NameErr…

f41f878

…or: global name 'pooler' is not defined" al iniciar el calculo.

l10n_es_aeat_mod349: Permitir seleccionar varios períodos en lugar de…

f93b5e9

… sólo uno y arrelgar un bug en la generación del modelo cuando el partner tiene el campo title lleno.

Commit inicial de una rama para la versión 7.0. Todos los módulos fue…

8093d12

…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True

[FIX] l10n_es_aeat_mod349: string sin cerrar en l10n_es_aeat_mod349-pot

6707be2

Launchpad automatic translations update.

41c21a2

[REF] cambiados los osv por orm.Model y orm.TransientModel, añadido e…

0a4f269

…n os import de objetos de OpenERP, el namespace openerp, refactorizado a PEP-8 y adaptacion vistas a formato 7

Launchpad automatic translations update.

81bda77

[IMP] Adaptación de los import a formato v7/v8

c064cc0

[IMP] l10n_es_aeat_mod349: Diversas mejoras y adaptación al nuevo l10…

8aff7c6

…n_es_aeat. * Traducciones. * Vista del wizard BOE * Vistas de facturas mejoradas * Icono * Asignar una clave a las facturas ya existentes.

[FIX] Adaptación a v8 de account_balance_reporting, l10n_es_aeat_mod3…

bfa60e8

…49 y l10n_es_payment_order

Launchpad automatic translations update.

82a4763

[MIG] l10n_es_aeat_mod349: Con nueva API

ef3bda0

[FIX] l10n_es_aeat_mod349: Se corrige un problema al calcular las fac…

9438776

…turas rectificativas a declarar.

[IMP] unificar campos en el módulo base + migrar a la nueva api.

ecc6a01

[FIX] l10n_es_aeat_mod349: Uso incorrecto de ondelete en definición d…

7763aea

…e o2m [FIX] l10n_es_aeat_mod349: Añadida restricción que impide eliminar facturas que aparezcan en registros de detalle

OCA Transbot updated translations from Transifex

1a56412

OCA Transbot updated translations from Transifex

cc449e4

[FIX] l10n_es_aeat_mod349: Name field in 349 conflict + add _rec_name

0332e82

OCA Transbot updated translations from Transifex

660c92b

[FIX] l10n_es_aeat_mod349: Don't depend on fiscal year code for the e…

ce15d39

…xport

[FIX]. l10n_es_aeat_mod349 . Recuperar funcionalidad perdida para pod…

fcb0518

…er calcular el 349 en períodos distintos de los fiscales

OCA Transbot updated translations from Transifex

434c8a8

[FIX] l10n_es_aeat_mod349: Search only one refund detail record

e7312f4

pedrobaeza reviewed Apr 25, 2024

View reviewed changes

HaraldPanten self-requested a review April 26, 2024 09:48

HaraldPanten requested changes Apr 26, 2024

View reviewed changes

HaraldPanten mentioned this pull request May 13, 2024

[16.0][IMP] l10n_es_aeat_mod349: functionality to get the country_code #3544

Merged

ACheung-FactorLibre and others added 4 commits May 14, 2024 09:05

[IMP] l10n_es_aeat_mod349: implement more error texts

2c993e3

[FIX] l10n_es_aeat_mod349: permitir a los comerciales ver sus facturas

2a61f1a

Regresión introducida en OCA@22d4155#diff-cf203442de5cb2bdf3aa371f214d1651256a58baa63ac5659c844287d53b5121. Fix OCA#3539. @moduon MT-5826

[IMP] l10n_es_aeat_mod349: functionality to get the country_code

6403590

[IMP] l10n_es_aeat_mod349: black, isort, prettier

3350ecc

manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_mod349 branch from a548794 to 424ca7c Compare May 14, 2024 07:18

[17.0][MIG] l10n_es_aeat_mod349: Migration to 17.0

f33e8be

manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_mod349 branch from 424ca7c to f33e8be Compare May 14, 2024 12:02

HaraldPanten approved these changes May 15, 2024

View reviewed changes

pedrobaeza approved these changes May 15, 2024

View reviewed changes

OCA-git-bot added the approved label May 15, 2024

OCA-git-bot added the bot is merging ⏳ label May 15, 2024

OCA-git-bot merged commit 28554bc into OCA:17.0 May 15, 2024
7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels May 15, 2024

HaraldPanten deleted the 17.0-mig-l10n_es_aeat_mod349 branch May 15, 2024 11:54

victoralmau mentioned this pull request Oct 3, 2024

[17.0][FIX] l10n_es_aeat_mod349: Fix tests compatibililty with account_edi_ubl_cii #3750

Merged

victoralmau added a commit to Tecnativa/l10n-spain that referenced this pull request Oct 4, 2024

[FIX] l10n_es_aeat_mod349: Remove peppol_eas field from tests

62ea4b4

Related to OCA#3503

victoralmau added a commit to Tecnativa/l10n-spain that referenced this pull request Oct 4, 2024

[FIX] l10n_es_aeat_mod349: Remove peppol_eas field from tests

993d384

Related to OCA#3503

victoralmau added a commit to Tecnativa/l10n-spain that referenced this pull request Oct 8, 2024

[FIX] l10n_es_aeat_mod349: Remove peppol_eas field from tests

95be6b8

Related to OCA#3503

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[17.0][MIG] l10n_es_aeat_mod349: Migration to version 17.0 #3503

[17.0][MIG] l10n_es_aeat_mod349: Migration to version 17.0 #3503

manuelregidor commented Mar 27, 2024

pedrobaeza Apr 25, 2024

HaraldPanten left a comment

manuelregidor commented Apr 29, 2024

HaraldPanten commented Apr 29, 2024

pedrobaeza commented Apr 29, 2024

manuelregidor commented May 14, 2024

HaraldPanten left a comment

pedrobaeza left a comment

OCA-git-bot commented May 15, 2024

OCA-git-bot commented May 15, 2024

	def get_taxes_for_company(self, company):
	"""Obtain the taxes corresponding to this line according the given company."""
	self.ensure_one()
	tax_ids = set()
	for tax_xmlid in self.tax_xmlid_ids:
	tax_id = company._get_tax_id_from_xmlid(tax_xmlid.name)
	if tax_id:
	tax_ids.add(tax_id)
	return self.env["account.tax"].browse(list(tax_ids))

[17.0][MIG] l10n_es_aeat_mod349: Migration to version 17.0 #3503

[17.0][MIG] l10n_es_aeat_mod349: Migration to version 17.0 #3503

Conversation

manuelregidor commented Mar 27, 2024

pedrobaeza Apr 25, 2024

Choose a reason for hiding this comment

HaraldPanten left a comment

Choose a reason for hiding this comment

manuelregidor commented Apr 29, 2024

HaraldPanten commented Apr 29, 2024

pedrobaeza commented Apr 29, 2024

manuelregidor commented May 14, 2024

HaraldPanten left a comment

Choose a reason for hiding this comment

pedrobaeza left a comment

Choose a reason for hiding this comment

OCA-git-bot commented May 15, 2024

OCA-git-bot commented May 15, 2024