From f32eeec4b5d670f5e4d33fd7a9dc28419d576988 Mon Sep 17 00:00:00 2001
From: genuineparts <bernhard@becast.at>
Date: Mon, 23 Jun 2025 20:27:41 +0200
Subject: [PATCH] removing more php warnings

---
 admin/editusers.apnl.php |  2 +-
 admin/module.apnl.php    | 46 ++++++++++++++++++++--------------------
 inc/panel.class.php      |  2 +-
 3 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/admin/editusers.apnl.php b/admin/editusers.apnl.php
index a63091a..fad2dd5 100644
--- a/admin/editusers.apnl.php
+++ b/admin/editusers.apnl.php
@@ -90,7 +90,7 @@ class editusers_panel extends admin_module{
 					$panel->formClose();
 					$panel->parse_page();
 				}
-			}elseif($_GET["action"]=="delete" && $_GET["user"]){
+			}elseif(isset($_GET["action"]) && $_GET["action"]=="delete" && isset($_GET["user"])){
 				$result=$db->query("SELECT * FROM `" . $config["prefix"] . "users` WHERE `uid`='".intval($_GET["user"])."'");
 				$data = $db->fetch_array($result);
 				$plugin->run_hook('admin_user_before_delete',array('data'=>$data));
diff --git a/admin/module.apnl.php b/admin/module.apnl.php
index f19b3e3..1d2212a 100644
--- a/admin/module.apnl.php
+++ b/admin/module.apnl.php
@@ -1,4 +1,4 @@
-<?php
+<?php global $session;
 /**
  * Project:     BeCast WebEngine - simple site engine
  * File:        /admin/module.apnl.php
@@ -67,15 +67,15 @@ class module_panel extends admin_module{
 			$panel->content.="<h2>Installierte Module</h2><br />
 			<table width=\"90%\" cellspacing=\"2\">
 					 <tr>	
-					 <th align=\"center\">Modul</th>
-					 <th align=\"center\">Version</th>
-					 <th align=\"center\">Autor</th>
-					 <th align=\"center\">Homepage</th>
-					 <th align=\"center\">Aktion</th>
+					 <th style=\"text-align='center';\">Modul</th>
+					 <th style=\"text-align='center';\">Version</th>
+					 <th style=\"text-align='center';\">Autor</th>
+					 <th style=\"text-align='center';\">Homepage</th>
+					 <th style=\"text-align='center';\">Aktion</th>
 					 </tr>";
 			if(!empty($core->mod_)){
 				foreach($core->mod_ as $mod){
-					If($mod["depends"]!=""){
+					if(isset($mod["depends"]) && $mod["depends"]!=""){
 					
 						$depends="";
 						$dependands=unserialize($mod["depends"]);
@@ -89,11 +89,11 @@ class module_panel extends admin_module{
 						$depends.='<br /><span style="font-size: xx-small;">h&auml;ngt ab von: '.$deps;
 					}
 					$panel->content.="<tr>
-					<td align=\"center\">".$mod["name"].$depends."</td>
-					<td align=\"center\">".$mod["version"]."</td>
-					<td align=\"center\">".$mod["author"]."</td>
-					<td align=\"center\"><a href=\"".$mod["url"]."\">".$mod["url"]."</a></td>
-					<td align=\"center\"><a onclick=\"return confirmLink(this, 'Willst du wirklich das Modul ".$mod["name"]." deinstallieren? Alle Daten diese Plugins werden gel&ouml;scht!')\" href=\"".$config["path"]."/admin/index.php?panel=module&amp;action=uninstall&amp;module=".$mod["file"]."\">Deinstallieren</a></td>
+					<td style=\"text-align='center';\">".$mod["name"].$depends."</td>
+					<td style=\"text-align='center';\">".$mod["version"]."</td>
+					<td style=\"text-align='center';\">".$mod["author"]."</td>
+					<td style=\"text-align='center';\"><a href=\"".$mod["url"]."\">".$mod["url"]."</a></td>
+					<td style=\"text-align='center';\"><a onclick=\"return confirmLink(this, 'Willst du wirklich das Modul ".$mod["name"]." deinstallieren? Alle Daten diese Plugins werden gel&ouml;scht!')\" href=\"".$config["path"]."/admin/index.php?panel=module&amp;action=uninstall&amp;module=".$mod["file"]."\">Deinstallieren</a></td>
 					</tr>";
 					unset($deps);
 					unset($depends);
@@ -109,17 +109,17 @@ class module_panel extends admin_module{
 			$panel->content.="<h2>Nicht Installierte Module</h2><br />";
 			$panel->content.="<table width=\"90%\" cellspacing=\"2\">
 					  <tr>	
-					  <th align=\"center\">Modul</th>
-					  <th align=\"center\">Version</th>
-					  <th align=\"center\">Autor</th>
-					  <th align=\"center\">Homepage</th>
-					  <th align=\"center\">Aktion</th>
+					  <th style=\"text-align='center';\">Modul</th>
+					  <th style=\"text-align='center';\">Version</th>
+					  <th style=\"text-align='center';\">Autor</th>
+					  <th style=\"text-align='center';\">Homepage</th>
+					  <th style=\"text-align='center';\">Aktion</th>
 					  </tr>";
 
 			$modules=$this->avaiable_modules();
 			if(is_array($modules)){
 				foreach($modules as $mod){
-					If($mod["depends"]!=""){
+					if(isset($mod["depends"]) && $mod["depends"]!=""){
 					
 						$depends="";
 						$dependands=explode(",",$mod["depends"]);
@@ -133,11 +133,11 @@ class module_panel extends admin_module{
 						$depends.='<br /><span style="font-size: xx-small;">h&auml;ngt ab von: '.$deps;
 					}
 					$panel->content.="<tr>
-					<td align=\"center\">".$mod["name"].$depends."</td>
-					<td align=\"center\">".$mod["version"]."</td>
-					<td align=\"center\">".$mod["author"]."</td>
-					<td align=\"center\"><a href=\"".$mod["url"]."\">".$mod["url"]."</a></td>
-					<td align=\"center\"><a href=\"".$config["path"]."/admin/index.php?panel=module&amp;action=install&amp;module=".$mod["file"]."\">Installieren</a></td>
+					<td style=\"text-align='center';\">".$mod["name"].$depends."</td>
+					<td style=\"text-align='center';\">".$mod["version"]."</td>
+					<td style=\"text-align='center';\">".$mod["author"]."</td>
+					<td style=\"text-align='center';\"><a href=\"".$mod["url"]."\">".$mod["url"]."</a></td>
+					<td style=\"text-align='center';\"><a href=\"".$config["path"]."/admin/index.php?panel=module&amp;action=install&amp;module=".$mod["file"]."\">Installieren</a></td>
 					</tr>";
 					unset($deps);
 					unset($depends);
diff --git a/inc/panel.class.php b/inc/panel.class.php
index 5e934a5..77cdc16 100644
--- a/inc/panel.class.php
+++ b/inc/panel.class.php
@@ -203,7 +203,7 @@ class panel {
 	}
 	function submit ($args=array(),$extra="",$return=FALSE)
 	{
-		$name = $args["name"] ? $args["name"] : 'submit';
+		$name = $args["name"] ?? 'submit';
 		$fld = '<input type="submit" name="'.$name.'" ' .$extra. ' />';  // html: form submit button
 		if(!$return){
 			$this->content .= $fld;