Hi Thorsten,

please take a look at https://github.com/wicketstuff/core/issues/680

Sven

Am 2. Oktober 2019 08:45:58 MESZ schrieb "Thorsten Schöning" 
<tschoen...@am-soft.de>:
>Guten Tag Sven Meier,
>am Dienstag, 1. Oktober 2019 um 20:53 schrieben Sie:
>
>> Actually you could argue whether it is practical that these events
>> are not delivered to invisible or disabled components by default -
>> something I wouldn't expect, since it requires to override
>> #canCallListener() or a clever solution like yours.
>
>+1, I was suprised as well in the past. AnnotationEventDispatcher is
>some kind of opt-in in the end already, so one can argue it should not
>care if to publish events or not at all. If receivers of events want
>to ignore some based on aspects like visibility of a component, if
>it's enabled or not etc., they could check that in the event handler 
>on their own. Would have the benefit of being able to check that for
>each and every individual event (handler) as well.
>
>> IMHO we should make this configurable[...]
>
>Such a configuration has been explicitly removed in the past in favour
>of calling "canCallListener":
>
>https://github.com/wicketstuff/core/commit/f2ee61fb48259f9bdbf682713b12738e596d3cc1#diff-e8dc60d585cc63b237e8bb83a2814c3b
>https://github.com/wicketstuff/core/commit/f2ee61fb48259f9bdbf682713b12738e596d3cc1#diff-e1aa43fd3aadd18631700c7dd4c63179
>
>So it might be worth it to consider re-introducing that configuration
>again, but with a changed default of "true" to send to invisible
>components.
>
>Mit freundlichen Grüßen,
>
>Thorsten Schöning
>
>-- 
>Thorsten Schöning       E-Mail: thorsten.schoen...@am-soft.de
>AM-SoFT IT-Systeme      http://www.AM-SoFT.de/
>
>Telefon...........05151-  9468- 55
>Fax...............05151-  9468- 88
>Mobil..............0178-8 9468- 04
>
>AM-SoFT GmbH IT-Systeme, Brandenburger Str. 7c, 31789 Hameln
>AG Hannover HRB 207 694 - Geschäftsführer: Andreas Muchow

Reply via email to