From 5359ce8ac74f21efc562461cd7e5874267e94c9b Mon Sep 17 00:00:00 2001
From: "kent1@arscenic.info" <>
Date: Fri, 27 Jan 2017 14:40:04 +0000
Subject: [PATCH] PSR SPIP
---
action/formidable_recuperer_fichier.php | 39 +++++++-------
...formidable_recuperer_fichier_par_email.php | 51 +++++++++----------
corbeille/formulaires_pre_suppression.php | 10 ++--
.../formulaires_reponses_pre_suppression.php | 10 ++--
echanger/formulaire/forms.php | 2 -
5 files changed, 53 insertions(+), 59 deletions(-)
diff --git a/action/formidable_recuperer_fichier.php b/action/formidable_recuperer_fichier.php
index af9220ff..5f2c8674 100644
--- a/action/formidable_recuperer_fichier.php
+++ b/action/formidable_recuperer_fichier.php
@@ -1,51 +1,52 @@
<?php
-if (!defined("_ECRIRE_INC_VERSION")) return;
+
+if (!defined('_ECRIRE_INC_VERSION')) {
+ return;
+}
+
include_spip('inc/autoriser');
include_spip('inc/formidable');
include_spip('inc/formidable_fichiers');
+
/**
- * Récupère, si on est autorisé à voir la réponse du formulaire,
- * un fichier
+ * Récupère, si on est autorisé à voir la réponse du formulaire,
+ * un fichier
* et l'envoi en http
**/
function action_formidable_recuperer_fichier() {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
$arg = unserialize($arg);
-
+
// test si autorisation de voir la réponse par cookie
- $cookie_ok = False;
+ $cookie_ok = false;
if (isset($arg['cookie'])) {
$nom_cookie = formidable_generer_nom_cookie($arg['formulaire']);
if (isset($_COOKIE[$nom_cookie])) {
if ($arg['cookie'] == sha1($_COOKIE[$nom_cookie].secret_du_site())) {
$cookie_bdd = sql_getfetsel('cookie', 'spip_formulaires_reponses', 'id_formulaires_reponse='.sql_quote($arg['reponse']));
if ($cookie_bdd = $_COOKIE[$nom_cookie]) {
- $cookie_ok = True;
+ $cookie_ok = true;
}
}
}
}
if (autoriser('voir', 'formulairesreponse', $arg['formulaire']) or $cookie_ok) {
$chemin_fichier = _DIR_FICHIERS_FORMIDABLE
- ."formulaire_".$arg['formulaire']
- ."/reponse_".$arg['reponse']
- ."/".$arg['saisie']
- ."/".$arg['fichier'];
- if (@file_exists($chemin_fichier)){
+ .'formulaire_'.$arg['formulaire']
+ .'/reponse_'.$arg['reponse']
+ .'/'.$arg['saisie']
+ .'/'.$arg['fichier'];
+ if (@file_exists($chemin_fichier)) {
$f = $arg['fichier'];
formidable_retourner_fichier($chemin_fichier, $f);
- }
- else {
+ } else {
include_spip('inc/minipres');
- echo minipres(_T("formidable:erreur_fichier_introuvable"));
+ echo minipres(_T('formidable:erreur_fichier_introuvable'));
}
} else {
include_spip('inc/minipres');
- echo minipres();
+ echo minipres();
}
-
- exit;
-
+ exit;
}
-
diff --git a/action/formidable_recuperer_fichier_par_email.php b/action/formidable_recuperer_fichier_par_email.php
index 47ecec75..6a324542 100644
--- a/action/formidable_recuperer_fichier_par_email.php
+++ b/action/formidable_recuperer_fichier_par_email.php
@@ -1,5 +1,8 @@
<?php
-if (!defined("_ECRIRE_INC_VERSION")) return;
+
+if (!defined('_ECRIRE_INC_VERSION')) {
+ return;
+}
include_spip('inc/autoriser');
include_spip('inc/formidable_fichiers');
/**
@@ -11,53 +14,47 @@ function action_formidable_recuperer_fichier_par_email() {
$arg = _request('arg');
$hash = _request('hash');
- include_spip("inc/securiser_action");
+ include_spip('inc/securiser_action');
$action = 'formidable_recuperer_fichier_par_email';
$pass = secret_du_site();
if ($hash==_action_auteur("$action-$arg", '', $pass, 'alea_ephemere')
- OR $hash==_action_auteur("$action-$arg", '', $pass, 'alea_ephemere_ancien')) {
-
+ or $hash==_action_auteur("$action-$arg", '', $pass, 'alea_ephemere_ancien')) {
$arg = unserialize($arg);
// Construire le chemin du fichier, en fonction de ce qu'on reçoit
- if (isset($arg['reponse'])) {
+ if (isset($arg['reponse'])) {
$chemin_fichier = _DIR_FICHIERS_FORMIDABLE
- ."formulaire_".$arg['formulaire']
- ."/reponse_".$arg['reponse']
- ."/".$arg['saisie']
- ."/".$arg['fichier'];
+ .'formulaire_'.$arg['formulaire']
+ .'/reponse_'.$arg['reponse']
+ .'/'.$arg['saisie']
+ .'/'.$arg['fichier'];
} elseif (isset($arg['timestamp'])) {
$chemin_fichier = _DIR_FICHIERS_FORMIDABLE
- . "timestamp/"
- . $arg['timestamp']."/"
- . $arg['saisie']."/"
+ . 'timestamp/'
+ . $arg['timestamp'].'/'
+ . $arg['saisie'].'/'
. $arg['fichier'];
} else {
- include_spip('inc/minipres');
- echo minipres(_T("formidable:erreur_fichier_introuvable"));
+ include_spip('inc/minipres');
+ echo minipres(_T('formidable:erreur_fichier_introuvable'));
}
-
+
// Vérifier que le fichier existe, qu'il n'est pas trop vieux, et l'envoyer le cas échéant
- if (@file_exists($chemin_fichier)){
+ if (@file_exists($chemin_fichier)) {
$f = $arg['fichier'];
$date = filemtime($chemin_fichier);
- if (_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL > 0 and $date + _FORMIDABLE_EXPIRATION_FICHIERS_EMAIL < time()) {// vérifier que le fichier n'est pas trop vieux
+ if (_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL > 0 and $date + _FORMIDABLE_EXPIRATION_FICHIERS_EMAIL < time()) {// vérifier que le fichier n'est pas trop vieux
include_spip('inc/minipres');
- echo minipres(_T("formidable:erreur_fichier_expire"));
+ echo minipres(_T('formidable:erreur_fichier_expire'));
} else {
formidable_retourner_fichier($chemin_fichier, $f);
}
- }
- else {
+ } else {
include_spip('inc/minipres');
- echo minipres(_T("formidable:erreur_fichier_introuvable"));
+ echo minipres(_T('formidable:erreur_fichier_introuvable'));
}
} else {
-
include_spip('inc/minipres');
- echo minipres();
+ echo minipres();
}
-
- exit;
-
+ exit;
}
-
diff --git a/corbeille/formulaires_pre_suppression.php b/corbeille/formulaires_pre_suppression.php
index 82cbd8d3..0900ab0c 100644
--- a/corbeille/formulaires_pre_suppression.php
+++ b/corbeille/formulaires_pre_suppression.php
@@ -1,7 +1,9 @@
<?php
+
if (!defined('_ECRIRE_INC_VERSION')) {
return;
}
+
include_spip('inc/formidable');
include_spip('base/abstract_sql');
@@ -12,15 +14,13 @@ include_spip('base/abstract_sql');
**/
function corbeille_formulaires_pre_suppression_dist($ids) {
foreach ($ids as $id_formulaire) {
- $res = sql_select('id_formulaires_reponse','spip_formulaires_reponses',"id_formulaire=$id_formulaire");
- while ($champ = sql_fetch($res)) {
+ $res = sql_select('id_formulaires_reponse', 'spip_formulaires_reponses', 'id_formulaire='.intval($id_formulaire));
+ while ($champ = sql_fetch($res)) {
$id_formulaires_reponse = $champ['id_formulaires_reponse'];
sql_delete('spip_formulaires_reponses_champs', "id_formulaires_reponse=$id_formulaires_reponse");
sql_delete('spip_formulaires_reponses', "id_formulaires_reponse=$id_formulaires_reponse");
}
formidable_effacer_fichiers_formulaire($id_formulaire);
- spip_log("Effacement des réponses du formulaire $id_formulaire via la corbeille", "formidable");
+ spip_log("Effacement des réponses du formulaire $id_formulaire via la corbeille", 'formidable');
}
}
-
-
diff --git a/corbeille/formulaires_reponses_pre_suppression.php b/corbeille/formulaires_reponses_pre_suppression.php
index 07f5d84f..57202f8b 100644
--- a/corbeille/formulaires_reponses_pre_suppression.php
+++ b/corbeille/formulaires_reponses_pre_suppression.php
@@ -11,12 +11,10 @@ include_spip('base/abstract_sql');
*
**/
function corbeille_formulaires_reponses_pre_suppression_dist($ids) {
- foreach ($ids as $id_formulaires_reponse){
- $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires_reponses',"id_formulaires_reponse=".$id_formulaires_reponse);
- formidable_effacer_fichiers_reponse($id_formulaire, $id_formulaires_reponse);
+ foreach ($ids as $id_formulaires_reponse) {
+ $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires_reponses', 'id_formulaires_reponse='.intval($id_formulaires_reponse));
+ formidable_effacer_fichiers_reponse($id_formulaire, $id_formulaires_reponse);
sql_delete('spip_formulaires_reponses_champs', "id_formulaires_reponse=$id_formulaires_reponse");
- spip_log("Effacement des champs de la réponse $id_formulaires_reponse via la corbeille", "formidable");
+ spip_log("Effacement des champs de la réponse $id_formulaires_reponse via la corbeille", 'formidable');
}
}
-
-
diff --git a/echanger/formulaire/forms.php b/echanger/formulaire/forms.php
index 0f18687c..8c0080cf 100644
--- a/echanger/formulaire/forms.php
+++ b/echanger/formulaire/forms.php
@@ -214,10 +214,8 @@ function forms_configure_traitement_formulaire($form, &$formulaire) {
'modifiable' => ($form['modifiable'] == 'oui') ? 'on' : '',
'multiple' => ($form['multiple'] == 'non') ? '' : 'on'
);
-
}
-
/**
* On a genere un fieldset pour chaque separateur de f&t
* il faut le peupler avec les saisies qui le suivent
--
GitLab