Page 1 of 1

Admin First/Last Category Sort bug

Posted: Wed Apr 28, 2010 12:35 pm
by 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

Re: Admin First/Last Category Sort bug

Posted: Wed Apr 28, 2010 12:50 pm
by mazhar
Here is the complete page of category sorting with fix applied.

Re: Admin First/Last Category Sort bug

Posted: Thu Apr 29, 2010 7:58 am
by 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
	    }

Re: Admin First/Last Category Sort bug

Posted: Wed Oct 27, 2010 11:48 am
by Falala
Thanks, this was very helpful.