From dc4c59a29258ffe69a43a0bd28af0bd446f2d620 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ma=C3=AFeul=20Rouquette?= <maieul@maieul.net> Date: Fri, 15 Sep 2023 17:14:03 +0200 Subject: [PATCH] style: un coup de PHPcbf --- .gitattributes | 1 + action/dupliquer_formulaire.php | 24 +- action/editer_formulaire.php | 44 +-- action/exporter_formulaire.php | 3 +- action/formidable_recuperer_fichier.php | 14 +- ...formidable_recuperer_fichier_par_email.php | 4 +- action/supprimer_formulaire.php | 2 +- action/supprimer_formulaires_reponse.php | 2 +- action/vider_formulaire.php | 8 +- afficher_diff/array_serialized.php | 2 - afficher_diff/formulaire_saisies.php | 2 - afficher_diff/formulaire_traitements.php | 2 - base/formidable_tables.php | 148 ++++----- controleurs/formulaires_reponses_champ.php | 8 +- corbeille/formulaires_pre_suppression.php | 2 +- .../formulaires_reponses_pre_suppression.php | 3 +- echanger/formulaire/forms.php | 130 ++++---- echanger/formulaire/wcs.php | 38 +-- echanger/formulaire/yaml.php | 12 +- formidable_administrations.php | 282 +++++++++--------- formidable_autorisations.php | 9 +- formidable_fonctions.php | 79 ++--- formidable_pipelines.php | 98 +++--- formulaires/editer_formulaire.php | 92 +++--- formulaires/editer_formulaire_champs.php | 168 +++++------ formulaires/editer_formulaire_traitements.php | 26 +- formulaires/editer_formulaires_reponse.php | 2 +- formulaires/exporter_formulaire_analyse.php | 31 +- formulaires/exporter_formulaire_reponses.php | 75 +++-- formulaires/formidable.php | 122 ++++---- formulaires/importer_formulaire.php | 10 +- genie/formidable_effacer_enregistrements.php | 4 +- genie/formidable_effacer_fichiers_email.php | 1 - genie/formidable_hasher_ip.php | 9 +- http/collectionjson/formulaires_reponses.php | 38 +-- inc/formidable.php | 116 +++---- inc/formidable_fichiers.php | 148 ++++----- inc/formidable_migrer_depuis_ft.php | 51 ++-- ...res_reponse_formulaires_reponses_champ.php | 2 +- phpcs.xml.dist | 16 + .../liste/formulaires_associer_fonctions.php | 1 + .../liste/formulaires_lies_fonctions.php | 1 + public/formidable_criteres.php | 30 +- spip-cli/FormidableExporterFormulaire.php | 14 +- traiter/email.php | 120 ++++---- traiter/enregistrement.php | 72 ++--- vues/formulaires_reponses_champ.php | 2 +- 47 files changed, 1058 insertions(+), 1010 deletions(-) create mode 100644 .gitattributes create mode 100644 phpcs.xml.dist diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 00000000..876174e2 --- /dev/null +++ b/.gitattributes @@ -0,0 +1 @@ +/phpcs.xml.dist export-ignore diff --git a/action/dupliquer_formulaire.php b/action/dupliquer_formulaire.php index afab7a44..55413ae6 100644 --- a/action/dupliquer_formulaire.php +++ b/action/dupliquer_formulaire.php @@ -20,35 +20,39 @@ function action_dupliquer_formulaire_dist($arg = null) { $id_formulaire = intval($arg); // Si le formulaire existe bien - if ($id_formulaire > 0 - and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id_formulaire)) { + if ( + $id_formulaire > 0 + and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire) + ) { include_spip('action/editer_formulaire'); // On enlève les champs inutiles unset($formulaire['id_formulaire']); // On modifie un peu le titre - $formulaire['titre'] = $formulaire['titre'].' '._T('formidable:formulaires_dupliquer_copie'); + $formulaire['titre'] = $formulaire['titre'] . ' ' . _T('formidable:formulaires_dupliquer_copie'); // On s'assure que l'identifiant n'existe pas déjà - $formulaire['identifiant'] = $formulaire['identifiant'].'_'.time(); + $formulaire['identifiant'] = $formulaire['identifiant'] . '_' . time(); // On insère un nouveau formulaire $id_formulaire = insert_formulaire(); // Si ça a marché on modifie les champs de base - if ($id_formulaire > 0 - and !($erreur = formulaire_set($id_formulaire, $formulaire))) { + if ( + $id_formulaire > 0 + and !($erreur = formulaire_set($id_formulaire, $formulaire)) + ) { // Et ensuite les saisies et les traitements sql_updateq( 'spip_formulaires', - array( + [ 'saisies' => $formulaire['saisies'], 'traitements' => $formulaire['traitements'] - ), - 'id_formulaire = '.$id_formulaire + ], + 'id_formulaire = ' . $id_formulaire ); // Tout a fonctionné. En fonction de la config, on attribue l'auteur courant $auteurs = lire_config('formidable/auteur'); if ($auteurs == 'on') { if ($id_auteur = session_get('id_auteur')) { // association (par défaut) du formulaire et de l'auteur courant - objet_associer(array('formulaire'=>$id_formulaire), array('auteur'=>$id_auteur)); + objet_associer(['formulaire' => $id_formulaire], ['auteur' => $id_auteur]); } } // Et on redirige vers la vue diff --git a/action/editer_formulaire.php b/action/editer_formulaire.php index 4c601c52..2055a33b 100644 --- a/action/editer_formulaire.php +++ b/action/editer_formulaire.php @@ -37,7 +37,7 @@ function action_editer_formulaire_dist($arg = null) { include_spip('inc/headers'); redirige_par_entete($redirect); } else { - return array($id_formulaire,$err); + return [$id_formulaire,$err]; } } @@ -47,31 +47,31 @@ function action_editer_formulaire_dist($arg = null) { * @return int id_formulaire */ function formulaire_inserer() { - $champs = array( + $champs = [ 'statut' => 'prop', 'date_creation' => date('Y-m-d H:i:s'), - ); + ]; // Envoyer aux plugins $champs = pipeline( 'pre_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_formulaires', - ), + ], 'data' => $champs - ) + ] ); $id_formulaire = sql_insertq('spip_formulaires', $champs); pipeline( 'post_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_formulaires', 'id_objet' => $id_formulaire - ), + ], 'data' => $champs - ) + ] ); return $id_formulaire; @@ -93,28 +93,30 @@ function formulaire_modifier($id_formulaire, $set = null) { // white list objet_info('formulaire', 'champs_editables'), // black list - array('statut'), + ['statut'], // donnees eventuellement fournies $set ); $invalideur = "id='id_formulaire/$id_formulaire'"; - if ($err = objet_modifier_champs( - 'formulaire', - $id_formulaire, - array( + if ( + $err = objet_modifier_champs( + 'formulaire', + $id_formulaire, + [ 'data' => $set, - 'nonvide' => array('titre' => _T('info_sans_titre')), + 'nonvide' => ['titre' => _T('info_sans_titre')], 'invalideur' => $invalideur, - ), - $c - )) { + ], + $c + ) + ) { return $err; } // Modification de statut, changement de rubrique ? - $c = collecter_requests(array('statut'), array(), $set); + $c = collecter_requests(['statut'], [], $set); include_spip('action/editer_objet'); $err = objet_instituer('formulaire', $id_formulaire, $c); diff --git a/action/exporter_formulaire.php b/action/exporter_formulaire.php index d32fa506..77c51ed9 100644 --- a/action/exporter_formulaire.php +++ b/action/exporter_formulaire.php @@ -14,7 +14,8 @@ function action_exporter_formulaire_dist($arg = null) { list($id_formulaire, $type_export) = preg_split('/[\W]/', $arg); - if ($id_formulaire > 0 + if ( + $id_formulaire > 0 and $type_export and $types_echange = echanges_formulaire_lister_disponibles() and $exporter = $types_echange['exporter'][$type_export] diff --git a/action/formidable_recuperer_fichier.php b/action/formidable_recuperer_fichier.php index 97d4cb91..713bd217 100644 --- a/action/formidable_recuperer_fichier.php +++ b/action/formidable_recuperer_fichier.php @@ -17,14 +17,14 @@ function action_formidable_recuperer_fichier() { $securiser_action = charger_fonction('securiser_action', 'inc'); $arg = $securiser_action(); $arg = json_decode($arg, true); - spip_log('Action recuperer_fichier : le arg est'.print_r($arg, true), 'formidable'._LOG_DEBUG); + spip_log('Action recuperer_fichier : le arg est' . print_r($arg, true), 'formidable' . _LOG_DEBUG); // test si autorisation de voir la réponse par cookie $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 ($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; } @@ -33,10 +33,10 @@ function action_formidable_recuperer_fichier() { } if (autoriser('voir', 'formulairesreponse', $arg['formulaire']) or $cookie_ok) { $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']; if (@file_exists($chemin_fichier)) { $f = $arg['fichier']; formidable_retourner_fichier($chemin_fichier, $f); diff --git a/action/formidable_recuperer_fichier_par_email.php b/action/formidable_recuperer_fichier_par_email.php index 45795b65..8adce86b 100644 --- a/action/formidable_recuperer_fichier_par_email.php +++ b/action/formidable_recuperer_fichier_par_email.php @@ -17,8 +17,8 @@ function action_formidable_recuperer_fichier_par_email() { include_spip('inc/securiser_action'); $action = 'formidable_recuperer_fichier_par_email'; $pass = secret_du_site(); - spip_log("Action recuperer_fichier_par_email : le arg est $arg et le hash $hash.", "formidable"._LOG_DEBUG); - spip_log("Du coup le resultat de action_auteur est ". _action_auteur("$action-$arg", 0, $pass, 'secret_de_formidable'), "formidable"._LOG_DEBUG); + spip_log("Action recuperer_fichier_par_email : le arg est $arg et le hash $hash.", 'formidable' . _LOG_DEBUG); + spip_log('Du coup le resultat de action_auteur est ' . _action_auteur("$action-$arg", 0, $pass, 'secret_de_formidable'), 'formidable' . _LOG_DEBUG); if ($hash == _action_auteur("$action-$arg", 0, $pass, 'secret_de_formidable')) { if (_request('base64')) { $arg = base64_decode($arg); diff --git a/action/supprimer_formulaire.php b/action/supprimer_formulaire.php index 00dbc407..6ae9f23c 100644 --- a/action/supprimer_formulaire.php +++ b/action/supprimer_formulaire.php @@ -18,7 +18,7 @@ function action_supprimer_formulaire_dist($arg = null) { // si id_formulaire n'est pas un nombre, on ne fait rien if ($id_formulaire = intval($arg)) { - $set = array('statut' => 'poubelle'); + $set = ['statut' => 'poubelle']; include_spip('action/editer_formulaire'); formulaire_modifier($id_formulaire, $set); } diff --git a/action/supprimer_formulaires_reponse.php b/action/supprimer_formulaires_reponse.php index 985e01fa..9bddc8ab 100644 --- a/action/supprimer_formulaires_reponse.php +++ b/action/supprimer_formulaires_reponse.php @@ -19,7 +19,7 @@ function action_supprimer_formulaires_reponse_dist($arg = null) { // si id_formulaires_reponse n'est pas un nombre, on ne fait rien if ($id_formulaires_reponse = intval($arg)) { include_spip('action/editer_objet'); - $set = array('statut' => 'refuse'); + $set = ['statut' => 'refuse']; objet_modifier('formulaires_reponse', $id_formulaires_reponse, $set); } } diff --git a/action/vider_formulaire.php b/action/vider_formulaire.php index 8ab7bb8a..548e4bb6 100644 --- a/action/vider_formulaire.php +++ b/action/vider_formulaire.php @@ -18,12 +18,14 @@ function action_vider_formulaire_dist($arg = null) { include_spip('inc/autoriser'); // si id_formulaires_reponse n'est pas un nombre, on ne fait rien - if ($id_formulaire = intval($arg) - and autoriser('instituer', 'formulairesreponse', $id_formulaire)) { + if ( + $id_formulaire = intval($arg) + and autoriser('instituer', 'formulairesreponse', $id_formulaire) + ) { // On supprime les réponse (statut => refuse) $ok = sql_updateq( 'spip_formulaires_reponses', - array('statut' => 'refuse'), + ['statut' => 'refuse'], 'id_formulaire=' . intval($id_formulaire) ); diff --git a/afficher_diff/array_serialized.php b/afficher_diff/array_serialized.php index 0956bd2e..26f81abd 100644 --- a/afficher_diff/array_serialized.php +++ b/afficher_diff/array_serialized.php @@ -34,5 +34,3 @@ function afficher_diff_array_serialized($champ, $old, $new, $format = 'diff') { $afficher_diff = charger_fonction('champ', 'afficher_diff', true); return $afficher_diff($champ, $old, $new, 'complet'); } - - diff --git a/afficher_diff/formulaire_saisies.php b/afficher_diff/formulaire_saisies.php index 47234ef3..a1f9fd01 100644 --- a/afficher_diff/formulaire_saisies.php +++ b/afficher_diff/formulaire_saisies.php @@ -19,5 +19,3 @@ function afficher_diff_formulaire_saisies($champ, $old, $new, $format = 'diff') $afficher_diff = charger_fonction('array_serialized', 'afficher_diff', true); return $afficher_diff($champ, $old, $new, 'complet'); } - - diff --git a/afficher_diff/formulaire_traitements.php b/afficher_diff/formulaire_traitements.php index ed9d5007..931ca99b 100644 --- a/afficher_diff/formulaire_traitements.php +++ b/afficher_diff/formulaire_traitements.php @@ -19,5 +19,3 @@ function afficher_diff_formulaire_traitements($champ, $old, $new, $format = 'dif $afficher_diff = charger_fonction('array_serialized', 'afficher_diff', true); return $afficher_diff($champ, $old, $new, 'complet'); } - - diff --git a/base/formidable_tables.php b/base/formidable_tables.php index 5324f13c..8dfa0bdd 100644 --- a/base/formidable_tables.php +++ b/base/formidable_tables.php @@ -31,10 +31,10 @@ function formidable_declarer_tables_interfaces($interfaces) { $interfaces['tables_jointures']['spip_articles'][] = 'formulaires_liens'; $interfaces['tables_jointures']['spip_rubriques'][] = 'formulaires_liens'; - $interfaces['exceptions_des_jointures']['spip_formulaires']['vu'] = array('spip_formulaires_liens', 'vu'); + $interfaces['exceptions_des_jointures']['spip_formulaires']['vu'] = ['spip_formulaires_liens', 'vu']; - $interfaces['table_des_traitements']['MESSAGE_RETOUR']['formulaires']= _TRAITEMENT_RACCOURCIS; - $interfaces['table_des_traitements']['URL_REDIRECT']['formulaires']= 'vider_url(%s)'; + $interfaces['table_des_traitements']['MESSAGE_RETOUR']['formulaires'] = _TRAITEMENT_RACCOURCIS; + $interfaces['table_des_traitements']['URL_REDIRECT']['formulaires'] = 'vider_url(%s)'; return $interfaces; @@ -51,21 +51,21 @@ function formidable_declarer_tables_interfaces($interfaces) { */ function formidable_declarer_tables_objets_sql($tables) { include_spip('inc/config'); - $tables['spip_formulaires'] = array( - 'type'=>'formulaire', + $tables['spip_formulaires'] = [ + 'type' => 'formulaire', 'titre' => "titre, '' AS lang", 'date' => 'date_creation', 'principale' => 'oui', 'texte_modifier' => 'formidable:editer_modifier_formulaire', - 'info_aucun_objet'=> 'formidable:info_aucun_formulaire', + 'info_aucun_objet' => 'formidable:info_aucun_formulaire', 'info_1_objet' => 'formidable:info_1_formulaire', 'info_nb_objets' => 'formidable:info_nb_formulaires', 'texte_creer_associer' => 'formidable:liens_creer_associer', 'texte_ajouter' => 'formidable:liens_ajouter', - 'champs_editables' => array('titre', 'identifiant', 'descriptif', 'css', 'message_retour', 'saisies', 'traitements', 'public','apres','url_redirect','resume_reponse'), - 'champs_versionnes' => array('titre', 'identifiant', 'descriptif', 'css', 'message_retour', 'saisies', 'traitements', 'public','apres','url_redirect','resume_reponse'), - 'field' => array( + 'champs_editables' => ['titre', 'identifiant', 'descriptif', 'css', 'message_retour', 'saisies', 'traitements', 'public','apres','url_redirect','resume_reponse'], + 'champs_versionnes' => ['titre', 'identifiant', 'descriptif', 'css', 'message_retour', 'saisies', 'traitements', 'public','apres','url_redirect','resume_reponse'], + 'field' => [ 'id_formulaire' => 'bigint(21) NOT NULL', 'identifiant' => 'varchar(200)', 'titre' => "text NOT NULL default ''", @@ -80,54 +80,54 @@ function formidable_declarer_tables_objets_sql($tables) { 'statut' => "varchar(10) NOT NULL default ''", 'date_creation' => "datetime NOT NULL DEFAULT '0000-00-00 00:00:00'", 'maj' => 'timestamp', - ), - 'key' => array( + ], + 'key' => [ 'PRIMARY KEY' => 'id_formulaire' - ), - 'join'=> array( + ], + 'join' => [ 'id_formulaire' => 'id_formulaire' - ), - 'statut'=> array( - array( + ], + 'statut' => [ + [ 'champ' => 'statut', 'publie' => 'publie', 'previsu' => 'publie,prop', - 'exception' => array('statut', 'tout'), - ) - ), + 'exception' => ['statut', 'tout'], + ] + ], 'texte_changer_statut' => 'formidable:changer_statut', - 'rechercher_champs' => array( + 'rechercher_champs' => [ 'titre' => 5, 'descriptif' => 3, 'identifiant' => 3, - ), - 'statut_titres' => array( - 'prop'=>'formidable:info_formulaire_propose', - 'publie'=>'formidable:info_formulaire_publie', - 'refuse'=>'formidable:info_formulaire_refuse', - 'poubelle'=>'formidable:info_formulaire_supprime' - ), - 'statut_textes_instituer' => array( + ], + 'statut_titres' => [ + 'prop' => 'formidable:info_formulaire_propose', + 'publie' => 'formidable:info_formulaire_publie', + 'refuse' => 'formidable:info_formulaire_refuse', + 'poubelle' => 'formidable:info_formulaire_supprime' + ], + 'statut_textes_instituer' => [ 'prop' => 'texte_statut_propose_evaluation', 'publie' => 'texte_statut_publie', - 'refuse'=>'formidable:texte_statut_refuse', + 'refuse' => 'formidable:texte_statut_refuse', 'poubelle' => 'texte_statut_poubelle', - ), - 'page' => (lire_config('formidable/activer_pages')=='on') ? 'formulaire' : false, + ], + 'page' => (lire_config('formidable/activer_pages') == 'on') ? 'formulaire' : false, - ); + ]; - $tables['spip_formulaires_reponses'] = array( - 'type_objet'=>'formulaires_reponse', - 'table_objet_surnoms' => array('formulairesreponse'), - 'type'=>'formulaires_reponse', - 'type_surnoms' => array('formulairesreponse'), + $tables['spip_formulaires_reponses'] = [ + 'type_objet' => 'formulaires_reponse', + 'table_objet_surnoms' => ['formulairesreponse'], + 'type' => 'formulaires_reponse', + 'type_surnoms' => ['formulairesreponse'], 'titre' => "'' AS titre, '' AS lang", 'date' => 'date', 'principale' => 'oui', - 'info_aucun_objet'=> 'formidable:info_aucune_reponse', + 'info_aucun_objet' => 'formidable:info_aucune_reponse', 'info_1_objet' => 'formidable:info_1_reponse', 'info_nb_objets' => 'formidable:info_nb_reponses', - 'field' => array( + 'field' => [ 'id_formulaires_reponse' => 'bigint(21) NOT NULL', 'id_formulaire' => 'bigint(21) NOT NULL default 0', 'date' => "datetime NOT NULL default '0000-00-00 00:00:00'", @@ -138,46 +138,46 @@ function formidable_declarer_tables_objets_sql($tables) { 'variable_php' => "varchar(255) NOT NULL default ''", 'statut' => "varchar(10) NOT NULL default ''", 'maj' => 'timestamp' - ), - 'champs_versionnes' => array('date_envoi'), - 'key' => array( + ], + 'champs_versionnes' => ['date_envoi'], + 'key' => [ 'PRIMARY KEY' => 'id_formulaires_reponse', 'KEY id_formulaire' => 'id_formulaire', 'KEY id_auteur' => 'id_auteur', 'KEY cookie' => 'cookie', 'KEY variable_php' => 'variable_php', - ), - 'join' => array( + ], + 'join' => [ 'id_formulaires_reponse' => 'id_formulaires_reponse', 'id_formulaire' => 'id_formulaire', 'id_auteur' => 'id_auteur' - ), - 'statut'=> array( - array( + ], + 'statut' => [ + [ 'champ' => 'statut', 'publie' => 'publie', 'previsu' => 'publie,prop', - 'exception' => array('statut', 'tout'), - ) - ), + 'exception' => ['statut', 'tout'], + ] + ], 'texte_changer_statut' => 'formulaires_reponse:changer_statut', - 'rechercher_champs' => array('ip'=>1), - 'rechercher_jointures' => array( - 'formulaires_reponses_champ' => array('valeur' => 4), - ), - 'statut_titres' => array( - 'prop'=>'formidable:info_reponse_proposee', - 'publie'=>'formidable:info_reponse_publiee', - 'refuse'=>'formidable:info_reponse_refusee', + 'rechercher_champs' => ['ip' => 1], + 'rechercher_jointures' => [ + 'formulaires_reponses_champ' => ['valeur' => 4], + ], + 'statut_titres' => [ + 'prop' => 'formidable:info_reponse_proposee', + 'publie' => 'formidable:info_reponse_publiee', + 'refuse' => 'formidable:info_reponse_refusee', 'poubelle' => 'formidable:info_reponse_poubelle' - ), - 'statut_textes_instituer' => array( + ], + 'statut_textes_instituer' => [ 'prop' => 'formidable:texte_statut_propose_evaluation', 'publie' => 'formidable:texte_statut_publie', 'refuse' => 'formidable:texte_statut_refusee', 'poubelle' => 'formidable:texte_statut_poubelle' - ), - ); + ], + ]; // Jointures sur les formulaires pour tous les objets $tables[]['tables_jointures'][] = 'formulaires_liens'; @@ -197,23 +197,23 @@ function formidable_declarer_tables_objets_sql($tables) { function formidable_declarer_tables_principales($tables_principales) { // Table formulaires_reponses_champs - $formulaires_reponses_champs = array( + $formulaires_reponses_champs = [ 'id_formulaires_reponses_champ' => 'bigint(21) NOT NULL', 'id_formulaires_reponse' => 'bigint(21) NOT NULL default 0', 'nom' => "varchar(255) NOT NULL default ''", 'valeur' => "text NOT NULL DEFAULT ''", 'maj' => 'timestamp' - ); - $formulaires_reponses_champs_cles = array( + ]; + $formulaires_reponses_champs_cles = [ 'PRIMARY KEY' => 'id_formulaires_reponses_champ', 'KEY id_formulaires_reponse' => 'id_formulaires_reponse', 'KEY nom' => 'nom', 'UNIQUE reponse' => 'id_formulaires_reponse,nom', - ); - $tables_principales['spip_formulaires_reponses_champs'] = array( + ]; + $tables_principales['spip_formulaires_reponses_champs'] = [ 'field' => &$formulaires_reponses_champs, 'key' => &$formulaires_reponses_champs_cles - ); + ]; return $tables_principales; } @@ -228,22 +228,22 @@ function formidable_declarer_tables_principales($tables_principales) { * Description complétée des tables **/ function formidable_declarer_tables_auxiliaires($tables_auxiliaires) { - $formulaires_liens = array( + $formulaires_liens = [ 'id_formulaire' => "bigint(21) DEFAULT '0' NOT NULL", 'id_objet' => "bigint(21) DEFAULT '0' NOT NULL", 'objet' => "VARCHAR (25) DEFAULT '' NOT NULL", 'vu' => "ENUM('non', 'oui') DEFAULT 'non' NOT NULL", - ); + ]; - $formulaires_liens_cles = array( + $formulaires_liens_cles = [ 'PRIMARY KEY' => 'id_formulaire,id_objet,objet', 'KEY id_formulaire' => 'id_formulaire' - ); + ]; - $tables_auxiliaires['spip_formulaires_liens'] = array( + $tables_auxiliaires['spip_formulaires_liens'] = [ 'field' => &$formulaires_liens, 'key' => &$formulaires_liens_cles - ); + ]; return $tables_auxiliaires; } diff --git a/controleurs/formulaires_reponses_champ.php b/controleurs/formulaires_reponses_champ.php index 73073ed9..30a9df63 100644 --- a/controleurs/formulaires_reponses_champ.php +++ b/controleurs/formulaires_reponses_champ.php @@ -31,14 +31,14 @@ function controleurs_formulaires_reponses_champ_dist($regs, $c = null) { $valeur = $data['valeur']; $saisie = saisies_chercher( formidable_deserialize($data['saisies']), - $nom + $nom ); $valeur = $data['valeur']; $n = new Crayon( $type . '-valeur-' . $id_formulaires_reponses_champ, $valeur, - array('controleur' => 'controleurs/formulaires_reponses_champ') + ['controleur' => 'controleurs/formulaires_reponses_champ'] ); $key = $n->key; @@ -51,7 +51,7 @@ function controleurs_formulaires_reponses_champ_dist($regs, $c = null) { $nom_crayons = 'content_' . $key . '_valeur'; $saisie['options']['nom'] = $nom_crayons; $valeur = formidable_deserialize($valeur); - $contexte = array('_saisies' => array($saisie), $nom_crayons => $valeur); + $contexte = ['_saisies' => [$saisie], $nom_crayons => $valeur]; $html = $n->formulaire($contexte); $status = $scripts = null; @@ -62,5 +62,5 @@ function controleurs_formulaires_reponses_champ_dist($regs, $c = null) { $scripts = '<link rel="stylesheet" type="text/css" href="' . find_in_path('css/ui/jquery-ui.css') . '" />'; } - return array($html . $scripts, $status); + return [$html . $scripts, $status]; } diff --git a/corbeille/formulaires_pre_suppression.php b/corbeille/formulaires_pre_suppression.php index 090c2bcd..dc1fb6b9 100644 --- a/corbeille/formulaires_pre_suppression.php +++ b/corbeille/formulaires_pre_suppression.php @@ -14,7 +14,7 @@ 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='.intval($id_formulaire)); + $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"); diff --git a/corbeille/formulaires_reponses_pre_suppression.php b/corbeille/formulaires_reponses_pre_suppression.php index 57202f8b..02128d9f 100644 --- a/corbeille/formulaires_reponses_pre_suppression.php +++ b/corbeille/formulaires_reponses_pre_suppression.php @@ -1,4 +1,5 @@ <?php + if (!defined('_ECRIRE_INC_VERSION')) { return; } @@ -12,7 +13,7 @@ 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='.intval($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'); diff --git a/echanger/formulaire/forms.php b/echanger/formulaire/forms.php index 00a484f2..f093874f 100644 --- a/echanger/formulaire/forms.php +++ b/echanger/formulaire/forms.php @@ -14,8 +14,8 @@ function echanger_formulaire_forms_importer_dist($fichier) { if ($arbre and is_array($arbre) and isset($arbre['forms'])) { foreach ($arbre['forms'] as $forms) { foreach ($forms['form'] as $form) { - $formulaire = array(); - $form_source = array( + $formulaire = []; + $form_source = [ 'id_form' => intval(trim(spip_xml_aplatit($form['id_form']))), 'titre' => trim(spip_xml_aplatit($form['titre'])), 'descriptif' => trim(spip_xml_aplatit($form['descriptif'])), @@ -25,7 +25,7 @@ function echanger_formulaire_forms_importer_dist($fichier) { 'modifiable' => trim(spip_xml_aplatit($form['modifiable'])), 'multiple' => trim(spip_xml_aplatit($form['multiple'])), 'champconfirm' => trim(spip_xml_aplatit($form['champconfirm'])), - ); + ]; // configurer le formulaire (titre etc) forms_configure_formulaire($form_source, $formulaire); @@ -33,26 +33,26 @@ function echanger_formulaire_forms_importer_dist($fichier) { // ajouter les champs de saisies foreach ($form['fields'] as $fields) { foreach ($fields['field'] as $field) { - $champ = array( - 'champ'=>trim(spip_xml_aplatit($field['champ'])), - 'titre'=>trim(spip_xml_aplatit($field['titre'])), - 'type'=>trim(spip_xml_aplatit($field['type'])), - 'obligatoire'=>trim(spip_xml_aplatit($field['obligatoire'])), - 'taille'=>trim(spip_xml_aplatit($field['taille'])), - 'extra_info'=>trim(spip_xml_aplatit($field['extra_info'])), - 'aide'=>trim(spip_xml_aplatit($field['aide'])), - 'saisie'=>trim(spip_xml_aplatit($field['saisie'])), - ); + $champ = [ + 'champ' => trim(spip_xml_aplatit($field['champ'])), + 'titre' => trim(spip_xml_aplatit($field['titre'])), + 'type' => trim(spip_xml_aplatit($field['type'])), + 'obligatoire' => trim(spip_xml_aplatit($field['obligatoire'])), + 'taille' => trim(spip_xml_aplatit($field['taille'])), + 'extra_info' => trim(spip_xml_aplatit($field['extra_info'])), + 'aide' => trim(spip_xml_aplatit($field['aide'])), + 'saisie' => trim(spip_xml_aplatit($field['saisie'])), + ]; // Les choix pour les types select et multiple if (isset($field['les_choix']) and is_array($field['les_choix'])) { - $champ['choix'] = array(); + $champ['choix'] = []; foreach ($field['les_choix'] as $les_choix) { foreach ($les_choix['un_choix'] as $un_choix) { - $champ['choix'][] = array( - 'choix'=>trim(spip_xml_aplatit($un_choix['choix'])), - 'titre'=>trim(spip_xml_aplatit($un_choix['titre'])), - ); + $champ['choix'][] = [ + 'choix' => trim(spip_xml_aplatit($un_choix['choix'])), + 'titre' => trim(spip_xml_aplatit($un_choix['titre'])), + ]; } } } @@ -86,34 +86,36 @@ function forms_importe_en_base($formulaire) { include_spip('action/editer_formulaire'); // On insère un nouveau formulaire // cas utilise par l'installation/import f&t - if (isset($formulaire['id_formulaire']) - and !sql_countsel('spip_formulaires', 'id_formulaire='.intval($formulaire['id_formulaire']))) { - $champs = array( + if ( + isset($formulaire['id_formulaire']) + and !sql_countsel('spip_formulaires', 'id_formulaire=' . intval($formulaire['id_formulaire'])) + ) { + $champs = [ 'id_formulaire' => $formulaire['id_formulaire'], 'statut' => 'prop', 'date_creation' => date('Y-m-d H:i:s'), - ); + ]; // Envoyer aux plugins $champs = pipeline( 'pre_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_formulaires', - ), + ], 'data' => $champs - ) + ] ); $id_formulaire = sql_insertq('spip_formulaires', $champs); pipeline( 'post_insertion', - array( - 'args' => array( + [ + 'args' => [ 'table' => 'spip_formulaires', 'id_objet' => $id_formulaire - ), + ], 'data' => $champs - ) + ] ); } else { $id_formulaire = formulaire_inserer(); @@ -130,13 +132,15 @@ function forms_importe_en_base($formulaire) { // si l'identifiant existe deja (multiples imports du meme form) // le dater - if (sql_countsel('spip_formulaires', 'identifiant='.sql_quote($formulaire['identifiant']))) { - $formulaire['identifiant'] .= '_'.date('Ymd_His'); + if (sql_countsel('spip_formulaires', 'identifiant=' . sql_quote($formulaire['identifiant']))) { + $formulaire['identifiant'] .= '_' . date('Ymd_His'); } // Si ok on modifie les champs de base - if ($id_formulaire > 0 - and !($erreur = formulaire_modifier($id_formulaire, $formulaire))) { + if ( + $id_formulaire > 0 + and !($erreur = formulaire_modifier($id_formulaire, $formulaire)) + ) { return $id_formulaire; } @@ -156,9 +160,9 @@ function forms_configure_formulaire($form, &$formulaire) { // Generer un identifiant // si id_form fourni, on s'en sert if (isset($form['id_form'])) { - $formulaire['identifiant'] = 'form_import_'.$form['id_form']; + $formulaire['identifiant'] = 'form_import_' . $form['id_form']; } else { - $formulaire['identifiant'] = 'form_import_'.preg_replace(',\W,', '_', strtolower($formulaire['titre'])); + $formulaire['identifiant'] = 'form_import_' . preg_replace(',\W,', '_', strtolower($formulaire['titre'])); } // Le descriptif @@ -168,11 +172,11 @@ function forms_configure_formulaire($form, &$formulaire) { $formulaire['message_retour'] = (isset($form['texte']) ? $form['texte'] : ''); if (!isset($formulaire['traitements'])) { - $formulaire['traitements'] = array(); + $formulaire['traitements'] = []; } if (!isset($formulaire['saisies'])) { - $formulaire['saisies'] = array(); + $formulaire['saisies'] = []; } } @@ -186,19 +190,19 @@ function forms_configure_traitement_formulaire($form, &$formulaire) { // Le traitement email if ($form['champconfirm']) { if (!isset($formulaire['traitements']['email'])) { - $formulaire['traitements']['email'] = array(); + $formulaire['traitements']['email'] = []; } $formulaire['traitements']['email']['champ_courriel_destinataire_form'] = $form['champconfirm']; } // $form['email'] est possiblement serialize - if (is_string($form['email']) and $a=unserialize($form['email'])) { + if (is_string($form['email']) and $a = unserialize($form['email'])) { $form['email'] = $a; } if (is_array($form['email'])) { if ($email_defaut = $form['email']['defaut']) { if (!isset($formulaire['traitements']['email'])) { - $formulaire['traitements']['email'] = array(); + $formulaire['traitements']['email'] = []; } $formulaire['traitements']['email']['destinataires_plus'] = $email_defaut; } @@ -209,11 +213,11 @@ function forms_configure_traitement_formulaire($form, &$formulaire) { } // Le traitement enregistrement : toujours - $formulaire['traitements']['enregistrement'] = array( + $formulaire['traitements']['enregistrement'] = [ 'moderation' => ($form['moderation'] == 'priori') ? 'priori' : 'posteriori', 'modifiable' => ($form['modifiable'] == 'oui') ? 'on' : '', 'multiple' => ($form['multiple'] == 'non') ? '' : 'on' - ); + ]; } /** @@ -224,13 +228,13 @@ function forms_configure_traitement_formulaire($form, &$formulaire) { * @return array */ function forms_regroupe_saisies_fieldset($saisies) { - $s = array(); + $s = []; $ins = &$s; foreach ($saisies as $k => $saisie) { - if ($saisie['saisie']=='fieldset') { + if ($saisie['saisie'] == 'fieldset') { if (!isset($saisies[$k]['saisies'])) { - $saisies[$k]['saisies'] = array(); + $saisies[$k]['saisies'] = []; } $ins = &$saisies[$k]['saisies']; $s[] = &$saisies[$k]; @@ -260,10 +264,10 @@ function forms_regroupe_saisies_fieldset($saisies) { */ function forms_champ_vers_saisie($champ) { // Le truc par défaut - $saisie = array( + $saisie = [ 'saisie' => 'input', - 'options' => array('size'=>40) - ); + 'options' => ['size' => 40] + ]; // On essaye de traduire tous les types de champs $type = $champ['type']; @@ -279,32 +283,32 @@ function forms_champ_vers_saisie($champ) { break; case 'date': $saisie['saisie'] = 'date'; - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'date' - ); + ]; break; case 'num': case 'monnaie': if (!isset($champ['taille']) or !intval($taille = $champ['taille'])) { - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'entier' - ); + ]; } else { - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'decimal' - ); - $saisie['verifier']['options'] = array('nb_decimales' => $taille); + ]; + $saisie['verifier']['options'] = ['nb_decimales' => $taille]; } break; case 'email': - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'email' - ); + ]; break; case 'telephone': - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'telephone' - ); + ]; break; case 'select': unset($saisie['options']['size']); @@ -333,7 +337,7 @@ function forms_champ_vers_saisie($champ) { break; case 'separateur': $saisie['saisie'] = 'fieldset'; - $saisie['saisies'] = array(); + $saisie['saisies'] = []; unset($saisie['options']['size']); break; case 'fichier': @@ -349,7 +353,7 @@ function forms_champ_vers_saisie($champ) { // Les choix pour les types select et multiple if (isset($champ['choix']) and is_array($champ['choix'])) { - $saisie['options']['datas'] = array();//on conserve datas, car c'est ainsi que dans les .yaml + $saisie['options']['datas'] = [];//on conserve datas, car c'est ainsi que dans les .yaml foreach ($champ['choix'] as $un_choix) { $choix = $un_choix['choix']; $titre = $un_choix['titre']; @@ -375,7 +379,7 @@ function forms_champ_vers_saisie($champ) { $saisie['options']['explication'] = $explication; } - if (isset($champ['saisie']) and $champ['saisie']=='non') { + if (isset($champ['saisie']) and $champ['saisie'] == 'non') { $saisie['options']['disable'] = 'on'; // masquer en JS, fallback $saisie['options']['afficher_si'] = 'false'; diff --git a/echanger/formulaire/wcs.php b/echanger/formulaire/wcs.php index dc4894c1..53d847c7 100644 --- a/echanger/formulaire/wcs.php +++ b/echanger/formulaire/wcs.php @@ -14,7 +14,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) { if ($arbre and is_array($arbre) and isset($arbre['formdef'])) { foreach ($arbre['formdef'] as $form) { - $formulaire = array(); + $formulaire = []; // Le titre $titre = filtrer_entites(trim(spip_xml_aplatit($form['name']))); @@ -25,14 +25,14 @@ function echanger_formulaire_wcs_importer_dist($fichier) { $deja = sql_getfetsel( 'id_formulaire', 'spip_formulaires', - 'identifiant = '.sql_quote($formulaire['identifiant']) + 'identifiant = ' . sql_quote($formulaire['identifiant']) ); if ($deja) { - $formulaire['identifiant'] = $formulaire['identifiant'].'_'.time(); + $formulaire['identifiant'] = $formulaire['identifiant'] . '_' . time(); } // Les champs - $formulaire['saisies'] = array(); + $formulaire['saisies'] = []; // Par défaut le conteneur c'est le formulaire $conteneur =& $formulaire; foreach ($form['fields'] as $fields) { @@ -40,10 +40,10 @@ function echanger_formulaire_wcs_importer_dist($fichier) { $changer_conteneur = false; // Le truc par défaut - $saisie = array( + $saisie = [ 'saisie' => 'input', - 'options' => array('size'=>40) - ); + 'options' => ['size' => 40] + ]; // Le label $saisie['options']['label'] = filtrer_entites(trim(spip_xml_aplatit($field['label']))); @@ -54,10 +54,10 @@ function echanger_formulaire_wcs_importer_dist($fichier) { case 'string': if ($size = intval(trim(spip_xml_aplatit($field['size'])))) { $saisie['options']['maxlength'] = $size; - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'taille', - 'options' => array('max'=>$size) - ); + 'options' => ['max' => $size] + ]; } break; case 'text': @@ -73,14 +73,14 @@ function echanger_formulaire_wcs_importer_dist($fichier) { } break; case 'date': - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'date' - ); + ]; break; case 'email': - $saisie['verifier'] = array( + $saisie['verifier'] = [ 'type' => 'email' - ); + ]; break; case 'item': unset($saisie['options']['size']); @@ -105,7 +105,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) { case 'page': $saisie['saisie'] = 'fieldset'; unset($saisie['options']['size']); - $saisie['saisies'] = array(); + $saisie['saisies'] = []; $changer_conteneur = true; // On remet le conteneur au niveau du formulaire $conteneur =& $formulaire; @@ -119,11 +119,11 @@ function echanger_formulaire_wcs_importer_dist($fichier) { if ($saisie) { // Les choix pour les types select if (isset($field['items']) and is_array($field['items'])) { - $saisie['options']['datas'] = array();//on conserve datas, car c'est ainsi que dans les .yaml + $saisie['options']['datas'] = [];//on conserve datas, car c'est ainsi que dans les .yaml foreach ($field['items'] as $items) { foreach ($items['item'] as $cle => $item) { $titre = filtrer_entites(trim($item)); - $saisie['options']['datas']['choix_'.$cle] = $titre;//on conserve datas, car c'est ainsi que dans les .yaml + $saisie['options']['datas']['choix_' . $cle] = $titre;//on conserve datas, car c'est ainsi que dans les .yaml } } } @@ -146,7 +146,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) { // Faut-il changer de conteneur ? if ($changer_conteneur) { - $conteneur =& $conteneur['saisies'][count($conteneur['saisies'])-1]; + $conteneur =& $conteneur['saisies'][count($conteneur['saisies']) - 1]; } } } @@ -172,6 +172,6 @@ function echanger_formulaire_wcs_importer_dist($fichier) { if ($id_formulaire and !$erreur) { return $id_formulaire; } else { - return _T('formidable:erreur_importer_wcs').' : '.$erreur; + return _T('formidable:erreur_importer_wcs') . ' : ' . $erreur; } } diff --git a/echanger/formulaire/yaml.php b/echanger/formulaire/yaml.php index 1a419ab0..a88b984e 100644 --- a/echanger/formulaire/yaml.php +++ b/echanger/formulaire/yaml.php @@ -17,7 +17,7 @@ function echanger_formulaire_yaml_exporter_dist($id_formulaire) { $formulaire = sql_fetsel( '*', 'spip_formulaires', - 'id_formulaire = '.$id_formulaire + 'id_formulaire = ' . $id_formulaire ); // On décompresse les trucs sérialisés @@ -29,8 +29,8 @@ function echanger_formulaire_yaml_exporter_dist($id_formulaire) { } Header('Content-Type: text/x-yaml;'); - Header('Content-Disposition: attachment; filename=formulaire-'.$formulaire['identifiant'].'.yaml'); - Header('Content-Length: '.strlen($export)); + Header('Content-Disposition: attachment; filename=formulaire-' . $formulaire['identifiant'] . '.yaml'); + Header('Content-Length: ' . strlen($export)); echo $export; exit(); } @@ -54,10 +54,10 @@ function echanger_formulaire_yaml_importer_dist($fichier) { $deja = sql_getfetsel( 'id_formulaire', 'spip_formulaires', - 'identifiant = '.sql_quote($formulaire['identifiant']) + 'identifiant = ' . sql_quote($formulaire['identifiant']) ); if ($deja) { - $formulaire['identifiant'] .= '_'.date('Ymd_His'); + $formulaire['identifiant'] .= '_' . date('Ymd_His'); } // On insère un nouveau formulaire @@ -79,6 +79,6 @@ function echanger_formulaire_yaml_importer_dist($fichier) { if ($id_formulaire and !$erreur) { return $id_formulaire; } else { - return _T('formidable:erreur_importer_yaml').' : '.$erreur; + return _T('formidable:erreur_importer_yaml') . ' : ' . $erreur; } } diff --git a/formidable_administrations.php b/formidable_administrations.php index 60790850..cbbc2166 100644 --- a/formidable_administrations.php +++ b/formidable_administrations.php @@ -26,113 +26,113 @@ function formidable_upgrade($nom_meta_base_version, $version_cible) { include_spip('base/abstract_sql'); include_spip('inc/formidable_migrer_depuis_ft'); - $maj = array(); - $maj['create'] = array( - array('maj_tables',array( + $maj = []; + $maj['create'] = [ + ['maj_tables',[ 'spip_formulaires', 'spip_formulaires_reponses', 'spip_formulaires_reponses_champs', - 'spip_formulaires_liens')), - array('formidable_importer_forms'), - array('formidable_importer_forms_donnees'), - array('formidable_associer_forms'), - array('formidable_creer_secret') - ); + 'spip_formulaires_liens']], + ['formidable_importer_forms'], + ['formidable_importer_forms_donnees'], + ['formidable_associer_forms'], + ['formidable_creer_secret'] + ]; // Ajout du choix de ce qu'on affiche à la fin des traitements - $maj['0.4.0'] = array(array('maj_tables',array('spip_formulaires'))); + $maj['0.4.0'] = [['maj_tables',['spip_formulaires']]]; // Ajout d'une URL de redirection - $maj['0.5.0'] = array(array('maj_tables',array('spip_formulaires'))); + $maj['0.5.0'] = [['maj_tables',['spip_formulaires']]]; // Modif du type du message de retour pour pouvoir mettre plus de chose - $maj['0.5.1'] = array(array('sql_alter','TABLE spip_formulaires CHANGE message_retour message_retour text NOT NULL default ""')); + $maj['0.5.1'] = [['sql_alter','TABLE spip_formulaires CHANGE message_retour message_retour text NOT NULL default ""']]; // Passer le champ saisies en longtext pour permettre d'y stocker des formulaires longs - $maj['0.5.2'] = array(array('sql_alter','TABLE spip_formulaires CHANGE saisies saisies longtext NOT NULL default ""')); + $maj['0.5.2'] = [['sql_alter','TABLE spip_formulaires CHANGE saisies saisies longtext NOT NULL default ""']]; // Ajouter un champ date de création - $maj['0.5.3'] = array(array('sql_alter','TABLE spip_formulaires ADD date_crea datetime NOT NULL DEFAULT "0000-00-00 00:00:00"')); + $maj['0.5.3'] = [['sql_alter','TABLE spip_formulaires ADD date_crea datetime NOT NULL DEFAULT "0000-00-00 00:00:00"']]; // Renommer la date de création (pas d'abbréviations dans les noms) - $maj['0.5.5'] = array(array('sql_alter','TABLE spip_formulaires CHANGE date_crea date_creation datetime NOT NULL DEFAULT "0000-00-00 00:00:00"')); + $maj['0.5.5'] = [['sql_alter','TABLE spip_formulaires CHANGE date_crea date_creation datetime NOT NULL DEFAULT "0000-00-00 00:00:00"']]; // statut publie sur les formulaires sans statut - $maj['0.5.6'] = array( - array('sql_updateq', 'spip_formulaires', array('statut'=>'publie'), 'statut='.sql_quote('')), - ); - $maj['0.6.0'] = array( - array('sql_alter','TABLE spip_formulaires_reponses_champs RENAME TO spip_formulaires_reponses_champs_bad'), - array('maj_tables',array('spip_formulaires_reponses_champs')), - array('formidable_transferer_reponses_champs'), - array('sql_drop_table','spip_formulaires_reponses_champs_bad'), - ); - $maj['0.6.1'] = array( - array('formidable_unifier_reponses_champs'), - ); - $maj['0.6.3'] = array( - array('sql_alter','TABLE spip_formulaires_reponses_champs ADD UNIQUE reponse (id_formulaires_reponse,nom)'), - ); - $maj['0.6.4'] = array( + $maj['0.5.6'] = [ + ['sql_updateq', 'spip_formulaires', ['statut' => 'publie'], 'statut=' . sql_quote('')], + ]; + $maj['0.6.0'] = [ + ['sql_alter','TABLE spip_formulaires_reponses_champs RENAME TO spip_formulaires_reponses_champs_bad'], + ['maj_tables',['spip_formulaires_reponses_champs']], + ['formidable_transferer_reponses_champs'], + ['sql_drop_table','spip_formulaires_reponses_champs_bad'], + ]; + $maj['0.6.1'] = [ + ['formidable_unifier_reponses_champs'], + ]; + $maj['0.6.3'] = [ + ['sql_alter','TABLE spip_formulaires_reponses_champs ADD UNIQUE reponse (id_formulaires_reponse,nom)'], + ]; + $maj['0.6.4'] = [ // champ resume_reponse - array('maj_tables',array('spip_formulaires')), - ); + ['maj_tables',['spip_formulaires']], + ]; // Pouvoir rendre un champ unique - $maj['0.6.5'] = array( + $maj['0.6.5'] = [ // champ resume_reponse - array('maj_tables',array('spip_formulaires')), - ); - $maj['0.6.6'] = array( - array('sql_updateq', 'spip_formulaires_reponses', array('statut' => 'refuse'), 'statut='.sql_quote('poubelle')), - ); + ['maj_tables',['spip_formulaires']], + ]; + $maj['0.6.6'] = [ + ['sql_updateq', 'spip_formulaires_reponses', ['statut' => 'refuse'], 'statut=' . sql_quote('poubelle')], + ]; // Ajouter un champ "css" sur les formulaires - $maj['0.7.0'] = array( - array('maj_tables', array('spip_formulaires')), - ); + $maj['0.7.0'] = [ + ['maj_tables', ['spip_formulaires']], + ]; // Migrer afficher_si_remplissage vers la checkbox - $maj['0.8.0'] = array( - array('formidable_migrer_formulaires_afficher_si_remplissage') - ); - $maj['0.9.0'] = array( - array('formidable_migrer_resume_reponse') - ); - $maj['0.10.0'] = array( - array('formidable_migrer_reglage_champ_unique') - ); - $maj['0.11.0'] = array( - array('formidable_migrer_anonymisation') - ); - $maj['0.12.0'] = array( - array('formidable_migrer_config') - ); - $maj['0.15.0'] = array( - array('formidable_effacer_traitement_enregistrer_sans_option') - ); - $maj['0.17.1'] = array( - array('sql_alter','TABLE spip_formulaires_reponses CHANGE variable_php variable_php VARCHAR(255) NOT NULL default ""') - ); - $maj['0.20.0'] = array( - array('sql_alter','TABLE spip_formulaires_reponses ADD column `date_envoi` datetime NOT NULL default "0000-00-00 00:00:00" AFTER `date`'), - array('sql_update', 'spip_formulaires_reponses', array('date_envoi' => 'date')), - ); - $maj['0.23.0'] = array( - array('formidable_saisies_identifier') - ); + $maj['0.8.0'] = [ + ['formidable_migrer_formulaires_afficher_si_remplissage'] + ]; + $maj['0.9.0'] = [ + ['formidable_migrer_resume_reponse'] + ]; + $maj['0.10.0'] = [ + ['formidable_migrer_reglage_champ_unique'] + ]; + $maj['0.11.0'] = [ + ['formidable_migrer_anonymisation'] + ]; + $maj['0.12.0'] = [ + ['formidable_migrer_config'] + ]; + $maj['0.15.0'] = [ + ['formidable_effacer_traitement_enregistrer_sans_option'] + ]; + $maj['0.17.1'] = [ + ['sql_alter','TABLE spip_formulaires_reponses CHANGE variable_php variable_php VARCHAR(255) NOT NULL default ""'] + ]; + $maj['0.20.0'] = [ + ['sql_alter','TABLE spip_formulaires_reponses ADD column `date_envoi` datetime NOT NULL default "0000-00-00 00:00:00" AFTER `date`'], + ['sql_update', 'spip_formulaires_reponses', ['date_envoi' => 'date']], + ]; + $maj['0.23.0'] = [ + ['formidable_saisies_identifier'] + ]; // Ajouter un champ "vu" pour les liaisons - $maj['1.0.0'] = array( - array('maj_tables', array('spip_formulaires_liens')), - array('formidable_verifier_liens_vu'), - ); + $maj['1.0.0'] = [ + ['maj_tables', ['spip_formulaires_liens']], + ['formidable_verifier_liens_vu'], + ]; // Ajouter un champ afficher_message_ok_traitements pour les formulaires - $maj['1.3.0'] = array( - array('maj_tables', array('spip_formulaires')), - ); + $maj['1.3.0'] = [ + ['maj_tables', ['spip_formulaires']], + ]; // Et le supprimer après car on a trouvé mieux. Cf discussion https://git.spip.net/spip-contrib-extensions/formidable/issues/58 if (lire_config($nom_meta_base_version) == '1.3.0') { - $maj['1.4.0'] = array( - array('sql_alter', 'TABLE spip_formulaires DROP afficher_message_ok_traitements' - ) - ); + $maj['1.4.0'] = [ + ['sql_alter', 'TABLE spip_formulaires DROP afficher_message_ok_traitements' + ] + ]; } - $maj['1.4.2'] = array( - array('formidable_creer_secret'), - ); + $maj['1.4.2'] = [ + ['formidable_creer_secret'], + ]; include_spip('base/upgrade'); maj_plugin($nom_meta_base_version, $version_cible, $maj); @@ -144,7 +144,7 @@ function formidable_unifier_reponses_champs() { $rows = sql_allfetsel( 'DISTINCT id_formulaires_reponses_champ,id_formulaires_reponse,nom,count(id_formulaires_reponse) AS N', 'spip_formulaires_reponses_champs', - 'nom LIKE '.sql_quote('multiple%').' OR nom LIKE '.sql_quote('mot%'), + 'nom LIKE ' . sql_quote('multiple%') . ' OR nom LIKE ' . sql_quote('mot%'), 'concat( id_formulaires_reponse, nom )', 'id_formulaires_reponse', '0,100', @@ -157,21 +157,21 @@ function formidable_unifier_reponses_champs() { $reponse = sql_allfetsel( '*', 'spip_formulaires_reponses_champs', - 'id_formulaires_reponse='.intval($row['id_formulaires_reponse']) + 'id_formulaires_reponse=' . intval($row['id_formulaires_reponse']) ); - spip_log('id_formulaires_reponse '.$row['id_formulaires_reponse'], 'formidable_unifier_reponses_champs'._LOG_INFO_IMPORTANTE); + spip_log('id_formulaires_reponse ' . $row['id_formulaires_reponse'], 'formidable_unifier_reponses_champs' . _LOG_INFO_IMPORTANTE); // on les reinsere un par un dans la nouvelle table propre - $data = array(); + $data = []; foreach ($reponse as $champ) { $data[$champ['nom']][] = $champ; } foreach ($data as $nom => $champs) { - if (count($champs)>1) { + if (count($champs) > 1) { #var_dump($champs); $keep = $champs[0]['id_formulaires_reponses_champ']; - $delete = array(); - $valeurs = array(); + $delete = []; + $valeurs = []; foreach ($champs as $champ) { $valeurs[] = $champ['valeur']; if ($champ['id_formulaires_reponses_champ'] !== $keep) { @@ -182,7 +182,7 @@ function formidable_unifier_reponses_champs() { #var_dump($valeurs); #var_dump($keep); #var_dump($delete); - sql_updateq('spip_formulaires_reponses_champs', array('valeur'=>$valeurs), 'id_formulaires_reponses_champ='.intval($keep)); + sql_updateq('spip_formulaires_reponses_champs', ['valeur' => $valeurs], 'id_formulaires_reponses_champ=' . intval($keep)); sql_delete('spip_formulaires_reponses_champs', sql_in('id_formulaires_reponses_champ', $delete)); //die(); } @@ -190,15 +190,15 @@ function formidable_unifier_reponses_champs() { #var_dump($data); //die('nothing?'); - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } } - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } - } while ($rows = sql_allfetsel('DISTINCT id_formulaires_reponses_champ,id_formulaires_reponse,nom,count( id_formulaires_reponse ) AS N', 'spip_formulaires_reponses_champs', 'nom LIKE '.sql_quote('multiple%').' OR nom LIKE '.sql_quote('mot%'), 'concat( id_formulaires_reponse, nom )', 'id_formulaires_reponse', '0,100', 'N>1')); + } while ($rows = sql_allfetsel('DISTINCT id_formulaires_reponses_champ,id_formulaires_reponse,nom,count( id_formulaires_reponse ) AS N', 'spip_formulaires_reponses_champs', 'nom LIKE ' . sql_quote('multiple%') . ' OR nom LIKE ' . sql_quote('mot%'), 'concat( id_formulaires_reponse, nom )', 'id_formulaires_reponse', '0,100', 'N>1')); //die('fini?'); } @@ -209,19 +209,19 @@ function formidable_transferer_reponses_champs() { do { foreach ($rows as $row) { // pour chaque reponse on recupere tous les champs - $reponse = sql_allfetsel('*', 'spip_formulaires_reponses_champs_bad', 'id_formulaires_reponse='.intval($row['id_formulaires_reponse'])); + $reponse = sql_allfetsel('*', 'spip_formulaires_reponses_champs_bad', 'id_formulaires_reponse=' . intval($row['id_formulaires_reponse'])); // on les reinsere un par un dans la nouvelle table propre foreach ($reponse as $champ) { sql_insertq('spip_formulaires_reponses_champs', $champ); } // et on les vire de la mauvaise - sql_delete('spip_formulaires_reponses_champs_bad', 'id_formulaires_reponse='.intval($row['id_formulaires_reponse'])); - if (time()>_TIME_OUT) { + sql_delete('spip_formulaires_reponses_champs_bad', 'id_formulaires_reponse=' . intval($row['id_formulaires_reponse'])); + if (time() > _TIME_OUT) { return; } } - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } } while ($rows = sql_allfetsel('DISTINCT id_formulaires_reponse', 'spip_formulaires_reponses_champs_bad', '', 'id_formulaires_reponse', '', '0,100')); @@ -235,27 +235,26 @@ function formidable_transferer_reponses_champs() { */ function formidable_migrer_reglage_champ_unique() { include_spip('inc/sql'); - $res = sql_select('id_formulaire,unicite,message_erreur_unicite,traitements','spip_formulaires'); + $res = sql_select('id_formulaire,unicite,message_erreur_unicite,traitements', 'spip_formulaires'); while ($row = sql_fetch($res)) { $traitements = unserialize($row['traitements']); if ($row['unicite'] != '') { if (!isset($traitements['enregistrement'])) { - $traitements['enregistrement'] = array(); + $traitements['enregistrement'] = []; } $traitements['enregistrement']['unicite'] = $row['unicite']; } if ($row['message_erreur_unicite'] != '') { if (!isset($traitements['enregistrement'])) { - $traitements['enregistrement'] = array(); + $traitements['enregistrement'] = []; } $traitements['enregistrement']['message_erreur_unicite'] = $row['message_erreur_unicite']; } $traitements = serialize($traitements); - sql_updateq('spip_formulaires',array('traitements'=>$traitements),'id_formulaire='.$row['id_formulaire']); - + sql_updateq('spip_formulaires', ['traitements' => $traitements], 'id_formulaire=' . $row['id_formulaire']); } - sql_alter("TABLE spip_formulaires DROP message_erreur_unicite, DROP unicite"); + sql_alter('TABLE spip_formulaires DROP message_erreur_unicite, DROP unicite'); } /** @@ -264,20 +263,20 @@ function formidable_migrer_reglage_champ_unique() { * * @return void */ -function formidable_migrer_formulaires_afficher_si_remplissage(){ +function formidable_migrer_formulaires_afficher_si_remplissage() { // selection include_spip('inc/saisies_migrer_afficher_si_remplissage'); - if ($res = sql_select(array('id_formulaire','saisies'), 'spip_formulaires')) { + if ($res = sql_select(['id_formulaire','saisies'], 'spip_formulaires')) { // boucler sur les resultats while ($row = sql_fetch($res)) { - $id_formulaire = $row["id_formulaire"]; + $id_formulaire = $row['id_formulaire']; if ($saisies = unserialize($row['saisies'])) { $saisies = saisies_migrer_afficher_si_remplissage($saisies); $saisies = serialize($saisies); sql_updateq( 'spip_formulaires', - array('saisies'=>$saisies), - "id_formulaire=".intval($id_formulaire) + ['saisies' => $saisies], + 'id_formulaire=' . intval($id_formulaire) ); } } @@ -297,29 +296,29 @@ function formidable_migrer_formulaires_afficher_si_remplissage(){ * @return void **/ function formidable_migrer_anonymisation() { - sql_alter("TABLE spip_formulaires_reponses ADD column `variable_php` bigint(21) NOT NULL default 0 AFTER `cookie`"); - sql_alter("TABLE spip_formulaires_reponses ADD INDEX (variable_php)"); - $res = sql_select("id_formulaire, traitements", "spip_formulaires"); + sql_alter('TABLE spip_formulaires_reponses ADD column `variable_php` bigint(21) NOT NULL default 0 AFTER `cookie`'); + sql_alter('TABLE spip_formulaires_reponses ADD INDEX (variable_php)'); + $res = sql_select('id_formulaire, traitements', 'spip_formulaires'); while ($row = sql_fetch($res)) { - $id_formulaire = $row["id_formulaire"]; - $traitements = unserialize($row["traitements"]); - $enregistrement = isset($traitements["enregistrement"]) ? $traitements["enregistrement"] : array(); + $id_formulaire = $row['id_formulaire']; + $traitements = unserialize($row['traitements']); + $enregistrement = isset($traitements['enregistrement']) ? $traitements['enregistrement'] : []; // A-ton l'option d'anonymisation activée? alors on migre, sinon rien à changer - if (isset($enregistrement['anonymiser']) and $enregistrement["anonymiser"] == "on") { - $enregistrement["variable_php"] = isset($enregistrement["anonymiser_variable"]) ? $enregistrement["anonymiser_variable"] : ''; - unset($enregistrement["anonymiser_variable"]); - if (isset($enregistrement["identification"]) and $enregistrement["identification"] == "id_auteur") { - $enregistrement["identification"] = "variable_php"; + if (isset($enregistrement['anonymiser']) and $enregistrement['anonymiser'] == 'on') { + $enregistrement['variable_php'] = isset($enregistrement['anonymiser_variable']) ? $enregistrement['anonymiser_variable'] : ''; + unset($enregistrement['anonymiser_variable']); + if (isset($enregistrement['identification']) and $enregistrement['identification'] == 'id_auteur') { + $enregistrement['identification'] = 'variable_php'; } // Mettre à jour le traitement - $traitements["enregistrement"] = $enregistrement; + $traitements['enregistrement'] = $enregistrement; $traitements = serialize($traitements); - sql_updateq("spip_formulaires", array("traitements" => $traitements), "id_formulaire=$id_formulaire"); + sql_updateq('spip_formulaires', ['traitements' => $traitements], "id_formulaire=$id_formulaire"); // Mettre à jour les réponses - $res_reponse = sql_select("id_auteur,id_formulaires_reponse", "spip_formulaires_reponses", "id_formulaire=$id_formulaire"); + $res_reponse = sql_select('id_auteur,id_formulaires_reponse', 'spip_formulaires_reponses', "id_formulaire=$id_formulaire"); while ($raw_reponse = sql_fetch($res_reponse)) { - $id_formulaires_reponse = $raw_reponse["id_formulaires_reponse"]; - sql_updateq("spip_formulaires_reponses", array("variable_php"=>$raw_reponse["id_auteur"], "id_auteur" => 0), "id_formulaires_reponse=$id_formulaires_reponse"); + $id_formulaires_reponse = $raw_reponse['id_formulaires_reponse']; + sql_updateq('spip_formulaires_reponses', ['variable_php' => $raw_reponse['id_auteur'], 'id_auteur' => 0], "id_formulaires_reponse=$id_formulaires_reponse"); } } } @@ -333,7 +332,7 @@ function formidable_migrer_anonymisation() { * @return void */ function formidable_migrer_resume_reponse() { - if ($res = sql_select(array('id_formulaire','traitements','resume_reponse'), 'spip_formulaires')) { + if ($res = sql_select(['id_formulaire','traitements','resume_reponse'], 'spip_formulaires')) { while ($row = sql_fetch($res)) { $id_formulaire = $row['id_formulaire']; $traitements = unserialize($row['traitements']); @@ -342,11 +341,11 @@ function formidable_migrer_resume_reponse() { if (isset($traitements['enregistrement'])) { $traitements['enregistrement']['resume_reponse'] = $resume_reponse; } else { - $traitements['enregistrement'] = array('resume_reponse' => $resume_reponse); + $traitements['enregistrement'] = ['resume_reponse' => $resume_reponse]; } sql_updateq( 'spip_formulaires', - array('traitements'=>serialize($traitements)), + ['traitements' => serialize($traitements)], "id_formulaire=$id_formulaire" ); } @@ -354,8 +353,7 @@ function formidable_migrer_resume_reponse() { } // suppression du champ - sql_alter("TABLE spip_formulaires DROP COLUMN resume_reponse"); - + sql_alter('TABLE spip_formulaires DROP COLUMN resume_reponse'); } @@ -368,14 +366,14 @@ function formidable_migrer_resume_reponse() { **/ function formidable_effacer_traitement_enregistrer_sans_option() { include_spip('inc/sql'); - $res = sql_select('id_formulaire,traitements','spip_formulaires'); + $res = sql_select('id_formulaire,traitements', 'spip_formulaires'); while ($row = sql_fetch($res)) { $traitements = unserialize($row['traitements']); - if ($traitements['enregistrement'] === array()) { + if ($traitements['enregistrement'] === []) { unset($traitements['enregistrement']); } $traitements = serialize($traitements); - sql_updateq('spip_formulaires',array('traitements'=>$traitements),'id_formulaire='.$row['id_formulaire']); + sql_updateq('spip_formulaires', ['traitements' => $traitements], 'id_formulaire=' . $row['id_formulaire']); } } @@ -384,9 +382,9 @@ function formidable_effacer_traitement_enregistrer_sans_option() { **/ function formidable_migrer_config() { include_spip('inc/config'); - $config = lire_config("formidable/analyse"); - effacer_config("formidable/analyse"); - ecrire_config("formidable", $config); + $config = lire_config('formidable/analyse'); + effacer_config('formidable/analyse'); + ecrire_config('formidable', $config); } /** @@ -397,11 +395,11 @@ function formidable_verifier_liens_vu() { include_spip('formidable_pipelines'); // Chercher tous les liens de form pas marqué comme vu - if ($liens = objet_trouver_liens(array('formulaire' => '*'), array('*' => '*'), array('vu="non"'))) { + if ($liens = objet_trouver_liens(['formulaire' => '*'], ['*' => '*'], ['vu="non"'])) { foreach ($liens as $lien) { $table = table_objet_sql($lien['objet']); $cle = id_table_objet($lien['objet']); - if ($contenu = sql_fetsel('*', $table, "$cle=".intval($lien['id_objet']))) { + if ($contenu = sql_fetsel('*', $table, "$cle=" . intval($lien['id_objet']))) { $contenu = implode(' ', $contenu); // On cherche les modèles insérés dans les textes du contenu @@ -410,7 +408,7 @@ function formidable_verifier_liens_vu() { // Est-ce qu'on trouve le form de ce lien dans les modèles trouvés if (isset($formulaires[$lien['id_formulaire']])) { // On lui ajoute vu=oui - objet_qualifier_liens(array('formulaire' => $lien['id_formulaire']), array($lien['objet'] => $lien['id_objet']), array('vu' => 'oui')); + objet_qualifier_liens(['formulaire' => $lien['id_formulaire']], [$lien['objet'] => $lien['id_objet']], ['vu' => 'oui']); } } } @@ -449,7 +447,7 @@ function formidable_vider_tables($nom_meta_base_version) { supprimer_repertoire(_DIR_FICHIERS_FORMIDABLE); // On efface la version enregistrée - effacer_config("formidable"); + effacer_config('formidable'); effacer_meta($nom_meta_base_version); } @@ -463,7 +461,7 @@ function formidable_saisies_identifier() { $saisies = unserialize($row['saisies']); $saisies = saisies_identifier($saisies); $saisies = serialize($saisies); - sql_updateq('spip_formulaires', array('saisies' => $saisies), 'id_formulaire='.$row['id_formulaire']); + sql_updateq('spip_formulaires', ['saisies' => $saisies], 'id_formulaire=' . $row['id_formulaire']); } } diff --git a/formidable_autorisations.php b/formidable_autorisations.php index a25c810f..0d186632 100644 --- a/formidable_autorisations.php +++ b/formidable_autorisations.php @@ -39,7 +39,7 @@ function formidable_autoriser_par_auteur($id_formulaire, $id_auteur = 0) { // Si on a bien un id_auteur if (($id_auteur = intval($id_auteur)) > 0) { // On cherche si cet auteur est lié à ce formulaire - $autorisations = objet_trouver_liens(array('formulaire' => $id_formulaire), array('auteur' => $id_auteur)); + $autorisations = objet_trouver_liens(['formulaire' => $id_formulaire], ['auteur' => $id_auteur]); $retour = count($autorisations) > 0; } @@ -78,7 +78,8 @@ function formidable_auteur_admin_reponse($qui) { * Fonction d'appel pour le pipeline * @pipeline autoriser */ -function formidable_autoriser() {} +function formidable_autoriser() { +} /** * Autorisation d'éditer un formulaire formidable @@ -154,7 +155,7 @@ function autoriser_formulaire_repondre_dist($faire, $type, $id, $qui, $opt) { $formulaire = $opt['formulaire']; } else { // Sinon on va le chercher - $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id); + $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id); } include_spip('formidable_fonctions'); @@ -227,7 +228,7 @@ function autoriser_formulairesreponse_instituer_dist($faire, $type, $id, $qui, $ $id_formulaire = intval($opt['id_formulaire']); } else { - $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires_reponses', 'id_formulaires_reponse = '.intval($id)); + $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires_reponses', 'id_formulaires_reponse = ' . intval($id)); } if ( diff --git a/formidable_fonctions.php b/formidable_fonctions.php index ccb55eec..42eecf7d 100644 --- a/formidable_fonctions.php +++ b/formidable_fonctions.php @@ -54,38 +54,41 @@ function balise_VOIR_REPONSE_dist($p) { * @return array|string */ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $sql_serveur = '', $type_retour = null, $sans_reponse = null) { - static $formulaires_saisies = array(); - static $reponses_valeurs = array(); + static $formulaires_saisies = []; + static $reponses_valeurs = []; include_spip('formidable_fonctions'); // Si pas déjà présent, on cherche les saisies de ce formulaire if (!isset($formulaires_saisies[$id_formulaire])) { $formulaires_saisies[$id_formulaire] = formidable_deserialize( - sql_getfetsel('saisies',//select - 'spip_formulaires',//from - 'id_formulaire = '.intval($id_formulaire),//where - '',//groupby - '',//orderby - '',//limit - '',//having - $sql_serveur - ) + sql_getfetsel( + 'saisies', //select + 'spip_formulaires', //from + 'id_formulaire = ' . intval($id_formulaire), //where + '', //groupby + '', //orderby + '', //limit + '', //having + $sql_serveur + ) ); } // Si pas déjà présent, on cherche les valeurs de cette réponse if (!isset($reponses_valeurs[$id_formulaires_reponse])) { - if ($champs = sql_allfetsel( - 'nom,valeur,id_formulaires_reponses_champ',//select - 'spip_formulaires_reponses_champs',//from - 'id_formulaires_reponse = '.intval($id_formulaires_reponse),//where - '',//groupby - '',//orderby - '',//limit - '',//having - $sql_serveur// - )) { + if ( + $champs = sql_allfetsel( + 'nom,valeur,id_formulaires_reponses_champ', //select + 'spip_formulaires_reponses_champs', //from + 'id_formulaires_reponse = ' . intval($id_formulaires_reponse), //where + '', //groupby + '', //orderby + '', //limit + '', //having + $sql_serveur// + ) + ) { foreach ($champs as $champ) { - $reponses_valeurs[$id_formulaires_reponse][$champ['nom']] = array( + $reponses_valeurs[$id_formulaires_reponse][$champ['nom']] = [ 'valeur' => formidable_ajouter_action_recuperer_fichier( formidable_deserialize($champ['valeur']), $champ['nom'], @@ -94,7 +97,7 @@ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $s $id_formulaires_reponse ), 'id' => $champ['id_formulaires_reponses_champ'] - ); + ]; } } } @@ -114,9 +117,9 @@ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $s if (isset($reponses_valeurs[$id_formulaires_reponse][$nom]['id'])) { $valeur = $reponses_valeurs[$id_formulaires_reponse][$nom]['id']; } else { - $valeur = sql_insertq('spip_formulaires_reponses_champs', array('id_formulaires_reponse' => $id_formulaires_reponse, 'nom' => $nom)); + $valeur = sql_insertq('spip_formulaires_reponses_champs', ['id_formulaires_reponse' => $id_formulaires_reponse, 'nom' => $nom]); } - return 'crayon '.'formulaires_reponses_champ-valeur-'. $valeur; + return 'crayon ' . 'formulaires_reponses_champ-valeur-' . $valeur; } // Si on trouve bien la saisie demandée if ($saisie = saisies_chercher($formulaires_saisies[$id_formulaire], $nom)) { @@ -128,12 +131,12 @@ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $s return recuperer_fond( 'saisies-vues/_base', array_merge( - array( + [ 'type_saisie' => $saisie['saisie'], 'valeur' => isset($reponses_valeurs[$id_formulaires_reponse][$nom]['valeur']) ? $reponses_valeurs[$id_formulaires_reponse][$nom]['valeur'] : '', 'valeur_uniquement' => ($type_retour == 'valeur_uniquement' ? 'oui' : 'non'), 'sans_reponse' => $sans_reponse, - ), + ], $saisie['options'] ) ); @@ -149,8 +152,8 @@ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $s * @return string */ function affiche_resume_reponse($id_formulaires_reponse, $id_formulaire = null, $modele_resume = null) { - static $modeles_resume = array(); - static $saisies_form = array(); + static $modeles_resume = []; + static $saisies_form = []; include_spip('formidable_fonctions'); @@ -158,12 +161,12 @@ function affiche_resume_reponse($id_formulaires_reponse, $id_formulaire = null, $id_formulaire = sql_getfetsel( 'id_formulaire', 'spip_formulaires_reponses', - 'id_formulaires_reponse='.intval($id_formulaires_reponse) + 'id_formulaires_reponse=' . intval($id_formulaires_reponse) ); } if (is_null($modele_resume) and !isset($modeles_resume[$id_formulaire])) { - $row = sql_fetsel('saisies, traitements', 'spip_formulaires', 'id_formulaire='.intval($id_formulaire)); + $row = sql_fetsel('saisies, traitements', 'spip_formulaires', 'id_formulaire=' . intval($id_formulaire)); $saisies = formidable_deserialize($row['saisies']); $saisies_form[$id_formulaire] = $saisies; $traitements = formidable_deserialize($row['traitements']); @@ -182,7 +185,7 @@ function affiche_resume_reponse($id_formulaires_reponse, $id_formulaire = null, return ''; } - $valeurs = array(); + $valeurs = []; $chaine = formidable_raccourcis_arobases_2_valeurs_champs( $modele_resume, $saisies, @@ -215,9 +218,9 @@ function formidable_nettoyer_saisie_vue($valeur) { $valeur = implode(', ', $valeur); } $valeur = supprimer_tags($valeur); - $valeur = str_replace("\n"," ",$valeur); - $valeur = str_replace("\r"," ",$valeur); - $valeur = str_replace("\t"," ",$valeur); + $valeur = str_replace("\n", ' ', $valeur); + $valeur = str_replace("\r", ' ', $valeur); + $valeur = str_replace("\t", ' ', $valeur); return $valeur; } @@ -252,8 +255,8 @@ function formidable_ajouter_action_recuperer_fichier($saisie_a_modifier, $nom_sa $saisie_a_modifier[$i]['url'] = $url; if (isset($valeur['extension'])) { $extension = strtolower($valeur['extension']); - if (in_array($extension,array('png','jpg','gif'))) { - $saisie_a_modifier[$i]['vignette'] = _DIR_FICHIERS_FORMIDABLE."formulaire_$id_formulaire/reponse_$id_formulaires_reponse/$nom_saisie/".$valeur['nom']; + if (in_array($extension, ['png','jpg','gif'])) { + $saisie_a_modifier[$i]['vignette'] = _DIR_FICHIERS_FORMIDABLE . "formulaire_$id_formulaire/reponse_$id_formulaires_reponse/$nom_saisie/" . $valeur['nom']; } else { $saisie_a_modifier[$i]['vignette'] = $vignette_par_defaut($extension, false); } @@ -283,7 +286,7 @@ function generer_titre_formulaires_reponse(int $id_formulaires_reponse, $champs) * @param array $not_in * @return string */ -function formidable_afficher_statut_si_different_de($statut,$not_in = array()) { +function formidable_afficher_statut_si_different_de($statut, $not_in = []) { if (!in_array($statut, $not_in)) { include_spip('inc/puce_statut'); return statut_titre('formidable', $statut); diff --git a/formidable_pipelines.php b/formidable_pipelines.php index 89cd170e..1cf22384 100644 --- a/formidable_pipelines.php +++ b/formidable_pipelines.php @@ -14,10 +14,10 @@ include_spip('inc/formidable_fichiers'); define( '_RACCOURCI_MODELE_FORMIDABLE', '(<(formulaire\|formidable|formidable|form)' # <modele - .'([0-9]*)\s*' # id - .'([|](?:<[^<>]*>|[^>])*)?' # |arguments (y compris des tags <...>) - .'>)' # fin du modele > - .'\s*(<\/a>)?' # eventuel </a> + . '([0-9]*)\s*' # id + . '([|](?:<[^<>]*>|[^>])*)?' # |arguments (y compris des tags <...>) + . '>)' # fin du modele > + . '\s*(<\/a>)?' # eventuel </a> ); /** @@ -37,8 +37,8 @@ function formidable_nospam_lister_formulaires($formulaires) { * @return array */ function formidable_trouve_liens($texte) { - $formulaires = array(); - if (preg_match_all(','._RACCOURCI_MODELE_FORMIDABLE.',ims', $texte, $regs, PREG_SET_ORDER)) { + $formulaires = []; + if (preg_match_all(',' . _RACCOURCI_MODELE_FORMIDABLE . ',ims', $texte, $regs, PREG_SET_ORDER)) { foreach ($regs as $r) { $id_formulaire = 0; if ($r[2] == 'formidable') { @@ -47,15 +47,15 @@ function formidable_trouve_liens($texte) { $id_formulaire = sql_getfetsel( 'id_formulaire', 'spip_formulaires', - 'identifiant='.sql_quote('form'.$r[3]) + 'identifiant=' . sql_quote('form' . $r[3]) ); } elseif ($r[2] == 'formulaire|formidable') { $args = ltrim($r[4], '|'); - if(strpos($args,'=')!==false) { + if (strpos($args, '=') !== false) { $args = explode('=', $args); $args = $args[1]; } - if(strpos($args,'|')!==false) { + if (strpos($args, '|') !== false) { $args = explode('|', $args); $args = trim(reset($args)); } @@ -65,7 +65,7 @@ function formidable_trouve_liens($texte) { $id_formulaire = sql_getfetsel( 'id_formulaire', 'spip_formulaires', - 'identifiant='.sql_quote($args) + 'identifiant=' . sql_quote($args) ); } } @@ -83,11 +83,12 @@ function formidable_trouve_liens($texte) { * @return mixed */ function formidable_post_edition($flux) { - if (isset($flux['args']['table']) + if ( + isset($flux['args']['table']) and $table = $flux['args']['table'] and $id_objet = intval($flux['args']['id_objet']) and $primary = id_table_objet($table) - and $row = sql_fetsel('*', $table, "$primary=".intval($id_objet)) + and $row = sql_fetsel('*', $table, "$primary=" . intval($id_objet)) ) { $objet = objet_type($table); $contenu = implode(' ', $row); @@ -97,10 +98,10 @@ function formidable_post_edition($flux) { // On cherche les liaisons "vu dans le texte" qui étaient là avant include_spip('action/editer_liens'); - $deja = objet_trouver_liens(array('formulaire' => '*'), array($objet => $id_objet), array('vu="oui"')); + $deja = objet_trouver_liens(['formulaire' => '*'], [$objet => $id_objet], ['vu="oui"']); // On compare pour savoir quelles liaisons "dans le texte" n'est plus là désormais - $del = array(); + $del = []; if (count($deja)) { foreach ($deja as $l) { if (!isset($formulaires[$l['id_formulaire']])) { @@ -111,14 +112,14 @@ function formidable_post_edition($flux) { // Pour les liens trouvés dans la dernière version des textes if (count($formulaires)) { - objet_associer(array('formulaire' => $formulaires), array($objet => $id_objet), array('vu' => 'oui')); - objet_qualifier_liens(array('formulaire' => $formulaires), array($objet => $id_objet), array('vu' => 'oui')); + objet_associer(['formulaire' => $formulaires], [$objet => $id_objet], ['vu' => 'oui']); + objet_qualifier_liens(['formulaire' => $formulaires], [$objet => $id_objet], ['vu' => 'oui']); } // Pour les liens qui étaient avant dans le texte mais qui n'y sont plus // => on garde toujours la liaison mais avec vu=non, à chacun de retirer la liaison si on ne veut plus du tout, comme pour les docs if (count($del)) { - objet_qualifier_liens(array('formulaire' => $del), array($objet => $id_objet), array('vu' => 'non')); + objet_qualifier_liens(['formulaire' => $del], [$objet => $id_objet], ['vu' => 'non']); } } @@ -131,15 +132,17 @@ function formidable_post_edition($flux) { * @return mixed */ function formidable_affiche_droite($flux) { - if ($e = trouver_objet_exec($flux['args']['exec']) + if ( + $e = trouver_objet_exec($flux['args']['exec']) and isset($e['type']) and $objet = $e['type'] and isset($flux['args'][$e['id_table_objet']]) and $id = $flux['args'][$e['id_table_objet']] - and sql_countsel('spip_formulaires_liens', 'objet='.sql_quote($objet).' AND id_objet='.intval($id))) { + and sql_countsel('spip_formulaires_liens', 'objet=' . sql_quote($objet) . ' AND id_objet=' . intval($id)) + ) { $flux['data'] .= recuperer_fond( 'prive/squelettes/inclure/formulaires_lies', - array('objet' => $objet, 'id_objet' => $id) + ['objet' => $objet, 'id_objet' => $id] ); } return $flux; @@ -156,26 +159,27 @@ function formidable_affiche_milieu($flux) { $texte = ''; $e = trouver_objet_exec($flux['args']['exec']); - if ($e and !$e['edition'] and isset($flux['args'][$e['id_table_objet']]) and in_array($e['table_objet_sql'], lire_config('formidable/objets', array()))) { - $texte .= recuperer_fond('prive/objets/editer/liens', array( + if ($e and !$e['edition'] and isset($flux['args'][$e['id_table_objet']]) and in_array($e['table_objet_sql'], lire_config('formidable/objets', []))) { + $texte .= recuperer_fond('prive/objets/editer/liens', [ 'table_source' => 'formulaires', 'objet' => $e['type'], 'id_objet' => $flux['args'][$e['id_table_objet']] - )); + ]); } if ($texte) { - if ($p=strpos($flux['data'], '<!--affiche_milieu-->')) { + if ($p = strpos($flux['data'], '<!--affiche_milieu-->')) { $flux['data'] = substr_replace($flux['data'], $texte, $p, 0); } else { $flux['data'] .= $texte; } } - if (in_array($flux['args']['exec'], array('accueil', 'suivi_edito')) + if ( + in_array($flux['args']['exec'], ['accueil', 'suivi_edito']) and lire_config('formidable/reponses_page_accueil') ) { - $flux['data'] .= recuperer_fond('prive/objets/liste/formulaires_reponses', array(), array('ajax' => true)); + $flux['data'] .= recuperer_fond('prive/objets/liste/formulaires_reponses', [], ['ajax' => true]); } return $flux; } @@ -194,12 +198,12 @@ function formidable_optimiser_base_disparus($flux) { $res = sql_select( 'id_formulaire AS id', 'spip_formulaires', - 'statut='.sql_quote('poubelle') + 'statut=' . sql_quote('poubelle') ); $res2 = sql_select( 'id_formulaire AS id', 'spip_formulaires', - 'statut='.sql_quote('poubelle') + 'statut=' . sql_quote('poubelle') );//Copie pour supprimer les fichiers // On génère la suppression @@ -222,12 +226,12 @@ function formidable_optimiser_base_disparus($flux) { $res = sql_select( 'id_formulaires_reponse AS id, id_formulaire AS form', 'spip_formulaires_reponses', - 'statut='.sql_quote('poubelle') + 'statut=' . sql_quote('poubelle') ); $res2 = sql_select( 'id_formulaires_reponse AS id, id_formulaire AS form', 'spip_formulaires_reponses', - 'statut='.sql_quote('poubelle') + 'statut=' . sql_quote('poubelle') ); //Copie pour la suppression des fichiers des réponses, c'est idiot de pas pouvoir faire une seule requete // On génère la suppression $flux['data'] += optimiser_sansref('spip_formulaires_reponses', 'id_formulaires_reponse', $res); @@ -268,7 +272,7 @@ function formidable_formidable_traitements($flux) { $position_enregistrement = array_search('enregistrement', $keys); if ($position_enregistrement > $position_email) { // si enregistrement après email - $nouveau_tab = array(); + $nouveau_tab = []; foreach ($keys as $key) { //on reconstruit le tableau, en inversant simplement email et enregistrement if ($key == 'email') { $nouveau_tab['enregistrement'] = $flux['data']['enregistrement']; @@ -290,9 +294,9 @@ function formidable_formidable_traitements($flux) { * @return array $flux **/ function formidable_taches_generales_cron($flux) { - $flux['formidable_hasher_ip'] = 24*3600; - $flux['formidable_effacer_fichiers_email'] = 24*3600; - $flux['formidable_effacer_enregistrements'] = 24*3600; + $flux['formidable_hasher_ip'] = 24 * 3600; + $flux['formidable_effacer_fichiers_email'] = 24 * 3600; + $flux['formidable_effacer_enregistrements'] = 24 * 3600; return $flux; } @@ -302,15 +306,15 @@ function formidable_taches_generales_cron($flux) { * @return array $flux; **/ function formidable_corbeille_table_infos($flux) { - $flux['formulaires']= array( - 'statut'=>'poubelle', - 'table'=>'formulaires', - 'tableliee'=>array('spip_formulaires_liens') - ); - $flux['formulaires_reponses']= array( - 'statut'=>'poubelle', - 'table'=>'formulaires_reponses' - ); + $flux['formulaires'] = [ + 'statut' => 'poubelle', + 'table' => 'formulaires', + 'tableliee' => ['spip_formulaires_liens'] + ]; + $flux['formulaires_reponses'] = [ + 'statut' => 'poubelle', + 'table' => 'formulaires_reponses' + ]; return $flux; } @@ -366,15 +370,15 @@ function formidable_crayons_verifier($flux) { $saisies = formidable_deserialize($data['saisies']); $saisie = saisies_chercher($saisies, $data['nom']); if (saisies_saisie_est_tabulaire($saisie)) { - $atrouver = 'content_'.$flux['args']['wid'].'_valeur'; + $atrouver = 'content_' . $flux['args']['wid'] . '_valeur'; $valeur = saisies_request($atrouver); $flux['data']['normaliser']['valeur'] = formidable_serialize($valeur); } saisies_set_request($data['nom'], $valeur); - $erreur = saisies_verifier(array($saisie), false); + $erreur = saisies_verifier([$saisie], false); if ($erreur) { - $flux['data']['erreurs']['valeur'] = implode('<br />',$erreur); + $flux['data']['erreurs']['valeur'] = implode('<br />', $erreur); } elseif (saisies_request($data['nom']) !== $valeur) { $flux['data']['normaliser']['valeur'] = $normaliser; } @@ -397,7 +401,7 @@ function formidable_crayons_verifier($flux) { **/ function formulaires_reponses_champ_revision($id, $champs, $type, $ref) { include_spip('inc/modifier'); - return objet_modifier_champs($type, $id, array(), $champs); + return objet_modifier_champs($type, $id, [], $champs); } /** diff --git a/formulaires/editer_formulaire.php b/formulaires/editer_formulaire.php index 5b33d734..a737b592 100644 --- a/formulaires/editer_formulaire.php +++ b/formulaires/editer_formulaire.php @@ -12,33 +12,33 @@ include_spip('inc/config'); function formulaires_editer_formulaire_saisies() { - $saisies = array( - array( + $saisies = [ + [ 'saisie' => 'input', - 'options' => array( + 'options' => [ 'nom' => 'titre', 'obligatoire' => 'on', 'label' => '<:formidable:editer_titre:>' - ) - ), - array( + ] + ], + [ 'saisie' => 'input', - 'options' => array ( + 'options' => [ 'nom' => 'identifiant', 'obligatoire' => 'on', 'label' => '<:formidable:editer_identifiant:>', 'explication' => '<:formidable:editer_identifiant_explication:>', - ), - 'verifier' => array( + ], + 'verifier' => [ 'type' => 'regex', - 'options' => array( + 'options' => [ 'modele' => '/^[\w]+$/' - ) - ), - ), - array( + ] + ], + ], + [ 'saisie' => 'textarea', - 'options' => array ( + 'options' => [ 'nom' => 'message_retour', 'label' => '<:formidable:editer_message_ok:>', 'explication' => '<:formidable:editer_message_ok_explication:>', @@ -46,70 +46,70 @@ function formulaires_editer_formulaire_saisies() { 'rows' => '5', 'inserer_barre' => 'edition', 'previsualisation' => 'oui', - ) - ), - array( + ] + ], + [ 'saisie' => 'textarea', - 'options' => array ( + 'options' => [ 'nom' => 'descriptif', 'label' => '<:formidable:editer_descriptif:>', 'explication' => '<:formidable:editer_descriptif_explication:>', 'rows' => '5', - ) - ), - array( + ] + ], + [ 'saisie' => 'input', - 'options' => array ( + 'options' => [ 'nom' => 'css', 'label' => '<:formidable:editer_css:>', 'rows' => '5', - ) - ), - array( + ] + ], + [ 'saisie' => 'selection', - 'options' => array ( + 'options' => [ 'nom' => 'apres', 'label' => '<:formidable:editer_apres_label:>', 'explication' => '<:formidable:editer_apres_explication:>', - 'data' => array( + 'data' => [ 'formulaire' => '<:formidable:editer_apres_choix_formulaire:>', 'valeurs' => '<:formidable:editer_apres_choix_valeurs:>', 'stats' => '<:formidable:editer_apres_choix_stats:>', 'rien' => '<:formidable:editer_apres_choix_rien:>', 'redirige' => '<:formidable:editer_apres_choix_redirige:>' - ), + ], 'cacher_option_intro' => 'on', 'defaut' => 'formulaire', - ) - ), - array( + ] + ], + [ 'saisie' => 'input', - 'options' => array( + 'options' => [ 'nom' => 'url_redirect', 'label' => '<:formidable:editer_redirige_url:>', 'obligatoire' => 'on', 'afficher_si' => '@apres@ == "redirige"', - ), - 'verifier' => array( + ], + 'verifier' => [ 'type' => 'url', - 'options' => array( + 'options' => [ 'type_protocole' => 'tous', - ) - ) - ), + ] + ] + ], - ); + ]; return $saisies; } function formulaires_editer_formulaire_charger($id_formulaire, $nouveau) { - $id_formulaire = intval($nouveau?0:$id_formulaire); + $id_formulaire = intval($nouveau ? 0 : $id_formulaire); include_spip('inc/editer'); // Est-ce qu'on a le droit ? if (!autoriser('editer', 'formulaire', $id_formulaire)) { - $contexte = array(); + $contexte = []; $contexte['editable'] = false; $contexte['message_erreur'] = _T('formidable:erreur_autorisation'); } else { @@ -121,13 +121,13 @@ function formulaires_editer_formulaire_charger($id_formulaire, $nouveau) { } function formulaires_editer_formulaire_verifier($id_formulaire, $nouveau) { - $id_formulaire = intval($nouveau?0:$id_formulaire); - $erreurs = array(); + $id_formulaire = intval($nouveau ? 0 : $id_formulaire); + $erreurs = []; include_spip('inc/editer'); if (!isset($erreurs['identifiant'])) { $identifiant = _request('identifiant'); - if (sql_getfetsel('id_formulaire', 'spip_formulaires', 'identifiant = '.sql_quote($identifiant).' AND id_formulaire != '.intval($id_formulaire))) { + if (sql_getfetsel('id_formulaire', 'spip_formulaires', 'identifiant = ' . sql_quote($identifiant) . ' AND id_formulaire != ' . intval($id_formulaire))) { // unicite de l'identifiant $erreurs['identifiant'] = _T('formidable:erreur_identifiant'); } @@ -150,7 +150,7 @@ function formulaires_editer_formulaire_traiter($id_formulaire, $nouveau) { if ($auteurs == 'on') { if ($id_auteur = session_get('id_auteur')) { // association (par défaut) du formulaire et de l'auteur courant - objet_associer(array('formulaire'=>$retours['id_formulaire']), array('auteur'=>$id_auteur)); + objet_associer(['formulaire' => $retours['id_formulaire']], ['auteur' => $id_auteur]); } } $retours['redirect'] = parametre_url(generer_url_ecrire('formulaire_edit'), 'id_formulaire', $retours['id_formulaire'], '&'); diff --git a/formulaires/editer_formulaire_champs.php b/formulaires/editer_formulaire_champs.php index c63d90bd..2e11f66f 100644 --- a/formulaires/editer_formulaire_champs.php +++ b/formulaires/editer_formulaire_champs.php @@ -7,13 +7,14 @@ if (!defined('_ECRIRE_INC_VERSION')) { function formulaires_editer_formulaire_champs_charger($id_formulaire) { $id_formulaire = intval($id_formulaire); - $contexte = array(); + $contexte = []; $contexte['id_formulaire'] = $id_formulaire; include_spip('formidable_fonctions'); // On teste si le formulaire existe - if ($id_formulaire - and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id_formulaire) + if ( + $id_formulaire + and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire) and autoriser('editer', 'formulaire', $id_formulaire) ) { $saisies = formidable_deserialize($formulaire['saisies']); @@ -25,131 +26,131 @@ function formulaires_editer_formulaire_champs_charger($id_formulaire) { $saisies = formidable_deserialize($old['saisies']); } if (!is_array($saisies)) { - $saisies = array(); + $saisies = []; } $contexte['_saisies'] = $saisies; $contexte['id'] = $id_formulaire; $contexte['saisie_id'] = "formidable_$id_formulaire"; // Les options globales que l'on permet de configurer pour le contexte de Formidables - $contexte['_options_globales'] = array( - array( + $contexte['_options_globales'] = [ + [ 'saisie' => 'fieldset', - 'options' => array( + 'options' => [ 'nom' => 'submit', 'label' => _T('formidable:editer_globales_submit_label') - ), - 'saisies' => array( - array( + ], + 'saisies' => [ + [ 'saisie' => 'input', - 'options' => array( + 'options' => [ 'nom' => 'texte_submit', 'label' => _T('formidable:editer_globales_texte_submit_label'), - ), - ), - array( + ], + ], + [ 'saisie' => 'textarea', - 'options' => array( + 'options' => [ 'nom' => 'afficher_si_submit', 'explication' => _T('saisies:option_afficher_si_explication'), 'label' => _T('formidable:editer_globales_afficher_si_submit_label'), 'rows' => '10', - ), - 'verifier' => array( + ], + 'verifier' => [ 'type' => 'afficher_si', - ) - ) - ) - ), - array( + ] + ] + ] + ], + [ 'saisie' => 'fieldset', - 'options' => array( + 'options' => [ 'nom' => 'etapes', 'label' => _T('formidable:editer_globales_etapes_label') - ), - 'saisies' => array( - array( + ], + 'saisies' => [ + [ 'saisie' => 'case', - 'options' => array( + 'options' => [ 'nom' => 'etapes_activer', 'label_case' => _T('formidable:editer_globales_etapes_activer_label_case'), 'conteneur_class' => 'pleine_largeur', 'explication' => _T('formidable:editer_globales_etapes_activer_explication'), - ), - ), - array( + ], + ], + [ 'saisie' => 'radio', - 'options' => array( + 'options' => [ 'nom' => 'etapes_presentation', 'label' => _T('formidable:editer_globales_etapes_presentation_label'), 'afficher_si' => '@etapes_activer@', - 'afficher_si_avec_post' => True, - 'data' => array( + 'afficher_si_avec_post' => true, + 'data' => [ 'defaut' => _T('formidable:editer_globales_etapes_presentation_defaut_label'), 'courante' => _T('formidable:editer_globales_etapes_presentation_courante_label'), - ), + ], 'defaut' => 'defaut', - ), - ), - array( + ], + ], + [ 'saisie' => 'input', - 'options' => array( + 'options' => [ 'nom' => 'etapes_suivant', 'label' => _T('formidable:editer_globales_etapes_suivant_label'), 'afficher_si' => '@etapes_activer@', - 'afficher_si_avec_post' => True, - ), - ), - array( + 'afficher_si_avec_post' => true, + ], + ], + [ 'saisie' => 'input', - 'options' => array( + 'options' => [ 'nom' => 'etapes_precedent', 'label' => _T('formidable:editer_globales_etapes_precedent_label'), 'afficher_si' => '@etapes_activer@', - 'afficher_si_avec_post' => True, - ), - ), - array( + 'afficher_si_avec_post' => true, + ], + ], + [ 'saisie' => 'case', - 'options' => array( + 'options' => [ 'nom' => 'etapes_precedent_suivant_titrer', 'label_case' => _T('formidable:editer_globales_etapes_precedent_suivant_titrer_label'), 'afficher_si' => '@etapes_activer@', - 'afficher_si_avec_post' => True, + 'afficher_si_avec_post' => true, 'conteneur_class' => 'pleine_largeur' - ) - ), - array( + ] + ], + [ 'saisie' => 'case', - 'options' => array( + 'options' => [ 'nom' => 'etapes_ignorer_recapitulatif', 'label_case' => _T('formidable:editer_globales_etapes_ignorer_recapitulatif_label_case'), 'afficher_si' => '@etapes_activer@', - 'afficher_si_avec_post' => True, + 'afficher_si_avec_post' => true, 'conteneur_class' => 'pleine_largeur', - ), - ), - ) - ), - array( + ], + ], + ] + ], + [ 'saisie' => 'fieldset', - 'options' => array( + 'options' => [ 'nom' => 'technique', 'label' => _T('formidable:editer_globales_technique_label') - ), - 'saisies' => array( - array( + ], + 'saisies' => [ + [ 'saisie' => 'case', - 'options' => array( + 'options' => [ 'nom' => 'verifier_valeurs_acceptables', 'label_case' => _T('saisies:verifier_valeurs_acceptables_label'), 'conteneur_class' => 'pleine_largeur', 'explication' => _T('saisies:verifier_valeurs_acceptables_explication'), - ) - ), - ) - ) - ); + ] + ], + ] + ] + ]; } return $contexte; @@ -157,22 +158,23 @@ function formulaires_editer_formulaire_champs_charger($id_formulaire) { function formulaires_editer_formulaire_champs_verifier($id_formulaire) { include_spip('inc/saisies'); - $erreurs = array(); + $erreurs = []; include_spip('formidable_fonctions'); // Si c'est pas une confirmation ni une annulation, ni un revert - if (!_request('enregistrer_confirmation') + if ( + !_request('enregistrer_confirmation') and !($annulation = _request('annulation')) and !_request('revert') - ) { + ) { // On récupère le formulaire dans la session $saisies_nouvelles = session_get("constructeur_formulaire_formidable_$id_formulaire"); - $md5_precedent_formulaire_initial = session_get("constructeur_formulaire_formidable_$id_formulaire".'_md5_formulaire_initial'); + $md5_precedent_formulaire_initial = session_get("constructeur_formulaire_formidable_$id_formulaire" . '_md5_formulaire_initial'); // On récupère les anciennes saisies $saisies_anciennes = sql_getfetsel( 'saisies', 'spip_formulaires', - 'id_formulaire = '.$id_formulaire + 'id_formulaire = ' . $id_formulaire ); if (!$saisies_anciennes) { return $erreurs; @@ -189,7 +191,7 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) { $md5_saisies_anciennes = md5(serialize($saisies_anciennes_str)); if ($md5_precedent_formulaire_initial and $md5_precedent_formulaire_initial != $md5_saisies_anciennes and !_request('id_version')) { session_set("constructeur_formulaire_formidable_$id_formulaire", $saisies_anciennes); - session_set("constructeur_formulaire_formidable_$id_formulaire".'_md5_formulaire_initial', $md5_saisies_anciennes); + session_set("constructeur_formulaire_formidable_$id_formulaire" . '_md5_formulaire_initial', $md5_saisies_anciennes); $erreurs['message_erreur'] = _T('formidable:erreur_saisies_modifiees_parallele'); $erreurs['saisies_modifiees_parallele'] = _T('formidable:erreur_saisies_modifiees_parallele'); return $erreurs; @@ -206,9 +208,9 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) { //On vérifie s'il y a pas d'incohérence dans les afficher_si $erreurs_afficher_si = saisies_verifier_coherence_afficher_si($saisies_nouvelles ?? []); if ($erreurs_afficher_si) { - $erreurs['saisies_incoherence_afficher_si'] = True; + $erreurs['saisies_incoherence_afficher_si'] = true; if ($erreurs['message_erreur'] ?? '') { - $erreurs['message_erreur'] .= '<br />'.$erreurs_afficher_si; + $erreurs['message_erreur'] .= '<br />' . $erreurs_afficher_si; } else { $erreurs['message_erreur'] = $erreurs_afficher_si; } @@ -227,13 +229,13 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) { function formulaires_editer_formulaire_champs_traiter($id_formulaire) { include_spip('inc/saisies'); include_spip('inc/formidable'); - $retours = array(); + $retours = []; $id_formulaire = intval($id_formulaire); include_spip('formidable_fonctions'); if (_request('revert')) { session_set("constructeur_formulaire_formidable_$id_formulaire"); - $retours = array('editable'=>true); + $retours = ['editable' => true]; } if (_request('enregistrer') or _request('enregistrer_confirmation')) { @@ -244,24 +246,24 @@ function formulaires_editer_formulaire_champs_traiter($id_formulaire) { $saisies_anciennes = formidable_deserialize(sql_getfetsel( 'saisies', 'spip_formulaires', - 'id_formulaire = '.$id_formulaire + 'id_formulaire = ' . $id_formulaire )); // On envoie les nouvelles dans la table include_spip('action/editer_objet'); - $err = objet_modifier('formulaire', $id_formulaire, array('saisies' => formidable_serialize(saisies_identifier($saisies_nouvelles)))); + $err = objet_modifier('formulaire', $id_formulaire, ['saisies' => formidable_serialize(saisies_identifier($saisies_nouvelles))]); // Si c'est bon on appelle d'éventuelles fonctions d'update des traitements // On reinitialise aussi les sessions // puis on renvoie vers la config des traitements if (!$err) { session_set("constructeur_formulaire_formidable_$id_formulaire"); - session_set("constructeur_formulaire_formidable_$id_formulaire".'_md5_formulaire_initial'); + session_set("constructeur_formulaire_formidable_$id_formulaire" . '_md5_formulaire_initial'); // On va chercher les traitements $traitements = formidable_deserialize(sql_getfetsel( 'traitements', 'spip_formulaires', - 'id_formulaire = '.$id_formulaire + 'id_formulaire = ' . $id_formulaire )); // Pour chaque traitements on regarde s'i y a une fonction d'update diff --git a/formulaires/editer_formulaire_traitements.php b/formulaires/editer_formulaire_traitements.php index e5eab33a..a3627d7e 100644 --- a/formulaires/editer_formulaire_traitements.php +++ b/formulaires/editer_formulaire_traitements.php @@ -9,13 +9,14 @@ include_spip('inc/formidable'); include_spip('inc/formidable_fichiers'); function formulaires_editer_formulaire_traitements_charger($id_formulaire) { - $contexte = array(); + $contexte = []; $id_formulaire = intval($id_formulaire); include_spip('formidable_fonctions'); // On teste si le formulaire existe - if ($id_formulaire - and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id_formulaire) + if ( + $id_formulaire + and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire) and autoriser('editer', 'formulaire', $id_formulaire) ) { $traitements = formidable_deserialize($formulaire['traitements']); @@ -26,10 +27,10 @@ function formulaires_editer_formulaire_traitements_charger($id_formulaire) { } $saisies = formidable_deserialize($formulaire['saisies']); if (!is_array($traitements)) { - $traitements = array(); + $traitements = []; } if (!is_array($saisies)) { - $saisies = array(); + $saisies = []; } $contexte['traitements'] = $traitements; $contexte['traitements_choisis'] = array_keys($traitements); @@ -112,7 +113,7 @@ function formulaires_editer_formulaire_traitements_charger($id_formulaire) { function formulaires_editer_formulaire_traitements_verifier($id_formulaire) { include_spip('inc/saisies'); - $erreurs = array(); + $erreurs = []; $traitements_disponibles = traitements_lister_disponibles(); // On regarde quels traitements sont demandés @@ -127,7 +128,8 @@ function formulaires_editer_formulaire_traitements_verifier($id_formulaire) { saisies_transformer_noms( $traitements_disponibles[$type_traitement]['options'], '/^.*$/', - "traitements[$type_traitement][\\0]") + "traitements[$type_traitement][\\0]" + ) ); $erreurs = array_merge($erreurs, $erreurs_traitement_brut); } @@ -136,7 +138,7 @@ function formulaires_editer_formulaire_traitements_verifier($id_formulaire) { } function formulaires_editer_formulaire_traitements_traiter($id_formulaire) { - $retours = array(); + $retours = []; $id_formulaire = intval($id_formulaire); include_spip('inc/formidable'); @@ -146,14 +148,16 @@ function formulaires_editer_formulaire_traitements_traiter($id_formulaire) { $traitements_choisis = _request('traitements_choisis'); if (!$traitements_choisis) { - $traitements_choisis = array(); + $traitements_choisis = []; } $traitements_choisis = array_flip($traitements_choisis); // Parmis toutes les options passées, ne prendre que celle des traitements choisis $traitements = array_intersect_key($traitements, $traitements_choisis); // On gère le cas des traitements sans options - $traitements_sans_option = array_map(function($i) {return [];}, array_diff_key($traitements_choisis, $traitements));//Même si pas d'option, on fait un pseudo tableau d'option + $traitements_sans_option = array_map(function ($i) { +return []; + }, array_diff_key($traitements_choisis, $traitements));//Même si pas d'option, on fait un pseudo tableau d'option $traitements = array_merge($traitements, $traitements_sans_option); // Si besoin, on crée un dossier pour stocker les fichiers @@ -163,7 +167,7 @@ function formulaires_editer_formulaire_traitements_traiter($id_formulaire) { } // Et on l'enregistre tel quel include_spip('action/editer_objet'); - $err = objet_modifier('formulaire', $id_formulaire, array('traitements' => formidable_serialize($traitements))); + $err = objet_modifier('formulaire', $id_formulaire, ['traitements' => formidable_serialize($traitements)]); // On va sur la page de visualisation quand c'est fini if (!$err) { diff --git a/formulaires/editer_formulaires_reponse.php b/formulaires/editer_formulaires_reponse.php index 1daa0441..8968adaf 100644 --- a/formulaires/editer_formulaires_reponse.php +++ b/formulaires/editer_formulaires_reponse.php @@ -9,7 +9,7 @@ function formulaires_editer_formulaires_reponse_charger($id_formulaires_reponse, include_spip('inc/editer'); // Est-ce qu'on a le droit ? if (!autoriser('modifier', 'formulaires_reponse', $id_formulaires_reponse)) { - $contexte = array(); + $contexte = []; $contexte['editable'] = false; $contexte['message_erreur'] = _T('formidable:erreur_autorisation'); } else { diff --git a/formulaires/exporter_formulaire_analyse.php b/formulaires/exporter_formulaire_analyse.php index 164c5173..961fde36 100644 --- a/formulaires/exporter_formulaire_analyse.php +++ b/formulaires/exporter_formulaire_analyse.php @@ -9,23 +9,23 @@ include_spip('inc/formidable'); include_spip('inc/config'); function formulaires_exporter_formulaire_analyse_charger($id_formulaire = 0) { - $contexte = array(); + $contexte = []; $contexte['id_formulaire'] = intval($id_formulaire); return $contexte; } function formulaires_exporter_formulaire_analyse_verifier($id_formulaire = 0) { - $erreurs = array(); + $erreurs = []; return $erreurs; } function formulaires_exporter_formulaire_analyse_traiter($id_formulaire = 0) { - $retours = array(); + $retours = []; if (_request('type_export') == 'csv') { action_exporter_analyse_reponses($id_formulaire); - } else if (_request('type_export') == 'xls') { + } elseif (_request('type_export') == 'xls') { action_exporter_analyse_reponses($id_formulaire, 'TAB'); } @@ -42,14 +42,15 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { // on ne fait des choses seulements si le formulaire existe et qu'il a des enregistrements include_spip('formidable_fonctions'); $ok = false; - if ($id_formulaire > 0 - and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id_formulaire) - and $reponses = sql_allfetsel('*', 'spip_formulaires_reponses', 'id_formulaire = '.$id_formulaire.' and statut = '.sql_quote('publie')) + if ( + $id_formulaire > 0 + and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire) + and $reponses = sql_allfetsel('*', 'spip_formulaires_reponses', 'id_formulaire = ' . $id_formulaire . ' and statut = ' . sql_quote('publie')) ) { include_spip('inc/saisies'); include_spip('classes/facteur'); include_spip('inc/filtres'); - $reponses_completes = array(); + $reponses_completes = []; $saisies = saisies_lister_par_nom(formidable_deserialize($formulaire['saisies']), false); @@ -59,7 +60,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { unset($saisies[$exclure]); } $res = sql_select( - array('nom, valeur'), + ['nom, valeur'], 'spip_formulaires_reponses_champs AS FRC, spip_formulaires_reponses AS FR, spip_formulaires AS F', @@ -68,7 +69,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { AND F.id_formulaire=$id_formulaire" ); - $valeurs = array(); + $valeurs = []; while ($r = sql_fetch($res)) { $valeurs[$r['nom']][] = formidable_deserialize($r['valeur']); } @@ -77,14 +78,14 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { $valeur = sql_getfetsel( 'valeur', 'spip_formulaires_reponses_champs', - 'id_formulaires_reponse = '.intval($reponse['id_formulaires_reponse']).' and nom = '.sql_quote($nom) + 'id_formulaires_reponse = ' . intval($reponse['id_formulaires_reponse']) . ' and nom = ' . sql_quote($nom) ); $valeur = formidable_deserialize($valeur); $reponse_complete[] = formidable_analyser_saisie($saisie, $valeurs, 0, true); } - $colonnes = array(_T('formidable:champs'), _T('formidable:sans_reponses')); + $colonnes = [_T('formidable:champs'), _T('formidable:sans_reponses')]; foreach ($reponse_complete as $reponses) { foreach ($reponses as $key => $reponse) { if ($key == 'header' || $key == 'sans_reponse') { @@ -96,7 +97,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { } } - $csv = array(); + $csv = []; foreach ($reponse_complete as $reponses) { foreach ($colonnes as $colonne) { $csv[$reponses['header']][$colonne] = @@ -110,7 +111,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { } $cpt_ligne = 1; - $reponses_completes = array(); + $reponses_completes = []; $reponses_completes[0] = $colonnes; foreach ($csv as $ligne => $colonnes) { $cpt_colonne = 0; @@ -121,7 +122,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') { } if ($reponses_completes and $exporter_csv = charger_fonction('exporter_csv', 'inc/', true)) { - $exporter_csv('analyses-formulaire-'.$formulaire['identifiant'], $reponses_completes, $delim); + $exporter_csv('analyses-formulaire-' . $formulaire['identifiant'], $reponses_completes, $delim); exit(); } } diff --git a/formulaires/exporter_formulaire_reponses.php b/formulaires/exporter_formulaire_reponses.php index 95fc82e1..bce3e367 100644 --- a/formulaires/exporter_formulaire_reponses.php +++ b/formulaires/exporter_formulaire_reponses.php @@ -10,14 +10,14 @@ include_spip('inc/formidable_fichiers'); include_spip('inc/config'); function formulaires_exporter_formulaire_reponses_charger($id_formulaire = 0) { - $contexte = array(); + $contexte = []; $contexte['id_formulaire'] = intval($id_formulaire); include_spip('formidable_fonctions'); // Vérifier si le formulaire dispose de champs "fichiers", pour proposer de ne pas les joindre à l'export - $saisies = sql_getfetsel('saisies','spip_formulaires','id_formulaire = '.intval($id_formulaire)); + $saisies = sql_getfetsel('saisies', 'spip_formulaires', 'id_formulaire = ' . intval($id_formulaire)); $saisies_par_type = saisies_lister_par_type(formidable_deserialize($saisies), false); - if(isset($saisies_par_type['fichiers'])){ + if (isset($saisies_par_type['fichiers'])) { $contexte['fichiers'] = true; } @@ -25,7 +25,7 @@ function formulaires_exporter_formulaire_reponses_charger($id_formulaire = 0) { } function formulaires_exporter_formulaire_reponses_verifier($id_formulaire = 0) { - $erreurs = array(); + $erreurs = []; if (_request('date_debut') && _request('date_fin')) { // Vérifions que la date debut soit < date de fin @@ -38,12 +38,12 @@ function formulaires_exporter_formulaire_reponses_verifier($id_formulaire = 0) { } function formulaires_exporter_formulaire_reponses_traiter($id_formulaire = 0) { - $retours = array(); + $retours = []; $statut_reponses = _request('statut_reponses'); // Normaliser la date $verifier = charger_fonction('verifier', 'inc/'); - $verifier(_request('date_debut'), 'date', array('normaliser' => 'datetime'), $date_debut); - $verifier(_request('date_fin'), 'date', array('normaliser' => 'datetime'), $date_fin); + $verifier(_request('date_debut'), 'date', ['normaliser' => 'datetime'], $date_debut); + $verifier(_request('date_fin'), 'date', ['normaliser' => 'datetime'], $date_fin); include_spip('inc/filtres'); $date_debut = vider_date($date_debut); @@ -57,12 +57,12 @@ function formulaires_exporter_formulaire_reponses_traiter($id_formulaire = 0) { if (_request('type_export') == 'csv') { $chemin = exporter_formulaires_reponses($id_formulaire, ',', $statut_reponses, $date_debut, $date_fin, $cle_ou_valeur, $ignorer_fichiers); if (pathinfo($chemin, PATHINFO_EXTENSION) === 'csv') { - $content_type = "text/comma-separated-values; charset=" . $GLOBALS['meta']['charset']; + $content_type = 'text/comma-separated-values; charset=' . $GLOBALS['meta']['charset']; } } elseif (_request('type_export') == 'xls') { $chemin = exporter_formulaires_reponses($id_formulaire, ';', $statut_reponses, $date_debut, $date_fin, $cle_ou_valeur, $ignorer_fichiers); if (pathinfo($chemin, PATHINFO_EXTENSION) === 'xls') { - $content_type = "text/comma-separated-values; charset=iso-8859-1"; + $content_type = 'text/comma-separated-values; charset=iso-8859-1'; } } @@ -104,7 +104,7 @@ function exporter_formulaires_reponses($id_formulaire, $delim = ',', $statut_rep return false; } - $fichier_csv = $exporter_csv('reponses-formulaire-' . $formulaire['identifiant'], $reponses_completes, ['delim'=>$delim, 'envoyer'=>false]); + $fichier_csv = $exporter_csv('reponses-formulaire-' . $formulaire['identifiant'], $reponses_completes, ['delim' => $delim, 'envoyer' => false]); // si pas de saisie fichiers, on envoie directement le csv if (!count($saisies_fichiers)) { @@ -127,13 +127,14 @@ function obtenir_formulaire_reponses($id_formulaire, $statut_reponses = 'publie' 'spip_formulaires_reponses', 'id_formulaire = ' . intval($id_formulaire) . ($statut_reponses == 'publie' ? ' and statut = "publie"' : '') . (strlen($date_debut) > 0 ? ' and date >= "' . $date_debut . '"' : '') - . (strlen($date_fin) > 0 ? ' and date <= "' . date('Y-m-d H:i:s', strtotime($date_fin. ' + 1 days')) . '"' : ''), + . (strlen($date_fin) > 0 ? ' and date <= "' . date('Y-m-d H:i:s', strtotime($date_fin . ' + 1 days')) . '"' : ''), '', 'date' - )) { - return array($formulaire, $reponses); + ) + ) { + return [$formulaire, $reponses]; } - return array(null, null); + return [null, null]; } @@ -145,17 +146,17 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, include_spip('formidable_fonctions'); $id_formulaire = $formulaire['id_formulaire']; - $reponses_completes = array(); - $saisies_fichiers = array(); + $reponses_completes = []; + $saisies_fichiers = []; // La première ligne des titres - $titres = array( + $titres = [ _T('formidable:id_formulaires_reponse'), _T('public:date'), _T('formidable:reponses_auteur'), - ); + ]; - if(lire_config('formidable/exporter_adresses_ip')){ + if (lire_config('formidable/exporter_adresses_ip')) { $titres[] = _T('formidable:reponses_ip'); } @@ -180,10 +181,10 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, // On passe la ligne des titres de colonnes dans un pipeline $titres = pipeline( 'formidable_exporter_formulaire_reponses_titres', - array( - 'args' => array('id_formulaire' => $id_formulaire, 'formulaire' => $formulaire), + [ + 'args' => ['id_formulaire' => $id_formulaire, 'formulaire' => $formulaire], 'data' => $titres, - ) + ] ); $reponses_completes[] = $titres; @@ -198,19 +199,19 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, $_reponses_valeurs = sql_allfetsel( 'id_formulaires_reponse, nom, valeur', 'spip_formulaires_reponses_champs', - array( + [ sql_in('id_formulaires_reponse', $ids_reponses), //sql_in('nom', array_keys($saisies)) // ralentit la requête, et inutile - ), + ], '', 'id_formulaires_reponse ASC' ); // grouper par identifiant de réponse - $reponses_valeurs = array(); + $reponses_valeurs = []; foreach ($_reponses_valeurs as $r) { if (empty($reponses_valeurs[$r['id_formulaires_reponse']])) { - $reponses_valeurs[$r['id_formulaires_reponse']] = array(); + $reponses_valeurs[$r['id_formulaires_reponse']] = []; } $reponses_valeurs[$r['id_formulaires_reponse']][$r['nom']] = $r['valeur']; } @@ -228,11 +229,11 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, } // Le début de la réponse avec les infos (date, auteur, etc) - $reponse_complete = array( + $reponse_complete = [ $reponse['id_formulaires_reponse'], $reponse['date'], $nom_auteur, - ); + ]; if (lire_config('formidable/exporter_adresses_ip')) { $reponse_complete[] = $reponse['ip']; } @@ -245,7 +246,6 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, foreach ($saisies as $nom => $saisie) { if ($saisie['saisie'] != 'explication') { - // Saisie de type fichier ? if (!$ignorer_fichiers && $saisie['saisie'] == 'fichiers') { $_valeurs = formidable_deserialize($valeurs[$nom]); @@ -269,21 +269,21 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, // On passe la ligne de réponse dans un pipeline $reponse_complete = pipeline( 'formidable_exporter_formulaire_reponses_reponse', - array( - 'args' => array( + [ + 'args' => [ 'id_formulaire' => $id_formulaire, 'formulaire' => $formulaire, 'reponse' => $reponse, - ), + ], 'data' => $reponse_complete, - ) + ] ); // On ajoute la ligne à l'ensemble des réponses $reponses_completes[] = $reponse_complete; } - return array($reponses_completes, $saisies_fichiers); + return [$reponses_completes, $saisies_fichiers]; } @@ -299,7 +299,7 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses, * @return string */ function formidable_generer_valeur_texte_saisie($valeur, $saisie, $cle_ou_valeur = 'valeur') { - static $resultats = array(); + static $resultats = []; include_spip('formidable_fonctions'); @@ -312,17 +312,16 @@ function formidable_generer_valeur_texte_saisie($valeur, $saisie, $cle_ou_valeur recuperer_fond( 'saisies-vues/_base', array_merge( - array( + [ 'valeur_uniquement' => 'oui', 'type_saisie' => $saisie['saisie'], 'valeur' => $valeur, 'cle_ou_valeur' => $cle_ou_valeur - ), + ], $saisie['options'] ) ) ); - } return $resultats[$hash]; diff --git a/formulaires/formidable.php b/formulaires/formidable.php index b7ba15c2..081ad7eb 100644 --- a/formulaires/formidable.php +++ b/formulaires/formidable.php @@ -21,7 +21,7 @@ include_spip('plugins/installer'); function formidable_id_formulaire($id) { // on utilise une static pour etre sur que si l'appel dans verifier() passe, celui dans traiter() passera aussi // meme si entre temps on perds la base - static $id_formulaires = array(); + static $id_formulaires = []; if (isset($id_formulaires[$id])) { return $id_formulaires[$id]; } @@ -36,9 +36,11 @@ function formidable_id_formulaire($id) { $id_formulaire = intval(sql_getfetsel('id_formulaire', 'spip_formulaires', $where)); - if ($id_formulaire + if ( + $id_formulaire and !test_espace_prive() - and !objet_test_si_publie('formulaire', $id_formulaire)) { + and !objet_test_si_publie('formulaire', $id_formulaire) + ) { return $id_formulaires[$id] = 0; } @@ -131,7 +133,7 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel $contexte['_css'] = $formulaire['css']; // Est-ce que la personne a le droit de répondre ? - if (autoriser('repondre', 'formulaire', $formulaire['id_formulaire'], null, array('formulaire' => $formulaire))) { + if (autoriser('repondre', 'formulaire', $formulaire['id_formulaire'], null, ['formulaire' => $formulaire])) { $traitements = formidable_deserialize($formulaire['traitements']); $contexte['mechantrobot'] = ''; @@ -145,10 +147,10 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel if (is_string($valeurs)) { $liste = explode(',', $valeurs); $liste = array_map('trim', $liste); - $valeurs = array(); + $valeurs = []; foreach ($liste as $i => $cle_ou_valeur) { - if ($i%2 == 0) { - $valeurs[$liste[$i]] = $liste[$i+1]; + if ($i % 2 == 0) { + $valeurs[$liste[$i]] = $liste[$i + 1]; } } } @@ -176,13 +178,13 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel $contexte['editable'] = false; // le formulaire a déjà été répondu. // peut être faut il afficher les statistiques des réponses - if ($formulaire['apres']=='stats') { + if ($formulaire['apres'] == 'stats') { // Nous sommes face à un sondage auquel on a déjà répondu ! // On remplace complètement l'affichage du formulaire // par un affichage du résultat de sondage ! - $contexte['_remplacer_formulaire'] = recuperer_fond('modeles/formulaire_analyse', array( + $contexte['_remplacer_formulaire'] = recuperer_fond('modeles/formulaire_analyse', [ 'id_formulaire' => $formulaire['id_formulaire'], - )); + ]); } else { $contexte['message_erreur'] = _T('formidable:traiter_enregistrement_erreur_deja_repondu'); $contexte['message_erreur_class'] = 'deja_repondu'; @@ -209,7 +211,7 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel set_request('cvtupload_etapes_files', null); set_request('_fichiers', null); unset($contexte['cvtupload_precharger_fichiers']); - $_FILES = array(); + $_FILES = []; set_request('cvtupload_fichiers_precedents', null); } return $contexte; @@ -327,7 +329,7 @@ function formulaires_formidable_verifier_traitements($id, $valeurs = [], $option foreach ($traitements as $type_traitement => $options) { if ($verifier_traitement = charger_fonction('verifier', "traiter/$type_traitement", true)) { $erreurs_traitements = $verifier_traitement( - array( + [ 'formulaire' => $formulaire, 'options' => $options, 'traitements' => $traitements, @@ -338,7 +340,7 @@ function formulaires_formidable_verifier_traitements($id, $valeurs = [], $option 'options_appel' => $options_appel, 'etapes' => $etapes, 'etape' => $etape - ), + ], $erreurs ); $erreurs = array_merge($erreurs, $erreurs_traitements); @@ -379,7 +381,7 @@ function formulaires_formidable_verifier_traitements($id, $valeurs = [], $option * Tableau des erreurs **/ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel = [], $deprecated_url_redirect = false, $deprecated_forcer_modif = false) { - $retours = array(); + $retours = []; include_spip('formidable_fonctions'); $options_appel = formulaires_formidable_normaliser_options_appel($options_appel, $deprecated_url_redirect, $deprecated_forcer_modif); @@ -390,12 +392,12 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel include_spip('inc/texte'); // POST Mortem de securite : on log le $_POST pour ne pas le perdre si quelque chose se passe mal include_spip('inc/json'); - $post = json_encode(array('post' => $_POST, 'files' => $_FILES)); - spip_log($post, 'formidable_post'._LOG_INFO_IMPORTANTE); + $post = json_encode(['post' => $_POST, 'files' => $_FILES]); + spip_log($post, 'formidable_post' . _LOG_INFO_IMPORTANTE); // On peut donner soit un id soit un identifiant if (!$id_formulaire = formidable_id_formulaire($id)) { - return array('message_erreur'=>_T('formidable:erreur_base')); + return ['message_erreur' => _T('formidable:erreur_base')]; } $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire); @@ -408,16 +410,16 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel // On passe les traitements en pipelines, pour s'assurer de l'ordre le cas échéant $traitements = pipeline( 'formidable_traitements', - array( - 'args' => array( + [ + 'args' => [ 'id_formulaire' => $id_formulaire, 'identifiant' => $formulaire['identifiant'] - ), + ], 'data' => $traitements - ) + ] ); // selon le choix, le formulaire se remet en route à la fin ou non - $retours['editable'] = ($formulaire['apres']=='formulaire'); + $retours['editable'] = ($formulaire['apres'] == 'formulaire'); $retours['formidable_afficher_apres'] = $formulaire['apres']; $retours['id_formulaire'] = $id_formulaire; @@ -446,7 +448,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel if ($formulaire['message_retour']) { $messages_ok .= _T_ou_typo( formidable_raccourcis_arobases_2_valeurs_champs( - wrap(trim(propre($formulaire['message_retour'])),'<div class="message_retour_defaut">'), + wrap(trim(propre($formulaire['message_retour'])), '<div class="message_retour_defaut">'), $saisies, [ 'sans_reponse' => '', @@ -463,24 +465,24 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel // et que B n'est pas fait quand il est appele, il peut rendre la main sans rien faire au premier coup // et sera rappele au second tour // note : depuis l'ajout du pipeline formidable_traitement, cette fonctionnalité n'a plus grand sens, mais on la maintient tout de même pour historique - $retours['traitements'] = array(); + $retours['traitements'] = []; $erreur_texte = ''; // Si on a des traitements if (is_array($traitements) and !empty($traitements)) { $maxiter = 5; - spip_log("Début des traitements pour le formulaire $id_formulaire", 'formidable'._LOG_INFO); + spip_log("Début des traitements pour le formulaire $id_formulaire", 'formidable' . _LOG_INFO); do { foreach ($traitements as $type_traitement => $options) { // si traitement deja appele, ne pas le relancer if (!isset($retours['traitements'][$type_traitement])) { if ($appliquer_traitement = charger_fonction($type_traitement, 'traiter/', true)) { - spip_log("Application du traitement $type_traitement", 'formidable'._LOG_INFO); + spip_log("Application du traitement $type_traitement", 'formidable' . _LOG_INFO); $retours['scripts_ok'] = ''; $retours['message_ok'] = ''; $retours = $appliquer_traitement( - array( + [ 'formulaire' => $formulaire, 'options' => $options, 'options_appel' => $options_appel, @@ -491,14 +493,14 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel 'id_formulaires_reponse' => $id_formulaires_reponse, 'forcer_modif' => $forcer_modif, 'message_retour_general' => $formulaire['message_retour'] - ), + ], $retours ); // Collecter les messages de retour afin de les formater correctement par la suite if ( !empty($retours['message_ok']) ) { - $messages_ok .= wrap($retours['message_ok'],'<div class="message_retour_traitement message_retour_'.$type_traitement.'">'); + $messages_ok .= wrap($retours['message_ok'], '<div class="message_retour_traitement message_retour_' . $type_traitement . '">'); } $messages_ok .= $retours['scripts_ok']; } else { @@ -511,8 +513,8 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel // si on ne peut pas traiter correctement, alerter le webmestre if (count($retours['traitements']) < count($traitements)) { $erreur_texte = "Impossible de traiter correctement le formulaire $id\n" - . 'Traitements attendus :'.implode(',', array_keys($traitements))."\n" - . 'Traitements realises :'.implode(',', array_keys($retours['traitements']))."\n"; + . 'Traitements attendus :' . implode(',', array_keys($traitements)) . "\n" + . 'Traitements realises :' . implode(',', array_keys($retours['traitements'])) . "\n"; } } else { $retours['message_erreur'] = _T('formidable:retour_aucun_traitement'); @@ -525,13 +527,13 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel // Inspecter les fichiers pour voir s'il y a des erreurs // Avertir l'utilisateur if (isset($retours['message_erreur'])) { - $retours['message_erreur'] .= '<br />'.$erreurs_fichiers['message_public']; + $retours['message_erreur'] .= '<br />' . $erreurs_fichiers['message_public']; } else { $retours['message_erreur'] = $erreurs_fichiers['message_public']; } // Avertir le ou la webmestre if (isset($retours['id_formulaires_reponse'])) { - $erreur_fichiers_sujet = '[ERREUR] Impossible de sauvegarder les fichiers de la réponse '.$retours['id_formulaires_reponse']." au formulaire $id"; + $erreur_fichiers_sujet = '[ERREUR] Impossible de sauvegarder les fichiers de la réponse ' . $retours['id_formulaires_reponse'] . " au formulaire $id"; } else { $erreur_fichiers_sujet = "[ERREUR] Impossible de sauvegarder les fichiers de la réponse au formulaire $id"; } @@ -554,7 +556,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel $description[$i]['url'] = formidable_generer_url_action_recuperer_fichier_email( $saisie, $desc['fichier'], - array('timestamp'=>$retours['timestamp']) + ['timestamp' => $retours['timestamp']] ); } } @@ -566,13 +568,13 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel // Si on fait une redirection // Et que l'on a enregistré le résultat // Alors, passer l'id de la réponse à la page - if (isset($retours['id_formulaires_reponse']) AND isset($retours['redirect'])) { + if (isset($retours['id_formulaires_reponse']) and isset($retours['redirect'])) { $retours['redirect'] = parametre_url($retours['redirect'], 'id_formulaires_reponse', $retours['id_formulaires_reponse'], '&'); } // lorsqu'on affichera à nouveau le html, // dire à cvt-upload de ne pas générer le html pour les résultats des saisies fichiers - if ($formulaire['apres']=='formulaire' and isset($retours['fichiers'])) { - $formidable_cvtupload_precharger_fichiers = array(); + if ($formulaire['apres'] == 'formulaire' and isset($retours['fichiers'])) { + $formidable_cvtupload_precharger_fichiers = []; set_request('_fichiers', null); set_request('_cvtupload_precharger_fichiers_forcer', true); foreach ($retours['fichiers'] as $champ => $valeur) { @@ -586,23 +588,23 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel } if (isset($retours['id_formulaires_reponse'])) { $chemin_fichier = _DIR_FICHIERS_FORMIDABLE - .'formulaire_'.$retours['id_formulaire'] - .'/reponse_'.$retours['id_formulaires_reponse'] - .'/'.$champ - .'/'.$nom_fichier; + . 'formulaire_' . $retours['id_formulaire'] + . '/reponse_' . $retours['id_formulaires_reponse'] + . '/' . $champ + . '/' . $nom_fichier; $formidable_cvtupload_precharger_fichiers[$champ][$i]['url'] = formidable_generer_url_action_recuperer_fichier($retours['id_formulaire'], $retours['id_formulaires_reponse'], $champ, $nom_fichier); $formidable_cvtupload_precharger_fichiers[$champ][$i]['chemin'] = $chemin_fichier; } elseif (isset($retours['timestamp'])) { $chemin_fichier = _DIR_FICHIERS_FORMIDABLE - .'timestamp/' - .$retours['timestamp'].'/' - .$champ.'/' - .$nom_fichier; + . 'timestamp/' + . $retours['timestamp'] . '/' + . $champ . '/' + . $nom_fichier; $formidable_cvtupload_precharger_fichiers[$champ][$i]['chemin'] = $chemin_fichier; $formidable_cvtupload_precharger_fichiers[$champ][$i]['url'] = formidable_generer_url_action_recuperer_fichier_email( $champ, $nom_fichier, - array('timestamp'=>$retours['timestamp']) + ['timestamp' => $retours['timestamp']] ); } } @@ -616,7 +618,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel if ($erreur_texte) { $erreur_sujet = "[ERREUR] Traitement Formulaire $id"; // dumper la saisie pour ne pas la perdre - $erreur_texte .= "\n".var_export($_REQUEST, true)."\n".var_export($_FILES, true); + $erreur_texte .= "\n" . var_export($_REQUEST, true) . "\n" . var_export($_FILES, true); $envoyer_mail = charger_fonction('envoyer_mail', 'inc'); $envoyer_mail($GLOBALS['meta']['email_webmaster'], $erreur_sujet, $erreur_texte); } @@ -678,7 +680,7 @@ function formidable_definir_contexte_avec_reponse($contexte, $id_formulaires_rep // On prépare des infos si jamais on a des champs fichiers $saisies = formidable_deserialize($contexte['_formidable']['saisies']); $saisies_fichiers = saisies_lister_avec_type($saisies, 'fichiers');// les saisies de type fichier - $fichiers = array(); + $fichiers = []; $id_formulaire = $contexte['_formidable']['id_formulaire']; // On va chercher tous les champs @@ -691,17 +693,17 @@ function formidable_definir_contexte_avec_reponse($contexte, $id_formulaires_rep // On remplit le contexte avec les résultats précédents foreach ($champs as $champ) { if (array_key_exists($champ['nom'], $saisies_fichiers)) { - $valeur= formidable_deserialize($champ['valeur']); + $valeur = formidable_deserialize($champ['valeur']); $nom = $champ['nom']; - $fichiers[$nom] = array(); + $fichiers[$nom] = []; $chemin = _DIR_FICHIERS_FORMIDABLE - ."formulaire_$id_formulaire/reponse_$id_formulaires_reponse/" - ."$nom/"; + . "formulaire_$id_formulaire/reponse_$id_formulaires_reponse/" + . "$nom/"; if (is_array($valeur)) { foreach ($valeur as $f => $fichier) { - $fichiers[$nom][$f]= array(); + $fichiers[$nom][$f] = []; $fichiers[$nom][$f]['url'] = formidable_generer_url_action_recuperer_fichier($id_formulaire, $id_formulaires_reponse, $champ['nom'], $fichier['nom']); - $fichiers[$nom][$f]['chemin'] = $chemin.$fichier['nom']; + $fichiers[$nom][$f]['chemin'] = $chemin . $fichier['nom']; } } } else { @@ -709,7 +711,7 @@ function formidable_definir_contexte_avec_reponse($contexte, $id_formulaires_rep $contexte[$champ['nom']] = is_array($test_array) ? $test_array : $champ['valeur']; } } - if ($fichiers != array()) {//s'il y a des fichiers dans les réponses + if ($fichiers != []) {//s'il y a des fichiers dans les réponses $contexte['cvtupload_precharger_fichiers'] = $fichiers; } return $contexte; @@ -729,15 +731,15 @@ function formidable_produire_messages_erreurs_fichiers($fichiers) { foreach ($fichiers as $champ => $description_champ) { foreach ($description_champ as $n => $description) { if (isset($description['erreur'])) { - $message_public .= $description['erreur']."\n"; + $message_public .= $description['erreur'] . "\n"; $message_webmestre .= "Pour le champ $champ[$n]:\n" - . '- Le fichier temporaire : '.$description['tmp_name']."\n" - . '- Ayant pour véritable nom : '.$description['nom']." \n"; + . '- Le fichier temporaire : ' . $description['tmp_name'] . "\n" + . '- Ayant pour véritable nom : ' . $description['nom'] . " \n"; } } } - if ($message_public !='') { - return array('message_public'=>$message_public, 'message_webmestre'=>$message_webmestre); + if ($message_public != '') { + return ['message_public' => $message_public, 'message_webmestre' => $message_webmestre]; } else { return ''; } diff --git a/formulaires/importer_formulaire.php b/formulaires/importer_formulaire.php index 09fd86c8..b585b985 100644 --- a/formulaires/importer_formulaire.php +++ b/formulaires/importer_formulaire.php @@ -10,11 +10,11 @@ include_spip('inc/config'); function formulaires_importer_formulaire_charger() { - $contexte = array(); + $contexte = []; // On va chercher toutes les fonctions d'importation existantes $types_echange = echanges_formulaire_lister_disponibles(); - $types_import = array(); + $types_import = []; foreach ($types_echange['importer'] as $type => $fonction) { $types_import[$type] = _T("formidable:echanger_formulaire_{$type}_importer"); } @@ -25,13 +25,13 @@ function formulaires_importer_formulaire_charger() { } function formulaires_importer_formulaire_verifier() { - $erreurs = array(); + $erreurs = []; return $erreurs; } function formulaires_importer_formulaire_traiter() { - $retours = array(); + $retours = []; if (!$_FILES['fichier']['error']) { $type_import = _request('type_import'); @@ -55,7 +55,7 @@ function formulaires_importer_formulaire_traiter() { if ($auteurs == 'on') { if ($id_auteur = session_get('id_auteur')) { // association (par défaut) du formulaire et de l'auteur courant - objet_associer(array('formulaire'=>$id_formulaire), array('auteur'=>$id_auteur)); + objet_associer(['formulaire' => $id_formulaire], ['auteur' => $id_auteur]); } } $retours['redirect'] = generer_url_ecrire('formulaire', "id_formulaire=$id_formulaire"); diff --git a/genie/formidable_effacer_enregistrements.php b/genie/formidable_effacer_enregistrements.php index 7d8a7d45..7a7fc561 100644 --- a/genie/formidable_effacer_enregistrements.php +++ b/genie/formidable_effacer_enregistrements.php @@ -1,6 +1,5 @@ <?php - if (!defined('_ECRIRE_INC_VERSION')) { return; } @@ -16,7 +15,8 @@ function genie_formidable_effacer_enregistrements($t) { while ($result = sql_fetch($res)) { $traitements = formidable_deserialize($result['traitements']); $id_formulaire = $result['id_formulaire']; - if (isset($traitements['enregistrement']['effacement']) + if ( + isset($traitements['enregistrement']['effacement']) and $traitements['enregistrement']['effacement'] == 'on' and isset($traitements['enregistrement']['effacement_delai']) ) { diff --git a/genie/formidable_effacer_fichiers_email.php b/genie/formidable_effacer_fichiers_email.php index d1fb2817..9f551b90 100644 --- a/genie/formidable_effacer_fichiers_email.php +++ b/genie/formidable_effacer_fichiers_email.php @@ -1,6 +1,5 @@ <?php - if (!defined('_ECRIRE_INC_VERSION')) { return; } diff --git a/genie/formidable_hasher_ip.php b/genie/formidable_hasher_ip.php index 7b2021a5..8e3b1d93 100644 --- a/genie/formidable_hasher_ip.php +++ b/genie/formidable_hasher_ip.php @@ -1,6 +1,5 @@ <?php - if (!defined('_ECRIRE_INC_VERSION')) { return; } @@ -14,17 +13,17 @@ if (!defined('_ECRIRE_INC_VERSION')) { **/ function genie_formidable_hasher_ip_dist($t) { if (!defined('_CNIL_PERIODE')) { - define('_CNIL_PERIODE', 3600*24*31*4); + define('_CNIL_PERIODE', 3600 * 24 * 31 * 4); } if (_CNIL_PERIODE) { - $critere_cnil = 'date<"'.date('Y-m-d', time()-_CNIL_PERIODE).'"' + $critere_cnil = 'date<"' . date('Y-m-d', time() - _CNIL_PERIODE) . '"' . ' AND statut != "spam"' . ' AND (ip LIKE "%.%" OR ip LIKE "%:%")'; # ipv4 ou ipv6 $c = sql_countsel('spip_formulaires_reponses', $critere_cnil); - if ($c>0) { + if ($c > 0) { spip_log("CNIL: masquer IP de $c réponses anciennes à formidable", 'formidable'); - sql_update('spip_formulaires_reponses', array('ip' => 'MD5(ip)'), $critere_cnil); + sql_update('spip_formulaires_reponses', ['ip' => 'MD5(ip)'], $critere_cnil); return $c; } } diff --git a/http/collectionjson/formulaires_reponses.php b/http/collectionjson/formulaires_reponses.php index 46440ea5..d1b34e6f 100644 --- a/http/collectionjson/formulaires_reponses.php +++ b/http/collectionjson/formulaires_reponses.php @@ -50,13 +50,13 @@ function http_collectionjson_formulaires_reponses_post_collection_dist($requete, $erreurs = $formidable_verifier($id_formulaire); // On passe les erreurs dans le pipeline "verifier" (par exemple pour Saisies) - $erreurs = pipeline('formulaire_verifier', array( - 'args' => array( + $erreurs = pipeline('formulaire_verifier', [ + 'args' => [ 'form' => 'formidable', - 'args' => array($id_formulaire), - ), + 'args' => [$id_formulaire], + ], 'data' => $erreurs, - )); + ]); // S'il y a des erreurs, on va générer un JSON les listant if ($erreurs) { @@ -64,23 +64,23 @@ function http_collectionjson_formulaires_reponses_post_collection_dist($requete, $reponse->headers->set('Content-Type', 'application/json'); $reponse->setCharset('utf-8'); - $json_reponse = array( - 'collection' => array( + $json_reponse = [ + 'collection' => [ 'version' => '1.0', 'href' => url_absolue(self('&')), - 'error' => array( + 'error' => [ 'title' => _T('erreur'), 'code' => 400, - ), - 'errors' => array(), - ), - ); + ], + 'errors' => [], + ], + ]; foreach ($erreurs as $nom => $erreur) { - $json_reponse['collection']['errors'][$nom] = array( + $json_reponse['collection']['errors'][$nom] = [ 'title' => $erreur, 'code' => 400, - ); + ]; } $reponse->setContent(json_encode($json_reponse)); } else { @@ -90,13 +90,13 @@ function http_collectionjson_formulaires_reponses_post_collection_dist($requete, $retours_formidable = $formidable_traiter($id_formulaire); // On passe dans le pipeline "traiter" (par exemple pour les quizz) - $retours_formidable = pipeline('formulaire_traiter', array( - 'args' => array( + $retours_formidable = pipeline('formulaire_traiter', [ + 'args' => [ 'form' => 'formidable', - 'args' => array($id_formulaire), - ), + 'args' => [$id_formulaire], + ], 'data' => $retours_formidable, - )); + ]); // Si à la fin on a bien un identifiant de réponse if ($id_formulaires_reponse = $retours_formidable['id_formulaires_reponse']) { diff --git a/inc/formidable.php b/inc/formidable.php index 7cda2605..8a53b49a 100644 --- a/inc/formidable.php +++ b/inc/formidable.php @@ -14,7 +14,7 @@ function traitements_lister_disponibles() { static $traitements = null; if (is_null($traitements)) { - $traitements = array(); + $traitements = []; $liste = find_all_in_path('traiter/', '.+[.]yaml$'); ksort($liste); @@ -22,7 +22,8 @@ function traitements_lister_disponibles() { foreach ($liste as $fichier => $chemin) { $type_traitement = preg_replace(',[.]yaml$,i', '', $fichier); // On ne garde que les traitements qui ont bien la fonction - if (charger_fonction($type_traitement, 'traiter', true) + if ( + charger_fonction($type_traitement, 'traiter', true) and ( is_array($traitement = traitements_charger_infos($type_traitement)) ) @@ -35,7 +36,7 @@ function traitements_lister_disponibles() { $traitements = pipeline( 'formidable_traitements', - array('data' => $traitements, 'args' => array()) + ['data' => $traitements, 'args' => []] ); return $traitements; } @@ -52,7 +53,7 @@ function traitements_charger_infos($type_traitement) { $traitement = yaml_decode_file($fichier); if (is_array($traitement)) { - $traitement += array('titre' => '', 'description' => '', 'icone' => ''); + $traitement += ['titre' => '', 'description' => '', 'icone' => '']; $traitement['titre'] = $traitement['titre'] ? _T_ou_typo($traitement['titre']) : $type_traitement; $traitement['description'] = $traitement['description'] ? _T_ou_typo($traitement['description']) : ''; $traitement['icone'] = $traitement['icone'] ? find_in_path($traitement['icone']) : ''; @@ -68,7 +69,7 @@ function traitements_charger_infos($type_traitement) { function echanges_formulaire_lister_disponibles() { // On va chercher toutes les fonctions existantes $liste = find_all_in_path('echanger/formulaire/', '.+[.]php$'); - $types_echange = array('exporter' => array(), 'importer' => array()); + $types_echange = ['exporter' => [], 'importer' => []]; if (count($liste)) { foreach ($liste as $fichier => $chemin) { $type_echange = preg_replace(',[.]php$,i', '', $fichier); @@ -93,7 +94,7 @@ function echanges_formulaire_lister_disponibles() { * @return string Retourne le nom du cookie */ function formidable_generer_nom_cookie($id_formulaire) { - return $GLOBALS['cookie_prefix'].'cookie_formidable_'.$id_formulaire; + return $GLOBALS['cookie_prefix'] . 'cookie_formidable_' . $id_formulaire; } @@ -122,8 +123,10 @@ function formidable_trouver_reponse_a_editer($id_formulaire, $id_formulaires_rep ); //A-t-on demandé de vérifier que l'auteur soit bien celui de la réponse? - if ($id_formulaires_reponse = intval($id_formulaires_reponse) - and $verifier_est_auteur == true) { + if ( + $id_formulaires_reponse = intval($id_formulaires_reponse) + and $verifier_est_auteur == true + ) { if (!is_array($reponses) or in_array($id_formulaires_reponse, $reponses) == false) { $id_formulaires_reponse = false; } @@ -132,11 +135,12 @@ function formidable_trouver_reponse_a_editer($id_formulaire, $id_formulaires_rep // Si multiple = non mais que c'est modifiable, alors on va chercher // la dernière réponse si elle existe - if ($options + if ( + $options and !$options['multiple'] and $options['modifiable'] and is_array($reponses) - ) { + ) { $id_formulaires_reponse = array_pop($reponses); } } @@ -152,7 +156,7 @@ function formidable_trouver_reponse_a_editer($id_formulaire, $id_formulaires_rep * @param string $anonymiser : si 'on', le formulaire doit-être anonymisé * @return unknown_type Retourne un tableau contenant les id des réponses si elles existent, sinon false **/ -function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identification = 'cookie', $variable_php_identification = '', $anonymiser='') { +function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identification = 'cookie', $variable_php_identification = '', $anonymiser = '') { global $auteur_session; $id_auteur = $auteur_session ? intval($auteur_session['id_auteur']) : 0; $nom_cookie = formidable_generer_nom_cookie($id_formulaire); @@ -171,43 +175,43 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific if ($id_auteur) { if ($anonymiser == 'on') { $id_auteur = formidable_hasher_id_auteur($id_auteur); - $where_id_auteur = 'variable_php="'.$id_auteur.'"'; + $where_id_auteur = 'variable_php="' . $id_auteur . '"'; } else { - $where_id_auteur = 'id_auteur='.$id_auteur; + $where_id_auteur = 'id_auteur=' . $id_auteur; } } if ($cookie) { - $where_cookie = 'cookie='.sql_quote($cookie); + $where_cookie = 'cookie=' . sql_quote($cookie); } if ($variable_php_identification) { - $where_variable_php = 'variable_php='.$variable_php_identification; + $where_variable_php = 'variable_php=' . $variable_php_identification; } // Comment identifie-t-on? Attention, le choix d'identification indique une PRIORITE, donc cela veut dire que les autres méthodes peuvent venir après, sauf dans le cas d'identification explicitement par id_reponse if ($choix_identification == 'cookie' or !$choix_identification) { if ($cookie) { - $where = array($where_cookie); + $where = [$where_cookie]; } else { - $where = array($where_id_auteur, $where_variable_php); + $where = [$where_id_auteur, $where_variable_php]; } } elseif ($choix_identification == 'id_auteur') { if ($id_auteur) { - $where = array($where_id_auteur); + $where = [$where_id_auteur]; } else { if ($anonymiser) { - $where = array($where_cookie); + $where = [$where_cookie]; } else { - $where = array($where_cookie, $where_variable_php); + $where = [$where_cookie, $where_variable_php]; } } } elseif ($choix_identification == 'variable_php') { if ($variable_php_identification) { - $where = array($where_variable_php); + $where = [$where_variable_php]; } else { - $where = array($where_cookie, $where_id_auteur); + $where = [$where_cookie, $where_id_auteur]; } } elseif ($choix_identification == 'id_reponse') {//Si le filtrage se fait par réponse, on prend tout (mais normalement on devrait pas aboutir ici si tel est le cas) - $where = array("1=1"); + $where = ['1=1']; } $where = array_filter($where);//Supprimer les wheres null $where = implode(' OR ', $where); @@ -215,11 +219,11 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific $reponses = sql_allfetsel( 'id_formulaires_reponse', 'spip_formulaires_reponses', - array( - array('=', 'id_formulaire', intval($id_formulaire)), - sql_in('statut', array('prop', 'publie')), + [ + ['=', 'id_formulaire', intval($id_formulaire)], + sql_in('statut', ['prop', 'publie']), $where - ), + ], '', 'date' ); @@ -238,13 +242,13 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific * @param array $env L'environnement, contenant normalement la réponse à la saisie * @return string Retour le HTML des vues */ -function formidable_analyser_saisie($saisie, $valeurs = array(), $reponses_total = 0, $format_brut = false) { +function formidable_analyser_saisie($saisie, $valeurs = [], $reponses_total = 0, $format_brut = false) { // Si le paramètre n'est pas bon ou que c'est un conteneur, on génère du vide if (!is_array($saisie) or (isset($saisie['saisies']) and $saisie['saisies'])) { return ''; } - $contexte = array('reponses_total'=>$reponses_total); + $contexte = ['reponses_total' => $reponses_total]; // On sélectionne le type de saisie $contexte['type_saisie'] = $saisie['saisie']; @@ -259,12 +263,14 @@ function formidable_analyser_saisie($saisie, $valeurs = array(), $reponses_total $contexte = array_merge($contexte, $options); // On récupère toutes les valeurs du champ - if (isset($valeurs[$contexte['nom']]) + if ( + isset($valeurs[$contexte['nom']]) and $valeurs[$contexte['nom']] - and is_array($valeurs[$contexte['nom']])) { + and is_array($valeurs[$contexte['nom']]) + ) { $contexte['valeurs'] = $valeurs[$contexte['nom']]; } else { - $contexte['valeurs'] = array(); + $contexte['valeurs'] = []; } // On génère la saisie @@ -289,7 +295,7 @@ function analyser_saisie($saisie) { return ''; } - $ligne = array(); + $ligne = []; switch ($saisie['type_saisie']) { case 'selecteur_rubrique': @@ -303,7 +309,7 @@ function analyser_saisie($saisie) { case 'selection_multiple': case 'choix_couleur': case 'checkbox': - $stats = array(); + $stats = []; foreach ($saisie['valeurs'] as $valeur) { if (is_array($valeur)) { foreach ($valeur as $choix) { @@ -335,7 +341,7 @@ function analyser_saisie($saisie) { } break; case 'destinataires': - $stats = array(); + $stats = []; foreach ($saisie['valeurs'] as $valeur) { foreach ($valeur as $choix) { if (isset($stats["choix-$choix"])) { @@ -408,7 +414,7 @@ function titre_nb_reponses($nb) { if ($nb == 1) { return _T('formidable:reponse_une'); } - return _T('formidable:reponses_nb', array('nb' => $nb)); + return _T('formidable:reponses_nb', ['nb' => $nb]); } /** @@ -420,7 +426,7 @@ function titre_nb_reponses($nb) { */ function md5_hex_to_dec($hex_str) { $arr = str_split($hex_str, 4); - $dec = array(); + $dec = []; foreach ($arr as $grp) { $dec[] = str_pad(hexdec($grp), 5, '0', STR_PAD_LEFT); } @@ -511,7 +517,8 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $ } } - $chaine = pipeline('formidable_pre_raccourcis_arobases', + $chaine = pipeline( + 'formidable_pre_raccourcis_arobases', [ 'data' => $chaine, 'args' => [ @@ -527,7 +534,7 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $ ] ] ); - $a_remplacer = array(); + $a_remplacer = []; if (preg_match_all('/@[\w]+@/', $chaine, $match)) { $a_remplacer = $match[0]; foreach ($a_remplacer as $cle => $val) { @@ -544,8 +551,8 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $ $a_remplacer = array_intersect_key($valeurs_libellees, $a_remplacer); } } - $a_remplacer = array_merge($a_remplacer,array('nom_site_spip' => lire_config('nom_site'))); - if (strpos($chaine,'@message_retour@') !== false) {// test pour éviter recurrence infinie + $a_remplacer = array_merge($a_remplacer, ['nom_site_spip' => lire_config('nom_site')]); + if (strpos($chaine, '@message_retour@') !== false) {// test pour éviter recurrence infinie $message_retour = sql_getfetsel('message_retour', 'spip_formulaires', "id_formulaire=$id_formulaire"); $message_retour = formidable_raccourcis_arobases_2_valeurs_champs( $message_retour, @@ -556,12 +563,13 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $ 'source' => $source, ] ); - $a_remplacer = array_merge($a_remplacer,array('message_retour' => $message_retour)); + $a_remplacer = array_merge($a_remplacer, ['message_retour' => $message_retour]); } } $a_remplacer = array_map('trim', $a_remplacer); $chaine = trim(_L($chaine, $a_remplacer)); - $chaine = pipeline('formidable_post_raccourcis_arobases', + $chaine = pipeline( + 'formidable_post_raccourcis_arobases', [ 'data' => $chaine, 'args' => [ @@ -589,11 +597,11 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $ */ function formidable_tableau_valeurs_saisies($saisies, $sans_reponse = true) { if (isset($valeurs)) { - return array($valeurs,$valeurs_libellees); + return [$valeurs,$valeurs_libellees]; } // On parcourt les champs pour générer le tableau des valeurs - static $valeurs = array(); - static $valeurs_libellees = array(); + static $valeurs = []; + static $valeurs_libellees = []; if ($sans_reponse === true) { $sans_reponse = _T('saisies:sans_reponse'); } @@ -619,18 +627,18 @@ function formidable_tableau_valeurs_saisies($saisies, $sans_reponse = true) { $valeurs_libellees[$champ] = formidable_nettoyer_saisie_vue(recuperer_fond( 'saisies-vues/_base', array_merge( - array( + [ 'type_saisie' => $saisies_par_nom[$champ]['saisie'], 'valeur' => $valeurs[$champ], 'valeur_uniquement' => 'oui', 'sans_reponse' => $sans_reponse - ), + ], $saisies_par_nom[$champ]['options'] ) )); } } - return array($valeurs, $valeurs_libellees); + return [$valeurs, $valeurs_libellees]; } @@ -647,10 +655,10 @@ function formidable_variable_php_identification($nom_variable, $id_formulaire) { //Pour compat ascendante if (isset($GLOBALS['formulaires']['variables_anonymisation'])) { $nom_variable = $GLOBALS['formulaires']['variables_anonymisation'][$nom_variable]; - $valeur_variable = eval ("return $nom_variable;"); + $valeur_variable = eval("return $nom_variable;"); } - if (in_array($nom_variable, array("remote_user", "php_auth_user"))) { + if (in_array($nom_variable, ['remote_user', 'php_auth_user'])) { $nom_variable = strtoupper($nom_variable); $valeur_variable = isset($_SERVER[$nom_variable]) ? $_SERVER[$nom_variable] : 0; } else { @@ -668,10 +676,10 @@ function formidable_variable_php_identification($nom_variable, $id_formulaire) { * @param string $id_auteur * @return string */ -function formidable_hasher_id_auteur($id_auteur="") { +function formidable_hasher_id_auteur($id_auteur = '') { include_spip('inc/securiser_action'); $pass = secret_du_site(); - return md5($pass.$id_auteur); + return md5($pass . $id_auteur); } /** @@ -695,6 +703,6 @@ function formidable_array_walk_recursive_strval(&$value, $key) { * @param array Le tableau * @return string **/ -function formidable_serialize(array $tab):string { +function formidable_serialize(array $tab): string { return json_encode($tab); } diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php index c8aa590b..4941424b 100644 --- a/inc/formidable_fichiers.php +++ b/inc/formidable_fichiers.php @@ -13,16 +13,16 @@ if (!defined('_FORMIDABLE_TAILLE_MAX_FICHIERS_EMAIL')) {// la taille maximum (en // Lieux de stockages des fichiers if (!defined('_DIR_FICHIERS')) { // En attendant que ce soit natif spip - define('_DIR_FICHIERS', _DIR_ETC.'fichiers/'); + define('_DIR_FICHIERS', _DIR_ETC . 'fichiers/'); } if (!defined('_DIR_FICHIERS_FORMIDABLE')) { - define('_DIR_FICHIERS_FORMIDABLE', _DIR_FICHIERS.'formidable/'); + define('_DIR_FICHIERS_FORMIDABLE', _DIR_FICHIERS . 'formidable/'); } if (!defined('_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL')) { // Combien de temps un lien par email dans fichier est valable (en seconde) - define('_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL', 24*3600); + define('_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL', 24 * 3600); } if (!defined('_FORMIDABLE_EFFACEMENT_FICHIERS_EMAIL')) { // Au bout de combien de temps efface-t-on les fichiers enregistrés lorsque le traitement est uniquement email? @@ -47,7 +47,7 @@ function formidable_creer_dossier_formulaire($id_formulaire, $forcer = false) { // Récuperer la liste des saisies de type fichier $saisies_fichiers = formulaires_formidable_fichiers($id_formulaire); - if (!is_array($saisies_fichiers) or $saisies_fichiers == array ()) { + if (!is_array($saisies_fichiers) or $saisies_fichiers == []) { //pas de saisie fichiers? return ''; } @@ -59,10 +59,10 @@ function formidable_creer_dossier_formulaire($id_formulaire, $forcer = false) { sous_repertoire(_DIR_FICHIERS_FORMIDABLE, '', true, true); $dossier = sous_repertoire(_DIR_FICHIERS_FORMIDABLE, $nom_dossier, false, true); if (strpos($dossier, "$nom_dossier/") === false) { - spip_log("Impossible d'écrire $nom_dossier", 'formidable'._LOG_ERREUR); + spip_log("Impossible d'écrire $nom_dossier", 'formidable' . _LOG_ERREUR); return _T( 'formidable:creer_dossier_formulaire_erreur_impossible_creer', - array('dossier'=>_DIR_FICHIERS_FORMIDABLE . $nom_dossier) + ['dossier' => _DIR_FICHIERS_FORMIDABLE . $nom_dossier] ); } @@ -73,31 +73,31 @@ function formidable_creer_dossier_formulaire($id_formulaire, $forcer = false) { // on crée un fichier de test, pour s'assurer // 1. Qu'on puisse écrire dans le rep // 2. Qu'on ne puisse pas accéder à ce fichier depuis l'exterieur. - $fichier = $dossier.'test.txt'; + $fichier = $dossier . 'test.txt'; $ecriture_ok = ecrire_fichier( $fichier, "Ce fichier n'est normalement pas lisible de l'extérieur. Si tel est le cas, il y a un souci de confidentialité.", false ); if ($ecriture_ok == false) { - spip_log("Impossible d'écrire dans $nom_dossier", 'formidable'._LOG_ERREUR); + spip_log("Impossible d'écrire dans $nom_dossier", 'formidable' . _LOG_ERREUR); return _T( 'formidable:creer_dossier_formulaire_erreur_impossible_ecrire', - array('dossier'=>_DIR_FICHIERS_FORMIDABLE . $nom_dossier) + ['dossier' => _DIR_FICHIERS_FORMIDABLE . $nom_dossier] ); } include_spip('inc/distant'); $url = url_absolue($fichier); - $data = recuperer_url($url, array( + $data = recuperer_url($url, [ 'methode' => 'HEAD' - )); + ]); if ($data and $data['status'] == 200) { // si on peut récuperer la page avec un statut http 200, c'est qu'il y a un problème - spip_log("$nom_dossier accessible en lecture depuis le web", 'formidable'._LOG_CRITIQUE); + spip_log("$nom_dossier accessible en lecture depuis le web", 'formidable' . _LOG_CRITIQUE); return _T( 'formidable:creer_dossier_formulaire_erreur_possible_lire_exterieur', - array('dossier'=>_DIR_FICHIERS_FORMIDABLE . $nom_dossier) + ['dossier' => _DIR_FICHIERS_FORMIDABLE . $nom_dossier] ); } // Et si tout va bien @@ -130,9 +130,9 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime } if (isset($options['id_formulaires_reponse'])) { - $dossier_reponse = 'reponse_'.$options['id_formulaires_reponse']; + $dossier_reponse = 'reponse_' . $options['id_formulaires_reponse']; } elseif (isset($options['timestamp'])) { - $dossier_reponse = 'reponse_'.$options['timestamp']; + $dossier_reponse = 'reponse_' . $options['timestamp']; } else { // si ni timestamp, ni id_formulaires_reponse => erreur return ''; } @@ -150,7 +150,7 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime $res = sql_select( 'mime_type', 'spip_types_documents', - 'mime_type='.sql_quote($mime).' and extension='.sql_quote($extension) + 'mime_type=' . sql_quote($mime) . ' and extension=' . sql_quote($extension) ); if (sql_count($res) == 0) { $zipper = true; @@ -164,7 +164,6 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime $zipper = false; } if (!isset($options['timestamp'])) { // si on enregistre la réponse en base - // d'abord, créer si besoin le dossier pour le formulaire, si on a une erreur, on ne déplace pas le fichier if (formidable_creer_dossier_formulaire($id_formulaire, true) != '') { return ''; @@ -172,7 +171,7 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime // puis on créer le dossier pour la réponse $dossier_reponse = sous_repertoire( - _DIR_FICHIERS_FORMIDABLE.$dossier_formulaire.'/', + _DIR_FICHIERS_FORMIDABLE . $dossier_formulaire . '/', $dossier_reponse, false, true @@ -195,37 +194,37 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime // on crée un fichier de test, pour s'assurer // 1. Qu'on puisse écrire dans le rep // 2. Qu'on ne puisse pas accéder à ce fichier depuis l'exterieur. - $fichier_test = $dossier.'test.txt'; + $fichier_test = $dossier . 'test.txt'; $ecriture_ok = ecrire_fichier( $fichier_test, "Ce fichier n'est normalement pas lisible de l'extérieur. Si tel est le cas, il y a un souci de confidentialité.", false ); if ($ecriture_ok == false) { - spip_log("Impossible d'écrire dans $dossier", 'formidable'._LOG_ERREUR); + spip_log("Impossible d'écrire dans $dossier", 'formidable' . _LOG_ERREUR); return ''; } // WARNING : ce test n'est pas fiable car il repose sur le fait que le serveur est capable d'acceder a ses propres URLs // ce qui n'est pas forcement vrai selon la config reseau etc. Peut provoquer un delai d'attente important si le firewall bloque la requete http include_spip('inc/distant'); $url = url_absolue($fichier_test); - $data = recuperer_url($url, array( + $data = recuperer_url($url, [ 'methode' => 'HEAD' - )); + ]); if ($data and $data['status'] === 200) { // si on peut récuperer la page avec un statut http 200, // c'est qu'il y a un problème. - spip_log("$dossier accessible en lecture depuis le web", 'formidable'._LOG_CRITIQUE); + spip_log("$dossier accessible en lecture depuis le web", 'formidable' . _LOG_CRITIQUE); return ''; } } // S'assurer qu'il n'y a pas un fichier du même nom à destination - $chemin_final = $dossier_champ.$nom; + $chemin_final = $dossier_champ . $nom; $n = 1; - $basename_sans_extension = substr_replace($basename, '', -strlen($extension)-1); + $basename_sans_extension = substr_replace($basename, '', -strlen($extension) - 1); while (@file_exists($chemin_final)) { - $nom = $basename_sans_extension."_$n.".$extension; - $chemin_final = $dossier_champ.$nom; + $nom = $basename_sans_extension . "_$n." . $extension; + $chemin_final = $dossier_champ . $nom; $n++; } if (!$zipper) { // si on ne zippe pas, c'est simple @@ -233,7 +232,7 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime spip_log("Enregistrement du fichier $chemin_final", 'formidable'); return basename($fichier_dest); } else { - spip_log("Pb lors de l'enregistrement du fichier $chemin_final", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'enregistrement du fichier $chemin_final", 'formidable' . _LOG_ERREUR); return ''; } } else { // si on doit zipper, c'est plus complexe @@ -245,12 +244,12 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime } spip_unlink($tmp_dir); @mkdir($tmp_dir); - if (!$fichier_tmp = deplacer_fichier_upload($fichier, $tmp_dir.'/'.$nom_dans_zip, false)) { - spip_log("Pb lors de l'enregistrement du fichier $tmp_dir/$nom_dans_zip", 'formidable'._LOG_ERREUR); + if (!$fichier_tmp = deplacer_fichier_upload($fichier, $tmp_dir . '/' . $nom_dans_zip, false)) { + spip_log("Pb lors de l'enregistrement du fichier $tmp_dir/$nom_dans_zip", 'formidable' . _LOG_ERREUR); return ''; } if (!$zip->emballer([$fichier_tmp], $tmp_dir)) { - spip_log("Pb lors de l'enregistrement du fichier $fichier", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'enregistrement du fichier $fichier", 'formidable' . _LOG_ERREUR); return ''; } else { //spip_unlink($fichier); On n'efface pas, car cela permet en cas de souci par ailleurs d'avoir au moins le fichier dans tmp/cvt_upload. CVT upload a son propre mécanisme de nettoyage de ce dossier. @@ -285,7 +284,7 @@ function formidable_retourner_fichier($chemin, $f, $content_type = '') { header('Expires: 0'); // set expiration time header('Cache-Control: must-revalidate, post-check=0, pre-check=0'); if ($cl = filesize($chemin)) { - header('Content-Length: '.$cl); + header('Content-Length: ' . $cl); } readfile($chemin); exit; @@ -308,34 +307,36 @@ function formidable_deplacer_fichiers_produire_vue_saisie($saisie, $options) { if (!isset($_FILES[$champ])) {//précaution return null; } - $description_fichiers = array(); + $description_fichiers = []; $mon_file = $_FILES[$champ]; $i = 0; while ($i < $nb_fichiers_max) { if ($mon_file['error'][$i] === 0) { // la saisie fichiers est forcément structurée sous la forme d'un tableau, // on peut donc vérifier qu'il n'y a pas d'erreur facilement - $description = array(); // tableau pour stocker la description de ce fichier + $description = []; // tableau pour stocker la description de ce fichier // les infos qu'on peut récuperer directement de $files $description['taille'] = $mon_file['size'][$i]; $description['mime'] = $mon_file['type'][$i]; // l'adresse du nouveau fichier, sans le chemin - if ($nouveau_nom = formidable_deplacer_fichier_emplacement_definitif( - $mon_file['tmp_name'][$i], - $mon_file['name'][$i], - $mon_file['type'][$i], - pathinfo($mon_file['name'][$i], PATHINFO_EXTENSION), - $champ, - $options - )) { + if ( + $nouveau_nom = formidable_deplacer_fichier_emplacement_definitif( + $mon_file['tmp_name'][$i], + $mon_file['name'][$i], + $mon_file['type'][$i], + pathinfo($mon_file['name'][$i], PATHINFO_EXTENSION), + $champ, + $options + ) + ) { $description['nom'] = $nouveau_nom; $description['extension'] = pathinfo($nouveau_nom, PATHINFO_EXTENSION); } else { $description['erreur'] = _T( 'formidable:erreur_deplacement_fichier', - array('nom'=>$mon_file['name'][$i]) + ['nom' => $mon_file['name'][$i]] ); $description['nom'] = $mon_file['name'][$i]; $description['tmp_name'] = $mon_file['tmp_name'][$i]; @@ -353,13 +354,13 @@ function formidable_deplacer_fichiers_produire_vue_saisie($saisie, $options) { * @return int 1 ou 0 selon que l'on a effacé ou non un répertoire **/ function formidable_effacer_fichiers_formulaire($id_formulaire) { - $chemin = _DIR_FICHIERS_FORMIDABLE."formulaire_$id_formulaire"; + $chemin = _DIR_FICHIERS_FORMIDABLE . "formulaire_$id_formulaire"; if (file_exists($chemin)) {// par sécurité if (supprimer_repertoire($chemin)) { spip_log("Effacement du dossier $chemin", 'formidable'); return 1; } else { - spip_log("Pb lors de l'effacement du dossier $chemin", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'effacement du dossier $chemin", 'formidable' . _LOG_ERREUR); return 0; } } @@ -373,13 +374,13 @@ function formidable_effacer_fichiers_formulaire($id_formulaire) { * @return int 1 ou 0 selon que l'on a effacé ou non un répertoire **/ function formidable_effacer_fichiers_reponse($id_formulaire, $id_formulaires_reponse) { - $chemin = _DIR_FICHIERS_FORMIDABLE."formulaire_$id_formulaire/reponse_$id_formulaires_reponse"; + $chemin = _DIR_FICHIERS_FORMIDABLE . "formulaire_$id_formulaire/reponse_$id_formulaires_reponse"; if (file_exists($chemin)) {// par sécurité if (supprimer_repertoire($chemin)) { spip_log("Effacement du dossier $chemin", 'formidable'); return 1; } else { - spip_log("Pb lors de l'effacement du dossier $chemin", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'effacement du dossier $chemin", 'formidable' . _LOG_ERREUR); return 0; } } @@ -393,21 +394,20 @@ function formidable_effacer_fichiers_reponse($id_formulaire, $id_formulaires_rep **/ function formidable_effacer_fichiers_champ($id_formulaire, $reponses, $champ) { if ($champ != '') { // on devrait pas arriver ici avec un $champ vide, mais prenons nos précaution - if (!is_array($reponses)) { - $reponses = array($reponses); + $reponses = [$reponses]; } - $rep_vide = array('.', '..', '.ok'); // si scandire retourne cela où inférieur, alors le dossier est vide + $rep_vide = ['.', '..', '.ok']; // si scandire retourne cela où inférieur, alors le dossier est vide foreach ($reponses as $rep) { - $chemin_reponse = _DIR_FICHIERS_FORMIDABLE."formulaire_$id_formulaire/reponse_$rep"; - $chemin_champ = $chemin_reponse.'/'.$champ; + $chemin_reponse = _DIR_FICHIERS_FORMIDABLE . "formulaire_$id_formulaire/reponse_$rep"; + $chemin_champ = $chemin_reponse . '/' . $champ; if (file_exists($chemin_champ)) { if (supprimer_repertoire($chemin_champ)) { spip_log("Effacement du dossier $chemin_champ", 'formidable'); } else { - spip_log("Pb lors de l'effacement du dossier $chemin_champ", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'effacement du dossier $chemin_champ", 'formidable' . _LOG_ERREUR); } if (count(array_diff(scandir($chemin_reponse), $rep_vide)) == 0) { // si jamais il ne reste plus aucun fichiers pour cette réponse, @@ -415,7 +415,7 @@ function formidable_effacer_fichiers_champ($id_formulaire, $reponses, $champ) { if (supprimer_repertoire($chemin_reponse)) { spip_log("Effacement du dossier $chemin_reponse", 'formidable'); } else { - spip_log("Pb lors de l'effacement du dossier $chemin_reponse", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'effacement du dossier $chemin_reponse", 'formidable' . _LOG_ERREUR); } } } @@ -432,7 +432,7 @@ function formidable_effacer_fichiers_email() { return 0; } $dossiers_effaces = 0; - $chemin = _DIR_FICHIERS_FORMIDABLE.'timestamp'; + $chemin = _DIR_FICHIERS_FORMIDABLE . 'timestamp'; $timestamp = time(); if (is_dir($chemin)) { $dossiers = scandir($chemin); @@ -447,7 +447,7 @@ function formidable_effacer_fichiers_email() { spip_log("Effacement du dossier $chemin_complet", 'formidable'); $dossiers_effaces++; } else { - spip_log("Pb lors de l'effacement du dossier $chemin_complet", 'formidable'._LOG_ERREUR); + spip_log("Pb lors de l'effacement du dossier $chemin_complet", 'formidable' . _LOG_ERREUR); } } } @@ -475,10 +475,10 @@ function formidable_zipper_reponses_formulaire($id_formulaire, $chemin_du_zip, $ $fichier_csv = deplacer_fichier_upload($fichier_csv, "$chemin_fichiers/$basename_csv", true); - if (!$zip->emballer(array_merge($saisies_fichiers,[$fichier_csv]), $chemin_fichiers)) { + if (!$zip->emballer(array_merge($saisies_fichiers, [$fichier_csv]), $chemin_fichiers)) { spip_log( "Impossible de créer le zip pour l'export des réponses du formulaire $id_formulaire", - 'formidable'._LOG_ERREUR + 'formidable' . _LOG_ERREUR ); return 0; } else { @@ -494,24 +494,24 @@ function formidable_zipper_reponses_formulaire($id_formulaire, $chemin_du_zip, $ * @param str $fichier **/ function formidable_generer_url_action_recuperer_fichier($id_formulaire, $id_formulaires_reponse, $saisie, $fichier) { - $param = array( + $param = [ 'formulaire' => $id_formulaire, 'reponse' => $id_formulaires_reponse, 'saisie' => $saisie, 'fichier' => $fichier - ); + ]; // Pour les utilisateurs non authentifiés, on se base sur le cookier $nom_cookie = formidable_generer_nom_cookie($id_formulaire); if (isset($_COOKIE[$nom_cookie])) { include_spip('inc/securiser_action'); - $param['cookie'] = sha1($_COOKIE[$nom_cookie].secret_du_site()); + $param['cookie'] = sha1($_COOKIE[$nom_cookie] . secret_du_site()); } $param = json_encode($param, JSON_UNESCAPED_SLASHES); $securiser_action = charger_fonction('securiser_action', 'inc'); $url = $securiser_action('formidable_recuperer_fichier', $param, '', false); - spip_log("recuperer_fichier : pour la $saisie et le fichier $fichier avec les options ".print_r($param,true)." : l'url est $url", "formidable"._LOG_DEBUG); + spip_log("recuperer_fichier : pour la $saisie et le fichier $fichier avec les options " . print_r($param, true) . " : l'url est $url", 'formidable' . _LOG_DEBUG); return $url; } @@ -524,25 +524,25 @@ function formidable_generer_url_action_recuperer_fichier($id_formulaire, $id_for **/ function formidable_generer_url_action_recuperer_fichier_email($saisie, $fichier, $options) { if (isset($options['id_formulaires_reponse'])) {//si reponses enregistrées - $arg = json_encode(array( + $arg = json_encode([ 'formulaire' => strval($options['id_formulaire']), 'reponse' => strval($options['id_formulaires_reponse']), 'fichier' => $fichier, 'saisie' => $saisie - )); + ]); } elseif (isset($options['timestamp'])) {//si par timestamp - $arg = json_encode(array( + $arg = json_encode([ 'timestamp' => strval($options['timestamp']), 'fichier' => $fichier, 'saisie' => $saisie - )); + ]); } $arg = base64_encode($arg); $pass = secret_du_site(); $action = 'formidable_recuperer_fichier_par_email'; $hash = _action_auteur("$action-$arg", 0, $pass, 'secret_de_formidable'); $url = generer_url_action($action, "arg=$arg&hash=$hash&base64=true", true, true); - spip_log("recuperer_fichier_par_email : pour la $saisie et le fichier $fichier avec les options ".print_r($options,true)." le arg est $arg et le hash $hash. Du coup l'url est $url", "formidable"._LOG_DEBUG); + spip_log("recuperer_fichier_par_email : pour la $saisie et le fichier $fichier avec les options " . print_r($options, true) . " le arg est $arg et le hash $hash. Du coup l'url est $url", 'formidable' . _LOG_DEBUG); return $url; } @@ -556,19 +556,19 @@ function formidable_generer_url_action_recuperer_fichier_email($saisie, $fichier * fichier => nom du fichier * @return string $chemin; **/ -function formidable_generer_chemin_fichier($param){ +function formidable_generer_chemin_fichier($param) { $chemin_fichier = ''; if (isset($param['reponse'])) { $chemin_fichier = _DIR_FICHIERS_FORMIDABLE - .'formulaire_'.$param['formulaire'] - .'/reponse_'.$param['reponse'] - .'/'.$param['saisie'] - .'/'.$param['fichier']; + . 'formulaire_' . $param['formulaire'] + . '/reponse_' . $param['reponse'] + . '/' . $param['saisie'] + . '/' . $param['fichier']; } elseif (isset($param['timestamp'])) { $chemin_fichier = _DIR_FICHIERS_FORMIDABLE . 'timestamp/' - . $param['timestamp'].'/' - . $param['saisie'].'/' + . $param['timestamp'] . '/' + . $param['saisie'] . '/' . $param['fichier']; } else { include_spip('inc/minipres'); diff --git a/inc/formidable_migrer_depuis_ft.php b/inc/formidable_migrer_depuis_ft.php index 2ca6b6a5..a374b16e 100644 --- a/inc/formidable_migrer_depuis_ft.php +++ b/inc/formidable_migrer_depuis_ft.php @@ -12,10 +12,10 @@ if (!defined('_ECRIRE_INC_VERSION')) { function formidable_associer_forms() { include_spip('inc/rechercher'); include_spip('inc/editer_liens'); - $forms = sql_allfetsel('*', 'spip_formulaires', 'identifiant REGEXP '.sql_quote('^form[0-9]+$')); + $forms = sql_allfetsel('*', 'spip_formulaires', 'identifiant REGEXP ' . sql_quote('^form[0-9]+$')); foreach ($forms as $form) { - if (!sql_countsel('spip_formulaires_liens', 'id_formulaire='.intval($form['id_formulaire']))) { - $articles = array(); + if (!sql_countsel('spip_formulaires_liens', 'id_formulaire=' . intval($form['id_formulaire']))) { + $articles = []; $id = $form['identifiant']; #var_dump($id); $res = recherche_en_base("/<{$id}[>|]/", 'article'); @@ -27,9 +27,9 @@ function formidable_associer_forms() { } #var_dump($form['id_formulaire']); #var_dump($articles); - objet_associer(array('formulaire' => array($form['id_formulaire'])), array('article' => $articles)); + objet_associer(['formulaire' => [$form['id_formulaire']]], ['article' => $articles]); } - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } } @@ -45,9 +45,9 @@ function formidable_importer_forms() { include_spip('echanger/formulaire/forms'); - $forms = sql_allfetsel('*', 'spip_forms', 'id_formulaire=0 AND type_form='.sql_quote('').' OR type_form='.sql_quote('sondage'), '', 'id_form'); + $forms = sql_allfetsel('*', 'spip_forms', 'id_formulaire=0 AND type_form=' . sql_quote('') . ' OR type_form=' . sql_quote('sondage'), '', 'id_form'); foreach ($forms as $form) { - $formulaire = array(); + $formulaire = []; // configurer le formulaire (titre etc) forms_configure_formulaire($form, $formulaire); @@ -57,9 +57,9 @@ function formidable_importer_forms() { // on peut faire ca aussi puisqu'on est a l'installation $formulaire['id_formulaire'] = $form['id_form']; - $fields = sql_allfetsel('*', 'spip_forms_champs', 'id_form='.intval($form['id_form']), '', 'rang'); + $fields = sql_allfetsel('*', 'spip_forms_champs', 'id_form=' . intval($form['id_form']), '', 'rang'); foreach ($fields as $field) { - $choix = sql_allfetsel('*', 'spip_forms_champs_choix', 'id_form='.intval($form['id_form']).' AND champ='.sql_quote($field['champ']), '', 'rang'); + $choix = sql_allfetsel('*', 'spip_forms_champs_choix', 'id_form=' . intval($form['id_form']) . ' AND champ=' . sql_quote($field['champ']), '', 'rang'); if (count($choix)) { $field['choix'] = $choix; } @@ -73,16 +73,16 @@ function formidable_importer_forms() { forms_configure_traitement_formulaire($form, $formulaire); // si ce formulaire a des reponses on le met en publie - if (sql_countsel('spip_forms_donnees', 'id_form='.intval($form['id_form']))) { + if (sql_countsel('spip_forms_donnees', 'id_form=' . intval($form['id_form']))) { $formulaire['statut'] = 'publie'; } $id_formulaire = forms_importe_en_base($formulaire); - spip_log('Import spip_forms #'.$form['id_form']." en spip_formulaires #$id_formulaire", 'maj'._LOG_INFO_IMPORTANTE); + spip_log('Import spip_forms #' . $form['id_form'] . " en spip_formulaires #$id_formulaire", 'maj' . _LOG_INFO_IMPORTANTE); - sql_update('spip_forms', array('id_formulaire' => $id_formulaire), 'id_form='.intval($form['id_form'])); + sql_update('spip_forms', ['id_formulaire' => $id_formulaire], 'id_form=' . intval($form['id_form'])); - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } } @@ -102,16 +102,16 @@ function formidable_importer_forms_donnees() { // table de correspondance id_form=>id_formulaire $rows = sql_allfetsel('id_form,id_formulaire', 'spip_forms', 'id_formulaire>0'); - $trans = array(); + $trans = []; foreach ($rows as $row) { $trans[$row['id_form']] = $row['id_formulaire']; } - $rows = sql_allfetsel('*', 'spip_forms_donnees', sql_in('id_form', array_keys($trans)).' AND id_formulaires_reponse=0', '', 'id_donnee', '0,100'); + $rows = sql_allfetsel('*', 'spip_forms_donnees', sql_in('id_form', array_keys($trans)) . ' AND id_formulaires_reponse=0', '', 'id_donnee', '0,100'); do { foreach ($rows as $row) { #var_dump($row); - $reponse = array( + $reponse = [ 'id_formulaires_reponse' => $row['id_donnee'], // conserver le meme id par facilite (on est sur une creation de base) 'id_formulaire' => $trans[$row['id_form']], 'date' => $row['date'], @@ -121,7 +121,7 @@ function formidable_importer_forms_donnees() { 'statut' => $row['statut'], 'url' => $row['url'], 'confirmation' => $row['confirmation'], - ); + ]; #var_dump($reponse); $id_formulaires_reponse = sql_insertq('spip_formulaires_reponses', $reponse); @@ -130,18 +130,18 @@ function formidable_importer_forms_donnees() { $donnees = sql_allfetsel( "$id_formulaires_reponse as id_formulaires_reponse,champ as nom,valeur", 'spip_forms_donnees_champs', - 'id_donnee='.intval($row['id_donnee']) + 'id_donnee=' . intval($row['id_donnee']) ); - $data = array(); + $data = []; foreach ($donnees as $donnee) { $data[$donnee['nom']][] = $donnee; } - $ins = array(); + $ins = []; foreach ($data as $nom => $valeurs) { if (count($valeurs) == 1) { $ins[] = reset($valeurs); } else { - $v = array(); + $v = []; foreach ($valeurs as $valeur) { $v[] = $valeur['valeur']; } @@ -151,16 +151,15 @@ function formidable_importer_forms_donnees() { } sql_insertq_multi('spip_formulaires_reponses_champs', $ins); // et on marque la donnee pour ne pas la rejouer - sql_update('spip_forms_donnees', array('id_formulaires_reponse' => $id_formulaires_reponse), 'id_donnee='.intval($row['id_donnee'])); + sql_update('spip_forms_donnees', ['id_formulaires_reponse' => $id_formulaires_reponse], 'id_donnee=' . intval($row['id_donnee'])); } - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } } - if (time()>_TIME_OUT) { + if (time() > _TIME_OUT) { return; } - } while ($rows = sql_allfetsel('*', 'spip_forms_donnees', sql_in('id_form', array_keys($trans)).' AND id_formulaires_reponse=0', '', 'id_donnee', '0,100')); + } while ($rows = sql_allfetsel('*', 'spip_forms_donnees', sql_in('id_form', array_keys($trans)) . ' AND id_formulaires_reponse=0', '', 'id_donnee', '0,100')); } } - diff --git a/inc/rechercher_joints_formulaires_reponse_formulaires_reponses_champ.php b/inc/rechercher_joints_formulaires_reponse_formulaires_reponses_champ.php index 1edad5a3..0fa32908 100644 --- a/inc/rechercher_joints_formulaires_reponse_formulaires_reponses_champ.php +++ b/inc/rechercher_joints_formulaires_reponse_formulaires_reponses_champ.php @@ -26,5 +26,5 @@ function inc_rechercher_joints_formulaires_reponse_formulaires_reponses_champ_di $serveur ); - return array($cle_depart, $cle_arrivee, $s); + return [$cle_depart, $cle_arrivee, $s]; } diff --git a/phpcs.xml.dist b/phpcs.xml.dist new file mode 100644 index 00000000..344df64c --- /dev/null +++ b/phpcs.xml.dist @@ -0,0 +1,16 @@ +<?xml version="1.0"?> +<ruleset> + <file>.</file> + <exclude-pattern>vendor/*</exclude-pattern> + <exclude-pattern>lang/*</exclude-pattern> + <exclude-pattern>tests/*</exclude-pattern> + + <rule ref="SPIP42"/> + + <config name="ignore_warnings_on_exit" value="1"/> + <arg name="cache" value=".php_cs.cache"/> + <arg name="report-full" value=".php_cs.txt"/> + <arg name="report-summary"/> + <arg value="s"/> +</ruleset> + diff --git a/prive/objets/liste/formulaires_associer_fonctions.php b/prive/objets/liste/formulaires_associer_fonctions.php index 12885a65..83725df5 100644 --- a/prive/objets/liste/formulaires_associer_fonctions.php +++ b/prive/objets/liste/formulaires_associer_fonctions.php @@ -1,4 +1,5 @@ <?php + /** * Fonctions du squelette associé * diff --git a/prive/objets/liste/formulaires_lies_fonctions.php b/prive/objets/liste/formulaires_lies_fonctions.php index 6a97b1d2..cd3a32dc 100644 --- a/prive/objets/liste/formulaires_lies_fonctions.php +++ b/prive/objets/liste/formulaires_lies_fonctions.php @@ -1,4 +1,5 @@ <?php + /** * Fonctions du squelette associé * diff --git a/public/formidable_criteres.php b/public/formidable_criteres.php index 96f97aff..1990f5ed 100644 --- a/public/formidable_criteres.php +++ b/public/formidable_criteres.php @@ -35,15 +35,15 @@ function critere_tri_selon_reponse_dist($idb, &$boucles, $crit) { $boucle = &$boucles[$idb]; $t = $boucle->id_table; - if ($t=='formulaires_reponses') { + if ($t == 'formulaires_reponses') { $not = $crit->not; - $_quoi = calculer_liste($crit->param[0], array(), $boucles, $boucles[$idb]->id_parent); - $k = count($boucle->join)+1; - $boucle->join[$k]= array("'$t'","'id_formulaires_reponse'"); - $boucle->from["L$k"]= 'spip_formulaires_reponses_champs'; - $op = array("'='", "'L$k.nom'", "sql_quote(".$_quoi.")"); - $boucle->where[]= array("'?'","!in_array($_quoi,array('date','id_formulaires_reponse'))",$op,"''"); - $boucle->order[]= "(in_array($_quoi,array('date','id_formulaires_reponse'))?'$t.'.$_quoi:'L$k.valeur')".($not?".' DESC'":""); + $_quoi = calculer_liste($crit->param[0], [], $boucles, $boucles[$idb]->id_parent); + $k = count($boucle->join) + 1; + $boucle->join[$k] = ["'$t'","'id_formulaires_reponse'"]; + $boucle->from["L$k"] = 'spip_formulaires_reponses_champs'; + $op = ["'='", "'L$k.nom'", 'sql_quote(' . $_quoi . ')']; + $boucle->where[] = ["'?'","!in_array($_quoi,array('date','id_formulaires_reponse'))",$op,"''"]; + $boucle->order[] = "(in_array($_quoi,array('date','id_formulaires_reponse'))?'$t.'.$_quoi:'L$k.valeur')" . ($not ? ".' DESC'" : ''); } } @@ -53,17 +53,17 @@ function critere_recherche_reponse_dist($idb, &$boucles, $crit) { global $table_des_tables; $boucle = &$boucles[$idb]; $t = $boucle->id_table; - if ($t=='formulaires_reponses') { + if ($t == 'formulaires_reponses') { if (isset($crit->param[0])) { - $_quoi = calculer_liste($crit->param[0], array(), $boucles, $boucles[$idb]->id_parent); + $_quoi = calculer_liste($crit->param[0], [], $boucles, $boucles[$idb]->id_parent); } else { $_quoi = '@$Pile[0]["recherche"]'; } - $k = count($boucle->join)+1; - $boucle->join[$k]= array($t,'id_formulaires_reponse'); - $boucle->from["L$k"]= 'spip_formulaires_reponses_champs'; - $op = array("'LIKE'","'L$k.valeur'","sql_quote(strpos($_quoi,'%')===false?'%'.".$_quoi.".'%':$_quoi)"); - $boucle->where[]= array("'?'",$_quoi,$op,"''"); + $k = count($boucle->join) + 1; + $boucle->join[$k] = [$t,'id_formulaires_reponse']; + $boucle->from["L$k"] = 'spip_formulaires_reponses_champs'; + $op = ["'LIKE'","'L$k.valeur'","sql_quote(strpos($_quoi,'%')===false?'%'." . $_quoi . ".'%':$_quoi)"]; + $boucle->where[] = ["'?'",$_quoi,$op,"''"]; } } diff --git a/spip-cli/FormidableExporterFormulaire.php b/spip-cli/FormidableExporterFormulaire.php index 085389b5..bd41c626 100644 --- a/spip-cli/FormidableExporterFormulaire.php +++ b/spip-cli/FormidableExporterFormulaire.php @@ -9,7 +9,6 @@ use Symfony\Component\Console\Input\InputOption; use Symfony\Component\Console\Output\OutputInterface; class FormidableExporterFormulaire extends Command { - protected $formats = ['csv', 'xls']; protected function configure() { @@ -50,21 +49,21 @@ class FormidableExporterFormulaire extends Command { $this->io->text("Exporter le formulaire n°<info>$id_formulaire</info> en <info>$format</info>"); $fichier = $this->exporterFormulaireFormidable($id_formulaire, $format); if ($fichier) { - $this->io->check("Export réussi"); + $this->io->check('Export réussi'); - $this->io->text(["", "<info>Chemin physique</info>"]); + $this->io->text(['', '<info>Chemin physique</info>']); $this->io->text(realpath($fichier)); - $this->io->text(["", "<info>Lien de téléchargement</info>"]); + $this->io->text(['', '<info>Lien de téléchargement</info>']); include_spip('inc/securiser_action'); - $args = "$id_formulaire:" . md5_file($fichier) . ":" . basename($fichier); + $args = "$id_formulaire:" . md5_file($fichier) . ':' . basename($fichier); $cle = calculer_cle_action($args); $action = generer_url_action('formidable_recuperer_export', "args=$args&cle=$cle", true); $this->io->text(url_absolue($action)); } else { - $this->io->fail("Echec de l’export"); + $this->io->fail('Echec de l’export'); } - $this->io->text(""); + $this->io->text(''); } /** @@ -105,6 +104,5 @@ class FormidableExporterFormulaire extends Command { break; } return $fichier; - } } diff --git a/traiter/email.php b/traiter/email.php index e053b0e7..a949d6d6 100644 --- a/traiter/email.php +++ b/traiter/email.php @@ -9,7 +9,8 @@ include_spip('inc/formidable_fichiers'); function traiter_email_dist($args, $retours) { // Vérifier si on doit envoyer en cas de modification de réponses - if (isset($retours['modification_reponse']) + if ( + isset($retours['modification_reponse']) and $retours['modification_reponse'] == true and isset($args['options']['modification_reponse']) and $args['options']['modification_reponse'] == 'on' @@ -20,7 +21,7 @@ function traiter_email_dist($args, $retours) { include_spip('inc/texte'); if (!isset($retours['fichiers'])) { - $retours['fichiers'] = array(); + $retours['fichiers'] = []; $ajouter_fichier = true; } else { $ajouter_fichier = false; @@ -35,16 +36,16 @@ function traiter_email_dist($args, $retours) { $saisies_fond_notification = $saisies; $traitements = $args['traitements']; $champs = saisies_lister_champs($saisies); - $destinataires = array(); + $destinataires = []; $taille_fichiers = 0; //taille des fichiers en email - $fichiers_facteur = array(); // tableau qui stockera les fichiers à envoyer avec facteur + $fichiers_facteur = []; // tableau qui stockera les fichiers à envoyer avec facteur if (isset($options['exclure_champs_email']) && $options['exclure_champs_email']) { - $champs_a_exclure = explode(",", $options['exclure_champs_email']); + $champs_a_exclure = explode(',', $options['exclure_champs_email']); $champs = array_diff($champs, $champs_a_exclure); foreach ($champs_a_exclure as $champ_a_exclure) { $champ_a_exclure = trim($champ_a_exclure); - $champ_a_exclure = str_replace('@','',$champ_a_exclure); - $saisies_fond_notification = saisies_supprimer($saisies_fond_notification,$champ_a_exclure); + $champ_a_exclure = str_replace('@', '', $champ_a_exclure); + $saisies_fond_notification = saisies_supprimer($saisies_fond_notification, $champ_a_exclure); } } $nom_site_spip = supprimer_tags(typo(lire_config('nom_site'))); @@ -91,16 +92,15 @@ function traiter_email_dist($args, $retours) { } } - $destinataires = pipeline('formidable_traiter_email_destinataires', array( + $destinataires = pipeline('formidable_traiter_email_destinataires', [ 'args' => array_merge( $args, - array( + [ 'id_formulaires_reponse' => isset($retours['id_formulaires_reponse']) ? $retours['id_formulaires_reponse'] : '', 'id_formulaire' => $args['formulaire']['id_formulaire']//Un peu redondant, mais ca aide - ) + ] ), - 'data' => $destinataires) - ); + 'data' => $destinataires]); // S'assurer que les destinataires ne soient pas en doublons @@ -171,24 +171,25 @@ function traiter_email_dist($args, $retours) { ); } if (!isset($sujet) or !$sujet) { - $sujet = _T('formidable:traiter_email_sujet', array('nom'=>$nom_envoyeur)); + $sujet = _T('formidable:traiter_email_sujet', ['nom' => $nom_envoyeur]); } if (isset($options['champ_sujet_modif_reponse']) and $options['champ_sujet_modif_reponse'] and $retours['modification_reponse']) { - $sujet = $sujet.' '._T('formidable:traitement_email_sujet_courriel_modif_reponse'); + $sujet = $sujet . ' ' . _T('formidable:traitement_email_sujet_courriel_modif_reponse'); } $sujet = filtrer_entites($sujet); // Mais quel va donc être le fond ? - if (find_in_path('notifications/formulaire_'.$formulaire['identifiant'].'_email.html')) { - $notification = 'notifications/formulaire_'.$formulaire['identifiant'].'_email'; + if (find_in_path('notifications/formulaire_' . $formulaire['identifiant'] . '_email.html')) { + $notification = 'notifications/formulaire_' . $formulaire['identifiant'] . '_email'; } else { $notification = 'notifications/formulaire_email'; } // Est-ce qu'on est assez léger pour joindre les pj $joindre_pj = false; - if ($taille_fichiers < 1024 * 1024 * _FORMIDABLE_TAILLE_MAX_FICHIERS_EMAIL + if ( + $taille_fichiers < 1024 * 1024 * _FORMIDABLE_TAILLE_MAX_FICHIERS_EMAIL and $traitements['email']['pj'] == 'on' ) { @@ -197,7 +198,7 @@ function traiter_email_dist($args, $retours) { // On génère le mail avec le fond $html = recuperer_fond( $notification, - array( + [ 'id_formulaire' => $args['id_formulaire'], 'id_formulaires_reponse' => isset($retours['id_formulaires_reponse']) ? $retours['id_formulaires_reponse'] : '', 'titre' => _T_ou_typo($formulaire['titre']), @@ -205,11 +206,11 @@ function traiter_email_dist($args, $retours) { 'saisies' => $saisies_fond_notification, 'valeurs' => $valeurs, 'masquer_liens' => $options['masquer_liens'], - 'ip' => $options['activer_ip']?$GLOBALS['ip']:'', + 'ip' => $options['activer_ip'] ? $GLOBALS['ip'] : '', 'envoi_precedent' => isset($retours['envoi_precedent']) ? $retours['envoi_precedent'] : '' , 'courriel_envoyeur' => $courriel_envoyeur, 'nom_envoyeur' => filtrer_entites($nom_envoyeur) - ) + ] ); // On génère le texte brut @@ -217,11 +218,11 @@ function traiter_email_dist($args, $retours) { $texte = facteur_mail_html2text($html); // On utilise la forme avancée de Facteur - $corps = array( + $corps = [ 'html' => $html, 'texte' => $texte, 'nom_envoyeur' => filtrer_entites($nom_envoyeur), - ); + ]; // Joindre les pj si léger if ($joindre_pj) { $corps['pieces_jointes'] = $fichiers_facteur; @@ -242,7 +243,7 @@ function traiter_email_dist($args, $retours) { // On envoie aux destinataires if ($destinataires) { - if (!$envoyer_mail($destinataires, $sujet, $corps, $courriel_from, 'X-Originating-IP: '.$GLOBALS['ip'])){ + if (!$envoyer_mail($destinataires, $sujet, $corps, $courriel_from, 'X-Originating-IP: ' . $GLOBALS['ip'])) { $erreur_envoi = true; } } @@ -266,8 +267,8 @@ function traiter_email_dist($args, $retours) { $sujet_accuse = filtrer_entites($sujet_accuse); // Mais quel va donc être le fond ? - if (find_in_path('notifications/formulaire_'.$formulaire['identifiant'].'_accuse.html')) { - $accuse = 'notifications/formulaire_'.$formulaire['identifiant'].'_accuse'; + if (find_in_path('notifications/formulaire_' . $formulaire['identifiant'] . '_accuse.html')) { + $accuse = 'notifications/formulaire_' . $formulaire['identifiant'] . '_accuse'; } else { $accuse = 'notifications/formulaire_accuse'; } @@ -276,7 +277,7 @@ function traiter_email_dist($args, $retours) { if (_FORMIDABLE_LIENS_FICHIERS_ACCUSE_RECEPTION == false) { $valeurs = vues_saisies_supprimer_action_recuperer_fichier_par_email($saisies, $valeurs); } - $parametres_accuse = array( + $parametres_accuse = [ 'id_formulaire' => $formulaire['id_formulaire'], 'id_formulaires_reponse' => isset($retours['id_formulaires_reponse']) ? $retours['id_formulaires_reponse'] : '', 'titre' => _T_ou_typo($formulaire['titre']), @@ -290,14 +291,15 @@ function traiter_email_dist($args, $retours) { ], ), 'traitements' => $traitements, - 'saisies' => $options['masquer_valeurs_accuse'] ? array() : $saisies_fond_notification, - 'valeurs' => $options['masquer_valeurs_accuse'] ? array() : $valeurs - ); - $parametres_accuse = pipeline('formidable_parametres_accuse', - array( - 'args' => array('id_formulaire' => $parametres_accuse['id_formulaire'], 'id_formulaires_reponse' => $parametres_accuse['id_formulaires_reponse']), + 'saisies' => $options['masquer_valeurs_accuse'] ? [] : $saisies_fond_notification, + 'valeurs' => $options['masquer_valeurs_accuse'] ? [] : $valeurs + ]; + $parametres_accuse = pipeline( + 'formidable_parametres_accuse', + [ + 'args' => ['id_formulaire' => $parametres_accuse['id_formulaire'], 'id_formulaires_reponse' => $parametres_accuse['id_formulaires_reponse']], 'data' => $parametres_accuse - ) + ] ); $html_accuse = recuperer_fond( @@ -315,11 +317,11 @@ function traiter_email_dist($args, $retours) { $nom_envoyeur_accuse = $nom_site_spip; } - $corps = array( + $corps = [ 'html' => $html_accuse, 'texte' => $texte, 'nom_envoyeur' => filtrer_entites($nom_envoyeur_accuse), - ); + ]; //A fortiori, si un courriel d'expéditeur est précisé pour l'AR, on l'utilise if ($options['courriel_envoyeur_accuse']) { @@ -341,7 +343,7 @@ function traiter_email_dist($args, $retours) { $corps['pieces_jointes'] = $fichiers_facteur; } - if(!$envoyer_mail($courriel_envoyeur, $sujet_accuse, $corps, $courriel_from_accuse, 'X-Originating-IP: '.$GLOBALS['ip'])){ + if (!$envoyer_mail($courriel_envoyeur, $sujet_accuse, $corps, $courriel_from_accuse, 'X-Originating-IP: ' . $GLOBALS['ip'])) { $erreur_envoi = true; } } @@ -351,7 +353,7 @@ function traiter_email_dist($args, $retours) { } } else { if (isset($retours['message_erreur'])) { - $retours['message_erreur'] .= "\n"._T('formidable:traiter_email_message_erreur'); + $retours['message_erreur'] .= "\n" . _T('formidable:traiter_email_message_erreur'); } else { $retours['message_erreur'] = _T('formidable:traiter_email_message_erreur'); } @@ -370,17 +372,17 @@ function traiter_email_dist($args, $retours) { **/ function formidable_traiter_email_champ_destinataires($champ) { if (!is_array($champ)) { - $champ = array($champ); + $champ = [$champ]; } //Trouver tout les id destinataires - $destinataires = array(); + $destinataires = []; foreach ($champ as $c) { $destinataires_c = _request($c); if (!is_array($destinataires_c)) { if (intval($destinataires_c)) { - $destinataires_c = array($destinataires_c); + $destinataires_c = [$destinataires_c]; } else { - $destinataires_c = array(); + $destinataires_c = []; } } $destinataires = array_merge($destinataires, $destinataires_c); @@ -410,7 +412,7 @@ function formidable_traiter_email_champ_destinataires($champ) { * Liste des destinataires, s'il y en a. **/ function formidable_traiter_email_destinataire_selon_champ($description) { - $destinataires = array(); + $destinataires = []; // 1 test à rechercher par ligne $descriptions = explode("\n", trim($description)); @@ -474,22 +476,22 @@ function formidable_traiter_email_destinataire_selon_champ($description) { function traiter_email_fichiers($saisie, $nom, $id_formulaire, $retours, $timestamp) { //Initialisation $id_formulaire = strval($id_formulaire);//précaution - $vue = array(); + $vue = []; if (isset($retours['id_formulaires_reponse']) and $id_formulaires_reponse = $retours['id_formulaires_reponse']) { // cas simple: les réponses ont été enregistrées if (isset($retours['fichiers'][$nom])) { // petite précaution - $options = array( + $options = [ 'id_formulaire' => $id_formulaire, 'id_formulaires_reponse' => $retours['id_formulaires_reponse'] - ); + ]; $vue = ajouter_action_recuperer_fichier_par_email($retours['fichiers'][$nom], $nom, $options); } } else { // si les réponses n'ont pas été enregistrées - $vue = formidable_deplacer_fichiers_produire_vue_saisie($saisie, array('id_formulaire' => $id_formulaire, 'timestamp' => $timestamp)); - $options = array( + $vue = formidable_deplacer_fichiers_produire_vue_saisie($saisie, ['id_formulaire' => $id_formulaire, 'timestamp' => $timestamp]); + $options = [ 'id_formulaire' => $id_formulaire, 'timestamp' => $timestamp - ); + ]; $vue = ajouter_action_recuperer_fichier_par_email($vue, $nom, $options); } @@ -518,7 +520,7 @@ function ajouter_action_recuperer_fichier_par_email($saisie_a_modifier, $nom_sai $saisie_a_modifier[$i]['url'] = $url; if (_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL > 0) { $saisie_a_modifier[$i]['fichier'] = $valeur['nom']; - $saisie_a_modifier[$i]['nom'] = '['._T('formidable:lien_expire', array('delai' => $delai)).'] '.$valeur['nom']; + $saisie_a_modifier[$i]['nom'] = '[' . _T('formidable:lien_expire', ['delai' => $delai]) . '] ' . $valeur['nom']; } else { $saisie_a_modifier[$i]['fichier'] = $valeur['nom']; $saisie_a_modifier[$i]['nom'] = $valeur['nom']; @@ -587,15 +589,15 @@ function formidable_calculer_taille_fichiers_saisie($saisie) { * @return array $tableau_facteur **/ function vue_fichier_to_tableau_facteur($vue) { - $tableau_facteur = array(); + $tableau_facteur = []; foreach ($vue as $fichier) { - $arg = base64_decode(parametre_url($fichier['url'],'arg')); + $arg = base64_decode(parametre_url($fichier['url'], 'arg')); $arg = json_decode($arg, true); - $tableau_facteur[] = array( + $tableau_facteur[] = [ 'chemin' => formidable_generer_chemin_fichier($arg), 'nom' => $fichier['fichier'], 'encodage' => 'base64', - 'mime' => $fichier['mime']); + 'mime' => $fichier['mime']]; } return $tableau_facteur; } @@ -607,16 +609,16 @@ function vue_fichier_to_tableau_facteur($vue) { **/ function secondes_en_jour($secondes) { //On ne peut pas utiliser date_create, car en PHP 5.2, et SPIP 3.0 est à partir de PHP 5.1… - $jours = floor($secondes/(24*3600)); - $heures = floor(($secondes-$jours*24*3600)/3600); - $minutes = floor(($secondes-$jours*24*3600-$heures*3600)/60); - $secondes = $secondes-$jours*24*3600-$heures*3600-$minutes*60; - $param = array( + $jours = floor($secondes / (24 * 3600)); + $heures = floor(($secondes - $jours * 24 * 3600) / 3600); + $minutes = floor(($secondes - $jours * 24 * 3600 - $heures * 3600) / 60); + $secondes = $secondes - $jours * 24 * 3600 - $heures * 3600 - $minutes * 60; + $param = [ 'j' => $jours, 'h' => $heures, 'm' => $minutes, 's' => $secondes - ); + ]; if ($jours > 0) { return _T('formidable:jours_heures_minutes_secondes', $param); } elseif ($heures > 0) { diff --git a/traiter/enregistrement.php b/traiter/enregistrement.php index 7f437520..18b4543a 100644 --- a/traiter/enregistrement.php +++ b/traiter/enregistrement.php @@ -9,7 +9,7 @@ function traiter_enregistrement_dist($args, $retours) { include_spip('inc/formidable'); include_spip('base/abstract_sql'); include_spip('inc/formidable'); - $retours['fichiers'] = array(); // on va stocker des infos sur les fichiers, pour les prochains traitement + $retours['fichiers'] = []; // on va stocker des infos sur les fichiers, pour les prochains traitement $options = $args['options']; $formulaire = $args['formulaire']; $id_formulaire = $args['id_formulaire']; @@ -21,7 +21,7 @@ function traiter_enregistrement_dist($args, $retours) { // traitement de l'anonymisation de l'auteur lorsque la méthode d'identification se fait par l'identifiant if ($options['anonymiser'] == 'on' and $id_auteur) { - if ($options['identification'] == 'id_auteur' ) { + if ($options['identification'] == 'id_auteur') { $variable_php = formidable_hasher_id_auteur($id_auteur); } $id_auteur = 0; @@ -37,7 +37,7 @@ function traiter_enregistrement_dist($args, $retours) { } // On crée un identifiant depuis l'éventuelle variable php d'identification - if ($options['identification'] == 'variable_php' ) { + if ($options['identification'] == 'variable_php') { $variable_php = formidable_variable_php_identification($options['variable_php'], $id_formulaire); } @@ -50,28 +50,29 @@ function traiter_enregistrement_dist($args, $retours) { // Si la moderation est a posteriori // ou que la personne est un·e boss, mais qu'on ne demande pas à modérer les boss, // on publie direct - if ($options['moderation'] == 'posteriori' + if ( + $options['moderation'] == 'posteriori' or (autoriser( 'instituer', 'formulairesreponse', $id_formulaires_reponse, null, - array('id_formulaire' => $id_formulaire, 'nouveau_statut' => 'publie') - ) - and !(isset($options['moderer_admins']) and $options['moderer_admins']=='on') + ['id_formulaire' => $id_formulaire, 'nouveau_statut' => 'publie'] + ) + and !(isset($options['moderer_admins']) and $options['moderer_admins'] == 'on') ) ) { - $statut='publie'; + $statut = 'publie'; } else { $statut = 'prop'; } // Si ce n'est pas une modif d'une réponse existante, on crée d'abord la réponse if (!$id_formulaires_reponse) { - spip_log("Création d'une nouvelle réponse pour le formulaire $id_formulaire", 'formidable'._LOG_INFO); + spip_log("Création d'une nouvelle réponse pour le formulaire $id_formulaire", 'formidable' . _LOG_INFO); $id_formulaires_reponse = sql_insertq( 'spip_formulaires_reponses', - array( + [ 'id_formulaire' => $id_formulaire, 'id_auteur' => $id_auteur, 'cookie' => $cookie, @@ -80,9 +81,9 @@ function traiter_enregistrement_dist($args, $retours) { 'date' => 'NOW()', 'date_envoi' => 'NOW()', 'statut' => $statut - ) + ] ); - spip_log("La réponse créée est $id_formulaires_reponse", 'formidable'._LOG_INFO); + spip_log("La réponse créée est $id_formulaires_reponse", 'formidable' . _LOG_INFO); // Si on a pas le droit de répondre plusieurs fois ou que les réponses seront modifiables, // il faut poser un cookie if (!$options['multiple'] or $options['modifiable']) { @@ -92,15 +93,15 @@ function traiter_enregistrement_dist($args, $retours) { } $retours['modification_reponse'] = false;// signaler aux traitements qu'il s'agit d'une nouvelle réponse } else { // si c'est une modif de réponse existante - spip_log("Modification de la réponse $id_formulaires_reponse", 'formidable'._LOG_INFO); + spip_log("Modification de la réponse $id_formulaires_reponse", 'formidable' . _LOG_INFO); // simple mise à jour du champ maj de la table spip_formulaires_reponses - $envoi_precedent = sql_getfetsel('date_envoi','spip_formulaires_reponses', "id_formulaires_reponse = $id_formulaires_reponse"); + $envoi_precedent = sql_getfetsel('date_envoi', 'spip_formulaires_reponses', "id_formulaires_reponse = $id_formulaires_reponse"); sql_updateq( 'spip_formulaires_reponses', - array( + [ 'maj' => 'NOW()', 'date_envoi' => 'NOW()', - ), + ], "id_formulaires_reponse = $id_formulaires_reponse" ); //effacer les fichiers existant @@ -110,22 +111,22 @@ function traiter_enregistrement_dist($args, $retours) { } // Si l'id n'a pas été créé correctement alors erreur if (!($id_formulaires_reponse > 0)) { - spip_log('La réponse n\'a pas été enregistrée', 'formidable'._LOG_ERREUR); - $retours['message_erreur'] .= "\n<br/>"._T('formidable:traiter_enregistrement_erreur_base'); + spip_log('La réponse n\'a pas été enregistrée', 'formidable' . _LOG_ERREUR); + $retours['message_erreur'] .= "\n<br/>" . _T('formidable:traiter_enregistrement_erreur_base'); } else { // Sinon on continue à mettre à jour - $champs = array(); - $insertions = array(); + $champs = []; + $insertions = []; foreach ($saisies as $nom => $saisie) { if ($saisie['saisie'] == 'fichiers') { // traiter à part le cas des saisies fichiers $valeur = traiter_enregistrement_fichiers($saisie, $id_formulaire, $id_formulaires_reponse); if (($valeur !== null)) { $champs[] = $nom; - $insertions[] = array( + $insertions[] = [ 'id_formulaires_reponse' => $id_formulaires_reponse, 'nom' => $nom, 'valeur' => is_array($valeur) ? formidable_serialize($valeur) : $valeur - ); + ]; $retours['fichiers'][$nom] = $valeur; } } @@ -133,11 +134,11 @@ function traiter_enregistrement_dist($args, $retours) { // Pour le saisies différentes de fichiers, // on ne prend que les champs qui ont effectivement été envoyés par le formulaire $champs[] = $nom; - $insertions[] = array( + $insertions[] = [ 'id_formulaires_reponse' => $id_formulaires_reponse, 'nom' => $nom, 'valeur' => is_array($valeur) ? formidable_serialize($valeur) : $valeur - ); + ]; } } // S'il y a bien des choses à modifier @@ -146,15 +147,15 @@ function traiter_enregistrement_dist($args, $retours) { // En effet, ils pouvaient y avoir des champs remplis lors du précédent enregistrement // Qui ne le sont plus au nouvel enregistrement, car la condition d'affichage (afficher_si) n'est plus remplie // Dans ce cas il ne faut pas qu'ils continuent à être stockés en base, car cela peut fausser les affichages divers (type tableaux et autres) - spip_log("Suppression des potentielles anciennes valeurs pour $id_formulaires_reponse", 'formidable'._LOG_INFO); + spip_log("Suppression des potentielles anciennes valeurs pour $id_formulaires_reponse", 'formidable' . _LOG_INFO); sql_delete( 'spip_formulaires_reponses_champs', - array( - 'id_formulaires_reponse = '.$id_formulaires_reponse - ) + [ + 'id_formulaires_reponse = ' . $id_formulaires_reponse + ] ); - spip_log("Insertion des nouvelles valeurs pour $id_formulaires_reponse", 'formidable'._LOG_INFO); + spip_log("Insertion des nouvelles valeurs pour $id_formulaires_reponse", 'formidable' . _LOG_INFO); // Puis on insère les nouvelles valeurs sql_insertq_multi( 'spip_formulaires_reponses_champs', @@ -188,12 +189,13 @@ function traiter_enregistrement_verifier_dist($args, $erreurs) { $options = $args['options']; $id_formulaires_reponse = $args['id_formulaires_reponse']; $etape = $args['etape']; - if (($unicite = $options['unicite'] ?? '') and !($erreurs[$unicite] ?? '') + if ( + ($unicite = $options['unicite'] ?? '') and !($erreurs[$unicite] ?? '') and ( $etape === null or ( - array_key_exists($unicite, saisies_lister_par_nom($args['etapes']['etape_'.$etape]['saisies'])) + array_key_exists($unicite, saisies_lister_par_nom($args['etapes']['etape_' . $etape]['saisies'])) ) ) ) { @@ -202,7 +204,7 @@ function traiter_enregistrement_verifier_dist($args, $erreurs) { } if ($id_formulaires_reponse != false) { - $unicite_exclure_reponse_courante = ' AND R.id_formulaires_reponse != '.$id_formulaires_reponse; + $unicite_exclure_reponse_courante = ' AND R.id_formulaires_reponse != ' . $id_formulaires_reponse; } else { $unicite_exclure_reponse_courante = ''; } @@ -216,8 +218,8 @@ function traiter_enregistrement_verifier_dist($args, $erreurs) { ON R.id_formulaires_reponse=C.id_formulaires_reponse', 'R.id_formulaire = ' . $id_formulaire . $unicite_exclure_reponse_courante . - ' AND C.nom='.sql_quote($unicite).' - AND C.valeur='.sql_quote(_request($unicite)).' + ' AND C.nom=' . sql_quote($unicite) . ' + AND C.valeur=' . sql_quote(_request($unicite)) . ' AND R.statut = "publie"' ); if (is_array($reponses) && count($reponses) > 0) { @@ -239,5 +241,5 @@ function traiter_enregistrement_verifier_dist($args, $erreurs) { * @return array|null un tableau organisé par fichier, contenant 'nom', 'extension','mime','taille' **/ function traiter_enregistrement_fichiers($saisie, $id_formulaire, $id_formulaires_reponse) { - return formidable_deplacer_fichiers_produire_vue_saisie($saisie, array('id_formulaire' => $id_formulaire, 'id_formulaires_reponse' => $id_formulaires_reponse)); + return formidable_deplacer_fichiers_produire_vue_saisie($saisie, ['id_formulaire' => $id_formulaire, 'id_formulaires_reponse' => $id_formulaires_reponse]); } diff --git a/vues/formulaires_reponses_champ.php b/vues/formulaires_reponses_champ.php index 3d19d370..6e0b98d1 100644 --- a/vues/formulaires_reponses_champ.php +++ b/vues/formulaires_reponses_champ.php @@ -21,5 +21,5 @@ function vues_formulaires_reponses_champ_dist($type, $modele, $id, $content, $wi ['spip_formulaires_reponses_champs AS c', 'spip_formulaires_reponses AS r', 'spip_formulaires AS f'], ["id_formulaires_reponses_champ = $id", 'r.id_formulaires_reponse = c.id_formulaires_reponse', 'f.id_formulaire = r.id_formulaire'] ); - return calculer_voir_reponse($data['id_formulaires_reponse'], $data['id_formulaire'], $data['nom']); + return calculer_voir_reponse($data['id_formulaires_reponse'], $data['id_formulaire'], $data['nom']); } -- GitLab