[SPIP Zone] r98150 - in _plugins_/fabrique

Le 01/06/2016 à 10:45, teddy.spip@gmail.com a écrit :

Author: teddy.spip@gmail.com
Date: 2016-06-01 10:45:23 +0200 (Wed, 01 Jun 2016)
New Revision: 98150

Modified:
   _plugins_/fabrique/branches/v1/formulaires/fabriquer_plugin.html
   _plugins_/fabrique/branches/v1/paquet.xml
   _plugins_/fabrique/trunk/formulaires/fabriquer_plugin.html
   _plugins_/fabrique/trunk/paquet.xml
Log:
Retrouver la liste des yaml qui se trouvent dans les répertoires 'saisies'. Up de z.

Details: Connexion · GitLab

Arf !

En fait je découvre Connexion · GitLab
Je pense qu'il faut modifier là bas plus tôt pour que si aucun répertoire n'est indiqué, qu'il utilise 'saisies' dans la fonction saisies_lister_disponibles() non ?

MM.

Le 1 juin 2016 à 10:54, Matthieu Marcillaud <marcimat@rezo.net> a écrit :

Le 01/06/2016 à 10:45, teddy.spip@gmail.com a écrit :

Author: teddy.spip@gmail.com
Date: 2016-06-01 10:45:23 +0200 (Wed, 01 Jun 2016)
New Revision: 98150

Modified:
   _plugins_/fabrique/branches/v1/formulaires/fabriquer_plugin.html
   _plugins_/fabrique/branches/v1/paquet.xml
   _plugins_/fabrique/trunk/formulaires/fabriquer_plugin.html
   _plugins_/fabrique/trunk/paquet.xml
Log:
Retrouver la liste des yaml qui se trouvent dans les répertoires
'saisies'. Up de z.

Details: Connexion · GitLab

Arf !

En fait je découvre Connexion · GitLab
Je pense qu'il faut modifier là bas plus tôt pour que si aucun répertoire
n'est indiqué, qu'il utilise 'saisies' dans la fonction
saisies_lister_disponibles() non ?

Ben, justement, je me posais la question… Le plus simple est ce que je
fais. Mais j'ai pensé à mettre une condition dans la fonction qui regarde
si $saisies_repertoires est vide, si oui, on le ré-alimente avec 'saisies'
car c'est le répertoire par défaut.

Qu'en penses-tu ?

Ybbet.

MM.

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

Oui, c’est ce que j’imaginais aussi. Après, techniquement, effectivement le changement ne modifiait rien en PHP ; c’est vraiment cet appel #VAL|saisies_lister_disponibles qui passait un argument vide qui n’existait pas avant. Et il semble que il n’y a que la Fabrique (sur la Zone) qui utilisait ça de la sorte. Donc bon… on peut peut être laisser comme tu as fait aussi finalement. MM.