Project

General

Profile

Evolution #4151

Le bouton de création rapide d'un mot ne sélectionne pas le groupe en cours du mot

Added by RealET 🔸 over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
06/14/2018
Due date:
% Done:

0%

Resolution:
fixed

Description

Avec les bouton d'action rapide dans la barre des gadgets :

  • quand on est dans la page d'un groupe de mot (?exec=groupe_mots&id_groupe=6), le bouton rapide reçoit bien id_groupe et propose de créer un mot dans ce groupe (cohérent avec le fonctionnement des rubriques et le bouton de création rapide d'un article)
  • mais si on est dans une page d'un mot (?exec=mot&id_mot=112) alors le bouton de création rapide d'un mot ne passe pas le id_groupe (incohérent avec les articles où quand on est déjà dans un article, le bouton de création rapide d'un article passe bien le id_rubrique)

Une lecture du code de ecrire/inc/bandeau.php montre que definir_barre_contexte ne marche que sur des rubriques ou articles et quand on est dans un groupe (mais pas quand on est dans un mot)
(Commentaire de b_b : Pour le bouton de création rapide, il passe bien l'id_groupe depuis la page de vue d'un groupe cf exec=groupe_mots&id_groupe=X. Reste à voir si on peut et si on doit reproduire le même comportement depuis la page de vue d'un mot.)

PS : ce ticket est créé à la demande de b_b pour séparer les problèmes évoqués dans #3976

4151.diff View (397 Bytes) b b, 06/14/2018 11:39 AM


Related issues

Related to SPIP - Roadmap #3844: Gérer la parenté dans la déclaration d'un objet éditorial. En cours 10/23/2016

Associated revisions

Revision 24004 (diff)
Added by bruno@eliaz.fr over 1 year ago

Fix #4151 : passer l'id_groupe en paramètre lors de la "création rapide" d'un mot depuis une page mot

... en passant id_groupe dans le contexte du bandeau

patch temporaire en attendant que #3844 soit réglé

Revision 24005 (diff)
Added by bruno@eliaz.fr over 1 year ago

report de r24004

Fix #4151 : passer l'id_groupe en paramètre lors de la "création rapide" d'un mot depuis une page mot

... en passant id_groupe dans le contexte du bandeau

patch temporaire en attendant que #3844 soit réglé

Revision 24006 (diff)
Added by bruno@eliaz.fr over 1 year ago

report de r24004

Fix #4151 : passer l'id_groupe en paramètre lors de la "création rapide" d'un mot depuis une page mot

... en passant id_groupe dans le contexte du bandeau

patch temporaire en attendant que #3844 soit réglé

History

#1 Updated by b b over 1 year ago

  • File 4151.diff added
  • Tracker changed from Anomalie to Evolution
  • Status changed from Nouveau to En cours

Ça se passe donc ici :

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/bandeau.php#L53

Le patch ci-joint corrige "le bug", mais ça me chagrine d'ajouter une mention aux mots clés dans le core alors que ceux-ci sont bien isolés dans un plugin dist.

#2 Updated by b b over 1 year ago

  • File deleted (4151.diff)

#3 Updated by b b over 1 year ago

Patch sans code de bug...

#4 Updated by RealET 🔸 over 1 year ago

Le patch est simple ;-) Merci

Pour la mention aux mots clefs dans le core, je comprend la réticence.

D'autant plus qu'en fait, ça rejoint #3844 et que si #3844 était résolu, je suppose que le code du bandeau pourrait devenir générique avec un parcours des parents possibles.

Donc, ton patch pourrait être intégré en dur en 3.2 et 3.1, et en mou dans 3.3 en attente de #3844 (avec mention dans ce dernier qu'il faudra aussi s'occuper du bandeau)

Qu'en dis-tu ?

#5 Updated by b b over 1 year ago

  • Related to Roadmap #3844: Gérer la parenté dans la déclaration d'un objet éditorial. added

#6 Updated by b b over 1 year ago

Voilà, j'ai fait le lien avec #3844, mais je pense qu'il faut attendre la résolution de ce dernier avant de faire quoi ce soit (histoire de ne pas "polluer" les branches stables avec mon patch).

#7 Updated by RealET 🔸 over 1 year ago

Je comprends ton envie d'attendre.
Mais je crains que #3844 prenne encore beaucoup de temps et surtout, ne soit pas backporté en 3.1 (voire en 3.2).
Donc, ton patch avec en plus un commentaire vers #3844 et un gros ToDo dans la remarque me semblerait bien pertinent pour avoir quelque chose qui marche avant d'avoir quelque chose de parfait plus tard.

#8 Updated by nicod _ over 1 year ago

Je suis un peu de cet avis, appliquer le patch pour 3.1 et 3.2
Dans l'absolu on pourrait faire mieux mais la correction est simple et n'ajoute pas de dépendance (avec un isset, c'est propre).

#9 Updated by b b over 1 year ago

pour avoir quelque chose qui marche avant d'avoir quelque chose de parfait plus tard

Cet argument m'a convaincu car je l'emploie souvent :p

D'autres avis ? J'envoie direct ?

PS : on pourrait même l'intégrer dans le trunk aussi.

#10 Updated by b b over 1 year ago

  • Status changed from En cours to Fermé
  • Resolution set to fixed

Et hop, on ferme :)

Also available in: Atom PDF