ZABBIX BUGS AND ISSUES

Screens are not cloned when performing full clone of the template.

Details

  • Zabbix ID:
    RTF

Description

Screens are not cloned when performing full clone of the template, but items, triggers, graphs and discovery rules are cloned.

Issue Links

Activity

Hide
richlv added a comment - - edited
Show
richlv added a comment - - edited (1) docs were added to http://www.zabbix.com/documentation/2.0/manual/appendix/api/templatescreen/copy , but still missing in http://www.zabbix.com/documentation/2.0/manual/appendix/api/changes_2.0 Alexey Fukalov RESOLVED <Eduard> OK, CLOSED
Hide
richlv added a comment - - edited

(2) there's something quite wrong with http://www.zabbix.com/documentation/2.0/manual/appendix/api/templatescreen - some links point to templatescreen, some to screen (and there are no corresponding pages for templatescreen). i'm afraid to touch it

Alexey Fukalov RESOLVED
<Eduard> CLOSED

Show
richlv added a comment - - edited (2) there's something quite wrong with http://www.zabbix.com/documentation/2.0/manual/appendix/api/templatescreen - some links point to templatescreen, some to screen (and there are no corresponding pages for templatescreen). i'm afraid to touch it Alexey Fukalov RESOLVED <Eduard> CLOSED
Hide
Eduards Samersovs added a comment - - edited

(3) Undefined index after full cloning just created template with 1 item, 1 graph, 1 screen with 1 graph screen item.
Undefined index: templateid [api/classes/CTemplateScreen.php:429]
Undefined index: templateid [api/classes/CTemplateScreen.php:431]

Alexey Fukalov RESOLVED
<Eduard> CLOSED

Show
Eduards Samersovs added a comment - - edited (3) Undefined index after full cloning just created template with 1 item, 1 graph, 1 screen with 1 graph screen item. Undefined index: templateid [api/classes/CTemplateScreen.php:429] Undefined index: templateid [api/classes/CTemplateScreen.php:431] Alexey Fukalov RESOLVED <Eduard> CLOSED
Hide
Eduards Samersovs added a comment - - edited

(4) Suggest not transform "*Ids" to array calling API. For example here:
$screensCopied = API::TemplateScreen()->copy(array(
'screenIds' => zbx_objectValues($screens, 'screenid'),
'templateIds' => array($templateid)
));

Alexey Fukalov RESOLVED
<Eduard> CLOSED

Show
Eduards Samersovs added a comment - - edited (4) Suggest not transform "*Ids" to array calling API. For example here: $screensCopied = API::TemplateScreen()->copy(array( 'screenIds' => zbx_objectValues($screens, 'screenid'), 'templateIds' => array($templateid) )); Alexey Fukalov RESOLVED <Eduard> CLOSED
Hide
Eduards Samersovs added a comment - - edited

(5) Suggest to use 'preservekeys' => true in CTemplateScreen->isReadable() and CTemplateScreen->isWritable()

Alexey Fukalov Output is set to COUNT there, so no need in preservekeys RESOLVED
<Eduard> Ok, CLOSED

Show
Eduards Samersovs added a comment - - edited (5) Suggest to use 'preservekeys' => true in CTemplateScreen->isReadable() and CTemplateScreen->isWritable() Alexey Fukalov Output is set to COUNT there, so no need in preservekeys RESOLVED <Eduard> Ok, CLOSED
Hide
Eduards Samersovs added a comment -

Tested!

Show
Eduards Samersovs added a comment - Tested!
Hide
Alexey Fukalov added a comment -

Fixed in 2.0.4rc1 r30554, pre-2.1.0 r30555.

Show
Alexey Fukalov added a comment - Fixed in 2.0.4rc1 r30554, pre-2.1.0 r30555.
Hide
Pavels Jelisejevs added a comment - - edited

(6) Please document the changes to the API in the 2.0 changelog.

Alexey Fukalov CLOSED

Show
Pavels Jelisejevs added a comment - - edited (6) Please document the changes to the API in the 2.0 changelog. Alexey Fukalov CLOSED
Hide
richlv added a comment - - edited

(7) this added a string that contains "Cannot change template for Screen", capitalising a word in the middle of the sentence.
we do not do that ( https://zabbix.org/wiki/Docs/specs/syntax#Syntax )

Alexey Fukalov Fixed in 2.0.4rc1 r30767, pre-2.1.0 r30768.

<richlv> CLOSED

Show
richlv added a comment - - edited (7) this added a string that contains "Cannot change template for Screen", capitalising a word in the middle of the sentence. we do not do that ( https://zabbix.org/wiki/Docs/specs/syntax#Syntax ) Alexey Fukalov Fixed in 2.0.4rc1 r30767, pre-2.1.0 r30768. <richlv> CLOSED

People

Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: