Admin First/Last Category Sort bug

[jmestep]

Mazhar, one of our clients is getting frustrated with this bug

http://bugs.ablecommerce.com/show_bug.cgi?id=8585

Is there a page code fix you could post here for it?
Thanks

[mazhar]

Here is the complete page of category sorting with fix applied.

[jmestep]

Thanks, Mazhar. I tweaked the code in a 703 store, so I'll post the pertinent section here in case someone doesn't want to replace the entire file.

Code: Select all

In Admin/Catalog/SortCategory.aspx -- around line 22
Change
tempVal = sel[tempIndex].value;
	    tempText = sel[tempIndex].text;
	    sel[tempIndex].text = sel[newIndex].text;
	    sel[tempIndex].value = sel[newIndex].value;
	    sel[newIndex].text = tempText;
	    sel[newIndex].value = tempVal;
	    sel.selectedIndex = newIndex

to 

for(;tempIndex>newIndex;tempIndex--)
	    {
	        nextIndex = tempIndex-1; 
	        tempVal = sel[tempIndex].value;
	        tempText = sel[tempIndex].text;
	        sel[tempIndex].text = sel[nextIndex].text;
	        sel[tempIndex].value = sel[nextIndex].value;
	        sel[nextIndex].text = tempText;
	        sel[nextIndex].value = tempVal;
	        sel.selectedIndex = newIndex
	    }

Forgot to post this part

Code: Select all

around line 53
change 

tempVal= sel[tempIndex].value;
	    tempText = sel[tempIndex].text;
	    sel[tempIndex].text = sel[newIndex].text;
	    sel[tempIndex].value = sel[newIndex].value;
	    sel[newIndex].text = tempText;
	    sel[newIndex].value = tempVal;
	    sel.selectedIndex = newIndex   

to

for(;tempIndex<newIndex;tempIndex++)
	    {
	        nextIndex = tempIndex+1;
	        tempVal= sel[tempIndex].value;
	        tempText = sel[tempIndex].text;
	        sel[tempIndex].text = sel[nextIndex].text;
	        sel[tempIndex].value = sel[nextIndex].value;
	        sel[nextIndex].text = tempText;
	        sel[nextIndex].value = tempVal;
	        sel.selectedIndex = newIndex
	    }

[Falala]

Thanks, this was very helpful.