fix(SQL): check if auto_increment and primary are rigth otherwis fix it

Author: J9rem

docs/users/fr/usage-avance.md

@@ -79,5 +79,13 @@ Par défaut la page "Modifier les paramètres utilisateurs" permet uniquement de
 Lorsqu'un utilisateur tente d'accéder à une page qui nécessite d'être connecté, c'est le formulaire de connexion par défaut qui s'affiche.
 **créez une page nommée PageLogin** permet de personnaliser cette page avec un message plus pédagogique et de bénéficier des options de connexions listées ci-dessus en utilisant le composant "Connexion"
 
+## Résoudre le souci de création de listes qui ne s'affichent pas
 
+### Description du problème
 
+ - après la création d'une liste, celle-ci ne s'affiche pas correctement dans la liste des listes (accessible avec [ce lien](?BazaR&vue=listes 'Liste des listes :ignore'))
+
+### Correction
+
+ 1. faire un `/update` sur votre wiki (accessible avec [ce lien](?GererMisesAJour/update 'Forcer la finalisation de la mise à jour :ignore'))
+ 2. vérifier si le problème est résolu. Sinon, suivre les [instructions réservées aux webmasters](./webmaster?id=r%c3%a9parer-la-structure-de-vos-bases-de-donn%c3%a9es)

docs/users/fr/webmaster.md

