style: un coup de phpcs

.gitignore

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

formidable_mailsubscribers_pipelines.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Plugin mailsubscribers
  *
@@ -36,7 +37,7 @@ function formidable_mailsubscribers_formulaire_charger($flux) {
 	if (isset($flux['data']['_formidable']['traitements'])) {
 		$traitements = function_exists('formidable_deserialize') ? formidable_deserialize($flux['data']['_formidable']['traitements']) : unserialize($flux['data']['_formidable']['traitements']);
 	} else {
-		$traitements = array();
+		$traitements = [];
 	}
 
 	if (
@@ -45,7 +46,7 @@ function formidable_mailsubscribers_formulaire_charger($flux) {
 		and in_array('mailsubscribe', array_keys($traitements))
 		and !(isset($flux['args']['args'][2]) and $flux['args']['args'][2])// argument 2 dans le tableau, 3 de l'appel à #FORMIDABLE_CHARGER, c'est la réponse qu'on précharge, dans ce cas garder les valeurs de la réponse, et ne pas prendre en session
 	) {
-		foreach (array('email', 'nom') as $champ) {
+		foreach (['email', 'nom'] as $champ) {
 			$nom_champ = (isset($traitements['mailsubscribe']["champ_${champ}_mailsubscribe"]) ? $traitements['mailsubscribe']["champ_${champ}_mailsubscribe"] : null);
 			if (
 				isset($GLOBALS['visiteur_session'][$champ])

phpcs.xml.dist

@@ -0,0 +1,15 @@
+<?xml version="1.0"?>
+<ruleset>
+    <file>.</file>
+    <exclude-pattern>vendor/*</exclude-pattern>
+    <exclude-pattern>lang/*</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>
+

traiter/mailsubscribe.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Traitement abonnement à des listes de diffusion à la saisie d'un formulaire
  *
@@ -55,22 +56,21 @@ function traiter_mailsubscribe_dist($args, $retours) {
 		$email = _request($champ_email)
 		and $listes = _request($champ_listes)
 	) {
-
 		include_spip('inc/config');
 		include_spip('inc/nospam');
 		include_spip('inc/filtres');
 		include_spip('base/abstract_sql');
 		include_spip('inc/mailsubscribers');
 
-		$options_subscribe = array();
+		$options_subscribe = [];
 
 		// Possibilité d'avoir un champs_liste qui renvoie juste un string, si par on propose un choix "je m'inscrit à la liste/je ne m'inscrit pas à la liste
 		if (!is_array($listes)) {
-			$listes = array($listes);
+			$listes = [$listes];
 		}
 
 		// S'assurer que les listes choisies sont ouvertes
-		$listes_dispo = mailsubscribers_listes(array('status' => 'open'));
+		$listes_dispo = mailsubscribers_listes(['status' => 'open']);
 		$listes = array_intersect($listes, array_keys($listes_dispo));
 
 		// Si la personne n'a choisi que des listes fermées, on n'inscrit à aucune liste, donc on n'inscrit nul part
@@ -95,9 +95,9 @@ function traiter_mailsubscribe_dist($args, $retours) {
 
 		// Message de retour en fonction du double opt-in
 		if (lire_config('mailsubscribers/double_optin', 0)) {
-			$message_ok = _T('newsletter:subscribe_message_ok_confirm', array('email' => "<b>$email</b>"));
+			$message_ok = _T('newsletter:subscribe_message_ok_confirm', ['email' => "<b>$email</b>"]);
 		} else {
-			$message_ok = _T('newsletter:subscribe_message_ok', array('email' => "<b>$email</b>"));
+			$message_ok = _T('newsletter:subscribe_message_ok', ['email' => "<b>$email</b>"]);
 		}
 
 		// Une constante pour déclencher l'action après le post, systématiquement
@@ -106,11 +106,12 @@ function traiter_mailsubscribe_dist($args, $retours) {
 		}
 
 		// Si nospam, ne déclencher l'action qu'après le POST
-		if (function_exists('nospam_confirm_action_html')
+		if (
+			function_exists('nospam_confirm_action_html')
 			and !(isset($retours['redirect']) and $retours['redirect'])
 			and _FORMIDABLE_MAILSUBSCRIBERS_NO_SPAM_CONFIRM_HTML
 		) {
-			$message_ok .= ' '.nospam_confirm_action_html('subscribe', "Subscribe $email " . json_encode($options_subscribe), array($email, $options_subscribe), 'newsletter/');
+			$message_ok .= ' ' . nospam_confirm_action_html('subscribe', "Subscribe $email " . json_encode($options_subscribe), [$email, $options_subscribe], 'newsletter/');
 		// Soit go go go direct
 		} else {
 			$newsletter_subscribe = charger_fonction('subscribe', 'newsletter');