Author: gvanmatre
Date: Sat Feb  4 06:56:54 2006
New Revision: 374876

URL: http://svn.apache.org/viewcvs?rev=374876&view=rev
Log:
Fixes for Bug 38493 and 38492 reported by Hermod Opstvedt.

Modified:
    
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/Clay.java
    
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/chain/PropertyActionCommand.java

Modified: 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/Clay.java
URL: 
http://svn.apache.org/viewcvs/struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/Clay.java?rev=374876&r1=374875&r2=374876&view=diff
==============================================================================
--- 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/Clay.java
 (original)
+++ 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/Clay.java
 Sat Feb  4 06:56:54 2006
@@ -373,15 +373,17 @@
             Iterator ci = child.getChildren().iterator();
             while (ci.hasNext()) {
                 UIComponent c = (UIComponent) ci.next();
-                c.encodeBegin(context);
-                
-                if (!c.getRendersChildren())
-                    recursiveRenderChildren(c, context);
-                else
-                    c.encodeChildren(context);
-                
-                c.encodeEnd(context);
-                c = null;
+                               if (c.isRendered()) {
+                                       c.encodeBegin(context);
+                                       
+                                       if (!c.getRendersChildren())
+                                               recursiveRenderChildren(c, 
context);
+                                       else
+                                               c.encodeChildren(context);
+                                       
+                                       c.encodeEnd(context);
+                                       c = null;
+                               }
             }
         } else {
             // let the component handle iterating over the children

Modified: 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/chain/PropertyActionCommand.java
URL: 
http://svn.apache.org/viewcvs/struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/chain/PropertyActionCommand.java?rev=374876&r1=374875&r2=374876&view=diff
==============================================================================
--- 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/chain/PropertyActionCommand.java
 (original)
+++ 
struts/shale/trunk/clay-plugin/src/java/org/apache/shale/clay/component/chain/PropertyActionCommand.java
 Sat Feb  4 06:56:54 2006
@@ -97,12 +97,7 @@
             
             if (child instanceof ActionSource) {
                 String expr = null;
-                
-                if (UIComponentTag.isValueReference(attributeBean.getValue()))
-                    expr = replaceMnemonic(clayContext);
-                else
-                    expr = attributeBean.getValue();
-
+                expr = replaceMnemonic(clayContext);
                 tagUtils.setAction(child, expr);
             } else {
                 log.error(messages.getMessage("property.action.error", new 
Object[] {attributeBean}));



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to