Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Uncategorized
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      327
    • Status Whiteboard:
      Hide

      EGTop5 effort_moderate

      Show
      EGTop5 effort_moderate

      Description

      the resetValue() method was added directly to UIInput, instead to a
      proper interface (EditableValueHolder).
      I guess this was done, to not break impls of that interface.

      It should have been added to the interface to start with as it simplifies many
      loops used for reseting EditableValueHolder of the whole tree. You cannot use
      instanceof UIInput for those as Trinidad input components, for example, does not
      extends UIInput, but does implement EditableValueHolder

      IMO this is wrong and should (at least in JSF2) be part of the
      EditableValueHolder interface.

      Since JSF2 will bring much more new bits, such an "enhancement" on the
      interface might be valueable.

        Activity

        Hide
        Ed Burns added a comment -

        effort_moderate

        Show
        Ed Burns added a comment - effort_moderate
        Hide
        Ed Burns added a comment -

        change target_milestone to 2.0

        Show
        Ed Burns added a comment - change target_milestone to 2.0
        Hide
        Ed Burns added a comment -

        Fix checked in.

        Show
        Ed Burns added a comment - Fix checked in.
        Hide
        Ed Burns added a comment -

        Prepare to delete api subcomponent

        Show
        Ed Burns added a comment - Prepare to delete api subcomponent
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            Ed Burns
            Reporter:
            mwessendorf
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: