fix coding standard violations

This commit is contained in:
Christian Zeitnitz 2021-03-27 18:39:42 +01:00
parent f583a3eeae
commit 34f46007bd
4 changed files with 70 additions and 49 deletions

View file

@ -26,7 +26,7 @@ php << _EOF_
require_once("includes/config.php"); require_once("includes/config.php");
require_once("includes/get_clients.php"); require_once("includes/get_clients.php");
load_client_config(); loadClientConfig();
setClientState("$state"); setClientState("$state");
?> ?>
_EOF_ _EOF_

View file

@ -113,7 +113,7 @@ function DisplayDashboard(&$extraFooterScripts)
// ------------------------ INFOS ABOUT THE CLIENT--------------------------------------------------------------- // ------------------------ INFOS ABOUT THE CLIENT---------------------------------------------------------------
$clientinfo=array("name"=>"none","type"=>-1,"connected"=>"n"); $clientinfo=array("name"=>"none","type"=>-1,"connected"=>"n");
$raspi_client=$_SESSION['wifi_client_interface']; $raspi_client=$_SESSION['wifi_client_interface'];
load_client_config(); loadClientConfig();
$all_clients = getClients(false); $all_clients = getClients(false);
$clientinfo = array("name" => "none", "connected" => "n"); $clientinfo = array("name" => "none", "connected" => "n");
if ( ($idx = findCurrentClientIndex($all_clients)) >= 0) $clientinfo = $all_clients["device"][$idx]; if ( ($idx = findCurrentClientIndex($all_clients)) >= 0) $clientinfo = $all_clients["device"][$idx];

View file

