Project

General

Profile

Anomalie #814

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

Added by Roland Telle over 7 years ago. Updated over 7 years ago.

Status:
Fermé
Priority:
Haut
Category:
Core
Target version:
-
Start date:
05/12/2014
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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 ;-)

#1

Updated by Johan Cwiklinski over 7 years ago

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 ?

#2

Updated by Roland Telle over 7 years ago

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).

#3

Updated by Roland Telle over 7 years ago

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):

#4

Updated by Johan Cwiklinski over 7 years ago

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);
         }
     }

#5

Updated by Johan Cwiklinski over 7 years ago

  • Target version deleted (0.8.0)
  • Category set to Core
  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski
#6

Updated by Johan Cwiklinski over 7 years ago

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

#7

Updated by Roland Telle over 7 years ago

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 ?

#8

Updated by Johan Cwiklinski over 7 years ago

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.

#9

Updated by Johan Cwiklinski over 7 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#10

Updated by Johan Cwiklinski over 7 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF