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.
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 ;-)
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 ?
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).
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):
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);
}
}
- Version cible
0.8.0 supprimé
- Catégorie mis à Core
- Statut changé de Nouveau à In Progress
- Assigné à mis à Johan Cwiklinski
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 ?
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.
- Statut changé de In Progress à Résolu
- % réalisé changé de 0 à 100
- Statut changé de Résolu à Fermé
Formats disponibles : Atom
PDF