Author: vmolotkov Date: 2007-11-26 05:09:53 -0500 (Mon, 26 Nov 2007) New Revision: 4251 Modified: branches/3.1.x/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/OrderingList.js Log: bug: RF-1398 Modified: branches/3.1.x/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/OrderingList.js =================================================================== --- branches/3.1.x/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/OrderingList.js 2007-11-26 08:54:41 UTC (rev 4250) +++ branches/3.1.x/ui/orderingList/src/main/resources/org/richfaces/renderkit/html/scripts/OrderingList.js 2007-11-26 10:09:53 UTC (rev 4251) @@ -32,23 +32,28 @@ this.selectedItems.sort(this.compareByRowIndex); var control; //FIXME - if ((this.shuttleItems.length <= 1) || (this.selectedItems.length == 0)) { - this.controlsProcessing(["first", "last", "down", "up"]); - } else if (this.selectedItems[0].rowIndex == 0) { - this.controlsProcessing(["first", "up"]); - } else if (this.selectedItems[this.selectedItems.length - 1].rowIndex == (this.shuttleItems.length - 1)) { - this.controlsProcessing(["down", "last"]); - } else { - this.controlsProcessing(); + this.controlsProcessing(["first", "last", "down", "up"], "enable"); + if ((this.shuttleItems.length <= 1) || (this.selectedItems.length == 0)) + this.controlsProcessing(["first", "last", "down", "up"], "disable"); + else { + if (this.selectedItems[0].rowIndex == 0) + this.controlsProcessing(["first", "up"], "disable"); + if (this.selectedItems[this.selectedItems.length - 1].rowIndex == (this.shuttleItems.length - 1)) + this.controlsProcessing(["down", "last"], "disable"); } }, - controlsProcessing : function(disabledControls) { + controlsProcessing : function(disabledControls , action) { for (var i = 0; i < this.controlList.length; i++) { control = this.controlList[i]; if (control != null) { - if (disabledControls != null && disabledControls.indexOf(control.action) != -1) control.doDisable(); - else control.doEnable(); + if (disabledControls != null && disabledControls.indexOf(control.action) != -1) { + if (action == "disable") { + control.doDisable(); + } else { + control.doEnable(); + } + } } } },