BVal
  1. BVal
  2. BVAL-32

Annotations associated with composited contraints must reflect inherited groups

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1-incubating
    • Fix Version/s: 0.1-incubating
    • Component/s: jsr303
    • Labels:
      None

      Description

      When enforcing group inheritance in constraint composition, the actual annotations must also have their groups() value modified. To do so, a new annotation proxy must be created that substitutes the obtained one.

      One gotcha in the solution, I made the class AnnotationProxy serializable as those proxies can be part of ConstraintDescriptorImpl,

      1 more test passes.

        Activity

        Hide
        Matt Benson added a comment -

        closing issues associated with existing releases

        Show
        Matt Benson added a comment - closing issues associated with existing releases
        Hide
        Roman Stumm added a comment -

        merged

        Show
        Roman Stumm added a comment - merged
        Hide
        Carlos Vara added a comment -

        You are right, sorry.

        Deleted duplicate patch and uploaded the good one.

        Show
        Carlos Vara added a comment - You are right, sorry. Deleted duplicate patch and uploaded the good one.
        Hide
        Donald Woods added a comment -

        Carlos, the attached patch is identical to the one for BVAL-31, so guessing you need to reattach?

        Show
        Donald Woods added a comment - Carlos, the attached patch is identical to the one for BVAL-31 , so guessing you need to reattach?
        Hide
        Carlos Vara added a comment -

        Attached patch and 1 test.

        Show
        Carlos Vara added a comment - Attached patch and 1 test.

          People

          • Assignee:
            Roman Stumm
            Reporter:
            Carlos Vara
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development