Re: Is there any way to extend ClientProperties

2016-11-25 Thread Maxim Solodovnik
False alarm with quick start :(
deleting ~/.m2/repository/org/apache/logging/log4j/log4j-core fixed jetty
Sorry for the noise

On Fri, Nov 25, 2016 at 10:31 PM, Maxim Solodovnik 
wrote:

> Sorry for OFF topic
>
> I'm struggling with git :(
> I do have clone of https://github.com/apache/wicket at
> https://github.com/solomax/wicket, but I don't have necessary branch :(((
> (WICKET-6056-client-properties)
>
> I did
> git remote add apache g...@github.com:apache/wicket.git
> git fetch apache
> git merge apache/master
> git push
>
> no branch :(
> I can attach patch to WICKET-6056, But I believe git will be more
> comfortable ...
> could you please help me to get required branch?
>
> Another issue 8.0.0-SNAPSHOP quickstart from here
> https://wicket.apache.org/start/quickstart.html (all default params,
> jetty) is not working :(
> Would appreciate any help on fixing this
>
> On Fri, Nov 25, 2016 at 3:57 AM, Martin Grigorov 
> wrote:
>
>> Hi Maxim,
>>
>> See Sven's changes in WICKET-6056 and my suggestion for the extra info
>> gathering.
>>
>> Martin Grigorov
>> Wicket Training and Consulting
>> https://twitter.com/mtgrigorov
>>
>> On Thu, Nov 24, 2016 at 1:02 PM, Maxim Solodovnik 
>> wrote:
>>
>> > Yep,
>> > Unfortunately It's not that easy to
>> > fix /wicket-core/src/main/java/org/apache/wicket/markup/html/
>> > pages/BrowserInfoPage.java/html
>> > I believe both need to be consistent
>> > Or maybe we can leave only JS version and remove static one?
>> >
>> > On Thu, Nov 24, 2016 at 6:56 PM, Martin Grigorov 
>> > wrote:
>> >
>> > > Hi Maxim,
>> > >
>> > > See org.apache.wicket.ajax.AjaxClientInfoBehavior works.
>> > > Looking at it I think you can easily extend it by using
>> monkey-patching
>> > > for Wicket.BrowserInfo.collect()
>> > >
>> > > Martin Grigorov
>> > > Wicket Training and Consulting
>> > > https://twitter.com/mtgrigorov
>> > >
>> > > On Thu, Nov 24, 2016 at 12:40 PM, Maxim Solodovnik <
>> solomax...@gmail.com
>> > >
>> > > wrote:
>> > >
>> > > > Hello Martin,
>> > > >
>> > > > One question before I'll start with this enhancement: Can pure JS
>> > > solution
>> > > > be used here? I can both create FORM and send all data via POST
>> without
>> > > any
>> > > > markup ...
>> > > > Would it be OK?
>> > > >
>> > > > On Wed, Nov 23, 2016 at 3:55 PM, Maxim Solodovnik <
>> > solomax...@gmail.com>
>> > > > wrote:
>> > > >
>> > > > > OK, will do, will take some time :)
>> > > > >
>> > > > > On Wed, Nov 23, 2016 at 3:54 PM, Martin Grigorov <
>> > mgrigo...@apache.org
>> > > >
>> > > > > wrote:
>> > > > >
>> > > > >> Try the PR and see whether it helps you to achieve what you need.
>> > > > >> If it does then we will process it sooner.
>> > > > >> If it doesn't then make it work :-)
>> > > > >>
>> > > > >> Martin Grigorov
>> > > > >> Wicket Training and Consulting
>> > > > >> https://twitter.com/mtgrigorov
>> > > > >>
>> > > > >> On Wed, Nov 23, 2016 at 9:44 AM, Maxim Solodovnik <
>> > > solomax...@gmail.com
>> > > > >
>> > > > >> wrote:
>> > > > >>
>> > > > >> > OK, how can I help?
>> > > > >> >
>> > > > >> > On Wed, Nov 23, 2016 at 3:40 PM, Martin Grigorov <
>> > > > mgrigo...@apache.org>
>> > > > >> > wrote:
>> > > > >> >
>> > > > >> > > See https://issues.apache.org/jira/browse/WICKET-6056
>> > > > >> > > It is almost possible to extend the BrowserInfoPage. It needs
>> > just
>> > > > >> little
>> > > > >> > > more work.
>> > > > >> > >
>> > > > >> > > Martin Grigorov
>> > > > >> > > Wicket Training and Consulting
>> > > > >> > > https://twitter.com/mtgrigorov
>> > > > >> > >
>> > > > >> > > On Tue, Nov 22, 2016 at 7:02 PM, Maxim Solodovnik <
>> > > > >> solomax...@gmail.com>
>> > > > >> > > wrote:
>> > > > >> > >
>> > > > >> > > > Does it make any sense?
>> > > > >> > > >
>> > > > >> > > > On Tue, Nov 22, 2016 at 10:25 AM, Maxim Solodovnik <
>> > > > >> > solomax...@gmail.com
>> > > > >> > > >
>> > > > >> > > > wrote:
>> > > > >> > > >
>> > > > >> > > > > And maybe protocol+context (or window.location.href) :)
>> > > > >> > > > > mod_proxy, mod_rewrite can change the URL a lot :)
>> > > > >> > > > >
>> > > > >> > > > > On Tue, Nov 22, 2016 at 10:18 AM, Maxim Solodovnik <
>> > > > >> > > solomax...@gmail.com
>> > > > >> > > > >
>> > > > >> > > > > wrote:
>> > > > >> > > > >
>> > > > >> > > > >> Hello All,
>> > > > >> > > > >>
>> > > > >> > > > >> I wonder is there any way to extend gathered
>> > > > >> org.apache.wicket.pro
>> > > > >> > > > >> tocol.http.ClientProperties?
>> > > > >> > > > >> I would like to have port in addition to hostname
>> > > > >> > > > >> Or maybe I can provide PR to wicket so it would have
>> this
>> > > info
>> > > > by
>> > > > >> > > > default?
>> > > > >> > > > >>
>> > > > >> > > > >> WDYT?
>> > > > >> > > > >>
>> > > > >> > > > >> --
>> > > > >> > > > >> WBR
>> > > > >> > > > >> Maxim aka solomax
>> > > > >> > > > >>
>> > > > >> > > > >
>> > > > >> > > > >
>> > > > >> > > > >
>> > > 

Re: Is there any way to extend ClientProperties

2016-11-25 Thread Maxim Solodovnik
Sorry for OFF topic

I'm struggling with git :(
I do have clone of https://github.com/apache/wicket at
https://github.com/solomax/wicket, but I don't have necessary branch :(((
(WICKET-6056-client-properties)

I did
git remote add apache g...@github.com:apache/wicket.git
git fetch apache
git merge apache/master
git push

no branch :(
I can attach patch to WICKET-6056, But I believe git will be more
comfortable ...
could you please help me to get required branch?

Another issue 8.0.0-SNAPSHOP quickstart from here
https://wicket.apache.org/start/quickstart.html (all default params, jetty)
is not working :(
Would appreciate any help on fixing this

On Fri, Nov 25, 2016 at 3:57 AM, Martin Grigorov 
wrote:

> Hi Maxim,
>
> See Sven's changes in WICKET-6056 and my suggestion for the extra info
> gathering.
>
> Martin Grigorov
> Wicket Training and Consulting
> https://twitter.com/mtgrigorov
>
> On Thu, Nov 24, 2016 at 1:02 PM, Maxim Solodovnik 
> wrote:
>
> > Yep,
> > Unfortunately It's not that easy to
> > fix /wicket-core/src/main/java/org/apache/wicket/markup/html/
> > pages/BrowserInfoPage.java/html
> > I believe both need to be consistent
> > Or maybe we can leave only JS version and remove static one?
> >
> > On Thu, Nov 24, 2016 at 6:56 PM, Martin Grigorov 
> > wrote:
> >
> > > Hi Maxim,
> > >
> > > See org.apache.wicket.ajax.AjaxClientInfoBehavior works.
> > > Looking at it I think you can easily extend it by using monkey-patching
> > > for Wicket.BrowserInfo.collect()
> > >
> > > Martin Grigorov
> > > Wicket Training and Consulting
> > > https://twitter.com/mtgrigorov
> > >
> > > On Thu, Nov 24, 2016 at 12:40 PM, Maxim Solodovnik <
> solomax...@gmail.com
> > >
> > > wrote:
> > >
> > > > Hello Martin,
> > > >
> > > > One question before I'll start with this enhancement: Can pure JS
> > > solution
> > > > be used here? I can both create FORM and send all data via POST
> without
> > > any
> > > > markup ...
> > > > Would it be OK?
> > > >
> > > > On Wed, Nov 23, 2016 at 3:55 PM, Maxim Solodovnik <
> > solomax...@gmail.com>
> > > > wrote:
> > > >
> > > > > OK, will do, will take some time :)
> > > > >
> > > > > On Wed, Nov 23, 2016 at 3:54 PM, Martin Grigorov <
> > mgrigo...@apache.org
> > > >
> > > > > wrote:
> > > > >
> > > > >> Try the PR and see whether it helps you to achieve what you need.
> > > > >> If it does then we will process it sooner.
> > > > >> If it doesn't then make it work :-)
> > > > >>
> > > > >> Martin Grigorov
> > > > >> Wicket Training and Consulting
> > > > >> https://twitter.com/mtgrigorov
> > > > >>
> > > > >> On Wed, Nov 23, 2016 at 9:44 AM, Maxim Solodovnik <
> > > solomax...@gmail.com
> > > > >
> > > > >> wrote:
> > > > >>
> > > > >> > OK, how can I help?
> > > > >> >
> > > > >> > On Wed, Nov 23, 2016 at 3:40 PM, Martin Grigorov <
> > > > mgrigo...@apache.org>
> > > > >> > wrote:
> > > > >> >
> > > > >> > > See https://issues.apache.org/jira/browse/WICKET-6056
> > > > >> > > It is almost possible to extend the BrowserInfoPage. It needs
> > just
> > > > >> little
> > > > >> > > more work.
> > > > >> > >
> > > > >> > > Martin Grigorov
> > > > >> > > Wicket Training and Consulting
> > > > >> > > https://twitter.com/mtgrigorov
> > > > >> > >
> > > > >> > > On Tue, Nov 22, 2016 at 7:02 PM, Maxim Solodovnik <
> > > > >> solomax...@gmail.com>
> > > > >> > > wrote:
> > > > >> > >
> > > > >> > > > Does it make any sense?
> > > > >> > > >
> > > > >> > > > On Tue, Nov 22, 2016 at 10:25 AM, Maxim Solodovnik <
> > > > >> > solomax...@gmail.com
> > > > >> > > >
> > > > >> > > > wrote:
> > > > >> > > >
> > > > >> > > > > And maybe protocol+context (or window.location.href) :)
> > > > >> > > > > mod_proxy, mod_rewrite can change the URL a lot :)
> > > > >> > > > >
> > > > >> > > > > On Tue, Nov 22, 2016 at 10:18 AM, Maxim Solodovnik <
> > > > >> > > solomax...@gmail.com
> > > > >> > > > >
> > > > >> > > > > wrote:
> > > > >> > > > >
> > > > >> > > > >> Hello All,
> > > > >> > > > >>
> > > > >> > > > >> I wonder is there any way to extend gathered
> > > > >> org.apache.wicket.pro
> > > > >> > > > >> tocol.http.ClientProperties?
> > > > >> > > > >> I would like to have port in addition to hostname
> > > > >> > > > >> Or maybe I can provide PR to wicket so it would have this
> > > info
> > > > by
> > > > >> > > > default?
> > > > >> > > > >>
> > > > >> > > > >> WDYT?
> > > > >> > > > >>
> > > > >> > > > >> --
> > > > >> > > > >> WBR
> > > > >> > > > >> Maxim aka solomax
> > > > >> > > > >>
> > > > >> > > > >
> > > > >> > > > >
> > > > >> > > > >
> > > > >> > > > > --
> > > > >> > > > > WBR
> > > > >> > > > > Maxim aka solomax
> > > > >> > > > >
> > > > >> > > >
> > > > >> > > >
> > > > >> > > >
> > > > >> > > > --
> > > > >> > > > WBR
> > > > >> > > > Maxim aka solomax
> > > > >> > > >
> > > > >> > >
> > > > >> >
> > > > >> >
> > > > >> >
> > > > >> > --
> > > > >> > WBR
> > > > >> > Maxim aka solomax
> > > > >> >
> > > > >>
> > > 

