Wicket
  1. Wicket
  2. WICKET-4462

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

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor 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); }

      }

        Activity

        Don Ngo created issue -
        Sven Meier made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Invalid [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development