[FIX] bunch of minor issues

bzr revid: xmo@openerp.com-20110906093516-3nkjngcmt2l3wex4

7 files changed, 21 insertions, 22 deletions

diff --git a/addons/web/static/src/css/base-ie7.css b/addons/web/static/src/css/base-ie7.css
index 35599dbdd4b..fa1f1fe74a6 100644
--- a/addons/web/static/src/css/base-ie7.css
+++ b/addons/web/static/src/css/base-ie7.css
@@ -1,4 +1,4 @@
 
 .openerp .menu span {
-    min-width: 0px;
+    min-width: 0;
 }
diff --git a/addons/web/static/src/css/base.css b/addons/web/static/src/css/base.css
index a4104e8a839..b46fef45fdc 100644
--- a/addons/web/static/src/css/base.css
+++ b/addons/web/static/src/css/base.css
@@ -174,7 +174,7 @@ ul.db_options li {
     filter: progid:DXImageTransform.Microsoft.gradient( startColorstr='#949292', endColorstr='#5B5A5A',GradientType=0 ); /* IE6-9 */
     border: none;
     /* overriding jquery ui */
-    -moz-border-radius: 0px; -webkit-border-radius: 0px; border-radius: 0px;
+    -moz-border-radius: 0; -webkit-border-radius: 0; border-radius: 0;
     display: block;
     font-weight: bold;
     text-transform: uppercase;
@@ -324,7 +324,7 @@ label.error {
     filter: progid:DXImageTransform.Microsoft.gradient( startColorstr='#949292', endColorstr='#5B5A5A',GradientType=0 ); /* IE6-9 */
     border: none;
     /* overriding jquery ui */
-    -moz-border-radius: 0px; -webkit-border-radius: 0px; border-radius: 0px;
+    -moz-border-radius: 0; -webkit-border-radius: 0; border-radius: 0;
 }
 .openerp .secondary_menu h4 {
     padding: 0 0 2px 10px;
@@ -820,7 +820,7 @@ label.error {
     padding: 2px;
 }
 .openerp td.oe_form_frame_cell.oe_form_group {
-    padding: 0px;
+    padding: 0;
 }
 .openerp .required.error {
     border: 1px solid #900;
diff --git a/addons/web/static/src/js/boot.js b/addons/web/static/src/js/boot.js
index c5d8bbfc1ca..2c00eb3d5a5 100644
--- a/addons/web/static/src/js/boot.js
+++ b/addons/web/static/src/js/boot.js
@@ -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);
         }
diff --git a/addons/web/static/src/js/core.js b/addons/web/static/src/js/core.js
index cfc7eab0b0e..4acc8ec15c3 100644
--- a/addons/web/static/src/js/core.js
+++ b/addons/web/static/src/js/core.js
@@ -553,14 +553,13 @@ 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(',');
                 var file_list = ["/web/static/lib/datejs/globalization/" +
-                    self.user_context.lang.replace("_", "-") + ".js",
-
-                    ];
+                    self.user_context.lang.replace("_", "-") + ".js"
+                ];
                 if(self.debug) {
                     self.rpc('/web/webclient/csslist', {"mods": modules}, self.do_load_css);
                     self.rpc('/web/webclient/jslist', {"mods": modules}, function(files) {
@@ -987,7 +986,7 @@ openerp.web.TranslationDataBase = openerp.web.Class.extend({
         var fcnt = function(str) {
             var tmp = self.get(str);
             return tmp === undefined ? str : tmp;
-        }
+        };
         fcnt.database = this;
         return fcnt;
     },
diff --git a/addons/web/static/src/js/data.js b/addons/web/static/src/js/data.js
index c4b489242fa..3797121756f 100644
--- a/addons/web/static/src/js/data.js
+++ b/addons/web/static/src/js/data.js
@@ -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;
@@ -691,7 +691,7 @@ openerp.web.BufferedDataSet = openerp.web.DataSetStatic.extend({
                 }
             }
             setTimeout(function () {completion.resolve(records);}, 0);
-        }
+        };
         if(to_get.length > 0) {
             var rpc_promise = this._super(to_get, fields, function(records) {
                 _.each(records, function(record, index) {
diff --git a/addons/web/static/src/js/form.js b/addons/web/static/src/js/form.js
index aac3b383dd3..7ed149305c0 100644
--- a/addons/web/static/src/js/form.js
+++ b/addons/web/static/src/js/form.js
@@ -214,7 +214,7 @@ openerp.web.FormView = openerp.web.View.extend( /** @lends openerp.web.FormView#
                         var ctx = widget.build_context ? widget.build_context() : {};
                         return ctx;
                     }
-                }
+                };
                 var parent_fields = null;
                 _.each(call[2].split(','), function(a, i) {
                     var field = _.trim(a);
@@ -1318,7 +1318,7 @@ openerp.web.form.dialog = function(content, options) {
     var dialog = new openerp.web.Dialog(null, options);
     dialog.$dialog = $(content).dialog(dialog.dialog_options);
     return dialog.$dialog;
-}
+};
 
 openerp.web.form.FieldMany2One = openerp.web.form.Field.extend({
     init: function(view, node) {
@@ -1411,13 +1411,13 @@ openerp.web.form.FieldMany2One = openerp.web.form.Field.extend({
             if (!self.$input.is(":focus") &&
                     !self.$input.autocomplete("widget").is(":visible") &&
                     !self.value) {
-                if(self.value === undefined && self.last_search.length > 0) {
+                if (self.value === undefined && self.last_search.length > 0) {
                     self._change_int_ext_value(self.last_search[0]);
                 } else {
                     self._change_int_ext_value(null);
                 }
             }
-        }
+        };
         this.$input.focusout(anyoneLoosesFocus);
 
         var isSelecting = false;
@@ -1479,7 +1479,7 @@ openerp.web.form.FieldMany2One = openerp.web.form.Field.extend({
                 }});
             }
             // quick create
-            var raw_result = _(data.result).map(function(x) {return x[1];})
+            var raw_result = _(data.result).map(function(x) {return x[1];});
             if (search_val.length > 0 &&
                 !_.include(raw_result, search_val) &&
                 (!self.value || search_val !== self.value[1])) {
@@ -2251,7 +2251,7 @@ openerp.web.form.FieldReference = openerp.web.form.Field.extend({
                     relation: null
                 }
             }
-        }
+        };
         this.get_fields_values = view.get_fields_values;
         this.do_onchange = this.on_form_changed = this.on_nop;
         this.widgets = {};
@@ -2361,7 +2361,7 @@ openerp.web.form.FieldBinary = openerp.web.form.Field.extend({
         } else {
             var url = '/web/binary/saveas?session_id=' + this.session.session_id + '&model=' +
                 this.view.dataset.model +'&id=' + (this.view.datarecord.id || '') + '&field=' + this.name +
-                '&fieldname=' + (this.node.attrs.filename || '') + '&t=' + (new Date().getTime())
+                '&fieldname=' + (this.node.attrs.filename || '') + '&t=' + (new Date().getTime());
             window.open(url);
         }
     },
@@ -2419,7 +2419,7 @@ openerp.web.form.FieldBinaryImage = openerp.web.form.FieldBinary.extend({
         this._super.apply(this, arguments);
         this.set_image_maxwidth();
         var url = '/web/binary/image?session_id=' + this.session.session_id + '&model=' +
-            this.view.dataset.model +'&id=' + (this.view.datarecord.id || '') + '&field=' + this.name + '&t=' + (new Date().getTime())
+            this.view.dataset.model +'&id=' + (this.view.datarecord.id || '') + '&field=' + this.name + '&t=' + (new Date().getTime());
         this.$image.attr('src', url);
     },
     set_image_maxwidth: function() {
diff --git a/addons/web/static/src/js/views.js b/addons/web/static/src/js/views.js
index e00b802c65f..8359a7e5d29 100644
--- a/addons/web/static/src/js/views.js
+++ b/addons/web/static/src/js/views.js
@@ -536,7 +536,7 @@ openerp.web.TranslateDialog = openerp.web.Dialog.extend({
                     self.$fields_form.find('.oe_trad_field[name="' + lg.code + '-' + f + '"]').val(values[0][f] || '').attr('data-value', values[0][f] || '');
                 });
                 deff.resolve();
-            }
+            };
             if (lg.code === self.view_language) {
                 var values = {};
                 _.each(self.translatable_fields_keys, function(field) {