Re: LDM reloaded several times

2016-11-25 Thread Ernesto Reinaldo Barreiro
@Sebastian,

Would a model that only detaches itself if there is no request target
scheduled after current work for your use case?

On Fri, Nov 25, 2016 at 4:00 PM, Sebastien  wrote:

> Hi Martin
>
> On Fri, Nov 25, 2016 at 3:52 PM, Martin Grigorov 
> wrote:
>
> > Hi Sebastien,
> >
> > Seems to work as designed! :-)
> >
>
> Yes definitely! sorry to have had a doubt! :D
>
>
> > The Ajax request is in a second request cycle so it loads it again.
> > I guess you can create some kind of ConversationModel (CDI conversations)
> > but this is totally different model impl.
> >
>
> Good to know, thanks!
>



-- 
Regards - Ernesto Reinaldo Barreiro


Re: LDM reloaded several times

2016-11-25 Thread Sebastien
Hi Martin

On Fri, Nov 25, 2016 at 3:52 PM, Martin Grigorov 
wrote:

> Hi Sebastien,
>
> Seems to work as designed! :-)
>

Yes definitely! sorry to have had a doubt! :D


> The Ajax request is in a second request cycle so it loads it again.
> I guess you can create some kind of ConversationModel (CDI conversations)
> but this is totally different model impl.
>

