[IMP]l10n_ar_account_reports: Mejoras del reporte Balance General

[feg-adhoc]

No description provided.

[roboadhoc]

[Copilot]

Pull request overview

Este PR mejora el reporte Estado Patrimonial / Balance General para AR desagregando el “Patrimonio Neto” en Capital, Reservas y Resultados, actualizando tanto la estructura del reporte como la lógica de tagueo de cuentas y agregando scripts de migración para upgrades.

Changes:

• Reemplaza el tag único ar_esp_patrimonio_neto por ar_esp_capital, ar_esp_reservas y ar_esp_resultados y ajusta el tagueo automático por prefijo de código (3.1/3.2/3.3).
• Refactoriza líneas del Balance Sheet para mostrar Patrimonio Neto con sublíneas (Capital/Reservas/Resultados) y nueva fórmula agregada.
• Añade migraciones pre/post para limpiar estructura anterior y recalcular tags en compañías AR; bump de versión a 18.0.1.5.0.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
l10n_ar_account_reports/models/account_chart_template.py	Ajusta el diccionario de tags y la asignación para cuentas de patrimonio neto por prefijo de código.
l10n_ar_account_reports/migrations/18.0.1.5.0/pre-migration.py	Remueve asignaciones del tag viejo y borra líneas viejas del reporte para evitar conflictos al actualizar.
l10n_ar_account_reports/migrations/18.0.1.5.0/post-migration.py	Reaplica tagueo automático a compañías AR tras el cambio de tags/estructura.
l10n_ar_account_reports/data/tags_data.xml	Elimina el tag viejo y crea los tres tags nuevos (Capital/Reservas/Resultados).
l10n_ar_account_reports/data/balance_sheet.xml	Cambia la estructura del Patrimonio Neto a children y actualiza fórmulas para sumar sublíneas.
l10n_ar_account_reports/manifest.py	Bump de versión a 18.0.1.5.0.

Comments suppressed due to low confidence (1)

l10n_ar_account_reports/migrations/18.0.1.5.0/post-migration.py:11

• Este post-migration corre _l10n_ar_account_reports_setup_account_tags() para todas las compañías AR. Ese método (en account_chart_template.py) hace múltiples write() por cuenta/tag, lo que puede alargar bastante el upgrade en bases grandes. Sugerencia: optimizar la asignación (writes vectorizados / SQL sobre la tabla m2m) o acotar el recalculado a cuentas afectadas (p.ej. las que tenían el tag viejo o las de patrimonio).

[Copilot]

En la pre-migración se eliminan las asignaciones del tag viejo, pero no se elimina el propio tag ar_esp_patrimonio_neto (ni su ir_model_data). Como el record fue removido del XML, va a quedar un tag “huérfano” en la BD y podría seguir usándose manualmente. Sugerencia: borrar también el registro de account.account.tag y su entrada de ir_model_data (idealmente con helpers de openupgrade) una vez removidas las relaciones.

Suggested change

	# Delete the old patrimonio_neto tag record itself
	openupgrade.logged_query(
	cr,
	"""
	DELETE FROM account_account_tag
	WHERE id IN (
	SELECT res_id
	FROM ir_model_data
	WHERE module = 'l10n_ar_account_reports'
	AND model = 'account.account.tag'
	AND name = 'ar_esp_patrimonio_neto'
	)
	""",
	)
	# Delete the ir_model_data entry for the old patrimonio_neto tag
	openupgrade.logged_query(
	cr,
	"""
	DELETE FROM ir_model_data
	WHERE module = 'l10n_ar_account_reports'
	AND model = 'account.account.tag'
	AND name = 'ar_esp_patrimonio_neto'
	""",
	)

[Copilot]

El docstring indica que los nuevos tags se asignan “based on the account type”, pero el split Capital/Reservas/Resultados depende del prefijo del código (3.1/3.2/3.3) además del tipo. Conviene ajustar la descripción para que refleje el criterio real (por código de cuenta), así evita confusión al mantener la migración.

Suggested change

	on the account type.
	primarily on the account code prefixes (3.1/3.2/3.3), taking into account
	the account type when needed.