utiliser documents au lieu de spip_documents dans la déclaration des traitements #4905

Closed
opened 5 months ago by cerdic · 6 comments
cerdic commented 5 months ago
Owner
https://discuter.spip.net/t/regression-sur-plugin-media-3-2-6/163225 https://git.spip.net/spip/medias/commit/90476e853579f1d85eedb9ea2059a94183ccea1c
b_b added the
bug
label 5 months ago
Poster
Owner

Pour garder une trace unique je recopie ma réponse au fil ici

Donc :

  • par défaut SPIP n’a jamais traité les raccourcis liens dans le champ crédits
  • par contre le plugin oEmbed ajoute cette feature via 0ab594c1b6/oembed_pipelines.php (L337)
  • et donc ensuite on a ajouté le traitement typo sur le champs crédits dans le core via 90476e8535

Le soucis ici vient du fait que dans oembed on déclare le traitement sur documents et dans le core sur spip_documents et pour de sombres raisons historiques, ce dernier prend la main même si le plugin oembed est bien chargé après medias et aurait du avoir le dernier mot.

Il se trouve que

Et pour conclure sur le sujet, c’est donc la déclaration avec la table complètes qui a la priorité quand elle est utilisée
https://git.spip.net/spip/spip/src/branch/master/ecrire/public/references.php#L803

Bref donc la solution ici c’est

  • corriger oEmbed pour déclarer avec spip_documents et pas documents
  • nettoyer dans le core pour mettre l’écriture courte partout pour que ce soit plus facilement surchargeable
Pour garder une trace unique je recopie ma réponse au fil ici Donc : * par défaut SPIP n’a jamais traité les raccourcis liens dans le champ crédits * par contre le plugin oEmbed ajoute cette feature via https://git.spip.net/spip-contrib-extensions/oembed/src/commit/0ab594c1b66edd3128ec92cd55a7617d0d41c15f/oembed_pipelines.php#L337 * et donc ensuite on a ajouté le traitement typo sur le champs crédits dans le core via https://git.spip.net/spip/medias/commit/90476e853579f1d85eedb9ea2059a94183ccea1c Le soucis ici vient du fait que dans oembed on déclare le traitement sur `documents` et dans le core sur `spip_documents` et pour de sombres raisons historiques, ce dernier prend la main **même** si le plugin oembed est bien chargé après medias et aurait du avoir le dernier mot. Il se trouve que * dans les plugins du core on utilise généralement l’écriture courte, sauf sur ce commit dans medias * dans le core lui meme on utilise l’écriture longue https://git.spip.net/spip/spip/src/branch/master/ecrire/public/interfaces.php#L817 1 Et pour conclure sur le sujet, c’est donc la déclaration avec la table complètes qui a la priorité quand elle est utilisée https://git.spip.net/spip/spip/src/branch/master/ecrire/public/references.php#L803 Bref donc la solution ici c’est * corriger oEmbed pour déclarer avec spip_documents et pas documents * nettoyer dans le core pour mettre l’écriture courte partout pour que ce soit plus facilement surchargeable
Poster
Owner

En lien avec spip-contrib-extensions/oembed#22 corrigé par a45db28d3a donc

En lien avec https://git.spip.net/spip-contrib-extensions/oembed/issues/22 corrigé par https://git.spip.net/spip-contrib-extensions/oembed/commit/a45db28d3ac6272d5a6ef958c142beea7251937c donc
Poster
Owner

Et donc 2 PR en lien avec ce ticket

Et donc 2 PR en lien avec ce ticket * dans le plugin medias https://git.spip.net/spip/medias/pulls/4906 * dans le core, par cohérence, https://git.spip.net/spip/spip/pulls/5315
Owner

Avant de fermer reste a question du report en 4.1.

Avant de fermer reste a question du report en 4.1.
Poster
Owner

un report serait pas du luxe pour le plugin medias, car cela a introduit un changement de comportement involontaire.

Pour le core ça peut attendre la 4.2 car c'est comme ça depuis mathusalem je crois

un report serait pas du luxe pour le plugin medias, car cela a introduit un changement de comportement involontaire. Pour le core ça peut attendre la 4.2 car c'est comme ça depuis mathusalem je crois
Owner

un report serait pas du luxe pour le plugin medias, car cela a introduit un changement de comportement involontaire.

Voilà qui est fait, on ferme.

> un report serait pas du luxe pour le plugin medias, car cela a introduit un changement de comportement involontaire. Voilà qui est fait, on ferme.
b_b closed this issue 5 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.