[19.0][MIG] l10n_it_central_journal_reportlab: Migration to 19.0#5077
Open
Borruso wants to merge 25 commits into
Open
[19.0][MIG] l10n_it_central_journal_reportlab: Migration to 19.0#5077Borruso wants to merge 25 commits into
Borruso wants to merge 25 commits into
Conversation
Member
|
/ocabot migration l10n_it_central_journal_reportlab |
Borruso
force-pushed
the
19.0-mig-l10n_it_central_journal_reportlab
branch
6 times, most recently
from
510d5a1 to
4b8d65f
Compare
|
è possibile fare rebase per testarla? |
Borruso
force-pushed
the
19.0-mig-l10n_it_central_journal_reportlab
branch
from
4b8d65f to
42bba6a
Compare
OCA-git-bot
added
mod:l10n_it_central_journal_reportlab
mmircoli-nexapp
approved these changes
May 5, 2026
mmircoli-nexapp
left a comment
mmircoli-nexapp
left a comment
There was a problem hiding this comment.
Review funzionale e ho provato anche la funzione ultima pagina stampata e anche la funzionalità della riga che in passato davano problemi e confermo che funziona corettamente
Contributor
|
This PR has the |
…le stampa reportlab
Currently translated at 88.3% (53 of 60 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_central_journal_reportlab Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_central_journal_reportlab/it/
Currently translated at 95.0% (57 of 60 strings) Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_central_journal_reportlab Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_central_journal_reportlab/it/
Currently translated at 100.0% (60 of 60 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_central_journal_reportlab Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_central_journal_reportlab/it/
Currently translated at 100.0% (61 of 61 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_central_journal_reportlab Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_central_journal_reportlab/it/
Currently translated at 100.0% (62 of 62 strings) Translation: l10n-italy-16.0/l10n-italy-16.0-l10n_it_central_journal_reportlab Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-l10n_it_central_journal_reportlab/it/
When extracting a grouped report, if `<` or `>` are in the account_name variable they are interpreted as html tags and an error is raised. This fixes the issue.
…eivable/payable accounts, otherwise use entry name
…hout entry name
We must exclude lines without account (like notes in invoice lines) because they are not significant for central journal;
otherwise:
l10n_it_central_journal_reportlab/wizard/print_giornale.py", line 471, in get_final_tables_report_giornale
else line["account_name"][user_lang]
~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^
TypeError: 'NoneType' object is not subscriptable
Also, lines without name (like draft invoices) must be printed without error. But default behaviour should be printing posted entries
Company does not need to be copied from date range.
…ssing account on move lines
…rom report This PR fixes exclude the lines of notes and section from the export
Borruso
force-pushed
the
19.0-mig-l10n_it_central_journal_reportlab
branch
from
42bba6a to
5361204
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#4930