Component.getModel() should use wildcard return value

[Grandiosa]

Shouldn't this method on Component

public ModelData getModel() {
    return model;
}

instead be

public ? extends ModelData getModel() {
    return model;
}

I have a typed Store<MyModel> and in my tablelistener I want to get a record from the store, like this:

  Record record = store.getRecord(item.getModel());  // item is selected TableItem

But this fails since the the getRecord expects a MyModel instance, so a cast is needed:

  Record record = store.getRecord((MyModel)item.getModel());

[zaccret]

You cannot do :

public ? extends ModelData getModel() {
    return model;
}

This is incorrect. It could be instead :

public T getModel() {
    return model;
}

and add the <T extends ModelData> parameter on Component class. Darell, what would you think of that ? Not too restrictive ?

[Grandiosa]

Oops, one should never write code directly in the wiki-editor... I had this construction in mind

    public List<? extends ModelData> getModel() {
        ...
    }

which works.
But components have one model only so i second your suggestion to parametrize the Component class.

[zaccret]

By the way, I think we (users) should avoid to use the Component.get/setModel methods. If you use a table binder, try binder.getSelection() instead.