[jira] Commented: (MYFACES-152) ResponseWriter.endDocument() abuse breaks ADF Faces

["Frank Fischer (JIRA)" <de...@myfaces.apache.org>]

    [ http://issues.apache.org/jira/browse/MYFACES-152?page=comments#action_12364610 ] 

Frank Fischer commented on MYFACES-152:
---------------------------------------

Bruno, as of revision 37812 I still have the issue that the clear_linkDummyForm() javascript is not rendered. Maybe the problem is in the using of facelets (1.0.7, 1.0.10 has major issues with myfaces 1.1.2 snapshots between 25.01.2006 and 31.01.2006: duplicate id), but honestly, I doubt it.

Best regards,

Frank

> ResponseWriter.endDocument() abuse breaks ADF Faces
> ---------------------------------------------------
>
>          Key: MYFACES-152
>          URL: http://issues.apache.org/jira/browse/MYFACES-152
>      Project: MyFaces
>         Type: Bug
>   Components: General
>     Versions: 1.0.9m9
>     Reporter: Adam Winer
>     Priority: Critical
>      Fix For: 1.1.2, Nightly

>
> I've been shown some problems lately with MyFaces 1.0.9 and ADF Faces.
> The problems specifically trace to MyFaces's use of
> ResponseWriter.endDocument() to output Javascript.  Since ADF Faces
> runs with its own RenderKit (and therefore its own ResponseWriter),
> this Javascript is getting dropped and not written.
> I'd recommend (both as JSF EG guy and ADF Faces guy) that this MyFaces
> code be moved *out* of ResponseWriter.endDocument().  Specifically:
> - ResponseWriter.endDocument() is not guaranteed to be called before
> the close of  or even the close of , and therefore this
> script cannot be safely output at this point.  It's quite likely that
> changes in JSF 1.2 will essentially guarantee that endDocument() is
> not called until the close of all output.
> - This is not really the intent of ResponseWriter.endDocument().  In
> HTML, it should be a no-op.  It's there for more bizarre scenarios
> like a ResponseWriter outputting a SOAP envelope around a response.
> - It's breaking ADF Faces. :)
> A significantly cleaner way to output needed Javascript is to add it
> as needed from the Renderers that require it (using a request-scoped
> attribute to track if its been added already).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira