Merge pull request #2099 from f0o/issue-2098

Fix bug involving multiple ifAlias overrides on the same device
This commit is contained in:
Neil Lathwood
2015-10-10 14:54:02 +01:00
2 changed files with 3 additions and 3 deletions
+2 -2
View File
@@ -30,10 +30,10 @@ if (!empty($ifName) && is_numeric($port_id)) {
if (dbUpdate(array('ifAlias'=>$descr), 'ports', '`port_id`=?', array($port_id)) > 0) {
$device = device_by_id_cache($device_id);
if ($descr === 'repoll') {
del_dev_attrib($device, 'ifName');
del_dev_attrib($device, 'ifName:'.$ifName);
}
else {
set_dev_attrib($device, 'ifName', $ifName);
set_dev_attrib($device, 'ifName:'.$ifName, 1);
}
$status = 'ok';
}
+1 -1
View File
@@ -318,7 +318,7 @@ foreach ($ports as $port) {
foreach ($data_oids as $oid) {
if ($oid == 'ifAlias') {
if (get_dev_attrib($device, 'ifName', $port['ifName'])) {
if (get_dev_attrib($device, 'ifName:'.$port['ifName'], 1)) {
$this_port['ifAlias'] = $port['ifAlias'];
}
}