ZABBIX BUGS AND ISSUES
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-5740

Incorrect discoveryids parameter in item, trigger and graph get methods

    Details

      Description

      The item, trigger and graph APIs have a "discoveryids" parameter, which is meant for working with prototypes, not with plain objects. When using it with prototypes, it returns objects that belong to the given discovery rules, but with plain objects it works incorrectly. It should be removed from these APIs.

        Activity

        Hide
        Toms added a comment -

        Fixed in dev. branch: svn://svn.zabbix.com/branches/dev/ZBX-5740

        Show
        Toms added a comment - Fixed in dev. branch: svn://svn.zabbix.com/branches/dev/ZBX-5740
        Hide
        Pavels Jelisejevs added a comment - - edited

        (1) Everywhere where you've changed API calls to prototype APIs, you also need to remove the array('flags' => ZBX_FLAG_DISCOVERY_CHILD) filter.

        Toms RESOLVED in r31137

        Pavels Jelisejevs CLOSED.

        Show
        Pavels Jelisejevs added a comment - - edited (1) Everywhere where you've changed API calls to prototype APIs, you also need to remove the array('flags' => ZBX_FLAG_DISCOVERY_CHILD) filter. Toms RESOLVED in r31137 Pavels Jelisejevs CLOSED.
        Hide
        Pavels Jelisejevs added a comment - - edited

        (2) "// TODO: should be used API::ItemPrototype()->get(), but 'graphids' not implemented yet"

        Actually, that's OK, graph and trigger prototypes can contain both items and item prototypes, that's why we use the item API.

        Toms RESOLVED in r31137

        Pavels Jelisejevs I've removed the second comment in r31141. CLOSED.

        Show
        Pavels Jelisejevs added a comment - - edited (2) "// TODO: should be used API::ItemPrototype()->get(), but 'graphids' not implemented yet" Actually, that's OK, graph and trigger prototypes can contain both items and item prototypes, that's why we use the item API. Toms RESOLVED in r31137 Pavels Jelisejevs I've removed the second comment in r31141. CLOSED.
        Hide
        Pavels Jelisejevs added a comment -

        TESTED.

        Show
        Pavels Jelisejevs added a comment - TESTED.
        Hide
        Toms added a comment -

        Fixed in 2.0.4rc1 r31149, 2.1.0 r31148

        Show
        Toms added a comment - Fixed in 2.0.4rc1 r31149, 2.1.0 r31148
        Hide
        richlv added a comment -

        actual change revisions seem to be 31147 and 31146, 9 & 8 only affected the changelog

        Show
        richlv added a comment - actual change revisions seem to be 31147 and 31146, 9 & 8 only affected the changelog

          People

          • Assignee:
            Toms
            Reporter:
            Pavels Jelisejevs
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: