fixed

On 10/1/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:

I just read and grokked the ResourceModel class. At the moment we use
a setter to set the component on the resource model. The setter is
package private, which is fairly odd. I found the whole structure
quite confusing: which instance is getting the component?

So I propose to change the code to this:

- void setComponent(Component c)
- {
-     component = c;
- }

public AssignmentWrapper(String key, String defaultValue, Component
component)
{
    super(key, defaultValue);
-     setComponent(component);
+    super.component = component;
}

wdyt?

Martijn

--
<a href="http://www.thebeststuffintheworld.com/vote_for/wicket";>Vote</a>
for <a href="http://www.thebeststuffintheworld.com/stuff/wicket
">Wicket</a>
at the <a href="http://www.thebeststuffintheworld.com/";>Best Stuff in
the World!</a>

Reply via email to