[ 
https://issues.apache.org/jira/browse/WICKET-3983?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13088810#comment-13088810
 ] 

Don Ngo commented on WICKET-3983:
---------------------------------

On second thought, I think if the LoadableDetachableModel is chainable, then I 
can do something like this:

public abstract class AuditableCoppoundLoadableDetachablePropertyModel<E 
extends Auditable, D extends AuditableDao<E>>
 extends LoadableDetachableModel<E> { 
public AuditableLoadableDetachableModel(E entity) { 
super(new CompoundPropertyModel<E>(entity)); 
} 
... 
} 

final class HolidayCompoundPropertyModel extends 
AuditableLoadableDetachableModel<Holiday> {
 } 

The HolidayCompoundPropertyModel class is now much simpler, no more inner class.

Martin/Igor: Your thoughts please.

> Update LoadableDetachableModel to include implementation of IChainingModel
> --------------------------------------------------------------------------
>
>                 Key: WICKET-3983
>                 URL: https://issues.apache.org/jira/browse/WICKET-3983
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.5-RC5.1
>            Reporter: Don Ngo
>            Assignee: Igor Vaynberg
>            Priority: Minor
>
> Please consider updating LoadableDetachableModel to include implementation of 
> IChainingModel.  Doing so will allow the LoadableDetachableModel to be 
> chained with other chainable models such as CompoundPropertyModel.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to