ZABBIX BUGS AND ISSUES

Various messages at linking templates

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: 1.8.3
  • Fix Version/s: 2.0.0
  • Component/s: Frontend (F)
  • Labels:
  • Environment:
    Zabbix pre1.8.3, r13531.
  • Zabbix ID:
    Reviewed 2.0
  1. link.jpg
    27 kB
    2010 Jul 22 17:11
  2. new_trigger.jpg
    21 kB
    2012 Jan 08 12:32
  3. re-link.jpg
    24 kB
    2010 Jul 22 17:13
  4. unlink.jpg
    24 kB
    2010 Jul 22 17:05
  5. unlink-and-clear.jpg
    29 kB
    2010 Jul 22 17:08

Activity

Alexander Vladishev made changes -
Field Original Value New Value
Attachment link.jpg [ 13441 ]
Alexander Vladishev made changes -
Attachment unlink.jpg [ 13442 ]
Alexander Vladishev made changes -
Attachment re-link.jpg [ 13443 ]
Alexander Vladishev made changes -
Attachment unlink-and-clear.jpg [ 13444 ]
Alexander Vladishev made changes -
Attachment link.jpg [ 13441 ]
Alexander Vladishev made changes -
Attachment link.jpg [ 13445 ]
Alexander Vladishev made changes -
Attachment re-link.jpg [ 13443 ]
Alexander Vladishev made changes -
Attachment re-link.jpg [ 13446 ]
Alexei Vladishev made changes -
Labels trivial usability
Fix Version/s 2.0 [ 10402 ]
Zabbix ID NA Reviewed 2.0
Alexander Sorokin made changes -
Assignee Alexander Sorokin [ sanja1989 ]
Alexander Sorokin made changes -
Status Open [ 1 ] In Progress [ 3 ]
Hide
Alexander Sorokin added a comment -

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

Show
Alexander Sorokin added a comment - Fixed in development branch svn://svn.zabbix.com/branches/dev/ZBX-2737
Alexander Sorokin made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Fix Version/s 1.8.10 [ 10506 ]
Resolution Fixed [ 1 ]
Hide
Igor Danoshaites added a comment -

<Igor> Dev branch (changes for v1.8) tested, all is fine.

Need the same changes for trunk version.

Show
Igor Danoshaites added a comment - <Igor> Dev branch (changes for v1.8) tested, all is fine. Need the same changes for trunk version.
Alexander Vladishev made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Alexander Vladishev made changes -
Status Resolved [ 5 ] Tested [ 10002 ]
Hide
richlv added a comment - - edited

(7) changelog entry has not been added to the trunk changelog

<sanja> RESOLVED

