ZABBIX BUGS AND ISSUES
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-4833

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

    Details

      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

            • Assignee:
              Alexey Fukalov
              Reporter:
              Igor Danoshaites
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: