<br><br><div class="gmail_quote">2010/3/29 Martin Marinschek <span dir="ltr"><<a href="mailto:mmarinschek@apache.org">mmarinschek@apache.org</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Used the wrong mail address.<br>
<br>
See original mail below. For my second point, this is of course only<br>
possible if there are no custom tag-handlers which do something<br>
dynamically. How could we handle such tag-handlers? I am not sure.<br>
Maybe a certain interface. Would need to be in the API. Or we do it<br>
implementation specific.<br>
<br></blockquote><div><br>Use a facesContext attribute as a flag for this one is enought, but if
this attribute<br>
is part of the spec, we could have a common way to handle those cases.<br>(see my response below)<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
best regards,<br>
<br>
Martin<br>
<br>
On Tue, Mar 30, 2010 at 4:18 AM, Martin Marinschek<br>
<<a href="mailto:martin.marinschek@gmail.com">martin.marinschek@gmail.com</a>> wrote:<br>
> Hi Leonardo,<br>
><br>
> yes. Two more things:<br>
><br>
> - you can keep the context-param, just change the default value<br></blockquote><div><br>Ok, I was thinking on put the original param facelets.BUILD_BEFORE_RESTORE, <br>but your idea sounds better.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> - it would be nice if you could skip this if there is no c:if,<br>
> c:forEach or ui:include on the page - that would help for performance<br>
><br></blockquote><div><br>Myfaces already has this behavior implemented. We use a param called <br>org.apache.myfaces.CLEAN_TRANSIENT_BUILD_ON_RESTORE, that<br>is set from c:forEach, ui:include, c:if or c:choose tag handlers to <br>
indicate that the current view should be refreshed. This behavior is <br>activated throught set org.apache.myfaces.REFRESH_TRANSIENT_BUILD_ON_PSS<br>param to "auto". Maybe the "auto" mode should be the default.<br>
<br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> best regards,<br>
><br>
> Martin<br>
<div><div></div><div class="h5">><br>
> On Tue, Mar 30, 2010 at 3:53 AM, Leonardo Uribe <<a href="mailto:lu4242@gmail.com">lu4242@gmail.com</a>> wrote:<br>
>><br>
>><br>
>> 2010/3/29 Andy Schwartz <<a href="mailto:andy.schwartz@oracle.com">andy.schwartz@oracle.com</a>><br>
>>><br>
>>> On 3/29/10 5:10 PM, Martin Marinschek wrote:<br>
>>>><br>
>>>> Hi Leonardo,<br>
>>>><br>
>>>><br>
>>>>><br>
>>>>> Ok, if I have a composite component the first thought is use c:if tag,<br>
>>>>> but<br>
>>>>> remember that with<br>
>>>>> partial state saving enabled this tag is evaluated when the view is<br>
>>>>> build,<br>
>>>>> not when it is<br>
>>>>> rendered. Other alternative is use a component that allows to render one<br>
>>>>><br>
>>>><br>
>>>> just to get this straight, cause I seemed to read this (IMHO, wrong<br>
>>>> assertion) several times from you already: c:if, c:forEach and<br>
>>>> ui:include _should_ be evaluated before rendering again. This is in<br>
>>>> Mojarra as of 2.0.3 (Ed, Ryan, correct me if I am wrong) and it should<br>
>>>> be in MyFaces as well, right?<br>
>>>><br>
>>><br>
>>> Haven't had a chance to review this entire thread (or any other recent<br>
>>> threads - sorry!), but just wanted to jump in to say that, yes, Ryan has<br>
>>> implemented an initial fix for this as part of this issue:<br>
>>><br>
>>> <a href="https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1313" target="_blank">https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1313</a><br>
>>><br>
>>> Would be good to do some additional testing with our various test cases<br>
>>> just to make sure that all is well.<br>
>>><br>
>><br>
>> Ok, that's great!. That means we need to make myfaces evaluate c:if,<br>
>> c:forEach and ui:include as default, not as an option activated by a param,<br>
>> like in myfaces right now. Thanks for the tip.<br>
>><br>
>> regards,<br>
>><br>
>> Leonardo Uribe<br>
>><br>
>>><br>
>>> Andy<br>
>>><br>
>>>> Anyway I support your basic motion: it would generally be good to be<br>
>>>> able to exchange the renderer as well.<br>
>>>><br>
>>>> best regards,<br>
>>>><br>
>>>> Martin<br>
>>>><br>
>>>><br>
>>><br>
>><br>
>><br>
><br>
><br>
><br>
</div></div>> --<br>
<div><div></div><div class="h5">><br>
> <a href="http://www.irian.at" target="_blank">http://www.irian.at</a><br>
><br>
> Your JSF powerhouse -<br>
> JSF Consulting, Development and<br>
> Courses in English and German<br>
><br>
> Professional Support for Apache MyFaces<br>
><br>
<br>
<br>
<br>
--<br>
<br>
<a href="http://www.irian.at" target="_blank">http://www.irian.at</a><br>
<br>
Your JSF powerhouse -<br>
JSF Consulting, Development and<br>
Courses in English and German<br>
<br>
Professional Support for Apache MyFaces<br>
</div></div></blockquote></div><br>