Hi Ernesto,

I think as long as the third party library is not Wicket 7.0 proofed you should 
use it careful. Because of this the migration guide is very useful - each 
framework should be checked when upgrade a major version.

Even on OS level there is a requirement note and sometimes API changes have 
impact on applications and their complete compatibility.

kind regards

Tobias

> Am 09.05.2016 um 20:55 schrieb Ernesto Reinaldo Barreiro <[email protected]>:
> 
> Martin,
> 
> Actually I was vaguely aware of having read the discussion related to the
> issue...
> 
> https://mail-archives.apache.org/mod_mbox/wicket-dev/201411.mbox/%3ccamomwmrvoeghovjj2u7pck9js2on-d28fww9fxck6xgyvcc...@mail.gmail.com%3E
> 
> It was a bit more  difficult to figure out this was the culprit in my case.
> IMHO changes like these are "little changes" that might break applications
> and leave users just wondering  what's going on for a while... particularly
> if your are using components integrating with third party JS libraries.
> 
> 
> 
> On Mon, May 9, 2016 at 4:49 PM, Martin Grigorov <[email protected]>
> wrote:
> 
>> 
>> https://cwiki.apache.org/confluence/display/WICKET/Migration+to+Wicket+7.0#MigrationtoWicket7.0-org.apache.wicket.ajax.attributes.AjaxRequestAttributes#eventPropagationisnowBUBBLEbydefaultWICKET-5198
>> 
>> Having so detailed migration guide sometimes is a drawback - people don't
>> want to read it completely :-)
>> 
>> Martin Grigorov
>> Wicket Training and Consulting
>> https://twitter.com/mtgrigorov
>> 
>> On Mon, May 9, 2016 at 4:23 PM, Ernesto Reinaldo Barreiro <
>> [email protected]> wrote:
>> 
>>> Martin.
>>> 
>>> Thanks for explanation. This little thing was preventing me from jumping
>> to
>>> wicket 7. Now I can migrate my project :-)
>>> 
>>> On Mon, May 9, 2016 at 4:17 PM, Martin Grigorov <[email protected]>
>>> wrote:
>>> 
>>>> On Mon, May 9, 2016 at 4:14 PM, Ernesto Reinaldo Barreiro <
>>>> [email protected]> wrote:
>>>> 
>>>>> Problem was
>> attributes.setEventPropagation(AjaxRequestAttributes.EventPropagation.STOP
>>>>> );
>>>>> 
>>>>> was need. Before STOP was the default, now it is BUBBLE. Just
>> wondering
>>>>> why default was changed?
>>>> It changed because BUBBLE is the default in the browser DOM.
>>>> 
>>>> 
>>>>> 
>>>>> On Mon, May 9, 2016 at 3:26 PM, Ernesto Reinaldo Barreiro <
>>>>> [email protected]> wrote:
>>>>> 
>>>>>> This seem to be some bug on the bootstrap layer NOT a Wicket bug:
>>>>>> stripping bootstrap specifics makes wicket AJAX work.
>>>>>> 
>>>>>> On Mon, May 9, 2016 at 3:02 PM, Ernesto Reinaldo Barreiro <
>>>>>> [email protected]> wrote:
>>>>>> 
>>>>>>> @Martin,
>>>>>>> 
>>>>>>> cleaned up working branch.
>>>>>>> 
>>>>>>> https://github.com/l0rdn1kk0n/wicket-bootstrap/pull/604
>>>>>>> 
>>>>>>> On Mon, May 9, 2016 at 1:40 PM, Ernesto Reinaldo Barreiro <
>>>>>>> [email protected]> wrote:
>>>>>>> 
>>>>>>>> Hi,
>>>>>>>> 
>>>>>>>> Sorry for the long time it took... Today I finally managed to find
>>>> some
>>>>>>>> free time to create something you can use to see the problem in
>>>> action:
>>>>>>>> 
>>>>>>>> https://github.com/l0rdn1kk0n/wicket-bootstrap/pull/603
>>>>>>>> 
>>>>>>>> Shows a component (Bootstrap radio choice) that does not work in
>>>> wicket
>>>>>>>> 7.3.0
>>>>>>>> 
>>>>>>>> and PR
>>>>>>>> 
>>>>>>>> https://github.com/l0rdn1kk0n/wicket-bootstrap/pull/602
>>>>>>>> 
>>>>>>>> shows the same component working on 6.3.
>>>>>>>> 
>>>>>>>> [image: Inline image 1]
>>>>>>>> 
>>>>>>>> @Martin,
>>>>>>>> 
>>>>>>>> I will later on create a new (cleaner) PR to replace 603 to add
>> only
>>>>>>>> radio choice renderer for 6.x.
>>>>>>>> 
>>>>>>>> Disclaimer: I do not know if this is an error introduced by the
>>>>>>>> bootstrap layer or if this can be reproduced with  a bare bones
>>> wicket
>>>>>>>> example.
>>>>>>>> 
>>>>>>>> 
>>>>>>>> On Wed, Nov 18, 2015 at 2:00 PM, Ernesto Reinaldo Barreiro <
>>>>>>>> [email protected]> wrote:
>>>>>>>> 
>>>>>>>>> Let me see is I find the time to created it (it uses my own
>>> bootstrap
>>>>>>>>> components)
>>>>>>>>> 
>>>>>>>>> On Wed, Nov 18, 2015 at 1:53 PM, Sven Meier <[email protected]>
>>> wrote:
>>>>>>>>> 
>>>>>>>>>> Hi,
>>>>>>>>>> 
>>>>>>>>>> with a quickstart it would be easy to spot the problem.
>>>>>>>>>> 
>>>>>>>>>> Regards
>>>>>>>>>> Sven
>>>>>>>>>> 
>>>>>>>>>> 
>>>>>>>>>>> On 18.11.2015 13:46, Ernesto Reinaldo Barreiro wrote:
>>>>>>>>>>> 
>>>>>>>>>>> Hi,
>>>>>>>>>>> 
>>>>>>>>>>> 
>>>>>>>>>>> If I "revert" to 6.21.0 problem disappears so this seems to be
>> a
>>>>>>>>>>> 7.1.0
>>>>>>>>>>> "issue".
>>>>>>>>>>> 
>>>>>>>>>>> On Tue, Nov 17, 2015 at 9:36 PM, Ernesto Reinaldo Barreiro <
>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>> 
>>>>>>>>>>> Well it is the same in the sense that no client side AJAX event
>>>>>>>>>>>> seems to
>>>>>>>>>>>> be triggered.
>>>>>>>>>>>> 
>>>>>>>>>>>> On Tue, Nov 17, 2015 at 9:24 PM, Ernesto Reinaldo Barreiro <
>>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>> 
>>>>>>>>>>>> Hi Sven,
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Thanks for the feedback. It should not be the case
>>>>>>>>>>>>> 
>>>>>>>>>>>>>  radioGroup = new RadioGroup<T>("radioGroup", getModel());
>>>>>>>>>>>>>         tuneRadioGroup(radioGroup);
>>>>>>>>>>>>>         radioGroup.setRenderBodyOnly(false);
>>>>>>>>>>>>>         radioGroup.setOutputMarkupId(true);
>>>>>>>>>>>>>         add(radioGroup);
>>>>>>>>>>>>>         if(changeHandler != null) {
>>>>>>>>>>>>>             radioGroup.add(new
>>>>>>>>>>>>> AjaxFormChoiceComponentUpdatingBehavior() {
>>>>>>>>>>>>> 
>>>>>>>>>>>>>                 @Override
>>>>>>>>>>>>>                 protected void onUpdate(AjaxRequestTarget
>>>> target)
>>>>>>>>>>>>> {
>>>>>>>>>>>>>                     changeHandler.onSelectionChanged(target,
>>>>>>>>>>>>> radioGroup.getModel().getObject());
>>>>>>>>>>>>>                 }
>>>>>>>>>>>>>             });
>>>>>>>>>>>>>         }
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> 
>>>>>>>>>>>>> On Tue, Nov 17, 2015 at 7:11 PM, Sven Meier <[email protected]
>>> 
>>>>>>>>>>>>> wrote:
>>>>>>>>>>>>> 
>>>>>>>>>>>>> Hi,
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> nothing specific I can think of ... perhaps WICKET-5476?
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> Regards
>>>>>>>>>>>>>> Sven
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> On 17.11.2015 18:55, Ernesto Reinaldo Barreiro wrote:
>>>>>>>>>>>>>> 
>>>>>>>>>>>>>> Hi,
>>>>>>>>>>>>>>> 
>>>>>>>>>>>>>>> I have a bootstrap based component that uses a RadioGroup
>>>>>>>>>>>>>>> + AjaxFormChoiceComponentUpdatingBehavior. This was working
>>>>>>>>>>>>>>> perfectly
>>>>>>>>>>>>>>> fine
>>>>>>>>>>>>>>> on wicket 6. Now is no loger working with wicket 7. Are
>> there
>>>> any
>>>>>>>>>>>>>>> (obvious)
>>>>>>>>>>>>>>> changes on the way RadioGroup works that might be
>> preventing
>>> it
>>>>>>>>>>>>>>> from
>>>>>>>>>>>>>>> working?
>>>> ---------------------------------------------------------------------
>>>>>>>>>>>>>> To unsubscribe, e-mail: [email protected]
>>>>>>>>>>>>>> For additional commands, e-mail:
>> [email protected]
>>>>>>>>>>>>> 
>>>>>>>>>>>>> --
>>>>>>>>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>>>>>>>>>> 
>>>>>>>>>>>> 
>>>>>>>>>>>> --
>>>>>>>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>> ---------------------------------------------------------------------
>>>>>>>>>> To unsubscribe, e-mail: [email protected]
>>>>>>>>>> For additional commands, e-mail: [email protected]
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> --
>>>>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>> --
>>>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>>>>> 
>>>>>>> 
>>>>>>> 
>>>>>>> --
>>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> --
>>>>>> Regards - Ernesto Reinaldo Barreiro
>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> Regards - Ernesto Reinaldo Barreiro
>>> 
>>> 
>>> 
>>> --
>>> Regards - Ernesto Reinaldo Barreiro
> 
> 
> 
> -- 
> Regards - Ernesto Reinaldo Barreiro

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to