Index: branches/RC/core/admin_templates/js/tree.js =================================================================== diff -u -N -r11835 -r11932 --- branches/RC/core/admin_templates/js/tree.js (.../tree.js) (revision 11835) +++ branches/RC/core/admin_templates/js/tree.js (.../tree.js) (revision 11932) @@ -504,7 +504,7 @@ TreeFolder.prototype.removeChildren = function() { - for (var i in this.Children) { + for (var i=0; i options['max_height']) { @@ -336,7 +336,7 @@ Form.SetMinHeights = function() { var used = 0; - for (var i in this.Controls) { + for (var i = 0; i < this.Controls.length; i++) { var options = this.Options[this.Controls[i]] if (options['real_height'] < options['min_height']) { options['real_height'] = options.min_height; Index: branches/RC/core/admin_templates/js/script.js =================================================================== diff -u -N -r11931 -r11932 --- branches/RC/core/admin_templates/js/script.js (.../script.js) (revision 11931) +++ branches/RC/core/admin_templates/js/script.js (.../script.js) (revision 11932) @@ -1492,8 +1492,8 @@ } padding = [parseInt(style.paddingTop), parseInt(style.paddingRight), parseInt(style.paddingBottom), parseInt(style.paddingLeft)] border = [parseInt(style.borderTopWidth), parseInt(style.borderRightWidth), parseInt(style.borderBottomWidth), parseInt(style.borderLeftWidth)] - for (var i in padding) if ( isNaN( padding[i] ) ) padding[i] = 0 - for (var i in border) if ( isNaN( border[i] ) ) border[i] = 0 + for (var i = 0; i < padding.length; i++) if ( isNaN( padding[i] ) ) padding[i] = 0 + for (var i = 0; i < border.length; i++) if ( isNaN( border[i] ) ) border[i] = 0 var result = new Object(); result.innerHeight = obj.clientHeight - padding[0] - padding[2]; Index: branches/RC/core/admin_templates/js/grid.js =================================================================== diff -u -N -r11929 -r11932 --- branches/RC/core/admin_templates/js/grid.js (.../grid.js) (revision 11929) +++ branches/RC/core/admin_templates/js/grid.js (.../grid.js) (revision 11932) @@ -314,7 +314,7 @@ this.prefix = prefix; this.class_on = class_on; this.class_off = class_off; - this.Items = new Array(); + this.Items = {}; this.LastSelectedSequence = 1; this.LastSelectedId = null; this.DblClick = dbl_click; @@ -474,7 +474,7 @@ Grid.prototype.SetDependantToolbarButtons = function($buttons, $direct, $mode) { if (!isset($direct)) $direct = true; // direct (use false for invert mode) if (!isset($mode)) $mode = 1; // enable/disable (use 2 for show/hide mode) - for (var i in $buttons) { + for (var i = 0; i < $buttons.length; i++) { this.DependantButtons.push(new Array($buttons[i], $direct, $mode)); } //this.DependantButtons = buttons; @@ -485,7 +485,7 @@ { // alert('prefix: ' + this.prefix + '; ' + $called_from + ' -> Grid.CheckDependencies'); var enabling = (this.CountSelected() > 0); - for (var i in this.DependantButtons) { + for (var i = 0; i < this.DependantButtons.length; i++) { if (this.DependantButtons[i][0].match("portal:(.*)")) { button_name = RegExp.$1; if (toolbar) { Index: branches/RC/core/admin_templates/js/grid_scroller.js =================================================================== diff -u -N -r11599 -r11932 --- branches/RC/core/admin_templates/js/grid_scroller.js (.../grid_scroller.js) (revision 11599) +++ branches/RC/core/admin_templates/js/grid_scroller.js (.../grid_scroller.js) (revision 11932) @@ -28,7 +28,7 @@ var value = sheetRules[ii].selectorText; if (value.match(',')) { // Mozilla does not break comma-separated selectors into separate rules... var subselectors = value.split(','); - for (var sub in subselectors) { + for (var sub = 0; sub < subselectors.length; sub++) { var real_name = subselectors[sub].replace(/^[ \t]*(.*)[ \t]*/, '$1'); this.Map[real_name] = sheetRules[ii] } Index: branches/RC/core/admin_templates/js/ajax.js =================================================================== diff -u -N -r11865 -r11932 --- branches/RC/core/admin_templates/js/ajax.js (.../ajax.js) (revision 11865) +++ branches/RC/core/admin_templates/js/ajax.js (.../ajax.js) (revision 11932) @@ -16,7 +16,7 @@ Request.timeout = 60000; //60 seconds Request.method = 'GET'; -Request.headers = new Array(); +Request.headers = {}; Request.params = null; Request.progressText = 'Loading ...'; @@ -92,7 +92,7 @@ } $request.setRequestHeader($header_name, Request.headers[$header_name]); } - Request.headers = new Array(); // reset header afterwards + Request.headers = {}; // reset header afterwards } Request.getRequest = function() { Index: branches/RC/core/admin_templates/js/form_controls.js =================================================================== diff -u -N -r11854 -r11932 --- branches/RC/core/admin_templates/js/form_controls.js (.../form_controls.js) (revision 11854) +++ branches/RC/core/admin_templates/js/form_controls.js (.../form_controls.js) (revision 11932) @@ -8,9 +8,9 @@ this.Permissions = new Array (); // action groups allowed this.Messages = new Array (); // various phrase (errors, confirmations, button titles) - this.Controls = new Array (); // controls used for editing list content + this.Controls = {}; // controls used for editing list content this.Records = new Array (); // data to be submitted (needs to be parsed using formatters) - this.Errors = new Array (); // error messages in fields + this.Errors = {}; // error messages in fields this.InEditing = false; } @@ -180,7 +180,7 @@ this.setControlValue($field_name, null); } - this.Errors = new Array (); + this.Errors = {}; } MultiInputControl.prototype.CancelEditing = function() { @@ -264,7 +264,7 @@ this.ProcessXMLNode($child, $child.tagName); } else if ($child.tagName == 'errors') { - this.Errors = new Array (); + this.Errors = {}; this.ProcessXMLNode($child, $child.tagName); } } @@ -307,7 +307,7 @@ } MultiInputControl.prototype.prepareRecord = function() { - var $record = new Array (); + var $record = {}; for (var $field_name in this.Controls) { $record[$field_name] = this.getControlValue($field_name); }