Details

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

      Operating System: All
      Platform: All

    • Issuezilla Id:
      172

      Description

      targetClass arg to constructor should not be any Class -
      it should an Enum class type.

      getAsString() method should throw ConverterException
      when a failed attempt to match value arg to one of constants.

      javadocs ned to be changed accordingly.

        Activity

        Hide
        rogerk added a comment -

        From reviewer:

        I can't comment on the issue tracker, so I'll do it here.

        The bundle looks good to me, with the exception of the HTML errors in the
        javadoc (,/code> instead of </code>). Assuming that's corrected (minor as it is)

        r=jdlee

        Show
        rogerk added a comment - From reviewer: I can't comment on the issue tracker, so I'll do it here. The bundle looks good to me, with the exception of the HTML errors in the javadoc (,/code> instead of </code>). Assuming that's corrected (minor as it is) r=jdlee
        Hide
        rogerk added a comment -

        Thanks for catching that.
        checked in.

        Show
        rogerk added a comment - Thanks for catching that. checked in.
        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:
            rogerk
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: