From ce9140e560d6b1d7e8dcf0cfd6b0e715bacba920 Mon Sep 17 00:00:00 2001
From: gpawel <gpawel>
Date: Mon, 6 Feb 2012 13:13:43 +0000
Subject: [PATCH] LMS-2773 YeastLab: managed properties stopped working

SVN: 24361
---
 .../structured/XmlStructuredPropertyConverter.java    | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/managed_property/structured/XmlStructuredPropertyConverter.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/managed_property/structured/XmlStructuredPropertyConverter.java
index 80f880171a8..38df64987e4 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/managed_property/structured/XmlStructuredPropertyConverter.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/shared/managed_property/structured/XmlStructuredPropertyConverter.java
@@ -35,6 +35,7 @@ import org.w3c.dom.NodeList;
 
 import ch.systemsx.cisd.base.exceptions.CheckedExceptionTunnel;
 import ch.systemsx.cisd.common.shared.basic.utils.StringUtils;
+import ch.systemsx.cisd.openbis.generic.shared.basic.dto.ManagedProperty;
 import ch.systemsx.cisd.openbis.generic.shared.basic.dto.api.IManagedProperty;
 import ch.systemsx.cisd.openbis.generic.shared.managed_property.api.EntityLinkElementKind;
 import ch.systemsx.cisd.openbis.generic.shared.managed_property.api.IElement;
@@ -61,13 +62,12 @@ public class XmlStructuredPropertyConverter implements IStructuredPropertyConver
 
     public List<IElement> convertToElements(IManagedProperty property)
     {
-        return property.isSpecialValue() ? Collections.<IElement> emptyList()
-                : convertStringToElements(property.getValue());
+        return convertStringToElements(property.getValue());
     }
 
     public List<IElement> convertStringToElements(String propertyValue)
     {
-        if (StringUtils.isBlank(propertyValue))
+        if (ManagedProperty.isSpecialValue(propertyValue) || StringUtils.isBlank(propertyValue))
         {
             return Collections.<IElement> emptyList();
         }
@@ -93,7 +93,7 @@ public class XmlStructuredPropertyConverter implements IStructuredPropertyConver
         root.setChildren(elements);
         return root;
     }
-    
+
     private Node transformToDOM(IElement element, Document document)
     {
         Node result = document.createElement(element.getName());
@@ -110,7 +110,6 @@ public class XmlStructuredPropertyConverter implements IStructuredPropertyConver
         return result;
     }
 
-
     private IElement transformFromDOM(Node node)
     {
         IElement result = createElementForNode(node);
@@ -172,7 +171,7 @@ public class XmlStructuredPropertyConverter implements IStructuredPropertyConver
         {
             String permId = getAttrValueOrFail(node, EntityLinkElement.PERMID_ATTR_NAME);
             return new EntityLinkElement(linkKind, permId);
-        } 
+        }
 
         // plain element, no special treatment needed
         return factory.createElement(nodeName);
-- 
GitLab