From 0fbe2e59b05647f12f81f4f3717322e9d90d5cc2 Mon Sep 17 00:00:00 2001
From: Neil McErlean <neil.mcerlean@alfresco.com>
Date: Wed, 6 May 2015 16:11:16 +0000
Subject: [PATCH] RM-2113. Acting on review comments.

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/modules/recordsmanagement/HEAD@103781 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../classification/UserQueryParams.java                  | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/classification/UserQueryParams.java b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/classification/UserQueryParams.java
index 8753859cec..0396d0f8d4 100644
--- a/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/classification/UserQueryParams.java
+++ b/rm-server/source/java/org/alfresco/module/org_alfresco_module_rm/classification/UserQueryParams.java
@@ -101,7 +101,14 @@ public final class UserQueryParams
 
         if (otherElems != null)
         {
-            elementList.addAll(asList(otherElems));
+            final List<T> tList = asList(otherElems);
+            final int firstNull = tList.indexOf(null);
+            if (firstNull != -1)
+            {
+                // "+ 2" so that position 1 points to 'firstElem' and so on through otherElems.
+                throw new IllegalArgumentException("Unexpected null element at position " + firstNull + 2);
+            }
+            elementList.addAll(tList);
         }
 
         return elementList;