135/netdisco
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

change double negative logic

Browse Source
This commit is contained in:
Oliver Gorwits
2019-10-30 19:50:04 +00:00
parent 441ac59661
commit 08969583cb
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/App/Netdisco/Util/Device.pm
View File

@@ -237,7 +237,7 @@ sub is_arpnipable {
my $device = get_device($ip) or return 0; my $device = get_device($ip) or return 0;
return _bail_msg("is_arpnipable: $device has no layer 3 capability") return _bail_msg("is_arpnipable: $device has no layer 3 capability")
unless ($device->has_layer(3) || !$device->in_storage()); if ($device->in_storage() and not $device->has_layer(3));
return _bail_msg("is_arpnipable: $device matched arpnip_no") return _bail_msg("is_arpnipable: $device matched arpnip_no")
if check_acl_no($device, 'arpnip_no'); if check_acl_no($device, 'arpnip_no');
@@ -288,8 +288,9 @@ Returns false if the host is not permitted to macsuck the target device.
sub is_macsuckable { sub is_macsuckable {
my $ip = shift; my $ip = shift;
my $device = get_device($ip) or return 0; my $device = get_device($ip) or return 0;
return _bail_msg("is_macsuckable: $device has no layer 2 capability") return _bail_msg("is_macsuckable: $device has no layer 2 capability")
unless ($device->has_layer(2) || !$device->in_storage()); if ($device->in_storage() and not $device->has_layer(2));
return _bail_msg("is_macsuckable: $device matched macsuck_no") return _bail_msg("is_macsuckable: $device matched macsuck_no")
if check_acl_no($device, 'macsuck_no'); if check_acl_no($device, 'macsuck_no');