Changeset 93637 in spip-zone


Ignore:
Timestamp:
Dec 13, 2015, 2:36:07 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/compresseur
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • _core_/plugins/compresseur/compresseur_fonctions.php

    r93609 r93637  
    3838 */
    3939function minifier($source, $format = null) {
    40         if (!$format AND preg_match(',\.(js|css)$,', $source, $r)) {
     40        if (!$format and preg_match(',\.(js|css)$,', $source, $r)) {
    4141                $format = $r[1];
    4242        }
     
    4848        // Si on n'importe pas, est-ce un fichier ?
    4949        if (!preg_match(',[\s{}],', $source)
    50                 AND preg_match(',\.' . $format . '$,i', $source, $r)
    51                 AND file_exists($source)
     50                and preg_match(',\.' . $format . '$,i', $source, $r)
     51                and file_exists($source)
    5252        ) {
    5353                // si c'est une css, il faut reecrire les url en absolu
     
    6363
    6464                if ((@filemtime($f) > @filemtime($source))
    65                         AND (!defined('_VAR_MODE') OR _VAR_MODE != 'recalcul')
     65                        and (!defined('_VAR_MODE') or _VAR_MODE != 'recalcul')
    6666                ) {
    6767                        return $f;
     
    128128        if (!defined('_INTERDIRE_COMPACTE_HEAD')) {
    129129                // dans l'espace prive on compacte toujours, c'est concu pour
    130                 if ((!test_espace_prive() AND $GLOBALS['meta']['auto_compress_css'] == 'oui') OR (test_espace_prive() AND !defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE'))) {
     130                if ((!test_espace_prive() and $GLOBALS['meta']['auto_compress_css'] == 'oui') or (test_espace_prive() and !defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE'))) {
    131131                        $flux = compacte_head_files($flux, 'css');
    132132                }
    133                 if ((!test_espace_prive() AND $GLOBALS['meta']['auto_compress_js'] == 'oui') OR (test_espace_prive() AND !defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE'))) {
     133                if ((!test_espace_prive() and $GLOBALS['meta']['auto_compress_js'] == 'oui') or (test_espace_prive() and !defined('_INTERDIRE_COMPACTE_HEAD_ECRIRE'))) {
    134134                        $flux = compacte_head_files($flux, 'js');
    135135                }
     
    166166
    167167        if (!file_exists($filename)
    168                 OR filesize($filename) > $maxsize
    169                 OR !lire_fichier($filename, $contenu)
     168                or filesize($filename) > $maxsize
     169                or !lire_fichier($filename, $contenu)
    170170        ) {
    171171                return $src;
     
    173173
    174174        if (!isset($mime[$extension])) {
    175                 if (isset($GLOBALS['tables_mime']) AND isset($GLOBALS['tables_mime'][$extension])) {
     175                if (isset($GLOBALS['tables_mime']) and isset($GLOBALS['tables_mime'][$extension])) {
    176176                        $mime[$extension] = $GLOBALS['tables_mime'][$extension];
    177177                }
     
    205205function filtre_embarque_src($img, $maxsize = 4096) {
    206206        $src = extraire_attribut($img, 'src');
    207         if ($src2 = filtre_embarque_fichier($src, "", $maxsize) AND $src2 != $src) {
     207        if ($src2 = filtre_embarque_fichier($src, "", $maxsize) and $src2 != $src) {
    208208                $img = inserer_attribut($img, 'src', $src2);
    209209        }
     
    211211        return $img;
    212212}
    213 
    214 ?>
  • _core_/plugins/compresseur/compresseur_ieconfig.php

    r93609 r93637  
    2727        return $table;
    2828}
    29 
    30 ?>
  • _core_/plugins/compresseur/compresseur_options.php

    r93609 r93637  
    1414// si un buffer est deja ouvert, stop
    1515if ($GLOBALS['flag_ob']
    16         AND defined('_AUTO_GZIP_HTTP') AND _AUTO_GZIP_HTTP
    17         AND strlen(ob_get_contents()) == 0
    18         AND !headers_sent()
     16        and defined('_AUTO_GZIP_HTTP') and _AUTO_GZIP_HTTP
     17        and strlen(ob_get_contents()) == 0
     18        and !headers_sent()
    1919) {
    2020
    2121        if (
    2222                // special bug de proxy
    23                 !(isset($_SERVER['HTTP_VIA']) AND preg_match(",NetCache|Hasd_proxy,i", $_SERVER['HTTP_VIA']))
     23                !(isset($_SERVER['HTTP_VIA']) and preg_match(",NetCache|Hasd_proxy,i", $_SERVER['HTTP_VIA']))
    2424                // special bug Netscape Win 4.0x
    25                 AND (strpos($_SERVER['HTTP_USER_AGENT'], 'Mozilla/4.0') === false)
     25                and (strpos($_SERVER['HTTP_USER_AGENT'], 'Mozilla/4.0') === false)
    2626                // special bug Apache2x
    2727                #&& !preg_match(",Apache(-[^ ]+)?/2,i", $_SERVER['SERVER_SOFTWARE'])
     
    3030                // si la compression est deja commencee, stop
    3131                # && !@ini_get("zlib.output_compression")
    32                 AND !@ini_get("output_handler")
    33                 AND !isset($_GET['var_mode']) # bug avec le debugueur qui appelle ob_end_clean()
     32                and !@ini_get("output_handler")
     33                and !isset($_GET['var_mode']) # bug avec le debugueur qui appelle ob_end_clean()
    3434        ) {
    3535                ob_start('ob_gzhandler');
  • _core_/plugins/compresseur/compresseur_pipeline.php

    r93609 r93637  
    9696        return $metas;
    9797}
    98 
    99 ?>
  • _core_/plugins/compresseur/formulaires/configurer_compresseur.php

    r93609 r93637  
    6969        return array('message_ok' => _T('config_info_enregistree'));
    7070}
    71 
    72 ?>
  • _core_/plugins/compresseur/inc/compresseur.php

    r93609 r93637  
    4040        $src = timestamp($src);
    4141        // option chargement JS async par jQl
    42         if (defined('_JS_ASYNC_LOAD') AND !test_espace_prive()) {
     42        if (defined('_JS_ASYNC_LOAD') and !test_espace_prive()) {
    4343                lire_fichier(find_in_path("lib/jQl/jQl.min.js"), $jQl);
    4444                if ($jQl) {
     
    100100        foreach ($balises as $s) {
    101101                if (extraire_attribut($s, 'rel') === 'stylesheet'
    102                         AND (!($type = extraire_attribut($s, 'type'))
    103                                 OR $type == 'text/css')
    104                         AND is_null(extraire_attribut($s, 'name')) # css nommee : pas touche
    105                         AND is_null(extraire_attribut($s, 'id'))   # idem
    106                         AND !strlen(strip_tags($s))
    107                         AND $src = preg_replace(",^$url_base,", _DIR_RACINE, extraire_attribut($s, 'href'))
     102                        and (!($type = extraire_attribut($s, 'type'))
     103                                or $type == 'text/css')
     104                        and is_null(extraire_attribut($s, 'name')) # css nommee : pas touche
     105                        and is_null(extraire_attribut($s, 'id'))   # idem
     106                        and !strlen(strip_tags($s))
     107                        and $src = preg_replace(",^$url_base,", _DIR_RACINE, extraire_attribut($s, 'href'))
    108108                ) {
    109109                        $files[$s] = $src;
     
    128128        foreach ($balises as $s) {
    129129                if (extraire_attribut($s, 'type') === 'text/javascript'
    130                         AND is_null(extraire_attribut($s, 'id')) # script avec un id : pas touche
    131                         AND $src = extraire_attribut($s, 'src')
    132                         AND !strlen(strip_tags($s))
     130                        and is_null(extraire_attribut($s, 'id')) # script avec un id : pas touche
     131                        and $src = extraire_attribut($s, 'src')
     132                        and !strlen(strip_tags($s))
    133133                ) {
    134134                        $files[$s] = $src;
     
    167167                if (
    168168                        preg_match(',^(' . $dir . ')(.*)$,', $src, $r)
    169                         OR (
     169                        or (
    170170                                // ou si c'est un fichier
    171171                                $src = preg_replace(',^' . preg_quote(url_de_base(), ',') . ',', '', $src)
    172172                                // enlever un timestamp eventuel derriere un nom de fichier statique
    173                                 AND $src2 = preg_replace(",[.]{$format}[?].+$,", ".$format", $src)
     173                                and $src2 = preg_replace(",[.]{$format}[?].+$,", ".$format", $src)
    174174                                // verifier qu'il n'y a pas de ../ ni / au debut (securite)
    175                                 AND !preg_match(',(^/|\.\.),', substr($src, strlen(_DIR_RACINE)))
     175                                and !preg_match(',(^/|\.\.),', substr($src, strlen(_DIR_RACINE)))
    176176                                // et si il est lisible
    177                                 AND @is_readable($src2)
     177                                and @is_readable($src2)
    178178                        )
    179179                ) {
     
    197197                $callbacks[] = protocole_implicite($url_base);
    198198        }
    199         if ($format == 'js' AND $GLOBALS['meta']['auto_compress_closure'] == 'oui') {
     199        if ($format == 'js' and $GLOBALS['meta']['auto_compress_closure'] == 'oui') {
    200200                $callbacks['all_min'] = 'minifier_encore_js';
    201201        }
     
    204204        include_spip('inc/compresseur_minifier');
    205205        if (list($src, $comms) = concatener_fichiers($files, $format, $callbacks)
    206                 AND $src
     206                and $src
    207207        ) {
    208208                $compacte_ecrire_balise = charger_fonction("compresseur_ecrire_balise_$format", '');
     
    234234                // les fonctions de preparation aux CSS peuvent etre personalisees
    235235                // via la globale $compresseur_filtres_css sous forme de tableau de fonctions ordonnees
    236                 if (isset($GLOBALS['compresseur_filtres_css']) AND is_array($GLOBALS['compresseur_filtres_css'])) {
    237                         $fonctions = $GLOBALS['compresseur_filtres_css']+$fonctions;
     236                if (isset($GLOBALS['compresseur_filtres_css']) and is_array($GLOBALS['compresseur_filtres_css'])) {
     237                        $fonctions = $GLOBALS['compresseur_filtres_css'] + $fonctions;
    238238                }
    239239        }
     
    278278
    279279        if ((@filemtime($file) > @filemtime($css))
    280                 AND (!defined('_VAR_MODE') OR _VAR_MODE != 'recalcul')
     280                and (!defined('_VAR_MODE') or _VAR_MODE != 'recalcul')
    281281        ) {
    282282                return $file;
     
    285285        if ($url_absolue_css == $css) {
    286286                if (strncmp($GLOBALS['meta']['adresse_site'] . "/", $css, $l = strlen($GLOBALS['meta']['adresse_site'] . "/")) != 0
    287                         OR !lire_fichier(_DIR_RACINE . substr($css, $l), $contenu)
     287                        or !lire_fichier(_DIR_RACINE . substr($css, $l), $contenu)
    288288                ) {
    289289                        include_spip('inc/distant');
     
    333333                        $f = chercher_filtre($f);
    334334                }
    335                 if ($f AND function_exists($f)) {
     335                if ($f and function_exists($f)) {
    336336                        $contenu = $f($contenu, $url_base, $filename);
    337337                }
     
    358358        preg_match_all(",@import ([^;]*);,UmsS", $contenu, $matches, PREG_SET_ORDER);
    359359
    360         if ($matches AND count($matches)) {
     360        if ($matches and count($matches)) {
    361361                foreach ($matches as $m) {
    362362                        $url = $media = $erreur = "";
     
    395395                                        $css = compresseur_callback_prepare_css($url);
    396396                                        if ($css == $url
    397                                                 OR !lire_fichier($css, $contenu_imported)
     397                                                or !lire_fichier($css, $contenu_imported)
    398398                                        ) {
    399399                                                $erreur = "Compresseur : url $url de <tt>" . $m[0] . ";</tt> non resolu dans <tt>$url_base</tt>";
     
    426426function css_regroup_atimport($nom_tmp, $nom) {
    427427        lire_fichier($nom_tmp, $contenu);
    428         if (!$contenu OR strpos($contenu, "@import") === false) {
     428        if (!$contenu or strpos($contenu, "@import") === false) {
    429429                return false;
    430430        } // rien a faire
  • _core_/plugins/compresseur/inc/compresseur_concatener.php

    r93609 r93637  
    6161                $nom = $dir . md5(serialize($files) . serialize($callbacks)) . ".$format";
    6262                if (
    63                         (defined('_VAR_MODE') AND _VAR_MODE == 'recalcul')
    64                         OR !file_exists($nom)
     63                        (defined('_VAR_MODE') and _VAR_MODE == 'recalcul')
     64                        or !file_exists($nom)
    6565                ) {
    6666                        $fichier = "";
     
    117117
    118118                        // calcul du % de compactage
    119                         $pc = intval(1000*strlen($fichier)/$total)/10;
     119                        $pc = intval(1000 * strlen($fichier) / $total) / 10;
    120120                        $comms = "compact [\n\t" . join("\n\t", $comms) . "\n] $pc%";
    121121                        $fichier = "/* $comms */\n\n" . $fichier;
     
    160160
    161161        // Le commentaire detaille n'apparait qu'au recalcul, pour debug
    162         return array($nom, (isset($comms) AND $comms) ? "<!-- $comms -->\n" : '');
     162        return array($nom, (isset($comms) and $comms) ? "<!-- $comms -->\n" : '');
    163163}
    164164
  • _core_/plugins/compresseur/inc/compresseur_embarquer.php

    r93609 r93637  
    4343        $filtre_embarque_fichier = chercher_filtre("filtre_embarque_fichier");
    4444        if (!defined("_CSS_EMBARQUE_FICHIER_MAX_SIZE")) {
    45                 define('_CSS_EMBARQUE_FICHIER_MAX_SIZE', 4*1024);
     45                define('_CSS_EMBARQUE_FICHIER_MAX_SIZE', 4 * 1024);
    4646        }
    4747
  • _core_/plugins/compresseur/inc/compresseur_minifier.php

    r93609 r93637  
    4545 */
    4646function minifier_css($contenu, $options = '') {
    47         if (is_string($options) AND $options) {
     47        if (is_string($options) and $options) {
    4848                if ($options == "all") // facile : media all => ne rien preciser
    4949                {
     
    5151                } elseif (
    5252                        strpos($contenu, "@media") == false
    53                         AND strpos($contenu, "@import") == false
    54                         AND strpos($contenu, "@font-face") == false
     53                        and strpos($contenu, "@import") == false
     54                        and strpos($contenu, "@font-face") == false
    5555                ) {
    5656                        $contenu = "@media $options {\n$contenu\n}\n";
     
    160160                // modele de sortie plus ou moins compact
    161161                $template = 'high';
    162                 if (isset($options['template']) AND in_array($options['template'], array('low', 'default', 'high', 'highest'))) {
     162                if (isset($options['template']) and in_array($options['template'], array('low', 'default', 'high', 'highest'))) {
    163163                        $template = $options['template'];
    164164                }
     
    254254                lire_fichier($nom, $content);
    255255                $dest = dirname($nom) . '/' . md5($content . $file) . '.js';
    256                 if (file_exists($dest) AND (!is_string($file) OR file_exists($file))) {
     256                if (file_exists($dest) and (!is_string($file) or file_exists($file))) {
    257257                        if (filesize($dest)) {
    258258                                return is_string($file) ? $file : $dest;
     
    265265        }
    266266
    267         if (!$file AND strlen($content) > 200000) {
     267        if (!$file and strlen($content) > 200000) {
    268268                return $content;
    269269        }
     
    277277                // 'SIMPLE_OPTIMIZATIONS', 'WHITESPACE_ONLY', 'ADVANCED_OPTIMIZATIONS'
    278278        );
    279         if (!$file OR strlen($content) < 200000) {
     279        if (!$file or strlen($content) < 200000) {
    280280                $datas['js_code'] = $content;
    281281        } else {
     
    289289                $boundary = -1);
    290290
    291         if ($cc AND !preg_match(',^\s*Error,', $cc)) {
     291        if ($cc and !preg_match(',^\s*Error,', $cc)) {
    292292                spip_log('Closure Compiler: success');
    293293                $cc = "/* $nom + Closure Compiler */\n" . $cc;
     
    348348
    349349        // si pas de contenu ni de balise html, ne rien faire
    350         if (!strlen($flux) OR strpos($flux, "<") === false) {
     350        if (!strlen($flux) or strpos($flux, "<") === false) {
    351351                return $flux;
    352352        }
Note: See TracChangeset for help on using the changeset viewer.