From 0fb4ddad47558200423d08276dc1fd9d61133c7c Mon Sep 17 00:00:00 2001 From: Roberts Lataria Date: Mon, 4 Apr 2022 13:24:24 +0300 Subject: ..F....... [ZBX-20534] fixed popup layout for importing when horizontal scroll was not visible --- ui/app/views/popup.import.compare.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'ui/app') diff --git a/ui/app/views/popup.import.compare.php b/ui/app/views/popup.import.compare.php index fc6af676071..26a9323b376 100644 --- a/ui/app/views/popup.import.compare.php +++ b/ui/app/views/popup.import.compare.php @@ -170,13 +170,13 @@ else { 'script_inline' => trim($this->readJsFile('popup.import.compare.js.php')), 'body' => !$data['diff'] ? (new CTableInfo()) - ->setNoDataMessage(_('No changes.')) - ->toString() + ->setNoDataMessage(_('No changes.')) + ->toString() : (new CForm()) - ->addClass('import-compare') ->addVar('import_overlayid', $data['import_overlayid']) - ->addItem(drawToc($data['diff_toc'])) - ->addItem(drawDiff($data['diff'])) + ->addItem( + (new CDiv([drawToc($data['diff_toc']), drawDiff($data['diff'])]))->addClass('import-compare') + ) ->toString(), 'buttons' => $buttons, 'no_changes' => !$data['diff'] -- cgit v1.2.3 From 5a78ddd14cd18257c80dbfd4cf6b9f46e53d1467 Mon Sep 17 00:00:00 2001 From: Eliza Sekace Date: Wed, 11 May 2022 14:43:31 +0300 Subject: ..F....... [ZBX-20669] fixed focus on fields when clicking on labels --- ui/app/partials/configuration.filter.items.php | 6 ++-- ui/app/partials/configuration.host.edit.html.php | 5 ++-- ui/app/partials/monitoring.latest.filter.php | 9 +++--- ui/app/partials/scheduledreport.formgrid.html.php | 2 +- ui/app/views/administration.token.edit.php | 2 +- ui/app/views/administration.user.edit.php | 2 +- ui/app/views/popup.condition.common.php | 34 +++++++++++------------ ui/app/views/popup.media.php | 2 +- ui/app/views/slareport.list.php | 4 +-- 9 files changed, 33 insertions(+), 33 deletions(-) (limited to 'ui/app') diff --git a/ui/app/partials/configuration.filter.items.php b/ui/app/partials/configuration.filter.items.php index f3ce4cbd54c..848abcece3e 100644 --- a/ui/app/partials/configuration.filter.items.php +++ b/ui/app/partials/configuration.filter.items.php @@ -52,7 +52,7 @@ zbx_add_post_js("var filterTypeSwitcher". // First column. $filter_column_1 - ->addRow((new CLabel(_('Host groups'), 'filter_groupid_ms')), + ->addRow((new CLabel(_('Host groups'), 'filter_groupids__ms')), (new CMultiSelect([ 'name' => 'filter_groupids[]', 'object_name' => 'hostGroup', @@ -69,7 +69,7 @@ $filter_column_1 ] ]))->setWidth(ZBX_TEXTAREA_FILTER_SMALL_WIDTH) ) - ->addRow((new CLabel(($data['context'] === 'host') ? _('Hosts') : _('Templates'), 'filter_hostid_ms')), + ->addRow((new CLabel(($data['context'] === 'host') ? _('Hosts') : _('Templates'), 'filter_hostids__ms')), (new CMultiSelect([ 'name' => 'filter_hostids[]', 'object_name' => ($data['context'] === 'host') ? 'hosts' : 'templates', @@ -96,7 +96,7 @@ $filter_column_1 ); if ($data['filter_data']['hosts']) { - $filter_column_1->addRow(_('Value mapping'), + $filter_column_1->addRow(new CLabel(_('Value mapping'), 'filter_valuemapids__ms'), (new CMultiSelect([ 'name' => 'filter_valuemapids[]', 'object_name' => 'valuemap_names', diff --git a/ui/app/partials/configuration.host.edit.html.php b/ui/app/partials/configuration.host.edit.html.php index e5b4c511ce6..a303200ff3a 100644 --- a/ui/app/partials/configuration.host.edit.html.php +++ b/ui/app/partials/configuration.host.edit.html.php @@ -207,7 +207,7 @@ else { $host_tab ->addItem([ - new CLabel(_('Templates')), + new CLabel(_('Templates'), 'add_templates__ms'), (new CFormField( (count($templates_field_items) > 1) ? (new CDiv($templates_field_items))->addClass('linked-templates') @@ -375,6 +375,7 @@ foreach ($data['inventory_fields'] as $inventory_no => $inventory_field) { $input_field = (new CTextArea('host_inventory['.$field_name.']', $data['host']['inventory'][$field_name])) ->setWidth(ZBX_TEXTAREA_BIG_WIDTH); } + else { $input_field = (new CTextBox('host_inventory['.$field_name.']', $data['host']['inventory'][$field_name])) ->setWidth(ZBX_TEXTAREA_BIG_WIDTH) @@ -414,7 +415,7 @@ foreach ($data['inventory_fields'] as $inventory_no => $inventory_field) { } $inventory_tab->addItem([ - new CLabel($inventory_field['title']), + new CLabel($inventory_field['title'], 'host_inventory['.$field_name.']'), new CFormField([$input_field, $inventory_item]) ]); } diff --git a/ui/app/partials/monitoring.latest.filter.php b/ui/app/partials/monitoring.latest.filter.php index 2afa2fda165..3ecfb6399fd 100644 --- a/ui/app/partials/monitoring.latest.filter.php +++ b/ui/app/partials/monitoring.latest.filter.php @@ -29,7 +29,7 @@ $filter_view_data = array_key_exists('filter_view_data', $data) ? $data['filter_ $left_column = (new CFormGrid()) ->addClass(CFormGrid::ZBX_STYLE_FORM_GRID_LABEL_WIDTH_TRUE) ->addItem([ - new CLabel(_('Host groups'), 'groupids__ms'), + new CLabel(_('Host groups'), 'groupids_#{uniqid}_ms'), new CFormField( (new CMultiSelect([ 'name' => 'groupids[]', @@ -53,7 +53,7 @@ $left_column = (new CFormGrid()) ) ]) ->addItem([ - new CLabel(_('Hosts'), 'hostids__ms'), + new CLabel(_('Hosts'), 'hostids_#{uniqid}_ms'), new CFormField( (new CMultiSelect([ 'name' => 'hostids[]', @@ -169,7 +169,7 @@ $right_column = (new CFormGrid()) new CFormField($tag_format_line) ]) ->addItem([ - new CLabel(_('Tag display priority')), + new CLabel(_('Tag display priority'), 'tag_priority_#{uniqid}'), new CFormField( (new CTextBox('tag_priority', $data['tag_priority'])) ->setWidth(ZBX_TEXTAREA_FILTER_STANDARD_WIDTH) @@ -179,12 +179,11 @@ $right_column = (new CFormGrid()) ) ]) ->addItem([ - new CLabel(_('Show details')), + new CLabel(_('Show details'), 'show_details'), new CFormField([ (new CCheckBox('show_details')) ->setChecked($data['show_details'] == 1) ->setUncheckedValue(0) - ->removeId() ]) ]); diff --git a/ui/app/partials/scheduledreport.formgrid.html.php b/ui/app/partials/scheduledreport.formgrid.html.php index b4f1576085f..04bb1e18606 100644 --- a/ui/app/partials/scheduledreport.formgrid.html.php +++ b/ui/app/partials/scheduledreport.formgrid.html.php @@ -104,7 +104,7 @@ $form_grid ) ]) ->addItem([ - new CLabel(_('Start time')), + new CLabel(_('Start time'), 'hours'), new CFormField( (new CDiv([ (new CNumericBox('hours', $data['hours'], 2)) diff --git a/ui/app/views/administration.token.edit.php b/ui/app/views/administration.token.edit.php index 5a7605268a9..e083f4c1c2b 100644 --- a/ui/app/views/administration.token.edit.php +++ b/ui/app/views/administration.token.edit.php @@ -78,7 +78,7 @@ $token_from_list = (new CFormList()) ->setChecked($data['expires_state']) ->setUncheckedValue('0') ) - ->addRow((new CLabel(_('Expires at')))->setAsteriskMark(), + ->addRow((new CLabel(_('Expires at'),'expires_at'))->setAsteriskMark(), (new CDateSelector('expires_at', $data['expires_at'])) ->setDateFormat(ZBX_FULL_DATE_TIME) ->setPlaceholder(_('YYYY-MM-DD hh:mm:ss')) diff --git a/ui/app/views/administration.user.edit.php b/ui/app/views/administration.user.edit.php index 8ffffe7203d..1e466b8b4e2 100644 --- a/ui/app/views/administration.user.edit.php +++ b/ui/app/views/administration.user.edit.php @@ -409,7 +409,7 @@ if ($data['action'] === 'user.edit') { ); } else { - $permissions_form_list->addRow((new CLabel(_('Role')))->setAsteriskMark(), $role_multiselect); + $permissions_form_list->addRow((new CLabel(_('Role'), 'roleid_ms'))->setAsteriskMark(), $role_multiselect); } if ($data['roleid']) { diff --git a/ui/app/views/popup.condition.common.php b/ui/app/views/popup.condition.common.php index 1415ca943af..6b6c64c608c 100644 --- a/ui/app/views/popup.condition.common.php +++ b/ui/app/views/popup.condition.common.php @@ -69,7 +69,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), [$operator, new CVar('operator', CONDITION_OPERATOR_EQUAL)]) - ->addRow(_('Tag'), $new_condition_tag); + ->addRow(new CLabel(_('Tag'), 'tag'), $new_condition_tag); break; // New event host group form elements. @@ -98,7 +98,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Host groups'), $hostgroup_multiselect); + ->addRow(new CLabel(_('Host groups'), 'groupids__ms'), $hostgroup_multiselect); break; // Event tag pair form elements. @@ -115,9 +115,9 @@ switch ($data['type']) { $inline_js .= $new_condition_newtag->getPostJS(); $form_list - ->addRow(_('Old tag name'), $new_condition_oldtag) + ->addRow(new CLabel(_('Old tag name'), 'oldtag'), $new_condition_oldtag) ->addRow(_('Operator'), [$operator, new CVar('operator', CONDITION_OPERATOR_EQUAL)]) - ->addRow(_('New tag name'), $new_condition_newtag); + ->addRow(new CLabel(_('New tag name'), 'newtag'), $new_condition_newtag); break; // Old|New event tag value form elements. @@ -135,9 +135,9 @@ switch ($data['type']) { $inline_js .= $new_condition_value->getPostJS(); $form_list - ->addRow(_('Tag'), $new_condition_tag) + ->addRow(new CLabel(_('Tag'), 'tag'), $new_condition_tag) ->addRow(_('Operator'), $operator) - ->addRow(_('Value'), $new_condition_value); + ->addRow(new CLabel(_('Value'), 'value'), $new_condition_value); break; } break; @@ -205,7 +205,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Triggers'), $trigger_multiselect); + ->addRow( new CLabel(_('Triggers'), 'trigger_new_condition_ms'), $trigger_multiselect); break; // Trigger severity form elements. @@ -248,7 +248,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Hosts'), $host_multiselect); + ->addRow(new CLabel(_('Hosts'), 'host_new_condition_ms'), $host_multiselect); break; // Host group form elements. @@ -279,7 +279,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Host groups'), $hostgroup_multiselect); + ->addRow(new CLabel(_('Host groups'), 'hostgroup_new_condition_ms'), $hostgroup_multiselect); break; // Problem is suppressed form elements. @@ -304,7 +304,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow((new CLabel(_('Tag')))->setAsteriskMark(), $new_condition_value); + ->addRow((new CLabel(_('Tag'), 'value'))->setAsteriskMark(), $new_condition_value); break; // Tag value form elements. @@ -320,9 +320,9 @@ switch ($data['type']) { $inline_js .= $new_condition_value->getPostJS(); $form_list - ->addRow((new CLabel(_('Tag')))->setAsteriskMark(), $new_condition_value2) + ->addRow((new CLabel(_('Tag'), 'value2'))->setAsteriskMark(), $new_condition_value2) ->addRow(_('Operator'), $operator) - ->addRow(_('Value'), $new_condition_value); + ->addRow(new CLabel(_('Value'), 'value'), $new_condition_value); break; // Template form elements. @@ -354,7 +354,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Templates'), $template_multiselect); + ->addRow(new CLabel(_('Templates'), 'template_new_condition_ms'), $template_multiselect); break; // Time period form elements. @@ -465,7 +465,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Discovery rules'), $drule_multiselect); + ->addRow(new CLabel(_('Discovery rules'), 'drule_new_condition_ms'), $drule_multiselect); break; // Discovery status form elements. @@ -517,7 +517,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow(_('Proxy'), $proxy_multiselect); + ->addRow(new CLabel(_('Proxy'), 'proxy_new_condition_ms'), $proxy_multiselect); break; // Received value form elements. @@ -532,7 +532,7 @@ switch ($data['type']) { ->setFocusableElementId('label-operator') ->addOptions(CSelect::createOptionsFromArray($operators_by_condition[CONDITION_TYPE_DVALUE])) ) - ->addRow(_('Value'), $new_condition_value); + ->addRow(new CLabel(_('Value'), 'value'), $new_condition_value); break; // Service port form elements. @@ -598,7 +598,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow((new CLabel(_('Value')))->setAsteriskMark(), $new_condition_value); + ->addRow((new CLabel(_('Value'), 'value'))->setAsteriskMark(), $new_condition_value); break; // Event type form elements. diff --git a/ui/app/views/popup.media.php b/ui/app/views/popup.media.php index 081d5d8480b..9b1718ec71b 100644 --- a/ui/app/views/popup.media.php +++ b/ui/app/views/popup.media.php @@ -76,7 +76,7 @@ $media_form = (new CFormList(_('Media'))) 'mediatype_send_to' ) ->addRow( - (new CLabel(_('Send to'), 'mediatype_email_send_to'))->setAsteriskMark(), + (new CLabel(_('Send to'), 'sendto_emails[0]'))->setAsteriskMark(), $email_send_to_table, 'mediatype_email_send_to' ) diff --git a/ui/app/views/slareport.list.php b/ui/app/views/slareport.list.php index 48562cba159..0dc67498e31 100644 --- a/ui/app/views/slareport.list.php +++ b/ui/app/views/slareport.list.php @@ -37,7 +37,7 @@ $filter = (new CFilter()) (new CFormGrid()) ->addClass(CFormGrid::ZBX_STYLE_FORM_GRID_LABEL_WIDTH_TRUE) ->addItem([ - new CLabel(_('SLA'), 'filter_slaid'), + new CLabel(_('SLA'), 'filter_slaid_ms'), new CFormField( (new CMultiSelect([ 'name' => 'filter_slaid', @@ -57,7 +57,7 @@ $filter = (new CFilter()) ] ]))->setWidth(ZBX_TEXTAREA_FILTER_STANDARD_WIDTH) ), - new CLabel(_('Service'), 'filter_serviceid'), + new CLabel(_('Service'), 'filter_serviceid_ms'), new CFormField( (new CMultiSelect([ 'name' => 'filter_serviceid', -- cgit v1.2.3 From eca402a8a2810500d65fe2e0f0dd5ca321902787 Mon Sep 17 00:00:00 2001 From: Eliza Sekace Date: Wed, 11 May 2022 16:56:14 +0300 Subject: ..F....... [ZBX-20669] fixed focus on fields when clicking label => Administration->Users and Configuration->Hosts->Web --- ui/app/views/administration.user.list.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui/app') diff --git a/ui/app/views/administration.user.list.php b/ui/app/views/administration.user.list.php index 4920b58891d..76255e3d866 100644 --- a/ui/app/views/administration.user.list.php +++ b/ui/app/views/administration.user.list.php @@ -70,7 +70,7 @@ $widget = (new CWidget()) (new CFormList())->addRow(_('Last name'), (new CTextBox('filter_surname', $data['filter']['surname']))->setWidth(ZBX_TEXTAREA_FILTER_SMALL_WIDTH) ), - (new CFormList())->addRow((new CLabel(_('User roles'), 'filter_roles')), + (new CFormList())->addRow((new CLabel(_('User roles'), 'filter_roles__ms')), (new CMultiSelect([ 'name' => 'filter_roles[]', 'object_name' => 'roles', -- cgit v1.2.3 From bc9df199662682d789f5ba2b6eba8d9f2a8dd4da Mon Sep 17 00:00:00 2001 From: Miks Kronkalns Date: Thu, 12 May 2022 12:52:28 +0300 Subject: ..F....... [ZBX-20669] fixed coding style --- ui/app/partials/configuration.host.edit.html.php | 1 - ui/app/views/administration.token.edit.php | 2 +- ui/app/views/popup.condition.common.php | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) (limited to 'ui/app') diff --git a/ui/app/partials/configuration.host.edit.html.php b/ui/app/partials/configuration.host.edit.html.php index a303200ff3a..4f6ee2a1a30 100644 --- a/ui/app/partials/configuration.host.edit.html.php +++ b/ui/app/partials/configuration.host.edit.html.php @@ -375,7 +375,6 @@ foreach ($data['inventory_fields'] as $inventory_no => $inventory_field) { $input_field = (new CTextArea('host_inventory['.$field_name.']', $data['host']['inventory'][$field_name])) ->setWidth(ZBX_TEXTAREA_BIG_WIDTH); } - else { $input_field = (new CTextBox('host_inventory['.$field_name.']', $data['host']['inventory'][$field_name])) ->setWidth(ZBX_TEXTAREA_BIG_WIDTH) diff --git a/ui/app/views/administration.token.edit.php b/ui/app/views/administration.token.edit.php index e083f4c1c2b..b7762bd64b9 100644 --- a/ui/app/views/administration.token.edit.php +++ b/ui/app/views/administration.token.edit.php @@ -78,7 +78,7 @@ $token_from_list = (new CFormList()) ->setChecked($data['expires_state']) ->setUncheckedValue('0') ) - ->addRow((new CLabel(_('Expires at'),'expires_at'))->setAsteriskMark(), + ->addRow((new CLabel(_('Expires at'), 'expires_at'))->setAsteriskMark(), (new CDateSelector('expires_at', $data['expires_at'])) ->setDateFormat(ZBX_FULL_DATE_TIME) ->setPlaceholder(_('YYYY-MM-DD hh:mm:ss')) diff --git a/ui/app/views/popup.condition.common.php b/ui/app/views/popup.condition.common.php index 6b6c64c608c..1366017d2cf 100644 --- a/ui/app/views/popup.condition.common.php +++ b/ui/app/views/popup.condition.common.php @@ -205,7 +205,7 @@ switch ($data['type']) { $form_list ->addRow(_('Operator'), $operator) - ->addRow( new CLabel(_('Triggers'), 'trigger_new_condition_ms'), $trigger_multiselect); + ->addRow(new CLabel(_('Triggers'), 'trigger_new_condition_ms'), $trigger_multiselect); break; // Trigger severity form elements. -- cgit v1.2.3 From b026108b8e2d628633602f1fa892ff26cbb1f332 Mon Sep 17 00:00:00 2001 From: Eliza Sekace Date: Thu, 12 May 2022 14:44:26 +0300 Subject: ..F....... [ZBX-20669] fixed focus on additional fields when clicking label --- ui/app/views/administration.user.token.edit.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'ui/app') diff --git a/ui/app/views/administration.user.token.edit.php b/ui/app/views/administration.user.token.edit.php index 85ef8584a9a..2768b815f71 100644 --- a/ui/app/views/administration.user.token.edit.php +++ b/ui/app/views/administration.user.token.edit.php @@ -60,11 +60,12 @@ $token_from_list = (new CFormList()) ->setChecked($data['expires_state']) ->setUncheckedValue('0') ) - ->addRow((new CLabel(_('Expires at')))->setAsteriskMark(), + ->addRow((new CLabel(_('Expires at'), 'expires_at'))->setAsteriskMark(), (new CDateSelector('expires_at', $data['expires_at'])) ->setDateFormat(ZBX_FULL_DATE_TIME) ->setPlaceholder(_('YYYY-MM-DD hh:mm:ss')) - ->setAriaRequired(), + ->setAriaRequired() + ->setId('expires-at-row'), 'expires-at-row' ) ->addRow(_('Enabled'), -- cgit v1.2.3 From a3780619d8e3b7d2c259450b17b6d4ffe530fe48 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andrejs=20Gri=C5=A1=C4=8Denko?= Date: Fri, 13 May 2022 17:39:38 +0300 Subject: ..F....... [ZBX-20847] fixed inability to attach any host groups after removing already attached ones --- ui/app/partials/js/configuration.host.edit.html.js.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui/app') diff --git a/ui/app/partials/js/configuration.host.edit.html.js.php b/ui/app/partials/js/configuration.host.edit.html.js.php index c102e0334bb..942ac098292 100644 --- a/ui/app/partials/js/configuration.host.edit.html.js.php +++ b/ui/app/partials/js/configuration.host.edit.html.js.php @@ -148,7 +148,7 @@ $groups_ms.on('change', (e) => { $groups_ms.multiSelect('setDisabledEntries', - [... document.querySelectorAll('[name^="groups["]')].map((input) => input.value) + [... this.form.querySelectorAll('[name^="groups["]')].map((input) => input.value) ); }); }, -- cgit v1.2.3 From d597bab24ae8a522d6bd44d0b2f41c34bfb8f0f4 Mon Sep 17 00:00:00 2001 From: Eliza Sekace Date: Mon, 23 May 2022 11:27:21 +0300 Subject: ..F....... [ZBX-20669] fixed focus on fields when clicking on labels, removed unnecessary lines --- ui/app/partials/configuration.host.edit.html.php | 12 +++++++++++- ui/app/views/administration.user.token.edit.php | 2 -- ui/app/views/popup.media.php | 2 +- 3 files changed, 12 insertions(+), 4 deletions(-) (limited to 'ui/app') diff --git a/ui/app/partials/configuration.host.edit.html.php b/ui/app/partials/configuration.host.edit.html.php index 4f6ee2a1a30..a1a2f97df3d 100644 --- a/ui/app/partials/configuration.host.edit.html.php +++ b/ui/app/partials/configuration.host.edit.html.php @@ -207,7 +207,17 @@ else { $host_tab ->addItem([ - new CLabel(_('Templates'), 'add_templates__ms'), + new CLabel([ + _('Templates'), + $host_is_discovered + ? makeHelpIcon([ + (new CList([ + _('Templates linked by host discovery cannot be unlinked.'), + _('Use host prototype configuration form to remove automatically linked templates on upcoming discovery.') + ])) + ]) + : null + ], 'add_templates__ms'), (new CFormField( (count($templates_field_items) > 1) ? (new CDiv($templates_field_items))->addClass('linked-templates') diff --git a/ui/app/views/administration.user.token.edit.php b/ui/app/views/administration.user.token.edit.php index 2768b815f71..51682ff5314 100644 --- a/ui/app/views/administration.user.token.edit.php +++ b/ui/app/views/administration.user.token.edit.php @@ -65,8 +65,6 @@ $token_from_list = (new CFormList()) ->setDateFormat(ZBX_FULL_DATE_TIME) ->setPlaceholder(_('YYYY-MM-DD hh:mm:ss')) ->setAriaRequired() - ->setId('expires-at-row'), - 'expires-at-row' ) ->addRow(_('Enabled'), (new CCheckBox('status', ZBX_AUTH_TOKEN_ENABLED)) diff --git a/ui/app/views/popup.media.php b/ui/app/views/popup.media.php index 9b1718ec71b..081d5d8480b 100644 --- a/ui/app/views/popup.media.php +++ b/ui/app/views/popup.media.php @@ -76,7 +76,7 @@ $media_form = (new CFormList(_('Media'))) 'mediatype_send_to' ) ->addRow( - (new CLabel(_('Send to'), 'sendto_emails[0]'))->setAsteriskMark(), + (new CLabel(_('Send to'), 'mediatype_email_send_to'))->setAsteriskMark(), $email_send_to_table, 'mediatype_email_send_to' ) -- cgit v1.2.3 From efeaa89c3cc051e550c7eaa0709f362d21843ccc Mon Sep 17 00:00:00 2001 From: Dace Petra Date: Thu, 26 May 2022 14:33:55 +0300 Subject: ..F....... [ZBX-20965] fixed positioning of change indicator and added help hint for the Position label in Units block of item value widget settings (dpetra). --- ui/app/views/monitoring.widget.item.view.php | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'ui/app') diff --git a/ui/app/views/monitoring.widget.item.view.php b/ui/app/views/monitoring.widget.item.view.php index a172ee2fff4..bab325bb799 100644 --- a/ui/app/views/monitoring.widget.item.view.php +++ b/ui/app/views/monitoring.widget.item.view.php @@ -168,6 +168,13 @@ function drawValueCell(array $cell_data): array { $item_content_div->addItem($item_decimals_div); } + // Units AFTER value. + if (array_key_exists('units', $cell_data['parts']) && $cell_data['units_pos'] == WIDGET_ITEM_POS_AFTER) { + $item_content_div->addItem($units_div); + } + + $item_cell[] = $item_content_div; + if (array_key_exists('change_indicator', $cell_data['parts'])) { $change_data = $cell_data['parts']['change_indicator']; $item_change_div = (new CDiv())->addClass('change-indicator'); @@ -191,13 +198,6 @@ function drawValueCell(array $cell_data): array { $item_content_div->addItem($item_change_div); } - // Units AFTER value. - if (array_key_exists('units', $cell_data['parts']) && $cell_data['units_pos'] == WIDGET_ITEM_POS_AFTER) { - $item_content_div->addItem($units_div); - } - - $item_cell[] = $item_content_div; - // Units BELOW value. if (array_key_exists('units', $cell_data['parts']) && $cell_data['units_pos'] == WIDGET_ITEM_POS_BELOW) { $item_cell[] = $units_div; -- cgit v1.2.3 From f21f77257fc9cad1362bb6d6925db1b37448dd8f Mon Sep 17 00:00:00 2001 From: Alexander Vladishev Date: Fri, 27 May 2022 16:28:08 +0000 Subject: A.F....... [ZBX-20233] fixed handling of empty configuration parameters $ZBX_SERVER and $ZBX_SERVER_PORT the same as if they were not defined --- ui/app/partials/layout.htmlpage.header.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ui/app') diff --git a/ui/app/partials/layout.htmlpage.header.php b/ui/app/partials/layout.htmlpage.header.php index 621bd06563e..5a343ccc248 100644 --- a/ui/app/partials/layout.htmlpage.header.php +++ b/ui/app/partials/layout.htmlpage.header.php @@ -24,7 +24,7 @@ * @var array $data */ -global $DB, $ZBX_SERVER, $ZBX_SERVER_NAME, $ZBX_SERVER_PORT; +global $DB, $ZBX_SERVER_NAME; $theme = ZBX_DEFAULT_THEME; $scripts = $data['javascript']['files']; -- cgit v1.2.3 From 0d4dccde82a4c55574582bf6198b61bf53d5d008 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andrejs=20Gri=C5=A1=C4=8Denko?= Date: Fri, 10 Jun 2022 15:29:52 +0300 Subject: ..F....... [ZBX-20534] removed extra div --- ui/app/views/popup.import.compare.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ui/app') diff --git a/ui/app/views/popup.import.compare.php b/ui/app/views/popup.import.compare.php index 26a9323b376..afa8d937708 100644 --- a/ui/app/views/popup.import.compare.php +++ b/ui/app/views/popup.import.compare.php @@ -173,10 +173,10 @@ else { ->setNoDataMessage(_('No changes.')) ->toString() : (new CForm()) + ->addClass('import-compare') ->addVar('import_overlayid', $data['import_overlayid']) - ->addItem( - (new CDiv([drawToc($data['diff_toc']), drawDiff($data['diff'])]))->addClass('import-compare') - ) + ->addItem(drawToc($data['diff_toc'])) + ->addItem(drawDiff($data['diff'])) ->toString(), 'buttons' => $buttons, 'no_changes' => !$data['diff'] -- cgit v1.2.3 From d6c72838b6dfc9d69d072e6a6da8ba112d23f1d5 Mon Sep 17 00:00:00 2001 From: Miks Kronkalns Date: Fri, 17 Jun 2022 10:19:56 +0300 Subject: ..F....... [ZBX-21185] fixed pattern selector when item name contains square brackets Merge in ZBX/zabbix from feature/ZBX-21185-6.1 to master * commit 'b74c96209b321e9d1ff7addae49d62996a7a6242': ..F....... [ZBX-21185] fixed pattern selector when item name contains square brackets (cherry picked from commit d5c14d1351c9b616776fd8ff37a3e63fde9d92b7) --- ui/app/controllers/CControllerPopupGeneric.php | 2 +- ui/app/views/popup.generic.php | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'ui/app') diff --git a/ui/app/controllers/CControllerPopupGeneric.php b/ui/app/controllers/CControllerPopupGeneric.php index 04cc5ed6ee1..09afd19ec0b 100644 --- a/ui/app/controllers/CControllerPopupGeneric.php +++ b/ui/app/controllers/CControllerPopupGeneric.php @@ -902,7 +902,7 @@ class CControllerPopupGeneric extends CController { case 'items': foreach ($records as $itemid => $row) { - $records[$row['name']] = ['itemid' => $row['name']] + $row; + $records[$row['name']] = ['pattern' => $row['name']] + $row; unset($records[$itemid]); } break; diff --git a/ui/app/views/popup.generic.php b/ui/app/views/popup.generic.php index 017049ba943..76d2a9533af 100644 --- a/ui/app/views/popup.generic.php +++ b/ui/app/views/popup.generic.php @@ -455,14 +455,14 @@ switch ($data['popup_type']) { $table->addRow([ $data['multiselect'] - ? new CCheckBox('item['.$item[$options['srcfld1']].']', $item['itemid']) + ? new CCheckBox('item['.$item['itemid'].']', $item['pattern']) : null, (new CLink($item['name'], 'javascript:void(0);')) ->onClick('javascript: addValue('. json_encode($options['reference']).', '. - json_encode($item['itemid']).', '. + json_encode($item['pattern']).', '. $options['parentid']. - ');'.$js_action_onclick), + ');'.$js_action_onclick), (new CDiv($item['key_']))->addClass(ZBX_STYLE_WORDWRAP), item_type2str($item['type']), itemValueTypeString($item['value_type']), @@ -473,7 +473,7 @@ switch ($data['popup_type']) { ]); $item = [ - 'id' => $item['itemid'], + 'id' => $item['pattern'], 'itemid' => $item['itemid'], 'name' => $options['patternselect'] ? $item['name'] -- cgit v1.2.3