spip
/
medias
16
1
Fork 7
Code Issues 32 Pull Requests 2 Releases Wiki Activity
Labels Milestones
New Issue

Refaire un alter sur le champ mode avant la migration des logos, par précaution #4889

Closed
opened 10 months ago by cerdic · 1 comments
cerdic commented 10 months ago
Owner

cf https://discuter.spip.net/t/probleme-bizarre-avec-les-logos-lors-de-la-migration-a-spip-4-plus-aucun-logos-lies-aux-articles-auteurs-rubriques-dans-lespace-prive/155168/9

J’ai fini par trouver la solution. Dans mon cas, dans la table spip_documents, le type du champ mode était resté en enum(vignette, image,document) au lieu d’être en varchar(10). Une mise à jour ancienne du plugin medias n’avait pas du se faire correctement !

La conséquence est que la fonction de migration des logos rencontre une erreur SQL car elle cherche à ajouter logoon comme mode et le enum() l’en empêche !

le changement de type raté n'avait pas vraiment de conséquences jusqu'ici, mais il bloque la migration des logos.
Du coup je me dis qu'un alter avant la migration serait une bonne précaution

cf https://discuter.spip.net/t/probleme-bizarre-avec-les-logos-lors-de-la-migration-a-spip-4-plus-aucun-logos-lies-aux-articles-auteurs-rubriques-dans-lespace-prive/155168/9 > J’ai fini par trouver la solution. Dans mon cas, dans la table spip_documents, le type du champ mode était resté en enum(vignette, image,document) au lieu d’être en varchar(10). Une mise à jour ancienne du plugin medias n’avait pas du se faire correctement ! > > La conséquence est que la fonction de migration des logos rencontre une erreur SQL car elle cherche à ajouter logoon comme mode et le enum() l’en empêche ! le changement de type raté n'avait pas vraiment de conséquences jusqu'ici, mais il bloque la migration des logos. Du coup je me dis qu'un alter *avant* la migration serait une bonne précaution
b_b added the
amélioration
 label 10 months ago
b_b added this to the spip-4.0 milestone 10 months ago
marcimat commented 10 months ago
Owner

En discutant, on se dit que peut être une commande spip-cli pour relancer cette migration medias:1.6.0 serait pas mal (et peut être utile dans d’autres circonstances)

En discutant, on se dit que peut être une commande spip-cli pour relancer cette migration medias:1.6.0 serait pas mal (et peut être utile dans d’autres circonstances)
👍 1
marcimat closed this issue 10 months ago
marcimat closed this issue 10 months ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
3.2
4.0
4.1
dev-sortable
dev/issue_4645_lien_fichier
issue_4480
master
spip-3.0
spip-3.1
spip-3.2
touti-patch-1
Labels
Apply labels
Clear labels
amélioration
Amélioration, nouvelle fonctionnalité bug
Ca ne fonctionne pas documentation
Documentation souhaite doublon
Ce ticket est un doublon invalide
Ticket invalide refusé
Ignoré, c'est comme Ca...
No Label amélioration bug documentation doublon invalide refusé
Milestone
Set milestone
Clear milestone
No items
No Milestone
spip-4.0
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes