Merged HEAD-BUG-FIX (4.3/Cloud) to HEAD (4.3/Cloud)

60032: Merged BRANCHES/DEV/43UIDEV to HEAD-BUG-FIX:
      This branch contained the Aikau and Surf framework updates implemented during the development hiatus post-4.2 completion. This features the change of Surf from 1.2.0 to Surf 1.2.1 which is for v4.3 releases only.


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

config/alfresco/model/shareUiModel.xml

@@ -21,6 +21,49 @@
          <title>Surf AMD Page</title>
          <parent>cm:content</parent>
       </type>
+
+      <!-- A Surf application type-->
+      <type name="surf:applicationType">
+         <title>Surf Application Type</title>
+         <parent>cm:folder</parent>
+         <properties>
+            <property name="surf:rootRage">
+               <type>d:text</type>
+               <mandatory enforced="true">true</mandatory>
+               <multiple>false</multiple>
+               <index enabled="true">
+                  <tokenised>false</tokenised>
+               </index>
+            </property>
+            <property name="surf:groupVisibility">
+               <type>d:text</type>
+               <mandatory>false</mandatory>
+               <multiple>false</multiple>
+               <index enabled="true">
+                  <tokenised>false</tokenised>
+               </index>
+            </property>
+         </properties>
+      </type>
+
+      <!-- A Surf application instance -->
+      <type name="surf:applicationInstance">
+         <title>Surf Application Instance</title>
+         <parent>cm:folder</parent>
+         <associations>
+            <association name="surf:applicationType">
+               <source>
+                  <mandatory>false</mandatory>
+                  <many>true</many>
+               </source>
+               <target>
+                  <class>surf:applicationType</class>
+                  <mandatory>true</mandatory>
+                  <many>false</many>
+               </target>
+            </association>
+         </associations>
+      </type>
       
    </types>
    

pom.xml

@@ -146,10 +146,11 @@
                 </exclusion>
             </exclusions>
         </dependency>
+        <!-- Dependency on Alfresco patched version of Rhino available in artfifact.alfresco.com -->
         <dependency>
-            <groupId>rhino</groupId>
+            <groupId>org.mozilla</groupId>
-            <artifactId>js</artifactId>
+            <artifactId>rhino</artifactId>
-            <version>1.6R7-patched</version>
+            <version>1.7R4-alfresco-patched</version>
         </dependency>
         <dependency>
             <groupId>org.springframework</groupId>
@@ -373,9 +374,9 @@
         </dependency>
         <!-- Use patched FreeMarker to fix ALF-10188 -->
         <dependency>
-            <groupId>org.freemarker</groupId>
+            <groupId>org.alfresco</groupId>
             <artifactId>freemarker</artifactId>
-            <version>2.3.18-alfresco-patched-2</version>
+            <version>2.3.20-alfresco-patched</version>
         </dependency>
         <dependency>
             <groupId>org.apache.cxf</groupId>

source/java/org/alfresco/repo/jscript/Search.java

@@ -533,6 +533,8 @@ public class Search extends BaseScopableProcessorExtension
     {
         Object[] results = null;
         
+        // convert values from JS to Java - may contain native JS object such as ConsString etc.
+        search = new ValueConverter().convertValueForJava(search);
         if (search instanceof Serializable)
         {
             Serializable obj = new ValueConverter().convertValueForRepo((Serializable)search);

source/java/org/alfresco/repo/jscript/ValueConverter.java

@@ -158,11 +158,11 @@ public class ValueConverter
         }
         else if (value instanceof ChildAssociation)
         {
-        	   value = ((ChildAssociation)value).getChildAssociationRef();
+            value = ((ChildAssociation)value).getChildAssociationRef();
         }
         else if (value instanceof Association)
         {
-        	   value = ((Association)value).getAssociationRef();
+            value = ((Association)value).getAssociationRef();
         }
         else if (value instanceof Wrapper)
         {
@@ -275,6 +275,12 @@ public class ValueConverter
             }
             value = list;
         }
+        else if (value instanceof CharSequence)
+        {
+            // Rhino has some interesting internal classes such as ConsString which cannot be cast to String
+            // but fortunately are instanceof CharSequence so we can toString() them.
+            value = value.toString();
+        }
         return value;
     }