Changeset 93643 in spip-zone

Timestamp:

Dec 13, 2015, 2:42:45 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/mediabox

Files:

• colorbox/fancybox/mediabox_config_theme.php (modified) (1 diff)
• formulaires/configurer_mediabox.php (modified) (3 diffs)
• mediabox_ieconfig.php (modified) (1 diff)
• mediabox_pipelines.php (modified) (5 diffs)

_core_/plugins/mediabox/colorbox/fancybox/mediabox_config_theme.php

@@ -13 +13 @@
         return $config;
 }
-
-?>

_core_/plugins/mediabox/formulaires/configurer_mediabox.php

@@ -22 +22 @@
                 $f = $d . "colorbox/";
                 if (@is_dir($f)) {
-                        $liste = preg_files($f, "colorbox[.]css$", $maxfiles-count($liste_fichiers), $recurs);
+                        $liste = preg_files($f, "colorbox[.]css$", $maxfiles - count($liste_fichiers), $recurs);
                         foreach ($liste as $chemin) {
                                 $nom = substr(dirname($chemin), strlen($f));
@@ -46 +46 @@
 function box_choisir_skin($skins, $selected, $name = 'skin') {
         $out = "";
-        if (!is_array($skins) OR !count($skins)) {
+        if (!is_array($skins) or !count($skins)) {
                 return $out;
         }
@@ -99 +99 @@
         return array('message_ok' => _T('config_info_enregistree'), 'editable' => true);
 }
-
-?>

_core_/plugins/mediabox/mediabox_ieconfig.php

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

_core_/plugins/mediabox/mediabox_pipelines.php

@@ -29 +29 @@
         ), $config);
 
-        if ((is_null($public) AND test_espace_prive()) OR $public === false) {
+        if ((is_null($public) and test_espace_prive()) or $public === false) {
                 $config = array_merge($config, array(
                         'active' => 'oui',
@@ -45 +45 @@
 
         // Gerer aussi les liens internes de SPIP
-        if (!test_espace_prive() AND $config['splash_url']) {
+        if (!test_espace_prive() and $config['splash_url']) {
                 include_spip("inc/filtres_ecrire");
                 $config['splash_url'] = url_absolue(extraire_attribut(lien_article_virtuel($config['splash_url']), 'href'));
@@ -51 +51 @@
 
         // charger la config du theme uniquement dans le public
-        if (!test_espace_prive() AND include_spip("colorbox/" . $config['skin'] . "/mediabox_config_theme")) {
+        if (!test_espace_prive() and include_spip("colorbox/" . $config['skin'] . "/mediabox_config_theme")) {
                 $config_theme = mediabox_config_theme();
                 $config = array_merge($config, $config_theme);
@@ -61 +61 @@
 function mediabox_insert_head_css($flux) {
         $config = mediabox_config();
-        if ($config['active'] == 'oui' AND $f = find_in_path((test_espace_prive() ? "prive/" : "") . "colorbox/" . $config['skin'] . '/colorbox.css')) {
+        if ($config['active'] == 'oui' and $f = find_in_path((test_espace_prive() ? "prive/" : "") . "colorbox/" . $config['skin'] . '/colorbox.css')) {
                 $flux .= '<link rel="stylesheet" href="' . direction_css($f) . '" type="text/css" media="all" />';
                 /**
@@ -134 +134 @@
         return $plugins;
 }
-
-?>