Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-4462

Update AbstractPropertyModel and PropertyModel constructor to include parameter for IModel<?>

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Invalid
    • Affects Version/s: 1.5.5
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Please consider adding another constructor to AbstractPropertyModel and PropertyModel class to include a parameter for IModel<?>. Since both of these classes implement the IChainingModel interface, I think it makes sense to include a parameter for IModel<?> in the constructor for these classes, as in the case of CompoundPropertyModel class.

      For what's it worth, below is my custom class to enhance the PropertyModel class:

      public class CustomPropertyModel<T> extends PropertyModel<T> {
      private static final long serialVersionUID = 1L;

      public CustomPropertyModel(IModel<T> model, String expression)

      { super(model.getObject(), expression); setChainedModel(model); }

      }

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              vvvngo Don Ngo
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: