[SPIP Zone] Plugin Coordonnées fonctions obsolètes

Bonjour,

Je suis en train de faire un plugin qui dépend de coordonnées et en me
plongeant dans le code de celui ci je me suis rendu compte qu'il se
trimballe un dossier /action remplie de fonctions obsolètes depuis spip
3.0

Toutes les fonctions, editer_XXX associer_XXX dissocier_XXX
supprimer_XXX sont disponible de façon générique depuis spip 3.0 me
semble-t-il?

Le pire c'est qu'il y a des commits de fait sur les fichiers de ce
dossier qui ne servent à rien si ce n'est qu'à maintenir du code
'obsolète'

Est ce que ça vous semble pertinent de virer le dossier /action pour la
branche v2 et le trunk?

http://zone.spip.org/trac/spip-zone/browser/plugins/coordonnees

Merci de votre attention.

Bonjour,

Non je pense que tout le contenu du dossier /action est nécessaire.
L'API générique c'est juste le fichier editer_XXX.php

- editer_[adresse|numero|email].php on en a besoin car il y a des traitements spécifiques par rapport à la fonction générique il me semble (pour les liaisons).

- Tous les autres fichiers associer_XXX, dissocier_XXX et supprimer_XXX sont nécessaires pour les boutons d'actions dans le BO.
Là c'est plus l'API générique, c'est propre au plugin.

Le 11/10/2016 à 10:31, Pierre a écrit :

Bonjour,

Je suis en train de faire un plugin qui dépend de coordonnées et en me
plongeant dans le code de celui ci je me suis rendu compte qu'il se
trimballe un dossier /action remplie de fonctions obsolètes depuis spip
3.0

Toutes les fonctions, editer_XXX associer_XXX dissocier_XXX
supprimer_XXX sont disponible de façon générique depuis spip 3.0 me
semble-t-il?

Le pire c'est qu'il y a des commits de fait sur les fichiers de ce
dossier qui ne servent à rien si ce n'est qu'à maintenir du code
'obsolète'

Est ce que ça vous semble pertinent de virer le dossier /action pour la
branche v2 et le trunk?

Connexion · GitLab

Merci de votre attention.

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

Ok effectivement, il faudrait modifier également modifier les boutons
d'actions.

De plus Marcimat m'a fortement déconseillé car certains plugins pourrait
utiliser les fonctions actuelles comme inserer_adresse.

Mais au départ, j'avais besoin de créer une interface clients pour un
site de ecommerce et j'avais trouver une solution qui me semblait
élégante afin d’étendre la table_objet adresse pour ajouter un champs
nom_prenom (ça serait plutôt destinataire au final..).

En effet le plugin coordonnée utilise la fonction saisie pour décrire le
formulaire:

Ce qui est bien car grâce au pipeline formulaire_saisies on peut venir
s’insérer facilement où on veut dans le formulaire:
http://spip.pastebin.fr/48128

ensuite il suffit simplement d'étendre la table de description de
adresse et d'ajouter les fonctions gestion des tables qui vont bien:
http://spip.pastebin.fr/48129

Bref tous ça pour dire, qu'au final la fonction revisions_adresses dans:

empêche d'étendre facilement le form vu que les champs_editables sont
écrits en dur...

Au final, si il n'ya pas d'inconvénient, je propose de commit ce genre
de modifications pour les fonctions revision_[adresses/mails/courriel]
http://spip.pastebin.fr/48127

sur le trunk et la branche V2

Merci

Le mardi 11 octobre 2016 à 11:46 +0200, tcharlss@bravecassine.com a
écrit :

Bonjour,

Non je pense que tout le contenu du dossier /action est nécessaire.
L'API générique c'est juste le fichier editer_XXX.php

- editer_[adresse|numero|email].php on en a besoin car il y a des
traitements spécifiques par rapport à la fonction générique il me semble
(pour les liaisons).

- Tous les autres fichiers associer_XXX, dissocier_XXX et supprimer_XXX
sont nécessaires pour les boutons d'actions dans le BO.
Là c'est plus l'API générique, c'est propre au plugin.

Le 11/10/2016 à 10:31, Pierre a écrit :
> Bonjour,
>
> Je suis en train de faire un plugin qui dépend de coordonnées et en me
> plongeant dans le code de celui ci je me suis rendu compte qu'il se
> trimballe un dossier /action remplie de fonctions obsolètes depuis spip
> 3.0
>
> Toutes les fonctions, editer_XXX associer_XXX dissocier_XXX
> supprimer_XXX sont disponible de façon générique depuis spip 3.0 me
> semble-t-il?
>
> Le pire c'est qu'il y a des commits de fait sur les fichiers de ce
> dossier qui ne servent à rien si ce n'est qu'à maintenir du code
> 'obsolète'
>
> Est ce que ça vous semble pertinent de virer le dossier /action pour la
> branche v2 et le trunk?
>
> Connexion · GitLab
>
> Merci de votre attention.
>
> ----
> spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone
>

editer_adresse.php.diff (1.35 KB)

Le 11/10/2016 à 13:27, Pierre a écrit :

empêche d'étendre facilement le form vu que les champs_editables sont
écrits en dur...

Bé oui, c'est sûr que la bonne pratique c'est d'aller chercher la *vraie* définition (modifiable) des champs_editables dans la description API de l'objet.

--
RastaPopoulos