Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[16.0] l10n_it_central_journal_reportlab: errore quando < o > sono nella variabile line.name #4396

Open
1 of 2 tasks
salvorapi opened this issue Oct 9, 2024 · 4 comments · Fixed by #4400
Open
1 of 2 tasks

[16.0] l10n_it_central_journal_reportlab: errore quando < o > sono nella variabile line.name #4396

salvorapi opened this issue Oct 9, 2024 · 4 comments · Fixed by #4400
Labels
14.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Comments

@salvorapi
Copy link
Contributor

salvorapi commented Oct 9, 2024 •
edited by SirAionTech
Loading

Module

l10n_it_central_journal_reportlab

Describe the bug

Stesso problema della issue #4153 però con la linea 403 line.name

Rif:

l10n-italy/l10n_it_central_journal_reportlab/wizard/print_giornale.py

Line 413 in 9076c72

name = Paragraph(str(line.name or ""), style_name)

To Reproduce

  1. Creare un movimento che contiene caratteri < o >
  2. Provare a stampare un report di l10n_it_central_journal_reportlab
@matteoopenf
Copy link
Contributor

perchè crei un movimento con quei caratteri?

@francesco-ooops
Copy link
Contributor

Perchè nulla glie lo impedisce 😅

@TheMule71 TheMule71 mentioned this issue Oct 11, 2024
Merged
@TheMule71 TheMule71 self-assigned this Oct 11, 2024
@techgrrow
Copy link
Contributor

Ciao, in teoria dovrebbe già essere stato mergiato questo fix #4143

@francesco-ooops francesco-ooops added needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue. labels Nov 8, 2024
@TheMule71
Copy link
Contributor

Ciao, in teoria dovrebbe già essere stato mergiato questo fix #4143

Mi pare fixi solo il campo due righe sopra (account_name e non name).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
14.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[16.0][FIX] l10n_it_central_journal_reportlab: escape Paragraph content TheMule71/l10n-italy
5 participants
@TheMule71 @salvorapi @matteoopenf @francesco-ooops @techgrrow