@ -7,40 +7,44 @@ function getClients($simple=true)
exec('ifconfig -a | grep -oP "^(?!lo)(\w*)"', $rawdevs); // all devices except loopback exec('ifconfig -a | grep -oP "^(?!lo)(\w*)"', $rawdevs); // all devices except loopback
$path=RASPI_CLIENT_SCRIPT_PATH; $path=RASPI_CLIENT_SCRIPT_PATH;
$cl=array(); $cl=array();
if(!empty($rawdevs) && is_array($rawdevs)) { if (!empty($rawdevs) && is_array($rawdevs)) {
$cl["clients"]=count($rawdevs); $cl["clients"]=count($rawdevs);
// search for possibly not connected modem // search for possibly not connected modem
exec("find /sys/bus/usb/devices/usb*/ -name dev ", $devtty); // search for ttyUSB exec("find /sys/bus/usb/devices/usb*/ -name dev ", $devtty); // search for ttyUSB
$devtty = preg_only_match("/(ttyUSB0)/", $devtty); $devtty = preg_only_match("/(ttyUSB0)/", $devtty);
if(empty(preg_only_match("/(ppp)[0-9]/", $rawdevs))) { if (empty(preg_only_match("/(ppp)[0-9]/", $rawdevs))) {
if(!empty($devtty)) { if (!empty($devtty)) {
$rawdevs[]="ppp0"; $rawdevs[]="ppp0";
exec("udevadm info --name='$devtty' 2> /dev/null"); exec("udevadm info --name='$devtty' 2> /dev/null");
} }
} }
foreach ($rawdevs as $i => $dev) { foreach ($rawdevs as $i => $dev) {
$cl["device"][$i]["name"]=$dev; $cl["device"][$i]["name"]=$dev;
if (preg_match("/^(\w+)[0-9]$/", $dev, $nam) === 1) { $nam=$nam[1]; if (preg_match("/^(\w+)[0-9]$/", $dev, $nam) === 1) {
} else { $nam="none"; $nam=$nam[1];
} else {
$nam="none";
} }
if (($n = array_search($nam, $_SESSION["net-device-name-prefix"])) === false) { $n = count($_SESSION["net-device-types"])-1; if (($n = array_search($nam, $_SESSION["net-device-name-prefix"])) === false) {
$n = count($_SESSION["net-device-types"])-1;
} }
$ty = $_SESSION["net-device-types"][$n]; $ty = $_SESSION["net-device-types"][$n];
$cl["device"][$i]["type"]=$ty; $cl["device"][$i]["type"]=$ty;
unset($udevinfo); unset($udevinfo);
exec("udevadm info /sys/class/net/$dev 2> /dev/null", $udevinfo); exec("udevadm info /sys/class/net/$dev 2> /dev/null", $udevinfo);
if ($nam == "ppp" && isset($devtty)) { exec("udevadm info --name='$devtty' 2> /dev/null", $udevinfo); if ($nam == "ppp" && isset($devtty)) {
exec("udevadm info --name='$devtty' 2> /dev/null", $udevinfo);
} }
if(!empty($udevinfo) && is_array($udevinfo)) { if (!empty($udevinfo) && is_array($udevinfo)) {
$model = preg_only_match("/ID_MODEL_ENC=(.*)$/", $udevinfo); $model = preg_only_match("/ID_MODEL_ENC=(.*)$/", $udevinfo);
if(empty($model) || preg_match("/^[0-9a-f]{4}$/", $model) === 1) { if (empty($model) || preg_match("/^[0-9a-f]{4}$/", $model) === 1) {
$model = preg_only_match("/ID_MODEL_FROM_DATABASE=(.*)$/", $udevinfo); $model = preg_only_match("/ID_MODEL_FROM_DATABASE=(.*)$/", $udevinfo);
} }
if(empty($model)) { if (empty($model)) {
$model = preg_only_match("/ID_OUI_FROM_DATABASE=(.*)$/", $udevinfo); $model = preg_only_match("/ID_OUI_FROM_DATABASE=(.*)$/", $udevinfo);
} }
$vendor = preg_only_match("/ID_VENDOR_ENC=(.*)$/", $udevinfo); $vendor = preg_only_match("/ID_VENDOR_ENC=(.*)$/", $udevinfo);
if(empty($vendor) || preg_match("/^[0-9a-f]{4}$/", $vendor) === 1) { if (empty($vendor) || preg_match("/^[0-9a-f]{4}$/", $vendor) === 1) {
$vendor = preg_only_match("/ID_VENDOR_FROM_DATABASE=(.*)$/", $udevinfo); $vendor = preg_only_match("/ID_VENDOR_FROM_DATABASE=(.*)$/", $udevinfo);
} }
$driver = preg_only_match("/ID_NET_DRIVER=(.*)$/", $udevinfo); $driver = preg_only_match("/ID_NET_DRIVER=(.*)$/", $udevinfo);
@ -62,8 +66,10 @@ function getClients($simple=true)
case "eth": case "eth":
unset($res); unset($res);
exec("ip link show $dev 2> /dev/null | grep -oP ' UP '", $res); exec("ip link show $dev 2> /dev/null | grep -oP ' UP '", $res);
if(empty($res) && empty($ipadd)) { $cl["device"][$i]["connected"] = "n"; if (empty($res) && empty($ipadd)) {
} else { $cl["device"][$i]["connected"] = "y"; $cl["device"][$i]["connected"] = "n";
} else {
$cl["device"][$i]["connected"] = "y";
} }
break; break;
case "wlan": case "wlan":
@ -72,30 +78,35 @@ function getClients($simple=true)
$cl["device"][$i]["isAP"] = !empty($retiw); $cl["device"][$i]["isAP"] = !empty($retiw);
unset($retiw); unset($retiw);
exec("iw dev $dev link 2> /dev/null", $retiw); exec("iw dev $dev link 2> /dev/null", $retiw);
if(!$simple && !empty($ssid=preg_only_match("/.*SSID: ([\w ]*).*/", $retiw)) ) { if (!$simple && !empty($ssid=preg_only_match("/.*SSID: ([\w ]*).*/", $retiw)) ) {
$cl["device"][$i]["connected"] = "y"; $cl["device"][$i]["connected"] = "y";
$cl["device"][$i]["ssid"] = $ssid; $cl["device"][$i]["ssid"] = $ssid;
$cl["device"][$i]["ap-mac"] = preg_only_match("/^Connected to ([0-9a-f\:]*).*$/", $retiw); $cl["device"][$i]["ap-mac"] = preg_only_match("/^Connected to ([0-9a-f\:]*).*$/", $retiw);
$sig = preg_only_match("/.*signal: (.*)$/", $retiw); $sig = preg_only_match("/.*signal: (.*)$/", $retiw);
$val = preg_only_match("/^([0-9\.-]*).*$/", $sig); $val = preg_only_match("/^([0-9\.-]*).*$/", $sig);
if (!is_numeric($val)) { $val = -100; if (!is_numeric($val)) {
$val = -100;
} }
if($val >= -50 ) { $qual=100; if ($val >= -50 ) {
} else if($val < -100) { $qual=0; $qual=100;
} else { $qual=round($val*2+200); } else if ($val < -100) {
$qual=0;
} else {
$qual=round($val*2+200);
} }
$cl["device"][$i]["signal"] = "$sig (".$qual."%)"; $cl["device"][$i]["signal"] = "$sig (".$qual."%)";
$cl["device"][$i]["bitrate"] = preg_only_match("/.*bitrate: ([0-9\.]* \w*\/s).*$/", $retiw); $cl["device"][$i]["bitrate"] = preg_only_match("/.*bitrate: ([0-9\.]* \w*\/s).*$/", $retiw);
$cl["device"][$i]["freq"] = preg_only_match("/.*freq: (.*)$/", $retiw); $cl["device"][$i]["freq"] = preg_only_match("/.*freq: (.*)$/", $retiw);
$cl["device"][$i]["ap-mac"] = preg_only_match("/^Connected to ([0-9a-f\:]*).*$/", $retiw); $cl["device"][$i]["ap-mac"] = preg_only_match("/^Connected to ([0-9a-f\:]*).*$/", $retiw);
} else { $cl["device"][$i]["connected"] = "n"; } else {
$cl["device"][$i]["connected"] = "n";
} }
break; break;
case "ppp": case "ppp":
unset($res); unset($res);
exec("ip link show $dev 2> /dev/null | grep -oP '( UP | UNKNOWN)'", $res); exec("ip link show $dev 2> /dev/null | grep -oP '( UP | UNKNOWN)'", $res);
if($simple) { if ($simple) {
if(empty($res)) { if (empty($res)) {
$cl["device"][$i]["connected"] = "n"; $cl["device"][$i]["connected"] = "n";
$cl["device"][$i]["signal"] = "-100 dB (0%)"; $cl["device"][$i]["signal"] = "-100 dB (0%)";
} else { } else {
@ -104,15 +115,18 @@ function getClients($simple=true)
} }
break; break;
} }
if(empty($res) && empty($ipadd)) { $cl["device"][$i]["connected"] = "n"; if (empty($res) && empty($ipadd)) {
} else { $cl["device"][$i]["connected"] = "y"; $cl["device"][$i]["connected"] = "n";
} else {
$cl["device"][$i]["connected"] = "y";
} }
unset($res); unset($res);
exec("$path/info_huawei.sh mode modem", $res); exec("$path/info_huawei.sh mode modem", $res);
$cl["device"][$i]["mode"] = $res[0]; $cl["device"][$i]["mode"] = $res[0];
unset($res); unset($res);
exec("$path/info_huawei.sh device modem", $res); exec("$path/info_huawei.sh device modem", $res);
if($res[0] != "none" ) { $cl["device"][$i]["model"] = $res[0]; if ($res[0] != "none" ) {
$cl["device"][$i]["model"] = $res[0];
} }
unset($res); unset($res);
exec("$path/info_huawei.sh signal modem", $res); exec("$path/info_huawei.sh signal modem", $res);
@ -131,18 +145,18 @@ function getClients($simple=true)
$cl["device"][$i]["mode"] = $res[0]; $cl["device"][$i]["mode"] = $res[0];
unset($res); unset($res);
exec("$path/info_huawei.sh device hilink $apiadd", $res); exec("$path/info_huawei.sh device hilink $apiadd", $res);
if($res[0] != "none" ) { $cl["device"][$i]["model"] = $res[0]; if ($res[0] != "none" ) {
$cl["device"][$i]["model"] = $res[0];
} }
unset($res); unset($res);
exec("$path/info_huawei.sh signal hilink $apiadd", $res); exec("$path/info_huawei.sh signal hilink $apiadd", $res);
$cl["device"][$i]["signal"] = $res[0]; $cl["device"][$i]["signal"] = $res[0];
unset($ipadd); unset($ipadd);
exec("$path/info_huawei.sh ipaddress hilink $apiadd", $ipadd); exec("$path/info_huawei.sh ipaddress hilink $apiadd", $ipadd);
if(!empty($ipadd) && $ipadd[0] !== "none" ) { if (!empty($ipadd) && $ipadd[0] !== "none" ) {
$cl["device"][$i]["connected"] = "y"; $cl["device"][$i]["connected"] = "y";
$cl["device"][$i]["wan_ip"] = $ipadd[0]; $cl["device"][$i]["wan_ip"] = $ipadd[0];
} } else {
else {
$cl["device"][$i]["connected"] = "n"; $cl["device"][$i]["connected"] = "n";
$cl["device"][$i]["wan_ip"] = "-"; $cl["device"][$i]["wan_ip"] = "-";
} }
@ -159,18 +173,18 @@ function getClients($simple=true)
if (!isset($cl["device"][$i]["signal"])) { if (!isset($cl["device"][$i]["signal"])) {
$cl["device"][$i]["signal"]= $cl["device"][$i]["connected"] == "n" ? "-100 dB (0%)": "0 dB (100%)";; $cl["device"][$i]["signal"]= $cl["device"][$i]["connected"] == "n" ? "-100 dB (0%)": "0 dB (100%)";;
} }
if (!isset($cl["device"][$i]["isAP"])) { $cl["device"][$i]["isAP"]=false; if (!isset($cl["device"][$i]["isAP"])) {
$cl["device"][$i]["isAP"]=false;
} }
} }
} }
return $cl; return $cl;
} }
function load_client_config() function loadClientConfig()
{ {
// load network device config file for UDEV rules into $_SESSION // load network device config file for UDEV rules into $_SESSION
if(true) { if (!isset($_SESSION["udevrules"])) {
// if(!isset($_SESSION["udevrules"])) {
$_SESSION["net-device-types"]=array(); $_SESSION["net-device-types"]=array();
$_SESSION["net-device-name-prefix"]=array(); $_SESSION["net-device-name-prefix"]=array();
try { try {
@ -194,14 +208,14 @@ function load_client_config()
function findCurrentClientIndex($clients) function findCurrentClientIndex($clients)
{ {
$devid = -1; $devid = -1;
if(!empty($clients)) { if (!empty($clients)) {
$ncl=$clients["clients"]; $ncl=$clients["clients"];
if($ncl > 0) { if ($ncl > 0) {
$ty=-1; $ty=-1;
foreach($clients["device"] as $i => $dev) { foreach ($clients["device"] as $i => $dev) {
$id=array_search($dev["type"],$_SESSION["net-device-types"]); $id=array_search($dev["type"], $_SESSION["net-device-types"]);
if($id >=0 && $_SESSION["udevrules"]["network_devices"][$id]["clientid"] > $ty && !$dev["isAP"]) { if ($id >=0 && $_SESSION["udevrules"]["network_devices"][$id]["clientid"] > $ty && !$dev["isAP"]) {
$ty=$id; $ty=$id;
$devid=$i; $devid=$i;
} }
} }
@ -215,9 +229,10 @@ function waitClientConnected($dev, $timeout=10)
do { do {
exec('ifconfig -a | grep -i '.$dev.' -A 1 | grep -oP "(?<=inet )([0-9]{1,3}\.){3}[0-9]{1,3}"', $res); exec('ifconfig -a | grep -i '.$dev.' -A 1 | grep -oP "(?<=inet )([0-9]{1,3}\.){3}[0-9]{1,3}"', $res);
$connected= !empty($res); $connected= !empty($res);
if(!$connected) { sleep(1); if (!$connected) {
sleep(1);
} }
} while(!$connected && --$timeout > 0); } while (!$connected && --$timeout > 0);
return $connected; return $connected;
} }
@ -227,13 +242,16 @@ function setClientState($state)
if (($idx = findCurrentClientIndex($clients)) >= 0) { if (($idx = findCurrentClientIndex($clients)) >= 0) {
$dev = $clients["device"][$idx]; $dev = $clients["device"][$idx];
exec('ifconfig -a | grep -i '.$dev["name"].' -A 1 | grep -oP "(?<=inet )([0-9]{1,3}\.){3}[0-9]{1,3}"', $res); exec('ifconfig -a | grep -i '.$dev["name"].' -A 1 | grep -oP "(?<=inet )([0-9]{1,3}\.){3}[0-9]{1,3}"', $res);
if (!empty($res)) { $connected=$res[0]; if (!empty($res)) {
$connected=$res[0];
} }
switch($dev["type"]) { switch($dev["type"]) {
case "wlan": case "wlan":
if($state =="up") { exec('sudo ip link set '.$dev["name"].' up'); if ($state =="up") {
exec('sudo ip link set '.$dev["name"].' up');
} }
if(!empty($connected) && $state =="down") { exec('sudo ip link set '.$dev["name"].' down'); if (!empty($connected) && $state =="down") {
exec('sudo ip link set '.$dev["name"].' down');
} }
break; break;
case "hilink": case "hilink":
@ -248,15 +266,18 @@ function setClientState($state)
exec('sudo '.RASPI_CLIENT_SCRIPT_PATH.'/onoff_huawei_hilink.sh -c '.$mode.' -h '.$ipadd.' -p '.$pin); exec('sudo '.RASPI_CLIENT_SCRIPT_PATH.'/onoff_huawei_hilink.sh -c '.$mode.' -h '.$ipadd.' -p '.$pin);
break; break;
case "ppp": case "ppp":
if($state == "up") { exec('sudo ifup '.$dev["name"]); if ($state == "up") {
exec('sudo ifup '.$dev["name"]);
} }
if(!empty($connected) && $state == "down") { exec('sudo ifdown '.$dev["name"]); if (!empty($connected) && $state == "down") {
exec('sudo ifdown '.$dev["name"]);
} }
break; break;
default: default:
break; break;
} }
if($state=="up") { waitClientConnected($dev["name"], 15); if ($state=="up") {
waitClientConnected($dev["name"], 15);
} }
} }
} }

View file

@ -22,7 +22,7 @@ function DisplayNetworkingConfig()
foreach ($interfaces as $interface) { foreach ($interfaces as $interface) {
exec("ip a show $interface", $$interface); exec("ip a show $interface", $$interface);
} }
load_client_config(); loadClientConfig();
$clients=getClients(); $clients=getClients();
echo renderTemplate("networking", compact( echo renderTemplate("networking", compact(
"status", "status",