Merge lp:~therp-nl/server-env-tools/decouple_oe_sa into lp:~server-env-tools-core-editors/server-env-tools/7.0
Proposed by
Holger Brunn (Therp)
Status: | Merged |
---|---|
Merged at revision: | 47 |
Proposed branch: | lp:~therp-nl/server-env-tools/decouple_oe_sa |
Merge into: | lp:~server-env-tools-core-editors/server-env-tools/7.0 |
Diff against target: |
194 lines (+151/-0) 7 files modified
disable_openerp_online/__init__.py (+24/-0) disable_openerp_online/__openerp__.py (+55/-0) disable_openerp_online/data/ir_cron.xml (+8/-0) disable_openerp_online/data/ir_ui_menu.xml (+7/-0) disable_openerp_online/model/__init__.py (+21/-0) disable_openerp_online/model/publisher_warranty_contract.py (+28/-0) disable_openerp_online/static/src/xml/base.xml (+8/-0) |
To merge this branch: | bzr merge lp:~therp-nl/server-env-tools/decouple_oe_sa |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stefan Rijnhart (Opener) | test, code review | Approve | |
Guewen Baconnier @ Camptocamp | code review, no test | Approve | |
Maxime Chambreuil (http://www.savoirfairelinux.com) | lgtm, no test | Approve | |
Ronald Portier (Therp) (community) | Approve | ||
Review via email: mp+179892@code.launchpad.net |
Description of the change
This module deactivates all bindings to openerp.com that come with the standard code:
- update notifier code is deactivated and the function is overwritten
- apps and updates menu items in settings are removed
- help and account menu items in user menu are removed
To post a comment you must log in.
Thanks! Works like a charm. The module name sounds a little unclear to me. How about 'disable_ openerp_ online' ?
Also, there is still a suspicious class "upload_ data_thread" in openerp/ tools/misc. py that refers to an OpenERP submission page in its 'run' method. I can't find any place where it is called, but maybe you could disarm this method by way of monkeypatching, just for the sake of completeness (if not paranoia)?