Availability |
Odoo Online
Odoo.sh
On Premise
|
Odoo Apps Dependencies |
Discuss (mail)
|
Lines of code | 72 |
Technical Name |
stop_auto_subscribe_partners |
License | AGPL-3 |
Website | https://www.cube48.de |
Versions | 11.0 10.0 13.0 12.0 15.0 14.0 16.0 |
Please log in to comment on this module
Report comment
Any abuse of this reporting system will be penalizedNeither designed nor coded to work at all.
I need to revise my rating. When I first tried this module in Odoo 11.0 it had an obvious bug that caused it to break the Odoo server. Once I had fixed that bug as described in my original rating it then broke the Odoo follow feature. In Odoo 11.0 that if the module is enabled using the settings checkbox then it completely disables the follow functionality of Odoo. Neither explicit follow from the Odoo UI nor following via the link in a follow invitation email will work properly. If the enable setting is unchecked then the Odoo follow works, but this module literally does nothing at all.
My choice was to uninstall, remove the app record from the DB, and remove the module from my addons directory in order to thoroughly expunge this module.
Excellent tool
Works like a charm... the only downside is that it will also prevent users to auto subscribe to new private channels in the discussion app.
Re: Excellent tool
Thank you for your feedback.
Useful, but needs some work
This is a useful module, but there is a critical bug and the code could use some touching up. The bug is at the line /stop_auto_subscribe_partners/models/mail_thread.py:23 app_stop_subscribe = True if ir_config.get_param('app_stop_subscribe') == "True" else False This needs to use the sudo() method on the ir_config model to allow non-admin users to read the config. You might also consider using the strtobool() function from the standard library to get a more robust and flexible conversion from the config string to the boolean value. I'd suggest this change: from distutils.util import strtobool app_stop_subscribe = bool(strtobool(ir_config.sudo().get_param('app_stop_subscribe'))) This change allowed me to continue using the module, which otherwise broke my application for my non-admin users.
Re: Useful, but needs some work
Hi, Thank you, for your detailed explanation, We are considered your feedback and our team will fix it.