Multilingual Content Fixes

- Fixed parsing of locale string to use I18NUtil.parseLocale and not Locale constructor
 - Fixed type conversion of properties returned from MLPropertyInterceptor's directNodeService
 - Fixed handling of ALL_LANGUAGES filter selection
 - Fixed MultilingualContentService NPE when checking for duplicate locales


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@5007 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

source/java/org/alfresco/repo/model/ml/MultilingualContentServiceImpl.java

@@ -217,12 +217,6 @@ public class MultilingualContentServiceImpl implements MultilingualContentServic
             // The aspect is present, so just ensure that the locale is correct
             nodeService.setProperty(contentNodeRef, ContentModel.PROP_LOCALE, locale);
         }
-        //get all the languages already there
-        Map<Locale, NodeRef> existingLanguages = this.getTranslations(mlContainerNodeRef);
-        if (existingLanguages.containsKey(locale))
-        {
-            throw new AlfrescoRuntimeException("Duplicate locale in document pool:" + locale.toString());
-        }
         
         // Do we make use of an existing container?
         if (mlContainerNodeRef == null)
@@ -232,6 +226,13 @@ public class MultilingualContentServiceImpl implements MultilingualContentServic
         }
         else
         {
+            // Check that the language is not duplicated
+            Map<Locale, NodeRef> existingLanguages = this.getTranslations(mlContainerNodeRef);
+            if (existingLanguages.containsKey(locale))
+            {
+                throw new AlfrescoRuntimeException("Duplicate locale in document pool: " + locale);
+            }
+            
             // Use the existing container
             nodeService.addChild(
                     mlContainerNodeRef,

source/java/org/alfresco/repo/node/MLPropertyInterceptor.java

@@ -28,6 +28,7 @@ import org.alfresco.service.cmr.dictionary.PropertyDefinition;
 import org.alfresco.service.cmr.repository.MLText;
 import org.alfresco.service.cmr.repository.NodeRef;
 import org.alfresco.service.cmr.repository.NodeService;
+import org.alfresco.service.cmr.repository.datatype.DefaultTypeConverter;
 import org.alfresco.service.namespace.QName;
 import org.aopalliance.intercept.MethodInterceptor;
 import org.aopalliance.intercept.MethodInvocation;
@@ -109,7 +110,16 @@ public class MLPropertyInterceptor implements MethodInterceptor
     @SuppressWarnings("unchecked")
     public Object invoke(MethodInvocation invocation) throws Throwable
     {
+        String methodName = invocation.getMethod().getName();
+        Object[] args = invocation.getArguments();
+
+        // What locale must be used for filtering
         Locale contentLocale = I18NUtil.getContentLocale();
+        if (logger.isDebugEnabled())
+        {
+            logger.debug("Intercepting method " + methodName + " using content filter " + contentLocale);
+        }
+        
         Object ret = null;
         
         // If isMLAware then no treatment is done, just return
@@ -119,7 +129,6 @@ public class MLPropertyInterceptor implements MethodInterceptor
             return invocation.proceed();
         }
         
-        String methodName = invocation.getMethod().getName();
         if (methodName.equals("getProperty"))
         {
             ret = invocation.proceed();
@@ -161,7 +170,7 @@ public class MLPropertyInterceptor implements MethodInterceptor
                 }
             }
             ret = convertedProperties;
-            // done
+            // Done
             if (logger.isDebugEnabled())
             {
                 logger.debug(
@@ -172,99 +181,38 @@ public class MLPropertyInterceptor implements MethodInterceptor
         }
         else if (methodName.equals("setProperties"))
         {
-            //get the raw properties and for every multi lingual property, 
+            NodeRef nodeRef = (NodeRef) args[0];
-            //take the value transmited as parameter and if type string put the value in
+            Map<QName, Serializable> newProperties =(Map<QName, Serializable>) args[1];
-            //the multilingual property.  If not just transfer property value
+            // Get the current properties for the node
-            NodeRef node = (NodeRef)invocation.getArguments()[0];
+            Map<QName, Serializable> currentProperties = directNodeService.getProperties(nodeRef);
-            //new properties
+            // Convert all properties
-            Map<QName, Serializable> newProperties =(Map<QName, Serializable>)(invocation.getArguments()[1]);
-            //get the raw property values
-            Map<QName, Serializable> previousProperties = directNodeService.getProperties(node);
-            //merge with previous properties only for MLText ones
             Map<QName, Serializable> convertedProperties = new HashMap<QName, Serializable>(newProperties.size() * 2);              
             for (Map.Entry<QName, Serializable> entry : newProperties.entrySet())
             {
-                 QName key = entry.getKey();
+                 QName propertyQName = entry.getKey();
-                 PropertyDefinition propertyDef = this.dictionaryService.getProperty(key);
+                 Serializable inboundValue = entry.getValue();
-                 if (propertyDef == null)
+                 // Get the current property value
-                 {
+                 Serializable currentValue = currentProperties.get(propertyQName);
-                     //no type found in the model, just transfer
+                 // Convert the inbound property value
-                     convertedProperties.put(key,newProperties.get(key));
+                 inboundValue = convertInboundProperty(contentLocale, nodeRef, propertyQName, inboundValue, currentValue);
-                     continue;
+                 // Put the value into the map
-                 }
+                 convertedProperties.put(propertyQName, inboundValue);
-                 //if incoming new property is MLText(based on the model) then 
+            }
-                 //transfer or merge depending on the incoming concrete type.
+            // Now complete the call by passing the converted properties
-                 //if incoming type is String and model indicates MLText then merge if something else just transfer
+            directNodeService.setProperties(nodeRef, convertedProperties);
-                 if(propertyDef.getDataType().getName().equals(DataTypeDefinition.MLTEXT) && 
+            // Done
-                         newProperties.get(key) instanceof String )
-                 {
-                     //get back previous value it should a MLText or null
-                     
-                     Serializable preVal = previousProperties.get(key);
-                     MLText newVal = new MLText();
-                     if (preVal == null || !(preVal instanceof MLText))
-                     {
-                         //second test is in case of the value was set before in the system without MLText
-                         //create a new MLText and transfer
-                         //if prevval wasn't null save value with as LOCAL.ENGLISH (purely arbitrary) to avoid any 
-                         //information loss
-                         if(preVal!= null)
-                             newVal.addValue(Locale.ENGLISH,(String)preVal);
-                     }
-                     else
-                     {
-                        newVal = (MLText)preVal; 
-                     }
-                     //use alternate locale
-                      newVal.addValue(
-                              contentLocale,
-                              (String)newProperties.get(key));
-                     //transfer
-                     convertedProperties.put(key,newVal);
-                     continue;
-                 }
-                 //normal process, just transfer
-                 convertedProperties.put(key,newProperties.get(key));
-            }//for
-            directNodeService.setProperties(node, convertedProperties);
         }
         else if (methodName.equals("setProperty"))
         {
             //check if the property is of type MLText
-            QName qPropName = (QName)invocation.getArguments()[1];
+            NodeRef nodeRef = (NodeRef) args[0];
-            PropertyDefinition propertyDef = this.dictionaryService.getProperty(qPropName);
+            QName propertyQName = (QName) args[1];
-            //if no type definition associated to the name then just proceed
+            Serializable inboundValue = (Serializable) args[2];
-            if (propertyDef != null && propertyDef.getDataType().getName().equals(DataTypeDefinition.MLTEXT))
+            // Convert the property
-            {
+            inboundValue = convertInboundProperty(contentLocale, nodeRef, propertyQName, inboundValue, null);
-                
+            // Pass this through to the node service
-                //check if property is multi valued or actual set value is MLText
+            directNodeService.setProperty(nodeRef, propertyQName, inboundValue);
-                // in that case delegate deletate
+            // Done
-                if (propertyDef.isMultiValued() || (invocation.getArguments()[2] instanceof MLText) )
-                {
-                    ret = invocation.proceed();
-                }
-                else
-                {
-                    //this is a multilingual mono valued property
-                    //then get the MLText value and set the linguistic value conrresponding 
-                    //of the preferate language
-                    NodeRef node = (NodeRef)invocation.getArguments()[0];
-                    MLText mlPropertyValue = (MLText)directNodeService.getProperty(node, qPropName);
-                    if (mlPropertyValue == null)
-                    {
-                        mlPropertyValue = new MLText();
-                    }
-                    mlPropertyValue.addValue(
-                            contentLocale,
-                            (String) invocation.getArguments()[2]);
-                    //set the value following according to the current locale
-                    directNodeService.setProperty(node, qPropName, mlPropertyValue);
-                }
-            }
-            else
-            {
-                ret = invocation.proceed();
-            }
         }
         else
         {
@@ -273,4 +221,68 @@ public class MLPropertyInterceptor implements MethodInterceptor
         // done
         return ret;
     }
+    
+    /**
+     * 
+     * @param inboundValue      The value that must be set
+     * @param currentValue      The current value of the property or <tt>null</tt> if not known
+     * @return                  Returns a potentially converted property that conforms to the model
+     */
+    private Serializable convertInboundProperty(
+            Locale contentLocale,
+            NodeRef nodeRef,
+            QName propertyQName,
+            Serializable inboundValue,
+            Serializable currentValue)
+    {
+        Serializable ret = null;
+        PropertyDefinition propertyDef = this.dictionaryService.getProperty(propertyQName);
+        //if no type definition associated to the name then just proceed
+        if (propertyDef != null && propertyDef.getDataType().getName().equals(DataTypeDefinition.MLTEXT))
+        {
+            // Don't mess with multivalued properties or instances already of type MLText
+            if (propertyDef.isMultiValued() || (inboundValue instanceof MLText) )
+            {
+                ret = inboundValue;
+            }
+            else
+            {
+                // This is a multilingual single-valued property
+                // Get the current value from the node service, if not provided
+                if (currentValue == null)
+                {
+                    currentValue = directNodeService.getProperty(nodeRef, propertyQName);
+                }
+                MLText currentMLValue = null;
+                if (currentValue == null)
+                {
+                    currentMLValue = new MLText();
+                }
+                else
+                {
+                    currentMLValue = DefaultTypeConverter.INSTANCE.convert(MLText.class, currentValue);
+                }
+                // Force the inbound value to be a String (it isn't MLText)
+                String inboundValueStr = DefaultTypeConverter.INSTANCE.convert(String.class, inboundValue);
+                // Add it to the current MLValue
+                currentMLValue.put(contentLocale, inboundValueStr);
+                // Done
+                ret = currentMLValue;
+            }
+        }
+        else            // It is not defined as d:mltext in the dictionary
+        {
+            ret = inboundValue;
+        }
+        // Done
+        if (logger.isDebugEnabled() && ret != inboundValue)
+        {
+            logger.debug("Converted inbound property: \n" +
+                    "   NodeRef:    " + nodeRef + "\n" +
+                    "   Property:   " + propertyQName + "\n" +
+                    "   Before:     " + inboundValue + "\n" +
+                    "   After:      " + ret);
+        }
+        return ret;
+    }
 }