@@ -316,8 +316,21 @@ Il va falloir adapter quelques points et sauver ensuite
 'mysql_password' => 'motdepassedevotredb',
 'base_url' =>'mettreicivotrenomdedomaine/?',
 ```
-      
-      
+
+## Réparer la structure de vos bases de données
+
+_Lorsque la structure de vos bases de données n'est pas correcte, des soucis peuvent survenir en particuliers lors de la création ou la modification des listes._
+
+ 1. tenter de forcer la finalisation d'une mise à jour avec le handler `/update` (accessible avec [ce lien](?GererMisesAJour/update 'Forcer la finalisation de la mise à jour :ignore'))
+ 2. si ça ne fonctionne pas:
+    1. se rendre dans l'interface de gestion de base de données du serveur concerné (`phpmyadmin`)
+    2. ouvrir en même temps le fichier `setup/sql/create-tables.sql` depuis votre wiki ([fichier à télécharger](setup/sql/create-tables.sql ':ignore'))
+    3. vérifier dans la structure de chaque table de votre serveur (`phpmyadmin`) que chaque colonne est correctement définie.
+    4. puis, dans cette ordre, modifier la colonne qui doit être en `AUTOINCREMENT` pour avoir `A.I.` cochée. Normalement, ceci corrige les soucis d'index pour la table concernée et définie la colonne comme primaire.
+    5. puis, pour les index (`KEY` dans le fichier `create-tables.sql`), définir manuellement chaque index pour la table concernée
+    6. vérifier que l'affichage et la modification des fiches fonctionnes à nouveau
+
+**Important** : il est vivement conseillé de faire une sauvegarde de votre base de données avant de faire les manipulations.
 
 
 

handlers/UpdateHandler.php

@@ -111,6 +111,9 @@ public function run()
             }
             $output .= $this->fixDefaultCommentsAcls();
 
+            // check if SQL tables are well defined
+            $output .= $this->checkSQLTablesThenFixThem($dbService);
+
             // update user table to increase size of password
             if ($this->wiki->services->has(PasswordHasherFactory::class)) {
                 $passwordHasherFactory = $this->getService(PasswordHasherFactory::class);
@@ -380,4 +383,134 @@ private function calcFieldToString(): string
 
         return $output;
     }
+
+    /**
+     * @param DbService $dbService
+     * @return string $output
+     */
+    private function checkSQLTablesThenFixThem($dbService): string
+    {
+        $output = "ℹ️ Checking SQL table structure... ";
+        try {
+            foreach ([
+                ['pages','id','int(10) unsigned NOT NULL AUTO_INCREMENT'],
+                ['links','id','int(10) unsigned NOT NULL AUTO_INCREMENT'],
+                ['nature','bn_id_nature','int(10) UNSIGNED NOT NULL AUTO_INCREMENT'],
+                ['referrers','id','int(10) unsigned NOT NULL AUTO_INCREMENT'],
+                ['triples','id','int(10) unsigned NOT NULL AUTO_INCREMENT'],
+            ] as $data) {
+                $output .= $this->checkThenUpdateColumnAutoincrement($dbService, $data[0], $data[1], $data[2]);
+            }
+            foreach ([
+                ['pages','id',['id']],
+                ['links','id',['id']],
+                ['nature','bn_id_nature',['bn_id_nature']],
+                ['referrers','id',['id']],
+                ['triples','id',['id']],
+                ['users','name',['name']],
+                ['acls','page_tag',['page_tag','privilege']],
+                ['acls','privilege',['page_tag','privilege']],
+            ] as $data) {
+                $output .= $this->checkThenUpdateColumnPrimary($dbService, $data[0], $data[1], $data[2]);
+            }
+            $output .= "✅ All is right !<br/>";
+        } catch (\Throwable $th) {
+            if ($th->getCode() ===1) {
+                $output .= "{$th->getMessage()} <br/>";
+            } else {
+                $output .= "❌ Not checkcked because of error during tests : {$th->getMessage()}! <br/>";
+            }
+        }
+
+        return $output;
+    }
+
+    /**
+     * check if a column has auto_increment then try to update it
+     * @param $dbService
+     * @param string $tableName
+     * @param string $columnName
+     * @param string $SQL_columnDef
+     * @return string $output
+     * @throws \Exception
+     */
+    private function checkThenUpdateColumnAutoincrement($dbService, string $tableName, string $columnName, string $SQL_columnDef): string
+    {
+        $output = "";
+        $data = $this->getColumnInfo($dbService, $tableName, $columnName);
+        if (empty($data['Extra']) || (is_string($data['Extra']) && strstr($data['Extra'], 'auto_increment') === false)) {
+            $output .= "<br/>  Updating `$columnName` in `$tableName`... ";
+            $dbService->query("ALTER TABLE {$dbService->prefixTable($tableName)} MODIFY COLUMN `$columnName` $SQL_columnDef;");
+            $data = $this->getColumnInfo($dbService, $tableName, $columnName);
+            if (empty($data['Extra']) || (is_string($data['Extra']) && strstr($data['Extra'], 'auto_increment') === false)) {
+                throw new \Exception("❌ tables `$tableName`, column `$columnName` not updated !", 1);
+            }
+        }
+        return $output;
+    }
+
+    /**
+     * check if a column is primary then try to update it
+     * @param $dbService
+     * @param string $tableName
+     * @param string $columnName
+     * @param array $newKeys
+     * @return string $output
+     * @throws \Exception
+     */
+    private function checkThenUpdateColumnPrimary($dbService, string $tableName, string $columnName, array $newKeys): string
+    {
+        $output = "";
+        $data = $this->getColumnInfo($dbService, $tableName, $columnName);
+        if (empty($data['Key']) || $data['Key'] !== "PRI") {
+            $output .= "<br/>  Updating key for `$columnName` in `$tableName`... ";
+            $newKeysFormatted = implode(
+                ',',
+                array_map(
+                    function ($key) use ($dbService) {
+                        return "`{$dbService->escape($key)}`";
+                    },
+                    array_filter($newKeys)
+                )
+            );
+            if (!empty($newKeysFormatted)) {
+                $data = $this->getColumnInfo($dbService, $tableName, 'index');
+                if (!empty($data)) {
+                    $dbService->query("ALTER TABLE {$dbService->prefixTable($tableName)} DROP PRIMARY KEY;");
+                }
+                $dbService->query("ALTER TABLE {$dbService->prefixTable($tableName)} ADD PRIMARY KEY($newKeysFormatted);");
+            }
+            $data = $this->getColumnInfo($dbService, $tableName, $columnName);
+            if (empty($data['Key']) || $data['Key'] !== "PRI") {
+                throw new \Exception("❌ tables `$tableName`, column `$columnName` key not updated !", 1);
+            }
+        }
+        return $output;
+    }
+
+    /**
+     * get columnInfo from a table
+     * @param DbService $dbService
+     * @param string $tableName
+     * @param string $columnName
+     * @return array $data
+     * @throws \Exception
+     */
+    private function getColumnInfo($dbService, string $tableName, string $columnName): array
+    {
+        if ($columnName == 'index') {
+            $result = $dbService->query("SHOW INDEX FROM {$dbService->prefixTable($tableName)};");
+            if (@mysqli_num_rows($result) === 0) {
+                return [];
+            }
+        } else {
+            $result = $dbService->query("SHOW COLUMNS FROM {$dbService->prefixTable($tableName)} LIKE '$columnName';");
+            if (@mysqli_num_rows($result) === 0) {
+                throw new \Exception("❌ tables `$tableName` not verified because error while getting `$columnName` column !", 1);
+            }
+        }
+        $data = mysqli_fetch_assoc($result);
+        mysqli_free_result($result);
+        return $data;
+    }
 }