Skip to content

[18.0][FIX] base_report_to_printer: neutralization scripts#433

Open
ferran-S73 wants to merge 1 commit intoOCA:18.0from
Studio73:18.0-fix-base_report_to_printer-neutralization
Open

[18.0][FIX] base_report_to_printer: neutralization scripts#433
ferran-S73 wants to merge 1 commit intoOCA:18.0from
Studio73:18.0-fix-base_report_to_printer-neutralization

Conversation

@ferran-S73
Copy link
Copy Markdown

Even with odoo neutralize script executed reports where still being sent to printers.

On one side the search for users behaviours finds them even though the printers where deactivated, thus the need to a add an active field to this model and set it to false in the script.

On the other side the general printer configured in a report needs to be cleaned or it will otherwise be found and used for printing.

Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the version from the commit message, and about the functionality, it's interesting, but please add all the stuff for the active field like banner.

@ferran-S73 ferran-S73 force-pushed the 18.0-fix-base_report_to_printer-neutralization branch from a2c0a7d to 1605a13 Compare March 16, 2026 07:50
@ferran-S73
Copy link
Copy Markdown
Author

@pedrobaeza removed version from commit message + rebased to resolve conflicts. As per the banner, are you talking about the ribbon with the text "archived" that is shown in some views? If that is the case I can't add it here since there is no actual form view for the printing.report.xml.action model.

@pedrobaeza pedrobaeza added this to the 18.0 milestone Mar 16, 2026
@pedrobaeza
Copy link
Copy Markdown
Member

OK, if all is handled from the list view, then it should be handled in a similar way to crons, that all are shown through a default filter, and an extra one should be to have a pill to identify the active/inactive ones in a column.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants