From 9f10421ac66634dc28cfbb8ccbbebe2540e56261 Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Mon, 7 Oct 2019 16:15:01 +0200 Subject: [PATCH] scripts: PSR12 style fixes test drive Mostly spacing around = , + - etc with a dash of conditional formatting madness, but rules are rules. --- .../OPNsense/CaptivePortal/generate_certs.php | 4 ++-- .../scripts/interfaces/list_interfaces.php | 1 + src/opnsense/scripts/shell/setaddr.php | 15 ++++++++++----- 3 files changed, 13 insertions(+), 7 deletions(-) diff --git a/src/opnsense/scripts/OPNsense/CaptivePortal/generate_certs.php b/src/opnsense/scripts/OPNsense/CaptivePortal/generate_certs.php index 8cbb255be..0528f3f6d 100755 --- a/src/opnsense/scripts/OPNsense/CaptivePortal/generate_certs.php +++ b/src/opnsense/scripts/OPNsense/CaptivePortal/generate_certs.php @@ -62,7 +62,7 @@ if (isset($configObj->OPNsense->captiveportal->zones)) { $output_pem_filename = "/var/etc/cert-cp-zone" . $zone_id . ".pem"; file_put_contents($output_pem_filename, $pem_content); chmod($output_pem_filename, 0600); - echo "certificate generated " .$output_pem_filename . "\n"; + echo "certificate generated " . $output_pem_filename . "\n"; // generate ca pem file if (!empty($cert->caref)) { $output_pem_filename = "/var/etc/ca-cp-zone" . $zone_id . ".pem"; @@ -70,7 +70,7 @@ if (isset($configObj->OPNsense->captiveportal->zones)) { $ca = ca_chain($cert); file_put_contents($output_pem_filename, $ca); chmod($output_pem_filename, 0600); - echo "certificate generated " .$output_pem_filename ."\n"; + echo "certificate generated " . $output_pem_filename . "\n"; } } } diff --git a/src/opnsense/scripts/interfaces/list_interfaces.php b/src/opnsense/scripts/interfaces/list_interfaces.php index a4c184890..cec8f7d04 100755 --- a/src/opnsense/scripts/interfaces/list_interfaces.php +++ b/src/opnsense/scripts/interfaces/list_interfaces.php @@ -1,5 +1,6 @@ #!/usr/local/bin/php 1) { echo "Available interfaces:\n\n"; - $x=1; + $x = 1; foreach ($ifdescrs as $iface => $ifcfg) { $config_descr = get_interface_config_description($iface); echo "{$x} - {$ifcfg['descr']} ({$config_descr})\n"; @@ -293,12 +293,17 @@ function console_configure_ip_address($version) $upperifname = strtoupper($interface); - if ($interface != 'wan' && $version === 6 && !empty($config['interfaces']['wan']['ipaddrv6']) && - $config['interfaces']['wan']['ipaddrv6'] == 'dhcp6' && console_prompt_for_yn(sprintf( + if ( + $interface != 'wan' + && $version === 6 + && !empty($config['interfaces']['wan']['ipaddrv6']) + && $config['interfaces']['wan']['ipaddrv6'] == 'dhcp6' + && console_prompt_for_yn(sprintf( 'Configure %s address %s interface via WAN tracking?', $label_IPvX, $upperifname - ), 'y')) { + ), 'y') + ) { $intip = 'track6'; $intbits = '64'; $isintdhcp = true; @@ -442,7 +447,7 @@ if (!empty($nameserver6)) { if (count($nameservers)) { $config['system']['dnsserver'] = $nameservers; for ($dnscounter = 1; $dnscounter < 9; $dnscounter++) { - $dnsgwname= "dns{$dnscounter}gw"; + $dnsgwname = "dns{$dnscounter}gw"; if (isset($config['system'][$dnsgwname])) { unset($config['system'][$dnsgwname]); }