Update for OGSpy 3.3.8 + PHP8

Retrait Table Mod_user_config de ce mod #3

Author: darknoon29

.editorconfig

@@ -1,36 +1,36 @@
-# This file is for unifying the coding style for different editors and IDEs
-# editorconfig.org
-
-root = true
-
-[*]
-end_of_line = lf
-charset = utf-8
-trim_trailing_whitespace = true
-insert_final_newline = true
-
-# Spaces in coffee
-[**.coffee]
-indent_style = space
-indent_size = 2
-
-[**.js]
-indent_style = space
-indent_size = 2
-
-# Tabs in less
-[**.less]
-indent_style = tab
-indent_size = 2
-
-[**.css]
-indent_style = tab
-indent_size = 2
-
-[**.php]
-indent_style = space
-indent_size = 4
-
-[**.html]
-indent_style = tab
-indent_size = 2
+# This file is for unifying the coding style for different editors and IDEs
+# editorconfig.org
+
+root = true
+
+[*]
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+# Spaces in coffee
+[**.coffee]
+indent_style = space
+indent_size = 2
+
+[**.js]
+indent_style = space
+indent_size = 2
+
+# Tabs in less
+[**.less]
+indent_style = tab
+indent_size = 2
+
+[**.css]
+indent_style = tab
+indent_size = 2
+
+[**.php]
+indent_style = space
+indent_size = 4
+
+[**.html]
+indent_style = tab
+indent_size = 2

admin.php

@@ -135,7 +135,7 @@
 echo help("attaques_Administration");
 echo "</font></b></legend>";
 // Formulaire des paramètres du module
-echo "<form name='form1' style=\"margin:0px;padding:0px; align-content: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
+echo "<form name='form1' style=\"margin:0px;padding:0px; alignment: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
 echo "<table width='60%' border='0'>
 <tr>
 <td class='c' colspan='2'>Paramètres du module</td>
@@ -179,7 +179,7 @@
 
 // Formulaire des BBcodes
 echo "<br />";
-echo "<form name='form2' style='margin:0px;padding:0px;align-content: center' action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
+echo "<form name='form2' style='margin:0px;padding:0px;alignment: center' action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
 echo "<table width='60%' border='0'>
 <tr>
   <td class='c' colspan='6'>BBCodes&nbsp;" . help("attaques_bbcolor") . "&nbsp;</td>
@@ -314,7 +314,7 @@
 if ($inval_id > 0) {
     //On serialize l'array pour le transmettre par $_POST dans le form
     $trans_id = json_encode($id);
-    echo "<form name='form4' style=\"margin:0;padding:0; align-content: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
+    echo "<form name='form4' style=\"margin:0;padding:0; alignment: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
     echo "<table width='60%' border='0'><tr>";
     echo "<th>Des données n'appartenant à aucun joueur actif ont été trouvées;
  dans la base de données</th>";
@@ -327,7 +327,7 @@
 }
 
 //Connexion Xtense2
-echo "<form name='form5' style=\"margin:0;padding:0; align-content: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
+echo "<form name='form5' style=\"margin:0;padding:0; alignment: center;\" action='index.php?action=attaques&page=admin' enctype='multipart/form-data' method='post'>";
 echo "<table width='60%' border='0'>
 <tr>
   <td class='c' colspan='5'>Xtense&nbsp;" . help("attaques_xtense") . "&nbsp;</td>

archives.php