Show
richlv added a comment - - edited (7) changelog entry has not been added to the trunk changelog <sanja> RESOLVED
richlv made changes -
Status Tested [ 10002 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Hide
Alexander Sorokin added a comment -

Fixed for trunk in development branch svn://svn.zabbix.com/branches/dev/ZBX-2737-trunk/

Show
Alexander Sorokin added a comment - Fixed for trunk in development branch svn://svn.zabbix.com/branches/dev/ZBX-2737-trunk/
Hide
Alexey Fukalov added a comment - - edited

(8) [GUI]
In many places messages looks like: "'Added new application "%2$s" to host "%1$s".'", i suggest change order of arguments. (yoda style? )

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (8) [GUI] In many places messages looks like: "'Added new application "%2$s" to host "%1$s".'", i suggest change order of arguments. (yoda style? ) <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(9) {GUI]
Class.capplication.php:635 Lets get host from db with one query and use api for this.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (9) {GUI] Class.capplication.php:635 Lets get host from db with one query and use api for this. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(10) [GUI]
class.cdiscoveryrule.php why message output is in "create" method if one already exist in "createReal"?
In "delete" same for all host with one query.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (10) [GUI] class.cdiscoveryrule.php why message output is in "create" method if one already exist in "createReal"? In "delete" same for all host with one query. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(11) [GUI]
class.cdrule.php:596 no point in the end.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (11) [GUI] class.cdrule.php:596 no point in the end. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(12) [GUI]
class.cgraph.php 2 tabs are used for indents.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (12) [GUI] class.cgraph.php 2 tabs are used for indents. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(13) [GUI]
class.cgraph.php same for one api request for all hosts, CHost->get have 'graphids' parameter.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (13) [GUI] class.cgraph.php same for one api request for all hosts, CHost->get have 'graphids' parameter. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(14) [GUI]
Everywhere where word "host" is added to message it should check if it's really a "host" not "template".

<sanja> RESOLVED by new messages syntax (19).

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (14) [GUI] Everywhere where word "host" is added to message it should check if it's really a "host" not "template". <sanja> RESOLVED by new messages syntax (19). <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(15) [GUI]
class.citem.php one api request for all hosts in "delete".

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (15) [GUI] class.citem.php one api request for all hosts in "delete". <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(16) [GUI]
class.citemprototype:814 there is "host" in message but item name is used as value.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (16) [GUI] class.citemprototype:814 there is "host" in message but item name is used as value. <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(17) [GUI]
class.cproxy.php:543 maybe there should be "proxy" in message instead of "host"?

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (17) [GUI] class.cproxy.php:543 maybe there should be "proxy" in message instead of "host"? <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexey Fukalov added a comment - - edited

(18) [GUI]
class.ctemplate.php all these get_host_by_* should be changed to API requests.

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (18) [GUI] class.ctemplate.php all these get_host_by_* should be changed to API requests. <sanja> RESOLVED <pavels> CLOSED.
Alexey Fukalov made changes -
Assignee Alexander Sorokin [ sanja1989 ] Alexei Fukalov [ vedmak ]
Alexey Fukalov made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Hide
Alexey Fukalov added a comment - - edited

(19) [GUI]
We decided that format for messages should be changed, so....

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexey Fukalov added a comment - - edited (19) [GUI] We decided that format for messages should be changed, so.... <sanja> RESOLVED <pavels> CLOSED.
Alexey Fukalov made changes -
Assignee Alexei Fukalov [ vedmak ] Alexander Sorokin [ sanja1989 ]
Alexei Vladishev made changes -
Fix Version/s 1.8.11 [ 10705 ]
Fix Version/s 1.8.10 [ 10506 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] In Progress [ 3 ]
Alexander Sorokin made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
Alexander Vladishev added a comment - - edited

(21) [API] Why here "API_OUTPUT_EXTEND"?
frontends/php/api/classes/class.ctrigger.php:1649
frontends/php/api/classes/class.capplication.php:583,614

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexander Vladishev added a comment - - edited (21) [API] Why here "API_OUTPUT_EXTEND"? frontends/php/api/classes/class.ctrigger.php:1649 frontends/php/api/classes/class.capplication.php:583,614 <sanja> RESOLVED <pavels> CLOSED.
Hide
Alexander Vladishev added a comment - - edited

(22) In all messages we should use "visible host name". I.e. a field "name" instead of "host"

<sanja> RESOLVED

<pavels> CLOSED.

Show
Alexander Vladishev added a comment - - edited (22) In all messages we should use "visible host name". I.e. a field "name" instead of "host" <sanja> RESOLVED <pavels> CLOSED.
Hide
richlv added a comment - - edited

(23) revisions 24458 and 24472 introduce incorrectly formatted sql queries. all sql queries should be reformatted in class.ctemplate.php

<sanja> RESOLVED

<pavels> CLOSED.

Show
richlv added a comment - - edited (23) revisions 24458 and 24472 introduce incorrectly formatted sql queries. all sql queries should be reformatted in class.ctemplate.php <sanja> RESOLVED <pavels> CLOSED.
Pavels Jelisejevs made changes -
Assignee Alexander Sorokin [ sanja1989 ] Pavels Jelisejevs [ jelisejev ]
Hide
Pavels Jelisejevs added a comment - - edited

(24) There are some places when detailed info messages don't appear at all, for instance, when updating an item prototype. Should they be added as well?

<sanja> RESOLVED

<pavels> CLOSED.

Show
Pavels Jelisejevs added a comment - - edited (24) There are some places when detailed info messages don't appear at all, for instance, when updating an item prototype. Should they be added as well? <sanja> RESOLVED <pavels> CLOSED.
Hide
Pavels Jelisejevs added a comment - - edited

(25) An error occurs when creating a trigger prototype:

mysql_real_escape_string() expects parameter 1 to be string, array given [include/db.inc.php:616]
Created: Trigger prototype "trigger2" on "".
reset() expects parameter 1 to be array, null given [api/classes/class.ctriggerprototype.php:1134]
Created: Trigger "trigger2" on "".

<sanja> RESOLVED

<richlv> modified lines in rev 24524 don't follow the coding guidelines

<sanja> RESOLVED

<pavels> CLOSED.

Show
Pavels Jelisejevs added a comment - - edited (25) An error occurs when creating a trigger prototype: mysql_real_escape_string() expects parameter 1 to be string, array given [include/db.inc.php:616] Created: Trigger prototype "trigger2" on "". reset() expects parameter 1 to be array, null given [api/classes/class.ctriggerprototype.php:1134] Created: Trigger "trigger2" on "". <sanja> RESOLVED <richlv> modified lines in rev 24524 don't follow the coding guidelines <sanja> RESOLVED <pavels> CLOSED.
Pavels Jelisejevs made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Assignee Pavels Jelisejevs [ jelisejev ] Alexander Sorokin [ sanja1989 ]
Hide
Pavels Jelisejevs added a comment - - edited

(26) Please review my changes in r24495.

<sanja> CLOSED

Show
Pavels Jelisejevs added a comment - - edited (26) Please review my changes in r24495. <sanja> CLOSED
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 1.8.11 [ 10705 ]
Resolution Fixed [ 1 ]
Pavels Jelisejevs made changes -
Assignee Alexander Sorokin [ sanja1989 ] Pavels Jelisejevs [ jelisejev ]
Hide
Pavels Jelisejevs added a comment - - edited

TESTED.

Before merging, please review my changes in r24539.

<sanja> CLOSED

Show
Pavels Jelisejevs added a comment - - edited TESTED. Before merging, please review my changes in r24539. <sanja> CLOSED
Pavels Jelisejevs made changes -
Status Resolved [ 5 ] Tested [ 10002 ]
Assignee Pavels Jelisejevs [ jelisejev ] Alexander Sorokin [ sanja1989 ]
Alexander Sorokin made changes -
Status Tested [ 10002 ] Closed [ 6 ]
Alexander Vladishev made changes -
Resolution Fixed [ 1 ]
Status Closed [ 6 ] Reopened [ 4 ]
Hide
Alexander Vladishev added a comment - - edited

(27) error displayed, when saving a new trigger.
https://support.zabbix.com/secure/attachment/17771/new_trigger.jpg

<sanja> RESOLVED

<Sasha> REOPENED Now for an every trigger we make a very difficult SQL statement:

SELECT DISTINCT h.hostid,h.name,f.triggerid FROM hosts h,functions f,items i WHERE EXISTS ( SELECT hh.hostid FROM hosts hh, hosts_groups hgg, rights r, users_groups ug WHERE hh.hostid=h.hostid AND hh.hostid=hgg.hostid AND r.id=hgg.groupid AND r.groupid=ug.usrgrpid AND ug.userid=3 AND r.permission>=2 AND NOT EXISTS( SELECT hggg.groupid FROM hosts_groups hggg, rights rr, users_groups gg WHERE hggg.hostid=hgg.hostid AND rr.id=hggg.groupid AND rr.groupid=gg.usrgrpid AND gg.userid=3 AND rr.permission<2)) AND (f.triggerid IN ('13022')) AND h.hostid=i.hostid AND f.itemid=i.itemid AND f.triggerid BETWEEN 000000000000000 AND 099999999999999 AND h.status IN (0,1) LIMIT 1 OFFSET 0

We can take a host name from a trigger expression.

<sanja> RESOLVED

<Sasha> REOPENED Now we actually double function implode_exp(). We can change SQL statement in function implode_exp() and get array of visible host names.

<sanja> RESOLVED

<Sasha> REOPENED

  • please review my changes in r24709.
    <sanja> CLOSED
  • frontends/php/api/classes/class.cusermacro.php:735,738 The message "Macro [%1$s:%2$s] already exists" doesn't correspond to the our specification
    <sanja> RESOLVED
    <Sasha> REOPENED
  • _() expects exactly 1 parameter, 3 given [api/classes/class.cusermacro.php:734]
  • _() expects exactly 1 parameter, 3 given [api/classes/class.cusermacro.php:737]
    <sanja> RESOLVED
    <Sasha> CLOSED
  • implode_exp should be return unique list of visible host names
    <sanja> RESOLVED
    <Sasha> REOPENED
  • better use array_uniq() function after the cycle
    <sanja> RESOLVED
    <Sasha> CLOSED
  • please use an "implode()" function instead "foreach ..."
    <sanja> RESOLVED
    <Sasha> REOPENED
  • better use zbx_objectValues() function
    -$hosts = array();
    -foreach ($dbTriggers[$trigger['triggerid']]['hosts'] as $host) { - $hosts[] = $host['name']; -}
    -
    +$hosts = zbx_objectValues($dbTriggers[$trigger['triggerid']]['hosts'], 'name');
    <sanja> RESOLVED
    <Sasha> CLOSED
  • better use single quotes in strings like ", "
    <sanja> RESOLVED
    <Sasha> CLOSED

<Sasha> TESTED
<Sasha> Merged to trunk. Rev. 24780

Show
Alexander Vladishev added a comment - - edited (27) error displayed, when saving a new trigger. https://support.zabbix.com/secure/attachment/17771/new_trigger.jpg <sanja> RESOLVED <Sasha> REOPENED Now for an every trigger we make a very difficult SQL statement: SELECT DISTINCT h.hostid,h.name,f.triggerid FROM hosts h,functions f,items i WHERE EXISTS ( SELECT hh.hostid FROM hosts hh, hosts_groups hgg, rights r, users_groups ug WHERE hh.hostid=h.hostid AND hh.hostid=hgg.hostid AND r.id=hgg.groupid AND r.groupid=ug.usrgrpid AND ug.userid=3 AND r.permission>=2 AND NOT EXISTS( SELECT hggg.groupid FROM hosts_groups hggg, rights rr, users_groups gg WHERE hggg.hostid=hgg.hostid AND rr.id=hggg.groupid AND rr.groupid=gg.usrgrpid AND gg.userid=3 AND rr.permission<2)) AND (f.triggerid IN ('13022')) AND h.hostid=i.hostid AND f.itemid=i.itemid AND f.triggerid BETWEEN 000000000000000 AND 099999999999999 AND h.status IN (0,1) LIMIT 1 OFFSET 0 We can take a host name from a trigger expression. <sanja> RESOLVED <Sasha> REOPENED Now we actually double function implode_exp(). We can change SQL statement in function implode_exp() and get array of visible host names. <sanja> RESOLVED <Sasha> REOPENED
  • please review my changes in r24709. <sanja> CLOSED
  • frontends/php/api/classes/class.cusermacro.php:735,738 The message "Macro [%1$s:%2$s] already exists" doesn't correspond to the our specification <sanja> RESOLVED <Sasha> REOPENED
  • _() expects exactly 1 parameter, 3 given [api/classes/class.cusermacro.php:734]
  • _() expects exactly 1 parameter, 3 given [api/classes/class.cusermacro.php:737] <sanja> RESOLVED <Sasha> CLOSED
  • implode_exp should be return unique list of visible host names <sanja> RESOLVED <Sasha> REOPENED
  • better use array_uniq() function after the cycle <sanja> RESOLVED <Sasha> CLOSED
  • please use an "implode()" function instead "foreach ..." <sanja> RESOLVED <Sasha> REOPENED
  • better use zbx_objectValues() function -$hosts = array(); -foreach ($dbTriggers[$trigger['triggerid']]['hosts'] as $host) { - $hosts[] = $host['name']; -} - +$hosts = zbx_objectValues($dbTriggers[$trigger['triggerid']]['hosts'], 'name'); <sanja> RESOLVED <Sasha> CLOSED
  • better use single quotes in strings like ", " <sanja> RESOLVED <Sasha> CLOSED
<Sasha> TESTED <Sasha> Merged to trunk. Rev. 24780
Alexander Vladishev made changes -
Attachment new_trigger.jpg [ 17771 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] In Progress [ 3 ]
Alexander Sorokin made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Alexander Vladishev made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] In Progress [ 3 ]
Alexander Sorokin made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Alexander Vladishev made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Alexander Vladishev made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
Alexander Vladishev added a comment - - edited

