Weld
  1. Weld
  2. WELD-902

CLONE - The qualified observer section is incorrect

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved (View Workflow)
    • Priority: Major Major
    • Resolution: Done
    • Affects Version/s: 1.1.1.Final
    • Fix Version/s: None
    • Component/s: Documentation
    • Labels:
      None
    • Affects:
      Documentation (Ref Guide, User Guide, etc.)
    • Estimated Difficulty:
      Low
    • Similar Issues:
      Show 10 results 

      Description

      The documentation at http://docs.jboss.org/weld/reference/1.0.0/en-US/html/events.html#d0e3980 is incorrect for the way qualified observers work. The related spec section is 10.2.

      The docs say all possible observers with any of the qualifiers will be called, which is incorrect. The full signature including all qualifiers must match for the observer to be invoked.

        Issue Links

          Activity

          Hide
          Christian Bauer
          added a comment -

          This was an incorrect change of the documentation, it is now wrong:

          "An observer method need not specify any event qualifiers—in this case it is interested in only unqualified events of a particular type."

          That's not the case (also see related issue of the clone source).

          http://seamframework.org/Community/EventObserversQualifiedOrNot

          Show
          Christian Bauer
          added a comment - This was an incorrect change of the documentation, it is now wrong: "An observer method need not specify any event qualifiers—in this case it is interested in only unqualified events of a particular type." That's not the case (also see related issue of the clone source). http://seamframework.org/Community/EventObserversQualifiedOrNot
          Hide
          Ales Justin
          added a comment -

          Jason, any update on this?

          Show
          Ales Justin
          added a comment - Jason, any update on this?
          Hide
          Jason Porter
          added a comment -

          Oh, I spoke to Pete about this a while ago. I think we have it correct, sorry I completely forgot this was assigned to me.

          Show
          Jason Porter
          added a comment - Oh, I spoke to Pete about this a while ago. I think we have it correct, sorry I completely forgot this was assigned to me.

            People

            • Assignee:
              Jason Porter
              Reporter:
              Christian Bauer
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: