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.