Les tables ne s'installent pas si l'on choisi un chiffre comme simple préfix #3626

Closed
opened 7 years ago by Franck · 10 comments
Franck commented 7 years ago

Bonjour
SPIP 3.1.0-rc3 [22681]
PHP 5.4.45 chez ovh

En voulant faire un test, je viens de me rendre compte que si comme préfix de tables au moment de l'installation, l'on choisi de ne mettre qu'un simple chiffre, alors l'installation ne fonctionne pas correctement.
Pour reproduire le problème, il suffit de mettre un simple chiffre à l'étape 2 de l'installation, exemple "8"
Quand on arrive à l'étape 3, il y a bien le message "La structure de votre base de données est installée"
Par contre, si je regarde à ce moment là les tables via phpmyadmin, elles sont absentes, et si je continu l’installation, à la fin, j'aurais comme message:
Site en travaux
Attention : un problème technique (serveur SQL) empêche l’accès à cette partie du site. Merci de votre compréhension.

Ce qui est logique car il n'y a pas de tables.

Franck

Bonjour SPIP 3.1.0-rc3 [22681] PHP 5.4.45 chez ovh En voulant faire un test, je viens de me rendre compte que si comme préfix de tables au moment de l'installation, l'on choisi de ne mettre qu'un simple chiffre, alors l'installation ne fonctionne pas correctement. Pour reproduire le problème, il suffit de mettre un simple chiffre à l'étape 2 de l'installation, exemple "8" Quand on arrive à l'étape 3, il y a bien le message "La structure de votre base de données est installée" Par contre, si je regarde à ce moment là les tables via phpmyadmin, elles sont absentes, et si je continu l’installation, à la fin, j'aurais comme message: _Site en travaux Attention : un problème technique (serveur SQL) empêche l’accès à cette partie du site. Merci de votre compréhension._ Ce qui est logique car il n'y a pas de tables. Franck
Owner

Je baisse la priorité. C'est pas un cas très courant

Je baisse la priorité. C'est pas un cas très courant
Poster

Hello
Pour info, je viens de faire un test d'installation en 3.0.22 svn et le problème est aussi présent, donc sans doute un vieux bug qui traine :-)
Bref, c'est beaucoup moins grave/urgent d'un coup :-)

Hello Pour info, je viens de faire un test d'installation en 3.0.22 svn et le problème est aussi présent, donc sans doute un vieux bug qui traine :-) Bref, c'est beaucoup moins grave/urgent d'un coup :-)
Owner

Assigné à marcimat

**Assigné à marcimat**
Owner

En rapport avec http://stackoverflow.com/questions/15917064/table-or-column-name-cannot-start-with-numeric
On devrait mettre un message d'erreur à l'installation ; effectivement ça ne passe pas sans backticker le nom de table dans les requêtes (8_articles), ce qu'on ne fait pas.

En rapport avec http://stackoverflow.com/questions/15917064/table-or-column-name-cannot-start-with-numeric On devrait mettre un message d'erreur à l'installation ; effectivement ça ne passe pas sans backticker le nom de table dans les requêtes (`8_articles`), ce qu'on ne fait pas.
b_b commented 5 years ago
Owner

Je vois qu'on effectue déjà une vérification/modification du préfixe saisie ici :

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/install/etape_3.php#L31

Peut-être serait-il intéressant d'ajouter un traitement automatique à ce niveau (puisqu'on le fait déjà) plutôt que d'afficher un message d'erreur ? D'autant plus qu'on affiche bien un avertissement "Ce préfixe s’écrit en lettres minuscules, non accentuées, et sans espace.", même si on accepte les chiffres...
Statut changé à En cours

Je vois qu'on effectue déjà une vérification/modification du préfixe saisie ici : https://core.spip.net/projects/spip/repository/entry/spip/ecrire/install/etape_3.php#L31 Peut-être serait-il intéressant d'ajouter un traitement automatique à ce niveau (puisqu'on le fait déjà) plutôt que d'afficher un message d'erreur ? D'autant plus qu'on affiche bien un avertissement "Ce préfixe s’écrit en lettres minuscules, non accentuées, et sans espace.", même si on accepte les chiffres... **Statut changé à En cours**
b_b commented 5 years ago
Owner

Voici un patch qui devrait permettre de supprimer automatiquement les chiffres en début de préfixe. Votre avis ?

Voici un patch qui devrait permettre de supprimer automatiquement les chiffres en début de préfixe. Votre avis ?
b_b commented 5 years ago
Owner

Patch à jour en déportant toutes ces corrections dans une fonction preparer_prefixe_tables() comme proposé par marcimat sur IRC.

Patch à jour en déportant toutes ces corrections dans une fonction `preparer_prefixe_tables()` comme proposé par marcimat sur IRC.
b_b commented 5 years ago
Owner
There is no content yet.
b_b commented 5 years ago
Owner
There is no content yet.
b_b commented 5 years ago
Owner

Et hop, on ferme :)
Statut changé à Fermé

Et hop, on ferme :) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.