Details

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

      Operating System: All
      Platform: Macintosh

    • Issuezilla Id:
      513

      Description

      The following should be reviewed/Changed. I think it makes sense to try and thro
      the same exception for the same reasons through out the API.

      ActionEvent states for the constructor.(since 1.2)
      Throws: IllegalArgumentException - if component is null

      I believe the following should follow suite, for consistency reasons.(These
      throw a NullPointerException for the same reason)

      These except UIComponent for arg:
      AfterAddToParentEvent
      AfterRestoreStateEvent
      BeforeRenderEvent
      ComponentSystemEvent
      InitialStateEvent

      So far the Impl seems to be throwing the IllegalStateException and not the doced
      NullPointerException. So the Impl needs no changes, just the API docs.

        Activity

        Hide
        rogerk added a comment -

        Ownership

        Show
        rogerk added a comment - Ownership
        Hide
        rogerk added a comment -

        This was fixed in the api javadocs. Also, some of these events don't exist in
        2.0 any longer.

        Show
        rogerk added a comment - This was fixed in the api javadocs. Also, some of these events don't exist in 2.0 any longer.
        Hide
        Ed Burns added a comment -

        Prepare to delete "spec" subcomponent.

        Show
        Ed Burns added a comment - Prepare to delete "spec" subcomponent.
        Hide
        Ed Burns added a comment -

        Move all to 1.2

        Show
        Ed Burns added a comment - Move all to 1.2
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            rogerk
            Reporter:
            dougd
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: