mots et groupes - API d'édition des objets #3181

Closed
opened 9 years ago by severo · 2 comments
severo commented 9 years ago

Les fonctions d'action du plugin mots ne semblent pas suivre les mêmes règles que l'API d'édition des objets, ce qui en permet pas d'utiliser ces fonctions génériques pour les mots ou les groupes de mots (me semble-t-il). Est-il envisageable de les adapter au format défini par l'API ?

Insertion

API : "objet_inserer":http://code.spip.net/autodoc/tree/ecrire/action/editer_objet.php.html#function_objet_inserer


bool | int objet_inserer( string $objet , int $id_parent = null , array | null $set = null )

Mot : "mot_inserer":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_mot.php.html#function_mot_inserer


int | bool mot_inserer( int $id_groupe )

Groupe de mots : "groupemots_inserer":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_groupe_mots.php.html#function_groupemots_inserer


int | bool groupemots_inserer( string $table = '' )

Modification

API : "objet_modifier":http://code.spip.net/autodoc/tree/ecrire/action/editer_objet.php.html#function_objet_modifier


mixed | string objet_modifier( string $objet , int $id , array | null $set = null )

Mot : "mot_modifier":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_mot.php.html#function_mot_modifier


string | null mot_modifier( int $id_mot , array | null $set = null )

Groupe mots : "groupemots_modifier":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_groupe_mots.php.html#function_groupemots_modifier


string | null groupemots_modifier( int $id_groupe , array | null $set = null )

Les fonctions d'action du plugin mots ne semblent pas suivre les mêmes règles que l'API d'édition des objets, ce qui en permet pas d'utiliser ces fonctions génériques pour les mots ou les groupes de mots (me semble-t-il). Est-il envisageable de les adapter au format défini par l'API ? ## Insertion API : "objet_inserer":http://code.spip.net/autodoc/tree/ecrire/action/editer_objet.php.html#function_objet_inserer<pre> bool | int objet_inserer( string $objet , int $id_parent = null , array | null $set = null ) </pre> Mot : "mot_inserer":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_mot.php.html#function_mot_inserer<pre> int | bool mot_inserer( int $id_groupe ) </pre> Groupe de mots : "groupemots_inserer":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_groupe_mots.php.html#function_groupemots_inserer<pre> int | bool groupemots_inserer( string $table = '' ) </pre> ## Modification API : "objet_modifier":http://code.spip.net/autodoc/tree/ecrire/action/editer_objet.php.html#function_objet_modifier<pre> mixed | string objet_modifier( string $objet , int $id , array | null $set = null ) </pre> Mot : "mot_modifier":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_mot.php.html#function_mot_modifier<pre> string | null mot_modifier( int $id_mot , array | null $set = null ) </pre> Groupe mots : "groupemots_modifier":http://code.spip.net/autodoc/tree/plugins-dist/mots/action/editer_groupe_mots.php.html#function_groupemots_modifier<pre> string | null groupemots_modifier( int $id_groupe , array | null $set = null ) </pre>
Owner

Ah oui il manque simplement la prise en compte du $set en second argument de l'insertion. Le reste est OK (fort logiquement le premier argument disparait puisqu'on sait qu'on porte sur un mot ou un groupe, et c'est bien mappé comme ça dans objet_modifier et objet_inserer que tu peux utiliser de façon générique sans avoir à savoir si une fonction existe ou non pour ton objet)
Assigné à cedric
Version cible mise à 3.1
Statut changé à En cours

Ah oui il manque simplement la prise en compte du $set en second argument de l'insertion. Le reste est OK (fort logiquement le premier argument disparait puisqu'on sait qu'on porte sur un mot ou un groupe, et c'est bien mappé comme ça dans objet_modifier et objet_inserer que tu peux utiliser de façon générique sans avoir à savoir si une fonction existe ou non pour ton objet) **Assigné à cedric** **Version cible mise à 3.1** **Statut changé à En cours**
Owner

http://zone.spip.org/trac/spip-zone/changeset/85596
http://zone.spip.org/trac/spip-zone/changeset/85597
http://zone.spip.org/trac/spip-zone/changeset/85598
http://zone.spip.org/trac/spip-zone/changeset/85599
http://zone.spip.org/trac/spip-zone/changeset/85600

On peut donc utiliser objet_modifier() et objet_inserer() pour tout objet sans se soucier de savoir si ils ont ou non une fonction dédiée, car elle sera automatiquement appelée.

A noter que c'est bien groupe_mots_inserer et groupe_mots_modifier qui sont standard.
Statut changé à Fermé

http://zone.spip.org/trac/spip-zone/changeset/85596 http://zone.spip.org/trac/spip-zone/changeset/85597 http://zone.spip.org/trac/spip-zone/changeset/85598 http://zone.spip.org/trac/spip-zone/changeset/85599 http://zone.spip.org/trac/spip-zone/changeset/85600 On peut donc utiliser objet_modifier() et objet_inserer() pour tout objet sans se soucier de savoir si ils ont ou non une fonction dédiée, car elle sera automatiquement appelée. A noter que c'est bien `groupe_mots_inserer` et `groupe_mots_modifier` qui sont standard. **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.