Beehive
  1. Beehive
  2. BEEHIVE-758

checker element of ControlInterface annotation does not work

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: v1m1
    • Fix Version/s: v1m1
    • Component/s: Controls
    • Labels:
      None

      Description

      The ControlInterface annatation contains a 'checker' element and a 'checkerClass' element. The 'checkerClass' element was deprecated and replaced by the 'checker' element. However, while a controlchecker specified by the 'checker' element is never run at controls compile time.

      'checkerClass' works as expected.

      1. BEEHIVE-758.diff
        1.0 kB
        Chad Schoettger

        Activity

        Hide
        Chad Schoettger added a comment -

        Attached patch, first check for the 'checker' element if not set then check for the deprecated 'checkerClass' element.

        Show
        Chad Schoettger added a comment - Attached patch, first check for the 'checker' element if not set then check for the deprecated 'checkerClass' element.
        Hide
        Eddie O'Neil added a comment -

        Hm. This seems like something to fix before the pending release – this is an entire annotation value that doesn't work because only the deprecated "checkerClass" is wired up.

        Show
        Eddie O'Neil added a comment - Hm. This seems like something to fix before the pending release – this is an entire annotation value that doesn't work because only the deprecated "checkerClass" is wired up.
        Hide
        Kenneth Tam added a comment -

        Agree. The diff looks good, I'll get it checked in.

        Show
        Kenneth Tam added a comment - Agree. The diff looks good, I'll get it checked in.
        Hide
        Eddie O'Neil added a comment -

        Fixed with Ken's SVN 178253.

        Show
        Eddie O'Neil added a comment - Fixed with Ken's SVN 178253.
        Hide
        James Song added a comment -

        This is fixed.

        Show
        James Song added a comment - This is fixed.

          People

          • Assignee:
            James Song
            Reporter:
            Chad Schoettger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development