(28) Cannot delete template:
pg_query(): Query failed: ERROR: missing FROM-clause entry for table "h" LINE 1: ...CT t.triggerid,t.description,t.flags,t.expression,h.name as ... ^ [include/db.inc.php:430]
...
<sanja> It is alredy fixed in ZBX-4519

Show
Alexander Vladishev added a comment - - edited (28) Cannot delete template: pg_query(): Query failed: ERROR: missing FROM-clause entry for table "h" LINE 1: ...CT t.triggerid,t.description,t.flags,t.expression,h.name as ... ^ [include/db.inc.php:430] ... <sanja> It is alredy fixed in ZBX-4519
Hide
Pavels Jelisejevs added a comment - - edited

(29) When adding a new trigger:

reset() expects parameter 1 to be array, null given [api/classes/class.ctrigger.php:1620]
Created: Trigger "321" on "".
<Slava> This can fix this bug:
$hostName=explode(':',$trigger['expression']);
info(_s('Created: Trigger "%1$s" on "%2$s".', $trigger['description'], str_replace('{','',reset($hostName))));

<richlv> better cross-merge the revision that fixed this in r24776

<Sasha> CLOSED duplicate. See (27)

Show
Pavels Jelisejevs added a comment - - edited (29) When adding a new trigger: reset() expects parameter 1 to be array, null given [api/classes/class.ctrigger.php:1620] Created: Trigger "321" on "". <Slava> This can fix this bug: $hostName=explode(':',$trigger['expression']); info(_s('Created: Trigger "%1$s" on "%2$s".', $trigger['description'], str_replace('{','',reset($hostName)))); <richlv> better cross-merge the revision that fixed this in r24776 <Sasha> CLOSED duplicate. See (27)
Pavels Jelisejevs made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Alexander Sorokin made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Hide
richlv added a comment - - edited

