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

130030 gjames: Manual merge of r129730 and r129731:mmuller:Add a specific error message for invalid maxItem and skipCount. REPO-1061


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@130291 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Gethin James
2016-09-06 15:38:14 +00:00
parent b4bbb859e9
commit 580b464ad2

View File

@@ -356,11 +356,20 @@ public interface RecognizedParamsExtractor
}
catch (NumberFormatException error)
{
String errorMsg = "Invalid paging parameters skipCount: " + skip + ", maxItems:" + maxItems;
if (rpeLogger().isDebugEnabled())
{
rpeLogger().debug("Invalid paging params skip: " + skip + ",maxItems:" + maxItems);
rpeLogger().debug(errorMsg);
}
throw new InvalidArgumentException();
if (skip == null)
{
errorMsg = "Invalid paging parameter maxItems:" + maxItems;
}
if (maxItems == null)
{
errorMsg = "Invalid paging parameter skipCount:" + skip;
}
throw new InvalidArgumentException(errorMsg);
}
return Paging.valueOf(skipped, max);