From cf974569aea19712d05210e1d4d2d16d55084e2d Mon Sep 17 00:00:00 2001
From: Gavin Cornwell <gavin.cornwell@alfresco.com>
Date: Thu, 9 Feb 2006 13:39:25 +0000
Subject: [PATCH] Fix for AWC-394

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@2333 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../web/ui/common/component/UIOutputText.java | 52 ++++++++++++++++++-
 .../web/ui/common/tag/OutputTextTag.java      | 13 +++++
 source/web/WEB-INF/alfresco.tld               |  6 +++
 source/web/jsp/wizard/new-rule/condition.jsp  | 14 ++---
 4 files changed, 77 insertions(+), 8 deletions(-)

diff --git a/source/java/org/alfresco/web/ui/common/component/UIOutputText.java b/source/java/org/alfresco/web/ui/common/component/UIOutputText.java
index 7cf2116bea..eb5eb999f1 100644
--- a/source/java/org/alfresco/web/ui/common/component/UIOutputText.java
+++ b/source/java/org/alfresco/web/ui/common/component/UIOutputText.java
@@ -18,10 +18,12 @@
 package org.alfresco.web.ui.common.component;
 
 import java.io.IOException;
+import java.net.URLEncoder;
 
 import javax.faces.component.UIOutput;
 import javax.faces.context.FacesContext;
 import javax.faces.context.ResponseWriter;
+import javax.faces.el.ValueBinding;
 
 /**
  * Component that simply renders text
@@ -30,6 +32,8 @@ import javax.faces.context.ResponseWriter;
  */
 public class UIOutputText extends UIOutput
 {
+   private Boolean encodeForJavaScript = null;
+   
    /**
     * Default constructor
     */
@@ -45,6 +49,40 @@ public class UIOutputText extends UIOutput
    {
       return "org.alfresco.faces.OutputText";
    }
+   
+   /**
+    * Sets whether the text should be encoded for JavaScript consumption
+    * 
+    * @param encodeForJavaScript true to escape text
+    */
+   public void setEncodeForJavaScript(boolean encodeForJavaScript)
+   {
+      this.encodeForJavaScript = Boolean.valueOf(encodeForJavaScript);
+   }
+
+   /**
+    * Returns whether the text is going to be encoded or not
+    * 
+    * @return true if the text is going to be encoded
+    */
+   public boolean isEncodeForJavaScript()
+   {
+      if (this.encodeForJavaScript == null)
+      {
+         ValueBinding vb = getValueBinding("encodeForJavaScript");
+         if (vb != null)
+         {
+            this.encodeForJavaScript = (Boolean)vb.getValue(getFacesContext());
+         }
+         
+         if (this.encodeForJavaScript == null)
+         {
+            this.encodeForJavaScript = Boolean.FALSE;
+         }
+      }
+      
+      return this.encodeForJavaScript.booleanValue();
+   }
 
    /**
     * @see javax.faces.component.UIComponentBase#encodeBegin(javax.faces.context.FacesContext)
@@ -57,6 +95,18 @@ public class UIOutputText extends UIOutput
       }
       
       ResponseWriter out = context.getResponseWriter();
-      out.write((String)getValue());
+      
+      String output = null;
+      
+      if (isEncodeForJavaScript())
+      {
+         output = URLEncoder.encode((String)getValue(), "UTF-8").replace('+', ' ');
+      }
+      else
+      {
+         output = (String)getValue();
+      }
+
+      out.write(output);
    }
 }
diff --git a/source/java/org/alfresco/web/ui/common/tag/OutputTextTag.java b/source/java/org/alfresco/web/ui/common/tag/OutputTextTag.java
index ad217b6342..147e5698ba 100644
--- a/source/java/org/alfresco/web/ui/common/tag/OutputTextTag.java
+++ b/source/java/org/alfresco/web/ui/common/tag/OutputTextTag.java
@@ -26,6 +26,7 @@ import javax.faces.component.UIComponent;
 public class OutputTextTag extends HtmlComponentTag
 {
    private String value;
+   private String encodeForJavaScript;
    
    /**
     * @see javax.faces.webapp.UIComponentTag#getComponentType()
@@ -51,6 +52,7 @@ public class OutputTextTag extends HtmlComponentTag
    {
       super.setProperties(component);
       setStringProperty(component, "value", this.value);
+      setBooleanProperty(component, "encodeForJavaScript", this.encodeForJavaScript);
    }
    
    /**
@@ -60,6 +62,7 @@ public class OutputTextTag extends HtmlComponentTag
    {
       super.release();
       this.value = null;
+      this.encodeForJavaScript = null;
    }
    
    /**
@@ -71,4 +74,14 @@ public class OutputTextTag extends HtmlComponentTag
    {
       this.value = value;
    }
+   
+   /**
+    * Set the encodeForJavaScript flag
+    * 
+    * @param encodeForJavaScript true to encode the text for use in JavaScript
+    */
+   public void setEncodeForJavaScript(String encodeForJavaScript)
+   {
+      this.encodeForJavaScript = encodeForJavaScript;
+   }
 }
