Changeset 101494 in spip-zone


Ignore:
Timestamp:
Jan 3, 2017, 11:01:22 AM (3 years ago)
Author:
kent1@…
Message:

On en profite pour un coup de PSR

Location:
_core_/plugins/urls_etendues/urls
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • _core_/plugins/urls_etendues/urls/arbo.php

    r101493 r101494  
    1111\***************************************************************************/
    1212
    13 if (!defined("_ECRIRE_INC_VERSION")) {
     13if (!defined('_ECRIRE_INC_VERSION')) {
    1414        return;
    1515} // securiser
     
    8181include_spip('inc/xcache');
    8282if (!function_exists('Cache')) {
    83         function Cache() { return null; }
     83        function Cache() {
     84                return null;
     85        }
    8486}
    8587
     
    191193        }
    192194
    193         return "";
     195        return '';
    194196}
    195197
     
    238240
    239241        include_spip('action/editer_url');
    240         if (!$url = url_nettoyer($objet['titre'], _URLS_ARBO_MAX, _URLS_ARBO_MIN, '-',
    241                 _url_arbo_minuscules ? 'spip_strtolower' : '')
    242         ) {
     242        if (!$url = url_nettoyer(
     243                $objet['titre'],
     244                _URLS_ARBO_MAX,
     245                _URLS_ARBO_MIN,
     246                '-',
     247                _url_arbo_minuscules ? 'spip_strtolower' : ''
     248        )) {
    243249                $url = $objet['id_objet'];
    244250        }
     
    246252        // le type ou son synonyme
    247253        $prefixe = url_arbo_type($objet['type']);
    248         if (strpos($prefixe,'<') !== false) {
     254        if (strpos($prefixe, '<') !== false) {
    249255                $prefixe = extraire_multi($prefixe);
    250256                $prefixe = textebrut($prefixe);
     
    298304        $desc = $trouver_table(table_objet($type));
    299305        $table = $desc['table'];
    300         $col_id = @$desc['key']["PRIMARY KEY"];
     306        $col_id = @$desc['key']['PRIMARY KEY'];
    301307        if (!$col_id) {
    302308                return false;
     
    312318        $champ_parent = url_arbo_parent($type);
    313319        $sel_parent = ', 0 as parent';
    314         $order_by_parent = "";
     320        $order_by_parent = '';
    315321        if ($champ_parent) {
    316322                // si un parent est fourni est qu'il est legitime, on recherche une URL pour ce parent
     
    319325                        and $url_verifier_parent_objet = charger_fonction('url_verifier_parent_objet', 'inc', true)
    320326                        and $url_verifier_parent_objet($type, $id_objet, $type_parent, $id_parent)) {
    321                         $sel_parent = ", ".intval($id_parent) . ' as parent';
     327                        $sel_parent = ', '.intval($id_parent) . ' as parent';
    322328                        // trouver l'url qui matche le parent en premier
    323                         $order_by_parent = "U.id_parent=".intval($id_parent)." DESC, ";
    324                 }
    325                 // sinon on prend son parent direct fourni par $champ_parent
    326                 else {
    327                         $sel_parent = ", O." . reset($champ_parent) . ' as parent';
     329                        $order_by_parent = 'U.id_parent='.intval($id_parent).' DESC, ';
     330                } else {
     331                        // sinon on prend son parent direct fourni par $champ_parent
     332                        $sel_parent = ', O.' . reset($champ_parent) . ' as parent';
    328333                        // trouver l'url qui matche le parent en premier
    329                         $order_by_parent = "O." . reset($champ_parent) . "=U.id_parent DESC, ";
     334                        $order_by_parent = 'O.' . reset($champ_parent) . '=U.id_parent DESC, ';
    330335                }
    331336        }
     
    336341
    337342        //  Recuperer une URL propre correspondant a l'objet.
    338         $row = sql_fetsel("U.url, U.date, U.id_parent, U.perma, U.langue, $champ_titre $sel_parent",
     343        $row = sql_fetsel(
     344                "U.url, U.date, U.id_parent, U.perma, U.langue, $champ_titre $sel_parent",
    339345                "$table AS O LEFT JOIN spip_urls AS U ON (U.type='$type' AND U.id_objet=O.$col_id)",
    340346                "O.$col_id=$id_objet",
    341347                '',
    342                 $order_by_parent . 'U.perma DESC, ' . $order_by_langue . 'U.date DESC', 1);
     348                $order_by_parent . 'U.perma DESC, ' . $order_by_langue . 'U.date DESC',
     349                1
     350        );
    343351        if ($row) {
    344352                $urls[$type][$id_objet] = $row;
     
    371379                if (!_url_arbo_multilang) {
    372380                        $contexte['langue'] = '';
    373                 }
    374                 elseif (_url_arbo_multilang === true) {
     381                } elseif (_url_arbo_multilang === true) {
    375382                        $contexte['langue'] = $GLOBALS['spip_lang'];
    376                 }
    377                 else {
     383                } else {
    378384                        $contexte['langue'] = _url_arbo_multilang;
    379385                }
     
    403409
    404410        if (!isset($urls[$type][$id_objet][$hash])) {
    405                 return "";
     411                return '';
    406412        } # objet inexistant
    407413
     
    416422                and ($u['perma'] or !$modifier_url)
    417423        ) {
    418                 return declarer_url_arbo_rec($url_propre, $type,
     424                return declarer_url_arbo_rec(
     425                        $url_propre,
     426                        $type,
    419427                        isset($u['parent']) ? $u['parent'] : 0,
    420                         isset($u['type_parent']) ? $u['type_parent'] : null, $contexte);
     428                        isset($u['type_parent']) ? $u['type_parent'] : null,
     429                        $contexte
     430                );
    421431        }
    422432
     
    425435        $urls_langues = array();
    426436        if (is_null($url_propre) or ($modifier_url and !$u['perma'])) {
    427 
    428437                $langues = array();
    429438                if (_url_arbo_multilang === true) {
    430439                        include_spip('inc/lang');
    431                         $langues = (isset($GLOBALS['meta']['langues_utilisees'])?$GLOBALS['meta']['langues_utilisees']:'');
    432                         $langues = explode(',',$langues);
     440                        $langues = (isset($GLOBALS['meta']['langues_utilisees']) ? $GLOBALS['meta']['langues_utilisees'] : '');
     441                        $langues = explode(',', $langues);
    433442                        if ($k = array_search(_LANGUE_PAR_DEFAUT, $langues)) {
    434443                                unset($langues[$k]);
     
    446455                $objets = urls_liste_objets();
    447456
    448                 foreach($langues as $l) {
     457                foreach ($langues as $l) {
    449458                        if ($l) {
    450459                                changer_langue($l);
    451460                        }
    452                         $urls_langues[$l] = pipeline('arbo_creer_chaine_url',
     461                        $urls_langues[$l] = pipeline(
     462                                'arbo_creer_chaine_url',
    453463                                array(
    454464                                        'data' => $url_propre,  // le vieux url_propre
     
    464474                                $urls_langues[$l] = $urls_langues[$l] . _url_arbo_sep_id . $id_objet;
    465475                        }
    466 
    467476                }
    468477                // retablir la $langue_courante par securite, au cas ou on a change de langue
     
    503512        // on enregistre toutes les langues
    504513        include_spip('action/editer_url');
    505         foreach($urls_langues as $langue => $url) {
     514        foreach ($urls_langues as $langue => $url) {
    506515                $set = array(
    507516                        'url' => $url,
     
    562571                        unset($contexte['lang']);
    563572                        $args = http_build_query($contexte);
    564                 }
    565                 elseif(isset($GLOBALS['spip_lang']) and $GLOBALS['spip_lang']) {
     573                } elseif (isset($GLOBALS['spip_lang']) and $GLOBALS['spip_lang']) {
    566574                        $c['langue'] = $GLOBALS['spip_lang'];
    567575                        $debut_langue = $c['langue'] .'/';
    568576                }
    569         }
    570         elseif (_url_arbo_multilang){
     577        } elseif (_url_arbo_multilang) {
    571578                $c['langue'] = _url_arbo_multilang;
    572579        }
     
    599606                        . url_arbo_terminaison($type);
    600607        } else {
    601 
    602608                // objet connu mais sans possibilite d'URL lisible, revenir au defaut
    603609                include_spip('base/connect_sql');
    604610                $id_type = id_table_objet($type);
    605                 $url = get_spip_script('./') . "?" . _SPIP_PAGE . "=$type&$id_type=$id";
     611                $url = get_spip_script('./') . '?' . _SPIP_PAGE . "=$type&$id_type=$id";
    606612        }
    607613
     
    743749                                }
    744750                        }
    745                 }
    746                 elseif(_url_arbo_multilang) {
     751                } elseif (_url_arbo_multilang) {
    747752                        $langue = _url_arbo_multilang;
    748753                }
     
    757762                        // si on est dans un contexte de parent, donne par le segment precedent,
    758763                        // prefixer le segment recherche avec ce contexte
    759                         $cp = "0"; // par defaut : parent racine, id=0
     764                        $cp = '0'; // par defaut : parent racine, id=0
    760765                        if ($dernier_parent_vu) {
    761766                                $cp = $parents_vus[$dernier_parent_vu];
    762767                        }
    763768                        // d'abord recherche avec prefixe parent, en une requete car aucun risque de colision
    764                         $row = sql_fetsel('id_objet, type, url',
     769                        $row = sql_fetsel(
     770                                'id_objet, type, url',
    765771                                'spip_urls',
    766772                                is_null($type)
    767                                         ? "url=" . sql_quote($url_segment, '', 'TEXT')
     773                                        ? 'url=' . sql_quote($url_segment, '', 'TEXT')
    768774                                        : sql_in('url', array("$type/$url_segment", $type)),
    769775                                '',
     
    771777                                // puis la bonne langue puis la langue ''
    772778                                // puis les deux segments puis 1 seul segment
    773                           //
     779                                //
    774780                                // si parent indefini on privilegie id_parent=0 avec la derniere clause du order
    775                                 (intval($cp) ? "id_parent=" . intval($cp) . " DESC, " : "id_parent>=0 DESC, ")
     781                                (intval($cp) ? 'id_parent=' . intval($cp) . ' DESC, ' : 'id_parent>=0 DESC, ')
    776782                                . ($langue?'langue='.sql_quote($langue).' DESC, ':'') ."langue='' DESC,"
    777                                 . "segments DESC, id_parent"
     783                                . 'segments DESC, id_parent'
    778784                        );
    779785                        if ($row) {
     
    833839                        // ne pas regenerer des segments arbo, mais rediriger vers la nouvelle URL
    834840                        // dans la nouvelle forme
    835                         if (strncmp($caller, "urls_", 5) == 0 and $caller !== "urls_decoder_url") {
     841                        if (strncmp($caller, 'urls_', 5) == 0 and $caller !== 'urls_decoder_url') {
    836842                                // en absolue, car assembler ne gere pas ce cas particulier
    837843                                include_spip('inc/filtres_mini');
     
    840846                                // securite contre redirection infinie
    841847                                if ($url_new !== $url_propre
    842                                         and rtrim($url_new, "/") !== rtrim($url_propre, "/")
     848                                        and rtrim($url_new, '/') !== rtrim($url_propre, '/')
    843849                                ) {
    844850                                        $url_redirect = url_absolue($url_new);
     
    859865                                if ($langue and _url_arbo_multilang === true) {
    860866                                        $url_arbo_new = "$langue/" . $url_arbo_new;
    861                                         if (strpos($args,'lang=') !== false) {
     867                                        if (strpos($args, 'lang=') !== false) {
    862868                                                parse_str($args, $cl);
    863869                                                unset($cl['lang']);
     
    899905                }
    900906        }
    901         if (!defined('_SET_HTML_BASE')){
    902                 define('_SET_HTML_BASE',1);
     907        if (!defined('_SET_HTML_BASE')) {
     908                define('_SET_HTML_BASE', 1);
    903909        }
    904910
  • _core_/plugins/urls_etendues/urls/html.php

    r94396 r101494  
    3232*/
    3333
    34 if (!defined("_ECRIRE_INC_VERSION")) {
     34if (!defined('_ECRIRE_INC_VERSION')) {
    3535        return;
    3636} // securiser
  • _core_/plugins/urls_etendues/urls/libres.php

    r94396 r101494  
    1111\***************************************************************************/
    1212
    13 if (!defined("_ECRIRE_INC_VERSION")) {
     13if (!defined('_ECRIRE_INC_VERSION')) {
    1414        return;
    1515} // securiser
    1616
    1717/*
    18 Ce jeu d'URLs est une variation de inc-urls-propres mais les urls 
     18Ce jeu d'URLs est une variation de inc-urls-propres mais les urls
    1919de differents types ne sont PAS distinguees par des marqueurs (_,-,+, etc.) ;
    2020*/
  • _core_/plugins/urls_etendues/urls/propres.php

    r100835 r101494  
    1111\***************************************************************************/
    1212
    13 if (!defined("_ECRIRE_INC_VERSION")) {
     13if (!defined('_ECRIRE_INC_VERSION')) {
    1414        return;
    1515} // securiser
     
    3939        "Mon-titre-d-article".
    4040
    41 La variante 'propres2' ajoutera '.html' aux adresses generees : 
     41La variante 'propres2' ajoutera '.html' aux adresses generees :
    4242        "Mon-titre-d-article.html"
    4343
     
    120120function urls_propres_creer_chaine_url($x) {
    121121        // NB: ici url_old ne sert pas, mais un plugin qui ajouterait une date
    122         // pourrait l'utiliser pour juste ajouter la 
     122        // pourrait l'utiliser pour juste ajouter la
    123123        $url_old = $x['data'];
    124124        $objet = $x['objet'];
     
    126126
    127127        include_spip('action/editer_url');
    128         if (!$url = url_nettoyer($objet['titre'], _URLS_PROPRES_MAX, _URLS_PROPRES_MIN, '-',
    129                 _url_minuscules ? 'spip_strtolower' : '')
    130         ) {
     128        if (!$url = url_nettoyer(
     129                $objet['titre'],
     130                _URLS_PROPRES_MAX,
     131                _URLS_PROPRES_MIN,
     132                '-',
     133                _url_minuscules ? 'spip_strtolower' : ''
     134        )) {
    131135                $url = $objet['type'] . $objet['id_objet'];
    132136        }
     
    145149        $table = $desc['table'];
    146150        $champ_titre = $desc['titre'] ? $desc['titre'] : 'titre';
    147         $col_id = @$desc['key']["PRIMARY KEY"];
     151        $col_id = @$desc['key']['PRIMARY KEY'];
    148152        if (!$col_id) {
    149153                return false;
     
    156160        // mais urls a 1 segment uniquement (pas d'urls /)
    157161        // de preference avec id_parent=0, puis perma, puis langue='' puis par date desc
    158         $row = sql_fetsel("U.url, U.date, U.id_parent, U.perma, $champ_titre",
     162        $row = sql_fetsel(
     163                "U.url, U.date, U.id_parent, U.perma, $champ_titre",
    159164                "$table AS O LEFT JOIN spip_urls AS U ON (U.type='$type' AND U.id_objet=O.$col_id)",
    160                 "O.$col_id=$id_objet AND (U.segments IS NULL OR U.segments=1)", '', 'U.id_parent=0 DESC, U.perma DESC, U.langue=\'\' DESC, U.date DESC',
    161                 1);
     165                "O.$col_id=$id_objet AND (U.segments IS NULL OR U.segments=1)",
     166                '',
     167                'U.id_parent=0 DESC, U.perma DESC, U.langue=\'\' DESC, U.date DESC',
     168                1
     169        );
    162170
    163171        // en SQLite le left join retourne du vide si il y a une url mais qui ne correspond pas pour la condition sur le segment
    164172        // on verifie donc que l'objet existe bien avant de sortir ou de creer une url pour cet objet
    165173        if (!$row) {
    166                 $row = sql_fetsel("'' as url, '' as date, 0 as id_parent, 0 as perma, $champ_titre",
     174                $row = sql_fetsel(
     175                        "'' as url, '' as date, 0 as id_parent, 0 as perma, $champ_titre",
    167176                        "$table AS O",
    168                         "O.$col_id=$id_objet");
     177                        "O.$col_id=$id_objet"
     178                );
    169179        }
    170180
    171181        if (!$row) {
    172                 return "";
     182                return '';
    173183        } # Quand $id_objet n'est pas un numero connu
    174184
     
    181191                // si on arrive pas a reinserer tel quel, on annule url_propre pour forcer un recalcul d'url
    182192                if (!url_insert($set, false, _url_propres_sep_id)) {
    183                         $url_propre = "";
     193                        $url_propre = '';
    184194                } else {
    185195                        $url_propre = $row['url'] = $set['url'];
     
    199209
    200210        // Sinon, creer une URL
    201         $url = pipeline('propres_creer_chaine_url',
     211        $url = pipeline(
     212                'propres_creer_chaine_url',
    202213                array(
    203214                        'data' => $url_propre,  // le vieux url_propre
    204                         'objet' => array_merge($row,
     215                        'objet' => array_merge(
     216                                $row,
    205217                                array('type' => $type, 'id_objet' => $id_objet)
    206218                        )
     
    290302                // on les encadre d'un - : /-1234-
    291303                if (is_numeric($url)) {
    292                         $url = "-" . $url . "-";
    293                 }
    294 
    295                 if (!defined('_SET_HTML_BASE') or !_SET_HTML_BASE) // Repositionne l'URL par rapport a la racine du site (#GLOBALS)
    296                 {
     304                        $url = '-' . $url . '-';
     305                }
     306
     307                if (!defined('_SET_HTML_BASE') or !_SET_HTML_BASE) {
     308                        // Repositionne l'URL par rapport a la racine du site (#GLOBALS)
    297309                        $url = str_repeat('../', $GLOBALS['profondeur_url']) . $url;
    298310                } else {
     
    300312                }
    301313        } else {
    302 
    303314                // objet connu mais sans possibilite d'URL lisible, revenir au defaut
    304315                include_spip('base/connect_sql');
    305316                $id_type = id_table_objet($type);
    306                 $url = _DIR_RACINE . get_spip_script('./') . "?" . _SPIP_PAGE . "=$type&$id_type=$id";
     317                $url = _DIR_RACINE . get_spip_script('./') . '?' . _SPIP_PAGE . "=$type&$id_type=$id";
    307318        }
    308319
  • _core_/plugins/urls_etendues/urls/propres2.php

    r94396 r101494  
    1111\***************************************************************************/
    1212
    13 if (!defined("_ECRIRE_INC_VERSION")) {
     13if (!defined('_ECRIRE_INC_VERSION')) {
    1414        return;
    1515} // securiser
  • _core_/plugins/urls_etendues/urls/propres_qs.php

    r94396 r101494  
    1111\***************************************************************************/
    1212
    13 if (!defined("_ECRIRE_INC_VERSION")) {
     13if (!defined('_ECRIRE_INC_VERSION')) {
    1414        return;
    1515} // securiser
  • _core_/plugins/urls_etendues/urls/simple.php

    r94396 r101494  
    3535
    3636        $url = _debut_urls_simple . $type
    37                 . "&" . id_table_objet($type) . "="
     37                . '&' . id_table_objet($type) . '='
    3838                . $id . _terminaison_urls_page;
    3939
Note: See TracChangeset for help on using the changeset viewer.