You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2007/10/24 19:44:23 UTC
svn commit: r587938 - in
/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit:
RendererUtils.java html/HtmlGridRendererBase.java
html/HtmlGroupRendererBase.java html/HtmlLinkRendererBase.java
html/HtmlTableRendererBase.java
Author: mmarinschek
Date: Wed Oct 24 10:44:21 2007
New Revision: 587938
URL: http://svn.apache.org/viewvc?rev=587938&view=rev
Log:
https://issues.apache.org/jira/browse/MYFACES-1751 (MYFACES-1751): reverted the changes - wrong assumption
Modified:
myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java
myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGridRendererBase.java
myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGroupRendererBase.java
myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java
myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java
Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java?rev=587938&r1=587937&r2=587938&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/RendererUtils.java Wed Oct 24 10:44:21 2007
@@ -433,8 +433,11 @@
}
child.encodeBegin(facesContext);
- if (!child.getRendersChildren()) {
+ if (child.getRendersChildren()) {
child.encodeChildren(facesContext);
+ }
+ else {
+ renderChildren(facesContext, child);
}
child.encodeEnd(facesContext);
}
Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGridRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGridRendererBase.java?rev=587938&r1=587937&r2=587938&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGridRendererBase.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGridRendererBase.java Wed Oct 24 10:44:21 2007
@@ -50,6 +50,18 @@
return true;
}
+ public void encodeBegin(FacesContext facesContext, UIComponent component)
+ throws IOException
+ {
+ // all work done in encodeEnd()
+ }
+
+ public void encodeChildren(FacesContext context, UIComponent component)
+ throws IOException
+ {
+ // all work done in encodeEnd()
+ }
+
public void encodeEnd(FacesContext facesContext, UIComponent component)
throws IOException
{
Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGroupRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGroupRendererBase.java?rev=587938&r1=587937&r2=587938&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGroupRendererBase.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlGroupRendererBase.java Wed Oct 24 10:44:21 2007
@@ -41,6 +41,16 @@
return true;
}
+ public void encodeBegin(FacesContext context, UIComponent component)
+ throws IOException
+ {
+ }
+
+ public void encodeChildren(FacesContext context, UIComponent component)
+ throws IOException
+ {
+ }
+
public void encodeEnd(FacesContext context, UIComponent component)
throws IOException
{
Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java?rev=587938&r1=587937&r2=587938&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java Wed Oct 24 10:44:21 2007
@@ -140,8 +140,6 @@
else {
throw new IllegalArgumentException("Unsupported component class " + component.getClass().getName());
}
-
- encodeChildren(facesContext, component);
}
protected void renderCommandLinkStart(FacesContext facesContext, UIComponent component,
Modified: myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java?rev=587938&r1=587937&r2=587938&view=diff
==============================================================================
--- myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java (original)
+++ myfaces/shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlTableRendererBase.java Wed Oct 24 10:44:21 2007
@@ -521,8 +521,6 @@
{
RendererUtils.checkParamValidity(facesContext, uiComponent, UIData.class);
- encodeChildren(facesContext, uiComponent);
-
ResponseWriter writer = facesContext.getResponseWriter();
writer.endElement(HTML.TABLE_ELEM);
HtmlRendererUtils.writePrettyLineSeparator(facesContext);