[SPIP Zone] r104899 - _core_/branches/spip-3.1/plugins/medias/formulaires

spip-zone-commit@rezo.net a écrit le 19/06/2017 à 10:22 :

Author: chankalan@choc0.net
Date: 2017-06-19 10:22:28 +0200 (Mon, 19 Jun 2017)
New Revision: 104899

Modified:
    _core_/branches/spip-3.1/plugins/medias/formulaires/inc-upload_document.html
    _core_/branches/spip-3.1/plugins/medias/formulaires/joindre_document.html
Log:
lorsqu'on fait "modifier" un document, ne pas proposer un multi upload lorsqu'on veut juste changer le fichier
https://core.spip.net/issues/3961

Details: Connexion · GitLab

C'est curieux, j'aurais codé de manière inverse :
#ENV{multi,oui}
pour n'avoir à modifier que les endroits n'ayant pas besoin de multi (avec multi=non en paramètre).
Ainsi, le code par défaut potentiellement utilisé ailleurs n'aurait aucun changement.

Mes 2 sous

--
RealET

salut RealET,
j’ai poussé le commit un peu rapidement pour SPIP 3.1, et finalement je sais pas s’il faut le revert… mais j’ai effectivement raisonné comme toi pour SPIP 3.2 :
En fait j’aurai pas dû pousser le premier commit, mais c’est fait, je le corrige pour que ce soit cohérent ?

chankalan@choc0.net a écrit le 19/06/2017 à 14:09 :

salut RealET,
j'ai poussé le commit un peu rapidement pour SPIP 3.1, et finalement je sais pas s'il faut le revert... mais j'ai effectivement raisonné comme toi pour SPIP 3.2 :
Connexion · GitLab
En fait j'aurai pas dû pousser le premier commit, mais c'est fait, je le corrige pour que ce soit cohérent ?

Ah oui !
Il faut toujours être cohérent au maximum.
Et la version pour 3.2 est plus mieux :wink:

PS : je suis pas core dev, alors, mon avis est à prendre pour ce qu'il est.

--
RealET

hop