odoo
/
odoo
2
0
Fork 0
Code Pull Requests Releases Activity

[IMP] When we go step by step, change the Case for a variable in a diff 

Is a little confufing, in our case 100% of time our team had problems confusing Action with action
Obtaining an error with prototype because incorrect declaration.

IMHO this is a No-Brain merge, but i explain well to be sure you understand my point.

bzr revid: nhomar@gmail.com-20130517205608-wykuz0oxjccf6021
This commit is contained in:
Nhomar Hernandez 2013-05-17 16:26:08 -04:30
parent c694837a64
commit f0ae319dec
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
addons/web/doc/module/static/src/js/first_module.js.2.diff
View File

@ -4,8 +4,8 @@
// static/src/js/first_module.js
openerp.web_example = function (instance) {
- console.log("Module loaded");
+ instance.web.client_actions.add('example.action', 'instance.web_example.action');
+ instance.web_example.action = function (parent, action) {
+ instance.web.client_actions.add('example.action', 'instance.web_example.Action');
+ instance.web_example.Action = function (parent, action) {
+ console.log("Executed the action", action);
+ };
};

4
addons/web/doc/module/static/src/js/first_module.js.3.diff
View File

@ -3,8 +3,8 @@
@@ -1,7 +1,11 @@
// static/src/js/first_module.js
openerp.web_example = function (instance) {
- instance.web.client_actions.add('example.action', 'instance.web_example.action');
- instance.web_example.action = function (parent, action) {
- instance.web.client_actions.add('example.action', 'instance.web_example.Action');
- instance.web_example.Action = function (parent, action) {
- console.log("Executed the action", action);
- };
+ instance.web.client_actions.add('example.action', 'instance.web_example.Action');