Changeset 93640 in spip-zone

Timestamp:

Dec 13, 2015, 2:39:43 PM (5 years ago)

Author:

marcimat@…

Message:

Meilleure compatibilité avec PSR-2 et nos règles d'écriture, en appliquant
différents fix avec php-cs-fixers. Fixers appliqués ici :

'encoding', utf8
'eof_ending', un saut de ligne en fin de fichier
'elseif', elseif plutôt que else if
'function_call_space', espaces sur fonctions
'function_declaration', espaces sur fonctions
'function_typehint_space', espaces sur fonctions
'linefeed', sauts de ligne \n uniquement
'lowercase_constants', true, false, null en munuscule
'lowercase_keywords', mots clés PHP en lowercase
'method_argument_space', espaces sur appels de fonctions
'multiple_use', use unique sur fonctions anonymes
'newline_after_open_tag', ouverture de php… et c'est tout sur cette ligne
'operators_spaces', espaces de part et d'autres des opérateurs binaires
'parenthesis', pas d'espace juste après parenthèse ouvrante, ou avant parenthèse fermante
'php_closing_tag', pas de fermeture de php
'short_tag', tag PHP corrects
'trailing_spaces', pas d'espace qui traîne en fin de ligne
'visibility', déclarer 'public / private / protected' sur les méthodes

Location:

_core_/plugins/forum

Files:

• action/editer_forum.php (modified) (3 diffs)
• action/instituer_forum.php (modified) (3 diffs)
• action/instituer_lot_forum.php (modified) (3 diffs)
• balise/formulaire_forum.php (modified) (3 diffs)
• balise/formulaire_forum_prive.php (modified) (1 diff)
• base/forum.php (modified) (1 diff)
• formulaires/activer_forums_objet.php (modified) (4 diffs)
• formulaires/configurer_forums_contenu.php (modified) (2 diffs)
• formulaires/configurer_forums_notifications.php (modified) (2 diffs)
• formulaires/configurer_forums_participants.php (modified) (2 diffs)
• formulaires/configurer_forums_prives.php (modified) (1 diff)
• formulaires/forum.php (modified) (15 diffs)
• formulaires/forum_prive.php (modified) (6 diffs)
• forum_administrations.php (modified) (2 diffs)
• forum_autoriser.php (modified) (3 diffs)
• forum_fonctions.php (modified) (1 diff)
• forum_ieconfig.php (modified) (1 diff)
• forum_pipelines.php (modified) (11 diffs)
• inc/article_accepter_forums_publics.php (modified) (1 diff)
• inc/email_notification_forum.php (modified) (2 diffs)
• inc/forum.php (modified) (4 diffs)
• inc/forum_insert.php (modified) (4 diffs)
• notifications/forumposte.php (modified) (5 diffs)
• notifications/forumvalide.php (modified) (4 diffs)
• prive/modeles/forum_fonctions.php (modified) (1 diff)
• public/forum.php (modified) (8 diffs)
• urls/generer_url_ecrire_forum.php (modified) (1 diff)
• urls/generer_url_forum.php (modified) (2 diffs)

_core_/plugins/forum/action/editer_forum.php

