ZABBIX BUGS AND ISSUES

Incorrect discoveryids parameter in item, trigger and graph get methods

Details

  • Zabbix ID:
    NA

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

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: