Merge lp:~openerp-dev/openerp-web/7.0-bug-1097757-ggh into lp:openerp-web/7.0
Proposed by
Farid Ghanchi
Status: | Needs review |
---|---|
Proposed branch: | lp:~openerp-dev/openerp-web/7.0-bug-1097757-ggh |
Merge into: | lp:openerp-web/7.0 |
Diff against target: |
34 lines (+10/-2) 2 files modified
addons/web/static/src/js/chrome.js (+9/-1) addons/web/static/src/xml/base.xml (+1/-1) |
To merge this branch: | bzr merge lp:~openerp-dev/openerp-web/7.0-bug-1097757-ggh |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Farid Ghanchi (community) | Needs Resubmitting | ||
Xavier (Open ERP) (community) | Needs Fixing | ||
Review via email: mp+145306@code.launchpad.net |
Description of the change
Hello,
I had fixed the issue of Deactivate Developer mode in 7.0.
About OpenERP >> Activate the Developer mode and Deactivate the Developer mode.
Thanks
To post a comment you must log in.
Unmerged revisions
- 3735. By Farid Ghanchi
-
[Fix] Used to deparam.querystring in deactivat devmode
- 3734. By Farid Ghanchi
-
[Merge] with trunk
- 3733. By Farid Ghanchi
-
[Fix]match debug only querystring(?#) in url
- 3732. By Farid Ghanchi
-
[FIx]Fixed the issue of Deactivate Developer mode with BBQ lib
- 3731. By Farid Ghanchi
-
[Fix ] FIxed the issue of Developer mode Deactivate
This is broken, for instance the first test will match any "debug" anywhere in the url, if somebody creates a page served through a controller on /debug or the web client is embedded in a page whose URL contains the word "debug", it'll match and this bit will believe it is in debug mode somehow.
Considering we are already using jquery-bbq for query parameters manipulation, I would suggest learning the library and using it for this case as well, instead of doing the same thing by hand incorrectly.