Anomalie #814
ferméGalette v0.8. Mise à jour de la base de données incomplète.
100%
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 tablegalette_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.0supprimé - 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
Appliqué par commit ef79ad8b806d8c7112045941432dfd66fde4a78d.