Project

General

Profile

Evolution #3976

Supprimer un comportement dérogatoire à la création d'un mot clé

Added by RealET 🔸 about 2 years ago. Updated about 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
07/16/2017
Due date:
% Done:

100%

Resolution:
fixed

Description

Consistance avec le comportement des autres objets de SPIP : quand on crée un nouveau mot clef, ne pas revenir à la page de liste qui contenait le bouton d'appel à création, mais bien arriver sur la page des propriétés du mot nouvellement créé.
Permet permet par exemple de créer plus rapidement un mot clef puis de lui rajouter un logo.
voir https://zone.spip.org/trac/spip-zone/changeset/93394 qui implémente ce comportement
et https://zone.spip.org/trac/spip-zone/changeset/93395 qui le supprime (« il semblerait qu'il y ait des usages où le comportement dérogatoire (j'aime bien ce mot) actuel est plus ergonomique »)

De plus : les bouton d'action rapide dans la barre des gadjets :
  • 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)

#3976.patch View (1.43 KB) RealET 🔸, 06/14/2018 10:45 AM

Associated revisions

Revision 110688 (diff)
Added by real3t@gmail.com over 1 year ago

fix #3976 : (Report r110686) Pouvoir revenir au mot clef nouvellement créé au lieu du comportement dérogatoire natif consistant à revenir à la page où était le bouton de création du mot.

Dans mes_options.php :
if (!defined('_MOTS_CREATION_RETOUR_MOT_CREE'))
define('_MOTS_CREATION_RETOUR_MOT_CREE', true);

History

#1 Updated by b b about 2 years ago

  • Target version set to 3.3

#2 Updated by b b about 2 years ago

Il serait intéressant de coller ici les arguments que Cedric t'avait exposé lors de votre discussion sur IRC.

#3 Updated by RealET 🔸 about 2 years ago

Ben justement, à l'époque, il n'y en avait pas eu. :-(

#4 Updated by b b about 2 years ago

Sur demande de Cerdic sur IRC, revert.

↑ ton log de commit m'a induit en erreur, hésite pas à lui signaler ce ticket pour qu'il vienne en causer ici :)

#5 Updated by RealET 🔸 about 2 years ago

Pour le 2e point (le bouton de création rapide), une lecture du code de ecrire/inc/bandeau.php montre que definir_barre_contexte ne marche que sur des rubriques.

Sans doute à mettre en relation avec #3844

#6 Updated by b b about 2 years ago

Il serait bien de créer un ticket par point, sans quoi on se retrouve avec des commentaires croisés comme ici :)

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 à vori si on peut et si on doit reproduire le même comportement depuis la page de vue d'un mot.

#7 Updated by RealET 🔸 over 1 year ago

En relisant les 2 commits, je me demande si une solution ne pourrait pas être un define qui ferait par défaut on passe |parametre_url{redirect,#SELF}
Et qu'on puisse le désactiver ce passage de paramètre.

Un avis ?

#8 Updated by RealET 🔸 over 1 year ago

b b a écrit :

Il serait bien de créer un ticket par point, sans quoi on se retrouve avec des commentaires croisés comme ici :)

Fait par #4151

#9 Updated by RealET 🔸 over 1 year ago

Voici un patch pour 3.2.1 qui ne change rien au comportement par défaut.
Et si dans mes_options.php on met :

define('_CREATION_MOT_RETOUR_MOT', 'oui');

Alors, on a le comportement non dérogatoire attendu ;-)

Je propose de commiter ça sur 3.3, 3.2 et 3.1 puisque ça ne change rien par défaut (mais que j'ai plein de site en 3.1 qui en auront besoin).

#10 Updated by b b over 1 year ago

Il faut faire appel à la nomenklatura sur ce coup car en l'état le define ne me semble pas très clair.

Comme le proposait rasta dans #4143 :

Ça me parait une bonne pratique de commencer par un identifiant commun à un groupement fonctionnel, comme pour un espace de nom.

Donc le define en question devrait amha commencer par _MOTS => _MOTS_RETOUR_***.

#11 Updated by b b over 1 year ago

  • Status changed from Nouveau to En cours

#12 Updated by RealET 🔸 over 1 year ago

_MOT_CREATION_RETOUR_MOT
_MOT_CREATION_NON_DEROGATOIRE

#13 Updated by RealET 🔸 over 1 year ago

Le seul define contenant MOTS est : _MAX_MOTS_LISTE

==> Je vais mettre : _MOTS_CREATION_RETOUR_MOT_CREE

#15 Updated by b b over 1 year ago

Heu, il aurait été intéressant d'en discuter un peu plus avant d'intégrer ça dans le trunk et les branches stables non ?

#16 Updated by RealET 🔸 over 1 year ago

Au pire, on change le nom, non ?

#17 Updated by Anonymous about 1 year ago

  • Status changed from En cours to Fermé
  • % Done changed from 0 to 100

Appliqué par commit r110688.

#18 Updated by b b about 1 year ago

  • Resolution set to fixed

Also available in: Atom PDF