[ZBX-10155] Trigger prototype without host:key references is saved in database Created: 2015 Dec 11 Updated: 2017 May 30 Resolved: 2016 Apr 04 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | Frontend (F) |
Affects Version/s: | 2.2.12rc1, 2.4.8rc1, 3.0.0alpha5 |
Fix Version/s: | 2.2.12rc1, 3.0.2rc1, 3.2.0alpha1 |
Type: | Incident report | Priority: | Trivial |
Reporter: | Glebs Ivanovskis (Inactive) | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | frontend, triggerprototypes | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Description |
Frontend allows user to successfully save trigger prototype with expression that does not contain any host:item references (for example "0=0"). Such trigger does not show up in frontend in connection to any host or template, obviously, but still can be seen in database (with expression "{}0=0"). In similar situation with triggers we show error message:
|
Comments |
Comment by vitalijs.cemeris (Inactive) [ 2015 Dec 16 ] |
RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-10155 |
Comment by Oleg Egorov (Inactive) [ 2016 Jan 27 ] |
(1) String changes? vitalijs.cemeris No translation strings changed |
Comment by Oleg Egorov (Inactive) [ 2016 Jan 28 ] |
TESTED for 2.2 But please close first sub-issue |
Comment by vitalijs.cemeris (Inactive) [ 2016 Jan 29 ] |
Fixed in:
sasha these changes are REVERTED in r58571, r58572, r58573 and r58574. See (4) for more details. |
Comment by vitalijs.cemeris (Inactive) [ 2016 Jan 29 ] |
(2) different fix for 2.4 and trunk should be provided gunarspujats CLOSED |
Comment by Oleg Egorov (Inactive) [ 2016 Jan 29 ] |
(3) 2.4 issue Result: vitalijs.cemeris RESOLVED in r58087 oleg.egorov CLOSED |
Comment by Oleg Egorov (Inactive) [ 2016 Feb 03 ] |
TESTED for 2.4 |
Comment by vitalijs.cemeris (Inactive) [ 2016 Feb 18 ] |
RESOLVED for 3.0 in r58512 oleg.egorov CLOSED |
Comment by Alexander Vladishev [ 2016 Feb 19 ] |
(4) I reverted all changes in stable branches because it must be fixed by other way. The method checkDiscoveryRuleCount() should be rewritten to prohibit such cases. vitalijs.cemeris RESOLVED for 2.2 oleg.egorov CLOSED sasha it is possible to create trigger prototype with lld-created item
REOPENED vitalijs.cemeris RESOLVED in r58967 oleg.egorov As discussed with iivs. Function getExpressionItems should be removed. And try to use API request instead SQL queries. vitalijs.cemeris RESOLVED for 2.2 sasha Another solution. Please take a look. Also added validation of permissions to hosts in the trigger expression. RESOLVED in r59178 (minor code improvements in r59179) gunarspujats CLOSED |
Comment by Gunars Pujats (Inactive) [ 2016 Mar 30 ] |
(5)
sasha Thanks! RESOLVED in r59189, r59190 and r59198. gunarspujats Minor code fix in r59211. sasha Thanks! CLOSED |
Comment by Gunars Pujats (Inactive) [ 2016 Mar 31 ] |
Tested, but close (5) before merging. |
Comment by Alexander Vladishev [ 2016 Apr 01 ] |
(6) Updated documentation: gunarspujats Updated documentation style. sasha Thanks! CLOSED |
Comment by Alexander Vladishev [ 2016 Apr 01 ] |
Fixed in:
|
Comment by Alexander Vladishev [ 2016 Apr 01 ] |
(7) Please review changes in 3.0 branch gunarspujats CLOSED |