diff --git a/formidable_fonctions.php b/formidable_fonctions.php index 2fc83feb90a5ccfc90116d11089c444ed9c9a253..58a39eb5a8f721b0c9a360728e24c805625eb2b5 100644 --- a/formidable_fonctions.php +++ b/formidable_fonctions.php @@ -52,11 +52,17 @@ function calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, $t // Si pas déjà présent, on cherche les saisies de ce formulaire if (!isset($formulaires_saisies[$id_formulaire])) { - $formulaires_saisies[$id_formulaire] = unserialize(sql_getfetsel('saisies', 'spip_formulaires', 'id_formulaire = '.intval($id_formulaire))); + $formulaires_saisies[$id_formulaire] = unserialize( + sql_getfetsel('saisies', 'spip_formulaires', 'id_formulaire = '.intval($id_formulaire)) + ); } // 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', 'spip_formulaires_reponses_champs', 'id_formulaires_reponse = '.intval($id_formulaires_reponse))) { + if ($champs = sql_allfetsel( + 'nom,valeur', + 'spip_formulaires_reponses_champs', + 'id_formulaires_reponse = '.intval($id_formulaires_reponse) + )) { foreach ($champs as $champ) { $reponses_valeurs[$id_formulaires_reponse][$champ['nom']] = $tenter_unserialize($champ['valeur']); } @@ -99,10 +105,18 @@ function affiche_resume_reponse($id_formulaires_reponse, $id_formulaire = null, static $modeles_vars = array(); if (is_null($id_formulaire)) { - $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) + ); } if (is_null($modele_resume) and !isset($modeles_resume[$id_formulaire])) { - $modeles_resume[$id_formulaire] = sql_getfetsel('resume_reponse', 'spip_formulaires', 'id_formulaire='.intval($id_formulaire)); + $modeles_resume[$id_formulaire] = sql_getfetsel( + 'resume_reponse', + 'spip_formulaires', + 'id_formulaire='.intval($id_formulaire) + ); } if (is_null($modele_resume)) { $modele_resume = $modeles_resume[$id_formulaire]; @@ -166,8 +180,13 @@ function formidable_ajouter_action_recuperer_fichier($saisie_a_modifier, $nom_sa $vignette_par_defaut = charger_fonction('vignette', 'inc/'); if (array_key_exists($nom_saisie, saisies_lister_avec_type($saisies_du_formulaire, 'fichiers'))) { //saisies SPIP if (isset($saisie_a_modifier) and is_array($saisie_a_modifier)) { - foreach ($saisie_a_modifier as $i => $valeur){ - $url = formidable_generer_url_action_recuperer_fichier($id_formulaire, $id_formulaires_reponse, $nom_saisie, $valeur['nom']); + foreach ($saisie_a_modifier as $i => $valeur) { + $url = formidable_generer_url_action_recuperer_fichier( + $id_formulaire, + $id_formulaires_reponse, + $nom_saisie, + $valeur['nom'] + ); $saisie_a_modifier[$i]['url'] = $url; $saisie_a_modifier[$i]['vignette'] = $vignette_par_defaut($valeur['extension'],false); } diff --git a/formidable_pipelines.php b/formidable_pipelines.php index 41deac2ef1e6c28c2c4f05ac4fb6923bd6a7498b..67be1c188b08879be31abf854746fae26027b7e8 100644 --- a/formidable_pipelines.php +++ b/formidable_pipelines.php @@ -44,7 +44,11 @@ function formidable_trouve_liens($texte) { if ($r[2] == 'formidable') { $id_formulaire = $r[3]; } elseif ($r[2] == 'form') { - $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires', 'identifiant='.sql_quote('form'.$r[3])); + $id_formulaire = sql_getfetsel( + 'id_formulaire', + 'spip_formulaires', + 'identifiant='.sql_quote('form'.$r[3]) + ); } elseif ($r[2] == 'formulaire|formidable') { $args = ltrim($r[4], '|'); $args = explode('=', $args); @@ -54,7 +58,11 @@ function formidable_trouve_liens($texte) { if (is_numeric($args)) { $id_formulaire = intval($args); } else { - $id_formulaire = sql_getfetsel('id_formulaire', 'spip_formulaires', 'identifiant='.sql_quote($args)); + $id_formulaire = sql_getfetsel( + 'id_formulaire', + 'spip_formulaires', + 'identifiant='.sql_quote($args) + ); } } if ($id_formulaire = intval($id_formulaire)) { @@ -114,7 +122,10 @@ function formidable_affiche_droite($flux) { 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))) { - $flux['data'] .= recuperer_fond('prive/squelettes/inclure/formulaires_lies', array('objet' => $objet, 'id_objet' => $id)); + $flux['data'] .= recuperer_fond( + 'prive/squelettes/inclure/formulaires_lies', + array('objet' => $objet, 'id_objet' => $id) + ); } return $flux; } @@ -124,9 +135,9 @@ function formidable_affiche_droite($flux) { **/ function formidable_affiche_milieu($flux) { include_spip('inc/config'); - $texte = ""; + $texte = ''; $e = trouver_objet_exec($flux['args']['exec']); - + if (!$e['edition'] and in_array($e['table_objet_sql'], lire_config('formidable/analyse/objets', array()))) { $texte .= recuperer_fond('prive/objets/editer/liens', array( 'table_source' => 'formulaires', @@ -134,16 +145,15 @@ function formidable_affiche_milieu($flux) { 'id_objet' => $flux['args'][$e['id_table_objet']] )); } - + if ($texte) { if ($p=strpos($flux['data'], '<!--affiche_milieu-->')) { $flux['data'] = substr_replace($flux['data'], $texte, $p, 0); - } - else { + } else { $flux['data'] .= $texte; } } - + return $flux; } @@ -172,12 +182,10 @@ function formidable_optimiser_base_disparus($flux) { // On génère la suppression $flux['data'] += optimiser_sansref('spip_formulaires', 'id_formulaire', $res); - while ($reponse = sql_fetch($res2)){ + while ($reponse = sql_fetch($res2)) { $flux['data'] += formidable_effacer_fichiers_formulaire($reponse['id']); } - - // les reponses qui sont associees a un formulaire inexistant $res = sql_select( 'R.id_formulaire AS id', @@ -192,7 +200,7 @@ function formidable_optimiser_base_disparus($flux) { 'id_formulaires_reponse AS id, id_formulaire AS form', 'spip_formulaires_reponses', 'statut='.sql_quote('poubelle') - ); + ); $res2 = sql_select( 'id_formulaires_reponse AS id, id_formulaire AS form', 'spip_formulaires_reponses', @@ -200,7 +208,7 @@ function formidable_optimiser_base_disparus($flux) { ); //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); - while ($reponse = sql_fetch($res2)){ + while ($reponse = sql_fetch($res2)) { $flux['data'] += formidable_effacer_fichiers_reponse($reponse['form'], $reponse['id']); } @@ -208,7 +216,8 @@ function formidable_optimiser_base_disparus($flux) { // les champs des reponses associes a une reponse inexistante $res = sql_select( 'C.id_formulaires_reponse AS id', - 'spip_formulaires_reponses_champs AS C LEFT JOIN spip_formulaires_reponses AS R ON C.id_formulaires_reponse=R.id_formulaires_reponse', + 'spip_formulaires_reponses_champs AS C + LEFT JOIN spip_formulaires_reponses AS R ON C.id_formulaires_reponse=R.id_formulaires_reponse', 'C.id_formulaires_reponse > 0 AND R.id_formulaires_reponse IS NULL' ); @@ -219,20 +228,20 @@ function formidable_optimiser_base_disparus($flux) { /** * S'assurer que le traitement email ait lieu après le traitement enregistrement - * + * * @pipeline formidable_traitements - * @param array $flux + * @param array $flux * @return array $flux **/ function formidable_formidable_traitements($flux) { if (isset($flux['data']['email']) and isset($flux['data']['enregistrement'])) { - $keys = array_keys($flux['data']); + $keys = array_keys($flux['data']); $position_email = array_search('email', $keys); $position_enregistrement = array_search('enregistrement', $keys); if ($position_enregistrement > $position_email) { // si enregistrement après email $nouveau_tab = array(); - foreach ($keys as $key) { //on reconstruit le tableau, en inversant simplement email et enregistrement + foreach ($keys as $key) { //on reconstruit le tableau, en inversant simplement email et enregistrement if ($key == 'email') { $nouveau_tab['enregistrement'] = $flux['data']['enregistrement']; } elseif ($key == 'enregistrement') { @@ -251,22 +260,22 @@ function formidable_formidable_traitements($flux) { /** Hasher les ip régulièrement * @param array $flux * @return array $flux -**/ -function formidable_taches_generales_cron($flux){ +**/ +function formidable_taches_generales_cron($flux) { $flux['formidable_hasher_ip'] = 24*3600; $flux['formidable_effacer_fichiers_email'] = 24*3600; return $flux; } -/** Déclarer les formulaires et les réponses +/** Déclarer les formulaires et les réponses * au plugin corbeille * @param array $flux; * @return array $flux; **/ -function formidable_corbeille_table_infos($flux){ +function formidable_corbeille_table_infos($flux) { $flux['formulaires']= array( 'statut'=>'poubelle', - 'table'=>'formulaires', + 'table'=>'formulaires', 'tableliee'=>array('spip_formulaires_liens') ); $flux['formulaires_reponses']= array( diff --git a/inc/formidable_fichiers.php b/inc/formidable_fichiers.php index 1a28689604b8e899498a6817e084fc4f69932ab0..0408ae56fe2105b2ad7340c97018cd5d2250bfe0 100644 --- a/inc/formidable_fichiers.php +++ b/inc/formidable_fichiers.php @@ -78,7 +78,7 @@ function formidable_creer_dossier_formulaire($id_formulaire, $forcer = false) { include_spip('inc/distant'); $url = url_absolue($fichier); - if ($data = recuperer_page($url) && $data != NULL) { + if ($data = recuperer_page($url) && $data != null) { // si on peut récuperer la page avec un statut http 200, // c'est qu'il y a un problème. recuperer_page() est obsolète en 3.1, mais recuperer_url() n'existe pas en 3.0 spip_log("$nom_dossier accessible en lecture depuis le web", 'formidable'._LOG_CRITIQUE); @@ -182,7 +182,7 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime } include_spip('inc/distant'); $url = url_absolue($fichier_test); - if ($data = recuperer_page($url) && $data != NULL) { + if ($data = recuperer_page($url) && $data != null) { // si on peut récuperer la page avec un statut http 200, // c'est qu'il y a un problème. // recuperer_page() est obsolète en 3.1, mais recuperer_url() n'existe pas en 3.0