ETHREEOH-2126 - WCM meta data extraction does not extract title when form is created

- metadata extractor was not firing 
 - titled aspect was applied after extraction thereby overwriting extracted value.

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

source/java/org/alfresco/repo/avm/AVMNodeService.java

@@ -1418,6 +1418,13 @@ public class AVMNodeService extends AbstractNodeServiceImpl implements NodeServi
         {
             throw new InvalidNodeRefException("Read only store.", nodeRef);
         }
+        
+        Map<QName, Serializable> propsBefore = null;
+        if (fInvokePolicies)
+        {
+            propsBefore = getProperties(nodeRef);
+        }
+        
         if (isBuiltInProperty(qname))
         {
             if (qname.equals(ContentModel.PROP_CONTENT))
@@ -1427,7 +1434,6 @@ public class AVMNodeService extends AbstractNodeServiceImpl implements NodeServi
                     fAVMService.setContentData(avmVersionPath.getSecond(), (ContentData)value);
                     if (fInvokePolicies)
                     {
-                        Map<QName, Serializable> propsBefore = getProperties(nodeRef);
                         Map<QName, Serializable> propsAfter = new HashMap<QName, Serializable>(propsBefore);
                         propsAfter.put(ContentModel.PROP_CONTENT, value);
                         invokeOnUpdateProperties(nodeRef, propsBefore, propsAfter);
@@ -1442,11 +1448,7 @@ public class AVMNodeService extends AbstractNodeServiceImpl implements NodeServi
         }
         try
         {
-            Map<QName, Serializable> propsBefore = null;
-            if (fInvokePolicies)
-            {
-                propsBefore = getProperties(nodeRef);
-            }
+
             PropertyDefinition propertyDef = dictionaryService.getProperty(qname);
             PropertyValue propertyValue = makePropertyValue(propertyDef, value);
             fAVMService.setNodeProperty(avmVersionPath.getSecond(), qname, propertyValue);

source/java/org/alfresco/repo/avm/AVMServiceImpl.java

@@ -386,10 +386,16 @@ public class AVMServiceImpl implements AVMService
      */
     public void createFile(String path, String name, InputStream in, List<QName> aspects, Map<QName, PropertyValue> properties)
     {
-        if (path == null || name == null || in == null || !FileNameValidator.isValid(name))
+        if (path == null || name == null || in == null)
         {
             throw new AVMBadArgumentException("Illegal argument.");
         }
+        
+        if(!FileNameValidator.isValid(name))
+        {
+            throw new AVMBadArgumentException("Illegal argument, name:" + name);
+        }
+        
         // Save the contents to temp space.
         File temp;
         try

source/java/org/alfresco/repo/domain/PropertyValue.java

@@ -670,7 +670,8 @@ public class PropertyValue implements Cloneable, Serializable
     /**
      * Construct a new property value.
      * 
-     * @param typeQName the dictionary-defined property type to store the property as
+     * @param typeQName the dictionary-defined property type to store the property as.   
+     * May be null in which case the type will be determined from the value parameter.
      * @param value the value to store.  This will be converted into a format compatible
      *      with the type given
      *