-
Notifications
You must be signed in to change notification settings - Fork 23k
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.
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
[FW][FIX] mail: show all attachment options #166086
[FW][FIX] mail: show all attachment options #166086
Conversation
This PR targets saas-17.1 and is part of the forward-port chain. Further PRs will be created up to master. More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port |
@reth-odoo ci/runbot failed on this forward-port PR |
@fw-bot retry |
b1b088d
to
d8f3a60
Compare
@robodoo r+ rebased |
@reth-odoo this PR was modified / updated and has become a normal PR. It should be merged the normal way (via @robodoo) |
The template for the attachment list adds a class to show multiple options when attachments can be deleted. However the condition to display the delete button is different from the condition to apply that style. Meaning if one condition `showDelete` is true while the other `isDeletable` is false it is impossible to access the download button as the correct style is not applied. - Fix the template to use the same condition everywhere - Fix the condition for showDelete to always be false is deletion is disallowed Issue noticeable since [1] when attachments were made conditionally deletable as opposed to always deletable previously. 1: 32a80a3 task-3519815 X-original-commit: 56b3f67
d8f3a60
to
3ce0d76
Compare
Switch to patchUserWithCleanup, user service not available and this is more reliable. |
The template for the attachment list adds a class to show multiple options when attachments can be deleted. However the condition to display the delete button is different from the condition to apply that style. Meaning if one condition `showDelete` is true while the other `isDeletable` is false it is impossible to access the download button as the correct style is not applied. - Fix the template to use the same condition everywhere - Fix the condition for showDelete to always be false is deletion is disallowed Issue noticeable since [1] when attachments were made conditionally deletable as opposed to always deletable previously. 1: 32a80a3 task-3519815 closes #166086 X-original-commit: 56b3f67 Signed-off-by: Renaud Thiry (reth) <reth@odoo.com>
The template for the attachment list adds a class to show multiple options when attachments can be deleted.
However the condition to display the delete button is different from the condition to apply that style.
Meaning if one condition
showDelete
is true while the otherisDeletable
is false it is impossible to access the download button as the correct style is not applied.Issue noticeable since [1] when attachments were made conditionally deletable as opposed to always deletable previously.
Steps to reproduce:
1: 32a80a3
task-3519815
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Forward-Port-Of: #165332