Easylize code and make proper compare column function.
Unit Tests Skipped
scythetwirler and me agreed that we don't see much need for this, but adding it doesn't hurt.
Considering that we are going to add more useful columns in D125 and D209, the feature will become useful then.
So agreeing to have it.
"sortOrder = "1"" in the default.cfg is unreadable, but don't see the point in adding code to translate it to "ascending" and "descending" back and forth.
Don't save the settings on updateGameList() and updatePlayerList(), but only when the user actually changed the sorting order.
Also don't write the file twice when changing two entries.
You can move the saving to new functions, change all 4 values in one go with CreateValue and then do a (single) WriteFile.
Also that what scythetwirler remarked.
|328 ↗||(On Diff #765)|
You should upload patches with 9999 lines of context, so that we can see to which function you added changes.
elexis i stated that arrows sorting shown are wrong direction cause in name f.e. a is lowest char z highest and arrow peak shows up when thers lowest char on first. so i changed in code.
i made multi column ordering. in case double values like buddied or online status. so u order like status then name.
its saved like buddies delimited with ","
sort column and order in user config
ther some indent error on 'case rating': if (i !=0)...
i'll fix if code final
changes from comments -> rid off
rid off not needed inline
after all changes maybe inline function
kill if unneed just unshift
Move sort by column to front
kill while one splice enough
unshift(splice(obj,1) || obj);
sync to current base maybe kill some vars
two lines sync no vars