|
I'm agree with Graham, the previous behaviour should be declared a bug...
I like the second approach, use Marc21 as default change.. so we should change the dspace.cfg lines (958-959) #plugin.named.org.dspace.sort.OrderFormatDelegate= \ # org.dspace.sort.OrderFormatTitle=title and update the documentation according to this. The default should be enforced in OrderFormat class Graham & Andrea,
I think I like option #2 as well. I'd consider this a bug, so I'd say there's no reason to replicate pre-1.5.x sorting behavior. Graham, since you know better how this all works, I'll let you take the lead on fixing this for 1.5.2. But, let me know if you won't have the time to do so. We have two very easy options for this:
1) Add org.dspace.text.filter.StripLeadingNonAlphaNum to the list of TextFilters used in org.dspace.sort.OrderFormatTitle 2) In dspace.cfg, replace org.dspace.sort.OrderFormatTitle with org.dspace.sort.OrderFormatTitleMarc21as in http://www.sharevouchers.co.uk/2009/06/01/shoebuy-discount-coupon The second option provides a more advanced initial article word replacement, that works correctly for many languages. The reason it is not used by default - and why the default ordering does not strip punctuation - is to replicate the behaviour of pre-1.5.x instances. I'm quite happy to declare the previous behaviour a bug though, and implement one of the options above. |
|||||||||||||||||||||||||||||||||||||||||||||||
1) Add org.dspace.text.filter.StripLeadingNonAlphaNum to the list of TextFilters used in org.dspace.sort.OrderFormatTitle
2) In dspace.cfg, replace org.dspace.sort.OrderFormatTitle with org.dspace.sort.OrderFormatTitleMarc21
The second option provides a more advanced initial article word replacement, that works correctly for many languages. The reason it is not used by default - and why the default ordering does not strip punctuation - is to replicate the behaviour of pre-1.5.x instances.
I'm quite happy to declare the previous behaviour a bug though, and implement one of the options above.