diff --git a/source/web/WEB-INF/alfresco.tld b/source/web/WEB-INF/alfresco.tld
index 36166f665d..f32dac1fed 100644
--- a/source/web/WEB-INF/alfresco.tld
+++ b/source/web/WEB-INF/alfresco.tld
@@ -1627,6 +1627,12 @@
          <required>true</required>
          <rtexprvalue>true</rtexprvalue>
       </attribute>
+      
+      <attribute>
+         <name>encodeForJavaScript</name>
+         <required>false</required>
+         <rtexprvalue>true</rtexprvalue>
+      </attribute>
    </tag>
    
    <tag>
diff --git a/source/web/jsp/wizard/new-rule/condition.jsp b/source/web/jsp/wizard/new-rule/condition.jsp
index 5685d29984..bf17c14538 100644
--- a/source/web/jsp/wizard/new-rule/condition.jsp
+++ b/source/web/jsp/wizard/new-rule/condition.jsp
@@ -245,7 +245,7 @@
        </tr>
     </table>
     
-    <script language="JavaScript1.2">
+    <script language="JavaScript1.5">
       function itemSelected(inputField)
       {
          if (inputField.selectedIndex == 0)
@@ -259,19 +259,19 @@
          
          // also check to see if the 'no-condition' option has been selected, if it has, change
          // the explanation text and the button label
-         var short_text = "<a:outputText value='#{msg.click_add_to_list}'/>";
-         var long_text = "<a:outputText value='#{msg.click_set_and_add}'/>";
-         var short_label = "<a:outputText value='#{msg.add_to_list_button}'/>";
-         var long_label = "<a:outputText value='#{msg.set_and_add_button}'/>";
+         var short_text = "<a:outputText value='#{msg.click_add_to_list}' />";
+         var long_text = "<a:outputText value='#{msg.click_set_and_add}' />";
+         var short_label = "<a:outputText value='#{msg.add_to_list_button}' encodeForJavaScript='true' />";
+         var long_label = "<a:outputText value='#{msg.set_and_add_button}' encodeForJavaScript='true' />";
          
          if (inputField.value == "no-condition")
          {
-            document.getElementById("new-rule-condition:set-add-button").value = short_label;
+            document.getElementById("new-rule-condition:set-add-button").value = decodeURI(short_label);
             document.getElementById("new-rule-condition:instruction-text").innerHTML = short_text;
          }
          else
          {
-            document.getElementById("new-rule-condition:set-add-button").value = long_label;
+            document.getElementById("new-rule-condition:set-add-button").value = decodeURI(long_label);
             document.getElementById("new-rule-condition:instruction-text").innerHTML = long_text;
          }
       }