(30) as discussed with zalex, string change that only changed [] to "" should also be reflected in all translations that had those strings translated. it should be made so that these translations are not lost (both msgid and msgstr must be properly changed). it should not be changed for languages that didn't have those strings translated.
if in doubt, consult translation maintainers : https://zabbix.org/wiki/Translating_Zabbix

<zalex> As I see this recommendation/requirement is totally ignored and at the moment (2012-05-13) is already not actual. All translated strings where [] have been changed to "" are lost.
CLOSED

Show
richlv added a comment - - edited (30) as discussed with zalex, string change that only changed [] to "" should also be reflected in all translations that had those strings translated. it should be made so that these translations are not lost (both msgid and msgstr must be properly changed). it should not be changed for languages that didn't have those strings translated. if in doubt, consult translation maintainers : https://zabbix.org/wiki/Translating_Zabbix <zalex> As I see this recommendation/requirement is totally ignored and at the moment (2012-05-13) is already not actual. All translated strings where [] have been changed to "" are lost. CLOSED
Hide
Oleksiy Zagorskyi added a comment -

Issue closed.

Show
Oleksiy Zagorskyi added a comment - Issue closed.
Oleksiy Zagorskyi made changes -
Status Resolved [ 5 ] Closed [ 6 ]
Assignee Alexander Sorokin [ sanja1989 ]

People

Vote (0)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: