[ZBX-9625] frontend allows to remove interfaces with items after failed update Created: 2015 Jun 09  Updated: 2017 May 30  Resolved: 2015 Sep 08

Status: Closed
Project: ZABBIX BUGS AND ISSUES
Component/s: Frontend (F)
Affects Version/s: 2.4.5
Fix Version/s: 2.4.7rc1, 3.0.0alpha2

Type: Incident report Priority: Minor
Reporter: richlv Assignee: Unassigned
Resolution: Fixed Votes: 0
Labels: interfaces
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

open a host that has an interface which can not be removed (because it has items).

enter an existing group name in the "New group" field, click Update - notice how that interface can be removed now. if host has some other interfaces left, clearing "New group" field and attempting to update again fails :

Interface is linked to item "Agent ping" on "Zabbix server".

frontend should not allow to remove such interfaces after a failed update (even if api catches that properly)



 Comments   
Comment by richlv [ 2015 Jun 09 ]

(1) the message is Interface is linked to item "Agent ping" on "Zabbix server". - shouldn't we use use item key, here not name ?

iivs We always write "name". Discussed with sasha. WON'T FIX.

Comment by Gunars Pujats (Inactive) [ 2015 Aug 12 ]

(2) No translation strings changed.

iivs CLOSED.

Comment by Gunars Pujats (Inactive) [ 2015 Aug 12 ]

RESOLVED in svn://svn.zabbix.com/branches/dev/ZBX-9625

Comment by Ivo Kurzemnieks [ 2015 Sep 01 ]

(3) I have a host with two agent interfaces, but one item. One interface is used by that item, so it is locked in place, the other is not and it is draggable. Using the same approach and intentionally failing the form, now both interfaces are locked, but second one can still be removed.

gunarspujats RESOLVED in r55353.

iivs CLOSED.

Comment by Ivo Kurzemnieks [ 2015 Sep 02 ]

(4) Coding style: please correct if statement in configuration.host.edit.js.php: 164 to

if (hostInterface.items > 0)

gunarspujats RESOLVED in r55353.

iivs CLOSED.

Comment by Ivo Kurzemnieks [ 2015 Sep 03 ]

TESTED.

Comment by Gunars Pujats (Inactive) [ 2015 Sep 03 ]

Fixed in:

  • pre-2.4.7 r55370
  • pre-3.0.0alpha2 (trunk) r55371
Generated at Fri Apr 26 23:10:17 EEST 2024 using Jira 9.12.4#9120004-sha1:625303b708afdb767e17cb2838290c41888e9ff0.