: Yes, I thought about adding some 'new syntax', but I opted for a separate 
'facet.sortorder' parameter,
: 
: mainly because I'm not familiar enough with the codebase to know what effect 
this might have on
: 
: backward compatibility. It would be easy enough to modify the patch I created 
to do it this way.

it shouldn't really affect anything -- it wouldn't really be new syntax, 
just extending hte existing "sort" param syntax to apply to the 
"facet.sort" param.  The only back compat concern is making sure we 
continue to support true/false as aliases, and having the default order 
match the current bahvior if asc/desc aren't specified.


-Hoss

Reply via email to