Incompatibilité SPIP 3.3 #1

Closed
opened 2 years ago by RealET · 6 comments
RealET commented 2 years ago
Collaborator

Bonjour,

Je teste le plugin avec SPIP 3.3.

J'ai une rubrique avec plein d'articles dedans : ecrire/?exec=rubrique&id_rubrique=5

Plugin inactif : la liste des articles est bien affichée
Plugin actif : aucun article n'est affiché

La liste des articles s'affiche bien si je rajoute return $interface; au début de
https://git.spip.net/spip-contrib-extensions/grappes/src/branch/master/base/grappes.php#L22

Par contre, les sites référencés sont correctements listés.

Bonjour, Je teste le plugin avec SPIP 3.3. J'ai une rubrique avec plein d'articles dedans : ecrire/?exec=rubrique&id_rubrique=5 Plugin inactif : la liste des articles est bien affichée Plugin actif : aucun article n'est affiché La liste des articles s'affiche bien si je rajoute return $interface; au début de https://git.spip.net/spip-contrib-extensions/grappes/src/branch/master/base/grappes.php#L22 Par contre, les sites référencés sont correctements listés.
Collaborator

Ça passe de

SELECT articles.id_article, 0 as points, '', articles.titre, articles.lang, articles.statut, articles.id_rubrique, articles.surtitre, articles.titre AS titre_rang, articles.soustitre, articles.date
FROM spip_articles AS `articles`
WHERE (articles.id_rubrique = 1)
	AND (articles.statut = 'prop')
GROUP BY articles.id_article
ORDER BY articles.date DESC, articles.titre

à

SELECT articles.id_article, 0 as points, '', articles.titre, articles.lang, articles.statut, articles.id_rubrique, articles.surtitre, L4.rang, articles.soustitre, articles.date
FROM spip_articles AS `articles`  
INNER JOIN spip_grappes_liens AS L4 ON ( L4.id_objet = articles.id_article AND L4.objet='article')
WHERE (articles.id_rubrique = 1)
	AND (articles.statut = 'prop')
GROUP BY articles.id_article
ORDER BY articles.date DESC, articles.titre

à cause de la ligne

  • $interface['tables_jointures']['spip_articles'][] = 'grappes_liens';

On voit qu’il crée la liaison à cause de la présence de 'rang' dans la table spip_grappes_liens.

Déjà je pense que 'rang' est mal nommé ici. On a maintenant plus ou moins adopté la convention d’utiliser 'rang_lien' sur les tables de liens, et 'rang' sur les objets éditoriaux.

Ensuite, bah c’est tout de même un bel imprévu cette liaison pour satisfaire 'rang'. Raison de plus pour ajouter 'rang' dans spip_articles :)

Ça passe de ```sql SELECT articles.id_article, 0 as points, '', articles.titre, articles.lang, articles.statut, articles.id_rubrique, articles.surtitre, articles.titre AS titre_rang, articles.soustitre, articles.date FROM spip_articles AS `articles` WHERE (articles.id_rubrique = 1) AND (articles.statut = 'prop') GROUP BY articles.id_article ORDER BY articles.date DESC, articles.titre ``` à ```sql SELECT articles.id_article, 0 as points, '', articles.titre, articles.lang, articles.statut, articles.id_rubrique, articles.surtitre, L4.rang, articles.soustitre, articles.date FROM spip_articles AS `articles` INNER JOIN spip_grappes_liens AS L4 ON ( L4.id_objet = articles.id_article AND L4.objet='article') WHERE (articles.id_rubrique = 1) AND (articles.statut = 'prop') GROUP BY articles.id_article ORDER BY articles.date DESC, articles.titre ``` à cause de la ligne - `$interface['tables_jointures']['spip_articles'][] = 'grappes_liens';` On voit qu’il crée la liaison à cause de la présence de 'rang' dans la table spip_grappes_liens. Déjà je pense que 'rang' est mal nommé ici. On a maintenant plus ou moins adopté la convention d’utiliser 'rang_lien' sur les tables de liens, et 'rang' sur les objets éditoriaux. Ensuite, bah c’est tout de même un bel imprévu cette liaison pour satisfaire 'rang'. Raison de plus pour ajouter 'rang' dans spip_articles :)
Poster
Collaborator

Concrètement, tu préconises quoi ?

De modifier Grappes pour changer partout rang en rang_lien (et faire la procédure de mise à jour qui va bien avec) ?

Et qui le fait ?
Toi ?
Moi ?

Concrètement, tu préconises quoi ? De modifier Grappes pour changer partout rang en rang_lien (et faire la procédure de mise à jour qui va bien avec) ? Et qui le fait ? Toi ? Moi ?
Collaborator

Heu oui, ça serait bien ce changement pour sûr.

Faudra bien le signaler, car ça risque de modifier des squelettes de gens qui auraient mis {par rang}.

Par contre, peu de chance que je le fasse moi ; j’ai pas utilisé ce plugin depuis des lustres :)

Heu oui, ça serait bien ce changement pour sûr. Faudra bien le signaler, car ça risque de modifier des squelettes de gens qui auraient mis `{par rang}`. Par contre, peu de chance que je le fasse moi ; j’ai pas utilisé ce plugin depuis des lustres :)
RealET closed this issue 2 years ago
Poster
Collaborator

C'est fait : j'ai fait une version 2.0 (pas encore le tag).

Est-ce que tu pourrais me mettre auteur de https://contrib.spip.net/Grappes-1-0 que je mettes à jour la doc ?

Merci d'avance.

C'est fait : j'ai fait une version 2.0 (pas encore le tag). Est-ce que tu pourrais me mettre auteur de https://contrib.spip.net/Grappes-1-0 que je mettes à jour la doc ? Merci d'avance.
Collaborator

Yep, c’est faut au fait :)

Yep, c’est faut au fait :)
Poster
Collaborator

Article mis à jour, merci
https://contrib.spip.net/Grappes-1-0-2-0

Article mis à jour, merci https://contrib.spip.net/Grappes-1-0-2-0
Sign in to join this conversation.
No Label
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.