Project

General

Profile

Bug #175

convert_real_interface_to_friendly_descr doesn't work for disabled interfaces

Added by Chris Buechler over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
11/22/2009
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.0
Affected Architecture:

Description

It needs to because some parts of the GUI use it to display interface info (system_gateways.php for one) where it currently shows a blank interface for disabled interfaces.

Associated revisions

Revision c795339e (diff)
Added by Ermal Luçi over 9 years ago

Resolves #175. Return description even for disabled interfaces. Teach the return_gateways_array() to return status for disabled interfaces if the caller requests it.

History

#1 Updated by Ermal Luçi over 9 years ago

  • Status changed from New to Rejected

This is not true please find the real cause of this.

#2 Updated by Chris Buechler over 9 years ago

  • Status changed from Rejected to New

It is absolutely true.

#3 Updated by Ermal Luçi over 9 years ago

As far as i can tell
[code]
$ifdescrs = get_configured_interface_with_descr(false, true);
foreach ($ifdescrs as $if => $ifname) {
if ($if $interface || $ifname $interface)
return $ifname;
}
[/code]

This is the code from that function so there is something else.

#4 Updated by Chris Buechler over 9 years ago

Yes, and get_configured_interface_with_descr(false,true) does return disabled interfaces as it should. I spent time digging through the functions adding debug logging seeing where it went wrong. The interface list works, but it later fails, and I'm not sure where. Config attached that replicates this, browse to system_gateways.php and see blank spot under "Interface" on the OPT1 route.

#5 Updated by Ermal Luçi over 9 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

#6 Updated by Chris Buechler over 9 years ago

fixed

Also available in: Atom PDF