Changeset 93633 in spip-zone

Timestamp:

Dec 13, 2015, 2:29:47 PM (4 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/breves

Files:

• action/editer_breve.php (modified) (3 diffs)
• base/breves.php (modified) (1 diff)
• breves_administrations.php (modified) (2 diffs)
• breves_autoriser.php (modified) (4 diffs)
• breves_ieconfig.php (modified) (1 diff)
• breves_pipelines.php (modified) (3 diffs)
• formulaires/configurer_breves.php (modified) (1 diff)
• formulaires/editer_breve.php (modified) (1 diff)

_core_/plugins/breves/action/editer_breve.php

@@ -198 +198 @@
 
         if (isset($c['statut'])
-                AND $c['statut']
-                AND $c['statut'] != $statut
-                AND autoriser('publierdans', 'rubrique', $id_rubrique)
+                and $c['statut']
+                and $c['statut'] != $statut
+                and autoriser('publierdans', 'rubrique', $id_rubrique)
         ) {
                 $statut = $champs['statut'] = $c['statut'];
@@ -209 +209 @@
         // de la rubrique actuelle
         if ($id_parent = intval($c['id_parent'])
-                AND $id_parent != $id_rubrique
-                AND (null !== ($lang = sql_getfetsel('lang', 'spip_rubriques',
+                and $id_parent != $id_rubrique
+                and (null !== ($lang = sql_getfetsel('lang', 'spip_rubriques',
                                 "id_parent=0 AND id_rubrique=" . intval($id_parent))))
         ) {
@@ -324 +324 @@
         return breve_modifier($id_breve, $set);
 }
-
-?>

_core_/plugins/breves/base/breves.php

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

_core_/plugins/breves/breves_administrations.php

@@ -38 +38 @@
                 $trouver_table = charger_fonction('trouver_table', 'base');
                 if ($desc = $trouver_table('spip_breves')
-                        AND isset($desc['exist']) AND $desc['exist']
+                        and isset($desc['exist']) and $desc['exist']
                 ) {
                         ecrire_meta($nom_meta_base_version, '1.0.0');
@@ -70 +70 @@
         effacer_meta($nom_meta_base_version);
 }
-
-?>

_core_/plugins/breves/breves_autoriser.php

@@ -54 +54 @@
 function autoriser_brevecreer_menu_dist($faire, $type, $id, $qui, $opt) {
         return ($GLOBALS['meta']["activer_breves"] != "non")
-        AND verifier_table_non_vide();
+        and verifier_table_non_vide();
 }
 
@@ -72 +72 @@
         return
                 ($GLOBALS['meta']["activer_breves"] != "non")
-                AND (sql_countsel('spip_rubriques') > 0);
+                and (sql_countsel('spip_rubriques') > 0);
 }
 
@@ -90 +90 @@
         return
                 $id
-                AND ($r['id_parent'] == 0)
-                AND ($GLOBALS['meta']["activer_breves"] != "non")
-                AND autoriser('voir', 'rubrique', $id);
+                and ($r['id_parent'] == 0)
+                and ($GLOBALS['meta']["activer_breves"] != "non")
+                and autoriser('voir', 'rubrique', $id);
 }
 
@@ -113 +113 @@
 
         return
-                $r AND (
-                ($r['statut'] == 'publie' OR (isset($opt['statut']) AND $opt['statut'] == 'publie'))
+                $r and (
+                ($r['statut'] == 'publie' or (isset($opt['statut']) and $opt['statut'] == 'publie'))
                         ? autoriser('publierdans', 'rubrique', $r['id_rubrique'], $qui, $opt)
                         : in_array($qui['statut'], array('0minirezo', '1comite'))
                 );
 }
-
-
-?>

_core_/plugins/breves/breves_ieconfig.php

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

_core_/plugins/breves/breves_pipelines.php

@@ -75 +75 @@
 function breves_affiche_enfants($flux) {
         if (isset($flux['args']['exec'])
-                AND $e = trouver_objet_exec($flux['args']['exec'])
-                AND $e['type'] == 'rubrique'
-                AND $e['edition'] == false
+                and $e = trouver_objet_exec($flux['args']['exec'])
+                and $e['type'] == 'rubrique'
+                and $e['edition'] == false
         ) {
                 $id_rubrique = $flux['args']['id_rubrique'];
@@ -173 +173 @@
 function breves_objet_compte_enfants($flux) {
         if ($flux['args']['objet'] == 'rubrique'
-                AND $id_rubrique = intval($flux['args']['id_objet'])
+                and $id_rubrique = intval($flux['args']['id_objet'])
         ) {
                 // juste les publies ?
@@ -329 +329 @@
         return $flux;
 }
-
-
-?>

_core_/plugins/breves/formulaires/configurer_breves.php

@@ -56 +56 @@
         return $res;
 }
-

_core_/plugins/breves/formulaires/editer_breve.php

@@ -194 +194 @@
                 $hidden);
 }
-
-?>