diff --git a/admin/addgroup.apnl.php b/admin/addgroup.apnl.php index c5948fb..fe8321f 100644 --- a/admin/addgroup.apnl.php +++ b/admin/addgroup.apnl.php @@ -1,4 +1,4 @@ -query("SELECT * FROM " . $config["prefix"] . "role_values"); - $panel->title="Gruppe anlegen"; - $panel->content='Hier kann man Benutzergruppen anlegen.'; - $panel->form(array("action"=>$config["path"]."/admin/index.php?panel=addgroup")); - $panel->content.="

Gruppename:

"; - $panel->field(array("name"=>"name","typ"=>"text","value"=>$data["name"])); - $panel->content.="

"; - $panel->content.=" - - - "; - while ($row = $db->fetch_array($result)){ - $panel->content.=""; - } - $panel->content.="
Rechte
".$row["text"].":
".$this->get_option($row["type"],$row["id"])."

"; - $panel->field(array("name"=>"send","typ"=>"hidden","value"=>"1")); - $panel->content.="
"; - $panel->submit(); - $panel->formClose(); + $result = $db->query("SELECT * FROM " . $config["prefix"] . "role_values"); + $panel->title="Gruppe anlegen"; + $panel->content='Hier kann man Benutzergruppen anlegen.'; + $panel->form(array("action"=>$config["path"]."/admin/index.php?panel=addgroup")); + $panel->content.="

Gruppename:

"; + $panel->field(array("name"=>"name","typ"=>"text","value"=>"")); + $panel->content.="

"; + $panel->content.=" + + + "; + while ($row = $db->fetch_array($result)){ + $panel->content.=""; + } + $panel->content.="
Rechte
".$row["text"].":
".$this->get_option($row["type"],$row["id"])."

