[ZBX-4425] When update item interface is set to null Created: 2011 Dec 08 Updated: 2017 May 30 Resolved: 2012 Apr 10 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | API (A), Frontend (F) |
Affects Version/s: | 1.9.8 (beta) |
Fix Version/s: | 1.9.9 (beta) |
Type: | Incident report | Priority: | Blocker |
Reporter: | Alexey Fukalov | Assignee: | Unassigned |
Resolution: | Fixed | Votes: | 0 |
Labels: | items, lld, trivial | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: | cannot_add_host.jpg | ||||||||
Issue Links: |
|
Comments |
Comment by richlv [ 2011 Dec 08 ] |
(1) i believe this is the 3rd time for the same bug... unittests <pavels> I've added a simple create/retrieve/delete unit test for the item API, plus a special test for this case ). RESOLVED. |
Comment by Pavels Jelisejevs (Inactive) [ 2011 Dec 13 ] |
RESOLVED. |
Comment by Alexey Fukalov [ 2011 Dec 14 ] |
(2) [GUI] <pavels> RESOLVED. <Vedmak> we decided to have next line in long "ifs" to be indented with two tabs. <pavels> RESOLVED. <Vedmak> CLOSED |
Comment by Pavels Jelisejevs (Inactive) [ 2011 Dec 16 ] |
Merged to trunk r24033. CLOSED. |
Comment by Alexander Vladishev [ 2011 Dec 20 ] |
When full-cloning a host there is a following error: https://support.zabbix.com/secure/attachment/17695/cannot_add_host.jpg <pavels> RESOLVED in /branches/dev/ <sasha> It do not agree! It's broken in r24033. In r23994 all work fine! <pavels> Item prototype cloning never supported interfaces, Prior to r24033 it, probably, just haven't shown any errors =) <pavels> CLOSED. |
Comment by Alexander Vladishev [ 2011 Dec 21 ] |
(1) frontends/php/api/classes/class.cdiscoveryrule.php:1080 In all places, except calculated item and trigger expressions, should be used 'Visible host name' <pavels> RESOLVED. <pavels> CLOSED. |
Comment by Alexander Vladishev [ 2011 Dec 21 ] |
(2) Please review my changes in r24136 <pavels> CLOSED. |
Comment by Alexander Vladishev [ 2011 Dec 21 ] |
Tested successfully! Please close point (1) before merge <pavels> There have been some conflicts when I merged the changes from trunk. Please review my commit r24156. <Sasha> RESOLVED I have corrected incorrectly merged changes. Please review them. <pavels> Thanks. CLOSED. |
Comment by Pavels Jelisejevs (Inactive) [ 2011 Dec 23 ] |
Merged the prototype interface fix to the trunk in r24203. CLOSED. |
Comment by richlv [ 2012 Apr 10 ] |
reopen to change labels |