Plugin Identifiants - commentaires

Plugin Identifiants

Bonjour tout le monde,

Petit retour @tcharlss : depuis presque un an, tout fonctionne très bien ;-)

françois

Plugin Identifiants

Bugs :
- lang > identifiants_fr.php
- ligne 44 > êtes-vous (avec un tiret)

Remarque :
- on peut configurer Groupes de mots-clés, Mots-clés et Sites référencés même si on n'a pas choisi de les utiliser dans Contenu du site

Tests :
Identifiants 1.1.16 :
SPIP3.2.9 :
- obligation de mettre à jour la ligne comme cedric https://git.spip.net/spip/spip/commit/f8c6aa9ac2dc43182faf0a2bcd38b6164c65c471 pour que Groupes-mots soit bien retrouvé

SPIP3.3-DEV avec correctif de cedric :
- tout fonctionne bien

Identifiants 2.0.2 -> activation et donc mise à jour de la 1.1.16 déjà installée :
SPIP3.2.9 :
- la colonne identifiant est bien ajoutée aux objets
- la migration des identifiants de la table vers les colonnes est OK
- la table identifiant est bien détruite
- obligation de mettre à jour la ligne comme cedric https://git.spip.net/spip/spip/commit/f8c6aa9ac2dc43182faf0a2bcd38b6164c65c471 pour que Groupes-mots soit bien retrouvé

SPIP3.3-DEV avec correctif de cedric :
- la colonne identifiant est bien ajoutée aux objets
- la migration des identifiants de la table vers les colonnes est OK
- la table identifiant est bien détruite
- tout fonctionne bien ;-)

Plugin Identifiants

Merci pour le ticket @tcharlss, et merci à @cedric pour le correctif !

Bien sûr, je vais tester ta V2.

Plugin Identifiants

C'est corrigé dans le core : https://git.spip.net/spip/spip/commit/f8c6aa9ac2dc43182faf0a2bcd38b6164c65c471

Plugin Identifiants

J'ai fait quelques tests et j'ai une erreur similaire avec la dist, hors plugins identifiants.
Comme tu dis que ça fonctionnait il y a quelque temps, ça voudrait dire que quelque chose a changé avec les jointures Spip, donc ticket : https://core.spip.net/issues/4701

Tout ça pousserait à sortir la v2 du plugin dis donc :)
La version est prête mais il reste à bien la tester pour s'assurer qu'il n'y a pas de régression.
Si tu as l'âme d'un aventurier tu peux l'essayer de suite, c'est la branche « master » sur le git (en prenant soin de faire une backup de la bdd avant au cas-où).

Plugin Identifiants

Je constate que sur une version « plus ancienne » de SPIP3.3-dev, toujours avec Identifiants 1.1.16, la requête est la suivante :

SELECT L1.id_objet, L1.id_objet AS id_groupe
FROM spip_identifiants AS L1 WHERE (L1.identifiant = 'boutique')
GROUP BY L1.id_objet

Plugin Identifiants

... et pour la requête :

SELECT L1.id_objet, L1.id_objet AS id_groupe
FROM spip_identifiants AS L1 WHERE (L1.objet = 'groupe') AND (L1.identifiant = 'boutique')
GROUP BY L1.id_objet

Plugin Identifiants

Est-ce que cela peut t'aider :

INSERT INTO spip_identifiants (identifiant, objet, id_objet, maj) VALUES
('boutique',	'groupe_mot',	2,	'2021-03-19 16:28:34');

Plugin Identifiants

Je suppose que le problème vient de cette partie de la requête, où « groupe_mot » est devenu « groupe » tout court :

array('=', 'L1.objet', sql_quote('groupe')),

Bon pour déboguer il faudrait que je puisse reproduire, s'il n'y a pas de choses confidentielles est-ce que tu pourrais transmettre un dump de ta table spip_identifiants ? (au format csv ou autre)

Ps : avec le mode debug en cliquant sur le lien « résultat » d'une boucle ça te donne la requête sql telle quelle, bien plus lisible :)

Plugin Identifiants

Tout fonctionnait déjà en 3.3 : la version date d'il y a 2 mois environ.

La requête : http://spip.pastebin.fr/88034

Plugin Identifiants

Hello,

Ça produit quelle requête sql ?
Et ça marche plus uniquement depuis le passage en 3.3 ?

Si c'est une histoire de jointure foireuse la v2 du plugin règlera sans doute le problème car ça n'utilise plus de table auxiliaire. Je devais finir les tests avant de releaser... Mais ça traîne :)

Précédemment, il me semble qu'il y avait 2 enregistrements pour le même identifiant : groupe_mot et groupe_mots.

Ah, ça je vois pas trop comment c'est possible. Le doublon a été supprimé à la main ?

Plugin Identifiants

Bonjour,

Suite à la mise à jour de SPIP 3.3.0-dev :


#TITRE

ne renvoi plus le titre du groupe de mots, alors qu'il n'y avait pas de souci auparavant.

Dans la table IDENTIFIANTS, l'identifiant toto est bien enregistré avec comme objet groupe_mot. Précédemment, il me semble qu'il y avait 2 enregistrements pour le même identifiant : groupe_mot et groupe_mots.

Plugin Identifiants

Salut pierre,

Oui bien vu, ça pourrait éviter une requête sql inutile.
À tester, et si ça fonctionne, go go go comme dirait l'autre !

Plugin Identifiants

Salut,

Pour la balise #IDENTIFIANT je tenterai bien un truc du genre pour éviter de passer par une fonction de calcul.

 // $p->code = 'identifiant_objet("' . $_objet . '",' . $_id . ')'; $p->code = champ_sql('identifiant') . '?: identifiant_objet("' . $_objet . '",' . $_id . ')';

J'ai supprimé une notice mais j'ai pas osé commit cette modif :)
https://zone.spip.net/trac/spip-zone/changeset/111925/spip-zone

Je ne suis pas certain que l'exécution de la fonction objet_identifiant soit réellement esquivée. des avis ?

Plugin Identifiants

Je découvre : excellent, merci tcharlss !

Plugin Identifiants

Ça devrait être réglé dans la vesion 1.1.9, merci du signalement.

Plugin Identifiants

Merci d'avoir signalé cette grosse omission !
Je regarde ça.

Plugin Identifiants

Le concept est sympa par contre j'ai rencontré un soucis : lorsque l'on supprime une rubrique qui a un identifiant, celui-ci n'est pas supprimé de la table spip_identifiants et ne peut donc être réattribué.

Plugin Identifiants

Bravo ! Bravissimo ! +++++ !