Author: awiner
Date: Thu Jan 18 09:48:23 2007
New Revision: 497512

URL: http://svn.apache.org/viewvc?view=rev&rev=497512
Log:
Last overlooked bit of ADFFACES-42:  don't block decode() calls to Renderer in 
base component classes

Modified:
    
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXGoTemplate.java
    
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXOutputTemplate.java
    
incubator/adffaces/trunk/trinidad/trinidad-api/src/test/java/org/apache/myfaces/trinidad/component/core/output/CoreOutputTextTest.java

Modified: 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXGoTemplate.java
URL: 
http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXGoTemplate.java?view=diff&rev=497512&r1=497511&r2=497512
==============================================================================
--- 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXGoTemplate.java
 (original)
+++ 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXGoTemplate.java
 Thu Jan 18 09:48:23 2007
@@ -28,9 +28,12 @@
 abstract public class UIXGoTemplate extends UIXComponentBase
 {
        
-       @Override
+  /* See ADFFACES-42: switching decode on and off should
+   * not be done up here.
+  @Override
   void __rendererDecode(FacesContext context)
   {
     // do nothing. we have no behaviour
   }
+  */
 }

Modified: 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXOutputTemplate.java
URL: 
http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXOutputTemplate.java?view=diff&rev=497512&r1=497511&r2=497512
==============================================================================
--- 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXOutputTemplate.java
 (original)
+++ 
incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXOutputTemplate.java
 Thu Jan 18 09:48:23 2007
@@ -28,9 +28,12 @@
 abstract public class UIXOutputTemplate extends UIXValue
 {
        
+  /* See ADFFACES-42: switching decode on and off should
+   * not be done up here.
   @Override
   void __rendererDecode(FacesContext context)
   {
     // do nothing. we have no behaviour
   }
+  */
 }

Modified: 
incubator/adffaces/trunk/trinidad/trinidad-api/src/test/java/org/apache/myfaces/trinidad/component/core/output/CoreOutputTextTest.java
URL: 
http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-api/src/test/java/org/apache/myfaces/trinidad/component/core/output/CoreOutputTextTest.java?view=diff&rev=497512&r1=497511&r2=497512
==============================================================================
--- 
incubator/adffaces/trunk/trinidad/trinidad-api/src/test/java/org/apache/myfaces/trinidad/component/core/output/CoreOutputTextTest.java
 (original)
+++ 
incubator/adffaces/trunk/trinidad/trinidad-api/src/test/java/org/apache/myfaces/trinidad/component/core/output/CoreOutputTextTest.java
 Thu Jan 18 09:48:23 2007
@@ -109,9 +109,10 @@
   {
     CoreOutputText component = new CoreOutputText();
     // we don't use renderers for decode for this component:
-    setRendererUsed(false);
+    // Or, we didn't.  But see ADFFACES-42
+    //    setRendererUsed(false);
     doTestApplyRequestValues(component);
-    setRendererUsed(true);
+    // setRendererUsed(true);
 
     component = new CoreOutputText();
     component.setRendered(false);


Reply via email to