Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: validation-1.0.0
    • Fix Version/s: validation-1.0.0
    • Component/s: validation
    • Labels:
      None

      Description

      ValidateMethodInterceptor should not be part of public API so make it package private.

        Activity

        Hide
        Hudson added a comment -

        Integrated in Onami-Validation #14 (See https://builds.apache.org/job/Onami-Validation/14/)
        ONAMI-80 Make interceptor package private and remove setter because it'll never be used (Revision 1444524)

        Result = FAILURE
        ash2k :
        Files :

        • /incubator/onami/sandbox/validation/src/main/java/org/apache/onami/validation/ValidateMethodInterceptor.java
        Show
        Hudson added a comment - Integrated in Onami-Validation #14 (See https://builds.apache.org/job/Onami-Validation/14/ ) ONAMI-80 Make interceptor package private and remove setter because it'll never be used (Revision 1444524) Result = FAILURE ash2k : Files : /incubator/onami/sandbox/validation/src/main/java/org/apache/onami/validation/ValidateMethodInterceptor.java
        Hide
        Mikhail Mazursky added a comment -

        Fixed in 1444524

        Show
        Mikhail Mazursky added a comment - Fixed in 1444524
        Hide
        Mikhail Mazursky added a comment -

        Also, setter method setValidatorFactory() is useless because it's not used and is out of reach in case ValidateMethodInterceptor class is not public. So it can be removed too.

        Show
        Mikhail Mazursky added a comment - Also, setter method setValidatorFactory() is useless because it's not used and is out of reach in case ValidateMethodInterceptor class is not public. So it can be removed too.

          People

          • Assignee:
            Mikhail Mazursky
            Reporter:
            Mikhail Mazursky
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development