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

[FIX] implicit globals: missing 'var' declarations 

did not touch gantt.js, this thing's a swap, there are implicit globals which are actually used globally

bzr revid: xmo@openerp.com-20110906093139-uermn7a9vo38l3kf
This commit is contained in:
Xavier Morel 2011-09-06 11:31:39 +02:00
parent a5a0878bba
commit 067e0b8827
8 changed files with 20 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
addons/web/static/src/js/boot.js
View File

@ -56,8 +56,8 @@ openerp.web = function(instance) {
openerp.web.formats(instance);
openerp.web.chrome(instance);
openerp.web.data(instance);
files = ["views","search","list","form","list_editable","web_mobile","view_tree","data_export"];
for(i=0; i<files.length; i++) {
var files = ["views","search","list","form","list_editable","web_mobile","view_tree","data_export"];
for(var i=0; i<files.length; i++) {
if(openerp.web[files[i]]) {
openerp.web[files[i]](instance);
}

2
addons/web/static/src/js/core.js
View File

@ -553,7 +553,7 @@ openerp.web.Session = openerp.web.CallbackEnabled.extend( /** @lends openerp.web
this.rpc('/web/session/modules', {}, function(result) {
self.module_list = result;
var lang = self.user_context.lang;
params = { mods: ["web"].concat(result), lang: lang};
var params = { mods: ["web"].concat(result), lang: lang};
self.rpc('/web/webclient/translations',params).then(function(transs) {
openerp.web._t.database.set_bundle(transs);
var modules = self.module_list.join(',');

2
addons/web/static/src/js/data.js
View File

@ -622,7 +622,7 @@ openerp.web.BufferedDataSet = openerp.web.DataSetStatic.extend({
record = record || _.detect(this.to_write, function(x) {return x.id === id;});
var dirty = false;
if (record) {
for (k in data) {
for (var k in data) {
if (record.values[k] === undefined || record.values[k] !== data[k]) {
dirty = true;
break;

2
addons/web_gantt/static/src/js/gantt.js
View File

@ -233,7 +233,7 @@ init: function(parent, element_id, dataset, view_id) {
evt_date = all_events[evt_id]['evt'][2];
evt_duration = all_events[evt_id]['evt'][3];
evt_str_date = this.convert_date_str(evt_date);
var evt_str_date = this.convert_date_str(evt_date);
evt_end_date = this.end_date(evt_str_date, evt_duration);
while (all_events[evt_id]['parent'] != "") {

14
addons/web_mobile/static/src/js/chrome_mobile.js
View File

@ -116,8 +116,8 @@ openerp.web_mobile.Header = openerp.web.Widget.extend({
self.$element.find("a").click(this.on_clicked);
},
on_clicked: function(ev) {
$opt = $(ev.currentTarget);
current_id = $opt.attr('id');
var $opt = $(ev.currentTarget);
var current_id = $opt.attr('id');
if (current_id == 'home') {
this.homepage = new openerp.web_mobile.HomePage(this, "oe_app");
this.homepage.start();
@ -137,9 +137,9 @@ openerp.web_mobile.Shortcuts = openerp.web.Widget.extend({
})
},
on_clicked: function(ev) {
$shortcut = $(ev.currentTarget);
id = $shortcut.data('menu');
res_id = $shortcut.data('res');
var $shortcut = $(ev.currentTarget);
var id = $shortcut.data('menu');
var res_id = $shortcut.data('res');
jQuery("#oe_header").find("h1").html($shortcut.data('name'));
this.listview = new openerp.web_mobile.ListView(this, "oe_app", res_id);
this.listview.start();
@ -162,7 +162,7 @@ openerp.web_mobile.Menu = openerp.web.Widget.extend({
this.$element.add(this.$secondary_menu).find("a").click(this.on_menu_click);
},
on_menu_click: function(ev, id) {
$menu = $(ev.currentTarget);
var $menu = $(ev.currentTarget);
id = $menu.data('menu');
for (var i = 0; i < this.data.data.children.length; i++) {
if (this.data.data.children[i].id == id) {
@ -190,7 +190,7 @@ openerp.web_mobile.Secondary = openerp.web.Widget.extend({
this.$element.add(this.$secondary_menu).find("a").click(this.on_menu_click);
},
on_menu_click: function(ev, id) {
$menu = $(ev.currentTarget);
var $menu = $(ev.currentTarget);
id = $menu.data('menu');
for (var i = 0; i < this.data.children.length; i++) {
if (this.data.children[i].id == id) {

6
addons/web_mobile/static/src/js/form_mobile.js
View File

@ -18,9 +18,9 @@ openerp.web_mobile.FormView = openerp.web.Widget.extend({
},
start: function() {
var self = this;
id = this.list_id;
model = this.action.res_model;
view_id = this.action.views[1][0];
var id = this.list_id;
var model = this.action.res_model;
var view_id = this.action.views[1][0];
this.dataset = new openerp.web.DataSetSearch(this, this.action.res_model, null, null);
this.dataset.read_slice([],{}, function (result) {
for (var i = 0; i < result.length; i++) {

4
addons/web_mobile/static/src/js/list_mobile.js
View File

@ -63,9 +63,9 @@ openerp.web_mobile.ListView = openerp.web.Widget.extend({
});
},
on_list_click: function(ev) {
$record = $(ev.currentTarget);
var $record = $(ev.currentTarget);
var self = this;
id = $record.data('id');
var id = $record.data('id');
this.formview = new openerp.web_mobile.FormView(this, "oe_app", id, this.action);
this.formview.start();
}

5
addons/web_mobile/static/src/web_mobile.html
View File

@ -15,7 +15,7 @@
<script type="text/javascript" src="/web_mobile/static/lib/jquery_mobile/js/jquery.mobile-1.0a4.1.js"></script>
<script type="text/javascript" src="/base/static/lib/qweb/qweb2.js"></script>
<script type="text/javascript" src="/base/static/src/js/boot.js"></script>
<script type="text/javascript" src="/base/static/src/js/boot.js"></script>
<script type="text/javascript" src="/base/static/src/js/core.js"></script>
<script type="text/javascript" src="/base/static/src/js/chrome.js"></script>
<script type="text/javascript" src="/base/static/src/js/data.js"></script>
@ -27,6 +27,7 @@
<script type="text/javascript" src="/web_mobile/static/src/js/form_mobile.js"></script>
<script type="text/javascript">
var QWeb;
$(function() {
QWeb = window.QWeb || new QWeb2.Engine();
var oe = openerp.init();
@ -38,4 +39,4 @@
<body>
<div id="moe" class="openerp"></div>
</body>
</html>
</html>