You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@wicket.apache.org by Martijn Dashorst <ma...@gmail.com> on 2016/11/10 21:30:55 UTC

Change return type of LoadableDetachabeModel.of() to a LDM

The LoadableDetachableModel factory method 'of' currently returns a
IModel. I propose to return a LoadableDetachableModel<T> so that users
can use the isAttached() method.

Querying the state of a LDM is something you typically need to do to
ensure proper loading, and prevent excessive detach/load calls.

Martijn

Re: Change return type of LoadableDetachabeModel.of() to a LDM

Posted by Martijn Dashorst <ma...@gmail.com>.

OK, done.

Martijn


On Fri, Nov 11, 2016 at 7:42 AM, Martin Grigorov <mg...@apache.org> wrote:
> +1
>
> Martin Grigorov
> Wicket Training and Consulting
> https://twitter.com/mtgrigorov
>
> On Thu, Nov 10, 2016 at 10:30 PM, Martijn Dashorst <
> martijn.dashorst@gmail.com> wrote:
>
>> The LoadableDetachableModel factory method 'of' currently returns a
>> IModel. I propose to return a LoadableDetachableModel<T> so that users
>> can use the isAttached() method.
>>
>> Querying the state of a LDM is something you typically need to do to
>> ensure proper loading, and prevent excessive detach/load calls.
>>
>> Martijn
>>



-- 
Become a Wicket expert, learn from the best: http://wicketinaction.com

Re: Change return type of LoadableDetachabeModel.of() to a LDM

Posted by Martin Grigorov <mg...@apache.org>.

+1

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

On Thu, Nov 10, 2016 at 10:30 PM, Martijn Dashorst <
martijn.dashorst@gmail.com> wrote:

> The LoadableDetachableModel factory method 'of' currently returns a
> IModel. I propose to return a LoadableDetachableModel<T> so that users
> can use the isAttached() method.
>
> Querying the state of a LDM is something you typically need to do to
> ensure proper loading, and prevent excessive detach/load calls.
>
> Martijn
>