Hi<br><br>I created an issue here:<br><br><a href="https://javaserverfaces-spec-public.dev.java.net/issues/show_bug.cgi?id=798">https://javaserverfaces-spec-public.dev.java.net/issues/show_bug.cgi?id=798</a><br><br>regards,<br>
<br>Leonardo Uribe<br><br><div class="gmail_quote">2010/5/5 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;">
Hi Leonardo,<br>
<br>
Ah well, I totally agree. Looks like the (wrong) remainder of a<br>
refactoring to me.<br>
<div><div></div><div class="h5"><br>
best regards,<br>
<br>
Martin<br>
<br>
On 5/5/10, Leonardo Uribe <<a href="mailto:lu4242@gmail.com">lu4242@gmail.com</a>> wrote:<br>
> Hi Martin<br>
><br>
> 2010/5/4 Martin Marinschek <<a href="mailto:mmarinschek@apache.org">mmarinschek@apache.org</a>><br>
><br>
>> Hi Leonardo,<br>
>><br>
>> just so that I get you right: you are saying that broadcast should be<br>
>> called in any case, not only if the behaviour is a ClientBehavior?<br>
>><br>
>><br>
> Yes, the Behavior interface has only one method: broadcast(). It does not<br>
> have sense to put a method on a base class that only will be called for an<br>
> specific child class (in this case ClientBehavior). I think the intention of<br>
> the programmer here is call it always.<br>
><br>
> best regards,<br>
><br>
> Leonardo Uribe<br>
><br>
><br>
>> (sorry if this question doesn't make a lot of sense; I am not too deep<br>
>> in the behavior stuff right now ;)<br>
>><br>
>> best regards,<br>
>><br>
>> Martin<br>
>><br>
>> On 5/5/10, Leonardo Uribe <<a href="mailto:lu4242@gmail.com">lu4242@gmail.com</a>> wrote:<br>
>> > Hi<br>
>> ><br>
>> > Checking the new Behavior api (for implement cc:clientBehavior), I note<br>
>> that<br>
>> > the javadoc of UIComponent.broadcast says this:<br>
>> ><br>
>> > "....Broadcast the specified FacesEvent to all registered event<br>
>> > listeners<br>
>> > who have expressed an interest in events of this type. Listeners are<br>
>> called<br>
>> > in the order in which they were added.<br>
>> ><br>
>> > If the event is an instance of BehaviorEvent and the current component<br>
>> > is<br>
>> > the source of the event call BehaviorEvent.getBehavior() to get the<br>
>> Behavior<br>
>> > for the event. If the behavior implements ClientBehavior, call<br>
>> > Behavior.broadcast(javax.faces.event.BehaviorEvent)}....."<br>
>> ><br>
>> > The wrong line is:<br>
>> ><br>
>> > ".....If the behavior implements ClientBehavior, call<br>
>> > Behavior.broadcast....."<br>
>> ><br>
>> > So, if a user try to create a custom Behavior, the method broadcast()<br>
>> will<br>
>> > be useless, and custom behaviors will not catch events. I think it is a<br>
>> bug<br>
>> > on the javadoc, so I'll correct it on myfaces.<br>
>> ><br>
>> > regards,<br>
>> ><br>
>> > Leonardo Uribe<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>
>><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>
</div></div></blockquote></div><br>