checkPortRole(): don't put "else" after "return"
authorDenis Ovsienko <denis@ovsienko.info>
Tue, 9 May 2017 08:28:16 +0000 (09:28 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Tue, 9 May 2017 08:28:52 +0000 (09:28 +0100)
wwwroot/inc/functions.php

index 42c9d9d18c675abd78ea0eacbb35abd5cd048d87..39d9f59e8a93672e0214f334cd0b7e9e6bccfa35 100644 (file)
@@ -6196,10 +6196,9 @@ function checkPortRole ($vswitch, $portinfo, $port_name, $port_order)
 
        if (! $remote_auto && ! $local_auto)
                return TRUE;
-       elseif ($remote_auto && $local_auto && $local_auto != $remote_auto && sameDomains ($vswitch['domain_id'], $remote_vswitch['domain_id']))
+       if ($remote_auto && $local_auto && $local_auto != $remote_auto && sameDomains ($vswitch['domain_id'], $remote_vswitch['domain_id']))
                return TRUE; // auto-calc link ends must belong to the same domain
-       else
-               return FALSE;
+       return FALSE;
 }
 
 # Convert InvalidArgException to InvalidRequestArgException with a choice of