Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: lang.*
    • Labels:
      None

      Description

      Methods for consideration:

      • isInstanceOf(Class<?> type, Object o);
      • isAssignable(Class<?> superType, Class<?> subType);
      • state(boolean condition);
      1. LANG_559.diff
        8 kB
        Valentin Rocher

        Issue Links

          Activity

          Mark Thomas made changes -
          Workflow jira [ 12483404 ] Default workflow, editable Closed status [ 12602445 ]
          Henri Yandell committed 918367 (1 file)
          Reviews: none

          Adding info on the LANG-559 commits

          Henri Yandell committed 918366 (2 files)
          Reviews: none

          Applying Valentin Rocher's patch from LANG-559, adding isInstanceOf and isAssignableFrom methods.

          Henri Yandell made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Hide
          Henri Yandell added a comment -

          Patch applied with change of name to isAssignableFrom.

          Show
          Henri Yandell added a comment - Patch applied with change of name to isAssignableFrom.
          Hide
          Henri Yandell added a comment -

          Patch looks good. Does this match your thoughts Paul?

          Should it be isAssignableFrom instead of isAssignable?

          Show
          Henri Yandell added a comment - Patch looks good. Does this match your thoughts Paul? Should it be isAssignableFrom instead of isAssignable?
          Valentin Rocher made changes -
          Attachment LANG_559.diff [ 12434680 ]
          Hide
          Valentin Rocher added a comment -

          Attached patch with tests

          Show
          Valentin Rocher added a comment - Attached patch with tests
          Hide
          Henri Yandell added a comment -

          Go with IllegalArgumentException. It was the parent class of IllegalClassException.

          Show
          Henri Yandell added a comment - Go with IllegalArgumentException. It was the parent class of IllegalClassException.
          Hide
          Valentin Rocher added a comment -

          I have a problem with this one : I've done the isInstance and isAssignable, but I don't know which exception to throw...I was thinking about IllegalClassException, but it seems to have disappeared from LANG, so I think there's a good reason...Any help ?

          Show
          Valentin Rocher added a comment - I have a problem with this one : I've done the isInstance and isAssignable, but I don't know which exception to throw...I was thinking about IllegalClassException, but it seems to have disappeared from LANG, so I think there's a good reason...Any help ?
          Henri Yandell made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Henri Yandell made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          pbenedict committed 895334 (1 file)
          Henri Yandell made changes -
          Component/s lang.* [ 12313203 ]
          Paul Benedict made changes -
          Link This issue blocks LANG-558 [ LANG-558 ]
          Paul Benedict made changes -
          Link This issue is depended upon by LANG-558 [ LANG-558 ]
          Paul Benedict made changes -
          Field Original Value New Value
          Link This issue blocks LANG-558 [ LANG-558 ]
          Paul Benedict created issue -

            People

            • Assignee:
              Paul Benedict
              Reporter:
              Paul Benedict
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development