From e3227a03cd3e98fa84326dc7c8dc5d27037a836c Mon Sep 17 00:00:00 2001
From: Mike Hatfield <mike.hatfield@alfresco.com>
Date: Fri, 15 Feb 2008 17:52:41 +0000
Subject: [PATCH] Fix for forum-reported NPE with ajax category selector

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@8297 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../ui/repo/component/BaseAjaxItemPicker.java | 29 ++++++++++---------
 1 file changed, 16 insertions(+), 13 deletions(-)

diff --git a/source/java/org/alfresco/web/ui/repo/component/BaseAjaxItemPicker.java b/source/java/org/alfresco/web/ui/repo/component/BaseAjaxItemPicker.java
index 83444d0317..9b49013a45 100644
--- a/source/java/org/alfresco/web/ui/repo/component/BaseAjaxItemPicker.java
+++ b/source/java/org/alfresco/web/ui/repo/component/BaseAjaxItemPicker.java
@@ -140,25 +140,28 @@ public abstract class BaseAjaxItemPicker extends UIInput
       Map requestMap = context.getExternalContext().getRequestParameterMap();
       String fieldId = getHiddenFieldName();
       String value = (String)requestMap.get(fieldId);
-      if (value.equals("empty"))
+      if (value != null)
       {
-         this.setSubmittedValue(new String("empty"));
-      }
-      else if (value != null && value.length() != 0)
-      {
-         if (getSingleSelect() == true)
+         if (value.equals("empty"))
          {
-            NodeRef ref = new NodeRef(value);
-            this.setSubmittedValue(ref);
+            this.setSubmittedValue(new String("empty"));
          }
-         else
+         else if (value.length() != 0)
          {
-            List<NodeRef> refs = new ArrayList<NodeRef>(5);
-            for (StringTokenizer t = new StringTokenizer(value, ","); t.hasMoreTokens(); /**/)
+            if (getSingleSelect() == true)
             {
-               refs.add(new NodeRef(t.nextToken()));
+               NodeRef ref = new NodeRef(value);
+               this.setSubmittedValue(ref);
+            }
+            else
+            {
+               List<NodeRef> refs = new ArrayList<NodeRef>(5);
+               for (StringTokenizer t = new StringTokenizer(value, ","); t.hasMoreTokens(); /**/)
+               {
+                  refs.add(new NodeRef(t.nextToken()));
+               }
+               this.setSubmittedValue(refs);
             }
-            this.setSubmittedValue(refs);
          }
       }
    }