Re: Mots-clés : manque un id_mot dans l e tableau de retour

Yohann Prigent <[email protected]>
Newsgroups gmane.comp.web.spip.devel
Message-ID <CAArjcA2xRNFO89QfUK1D0p=z3JjBbK_pdOOKDzEBMk12YR6iBg@mail.gmail.com>
Hmm, mais étant donné que le formulaire d’édition de mot ne se fait qu’en
Ajax, ceci expliquerait qu’id_mot n’est jamais renvoyé ? Car en tout cas,
dans le pipeline formulaire_traiter utilisé par le plugin SEO, il n’arrive
jamais à récupérer id_mot dans $flux[‘data’].
Il y a une raison pour ne pas renvoyer systématiquement id_mot ?

-- 
Yohann Prigent

From: Bruno Bergot <[email protected]> <[email protected]>
Reply: Bruno Bergot <[email protected]> <[email protected]>
Date: August 23, 2016 at 1:24:34 PM
To: Yohann Prigent <[email protected]>
<[email protected]>, Spip-dev
SPIP <[email protected]> <[email protected]>
Subject:  Re: [spip-dev] Mots-clés : manque un id_mot dans le tableau de
retour

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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.