Specify behavior of h:outputText and h:inputText

[glassfishrobot]

The behavior of h:outputText and h:inputText for rendering children is currently
unspecified.

The RI behavior for outputText is odd - it renders the children first, then the
value of the tag.

After extensive discussion in the EG, there was a rough consensus that children
for outputText not be rendered. During the process of implementing the fix, I
discovered that inputText had a similar undesired behavior, and also changed
that to not be rendered.

I've marked this P1, since this is slated for inclusion in the very next MR of
JSF, and the RI has already changed behavior to not render children.

Environment

Operating System: All
Platform: All

Affected Versions

[2.0]

[glassfishrobot]

Reported by driscoll

[glassfishrobot]

@edburns said:
Move to unscheduled target milestone

[glassfishrobot]

@edburns said:
Prepare to delete api subcomponent

[glassfishrobot]

rogerk said:
cat1

[glassfishrobot]

@edburns said:
rk

[glassfishrobot]

@edburns said:
2.0 rev a

[glassfishrobot]

rogerk said:
The RI has added a configuration option to enable the rendering of children.
See: https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1154
And, by default, the rendering of children should not be done.
Currently, the renderkit docs for text rendering say:

"This component is responsible for rendering its children."

I suppose we could add a clarification like:

"Implementations may choose to disable the rendering of children by default."

[glassfishrobot]

rogerk said:
Ownership

[glassfishrobot]

@edburns said:
These are valid 2.0 Rev a issues

[glassfishrobot]

@edburns said:

• move to 2.1. When you do fix it, you can use this as the log message
  text.

• modify javax.faces. {Output,Input}

  javax.faces.Text renderer to not
  render children by default, but do allow the possibility of a config
  option to control rendering.

About to attach patch.

[glassfishrobot]

@edburns said:
Created an attachment (id=232)
Fix for this bug, first iteration

[glassfishrobot]

File: diffs.txt
Attached By: @edburns

[glassfishrobot]

rogerk said:
triage

[glassfishrobot]

@edburns said:
Change target milestone.

[glassfishrobot]

rogerk said:
triage

[glassfishrobot]

@edburns said:
Committed to trunk.

Sending jsf-api/doc/standard-html-renderkit-base.xml
Sending jsf-api/doc/standard-html-renderkit.xml
Sending jsf-api/src/main/java/javax/faces/context/ExternalContextWrapper.java
Sending jsf-api/src/main/java/javax/faces/webapp/FacesServlet.java
Sending jsf-api/src/main/resources/overview.html
Sending jsf-tools/src/main/resources/com/sun/faces/generate/facesdoc/stylesheet.css
Transmitting file data ......
Committed revision 9144.

[glassfishrobot]

Marked as fixed on Thursday, June 9th 2011, 1:53:46 am

[glassfishrobot]

@manfredriem said:
Closing resolved issue out

[glassfishrobot]

This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-585