[ZBX-5740] Incorrect discoveryids parameter in item, trigger and graph get methods Created: 2012 Oct 25 Updated: 2017 May 30 Resolved: 2012 Oct 30 |
|
| Status: | Closed |
| Project: | ZABBIX BUGS AND ISSUES |
| Component/s: | API (A) |
| Affects Version/s: | 2.1.0 |
| Fix Version/s: | 2.0.4rc1, 2.1.0 |
| Type: | Incident report | Priority: | Major |
| Reporter: | Pavels Jelisejevs (Inactive) | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | api, graph, item, trigger | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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. |
| Comments |
| Comment by Toms (Inactive) [ 2012 Oct 26 ] |
|
Fixed in dev. branch: svn://svn.zabbix.com/branches/dev/ZBX-5740 |
| Comment by Pavels Jelisejevs (Inactive) [ 2012 Oct 30 ] |
|
(1) Everywhere where you've changed API calls to prototype APIs, you also need to remove the array('flags' => ZBX_FLAG_DISCOVERY_CHILD) filter. tomtom RESOLVED in r31137 jelisejev CLOSED. |
| Comment by Pavels Jelisejevs (Inactive) [ 2012 Oct 30 ] |
|
(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. tomtom RESOLVED in r31137 jelisejev I've removed the second comment in r31141. CLOSED. |
| Comment by Pavels Jelisejevs (Inactive) [ 2012 Oct 30 ] |
|
TESTED. |
| Comment by Toms (Inactive) [ 2012 Oct 30 ] |
|
Fixed in 2.0.4rc1 r31149, 2.1.0 r31148 |
| Comment by richlv [ 2012 Oct 30 ] |
|
actual change revisions seem to be 31147 and 31146, 9 & 8 only affected the changelog |