Re: Mots-clés : manque un id_mot da ns le tableau de retour

Hello,

sur ce coup là Yohann a raison, il est de bon usage que le tableau en 
retour de traiter contienne l'id de l'objet qui a été créé ou modifié.
C'est fait par defaut pour la plupart des formulaires cvt editer_xx par
https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/editer.php#L82
mais comme le formulaire editer_mot ne passe pas par là (pour des 
raisons que j'ai oublié mais qui doivent tenir à de l'histoire plus ou 
moins justifiée), du coup il ne suit pas cette convention.

Il faut avouer que c'est moyennement documentée et affirmé comme 
convention, c'est plutôt un usage qui facilite les choses pour les 
pipelines.
Ça me parait donc légitime que le formulaire editer_mot fasse de même.

-- 
Cédric



Bruno Bergot a écrit :
> Hop,
>
> Le 23/08/2016 à 12:28, Yohann Prigent a écrit :
>> Hello,
>>
>> Dans le plugin mots-clés, il manque un id_mot dans le tableau retourné
>> par
>> le CVT :
>> http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/mots/formulaires/editer_mot.php#L240.
>>
>> Les autres objets retournent bien cet id_objet, mais pas ce plugin. Du
>> coup, le plugin SEO est planté sur les mots-clés.
>>
>
> À la lecture du code je ne vois aucun manque. L'id_mot est bien ajouté à
> l'url de retour à condition que :
>
> - un paramètre retour soit bien passé au formulaire, cf
> http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/mots/formulaires/editer_mot.php#L253
>
> - le paramètre retour ne soit pas un retour ajax, cf
> http://zone.spip.org/trac/spip-zone/browser/_core_/plugins/mots/prive/squelettes/contenu/mot_edit.html#L17
>
>
> Bref, le principe est le même que pour les formulaires editer du core, cf :
>
> https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/editer.php#L98
>
>
>
>> Pas d’objection à ce que je le rajoute ?
>>
>
> Du coup, si, objection, à moins que j'ai mal compris le problème et que
> tu nous donnes plus de détails sur ton cas d'usage ;)
>
> ++
> b_b
> _______________________________________________
> liste: http://listes.rezo.net/mailman/listinfo/spip-dev
> doc: http://www.spip.net/
> dev: http://trac.rezo.net/trac/spip/
> irc://irc.freenode.net/spip