From 8b0fa4a821f7416b2f3e857efcf6dca21fe76c4c Mon Sep 17 00:00:00 2001
From: Roy Wetherall <roy.wetherall@alfresco.com>
Date: Fri, 20 Jan 2006 11:06:44 +0000
Subject: [PATCH] - Fixed bug raised on forum that category values can be
 applied more than once.

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@2156 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../repo/action/executer/LinkCategoryActionExecuter.java   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/source/java/org/alfresco/repo/action/executer/LinkCategoryActionExecuter.java b/source/java/org/alfresco/repo/action/executer/LinkCategoryActionExecuter.java
index a6746d1947..d4af818515 100644
--- a/source/java/org/alfresco/repo/action/executer/LinkCategoryActionExecuter.java
+++ b/source/java/org/alfresco/repo/action/executer/LinkCategoryActionExecuter.java
@@ -131,8 +131,11 @@ public class LinkCategoryActionExecuter extends ActionExecuterAbstractBase
                         // Append the category value to the existing values
                         Serializable value = this.nodeService.getProperty(actionedUponNodeRef, categoryProperty);
                         Collection<NodeRef> categories = DefaultTypeConverter.INSTANCE.getCollection(NodeRef.class, value);
-                        categories.add(categoryValue);
-                        this.nodeService.setProperty(actionedUponNodeRef, categoryProperty, (Serializable)categories);
+                        if (categories.contains(categoryValue) == false)
+                        {
+                            categories.add(categoryValue);
+                            this.nodeService.setProperty(actionedUponNodeRef, categoryProperty, (Serializable)categories);
+                        }
                     }
                 }
             }