@@ -53 +53 @@
                 $cles = array();
                 foreach (array('id_objet', 'objet') as $k) {
-                        if (isset($c[$k]) AND $c[$k]) {
+                        if (isset($c[$k]) and $c[$k]) {
                                 $cles[$k] = $c[$k];
                         }
@@ -61 +61 @@
                 // (non autorise en standard mais utile pour des crayons)
                 // on deplace tout le thread {sauf les originaux}.
-                if (count($cles) AND $id_thread) {
+                if (count($cles) and $id_thread) {
                         spip_log("update thread id_thread=$id_thread avec " . var_export($cles, 1), 'forum.' . _LOG_INFO_IMPORTANTE);
                         sql_updateq("spip_forum", $cles, "id_thread=" . $id_thread . " AND statut!='original'");
@@ -69 +69 @@
                 // s'il y a vraiment eu une modif et que le message est public
                 // on enregistre le nouveau date_thread
-                if ($err === '' AND $t['statut'] == 'publie') {
+                if ($err === '' and $t['statut'] == 'publie') {
                         // on ne stocke ni le numero IP courant ni le nouvel id_auteur
                         // dans le message modifie (trop penible a l'usage) ; mais du

_core_/plugins/forum/action/instituer_forum.php

@@ -51 +51 @@
 
         // Notifier de la publication du message, s'il etait 'prop'
-        if ($old == 'prop' AND $statut == 'publie') {
+        if ($old == 'prop' and $statut == 'publie') {
                 if ($notifications = charger_fonction('notifications', 'inc')) {
                         $notifications('forumvalide', $id_forum);
@@ -64 +64 @@
         // car la date_thread aurait cette derniere date alors que pas le message
         // mais c'est au mieux de ce que l'on peut faire quand on depublie un SPAM ou supprime un message
-        if ($statut == 'publie' OR $old == 'publie') {
+        if ($statut == 'publie' or $old == 'publie') {
                 if ($statut == 'publie'
-                        OR !($date_thread = sql_getfetsel("date_heure", "spip_forum",
+                        or !($date_thread = sql_getfetsel("date_heure", "spip_forum",
                                 "statut='publie' AND id_thread=" . $row['id_thread'], "", "date_heure DESC", "0,1"))
                 ) {
@@ -95 +95 @@
         );
 }
-
-?>

_core_/plugins/forum/action/instituer_lot_forum.php

@@ -32 +32 @@
                  */
                 if (preg_match(",^(\w+)-,", $arg, $match)
-                        AND in_array($statut = $match[1], array('publie', 'off', 'spam'))
+                        and in_array($statut = $match[1], array('publie', 'off', 'spam'))
                 ) {
-                        $arg = substr($arg, strlen($statut)+1);
+                        $arg = substr($arg, strlen($statut) + 1);
 
                         $arg = explode('/', $arg);
@@ -71 +71 @@
                  */
                 elseif (preg_match(",^(\w+)$,", $arg, $match)
-                        AND in_array($statut = $match[1], array('publie', 'off', 'spam'))
-                        AND $id = _request('ids')
-                        AND is_array($id)
+                        and in_array($statut = $match[1], array('publie', 'off', 'spam'))
+                        and $id = _request('ids')
+                        and is_array($id)
                 ) {
 
@@ -96 +96 @@
 
 }
-
-?>

_core_/plugins/forum/balise/formulaire_forum.php

@@ -206 +206 @@
 
         // et si on n'a toujours pas ce qu'on souhaite, on tente de le trouver dans un forum existant...
-        if (($objet == 'forum' OR !$id_objet) and $id_forum) {
+        if (($objet == 'forum' or !$id_objet) and $id_forum) {
                 if ($objet = sql_fetsel(array('id_objet', 'objet'), 'spip_forum', 'id_forum=' . intval($id_forum))) {
                         $id_objet = $objet['id_objet'];
@@ -217 +217 @@
         }
         // vraiment la... faut pas exagerer !
-        if ($objet_obligatoire AND !$id_objet) {
+        if ($objet_obligatoire and !$id_objet) {
                 return false;
         }
@@ -223 +223 @@
         return array($objet, $id_objet, $url);
 }
-
-?>

_core_/plugins/forum/balise/formulaire_forum_prive.php

@@ -126 +126 @@
                 array($objet, $id_objet, $id_forum, $forcer_previsu, $statut, $retour);
 }
-
-?>

_core_/plugins/forum/base/forum.php

@@ -148 +148 @@
         return $tables;
 }
-
-?>

_core_/plugins/forum/formulaires/activer_forums_objet.php

@@ -31 +31 @@
 function get_forums_publics($id_objet = 0, $objet = 'article') {
 
-        if ($objet == 'article' AND $id_objet) {
+        if ($objet == 'article' and $id_objet) {
                 $obj = sql_fetsel("accepter_forum", "spip_articles", "id_article=" . intval($id_objet));
 
@@ -62 +62 @@
                 "objet=" . sql_quote($objet) . " AND id_objet=" . intval($id_objet) . " AND statut IN ('publie', 'off', 'prop', 'spam')");
         $editable = ($objet == 'article') ? true : false;
-        if (!$editable AND !$nb_forums) {
+        if (!$editable and !$nb_forums) {
                 return false;
         }
@@ -86 +86 @@
 function formulaires_activer_forums_objet_traiter_dist($id_objet, $objet = 'article') {
         include_spip('inc/autoriser');
-        if ($objet == 'article' AND autoriser('modererforum', $objet, $id_objet)) {
+        if ($objet == 'article' and autoriser('modererforum', $objet, $id_objet)) {
                 $statut = _request('accepter_forum');
                 include_spip('base/abstract_sql');
@@ -100 +100 @@
         return array('message_ok' => _T('config_info_enregistree'), 'editable' => true);
 }
-
-?>

_core_/plugins/forum/formulaires/configurer_forums_contenu.php

@@ -31 +31 @@
         $erreurs = array();
 
-        if (!_request('forums_titre') AND !_request('forums_texte') AND !_request('forums_urlref')) {
+        if (!_request('forums_titre') and !_request('forums_texte') and !_request('forums_urlref')) {
                 $erreurs['forums_titre'] = _T('info_obligatoire');
         }
@@ -56 +56 @@
         return array('message_ok' => _T('config_info_enregistree'));
 }
-
-?>

_core_/plugins/forum/formulaires/configurer_forums_notifications.php

@@ -25 +25 @@
         unset($l['forum:info_pas_de_forum']);
         foreach ($l as $desc => $val) {
-                $valeurs['prevenir_auteurs_' . $val] = (($m == 'oui') OR strpos($m, ",$val,") !== false);
+                $valeurs['prevenir_auteurs_' . $val] = (($m == 'oui') or strpos($m, ",$val,") !== false);
         }
 
@@ -44 +44 @@
         return array('message_ok' => _T('config_info_enregistree'));
 }
-
-?>

_core_/plugins/forum/formulaires/configurer_forums_participants.php

@@ -29 +29 @@
         $purger_skel = false;
         if ($accepter_forum = _request('forums_publics')
-                AND ($accepter_forum != $GLOBALS['meta']["forums_publics"])
+                and ($accepter_forum != $GLOBALS['meta']["forums_publics"])
         ) {
                 $purger_skel = true;
@@ -59 +59 @@
         return array('message_ok' => _T('config_info_enregistree'));
 }
-
-?>

_core_/plugins/forum/formulaires/configurer_forums_prives.php

@@ -42 +42 @@
         return $res;
 }
-
-?>

_core_/plugins/forum/formulaires/forum.php

@@ -93 +93 @@
         // exiger l'authentification des posteurs pour les forums sur abo
         if ($accepter_forum == "abo") {
-                if (!isset($GLOBALS["visiteur_session"]['statut']) OR !$GLOBALS["visiteur_session"]['statut']) {
+                if (!isset($GLOBALS["visiteur_session"]['statut']) or !$GLOBALS["visiteur_session"]['statut']) {
                         return array(
                                 'action' => '', #ne sert pas dans ce cas, on la vide pour mutualiser le cache
@@ -118 +118 @@
 
         // par défaut, on force la prévisualisation du message avant de le poster
-        if (($forcer_previsu == 'non') OR (empty($forcer_previsu) AND $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
+        if (($forcer_previsu == 'non') or (empty($forcer_previsu) and $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
                 $forcer_previsu = 'non';
         } else {
@@ -196 +196 @@
 function forum_fichier_tmp($arg) {
 # astuce : mt_rand pour autoriser les hits simultanes
-        while (($alea = time()+@mt_rand())+intval($arg)
-                AND @file_exists($f = _DIR_TMP . "forum_$alea.lck")) {
+        while (($alea = time() + @mt_rand()) + intval($arg)
+                and @file_exists($f = _DIR_TMP . "forum_$alea.lck")) {
         };
         spip_touch($f);
@@ -206 +206 @@
                 while (($file = @readdir($dh)) !== false) {
                         if (preg_match('/^forum_([0-9]+)\.lck$/', $file)
-                                AND (time()-@filemtime(_DIR_TMP . $file) > 4*3600)
+                                and (time() - @filemtime(_DIR_TMP . $file) > 4 * 3600)
                         ) {
                                 spip_unlink(_DIR_TMP . $file);
@@ -247 +247 @@
 
         // par défaut, on force la prévisualisation du message avant de le poster
-        if (($forcer_previsu == 'non') OR (empty($forcer_previsu) AND $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
+        if (($forcer_previsu == 'non') or (empty($forcer_previsu) and $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
                 $forcer_previsu = 'non';
         } else {
@@ -264 +264 @@
 
         if (isset($_FILES['ajouter_document'])
-                AND $_FILES['ajouter_document']['tmp_name']
+                and $_FILES['ajouter_document']['tmp_name']
         ) {
 
@@ -274 +274 @@
                         // verifier si on possede la cle (ie on est autorise a poster)
                         // (sinon tant pis) ; cf. charger.php pour la definition de la cle
-                        OR _request('cle_ajouter_document') != calculer_cle_action($a = "ajouter-document-$objet-$id_objet")
+                        or _request('cle_ajouter_document') != calculer_cle_action($a = "ajouter-document-$objet-$id_objet")
                 ) {
                         $erreurs['document_forum'] = _T('forum:documents_interdits_forum');
@@ -306 +306 @@
                         if (isset($erreurs['document_forum'])) {
                                 spip_unlink($tmp . '.bin');
-                                unset ($_FILES['ajouter_document']);
+                                unset($_FILES['ajouter_document']);
                         } else {
                                 $doc['tmp_name'] = $tmp . '.bin';
@@ -314 +314 @@
         } // restaurer/supprimer le document eventuellement uploade au tour precedent
         elseif (isset($GLOBALS['visiteur_session']['tmp_forum_document'])
-                AND $tmp = $GLOBALS['visiteur_session']['tmp_forum_document']
-                AND file_exists($tmp . '.bin')
+                and $tmp = $GLOBALS['visiteur_session']['tmp_forum_document']
+                and file_exists($tmp . '.bin')
         ) {
                 if (_request('supprimer_document_ajoute')) {
@@ -328 +328 @@
         $min_length = (defined('_FORUM_LONGUEUR_MINI') ? _FORUM_LONGUEUR_MINI : 10);
         if (strlen($texte = _request('texte')) < $min_length
-                AND !$ajouter_mot AND $GLOBALS['meta']['forums_texte'] == 'oui'
+                and !$ajouter_mot and $GLOBALS['meta']['forums_texte'] == 'oui'
         ) {
                 $erreurs['texte'] = _T($min_length == 10 ? 'forum:forum_attention_dix_caracteres' : 'forum:forum_attention_nb_caracteres_mini',
                         array('min' => $min_length));
         } elseif (defined('_FORUM_LONGUEUR_MAXI')
-                AND _FORUM_LONGUEUR_MAXI > 0
-                AND strlen($texte) > _FORUM_LONGUEUR_MAXI
+                and _FORUM_LONGUEUR_MAXI > 0
+                and strlen($texte) > _FORUM_LONGUEUR_MAXI
         ) {
                 $erreurs['texte'] = _T('forum:forum_attention_trop_caracteres',
@@ -343 +343 @@
         }
 
-        if (array_reduce($_POST, 'reduce_strlen', (20*1024)) < 0) {
+        if (array_reduce($_POST, 'reduce_strlen', (20 * 1024)) < 0) {
                 $erreurs['erreur_message'] = _T('forum:forum_message_trop_long');
         } else {
                 // Ne pas autoriser d'envoi hacke si forum sur abonnement
                 if (controler_forum($objet, $id_objet) == 'abo'
-                        AND !test_espace_prive()
+                        and !test_espace_prive()
                 ) {
                         if (!isset($GLOBALS['visiteur_session'])
-                                OR !isset($GLOBALS['visiteur_session']['statut'])
+                                or !isset($GLOBALS['visiteur_session']['statut'])
                         ) {
                                 $erreurs['erreur_message'] = _T('forum_non_inscrit');
@@ -361 +361 @@
 
         if (strlen($titre = _request('titre')) < 3
-                AND $GLOBALS['meta']['forums_titre'] == 'oui'
+                and $GLOBALS['meta']['forums_titre'] == 'oui'
         ) {
                 $erreurs['titre'] = _T('forum:forum_attention_trois_caracteres');
         }
 
-        if (!count($erreurs) AND !_request('confirmer_previsu_forum')) {
-                if (!_request('envoyer_message') OR $forcer_previsu <> 'non') {
+        if (!count($erreurs) and !_request('confirmer_previsu_forum')) {
+                if (!_request('envoyer_message') or $forcer_previsu <> 'non') {
                         $previsu = inclure_previsu($texte, $titre, _request('url_site'), _request('nom_site'), _request('ajouter_mot'),
                                 $doc,
@@ -379 +379 @@
         if (!count($erreurs)) {
                 if (!test_espace_prive()
-                        AND $forcer_previsu <> 'non'
-                        AND forum_insert_noprevisu()
+                        and $forcer_previsu <> 'non'
+                        and forum_insert_noprevisu()
                 ) {
                         $erreurs['erreur_message'] = _T('forum:forum_acces_refuse');
@@ -525 +525 @@
 
         if (defined('_FORUM_AUTORISER_POST_ID_FORUM')
-                AND _FORUM_AUTORISER_POST_ID_FORUM
-                AND _request('id_forum')
+                and _FORUM_AUTORISER_POST_ID_FORUM
+                and _request('id_forum')
         ) {
                 $id_forum = _request('id_forum');
@@ -563 +563 @@
         return $res;
 }
-
-
-?>

_core_/plugins/forum/formulaires/forum_prive.php

@@ -37 +37 @@
         }
         // si objet, il faut un titre, sinon on est dans un statut privrac/privadm qui permet un forum sans objet
-        if ($objet AND $id_objet AND !$titre = $f($objet, $id_objet, $id_forum, false)) {
+        if ($objet and $id_objet and !$titre = $f($objet, $id_objet, $id_forum, false)) {
                 return false;
         }
@@ -57 +57 @@
 
         // par défaut, on force la prévisualisation du message avant de le poster
-        if (($forcer_previsu == 'non') OR (empty($forcer_previsu) AND $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
+        if (($forcer_previsu == 'non') or (empty($forcer_previsu) and $GLOBALS['meta']["forums_forcer_previsu"] == "non")) {
                 $forcer_previsu = 'non';
         } else {
@@ -101 +101 @@
         $min_length = (defined('_FORUM_LONGUEUR_MINI') ? _FORUM_LONGUEUR_MINI : 10);
         if (strlen($texte = _request('texte')) < $min_length
-                AND !_request('ajouter_mot') AND $GLOBALS['meta']['forums_texte'] == 'oui'
+                and !_request('ajouter_mot') and $GLOBALS['meta']['forums_texte'] == 'oui'
         ) {
                 $erreurs['texte'] = _T($min_length == 10 ? 'forum:forum_attention_dix_caracteres' : 'forum:forum_attention_nb_caracteres_mini',
@@ -107 +107 @@
         } else {
                 if (defined('_FORUM_LONGUEUR_MAXI')
-                        AND _FORUM_LONGUEUR_MAXI > 0
-                        AND strlen($texte) > _FORUM_LONGUEUR_MAXI
+                        and _FORUM_LONGUEUR_MAXI > 0
+                        and strlen($texte) > _FORUM_LONGUEUR_MAXI
                 ) {
                         $erreurs['texte'] = _T('forum:forum_attention_trop_caracteres',
@@ -119 +119 @@
 
         if (strlen($titre = _request('titre')) < 3
-                AND $GLOBALS['meta']['forums_titre'] == 'oui'
+                and $GLOBALS['meta']['forums_titre'] == 'oui'
         ) {
                 $erreurs['titre'] = _T('forum:forum_attention_trois_caracteres');
         }
 
-        if (array_reduce($_POST, 'reduce_strlen', (20*1024)) < 0) {
+        if (array_reduce($_POST, 'reduce_strlen', (20 * 1024)) < 0) {
                 $erreurs['erreur_message'] = _T('forum:forum_message_trop_long');
         }
 
-        if (!count($erreurs) AND !_request('envoyer_message') AND !_request('confirmer_previsu_forum')) {
+        if (!count($erreurs) and !_request('envoyer_message') and !_request('confirmer_previsu_forum')) {
                 $previsu = inclure_forum_prive_previsu($texte, $titre, _request('url_site'), _request('nom_site'),
                         _request('ajouter_mot'));
@@ -200 +200 @@
         return $res;
 }
-
-
-?>

_core_/plugins/forum/forum_administrations.php

@@ -37 +37 @@
                 $trouver_table(''); // vider le cache des descriptions !
                 if ($desc = $trouver_table('spip_forum')
-                        AND isset($desc['field']['id_article'])
+                        and isset($desc['field']['id_article'])
                 ) {
                         ecrire_meta($nom_meta_base_version, '1.0.0');
@@ -118 +118 @@
         effacer_meta($nom_meta_base_version);
 }
-
-?>

_core_/plugins/forum/forum_autoriser.php

@@ -39 +39 @@
 function autoriser_foruminternesuivi_menu_dist($faire, $type = '', $id = 0, $qui = null, $opt = null) {
         if ((($GLOBALS['meta']['forum_prive'] == 'non') && ($GLOBALS['meta']['forum_prive_admin'] == 'non'))
-                OR (($GLOBALS['meta']['forum_prive'] == 'non') && ($qui['statut'] == '1comite'))
+                or (($GLOBALS['meta']['forum_prive'] == 'non') && ($qui['statut'] == '1comite'))
         ) {
                 return false;
@@ -123 +123 @@
                 include_spip('inc/forum');
                 if ($racine = racine_forum($id)
-                        AND list($objet, $id_objet,) = $racine
-                        AND $objet
+                        and list($objet, $id_objet, ) = $racine
+                        and $objet
                 ) {
                         return autoriser('modererforum', $objet, $id_objet);
@@ -247 +247 @@
         return true;
 }
-
-?>

_core_/plugins/forum/forum_fonctions.php

@@ -96 +96 @@
         return $texte;
 }
-
-?>

_core_/plugins/forum/forum_ieconfig.php

@@ -37 +37 @@
         return $table;
 }
-
-?>

_core_/plugins/forum/forum_pipelines.php

@@ -45 +45 @@
                         $texte .= "\n<br />" . $lien;
                 }
-                if (strlen($texte) AND $GLOBALS['meta']['forum_prive_objets'] != 'non') {
+                if (strlen($texte) and $GLOBALS['meta']['forum_prive_objets'] != 'non') {
                         $cpt2 = sql_countsel("spip_articles", "statut='prop'");
                         if ($cpt2) {
@@ -118 +118 @@
 function forum_afficher_fiche_objet($flux) {
         if ($type = $flux['args']['type']
-                AND $table_sql = table_objet_sql($type)
-                AND in_array($table_sql, explode(',', $GLOBALS['meta']['forum_prive_objets']))
+                and $table_sql = table_objet_sql($type)
+                and in_array($table_sql, explode(',', $GLOBALS['meta']['forum_prive_objets']))
         ) {
                 $id = $flux['args']['id'];
@@ -135 +135 @@
                 $id_rubrique = $flux['args']['id'];
                 if (autoriser('publierdans', 'rubrique', $id_rubrique)
-                        AND !sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique))
+                        and !sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique))
                 ) {
                         include_spip('inc/forum');
@@ -161 +161 @@
 function forum_afficher_config_objet($flux) {
         if (($type = $flux['args']['type'])
-                AND $id = $flux['args']['id']
+                and $id = $flux['args']['id']
         ) {
                 if (autoriser('modererforum', $type, $id)) {
@@ -183 +183 @@
                 $statut = $flux['args']['statut'];
                 if ($GLOBALS['meta']['forum_prive_objets'] != 'non'
-                        AND $statut == 'prop'
+                        and $statut == 'prop'
                 ) {
                         $flux['data'] .= "<p class='article_prop'>" . _T('forum:text_article_propose_publication_forum') . '</p>';
@@ -200 +200 @@
 function forum_boite_infos($flux) {
         if ($flux['args']['type'] == 'rubrique'
-                AND $id_rubrique = $flux['args']['id']
+                and $id_rubrique = $flux['args']['id']
         ) {
                 if (autoriser('publierdans', 'rubrique', $id_rubrique)
                         // [doc] d'ou il vient ce row ?
-                        AND (!isset($flux['args']['row']['id_parent']) OR !$flux['args']['row']['id_parent'])
+                        and (!isset($flux['args']['row']['id_parent']) or !$flux['args']['row']['id_parent'])
                 ) {
                         include_spip('inc/forum');
@@ -222 +222 @@
                 }
         } elseif ($flux['args']['type'] == 'auteur'
-                AND $id_auteur = $flux['args']['id']
+                and $id_auteur = $flux['args']['id']
         ) {
                 if ($nb = sql_countsel('spip_forum', "statut!='poubelle' AND id_auteur=" . intval($id_auteur))) {
@@ -338 +338 @@
         ## definir a 0 pour desactiver
         if (!defined('_CNIL_PERIODE')) {
-                define('_CNIL_PERIODE', 3600*24*31*4);
+                define('_CNIL_PERIODE', 3600 * 24 * 31 * 4);
         }
         if (_CNIL_PERIODE) {
-                $critere_cnil = 'date_heure<"' . date('Y-m-d', time()-_CNIL_PERIODE) . '"'
+                $critere_cnil = 'date_heure<"' . date('Y-m-d', time() - _CNIL_PERIODE) . '"'
                         . ' AND statut != "spam"'
                         . ' AND (ip LIKE "%.%" OR ip LIKE "%:%")'; # ipv4 ou ipv6
@@ -382 +382 @@
         # Pour les forums, unifier par id_thread et forcer statut='publie'
         if ($flux['args']['type'] == 'forum'
-                AND $points = $flux['data']
+                and $points = $flux['data']
         ) {
                 $serveur = $flux['args']['serveur'];
@@ -414 +414 @@
 function forum_rubrique_encours($flux) {
         if (strlen($flux['data'])
-                AND $GLOBALS['meta']['forum_prive_objets'] != 'non'
+                and $GLOBALS['meta']['forum_prive_objets'] != 'non'
         ) {
                 $flux['data'] = _T('forum:texte_en_cours_validation_forum') . $flux['data'];
@@ -440 +440 @@
         return $objets;
 }
-
-?>

_core_/plugins/forum/inc/article_accepter_forums_publics.php

@@ -22 +22 @@
         return substr($accepter_forum, 0, 3);
 }
-
-
-?>

_core_/plugins/forum/inc/email_notification_forum.php

@@ -77 +77 @@
                         $traitement = (isset($GLOBALS['table_des_traitements'][$champ]) ? $GLOBALS['table_des_traitements'][$champ] : null);
                         if (is_array($traitement)
-                                AND (isset($traitement[$table_objet]) OR isset($traitement[0]))
+                                and (isset($traitement[$table_objet]) or isset($traitement[0]))
                         ) {
                                 $traitement = $traitement[isset($traitement[$table_objet]) ? $table_objet : 0];
@@ -84 +84 @@
                         }
 
-                        $links = $links+extraire_balises($v, 'a');
+                        $links = $links + extraire_balises($v, 'a');
                 }
                 $links = extraire_attribut($links, 'href');

_core_/plugins/forum/inc/forum.php

@@ -131 +131 @@
 
         if ($row['id_parent']
-                AND $row['id_thread'] != $id_forum
+                and $row['id_thread'] != $id_forum
         ) // eviter boucle infinie
         {
@@ -244 +244 @@
         } else {
                 include_spip('base/objets');
-                if ($publie AND !objet_test_si_publie($objet, $id_objet)) {
+                if ($publie and !objet_test_si_publie($objet, $id_objet)) {
                         return false;
                 }
@@ -251 +251 @@
         }
 
-        if ($titre AND $id_forum) {
+        if ($titre and $id_forum) {
                 $titre_m = sql_getfetsel('titre', 'spip_forum', "id_forum = " . intval($id_forum));
                 if (!$titre_m) {
@@ -357 +357 @@
 // http://code.spip.net/@reduce_strlen
 function reduce_strlen($n, $c) {
-        return $n-(is_string($c) ? strlen($c) : 0);
-}
-
-?>
+        return $n - (is_string($c) ? strlen($c) : 0);
+}

_core_/plugins/forum/inc/forum_insert.php

@@ -57 +57 @@
         if (!in_array($force_statut, array('privrac', 'privadm'))) {
                 if (!strlen($objet)
-                        OR !intval($id_objet)
+                        or !intval($id_objet)
                 ) {
                         spip_log("Erreur insertion forum sur objet='$objet', id_objet=$id_objet", 'forum.' . _LOG_ERREUR);
@@ -105 +105 @@
         if (!in_array($statut, array('privrac', 'privadm'))) {
                 // si le statut est vide, c'est qu'on ne veut pas de ce presume spam !
-                if (!$statut OR !$objet OR !$id_objet) {
+                if (!$statut or !$objet or !$id_objet) {
                         $args = func_get_args();
                         spip_log("Erreur sur forum_insert_base " . var_export($args, 1), 'forum.' . _LOG_ERREUR);
@@ -149 +149 @@
                 // Ajouter un document
                 if (isset($_FILES['ajouter_document'])
-                        AND $_FILES['ajouter_document']['tmp_name']
+                        and $_FILES['ajouter_document']['tmp_name']
                 ) {
                         $files[] = array(
@@ -192 +192 @@
         return $champs;
 }
-
-?>

_core_/plugins/forum/notifications/forumposte.php

@@ -31 +31 @@
 
         // plugin notification si present
-        $prevenir_auteurs = isset($GLOBALS['notifications']['prevenir_auteurs']) AND $GLOBALS['notifications']['prevenir_auteurs'];
+        $prevenir_auteurs = isset($GLOBALS['notifications']['prevenir_auteurs']) and $GLOBALS['notifications']['prevenir_auteurs'];
         // sinon voie normale
-        if ($t['objet'] == 'article' AND !$prevenir_auteurs) {
+        if ($t['objet'] == 'article' and !$prevenir_auteurs) {
                 $s = sql_getfetsel('accepter_forum', 'spip_articles', "id_article=" . $t['id_objet']);
                 if (!$s) {
@@ -40 +40 @@
 
                 $prevenir_auteurs = (strpos(@$GLOBALS['meta']['prevenir_auteurs'], ",$s,") !== false
-                        OR @$GLOBALS['meta']['prevenir_auteurs'] === 'oui'); // compat
+                        or @$GLOBALS['meta']['prevenir_auteurs'] === 'oui'); // compat
         }
 
@@ -58 +58 @@
 
                 while ($qui = sql_fetch($result)) {
-                        if ($qui['email'] AND autoriser('modererforum', $t['objet'], $t['id_objet'], $qui['id_auteur'])) {
+                        if ($qui['email'] and autoriser('modererforum', $t['objet'], $t['id_objet'], $qui['id_auteur'])) {
                                 $tous[] = $qui['email'];
                         }
@@ -67 +67 @@
         $destinataires = pipeline('notifications_destinataires',
                 array(
-                        'args' => array('quoi' => $quoi, 'id' => $id_forum, 'options' => $options)
-                ,
+                        'args' => array('quoi' => $quoi, 'id' => $id_forum, 'options' => $options),
                         'data' => $tous
                 )
@@ -93 +92 @@
         }
 }
-
-?>

_core_/plugins/forum/notifications/forumvalide.php

@@ -36 +36 @@
         if (!$t
                 // forum sur un message prive : pas de notification ici (cron)
-                OR @$t['statut'] == 'perso'
+                or @$t['statut'] == 'perso'
         ) {
                 return;
@@ -42 +42 @@
 
         // plugin notification si present
-        $prevenir_auteurs = isset($GLOBALS['notifications']['prevenir_auteurs']) AND $GLOBALS['notifications']['prevenir_auteurs'];
+        $prevenir_auteurs = isset($GLOBALS['notifications']['prevenir_auteurs']) and $GLOBALS['notifications']['prevenir_auteurs'];
         // sinon voie normale
-        if ($t['objet'] == 'article' AND !$prevenir_auteurs) {
+        if ($t['objet'] == 'article' and !$prevenir_auteurs) {
                 $s = sql_getfetsel('accepter_forum', 'spip_articles', "id_article=" . $t['id_objet']);
                 if (!$s) {
@@ -51 +51 @@
 
                 $prevenir_auteurs = (strpos(@$GLOBALS['meta']['prevenir_auteurs'], ",$s,") !== false
-                        OR @$GLOBALS['meta']['prevenir_auteurs'] === 'oui'); // compat
+                        or @$GLOBALS['meta']['prevenir_auteurs'] === 'oui'); // compat
         }
 
@@ -107 +107 @@
 
 }
-
-?>

_core_/plugins/forum/prive/modeles/forum_fonctions.php

@@ -24 +24 @@
         // sinon on fait une requete groupee pour essayer de ne le faire qu'une fois pour toute la liste
         $emails = sql_allfetsel("DISTINCT email_auteur", "spip_forum",
-                "id_forum>" . intval($id_forum-50) . " AND id_forum<" . intval($id_forum+50));
+                "id_forum>" . intval($id_forum - 50) . " AND id_forum<" . intval($id_forum + 50));
         $emails = array_map('reset', $emails);
         $emails = array_filter($emails);

_core_/plugins/forum/public/forum.php

@@ -27 +27 @@
         // de meme qu'un critere sur {id_forum} ou {id_parent}
         if (!isset($boucle->modificateur['tout'])
-                AND !isset($boucle->modificateur['plat'])
-                AND !isset($boucle->modificateur['criteres']['id_forum'])
-                AND !isset($boucle->modificateur['criteres']['id_parent'])
+                and !isset($boucle->modificateur['plat'])
+                and !isset($boucle->modificateur['criteres']['id_forum'])
+                and !isset($boucle->modificateur['criteres']['id_parent'])
         ) {
                 array_unshift($boucle->where, array("'='", "'$id_table." . "id_parent'", 0));
@@ -110 +110 @@
         // a la boucle mere de reserver son id_primary
         if ($p->id_boucle
-                AND isset($p->boucles[$p->id_boucle])
-                AND $primary = $p->boucles[$p->id_boucle]->primary
+                and isset($p->boucles[$p->id_boucle])
+                and $primary = $p->boucles[$p->id_boucle]->primary
         ) {
                 $_type = _q($p->boucles[$p->id_boucle]->type_requete);
@@ -169 +169 @@
         if ($id_parent = intval($reponse)) {
                 if ($type
-                        AND $type != 'forums'
-                        AND $primary
+                        and $type != 'forums'
+                        and $primary
                 ) {
                         $forum = array('objet' => $type, 'id_objet' => $primary);
@@ -186 +186 @@
 
         // Ce n'est pas une reponse, on prend la boucle mere
-        if ($type AND $primary) {
+        if ($type and $primary) {
                 return id_table_objet($type) . '=' . intval($primary);
         }
@@ -194 +194 @@
         foreach ($env as $k => $v) {
                 if (preg_match(',^id_([a-z_]+)$,S', $k)
-                        AND $id = intval($v)
+                        and $id = intval($v)
                 ) {
                         return id_table_objet($k) . '=' . $v;
@@ -229 +229 @@
 // http://code.spip.net/@lang_parametres_forum
 function lang_parametres_forum($qs, $lang) {
-        if (is_array($lang) AND preg_match(',id_([a-z_]+)=([0-9]+),', $qs, $r)) {
+        if (is_array($lang) and preg_match(',id_([a-z_]+)=([0-9]+),', $qs, $r)) {
                 $id = 'id_' . $r[1];
                 if ($t = $lang[$id]) {
@@ -237 +237 @@
         // Si ce n'est pas la meme que celle du site, l'ajouter aux parametres
 
-        if ($lang AND $lang <> $GLOBALS['meta']['langue_site']) {
+        if ($lang and $lang <> $GLOBALS['meta']['langue_site']) {
                 return $qs . "&lang=" . $lang;
         }
@@ -251 +251 @@
         return $code;
 }
-
-?>

_core_/plugins/forum/urls/generer_url_ecrire_forum.php

@@ -18 +18 @@
 function urls_generer_url_ecrire_forum_dist($id, $args = '', $ancre = '', $public = null, $connect = '') {
         $a = "id_forum=" . intval($id);
-        if (is_null($public) AND !$connect) {
+        if (is_null($public) and !$connect) {
                 $public = objet_test_si_publie('forum', $id, $connect);
         }
-        $h = ($public OR $connect)
+        $h = ($public or $connect)
                 ? generer_url_entite_absolue($id, 'forum', $args, $ancre, $connect)
                 : (generer_url_ecrire('controler_forum', "debut_forum=@$id" . ($args ? "&$args" : ''))

_core_/plugins/forum/urls/generer_url_forum.php

@@ -18 +18 @@
         if ($id_forum = intval($id_forum)) {
                 include_spip('inc/forum');
-                list($type, $id,) = racine_forum($id_forum);
+                list($type, $id, ) = racine_forum($id_forum);
                 if ($type) {
                         if (!$ancre) {
@@ -30 +30 @@
         return '';
 }
-
-?>