Merged 5.2.N (5.2.1) to HEAD (5.2)

130555 jvonka: REPO-1059: REST API: Invalid orderBy field direction is ignored (hence defaults to ascending rather than returning an error)
   - implement fix & update test
   - note: InvalidArgumentException maps to HTTP 400
   - note: we already have tests for case-insensitive (ie. we allow ASC, Asc, AsC, DESC, Desc, DeSC, or similar)


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@132144 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

source/java/org/alfresco/rest/framework/tools/RecognizedParamsExtractor.java

@@ -315,6 +315,7 @@ public interface RecognizedParamsExtractor
                         {
                             rpeLogger().debug("Invalid sort order definition (" + sortDef + ").  Valid values are " + SortColumn.ASCENDING + " or "
                                         + SortColumn.DESCENDING + ".");
+                            throw new InvalidArgumentException("Unknown sort order direction: "+sortDef+" expected: asc or desc");
                         }
                     }
                     sortedColumns.add(new SortColumn(columnName, SortColumn.ASCENDING.equals(sortOrder)));