[SPIP Zone] [Spip-zone-commit] r31460 - in /_plugins_/gestion_documents/formulaires: inc-upload_document.html joindre_document.html

Archives Zone
1

Hello,

Le squelette du formulaire prevoyait deja la possibilité de desactiver les modes ftp et mediatheque en passant #ENV{mediatheque} et #ENV{proposer_ftp}

Pourquoi ne pas les utiliser plutot qu’un alourdissement du html (dont je n’etais deja pas très content ici, mais qui du coup frise l’horrible). Si necessaire il faut ajouter une option au #FORMULAIRE correspondant pour que l’on puisse selectionner les modes voulus ou non.
Masquer en css me plait d’autant moins que le html est toujours là, et le code sous jacent aussi, et permet donc hypothetiquement à un utilisateur malveillant de l’utiliser pour injecter du contenu, en annulant simplement le display:none;

Cédric

Le 10 septembre 2009 09:11, <rastapopoulos@spip.org> a écrit :

Author: rastapopoulos@spip.org
Date: Thu Sep 10 09:10:59 2009
New Revision: 31460

Log:
On ajoute une classe pour chaque type d’ajout de document. Cela permet de masquer rapidement un choix d’ajout sans pour autant tout surcharger.

C’est pratique quand on utilise le formulaire en partie publique, et qu’on ne veut sélectionner que tel ou tel mode. Surtout tant que l’ajout par la médiathèque n’est pas correct en partie publique.

Modified:
plugins/gestion_documents/formulaires/inc-upload_document.html
plugins/gestion_documents/formulaires/joindre_document.html

Modified: plugins/gestion_documents/formulaires/inc-upload_document.html

— plugins/gestion_documents/formulaires/inc-upload_document.html (original)
+++ plugins/gestion_documents/formulaires/inc-upload_document.html Thu Sep 10 09:10:59 2009
@@ -13,12 +13,12 @@

<:gestdoc:bouton_download_depuis:> - <:gestdoc:bouton_download_local:> + <:gestdoc:bouton_download_local:> [(#ENV{mediatheque,''}|oui) - | <:gestdoc:bouton_download_par_mediatheque:>] + | <:gestdoc:bouton_download_par_mediatheque:>] [(#ENV{proposer_ftp,''}|oui) - | <:gestdoc:bouton_download_par_ftp:>] - | <:gestdoc:bouton_download_sur_le_web:> + | <:gestdoc:bouton_download_par_ftp:>] + | <:gestdoc:bouton_download_sur_le_web:>

[ @@ -50,11 +50,11 @@
<:gestdoc:bouton_download_depuis:> - <:gestdoc:bouton_download_local:> - | <:gestdoc:bouton_download_par_mediatheque:> + <:gestdoc:bouton_download_local:> + | <:gestdoc:bouton_download_par_mediatheque:> [(#ENV{proposer_ftp,''}|oui) - | <:gestdoc:bouton_download_par_ftp:>] - | <:gestdoc:bouton_download_sur_le_web:> + | <:gestdoc:bouton_download_par_ftp:>] + | <:gestdoc:bouton_download_sur_le_web:>

@@ -83,11 +83,11 @@ ]
<:gestdoc:bouton_download_depuis:> - <:gestdoc:bouton_download_local:> + <:gestdoc:bouton_download_local:> [(#ENV{mediatheque,''}|oui) - | <:gestdoc:bouton_download_par_mediatheque:>] - | <:gestdoc:bouton_download_par_ftp:> - | <:gestdoc:bouton_download_sur_le_web:> + | <:gestdoc:bouton_download_par_mediatheque:>] + | <:gestdoc:bouton_download_par_ftp:> + | <:gestdoc:bouton_download_sur_le_web:>
[(#ENV*{_options_upload_ftp}|oui)

@@ -109,12 +109,12 @@
<:gestdoc:bouton_download_depuis:> - <:gestdoc:bouton_download_local:> + <:gestdoc:bouton_download_local:> [(#ENV{mediatheque,''}|oui) - | <:gestdoc:bouton_download_par_mediatheque:>] + | <:gestdoc:bouton_download_par_mediatheque:>] [(#ENV{proposer_ftp,''}|oui) - | <:gestdoc:bouton_download_par_ftp:>] - | <:gestdoc:bouton_download_sur_le_web:> + | <:gestdoc:bouton_download_par_ftp:>] + | <:gestdoc:bouton_download_sur_le_web:>

[

Modified: plugins/gestion_documents/formulaires/joindre_document.html

(empty)

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

2

Le 10/09/2009 09:39, Cédric Morin a écrit :

Hello,

Le squelette du formulaire prevoyait deja la possibilité de desactiver
les modes ftp et mediatheque en passant #ENV{mediatheque} et
#ENV{proposer_ftp}

Pourquoi ne pas les utiliser plutot qu'un alourdissement du html (dont
je n'etais deja pas très content ici, mais qui du coup frise
l'horrible). Si necessaire il faut ajouter une option au #FORMULAIRE
correspondant pour que l'on puisse selectionner les modes voulus ou non.
Masquer en css me plait d'autant moins que le html est toujours là, et
le code sous jacent aussi, et permet donc hypothetiquement à un
utilisateur malveillant de l'utiliser pour injecter du contenu, en
annulant simplement le display:none;

Oui, c'est moche.
J'ai enlevé et j'ai ajouté deux options facultatives à la fin du formulaire, qui sont vraies par défaut.

--
RastaPopoulos