Good to know, thanks!


Re: LDM reloaded several times

2016-11-25 Thread Martin Grigorov
Hi Sebastien,

Seems to work as designed! :-)
The Ajax request is in a second request cycle so it loads it again.
I guess you can create some kind of ConversationModel (CDI conversations)
but this is totally different model impl.

Martin Grigorov
Wicket Training and Consulting
https://twitter.com/mtgrigorov

On Fri, Nov 25, 2016 at 3:47 PM, Sebastien  wrote:

> Hi all,
>
> Following a recent discussion about LDM being reloaded several times [1],
> I've prepared a quickstart [2].
>
> My concern was the following: I was thinking that a LDM relying on another
> LDM, and reused on several components may have been reloaded several times
> (because of the component's #reload which could have been called to early).
>
> It seems to not be the case. Good news!
>
> Then, I realized that I was using the LDM as a component model *and* in an
> AjaxBehavior [3]. Of course, the AjaxBehavior initiate a new request,
> causing the LDM to be reloaded.
>
> I do not expect a solution here, it's just a wrong usage on my side. But as
> I started the discussion, I wanted to loop the loop! :)
>
> Thanks & best regards,
> Sebastien.
>
> [1] http://markmail.org/message/lmg2zrfuovn4fhgg
> [2] https://github.com/sebfz1/wicket-quickstart-ldm
> [3]
> https://github.com/sebfz1/wicket-quickstart-ldm/blob/master/src/main/java/
> quickstart/QuickstartPage2.java
>


LDM reloaded several times

2016-11-25 Thread Sebastien
Hi all,

Following a recent discussion about LDM being reloaded several times [1],
I've prepared a quickstart [2].

My concern was the following: I was thinking that a LDM relying on another
LDM, and reused on several components may have been reloaded several times
(because of the component's #reload which could have been called to early).

It seems to not be the case. Good news!

Then, I realized that I was using the LDM as a component model *and* in an
AjaxBehavior [3]. Of course, the AjaxBehavior initiate a new request,
causing the LDM to be reloaded.

I do not expect a solution here, it's just a wrong usage on my side. But as
I started the discussion, I wanted to loop the loop! :)

Thanks & best regards,
Sebastien.

[1] http://markmail.org/message/lmg2zrfuovn4fhgg
[2] https://github.com/sebfz1/wicket-quickstart-ldm
[3]
https://github.com/sebfz1/wicket-quickstart-ldm/blob/master/src/main/java/quickstart/QuickstartPage2.java