PortletLuceneComparator does not implement Comparator interface properly

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 6.0.6 GA, 6.1.0 CE RC1
• Fix Version/s: --Sprint 12/11, 6.1.0 CE RC1
• Component/s: Operations > Performance
• Labels: None
• Environment: Windows xp, 6.0.6 GA
• Similar Issues: Show 5 results ; ;

  LPS-30296	Dynamic queries does not sort properly on multiple comparator fields
  LPS-8691	Service builder doesn't generate model interfaces, only implementation
  LPS-32589	The Document implementation does not obey its interface when adding numeric fields
  LPS-8264	Workflow comparators in kernel currently specific for jbpm.
  LPS-31166	The resourceURL taglib should return an instance of a class implementing the resourceURL interface when using the varImpl attribute

PortletLuceneComparator does not implement Comparator interface properly. Checking interface documentation states "The implementor must ensure that sgn(compare(x, y)) == -sgn(compare(y, x)) for all x and y." - It returns 1 only if the first argument is PortletKeys.DOCUMENT_LIBRARY, it should also return -1 if the second argument is PortletKeys.DOCUMENT_LIBRARY.

I have run a simulated test that successfully runs with new code.

Attached is possible fix.