[MYFACES-2493] ViewMetadata facelets compiler should not output DTD, <?xml or any UIInstruction outside f:metadata

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0-alpha
• Fix Version/s: 2.0.0-beta
• Component/s: JSR-314
• Assignee: Leonardo Uribe
• Reporter: Leonardo Uribe
• Created: 19/Jan/10 00:51
• Updated: 07/May/10 01:02
• Resolved: 19/Jan/10 00:52

Description

This does not have any side effects but originally, FaceletViewHandler used this line on renderView method:

// build view - but not if we're in "buildBeforeRestore"
// land and we've already got a populated view. Note
// that this optimizations breaks if there's a "c:if" in
// the page that toggles as a result of request processing -
// should that be handled? Or
// is this optimization simply so minor that it should just
// be trimmed altogether?
if (!this.buildBeforeRestore || viewToRender.getChildren().isEmpty())

{ this.buildView(context, viewToRender); }

Since we need to solve MYFACES-2483 Find a way to allow c:if work with partial state saving enabled, we need a way to detect when a view is "updated", and check if UIViewRoot has children is ideal.