[spip-dev] [spip-commit] r21682 - in spip: ecrire/action prive/formulaires

Dev
1

Hop,

Author: cedric@yterium.com
Date: 2014-10-23 18:13:05 +0200 (jeu, 23 oct 2014)
New Revision: 21682

Log:
Fix #2759 : messages d'erreur sur le formulaire edition logo
+ refactoring : l'action iconifier disparait totalement, le code qu'on en utilisait est integre a formulaires/editer_logo.php en supprimant le code mort

Juste une petite question au sujet du retrait de l'action iconifier. Après recherche il y quelques plugins qui utilisent cette action sur la zone.

N'est-ce pas un peu bourrin de virer cette action dans la 3.1 ?

Cela risque de forcer la création de branches spécifiques ou d'effectuer des tests sur la présence de l'action en question dans les plugins qui l'utilisent.

À moins qu'il y ait une redirection de secours sur cette fonction ?

2

Re,

Juste une petite question au sujet du retrait de l'action iconifier.
Après recherche il y quelques plugins qui utilisent cette action sur la
zone.

Voici le résultat d'un rep sur l'action iconifier sur le dossier plugins de la zone :

grep -nHIrF --exclude=*.svn "action/iconifier" .

./kconf/action/kconf_admin.php:20: include_spip("action/iconifier");
./spip_objets/formulaires/editer_logo.php:128: include_spip('action/iconifier');
./cameras/cameras_fonctions.php:11: include_spip('action/iconifier');
./crayons/inc/crayons.php:161: include_spip('action/iconifier');
./ajaxforms/formulaires/editer_logo.php:124: include_spip('action/iconifier');
./tickets/trunk/inc/migration_200.php:212: include_spip('action/iconifier');
./inscription/trunk/inscription3_pipelines.php:675: include_spip('action/iconifier');
./inscription/branches/inscription3_2x/inscription3_pipelines.php:642: include_spip('action/iconifier');
./inscription/branches/inscription3_2x/formulaires/inscription3.php:369: include_spip('action/iconifier');
./inscription/branches/inscription2_2_0/formulaires/inscription2.php:427: include_spip('action/iconifier');
./diogene/diogene/trunk/diogene_pipelines.php:819: include_spip('action/iconifier');
./spipservice/trunk/inc/spip_service_core.php:92: include_spip('action/iconifier');
./spipservice/trunk/inc/spip_service_core.php:636: include_spip('action/iconifier');
./spipservice/branches/2.1/inc/spip_service_core.php:84: include_spip('action/iconifier');
./spipservice/branches/2.1/inc/spip_service_core.php:651: include_spip('action/iconifier');
./nouvelle_version/action/dupliquer.php:91: include_spip('action/iconifier');
./logo/prive/formulaires/editer_logo.php:186: include_spip('action/iconifier');
./plugin_actualite/formulaires/editer_logo.php:128: include_spip('action/iconifier');
./duplicator/trunk/inc/duplicator.php:208: include_spip('action/iconifier');
./duplicator/branches/v0/action/dupliquer.php:207: include_spip('action/iconifier');
./titre_de_logo/trunk/formulaires/editer_logo.php:195: include_spip('action/iconifier');
./sel/formulaires/logo_auteur.php:175: include_spip('action/iconifier');
./verifcore/refcorespip191.txt:699:ecrire/action/iconifier.php:3349:1155592721
./boussole/trunk/inc/iconifier_site.php:33: include_spip('action/iconifier');

3

J'utilise aussi l'action spip_image_ajouter dans un plugin (un projet bien spécifique pas sur la zone).

Il faudra que je revois cette partie, mais bon, heureusement que j'ai vu passer ce commit, je saurai au moins ou chercher quand les utilisateurs me remonteront le bug après la mise à jour.

4

Bon j'ai été un peu violent. En même temps cette action est vraiment à reformer. Donc éventuellement je la remets pour la compat mais en même temps avec une vrai fonction api propre, et on virera la vieille fonction a la release suivante.

Cédric

nicod_ a écrit:

5

J’ai donc remis une API propre action/editer_logo dans la 3.1, et voici le pattern, sur le plugin crayons, pour la mise a jour des codes qui utilisaient l’ancienne fonction :

http://zone.spip.org/trac/spip-zone/changeset/85713

La vieille fonction étant vraiment très moche et incomprehensible je ne suis pas trop motivé pour ecrire un helper qui prend ses anciens appels et les renvoi vers la nouvelle API… Mais si tu as le courage on peut ajouter ça pour ne casser aucun plugin.

Cédric

6

Ecrire un helper moche qu'on va trainer encore longtemps, ou bien reprendre les quelques plugins qui l'utilisent, et qui de toute façon sont à retester pour passer en 3.1 ?
L'exemple de portage que tu donnes sur crayons est très clair, l'API est beaucoup plus facile à utiliser.

Le mieux serait peut être une annonce générale sur les listes, avec la liste des plugins impactés ?

7

Faut le mettre clairement dans la note de release, et fixer les plugins de la zone, ça devrait suffire comme ça

8

Moi ça ne me gène pas que des plugins aient quelques soucis provisoires de compatibilité entre la 3.0 et la 3.1.

Si c’est clairement dit dans les logs, on sait ce qu’il faut changer.

Au pire on refait un plugin de type “grenier” qui permet d’utiliser d’anciennes fonctions obsolètes