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

        Carlos Vara created issue -
        Hide
        Carlos Vara added a comment -

        Attached patch and 1 test.

        Show
        Carlos Vara added a comment - Attached patch and 1 test.
        Carlos Vara made changes -
        Field Original Value New Value
        Attachment bval-jsr303.patch [ 12443298 ]
        Donald Woods made changes -
        Assignee Donald Woods [ drwoods ]
        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?
        Donald Woods made changes -
        Assignee Donald Woods [ drwoods ]
        Carlos Vara made changes -
        Attachment bval-jsr303.patch [ 12443298 ]
        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.
        Carlos Vara made changes -
        Attachment bval-jsr303.patch [ 12443369 ]
        Roman Stumm made changes -
        Assignee Roman Stumm [ romanstumm ]
        Hide
        Roman Stumm added a comment -

        merged

        Show
        Roman Stumm added a comment - merged
        Roman Stumm made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Roman Stumm made changes -
        Fix Version/s 0.1-incubating [ 12314849 ]
        Hide
        Matt Benson added a comment -

        closing issues associated with existing releases

        Show
        Matt Benson added a comment - closing issues associated with existing releases
        Matt Benson made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 16h 9m 1 Roman Stumm 03/May/10 09:39
        Resolved Resolved Closed Closed
        704d 13h 49m 1 Matt Benson 06/Apr/12 23:29

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development