"; + $panel->field(array("name"=>"send","typ"=>"hidden","value"=>"1")); + $panel->content.="
"; + $panel->submit(); + $panel->formClose(); } } diff --git a/admin/banips.apnl.php b/admin/banips.apnl.php index df3253c..35e4415 100644 --- a/admin/banips.apnl.php +++ b/admin/banips.apnl.php @@ -40,7 +40,7 @@ class banips_panel extends admin_module{ $db->query("INSERT INTO `" . $config["prefix"] . "banned_ips` (`ip`,`date`,`reason`) VALUES ('".$db->escape($_POST['ip'])."','".time()."','".$db->escape($_POST["reason"])."')"); $panel->admin_message($lang->_('SUCCESS'),$lang->_('DATASAVED'),True,"banips",3); } - }elseif($_GET["action"]=='delete' && $_GET['id']){ + }elseif(isset($_GET['action']) && $_GET["action"]=='delete' && $_GET['id']){ $db->query("DELETE FROM `" . $config["prefix"] . "banned_ips` WHERE `id`='".intval($_GET['id'])."'"); $panel->admin_message("Ban lifted","The ban was lifted.",True,"banips",3); }else{ diff --git a/admin/config.apnl.php b/admin/config.apnl.php index a944f6c..68494cb 100644 --- a/admin/config.apnl.php +++ b/admin/config.apnl.php @@ -121,7 +121,7 @@ class config_panel extends admin_module{ return $text; } - if(strstr($option,"wysiwyg")){ + if(str_contains($option, "wysiwyg")){ $string=str_replace("wysiwyg(","",substr($option, 0, -1)); $sarr=explode(";",$string); foreach($sarr as $str){ @@ -137,15 +137,15 @@ class config_panel extends admin_module{ return $text; } - if(strstr($option,"textarea")){ + if(str_contains($option, "textarea")){ $string=str_replace("textarea(","",substr($option, 0, -1)); $sarr=explode(";",$string); - foreach($sarr as $str){ - $substr=explode("|",$str); - $a_name[]=$substr[0]; - $value[]=$substr[1]; - } - if(is_array($sarr)){ + if(count($sarr)>1) { + foreach ($sarr as $str) { + $substr = explode("|", $str); + $a_name[] = $substr[0]; + $value[] = $substr[1]; + } $text=$panel->textarea(array("name"=>$name,"value"=>$config[$name], $a_name[0]=>$value[0],$a_name[1]=>$value[1]),"",TRUE); }else{ $text=$panel->textarea(array("name"=>$name,"value"=>$config[$name]),"",TRUE); @@ -153,7 +153,7 @@ class config_panel extends admin_module{ return $text; } - if(strstr($option,"select")){ + if(str_contains($option, "select")){ //$text=$panel->field("text",$name,"value=\"".$config[$name]."\"",TRUE); $string=str_replace("select(","",substr($option, 0, -1)); $sarr=explode(";",$string); diff --git a/admin/editgroup.apnl.php b/admin/editgroup.apnl.php index 0642d1a..4a6ef13 100644 --- a/admin/editgroup.apnl.php +++ b/admin/editgroup.apnl.php @@ -1,4 +1,4 @@ - Rechte "; - $result = $db->query("SELECT * FROM " . $config["prefix"] . "role_values"); + $drow = array(); + $result = $db->query("SELECT * FROM `" . $config["prefix"] . "role_values`"); while ($row = $db->fetch_array($result)){ - $rval =$db->fetch_array($db->query("SELECT value FROM `" . $config["prefix"] . "roleset` WHERE role_value_id='".$row["id"]."' AND role_id='".$data["id"]."'")); - $row["value"]=$rval["value"]; - $rdata[]=$row; - } - foreach ($rdata as $d){ - $panel->content.="".$d["text"].":
".$this->get_option($d["type"],$d["id"],$d["value"]).""; + $rval =$db->fetch_array($db->query("SELECT `value` FROM `" . $config["prefix"] . "roleset` WHERE role_value_id='".$row["id"]."' AND role_id='".$data["id"]."'")); + if(empty($rval["value"])){ + $rval["value"] = 0; + } + $panel->content.="".$row["text"].":
".$this->get_option($row["type"],$row["id"],$rval["value"]).""; } $panel->content.="
"; $panel->field(array("name"=>"send","typ"=>"hidden","value"=>"1")); @@ -83,7 +83,7 @@ class editgroup_panel extends admin_module{ $panel->parse_page(); } - }elseif($_GET["action"]=="delete" && $_GET["group"]){ + }elseif(isset($_GET["action"]) && $_GET["action"]=="delete" && isset($_GET["group"])){ $result=$db->query("SELECT id FROM `" . $config["prefix"] . "role` where `role_name`='User' LIMIT 1"); $row = $db->fetch_array($result); $db->query("UPDATE `" . $config["prefix"] . "users` SET `role`='".$row["id"]."' WHERE `role`='".intval($_GET["group"])."'"); diff --git a/inc/panel.class.php b/inc/panel.class.php index 77cdc16..a91f72e 100644 --- a/inc/panel.class.php +++ b/inc/panel.class.php @@ -26,7 +26,7 @@ */ $module["panel"]["name"]="Panel Class"; -$module["panel"]["ver"]="0.3.6"; +$module["panel"]["ver"]="0.3.7"; class panel { var string $sidebar=""; var string $page=""; @@ -156,7 +156,7 @@ class panel { function radio ($args=array(),$extra="",$return=FALSE) { - $name = $args["name"] ? $args["name"] : 'radiobutton'; + $name = $args["name"] ?? 'radiobutton'; $value = $args["value"]; if($value!=""){ $extra .= ' value="'.$value.'"'; @@ -191,9 +191,9 @@ class panel { function field ($args=array(),$extra="",$return=FALSE) { - $typ = $args["typ"] ? $args["typ"] : 'text'; - $name = $args["name"] ? $args["name"] : 'textfield'; - $value = $args["value"] ? $args["value"] : ''; + $typ = $args["typ"] ?? 'text'; + $name = $args["name"] ?? 'textfield'; + $value = $args["value"] ?? ''; $fld = ''; // html: form field if(!$return){ $this->content .= $fld;