summaryrefslogtreecommitdiff
path: root/addons/website_slides/static
diff options
context:
space:
mode:
authorAdrian Torres <adt@odoo.com>2020-03-11 14:13:17 +0000
committerAdrian Torres <adt@odoo.com>2020-03-11 18:52:52 +0000
commitcd4b50de6f5a6bfa063b2d8268cb8bc58c6a74f6 (patch)
treed19a6b39277b70da8d3dcd8a3f12a959047cabc0 /addons/website_slides/static
parent29e0227081ccf51f1a89305dc52f2470d4279b2c (diff)
[FIX] base: don't uninstall already uninstalled modules
Before this commit: - Install some random module that can be uninstalled (so not base) - Open the uninstall wizard for said module in two different tabs / windows / whatever - In one tab, confirm the module uninstall and wait for it to be done - As soon as the other tab is done with the uninstall, go to the second one with the uninstall wizard still open, and proceed with the second uninstall - Boom, the registry crashes and completely fucks up the DB because there's no check at all that prevents the uninstall of already uninstalled modules. After this commit: - `ir.module.module.button_uninstall` will check if all the modules being uninstalled are in the installed state and if not a UserError will be raised, preventing a second uninstall of the module which could potentially break the DB Do note that this fix is LOCAL, the problem is however more or less global, wherever there's user-actionable buttons that should only be pressed once there's a potential for bugs / breakage if a similar fix is not implemented locally. Perhaps a more global fix should be implemented eventually, but it's generally less annoying for business cases since those probably won't break the registry, see task 1859014. opw-2213679 opw-2212594 opw-2206446 closes odoo/odoo#47450 X-original-commit: 8c1bb22ec0222dca652e0649454b986c06cb1368 Signed-off-by: Adrian Torres (adt) <adt@odoo.com>
Diffstat (limited to 'addons/website_slides/static')
0 files changed, 0 insertions, 0 deletions