You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@wicket.apache.org by Jorge Gallardo <jo...@gmail.com> on 2011/09/21 17:08:18 UTC
Glitch in PageParameter API
Folks,
In an attempt to create my own implementation of PageParameterEncoder and
PageParameters, I encountered the following problem with the API:
The interface INamedParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html>
defines
the method getAllNamed()<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()>
that
returns a list of
NamedPairs<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html>
defined
as inner class of
PageParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html>
.
I believe that an interface shouldnt be referencing any element from an
implementation to be a true extensible interface.
A quick fix to this could be to convert NamedPair to a standard class and
make it extensible as well.
Thoughts?
--
Jorge Gallardo
----------------------------------------
jorgeagallardo@gmail.com
Re: Glitch in PageParameter API
Posted by Jorge Gallardo <jo...@gmail.com>.
Thx Igor, much appreciated
On Thu, Sep 22, 2011 at 9:10 PM, Igor Vaynberg <ig...@gmail.com>wrote:
> done
>
> -igor
>
>
> On Thu, Sep 22, 2011 at 6:45 PM, Jorge Gallardo
> <jo...@gmail.com> wrote:
> > Igor,
> >
> > The inner class has a private constructor, hence I cannot extend it.
> > Could you please at least in the interim make the constructor public?
> >
> > Thanks
> > JG
> >
> > On Wed, Sep 21, 2011 at 3:19 PM, Igor Vaynberg <igor.vaynberg@gmail.com
> >wrote:
> >
> >> there is no need for a workaround, this is a codesmell - nothing more.
> >>
> >> -igor
> >>
> >>
> >> On Wed, Sep 21, 2011 at 1:12 PM, Jorge Gallardo
> >> <jo...@gmail.com> wrote:
> >> > https://issues.apache.org/jira/browse/WICKET-4076 created
> >> >
> >> > What is your suggested workaround?
> >> >
> >> > Thanks
> >> > JG
> >> >
> >> > On Wed, Sep 21, 2011 at 3:06 PM, Igor Vaynberg <
> igor.vaynberg@gmail.com
> >> >wrote:
> >> >
> >> >> sure, as long as INamedParameters#getAllNamed() still uses
> >> >> PageParameters#NamedPair which of course will not fix anything....
> >> >>
> >> >> -igor
> >> >>
> >> >> On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <
> bruno.borges@gmail.com>
> >> >> wrote:
> >> >> > Well, it could be done in 1.5.x this way:
> >> >> >
> >> >> > PageParameters.NamedPairs extends INamedParameters.NamedPairs
> >> >> >
> >> >> > In 1.6 then PP.NP could be dropped.
> >> >> >
> >> >> > What about that?
> >> >> >
> >> >> >
> >> >> > *Bruno Borges*
> >> >> > (21) 7672-7099
> >> >> > *www.brunoborges.com*
> >> >> >
> >> >> >
> >> >> >
> >> >> > On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <
> >> igor.vaynberg@gmail.com
> >> >> >wrote:
> >> >> >
> >> >> >> cant do in 1.5.x. make a ticket and schedule for 1.6.0
> >> >> >>
> >> >> >> -igor
> >> >> >>
> >> >> >>
> >> >> >> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <
> >> mgrigorov@apache.org>
> >> >> >> wrote:
> >> >> >> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
> >> >> >> > <jo...@gmail.com> wrote:
> >> >> >> >> Folks,
> >> >> >> >>
> >> >> >> >> In an attempt to create my own implementation of
> >> PageParameterEncoder
> >> >> >> and
> >> >> >> >> PageParameters, I encountered the following problem with the
> API:
> >> >> >> >>
> >> >> >> >> The interface INamedParameters<
> >> >> >>
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
> >> >> >> >
> >> >> >> >> defines
> >> >> >> >> the method getAllNamed()<
> >> >> >>
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
> >> >> >> >
> >> >> >> >> that
> >> >> >> >> returns a list of
> >> >> >> >> NamedPairs<
> >> >> >>
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
> >> >> >> >
> >> >> >> >> defined
> >> >> >> >> as inner class of
> >> >> >> >> PageParameters<
> >> >> >>
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
> >> >> >> >
> >> >> >> >> .
> >> >> >> >>
> >> >> >> >> I believe that an interface shouldnt be referencing any element
> >> from
> >> >> an
> >> >> >> >> implementation to be a true extensible interface.
> >> >> >> >> A quick fix to this could be to convert NamedPair to a standard
> >> class
> >> >> >> and
> >> >> >> >> make it extensible as well.
> >> >> >> > Or make it INamedParameters.NamedPair
> >> >> >> >>
> >> >> >> >> Thoughts?
> >> >> >> >>
> >> >> >> >> --
> >> >> >> >> Jorge Gallardo
> >> >> >> >> ----------------------------------------
> >> >> >> >> jorgeagallardo@gmail.com
> >> >> >> >>
> >> >> >> >
> >> >> >> >
> >> >> >> >
> >> >> >> > --
> >> >> >> > Martin Grigorov
> >> >> >> > jWeekend
> >> >> >> > Training, Consulting, Development
> >> >> >> > http://jWeekend.com
> >> >> >> >
> >> >> >> >
> >> ---------------------------------------------------------------------
> >> >> >> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> >> >> > For additional commands, e-mail: users-help@wicket.apache.org
> >> >> >> >
> >> >> >> >
> >> >> >>
> >> >> >>
> ---------------------------------------------------------------------
> >> >> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> >> >> For additional commands, e-mail: users-help@wicket.apache.org
> >> >> >>
> >> >> >>
> >> >> >
> >> >>
> >> >> ---------------------------------------------------------------------
> >> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> >> For additional commands, e-mail: users-help@wicket.apache.org
> >> >>
> >> >>
> >> >
> >> >
> >> > --
> >> > Jorge Gallardo
> >> > ----------------------------------------
> >> > jorgeagallardo@gmail.com
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> For additional commands, e-mail: users-help@wicket.apache.org
> >>
> >>
> >
> >
> > --
> > Jorge Gallardo
> > ----------------------------------------
> > jorgeagallardo@gmail.com
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>
--
Jorge Gallardo
----------------------------------------
jorgeagallardo@gmail.com
Re: Glitch in PageParameter API
Posted by Igor Vaynberg <ig...@gmail.com>.
done
-igor
On Thu, Sep 22, 2011 at 6:45 PM, Jorge Gallardo
<jo...@gmail.com> wrote:
> Igor,
>
> The inner class has a private constructor, hence I cannot extend it.
> Could you please at least in the interim make the constructor public?
>
> Thanks
> JG
>
> On Wed, Sep 21, 2011 at 3:19 PM, Igor Vaynberg <ig...@gmail.com>wrote:
>
>> there is no need for a workaround, this is a codesmell - nothing more.
>>
>> -igor
>>
>>
>> On Wed, Sep 21, 2011 at 1:12 PM, Jorge Gallardo
>> <jo...@gmail.com> wrote:
>> > https://issues.apache.org/jira/browse/WICKET-4076 created
>> >
>> > What is your suggested workaround?
>> >
>> > Thanks
>> > JG
>> >
>> > On Wed, Sep 21, 2011 at 3:06 PM, Igor Vaynberg <igor.vaynberg@gmail.com
>> >wrote:
>> >
>> >> sure, as long as INamedParameters#getAllNamed() still uses
>> >> PageParameters#NamedPair which of course will not fix anything....
>> >>
>> >> -igor
>> >>
>> >> On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <br...@gmail.com>
>> >> wrote:
>> >> > Well, it could be done in 1.5.x this way:
>> >> >
>> >> > PageParameters.NamedPairs extends INamedParameters.NamedPairs
>> >> >
>> >> > In 1.6 then PP.NP could be dropped.
>> >> >
>> >> > What about that?
>> >> >
>> >> >
>> >> > *Bruno Borges*
>> >> > (21) 7672-7099
>> >> > *www.brunoborges.com*
>> >> >
>> >> >
>> >> >
>> >> > On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <
>> igor.vaynberg@gmail.com
>> >> >wrote:
>> >> >
>> >> >> cant do in 1.5.x. make a ticket and schedule for 1.6.0
>> >> >>
>> >> >> -igor
>> >> >>
>> >> >>
>> >> >> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <
>> mgrigorov@apache.org>
>> >> >> wrote:
>> >> >> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
>> >> >> > <jo...@gmail.com> wrote:
>> >> >> >> Folks,
>> >> >> >>
>> >> >> >> In an attempt to create my own implementation of
>> PageParameterEncoder
>> >> >> and
>> >> >> >> PageParameters, I encountered the following problem with the API:
>> >> >> >>
>> >> >> >> The interface INamedParameters<
>> >> >>
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
>> >> >> >
>> >> >> >> defines
>> >> >> >> the method getAllNamed()<
>> >> >>
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
>> >> >> >
>> >> >> >> that
>> >> >> >> returns a list of
>> >> >> >> NamedPairs<
>> >> >>
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
>> >> >> >
>> >> >> >> defined
>> >> >> >> as inner class of
>> >> >> >> PageParameters<
>> >> >>
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
>> >> >> >
>> >> >> >> .
>> >> >> >>
>> >> >> >> I believe that an interface shouldnt be referencing any element
>> from
>> >> an
>> >> >> >> implementation to be a true extensible interface.
>> >> >> >> A quick fix to this could be to convert NamedPair to a standard
>> class
>> >> >> and
>> >> >> >> make it extensible as well.
>> >> >> > Or make it INamedParameters.NamedPair
>> >> >> >>
>> >> >> >> Thoughts?
>> >> >> >>
>> >> >> >> --
>> >> >> >> Jorge Gallardo
>> >> >> >> ----------------------------------------
>> >> >> >> jorgeagallardo@gmail.com
>> >> >> >>
>> >> >> >
>> >> >> >
>> >> >> >
>> >> >> > --
>> >> >> > Martin Grigorov
>> >> >> > jWeekend
>> >> >> > Training, Consulting, Development
>> >> >> > http://jWeekend.com
>> >> >> >
>> >> >> >
>> ---------------------------------------------------------------------
>> >> >> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> >> > For additional commands, e-mail: users-help@wicket.apache.org
>> >> >> >
>> >> >> >
>> >> >>
>> >> >> ---------------------------------------------------------------------
>> >> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> >> For additional commands, e-mail: users-help@wicket.apache.org
>> >> >>
>> >> >>
>> >> >
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> For additional commands, e-mail: users-help@wicket.apache.org
>> >>
>> >>
>> >
>> >
>> > --
>> > Jorge Gallardo
>> > ----------------------------------------
>> > jorgeagallardo@gmail.com
>> >
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> For additional commands, e-mail: users-help@wicket.apache.org
>>
>>
>
>
> --
> Jorge Gallardo
> ----------------------------------------
> jorgeagallardo@gmail.com
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: Glitch in PageParameter API
Posted by Jorge Gallardo <jo...@gmail.com>.
Igor,
The inner class has a private constructor, hence I cannot extend it.
Could you please at least in the interim make the constructor public?
Thanks
JG
On Wed, Sep 21, 2011 at 3:19 PM, Igor Vaynberg <ig...@gmail.com>wrote:
> there is no need for a workaround, this is a codesmell - nothing more.
>
> -igor
>
>
> On Wed, Sep 21, 2011 at 1:12 PM, Jorge Gallardo
> <jo...@gmail.com> wrote:
> > https://issues.apache.org/jira/browse/WICKET-4076 created
> >
> > What is your suggested workaround?
> >
> > Thanks
> > JG
> >
> > On Wed, Sep 21, 2011 at 3:06 PM, Igor Vaynberg <igor.vaynberg@gmail.com
> >wrote:
> >
> >> sure, as long as INamedParameters#getAllNamed() still uses
> >> PageParameters#NamedPair which of course will not fix anything....
> >>
> >> -igor
> >>
> >> On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <br...@gmail.com>
> >> wrote:
> >> > Well, it could be done in 1.5.x this way:
> >> >
> >> > PageParameters.NamedPairs extends INamedParameters.NamedPairs
> >> >
> >> > In 1.6 then PP.NP could be dropped.
> >> >
> >> > What about that?
> >> >
> >> >
> >> > *Bruno Borges*
> >> > (21) 7672-7099
> >> > *www.brunoborges.com*
> >> >
> >> >
> >> >
> >> > On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <
> igor.vaynberg@gmail.com
> >> >wrote:
> >> >
> >> >> cant do in 1.5.x. make a ticket and schedule for 1.6.0
> >> >>
> >> >> -igor
> >> >>
> >> >>
> >> >> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <
> mgrigorov@apache.org>
> >> >> wrote:
> >> >> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
> >> >> > <jo...@gmail.com> wrote:
> >> >> >> Folks,
> >> >> >>
> >> >> >> In an attempt to create my own implementation of
> PageParameterEncoder
> >> >> and
> >> >> >> PageParameters, I encountered the following problem with the API:
> >> >> >>
> >> >> >> The interface INamedParameters<
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
> >> >> >
> >> >> >> defines
> >> >> >> the method getAllNamed()<
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
> >> >> >
> >> >> >> that
> >> >> >> returns a list of
> >> >> >> NamedPairs<
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
> >> >> >
> >> >> >> defined
> >> >> >> as inner class of
> >> >> >> PageParameters<
> >> >>
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
> >> >> >
> >> >> >> .
> >> >> >>
> >> >> >> I believe that an interface shouldnt be referencing any element
> from
> >> an
> >> >> >> implementation to be a true extensible interface.
> >> >> >> A quick fix to this could be to convert NamedPair to a standard
> class
> >> >> and
> >> >> >> make it extensible as well.
> >> >> > Or make it INamedParameters.NamedPair
> >> >> >>
> >> >> >> Thoughts?
> >> >> >>
> >> >> >> --
> >> >> >> Jorge Gallardo
> >> >> >> ----------------------------------------
> >> >> >> jorgeagallardo@gmail.com
> >> >> >>
> >> >> >
> >> >> >
> >> >> >
> >> >> > --
> >> >> > Martin Grigorov
> >> >> > jWeekend
> >> >> > Training, Consulting, Development
> >> >> > http://jWeekend.com
> >> >> >
> >> >> >
> ---------------------------------------------------------------------
> >> >> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> >> > For additional commands, e-mail: users-help@wicket.apache.org
> >> >> >
> >> >> >
> >> >>
> >> >> ---------------------------------------------------------------------
> >> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> >> For additional commands, e-mail: users-help@wicket.apache.org
> >> >>
> >> >>
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> For additional commands, e-mail: users-help@wicket.apache.org
> >>
> >>
> >
> >
> > --
> > Jorge Gallardo
> > ----------------------------------------
> > jorgeagallardo@gmail.com
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>
--
Jorge Gallardo
----------------------------------------
jorgeagallardo@gmail.com
Re: Glitch in PageParameter API
Posted by Igor Vaynberg <ig...@gmail.com>.
there is no need for a workaround, this is a codesmell - nothing more.
-igor
On Wed, Sep 21, 2011 at 1:12 PM, Jorge Gallardo
<jo...@gmail.com> wrote:
> https://issues.apache.org/jira/browse/WICKET-4076 created
>
> What is your suggested workaround?
>
> Thanks
> JG
>
> On Wed, Sep 21, 2011 at 3:06 PM, Igor Vaynberg <ig...@gmail.com>wrote:
>
>> sure, as long as INamedParameters#getAllNamed() still uses
>> PageParameters#NamedPair which of course will not fix anything....
>>
>> -igor
>>
>> On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <br...@gmail.com>
>> wrote:
>> > Well, it could be done in 1.5.x this way:
>> >
>> > PageParameters.NamedPairs extends INamedParameters.NamedPairs
>> >
>> > In 1.6 then PP.NP could be dropped.
>> >
>> > What about that?
>> >
>> >
>> > *Bruno Borges*
>> > (21) 7672-7099
>> > *www.brunoborges.com*
>> >
>> >
>> >
>> > On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <igor.vaynberg@gmail.com
>> >wrote:
>> >
>> >> cant do in 1.5.x. make a ticket and schedule for 1.6.0
>> >>
>> >> -igor
>> >>
>> >>
>> >> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <mg...@apache.org>
>> >> wrote:
>> >> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
>> >> > <jo...@gmail.com> wrote:
>> >> >> Folks,
>> >> >>
>> >> >> In an attempt to create my own implementation of PageParameterEncoder
>> >> and
>> >> >> PageParameters, I encountered the following problem with the API:
>> >> >>
>> >> >> The interface INamedParameters<
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
>> >> >
>> >> >> defines
>> >> >> the method getAllNamed()<
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
>> >> >
>> >> >> that
>> >> >> returns a list of
>> >> >> NamedPairs<
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
>> >> >
>> >> >> defined
>> >> >> as inner class of
>> >> >> PageParameters<
>> >>
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
>> >> >
>> >> >> .
>> >> >>
>> >> >> I believe that an interface shouldnt be referencing any element from
>> an
>> >> >> implementation to be a true extensible interface.
>> >> >> A quick fix to this could be to convert NamedPair to a standard class
>> >> and
>> >> >> make it extensible as well.
>> >> > Or make it INamedParameters.NamedPair
>> >> >>
>> >> >> Thoughts?
>> >> >>
>> >> >> --
>> >> >> Jorge Gallardo
>> >> >> ----------------------------------------
>> >> >> jorgeagallardo@gmail.com
>> >> >>
>> >> >
>> >> >
>> >> >
>> >> > --
>> >> > Martin Grigorov
>> >> > jWeekend
>> >> > Training, Consulting, Development
>> >> > http://jWeekend.com
>> >> >
>> >> > ---------------------------------------------------------------------
>> >> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> > For additional commands, e-mail: users-help@wicket.apache.org
>> >> >
>> >> >
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> >> For additional commands, e-mail: users-help@wicket.apache.org
>> >>
>> >>
>> >
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> For additional commands, e-mail: users-help@wicket.apache.org
>>
>>
>
>
> --
> Jorge Gallardo
> ----------------------------------------
> jorgeagallardo@gmail.com
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: Glitch in PageParameter API
Posted by Jorge Gallardo <jo...@gmail.com>.
https://issues.apache.org/jira/browse/WICKET-4076 created
What is your suggested workaround?
Thanks
JG
On Wed, Sep 21, 2011 at 3:06 PM, Igor Vaynberg <ig...@gmail.com>wrote:
> sure, as long as INamedParameters#getAllNamed() still uses
> PageParameters#NamedPair which of course will not fix anything....
>
> -igor
>
> On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <br...@gmail.com>
> wrote:
> > Well, it could be done in 1.5.x this way:
> >
> > PageParameters.NamedPairs extends INamedParameters.NamedPairs
> >
> > In 1.6 then PP.NP could be dropped.
> >
> > What about that?
> >
> >
> > *Bruno Borges*
> > (21) 7672-7099
> > *www.brunoborges.com*
> >
> >
> >
> > On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <igor.vaynberg@gmail.com
> >wrote:
> >
> >> cant do in 1.5.x. make a ticket and schedule for 1.6.0
> >>
> >> -igor
> >>
> >>
> >> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <mg...@apache.org>
> >> wrote:
> >> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
> >> > <jo...@gmail.com> wrote:
> >> >> Folks,
> >> >>
> >> >> In an attempt to create my own implementation of PageParameterEncoder
> >> and
> >> >> PageParameters, I encountered the following problem with the API:
> >> >>
> >> >> The interface INamedParameters<
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
> >> >
> >> >> defines
> >> >> the method getAllNamed()<
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
> >> >
> >> >> that
> >> >> returns a list of
> >> >> NamedPairs<
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
> >> >
> >> >> defined
> >> >> as inner class of
> >> >> PageParameters<
> >>
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
> >> >
> >> >> .
> >> >>
> >> >> I believe that an interface shouldnt be referencing any element from
> an
> >> >> implementation to be a true extensible interface.
> >> >> A quick fix to this could be to convert NamedPair to a standard class
> >> and
> >> >> make it extensible as well.
> >> > Or make it INamedParameters.NamedPair
> >> >>
> >> >> Thoughts?
> >> >>
> >> >> --
> >> >> Jorge Gallardo
> >> >> ----------------------------------------
> >> >> jorgeagallardo@gmail.com
> >> >>
> >> >
> >> >
> >> >
> >> > --
> >> > Martin Grigorov
> >> > jWeekend
> >> > Training, Consulting, Development
> >> > http://jWeekend.com
> >> >
> >> > ---------------------------------------------------------------------
> >> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> > For additional commands, e-mail: users-help@wicket.apache.org
> >> >
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> >> For additional commands, e-mail: users-help@wicket.apache.org
> >>
> >>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>
--
Jorge Gallardo
----------------------------------------
jorgeagallardo@gmail.com
Re: Glitch in PageParameter API
Posted by Igor Vaynberg <ig...@gmail.com>.
sure, as long as INamedParameters#getAllNamed() still uses
PageParameters#NamedPair which of course will not fix anything....
-igor
On Wed, Sep 21, 2011 at 12:55 PM, Bruno Borges <br...@gmail.com> wrote:
> Well, it could be done in 1.5.x this way:
>
> PageParameters.NamedPairs extends INamedParameters.NamedPairs
>
> In 1.6 then PP.NP could be dropped.
>
> What about that?
>
>
> *Bruno Borges*
> (21) 7672-7099
> *www.brunoborges.com*
>
>
>
> On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <ig...@gmail.com>wrote:
>
>> cant do in 1.5.x. make a ticket and schedule for 1.6.0
>>
>> -igor
>>
>>
>> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <mg...@apache.org>
>> wrote:
>> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
>> > <jo...@gmail.com> wrote:
>> >> Folks,
>> >>
>> >> In an attempt to create my own implementation of PageParameterEncoder
>> and
>> >> PageParameters, I encountered the following problem with the API:
>> >>
>> >> The interface INamedParameters<
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
>> >
>> >> defines
>> >> the method getAllNamed()<
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
>> >
>> >> that
>> >> returns a list of
>> >> NamedPairs<
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
>> >
>> >> defined
>> >> as inner class of
>> >> PageParameters<
>> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
>> >
>> >> .
>> >>
>> >> I believe that an interface shouldnt be referencing any element from an
>> >> implementation to be a true extensible interface.
>> >> A quick fix to this could be to convert NamedPair to a standard class
>> and
>> >> make it extensible as well.
>> > Or make it INamedParameters.NamedPair
>> >>
>> >> Thoughts?
>> >>
>> >> --
>> >> Jorge Gallardo
>> >> ----------------------------------------
>> >> jorgeagallardo@gmail.com
>> >>
>> >
>> >
>> >
>> > --
>> > Martin Grigorov
>> > jWeekend
>> > Training, Consulting, Development
>> > http://jWeekend.com
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> > For additional commands, e-mail: users-help@wicket.apache.org
>> >
>> >
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
>> For additional commands, e-mail: users-help@wicket.apache.org
>>
>>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: Glitch in PageParameter API
Posted by Bruno Borges <br...@gmail.com>.
Well, it could be done in 1.5.x this way:
PageParameters.NamedPairs extends INamedParameters.NamedPairs
In 1.6 then PP.NP could be dropped.
What about that?
*Bruno Borges*
(21) 7672-7099
*www.brunoborges.com*
On Wed, Sep 21, 2011 at 12:13 PM, Igor Vaynberg <ig...@gmail.com>wrote:
> cant do in 1.5.x. make a ticket and schedule for 1.6.0
>
> -igor
>
>
> On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <mg...@apache.org>
> wrote:
> > On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
> > <jo...@gmail.com> wrote:
> >> Folks,
> >>
> >> In an attempt to create my own implementation of PageParameterEncoder
> and
> >> PageParameters, I encountered the following problem with the API:
> >>
> >> The interface INamedParameters<
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html
> >
> >> defines
> >> the method getAllNamed()<
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()
> >
> >> that
> >> returns a list of
> >> NamedPairs<
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html
> >
> >> defined
> >> as inner class of
> >> PageParameters<
> http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html
> >
> >> .
> >>
> >> I believe that an interface shouldnt be referencing any element from an
> >> implementation to be a true extensible interface.
> >> A quick fix to this could be to convert NamedPair to a standard class
> and
> >> make it extensible as well.
> > Or make it INamedParameters.NamedPair
> >>
> >> Thoughts?
> >>
> >> --
> >> Jorge Gallardo
> >> ----------------------------------------
> >> jorgeagallardo@gmail.com
> >>
> >
> >
> >
> > --
> > Martin Grigorov
> > jWeekend
> > Training, Consulting, Development
> > http://jWeekend.com
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> > For additional commands, e-mail: users-help@wicket.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>
Re: Glitch in PageParameter API
Posted by Igor Vaynberg <ig...@gmail.com>.
cant do in 1.5.x. make a ticket and schedule for 1.6.0
-igor
On Wed, Sep 21, 2011 at 8:12 AM, Martin Grigorov <mg...@apache.org> wrote:
> On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
> <jo...@gmail.com> wrote:
>> Folks,
>>
>> In an attempt to create my own implementation of PageParameterEncoder and
>> PageParameters, I encountered the following problem with the API:
>>
>> The interface INamedParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html>
>> defines
>> the method getAllNamed()<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()>
>> that
>> returns a list of
>> NamedPairs<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html>
>> defined
>> as inner class of
>> PageParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html>
>> .
>>
>> I believe that an interface shouldnt be referencing any element from an
>> implementation to be a true extensible interface.
>> A quick fix to this could be to convert NamedPair to a standard class and
>> make it extensible as well.
> Or make it INamedParameters.NamedPair
>>
>> Thoughts?
>>
>> --
>> Jorge Gallardo
>> ----------------------------------------
>> jorgeagallardo@gmail.com
>>
>
>
>
> --
> Martin Grigorov
> jWeekend
> Training, Consulting, Development
> http://jWeekend.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org
Re: Glitch in PageParameter API
Posted by Martin Grigorov <mg...@apache.org>.
On Wed, Sep 21, 2011 at 6:08 PM, Jorge Gallardo
<jo...@gmail.com> wrote:
> Folks,
>
> In an attempt to create my own implementation of PageParameterEncoder and
> PageParameters, I encountered the following problem with the API:
>
> The interface INamedParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html>
> defines
> the method getAllNamed()<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/INamedParameters.html#getAllNamed()>
> that
> returns a list of
> NamedPairs<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.NamedPair.html>
> defined
> as inner class of
> PageParameters<http://wicket.apache.org/apidocs/1.5/org/apache/wicket/request/mapper/parameter/PageParameters.html>
> .
>
> I believe that an interface shouldnt be referencing any element from an
> implementation to be a true extensible interface.
> A quick fix to this could be to convert NamedPair to a standard class and
> make it extensible as well.
Or make it INamedParameters.NamedPair
>
> Thoughts?
>
> --
> Jorge Gallardo
> ----------------------------------------
> jorgeagallardo@gmail.com
>
--
Martin Grigorov
jWeekend
Training, Consulting, Development
http://jWeekend.com
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org