@@ -69,15 +69,15 @@ function selectionner() {
     $nb_result = $db->sql_numrows($result);
 
     if ($nb_result == 0) {
-        $date_from = date("M Y", $date_from);
+        $date_from = strftime("%b %Y", $date_from);
 
         echo "<fieldset><legend><b><span style=\"color: #0080FF; \">Archives des attaques du mois de " . $date_from . "</span></b></legend>";
         echo "<span style=\"color: #FF0000; \">Vous n'avez pas de resultat pour cette date</span>";
         echo "</fieldset><br><br>";
     } else {
         list($archives_nb_attaques, $archives_date, $archives_metal, $archives_cristal, $archives_deut, $archives_pertes, $archives_recy_metal, $archives_recy_cristal, $archives_id) = $db->sql_fetch_row($resultgains);
 
-        $date_from = date("M Y", $date_from);
+        $date_from = strftime("%b %Y", $date_from);
 
         //On fait les calculs du total des gains, et la rentabilité, et du total des recyclages
         $total_gains = $archives_metal + $archives_cristal + $archives_deut;

attack_include.php

@@ -62,10 +62,14 @@ function mod_get_user_option($user_id, $param)
 
     $query = "SELECT `value` FROM `" . TABLE_MOD_USER_CFG . "` WHERE `mod`='Attaques' and `user_id`=" . $user_id . " and `config`='" . $param . "'";
     $result = $db->sql_query($query);
-    $user_config = $db->sql_fetch_row($result);
-    $user_value = $user_config['value'];
 
-    return $user_value;
+    if ($db->sql_numrows() > 0) {
+        $userConfig = $db->sql_fetch_row($result);
+        $userValue = $userConfig['value'];
+        return $userValue;
+    }
+
+    return false;
 }
 
 /**

attaques.php

@@ -25,8 +25,8 @@
 $septjours = $date - 7;
 $yesterday = $date - 1;
 
-if ($septjours < 1) $septjours = 1;
-if ($yesterday < 1) $yesterday = 1;
+$septjours = $septjours ?? 1;
+$yesterday = $yesterday ?? 1;
 
 
 //On verifie si il y a des attaques qui ne sont pas du mois actuel
@@ -59,8 +59,8 @@
 
         list($metal_recy, $cristal_recy) = $db->sql_fetch_row($resultrecy);
         // Sql Fetch Row can set values to null if no result
-        if ($metal_recy == null) $metal_recy = 0;
-        if ($cristal_recy == null) $cristal_recy = 0;
+        $metal_recy = $metal_recy ?? 0;
+        $cristal_recy = $cristal_recy ?? 0;
 
         //On sauvegarde les résultats
         $query = "INSERT INTO " . TABLE_ATTAQUES_ARCHIVES . " (`archives_user_id` , `archives_nb_attaques` , `archives_date` , `archives_metal` , `archives_cristal` , `archives_deut` , `archives_pertes`, `archives_recy_metal`, `archives_recy_cristal` )
@@ -80,7 +80,7 @@
         $bbcode .= "du 01/" . $month . "/" . $year . " au 31/" . $month . "/" . $year . "\n\n";
 
         while (list($attack_coord, $attack_date, $attack_metal, $attack_cristal, $attack_deut, $attack_pertes) = $db->sql_fetch_row($list)) {
-            $attack_date = date("d M Y H:i", $attack_date);
+            $attack_date = date('d M Y H:i:s', $attack_date);
             $bbcode .= "Le " . $attack_date . " victoire en " . $attack_coord . ".\n";
             $bbcode .= "[color=" . $bbcolor['m_g'] . "]" . $attack_metal . "[/color] de métal, [color=" . $bbcolor['c_g'] . "]" . $attack_cristal . "[/color] de cristal et [color=" . $bbcolor['d_g'] . "]" . $attack_deut . "[/color] de deutérium ont été rapportés.\n";
             $bbcode .= "Les pertes s'&eacute;lèvent à [color=" . $bbcolor['perte'] . "]" . $attack_pertes . "[/color].\n\n";
@@ -102,7 +102,7 @@
     //On ajoute l'action dans le log
     $line = "La listes des attaques de " . $user_data['user_name'] . " a &eacute;t&eacute; supprimée. Les gains ont été archivés dans le module de gestion des attaques";
     $fichier = "log_" . date("ymd") . '.log';
-    $line = "/*" . date("d/m/Y H:i:s") . '*/ ' . $line;
+    $line = "/*" . date("d m Y H:i:s") . '*/ ' . $line;
     write_file(PATH_LOG_TODAY . $fichier, "a", $line);
 
     exit;
@@ -147,8 +147,9 @@
 
 $result = $db->sql_query($query);
 $users = array();
-while ($row = $db->sql_fetch_row($result))
+while ($row = $db->sql_fetch_row($result)) {
     $users[$row[0]] = $row[1];
+}
 
 // Si un utilisateur a été sélectionné, on vérifie que l'on peut afficher les rapports de celui-ci
 if (isset($pub_user_id) && isset($users[$pub_user_id]))
@@ -228,8 +229,8 @@
 $resultgains = $db->sql_query($query);
 
 //On récupère la date au bon format
-$pub_date_from = date("d M Y H:i", $pub_date_from);
-$pub_date_to = date("d M Y H:i", $pub_date_to);
+$pub_date_from = date('d M Y', $pub_date_from);
+$pub_date_to = date('d M Y', $pub_date_to);
 
 
 //Création du field pour choisir l'affichage (attaque du jour, de la semaine ou du mois
@@ -276,6 +277,11 @@
 //Résultat requete
 list($attack_metal, $attack_cristal, $attack_deut, $attack_pertes) = $db->sql_fetch_row($resultgains);
 
+// Valeurs par défaut
+$attack_metal = $attack_metal ?? 0;
+$attack_cristal = $attack_cristal ?? 0;
+$attack_deut = $attack_deut ?? 0;
+$attack_pertes = $attack_pertes ?? 0;
 
 //Calcul des gains totaux
 $totalgains = $attack_metal + $attack_cristal + $attack_deut;
@@ -330,7 +336,7 @@
 echo "<tr>";
 
 while (list($attack_coord, $attack_date, $attack_metal, $attack_cristal, $attack_deut, $attack_pertes, $attack_id) = $db->sql_fetch_row($result)) {
-    $attack_date = date("d M Y H:i", $attack_date);
+    $attack_date = strftime("%d %b %Y %Hh%M", $attack_date);
     $attack_metal = number_format($attack_metal, 0, ',', ' ');
     $attack_cristal = number_format($attack_cristal, 0, ',', ' ');
     $attack_deut = number_format($attack_deut, 0, ',', ' ');

bbcode.php

@@ -65,8 +65,8 @@
 $nb_recy = $db->sql_numrows($recyclages);
 
 //On récupère la date au bon format
-$pub_date_from = date("d M Y", $pub_date_from);
-$pub_date_to = date("d M Y", $pub_date_to);
+$pub_date_from = date('d M Y', $pub_date_from);
+$pub_date_to = date('d M Y', $pub_date_to);
 
 //Création du field pour choisir l'affichage (attaque du jour, de la semaine ou du mois
 echo "<fieldset><legend><b><font color='#0080FF'>Date d'affichage des attaques et des recyclages en BBCode ";
@@ -97,15 +97,19 @@
 $bbcode .= "du " . $pub_date_from . " au " . $pub_date_to . "\n\n";
 
 //Résultat requete
-while (list($attack_coord, $attack_date, $attack_metal, $attack_cristal, $attack_deut, $attack_pertes) = $db->sql_fetch_row($attaques)) {
-    $attack_date = date("d M Y H:i", $attack_date);
-    $attack_metal = number_format($attack_metal, 0, ',', ' ');
-    $attack_cristal = number_format($attack_cristal, 0, ',', ' ');
-    $attack_deut = number_format($attack_deut, 0, ',', ' ');
-    $attack_pertes = number_format($attack_pertes, 0, ',', ' ');
-    $bbcode .= "Le " . $attack_date . " victoire en " . $attack_coord . ".\n";
-    $bbcode .= "[color=" . $bbcolor['m_g'] . "]" . $attack_metal . "[/color] de métal, [color=" . $bbcolor['c_g'] . "]" . $attack_cristal . "[/color] de cristal et [color=" . $bbcolor['d_g'] . "]" . $attack_deut . "[/color] de deuterium ont été rapportés.\n";
-    $bbcode .= "Les pertes s'élèvent à [color=" . $bbcolor['perte'] . "]" . $attack_pertes . "[/color].\n\n";
+if ($nb_attack > 0) {
+    while (list($attack_coord, $attack_date, $attack_metal, $attack_cristal, $attack_deut, $attack_pertes) = $db->sql_fetch_row($attaques)) {
+        $attack_date = date('d M Y H:i', $attack_date);
+        $attack_metal = number_format($attack_metal, 0, ',', ' ');
+        $attack_cristal = number_format($attack_cristal, 0, ',', ' ');
+        $attack_deut = number_format($attack_deut, 0, ',', ' ');
+        $attack_pertes = number_format($attack_pertes, 0, ',', ' ');
+        $bbcode .= "Le " . $attack_date . " victoire en " . $attack_coord . ".\n";
+        $bbcode .= "[color=" . $bbcolor['m_g'] . "]" . $attack_metal . "[/color] de métal, [color=" . $bbcolor['c_g'] . "]" . $attack_cristal . "[/color] de cristal et [color=" . $bbcolor['d_g'] . "]" . $attack_deut . "[/color] de deuterium ont été rapportés.\n";
+        $bbcode .= "Les pertes s'élèvent à [color=" . $bbcolor['perte'] . "]" . $attack_pertes . "[/color].\n\n";
+    }
+} else {
+    $bbcode = "Aucune Attaque enregistrée sur la période\n";
 }
 $bbcode .= "[url=http://www.ogsteam.eu/]Généré par OGSpy et le module de gestion des attaques[/url]";
 
@@ -124,12 +128,16 @@
 $bbcode .= "du " . $pub_date_from . " au " . $pub_date_to . "\n\n";
 
 //Résultat requete
-while (list($recy_coord, $recy_date, $recy_metal, $recy_cristal,) = $db->sql_fetch_row($recyclages)) {
-    $recy_date = date("d M Y H:i", $recy_date);
-    $recy_metal = number_format($recy_metal, 0, ',', ' ');
-    $recy_cristal = number_format($recy_cristal, 0, ',', ' ');
-    $bbcode .= "Le " . $recy_date . " recyclage en " . $recy_coord . ".\n";
-    $bbcode .= "[color=" . $bbcolor['m_r'] . "]" . $recy_metal . "[/color] de métal, [color=" . $bbcolor['c_r'] . "]" . $recy_cristal . "[/color] de cristal ont été recyclés.\n\n";
+if ($nb_recy > 0) {
+    while (list($recy_coord, $recy_date, $recy_metal, $recy_cristal,) = $db->sql_fetch_row($recyclages)) {
+        $recy_date = date('d M Y H:i', $recy_date);
+        $recy_metal = number_format($recy_metal, 0, ',', ' ');
+        $recy_cristal = number_format($recy_cristal, 0, ',', ' ');
+        $bbcode .= "Le " . $recy_date . " recyclage en " . $recy_coord . ".\n";
+        $bbcode .= "[color=" . $bbcolor['m_r'] . "]" . $recy_metal . "[/color] de métal, [color=" . $bbcolor['c_r'] . "]" . $recy_cristal . "[/color] de cristal ont été recyclés.\n\n";
+    }
+} else {
+    $bbcode = "Aucun Recyclage enregistré sur la période\n";
 }
 $bbcode .= "[url=http://www.ogsteam.eu/]Généré par OGSpy et le module de gestion des attaques[/url]";
 
@@ -148,6 +156,14 @@
 list($attack_metal, $attack_cristal, $attack_deut, $attack_pertes) = $db->sql_fetch_row($resultgains);
 list($recy_metal, $recy_cristal) = $db->sql_fetch_row($resultgains_recy);
 
+// Valeurs par Défaut si retour = null
+$attack_metal = $attack_metal ?? 0;
+$attack_cristal = $attack_cristal ?? 0;
+$attack_deut = $attack_deut ?? 0;
+$attack_pertes = $attack_pertes ?? 0;
+$recy_metal = $attack_cristal ?? 0;
+$recy_cristal = $attack_cristal ?? 0;
+
 //Calcul des gains totaux
 $totalgains = $attack_metal + $attack_cristal + $attack_deut + $recy_metal + $recy_cristal;
 
@@ -176,8 +192,12 @@
 $bbcode .= "Nombre de recyclages durant cette periode : " . $nb_recy . "\n\n";
 $bbcode .= "Metal recyclé : [color=" . $bbcolor['m_r'] . "]" . $recy_metal . "[/color]\n";
 $bbcode .= "Cristal recyclé : [color=" . $bbcolor['c_r'] . "]" . $recy_cristal . "[/color]\n\n";
-if ($renta > 0) $bbcode .= "Rentabilité : [color=" . $bbcolor['renta'] . "]" . $renta . "[/color]\n\n";
-else $bbcode .= "Rentabilité : [color=" . $bbcolor['perte'] . "]" . $renta . "[/color]\n\n";
+if ($renta > 0) {
+    $bbcode .= "Rentabilité : [color=" . $bbcolor['renta'] . "]" . $renta . "[/color]\n\n";
+}else {
+    $bbcode .= "Rentabilité : [color=" . $bbcolor['perte'] . "]" . $renta . "[/color]\n\n";
+}
+
 $bbcode .= "[url=http://www.ogsteam.eu/]Généré par OGSpy et le module de gestion des attaques[/url]";
 
 
@@ -191,4 +211,4 @@
 echo "<br>";
 echo "</fieldset>";
 echo "<br/>";
-?>
+?>

bilan.php

@@ -91,8 +91,8 @@
 
 
 //On récupère la date au bon format
-$pub_date_from = date("d M Y H:i", $pub_date_from);
-$pub_date_to = date("d M Y H:i", $pub_date_to);
+$pub_date_from = date('d M Y', $pub_date_from);
+$pub_date_to = date('d M Y', $pub_date_to);
 
 
 //Création du field pour choisir l'affichage (attaque du jour, de la semaine ou du mois
@@ -140,6 +140,14 @@
 //Résultat requete
 list($recy_metal, $recy_cristal) = $db->sql_fetch_row($resultgainsrecy);
 
+// Valeurs par Défaut si retour = null
+$attack_metal = $attack_metal ?? 0;
+$attack_cristal = $attack_cristal ?? 0;
+$attack_deut = $attack_deut ?? 0;
+$attack_pertes = $attack_pertes ?? 0;
+$recy_metal = $attack_cristal ?? 0;
+$recy_cristal = $attack_cristal ?? 0;
+
 //Calcul des gains totaux
 $totalgains = $attack_metal + $attack_cristal + $attack_deut;