[16.0][FIX] l10n_es_aeat_mod303_vat_prorate: Add analytic_distribution only if prorate_analytic_account_id is set

OCA / l10n-spain

Odoo Spain Localization

https://www.aeodoo.org/estado-localizacion

GNU Affero General Public License v3.0

265 stars 513 forks source link

[16.0][FIX] l10n_es_aeat_mod303_vat_prorate: Add analytic_distribution only if prorate_analytic_account_id is set #3620

Closed victoralmau closed 2 months ago

victoralmau commented 2 months ago

Add analytic_distribution only if prorate_analytic_account_id is set

Caso de uso

Definir distribución analítica "incorrecta" (no vinculada a ninguna cuenta analítica).

Error:

File "/opt/odoo/auto/addons/account_financial_report/models/account_move_line.py", line 21, in _compute_analytic_account_ids
    for account_id in map(int, record.analytic_distribution):
ValueError: invalid literal for int() with base 10: 'false'

@Tecnativa

OCA-git-bot commented 2 months ago

Hi @pedrobaeza, some modules you are maintaining are being modified, check this out!

pedrobaeza commented 2 months ago

Indica el problema que estás intentando resolver.

victoralmau commented 2 months ago

Actualizado el caso de uso y el error que sucedía antes de este cambio.

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-3620-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

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