style: `&&`->`and`, `or` => `||`, ainsi que quelques `==`->`===`

pull/167/head
Maïeul 2 weeks ago
parent dc4c59a292
commit 5bddd9a0c9

2
.gitignore vendored

@ -0,0 +1,2 @@
/.php_cs.cache
/.php_cs.txt

@ -22,7 +22,7 @@ function action_dupliquer_formulaire_dist($arg = null) {
// Si le formulaire existe bien
if (
$id_formulaire > 0
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire)
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire))
) {
include_spip('action/editer_formulaire');
// On enlève les champs inutiles
@ -36,7 +36,7 @@ function action_dupliquer_formulaire_dist($arg = null) {
// Si ça a marché on modifie les champs de base
if (
$id_formulaire > 0
and !($erreur = formulaire_set($id_formulaire, $formulaire))
&& !($erreur = formulaire_set($id_formulaire, $formulaire))
) {
// Et ensuite les saisies et les traitements
sql_updateq(

@ -16,9 +16,9 @@ function action_exporter_formulaire_dist($arg = null) {
if (
$id_formulaire > 0
and $type_export
and $types_echange = echanges_formulaire_lister_disponibles()
and $exporter = $types_echange['exporter'][$type_export]
&& $type_export
&& ($types_echange = echanges_formulaire_lister_disponibles())
&& ($exporter = $types_echange['exporter'][$type_export])
) {
$exporter($id_formulaire);
}

@ -34,12 +34,12 @@ function action_formidable_recuperer_export() {
list($id_formulaire, $md5, $filename) = array_pad(explode(':', $args, 3), 3, null);
if (
!$id_formulaire
or !$md5
or !$filename
or false !== stripos($filename, '/')
or false !== stripos($filename, '\\')
or empty($GLOBALS['visiteur_session']['id_auteur'])
or !autoriser('voir', 'formulairesreponse', $id_formulaire)
|| !$md5
|| !$filename
|| false !== stripos($filename, '/')
|| false !== stripos($filename, '\\')
|| empty($GLOBALS['visiteur_session']['id_auteur'])
|| !autoriser('voir', 'formulairesreponse', $id_formulaire)
) {
echo minipres();
exit;

@ -31,7 +31,7 @@ function action_formidable_recuperer_fichier() {
}
}
}
if (autoriser('voir', 'formulairesreponse', $arg['formulaire']) or $cookie_ok) {
if (autoriser('voir', 'formulairesreponse', $arg['formulaire']) || $cookie_ok) {
$chemin_fichier = _DIR_FICHIERS_FORMIDABLE
. 'formulaire_' . $arg['formulaire']
. '/reponse_' . $arg['reponse']

@ -32,7 +32,7 @@ function action_formidable_recuperer_fichier_par_email() {
if (@file_exists($chemin_fichier)) {
$f = $arg['fichier'];
$date = filemtime($chemin_fichier);
if (_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL > 0 and $date + _FORMIDABLE_EXPIRATION_FICHIERS_EMAIL < time()) {// vérifier que le fichier n'est pas trop vieux
if (_FORMIDABLE_EXPIRATION_FICHIERS_EMAIL > 0 && $date + _FORMIDABLE_EXPIRATION_FICHIERS_EMAIL < time()) {// vérifier que le fichier n'est pas trop vieux
include_spip('inc/minipres');
echo minipres(_T('formidable:erreur_fichier_expire'));
} else {

@ -19,8 +19,8 @@ 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)
($id_formulaire = intval($arg))
&& autoriser('instituer', 'formulairesreponse', $id_formulaire)
) {
// On supprime les réponse (statut => refuse)
$ok = sql_updateq(

@ -58,7 +58,7 @@ function controleurs_formulaires_reponses_champ_dist($regs, $c = null) {
// probablement pas la meilleure idée du siècle…
// mais tenter d'afficher correctement le picker de date du plugin saisies dans l'espace public
if (!test_espace_prive() and $saisie['saisie'] == 'date') {
if (!test_espace_prive() && $saisie['saisie'] === 'date') {
$scripts = '<link rel="stylesheet" type="text/css" href="' . find_in_path('css/ui/jquery-ui.css') . '" />';
}

@ -11,7 +11,7 @@ function echanger_formulaire_forms_importer_dist($fichier) {
include_spip('inc/xml');
$arbre = spip_xml_load($fichier, false);
if ($arbre and is_array($arbre) and isset($arbre['forms'])) {
if (isset($arbre['forms'])) {
foreach ($arbre['forms'] as $forms) {
foreach ($forms['form'] as $form) {
$formulaire = [];
@ -45,7 +45,7 @@ function echanger_formulaire_forms_importer_dist($fichier) {
];
// Les choix pour les types select et multiple
if (isset($field['les_choix']) and is_array($field['les_choix'])) {
if (is_array($field['les_choix'] ?? '')) {
$champ['choix'] = [];
foreach ($field['les_choix'] as $les_choix) {
foreach ($les_choix['un_choix'] as $un_choix) {
@ -88,7 +88,7 @@ function forms_importe_en_base($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']))
&& !sql_countsel('spip_formulaires', 'id_formulaire=' . intval($formulaire['id_formulaire']))
) {
$champs = [
'id_formulaire' => $formulaire['id_formulaire'],
@ -139,7 +139,7 @@ function forms_importe_en_base($formulaire) {
// Si ok on modifie les champs de base
if (
$id_formulaire > 0
and !($erreur = formulaire_modifier($id_formulaire, $formulaire))
&& !(formulaire_modifier($id_formulaire, $formulaire))
) {
return $id_formulaire;
}
@ -196,7 +196,7 @@ function forms_configure_traitement_formulaire($form, &$formulaire) {
}
// $form['email'] est possiblement serialize
if (is_string($form['email']) and $a = unserialize($form['email'])) {
if (is_string($form['email']) && ($a = unserialize($form['email']))) {
$form['email'] = $a;
}
if (is_array($form['email'])) {
@ -289,7 +289,7 @@ function forms_champ_vers_saisie($champ) {
break;
case 'num':
case 'monnaie':
if (!isset($champ['taille']) or !intval($taille = $champ['taille'])) {
if (!isset($champ['taille']) || !intval($taille = $champ['taille'])) {
$saisie['verifier'] = [
'type' => 'entier'
];
@ -352,7 +352,7 @@ function forms_champ_vers_saisie($champ) {
}
// Les choix pour les types select et multiple
if (isset($champ['choix']) and is_array($champ['choix'])) {
if (is_array($champ['choix'] ?? '')) {
$saisie['options']['datas'] = [];//on conserve datas, car c'est ainsi que dans les .yaml
foreach ($champ['choix'] as $un_choix) {
$choix = $un_choix['choix'];
@ -365,21 +365,21 @@ function forms_champ_vers_saisie($champ) {
$saisie['options']['nom'] = $champ['champ'];
// Le label
if (isset($champ['titre']) and $champ['titre']) {
if ($champ['titre'] ?? '') {
$saisie['options']['label'] = $champ['titre'];
}
// Obligatoire
if (isset($champ['obligatoire']) and $champ['obligatoire'] == 'oui') {
if (($champ['obligatoire'] ?? '') === 'oui') {
$saisie['options']['obligatoire'] = 'on';
}
// Explication éventuelle
if (isset($champ['aide']) and $explication = $champ['aide']) {
$saisie['options']['explication'] = $explication;
if ($champ['aide'] ?? '') {
$saisie['options']['explication'] = $champ['aide'];
}
if (isset($champ['saisie']) and $champ['saisie'] == 'non') {
if (($champ['saisie'] ?? '') === 'non') {
$saisie['options']['disable'] = 'on';
// masquer en JS, fallback
$saisie['options']['afficher_si'] = 'false';

@ -12,7 +12,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) {
include_spip('inc/formidable');
$arbre = spip_xml_load($fichier, false);
if ($arbre and is_array($arbre) and isset($arbre['formdef'])) {
if (isset($arbre['formdef'])) {
foreach ($arbre['formdef'] as $form) {
$formulaire = [];
@ -118,7 +118,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) {
// On continue seulement si on a toujours une saisie
if ($saisie) {
// Les choix pour les types select
if (isset($field['items']) and is_array($field['items'])) {
if (is_array($field['items'] ?? '')) {
$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) {
@ -169,7 +169,7 @@ function echanger_formulaire_wcs_importer_dist($fichier) {
}
}
if ($id_formulaire and !$erreur) {
if ($id_formulaire && !$erreur) {
return $id_formulaire;
} else {
return _T('formidable:erreur_importer_wcs') . ' : ' . $erreur;

@ -42,7 +42,7 @@ function echanger_formulaire_yaml_importer_dist($fichier) {
$erreur = '';
// Si on a bien recupere une chaine on tente de la decoder
if (lire_fichier($fichier, $yaml) and $yaml) {
if (lire_fichier($fichier, $yaml) && $yaml) {
include_spip('inc/yaml');
$formulaire = yaml_decode($yaml);
// Si le decodage marche on importe alors le contenu
@ -76,7 +76,7 @@ function echanger_formulaire_yaml_importer_dist($fichier) {
}
}
if ($id_formulaire and !$erreur) {
if ($id_formulaire && !$erreur) {
return $id_formulaire;
} else {
return _T('formidable:erreur_importer_yaml') . ' : ' . $erreur;

@ -304,10 +304,10 @@ function formidable_migrer_anonymisation() {
$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') {
if (($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') {
if (($enregistrement['identification'] ?? '') === 'id_auteur') {
$enregistrement['identification'] = 'variable_php';
}
// Mettre à jour le traitement

@ -59,14 +59,16 @@ function formidable_autoriser_par_auteur($id_formulaire, $id_auteur = 0) {
*/
function formidable_auteur_admin_reponse($qui) {
$retour = false;
$statut = $qui['statut'] ?? '';
$restreint = $qui['restreint'] ?? '';
if (
// Si c'est un admin complet
(isset($qui['statut']) and $qui['statut'] == '0minirezo' and !$qui['restreint'])
($statut === '0minirezo' && !$restreint)
// Ou un admin restreint et qu'on a autorisé les restreints entiers à être admins des formulaires
or ($qui['restreint'] and lire_config('formidable/autoriser_admin_restreint'))
|| ($statut === '0minirezo' && $restreint && lire_config('formidable/autoriser_admin_restreint'))
// Ou qu'on a autorisé les auteurs à être liés et donc admins de formulaires ET de leurs réponses
or (lire_config('formidable/auteur') and lire_config('formidable/admin_reponses_auteur'))
|| (lire_config('formidable/auteur') && lire_config('formidable/admin_reponses_auteur'))
) {
$retour = true;
}
@ -95,14 +97,15 @@ function formidable_autoriser() {
**/
function autoriser_formulaire_editer_dist($faire, $type, $id, $qui, $opt) {
$ok = false;
$statut = $qui['statut'] ?? '';
$restreint = $qui['restreint'] ?? '';
if (
// Si on est admin complet
(isset($qui['statut']) and $qui['statut'] == '0minirezo' and !$qui['restreint'])
($statut === '0minirezo' && !$restreint)
// Ou admin restreint avec l'option droit pour admin restreint
or (isset($qui['statut']) and $qui['statut'] == '0minirezo' and $qui['restreint'] and lire_config('formidable/autoriser_admin_restreint'))
|| ($statut === '0minirezo' && $statut && lire_config('formidable/autoriser_admin_restreint'))
// Ou si ya les auteurs liés et qu'on est bien lié à ce formulaire
or (lire_config('formidable/auteur') and formidable_autoriser_par_auteur($id, $qui['id_auteur']))
|| (lire_config('formidable/auteur') && formidable_autoriser_par_auteur($id, $qui['id_auteur']))
) {
$ok = true;
}
@ -123,7 +126,7 @@ function autoriser_formulaire_editer_dist($faire, $type, $id, $qui, $opt) {
* @return bool true s'il a le droit, false sinon
**/
function autoriser_formulaires_menu_dist($faire, $type, $id, $qui, $opt) {
if (isset($qui['statut']) and ($qui['statut'] == '1comite' or $qui['statut'] == '0minirezo')) {
if (in_array($qui['statut'] ?? '', ['1comite', '0minirezo'])) {
return true;
} else {
return false;
@ -162,7 +165,8 @@ function autoriser_formulaire_repondre_dist($faire, $type, $id, $qui, $opt) {
$traitements = formidable_deserialize($formulaire['traitements']);
// S'il n'y a pas d'enregistrement, c'est forcément bon
if (!isset($traitements['enregistrement']) or !($options = $traitements['enregistrement'])) {
$options = $traitements['enregistrement'] ?? '';
if (!$options) {
return true;
} else {
// Sinon faut voir les options
@ -235,11 +239,11 @@ function autoriser_formulairesreponse_instituer_dist($faire, $type, $id, $qui, $
// Si on a bien trouvé le formulaire parent
$id_formulaire
// Si on peut éditer le formulaire parent
and autoriser('editer', 'formulaire', $id_formulaire, $qui, $opt)
and (
&& autoriser('editer', 'formulaire', $id_formulaire, $qui, $opt)
&& (
// Et soit on est admin
(isset($qui['statut']) and $qui['statut'] == '0minirezo')
or
(($qui['statut'] ?? '') === '0minirezo')
||
// Soit c'est autorisé pour les auteurs liés (et si on est là c'est forcément qu'on est auteur lié puisqu'autorisé à éditer)
lire_config('formidable/admin_reponses_auteur')
)

@ -165,7 +165,7 @@ function affiche_resume_reponse($id_formulaires_reponse, $id_formulaire = null,
);
}
if (is_null($modele_resume) and !isset($modeles_resume[$id_formulaire])) {
if (is_null($modele_resume) && !isset($modeles_resume[$id_formulaire])) {
$row = sql_fetsel('saisies, traitements', 'spip_formulaires', 'id_formulaire=' . intval($id_formulaire));
$saisies = formidable_deserialize($row['saisies']);
$saisies_form[$id_formulaire] = $saisies;
@ -244,7 +244,7 @@ function formidable_ajouter_action_recuperer_fichier($saisie_a_modifier, $nom_sa
$id_formulaires_reponse = strval($id_formulaires_reponse);
$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)) {
if (isset($saisie_a_modifier) && is_array($saisie_a_modifier)) {
foreach ($saisie_a_modifier as $i => $valeur) {
$url = formidable_generer_url_action_recuperer_fichier(
$id_formulaire,

@ -84,11 +84,10 @@ function formidable_trouve_liens($texte) {
*/
function formidable_post_edition($flux) {
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))
($table = $flux['args']['table'] ?? '')
&& ($id_objet = intval($flux['args']['id_objet']))
&& ($primary = id_table_objet($table))
&& ($row = sql_fetsel('*', $table, "$primary=" . intval($id_objet)))
) {
$objet = objet_type($table);
$contenu = implode(' ', $row);
@ -133,12 +132,10 @@ function formidable_post_edition($flux) {
*/
function formidable_affiche_droite($flux) {
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))
($e = trouver_objet_exec($flux['args']['exec']))
&& ($objet = $e['type'] ?? '')
&& ($id = $flux['args'][$e['id_table_objet']] ?? '')
&& sql_countsel('spip_formulaires_liens', 'objet=' . sql_quote($objet) . ' AND id_objet=' . intval($id))
) {
$flux['data'] .= recuperer_fond(
'prive/squelettes/inclure/formulaires_lies',
@ -159,7 +156,7 @@ 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', []))) {
if ($e && !$e['edition'] && isset($flux['args'][$e['id_table_objet']]) && in_array($e['table_objet_sql'], lire_config('formidable/objets', []))) {
$texte .= recuperer_fond('prive/objets/editer/liens', [
'table_source' => 'formulaires',
'objet' => $e['type'],
@ -177,7 +174,7 @@ function formidable_affiche_milieu($flux) {
if (
in_array($flux['args']['exec'], ['accueil', 'suivi_edito'])
and lire_config('formidable/reponses_page_accueil')
&& lire_config('formidable/reponses_page_accueil')
) {
$flux['data'] .= recuperer_fond('prive/objets/liste/formulaires_reponses', [], ['ajax' => true]);
}
@ -266,7 +263,7 @@ function formidable_optimiser_base_disparus($flux) {
* @return array $flux
**/
function formidable_formidable_traitements($flux) {
if (isset($flux['data']['email']) and isset($flux['data']['enregistrement'])) {
if (isset($flux['data']['email']) && isset($flux['data']['enregistrement'])) {
$keys = array_keys($flux['data']);
$position_email = array_search('email', $keys);
$position_enregistrement = array_search('enregistrement', $keys);
@ -355,7 +352,7 @@ function formidable_crayons_controleur($flux) {
*/
function formidable_crayons_verifier($flux) {
// Le nom du modèle envoyé par le controleur/formulaires_reponses_champs.
if ($flux['args']['modele'] !== 'valeur' or $flux['args']['type'] !== 'formulaires_reponses_champ') {
if ($flux['args']['modele'] !== 'valeur' || $flux['args']['type'] !== 'formulaires_reponses_champ') {
return $flux;
}
include_spip('inc/saisies');

@ -142,9 +142,9 @@ function formulaires_editer_formulaire_traiter($id_formulaire, $nouveau) {
$retours = formulaires_editer_objet_traiter('formulaire', $id_formulaire);
// S'il n'y a pas d'erreur et que le formulaire est bien là
if (empty($retours['message_erreur']) and $retours['id_formulaire'] > 0) {
if (empty($retours['message_erreur']) && $retours['id_formulaire'] > 0) {
// Si c'était un nouveau on reste sur l'édition
if (!intval($id_formulaire) and $nouveau == 'oui') {
if (!intval($id_formulaire) && $nouveau === 'oui') {
// Tout a fonctionné. En fonction de la config, on attribue l'auteur courant
$auteurs = lire_config('formidable/auteur');
if ($auteurs == 'on') {

@ -14,8 +14,8 @@ function formulaires_editer_formulaire_champs_charger($id_formulaire) {
// On teste si le formulaire existe
if (
$id_formulaire
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire)
and autoriser('editer', 'formulaire', $id_formulaire)
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire))
&& autoriser('editer', 'formulaire', $id_formulaire)
) {
$saisies = formidable_deserialize($formulaire['saisies']);
@ -163,8 +163,8 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) {
// Si c'est pas une confirmation ni une annulation, ni un revert
if (
!_request('enregistrer_confirmation')
and !($annulation = _request('annulation'))
and !_request('revert')
&& !($annulation = _request('annulation'))
&& !_request('revert')
) {
// On récupère le formulaire dans la session
$saisies_nouvelles = session_get("constructeur_formulaire_formidable_$id_formulaire");
@ -189,7 +189,11 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) {
$saisies_anciennes_str = $saisies_anciennes;
array_walk_recursive($saisies_anciennes_str, 'formidable_array_walk_recursive_strval');
$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')) {
if (
$md5_precedent_formulaire_initial
&& $md5_precedent_formulaire_initial != $md5_saisies_anciennes
&& !_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);
$erreurs['message_erreur'] = _T('formidable:erreur_saisies_modifiees_parallele');
@ -215,7 +219,7 @@ function formulaires_editer_formulaire_champs_verifier($id_formulaire) {
$erreurs['message_erreur'] = $erreurs_afficher_si;
}
}
} elseif (isset($annulation) and $annulation) {
} elseif ($annulation ?? '') {
// Si on annule on génère une erreur bidon juste pour réafficher le formulaire
$erreurs['pouetpouet'] = true;
$erreurs['message_erreur'] = '';
@ -238,7 +242,7 @@ function formulaires_editer_formulaire_champs_traiter($id_formulaire) {
$retours = ['editable' => true];
}
if (_request('enregistrer') or _request('enregistrer_confirmation')) {
if (_request('enregistrer') || _request('enregistrer_confirmation')) {
// On récupère le formulaire dans la session
$saisies_nouvelles = session_get("constructeur_formulaire_formidable_$id_formulaire");

@ -16,8 +16,8 @@ function formulaires_editer_formulaire_traitements_charger($id_formulaire) {
// On teste si le formulaire existe
if (
$id_formulaire
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire)
and autoriser('editer', 'formulaire', $id_formulaire)
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire))
&& autoriser('editer', 'formulaire', $id_formulaire)
) {
$traitements = formidable_deserialize($formulaire['traitements']);
if ($id_version = _request('id_version')) {

@ -44,8 +44,8 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') {
$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'))
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire))
&& ($reponses = sql_allfetsel('*', 'spip_formulaires_reponses', 'id_formulaire = ' . $id_formulaire . ' and statut = ' . sql_quote('publie')))
) {
include_spip('inc/saisies');
include_spip('classes/facteur');
@ -121,7 +121,7 @@ function action_exporter_analyse_reponses($id_formulaire, $delim = ',') {
$cpt_ligne++;
}
if ($reponses_completes and $exporter_csv = charger_fonction('exporter_csv', 'inc/', true)) {
if ($reponses_completes && ($exporter_csv = charger_fonction('exporter_csv', 'inc/', true))) {
$exporter_csv('analyses-formulaire-' . $formulaire['identifiant'], $reponses_completes, $delim);
exit();
}

@ -95,7 +95,7 @@ function exporter_formulaires_reponses($id_formulaire, $delim = ',', $statut_rep
return false;
}
list($formulaire, $reponses) = obtenir_formulaire_reponses($id_formulaire, $statut_reponses, $date_debut, $date_fin);
if (!$formulaire or !$reponses) {
if (!$formulaire || !$reponses) {
return false;
}
@ -121,8 +121,8 @@ function obtenir_formulaire_reponses($id_formulaire, $statut_reponses = 'publie'
// on fait des choses seulements si le formulaire existe et qu'il a des enregistrements
if (
$id_formulaire > 0
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire)
and $reponses = sql_allfetsel(
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . $id_formulaire))
&& ($reponses = sql_allfetsel(
'*',
'spip_formulaires_reponses',
'id_formulaire = ' . intval($id_formulaire) . ($statut_reponses == 'publie' ? ' and statut = "publie"' : '')
@ -130,7 +130,7 @@ function obtenir_formulaire_reponses($id_formulaire, $statut_reponses = 'publie'
. (strlen($date_fin) > 0 ? ' and date <= "' . date('Y-m-d H:i:s', strtotime($date_fin . ' + 1 days')) . '"' : ''),
'',
'date'
)
))
) {
return [$formulaire, $reponses];
}
@ -250,7 +250,7 @@ function preparer_formulaire_reponses($formulaire, $reponses, $statut_reponses,
if (!$ignorer_fichiers && $saisie['saisie'] == 'fichiers') {
$_valeurs = formidable_deserialize($valeurs[$nom]);
//tester s'il y a des saisies parmi les fichiers
if (is_array($_valeurs) and $_valeurs) {
if (is_array($_valeurs) && $_valeurs) {
$chemin = _DIR_FICHIERS_FORMIDABLE . 'formulaire_' . $id_formulaire . '/reponse_' . $reponse['id_formulaires_reponse'];
foreach ($_valeurs as $v) {
$chemin_fichier = $chemin . '/' . $saisie['options']['nom'] . '/' . $v['nom'];

@ -38,8 +38,8 @@ function formidable_id_formulaire($id) {
if (
$id_formulaire
and !test_espace_prive()
and !objet_test_si_publie('formulaire', $id_formulaire)
&& !test_espace_prive()
&& !objet_test_si_publie('formulaire', $id_formulaire)
) {
return $id_formulaires[$id] = 0;
}
@ -69,17 +69,17 @@ function formulaires_formidable_saisies_dist($id, $valeurs = [], $options_appel
$saisies = [];
include_spip('formidable_fonctions');
if (
$id_formulaire = formidable_id_formulaire($id)
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . intval($id_formulaire))
($id_formulaire = formidable_id_formulaire($id))
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . intval($id_formulaire)))
) {
$saisies = formidable_deserialize($formulaire['saisies']);
// Si on est en train de réafficher les valeurs postées,
// ne pas afficher les saisies hidden
if (
$formulaire['apres'] == 'valeurs'
and _request('formidable_afficher_apres') == 'valeurs'
and _request('formidable_traiter_ok') == true
$formulaire['apres'] === 'valeurs'
&& _request('formidable_afficher_apres') === 'valeurs'
&& _request('formidable_traiter_ok') == true
) {
$champs_hidden = saisies_lister_avec_type($saisies, 'hidden');
foreach ($champs_hidden as $champ => $desc) {
@ -157,7 +157,7 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel
// Si on a un tableau,
// alors on écrase avec les valeurs données depuis l'appel
if ($valeurs and is_array($valeurs)) {
if ($valeurs && is_array($valeurs)) {
$contexte = array_merge($contexte, $valeurs);
}
}
@ -204,7 +204,7 @@ function formulaires_formidable_charger_dist($id, $valeurs = [], $options_appel
}
$contexte['formidable_afficher_apres'] = $formulaire['apres'];
// Si le formulaire via d'être posté, ne pas preremplir le nouveau formulaire avec les valeurs postées
if ($formulaire['apres'] == 'formulaire' and _request('formidable_traiter_ok')) {
if ($formulaire['apres'] === 'formulaire' && _request('formidable_traiter_ok')) {
foreach (saisies_lister_par_nom(formidable_deserialize($formulaire['saisies'])) as $nom => $valeur) {
set_request($nom, null);
}
@ -249,7 +249,7 @@ function formulaires_formidable_verifier_post_saisies_dist($id, $valeurs = [], $
$saisies = saisies_chercher_formulaire('formidable', [$id, $valeurs, $options_appel, $deprecated_url_redirect, $deprecated_forcer_modif]);
// Si on n'est pas dans un formulaire à étape, on lance les vérifications des traitements
if ($saisies and !saisies_lister_par_etapes($saisies)) {
if ($saisies && !saisies_lister_par_etapes($saisies)) {
$erreurs = formulaires_formidable_verifier_traitements($id, $valeurs, $options_appel);
}
@ -290,7 +290,7 @@ function formulaires_formidable_verifier_etape_post_saisies_dist($etape, $id, $v
$saisies = saisies_chercher_formulaire('formidable', [$id, $valeurs, $options_appel, $deprecated_url_redirect, $deprecated_forcer_modif]);
// On lance les vérifications propres aux traitements à chaque étape, pour avoir les messages d'erreurs à chaque étape
if ($saisies and $etapes = saisies_lister_par_etapes($saisies)) {
if ($saisies && ($etapes = saisies_lister_par_etapes($saisies))) {
$erreurs = formulaires_formidable_verifier_traitements($id, $valeurs, $options_appel, $etapes, $etape);
}
@ -318,10 +318,10 @@ function formulaires_formidable_verifier_traitements($id, $valeurs = [], $option
include_spip('formidable_fonctions');
if (
$id_formulaire = formidable_id_formulaire($id)
and $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . intval($id_formulaire))
and $traitements = formidable_deserialize($formulaire['traitements'])
and is_array($traitements)
($id_formulaire = formidable_id_formulaire($id))
&& ($formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = ' . intval($id_formulaire)))
&& ($traitements = formidable_deserialize($formulaire['traitements']))
&& is_array($traitements)
) {
$saisies = formidable_deserialize($formulaire['saisies']);
@ -348,7 +348,7 @@ function formulaires_formidable_verifier_traitements($id, $valeurs = [], $option
}
}
// Si jamais on raffiche le formulaire, le reafficher à l'étape 0
if ($formulaire['apres'] == 'formulaire' and $etapes) {
if ($formulaire['apres'] === 'formulaire' && $etapes) {
set_request('_etape', '1');
}
return $erreurs;
@ -429,7 +429,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel
// Si on a une redirection valide
if ($url_redirect) {
$retours['redirect'] = $url_redirect;
} elseif (($formulaire['apres'] == 'redirige') and ($formulaire['url_redirect'] != '')) {
} elseif (($formulaire['apres'] === 'redirige') && ($formulaire['url_redirect'] != '')) {
refuser_traiter_formulaire_ajax();
// traiter les raccourcis artX, brX
include_spip('inc/lien');
@ -470,7 +470,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel
// Si on a des traitements
if (is_array($traitements) and !empty($traitements)) {
if (is_array($traitements) && !empty($traitements)) {
$maxiter = 5;
spip_log("Début des traitements pour le formulaire $id_formulaire", 'formidable' . _LOG_INFO);
do {
@ -509,7 +509,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel
}
}
}
} while (count($retours['traitements']) < count($traitements) and $maxiter--);
} while (count($retours['traitements']) < count($traitements) && $maxiter--);
// 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"
@ -568,12 +568,12 @@ 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']) && 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'])) {
if ($formulaire['apres'] === 'formulaire' && isset($retours['fichiers'])) {
$formidable_cvtupload_precharger_fichiers = [];
set_request('_fichiers', null);
set_request('_cvtupload_precharger_fichiers_forcer', true);
@ -612,7 +612,7 @@ function formulaires_formidable_traiter_dist($id, $valeurs = [], $options_appel
set_request('_formidable_cvtupload_precharger_fichiers', $formidable_cvtupload_precharger_fichiers);
}
// si aucun traitement, alerter le webmestre pour ne pas perdre les donnees
if (!$erreur_texte and !count($retours['traitements'])) {
if (!$erreur_texte && !count($retours['traitements'])) {
$erreur_texte = "Aucun traitement pour le formulaire $id\n";
}
if ($erreur_texte) {

@ -16,9 +16,8 @@ function genie_formidable_effacer_enregistrements($t) {
$traitements = formidable_deserialize($result['traitements']);
$id_formulaire = $result['id_formulaire'];
if (
isset($traitements['enregistrement']['effacement'])
and $traitements['enregistrement']['effacement'] == 'on'
and isset($traitements['enregistrement']['effacement_delai'])
($traitements['enregistrement']['effacement'] ?? '') === 'on'
&& isset($traitements['enregistrement']['effacement_delai'])
) {
$delai = intval($traitements['enregistrement']['effacement_delai']);
if ($delai > 0) {

@ -21,19 +21,14 @@ function http_collectionjson_formulaires_reponses_post_collection_dist($requete,
// On teste si on a bien du contenu au bon format
if (
$contenu = $requete->getContent()
and $json = json_decode($contenu, true)
and is_array($json)
and isset($json['collection']['items'][0]['data'])
and $data = $json['collection']['items'][0]['data']
and is_array($data)
($contenu = $requete->getContent())
&& ($json = json_decode($contenu, true))
&& ($data = $json['collection']['items'][0]['data'] ?? '')
&& is_array($data)
) {
// Pour chaque champ envoyé, ça dépend ce que c'est
foreach ($data as $champ) {
if (
isset($champ['name'])
and isset($champ['value'])
) {
if (isset($champ['name']) && isset($champ['value'])) {
// Si c'est id_formulaire, on le garde de côté pour l'autorisation et pour le traitement
if ($champ['name'] == 'id_formulaire') {
$id_formulaire = intval($champ['value']);
@ -44,7 +39,7 @@ function http_collectionjson_formulaires_reponses_post_collection_dist($requete,
}
// On teste l'autorisation
if ($id_formulaire > 0 and autoriser('repondre', 'formulaire', $id_formulaire)) {
if ($id_formulaire > 0 && autoriser('repondre', 'formulaire', $id_formulaire)) {
// On vérifie maintenant les erreurs
$formidable_verifier = charger_fonction('verifier', 'formulaires/formidable');
$erreurs = $formidable_verifier($id_formulaire);

@ -24,7 +24,7 @@ function traitements_lister_disponibles() {
// On ne garde que les traitements qui ont bien la fonction
if (
charger_fonction($type_traitement, 'traiter', true)
and (
&& (
is_array($traitement = traitements_charger_infos($type_traitement))
)
) {
@ -110,24 +110,25 @@ function formidable_generer_nom_cookie($id_formulaire) {
*
*/
function formidable_trouver_reponse_a_editer($id_formulaire, $id_formulaires_reponse, $options, $verifier_est_auteur = false) {
$id_formulaires_reponse = intval($id_formulaires_reponse);
// Si on passe un identifiant de reponse, on edite cette reponse si elle existe
if ($id_formulaires_reponse = intval($id_formulaires_reponse) and ($verifier_est_auteur == false or $options['identification'] == 'id_reponse')) {
if ($id_formulaires_reponse && ($verifier_est_auteur == false || $options['identification'] == 'id_reponse')) {
return $id_formulaires_reponse;
} else {
// calcul des paramètres d'anonymisation
$reponses = formidable_verifier_reponse_formulaire(
$id_formulaire,
isset($options['identification']) ? $options['identification'] : '',
isset($options['variable_php']) ? $options['variable_php'] : ''
$options['identification'] ?? '',
$options['variable_php'] ?? ''
);
//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
($id_formulaires_reponse = intval($id_formulaires_reponse))
&& $verifier_est_auteur == true
) {
if (!is_array($reponses) or in_array($id_formulaires_reponse, $reponses) == false) {
if (!is_array($reponses) || in_array($id_formulaires_reponse, $reponses) == false) {
$id_formulaires_reponse = false;
}
return $id_formulaires_reponse;
@ -136,10 +137,9 @@ 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
and !$options['multiple']
and $options['modifiable']
and is_array($reponses)
!($options['multiple'] ?? '')
&& ($options['modifiable'] ?? '')
&& is_array($reponses)
) {
$id_formulaires_reponse = array_pop($reponses);
}
@ -164,7 +164,7 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific
$variable_php_identification = formidable_variable_php_identification($variable_php_identification, $id_formulaire);
// ni cookie ni id, ni variable_php, on ne peut rien faire
if (!$cookie and !$id_auteur and !$variable_php_identification) {
if (!$cookie && !$id_auteur && !$variable_php_identification) {
return false;
}
@ -188,7 +188,7 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific
}
// 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 ($choix_identification === 'cookie' || !$choix_identification) {
if ($cookie) {
$where = [$where_cookie];
} else {
@ -244,7 +244,7 @@ function formidable_verifier_reponse_formulaire($id_formulaire, $choix_identific
*/
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'])) {
if (!is_array($saisie) || $saisie['saisies'] ?? '') {
return '';
}
@ -264,9 +264,7 @@ function formidable_analyser_saisie($saisie, $valeurs = [], $reponses_total = 0,
// On récupère toutes les valeurs du champ
if (
isset($valeurs[$contexte['nom']])
and $valeurs[$contexte['nom']]
and is_array($valeurs[$contexte['nom']])
is_array($valeurs[$contexte['nom']] ?? '')
) {
$contexte['valeurs'] = $valeurs[$contexte['nom']];
} else {
@ -291,7 +289,7 @@ function formidable_analyser_saisie($saisie, $valeurs = [], $reponses_total = 0,
* @return array Tableau contenant une ligne
*/
function analyser_saisie($saisie) {
if (!isset($saisie['type_saisie']) or $saisie['type_saisie'] == '') {
if (($saisie['type_saisie'] ?? '') === '') {
return '';
}
@ -507,9 +505,9 @@ function formidable_raccourcis_arobases_2_valeurs_champs(string $chaine, array $
$contexte = $options['contexte'] ?? '';
if ($source == 'request') {
if ($source === 'request') {
list($valeurs, $valeurs_libellees) = formidable_tableau_valeurs_saisies($saisies, $sans_reponse);
} elseif ($source == 'base' and $id_formulaires_reponse and $id_formulaire) {
} elseif ($source === 'base' && $id_formulaires_reponse && $id_formulaire) {
$saisies = saisies_lister_par_nom($saisies, false);
foreach ($saisies as $nom => $saisie) {
$valeurs[$nom] = formidable_nettoyer_saisie_vue(saisies_tableau2chaine(calculer_voir_reponse($id_formulaires_reponse, $id_formulaire, $nom, '', 'brut', $sans_reponse)));
@ -618,7 +616,7 @@ function formidable_tableau_valeurs_saisies($saisies, $sans_reponse = true) {
// il peut y avoir eu une modification entre le moment où l'utilisateur a vu le formulaire et maintenant
foreach ($champs as $champ) {
if (array_key_exists($champ, $saisies_fichiers)) {// si on a affaire à une saisie de type fichiers, on considère qu'il n'y pas vraiment de valeur brute
} elseif ($saisies_par_nom[$champ]['saisie'] == 'explication' and isset($saisies_par_nom_apres_afficher_si[$champ])) {
} elseif ($saisies_par_nom[$champ]['saisie'] === 'explication' && isset($saisies_par_nom_apres_afficher_si[$champ])) {
$valeurs[$champ] = $saisies_par_nom_apres_afficher_si[$champ]['options']['texte'];
$valeurs_libellees[$champ] = $valeurs[$champ];
} else {

@ -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 == []) {
if (!is_array($saisies_fichiers) || $saisies_fichiers == []) {
//pas de saisie fichiers?
return '';
}
@ -92,7 +92,7 @@ function formidable_creer_dossier_formulaire($id_formulaire, $forcer = false) {
$data = recuperer_url($url, [
'methode' => 'HEAD'
]);
if ($data and $data['status'] == 200) {
if (($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);
return _T(
@ -211,7 +211,7 @@ function formidable_deplacer_fichier_emplacement_definitif($fichier, $nom, $mime
$data = recuperer_url($url, [
'methode' => 'HEAD'
]);
if ($data and $data['status'] === 200) {
if (($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);

@ -20,7 +20,7 @@ function formidable_associer_forms() {
#var_dump($id);
$res = recherche_en_base("/<{$id}[>|]/", 'article');
#var_dump($res);
if (count($res) and isset($res['article'])) {
if (isset($res['article'])) {
foreach ($res['article'] as $id_article => $details) {
$articles[] = $id_article;
}

@ -10,10 +10,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'])
and $retours['modification_reponse'] == true
and isset($args['options']['modification_reponse'])
and $args['options']['modification_reponse'] == 'on'
($retours['modification_reponse'] ?? '') == true
($args['options']['modification_reponse'] ?? '') == 'on'
) {
$retours['traitements']['email'] = true;
return $retours;
@ -139,7 +137,7 @@ function traiter_email_dist($args, $retours) {
}
}
// Si on a bien des destinataires, on peut continuer
if ($destinataires or ($courriel_envoyeur and $options['activer_accuse'])) {
if ($destinataires || ($courriel_envoyeur && ($options['activer_accuse'] ?? ''))) {
include_spip('inc/filtres');
include_spip('inc/texte');
@ -154,7 +152,7 @@ function traiter_email_dist($args, $retours) {
]
);
}
if (!isset($nom_envoyeur) or !$nom_envoyeur) {
if (!($nom_envoyeur ?? '')) {
$nom_envoyeur = $nom_site_spip;
}
@ -170,11 +168,11 @@ function traiter_email_dist($args, $retours) {
]
);
}
if (!isset($sujet) or !$sujet) {
if (!($sujet ?? '')) {
$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']) {
if (($options['champ_sujet_modif_reponse'] ?? '') && $retours['modification_reponse']) {
$sujet = $sujet . ' ' . _T('formidable:traitement_email_sujet_courriel_modif_reponse');
}
@ -190,8 +188,8 @@ function traiter_email_dist($args, $retours) {
$joindre_pj = false;
if (
$taille_fichiers < 1024 * 1024 * _FORMIDABLE_TAILLE_MAX_FICHIERS_EMAIL
and
$traitements['email']['pj'] == 'on'
&&
$traitements['email']['pj'] === 'on'
) {
$joindre_pj = true;
}
@ -249,7 +247,7 @@ function traiter_email_dist($args, $retours) {
}
// Si besoin, on envoie l'accusé de réception
if ($courriel_envoyeur and $options['activer_accuse']) {
if ($courriel_envoyeur && ($options['activer_accuse'] ?? '')) {
// On récupère le sujet s'il existe sinon on le construit
if ($options['sujet_accuse']) {
$sujet_accuse = formidable_raccourcis_arobases_2_valeurs_champs(
@ -261,7 +259,7 @@ function traiter_email_dist($args, $retours) {
]
);
}
if (!isset($sujet_accuse) or !$sujet_accuse) {
if (!($sujet_accuse ?? '')) {
$sujet_accuse = _T('formidable:traiter_email_sujet_accuse');
}
$sujet_accuse = filtrer_entites($sujet_accuse);
@ -331,7 +329,7 @@ function traiter_email_dist($args, $retours) {
}
//Et on teste si on doit mettre cela en from ou en reply-to
if ($options['activer_vrai_envoyeur'] and $courriel_envoyeur_accuse) {
if (($options['activer_vrai_envoyeur'] ?? '') && $courriel_envoyeur_accuse) {
$courriel_from_accuse = $courriel_envoyeur_accuse;
} elseif ($courriel_envoyeur_accuse) {
$corps['repondre_a'] = $courriel_envoyeur_accuse;
@ -339,7 +337,7 @@ function traiter_email_dist($args, $retours) {
}
// Joindre les pj si léger et nécessaire
if ($joindre_pj and _FORMIDABLE_LIENS_FICHIERS_ACCUSE_RECEPTION == false) {
if ($joindre_pj && _FORMIDABLE_LIENS_FICHIERS_ACCUSE_RECEPTION == false) {
$corps['pieces_jointes'] = $fichiers_facteur;
}
@ -477,8 +475,8 @@ function traiter_email_fichiers($saisie, $nom, $id_formulaire, $retours, $timest
//Initialisation
$id_formulaire = strval($id_formulaire);//précaution
$vue = [];
if (isset($retours['id_formulaires_reponse']) and $id_formulaires_reponse = $retours['id_formulaires_reponse']) { // cas simple: les réponses ont été enregistrées
$id_formulaires_reponse = $retours['id_formulaires_reponse'] ?? '';
if ($id_formulaires_reponse) { // cas simple: les réponses ont été enregistrées
if (isset($retours['fichiers'][$nom])) { // petite précaution
$options = [
'id_formulaire' => $id_formulaire,

@ -20,8 +20,8 @@ function traiter_enregistrement_dist($args, $retours) {
$GLOBALS['visiteur_session']['id_auteur'] : 0) : 0;
// 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['anonymiser'] === 'on' && $id_auteur) {
if ($options['identification'] === 'id_auteur') {
$variable_php = formidable_hasher_id_auteur($id_auteur);
}
$id_auteur = 0;
@ -51,15 +51,15 @@ function traiter_enregistrement_dist($args, $retours) {
// 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'
or (autoriser(
$options['moderation'] === 'posteriori'
|| (autoriser(
'instituer',
'formulairesreponse',
$id_formulaires_reponse,
null,
['id_formulaire' => $id_formulaire, 'nouveau_statut' => 'publie']
)
and !(isset($options['moderer_admins']) and $options['moderer_admins'] == 'on')
&& !(($options['moderer_admins'] ?? '') === 'on')
)
) {
$statut = 'publie';
@ -86,7 +86,7 @@ function traiter_enregistrement_dist($args, $retours) {
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']) {
if (!$options['multiple'] || $options['modifiable']) {
include_spip('inc/cookie');
// Expiration dans 30 jours
spip_setcookie($nom_cookie, $_COOKIE[$nom_cookie] = $cookie, time() + 30 * 24 * 3600);
@ -129,8 +129,7 @@ function traiter_enregistrement_dist($args, $retours) {
];
$retours['fichiers'][$nom] = $valeur;
}
}
elseif (($valeur = _request($nom)) !== null or saisies_saisie_est_tabulaire($saisie)) {
} elseif (($valeur = _request($nom)) !== null || saisies_saisie_est_tabulaire($saisie)) {
// Pour le saisies différentes de fichiers,
// on ne prend que les champs qui ont effectivement été envoyés par le formulaire
$champs[] = $nom;
@ -170,9 +169,8 @@ function traiter_enregistrement_dist($args, $retours) {
//Invalider le cache le cas échéant
if (
isset($options['invalider'])
and $options['invalider']
and $statut == 'pub'
($options['invalider'] ?? '')
&& $statut === 'pub'
) {
include_spip('inc/invalideur');
suivre_invalideur("formulaires_reponse/$id_formulaires_reponse");
@ -190,11 +188,11 @@ function traiter_enregistrement_verifier_dist($args, $erreurs) {
$id_formulaires_reponse = $args['id_formulaires_reponse'];
$etape = $args['etape'];
if (
($unicite = $options['unicite'] ?? '') and !($erreurs[$unicite] ?? '')
and
($unicite = $options['unicite'] ?? '') && !($erreurs[$unicite] ?? '')
&&
(
$etape === null
or (
|| (
array_key_exists($unicite, saisies_lister_par_nom($args['etapes']['etape_' . $etape]['saisies']))
)
)

Loading…
Cancel
Save