Projet

Général

Profil

Actions

Anomalie #814

fermé

Galette v0.8. Mise à jour de la base de données incomplète.

Ajouté par Roland Telle il y a presque 10 ans. Mis à jour il y a presque 10 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Catégorie:
Core
Version cible:
-
Début:
12/05/2014
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

Description

Installation en local sous Xampp 1.8.3 avec MySQL 5.6.16, PHP 5.5.11
Il s'agit de la mise à jour d'une version 0.7.8 vers la version 0.8

L'installation détecte correctement la version précédente. La table galette_logs est bien mise à jour mais le script affiche aussitôt "Impossible de lancer le script de mise à jour 0.8"

Accessoirement, corriger la coquille du message d'alerte "La base de données n'a pas été mises à jour !" en mettant "mises" au singulier ;-)

Mis à jour par Johan Cwiklinski il y a presque 10 ans

Il n'y a rien dans les logs d'apache et/ou de Galette ? La seule chose qui change pour le moment dans la base en 0.8, c'est la taille du champ ip_log de la table galette_log qui passe à 46 caractères ; est-ce que ça a été changé ou non ?

Mis à jour par Roland Telle il y a presque 10 ans

Johan Cwiklinski a écrit :

Il n'y a rien dans les logs d'apache et/ou de Galette ?

Rien nulle part.

La seule chose qui change pour le moment dans la base en 0.8, c'est la taille du champ ip_log de la table galette_log qui passe à 46 caractères ; est-ce que ça a été changé ou non ?

Je confirme ce que j'ai écrit dans mon courriel initial : La table galette_logs est bien mise à jour (et le champ ip_log est bien à 46 caractères).

Mis à jour par Roland Telle il y a presque 10 ans

Installation en local sous Xampp 1.8.3 avec MySQL 5.6.16, PHP 5.5.11
J'ai refait une installation neuve c'est-à-dire une 0.7.8 que j'ai mise à jour avec une 0.8 nightly après avoir écrasé les scripts de la 0.7.8 avec ceux de la 0.8.
ce qui donne la même erreur comme suit (avec une ligne qui liste les fichiers de data/logs):

Mis à jour par Johan Cwiklinski il y a presque 10 ans

Je n'ai pas le temps de tester ça en profondeur pour le moment, mais j'ai remarqué quelques petites coquilles du côté des codes de retour. Je pense que ça devrait corriger le problème :

diff --git a/galette/lib/Galette/Core/Install.php b/galette/lib/Galette/Core/Install.php
index e89557e..0d78a50 100644
--- a/galette/lib/Galette/Core/Install.php
+++ b/galette/lib/Galette/Core/Install.php
@@ -685,7 +685,8 @@ class Install
         }

         if ( $sql_query !== '' ) {
-            $this->executeSql($zdb, $sql_query);
+            $sql_res = $this->executeSql($zdb, $sql_query);
+            $fatal_error = !$sql_res;
         }
         return !$fatal_error;
     }
@@ -764,7 +765,7 @@ class Install
         }

         $this->_report = array_merge($this->_report, $queries_results);
-        return $fatal_error;
+        return !$fatal_error;
     }

     /**
diff --git a/galette/lib/Galette/Updater/AbstractUpdater.php b/galette/lib/Galette/Updater/AbstractUpdater.php
index a1ca3b5..4a76bab 100644
--- a/galette/lib/Galette/Updater/AbstractUpdater.php
+++ b/galette/lib/Galette/Updater/AbstractUpdater.php
@@ -188,7 +188,7 @@ abstract class AbstractUpdater
         ) . "\n";

         if ( $sql_query !== '' ) {
-            $installer->executeSql($zdb, $sql_query);
+            return $installer->executeSql($zdb, $sql_query);
         }
     }

Mis à jour par Johan Cwiklinski il y a presque 10 ans

  • Version cible 0.8.0 supprimé
  • Catégorie mis à Core
  • Statut changé de Nouveau à In Progress
  • Assigné à mis à Johan Cwiklinski

Mis à jour par Johan Cwiklinski il y a presque 10 ans

L'affichage du contenu du dossier est corrigé dans le commit f25e7521e77cedefff0a8d69cae35c6ff41284f4 :)

Mis à jour par Roland Telle il y a presque 10 ans

Johan Cwiklinski a écrit :

[...] Je pense que ça devrait corriger le problème : [...]

C'est p-a-r-f-a-i-t ! Merci ;-)

Y a-t-il lieu de procéder au test des plugins ou non ?

Mis à jour par Johan Cwiklinski il y a presque 10 ans

Roland Telle a écrit :

Johan Cwiklinski a écrit :

[...] Je pense que ça devrait corriger le problème : [...]

C'est p-a-r-f-a-i-t ! Merci ;-)

Super :)

Y a-t-il lieu de procéder au test des plugins ou non ?

Oui, l'accès à la base de données ayant été fortement modifié, les plugins sont impactés également.

Mis à jour par Johan Cwiklinski il y a presque 10 ans

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a presque 10 ans

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF