Re: [Wicket-user] hey, I was uisng that "target" model

[Eelco Hillenius]

What are we going to do with this?

In my opinion these things should be fixed in CompoundPropertyModel:

* Use consistent naming. Currently, the field is called target, but
the the constructor argument is called model. We need to find the best
name and use this throughout the class.
* Once we decide on the field name, we should expose a getter and
setter for that field.
* I think the current setObject implementation is wrong. It should
either be like getObject, thus:
  if (target instanceof IModel) {
((IModel)target).setObject(object);
  } else {
this.target = object; // yes, naming should change
  }
  or we should make this a read-only model.

WDYT?

Eelco

> Yes, that's it.
>
> Johan Compagner wrote:
> > ahh wait i see that the getObject does  a recursive call if it is a model
> > So you want the internal model instead of the real model object ?

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Re: [Wicket-user] hey, I was uisng that "target" model

[Nathan Hamblen]

Yes, that's it.

Johan Compagner wrote:
> ahh wait i see that the getObject does  a recursive call if it is a model
> So you want the internal model instead of the real model object ?
> 
> johan
> 
> 
> On 3/21/07, * Johan Compagner* <[EMAIL PROTECTED] 
> > wrote:
> 
> getObject() of CompoundPropertyModel itself == the target.
> 
> 
> 
> 
> On 3/21/07, *Nathan Hamblen* < [EMAIL PROTECTED]
> > wrote:
> 
> Now that 2.0-style compound models are in 1.x snapshots, I have a
> question for people that have been using them: how do you work
> without
> CompoundPropertyModel.getNetstedModel()? For me, this is often a
> special
> model that I want to do something with, but in the newer style it's
> called the "target" and isn't, as far as I can tell, accessible
> from
> outside the compound.
> 
> For now I'm holding on to my reference to the model that I'm setting
> inside the compound, but that's not very satisfying or flexible.
> 
> Nathan
> 


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Re: [Wicket-user] hey, I was uisng that "target" model

[Johan Compagner]

ahh wait i see that the getObject does  a recursive call if it is a model
So you want the internal model instead of the real model object ?

johan


On 3/21/07, Johan Compagner <[EMAIL PROTECTED]> wrote:


getObject() of CompoundPropertyModel itself == the target.



On 3/21/07, Nathan Hamblen <[EMAIL PROTECTED] > wrote:
>
> Now that 2.0-style compound models are in 1.x snapshots, I have a
> question for people that have been using them: how do you work without
> CompoundPropertyModel.getNetstedModel()? For me, this is often a special
> model that I want to do something with, but in the newer style it's
> called the "target" and isn't, as far as I can tell, accessible from
> outside the compound.
>
> For now I'm holding on to my reference to the model that I'm setting
> inside the compound, but that's not very satisfying or flexible.
>
> Nathan
>
>
> -
>
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share
> your
> opinions on IT & business topics through brief surveys-and earn cash
>
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> ___
> Wicket-user mailing list
> Wicket-user@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wicket-user
>


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Re: [Wicket-user] hey, I was uisng that "target" model

[Johan Compagner]

getObject() of CompoundPropertyModel itself == the target.



On 3/21/07, Nathan Hamblen <[EMAIL PROTECTED]> wrote:


Now that 2.0-style compound models are in 1.x snapshots, I have a
question for people that have been using them: how do you work without
CompoundPropertyModel.getNetstedModel()? For me, this is often a special
model that I want to do something with, but in the newer style it's
called the "target" and isn't, as far as I can tell, accessible from
outside the compound.

For now I'm holding on to my reference to the model that I'm setting
inside the compound, but that's not very satisfying or flexible.

Nathan


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share
your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

[Wicket-user] hey, I was uisng that "target" model

[Nathan Hamblen]

Now that 2.0-style compound models are in 1.x snapshots, I have a 
question for people that have been using them: how do you work without 
CompoundPropertyModel.getNetstedModel()? For me, this is often a special 
model that I want to do something with, but in the newer style it's 
called the "target" and isn't, as far as I can tell, accessible from 
outside the compound.

For now I'm holding on to my reference to the model that I'm setting 
inside the compound, but that's not very satisfying or flexible.

Nathan


-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user