Changeset 28525 in spip-zone


Ignore:
Timestamp:
May 10, 2009, 10:04:48 AM (13 years ago)
Author:
cedric@…
Message:

[24806] n'est pas une bonne solution :

  • le pipeline affichage_final est un pipe non cache, appele a chaque hit,

il faut donc veiller a ne pas charger son calcul

  • l'insertion a la main par le webmestre est faite pour 2 raisons possibles
    • optimisation des performance
    • prendre la main pour maitriser la place et le rendu

avec [24806] tous les avantages de l'insertion manuelle sont perdus :

  • la perf reste degradee par plusieurs regexp calculee dans tous les cas
  • le webmestre se fait imposer le lien vers le moteur de recherche meme si il n'en veut pas

On revert en vertu du principe KISS

Location:
_plugins_/_stable_/skiplink
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/_stable_/skiplink/insere_skiplink.php

    r28406 r28525  
    33# inserer les liens dans la page
    44function skiplink_affichage_final(&$page) {
    5        
    6        
    7 
    85        // ne pas se fatiguer si pas HTML
    96        if (!$GLOBALS['html']
     7          OR strpos($page, 'id="raccourcis"') # pas deux fois, au cas ou !
    108        )
    119                return $page;
    1210
    13     $recherche_existe = (preg_match(',<input.*?name[ ]*=[ ]*"recherche".*?>,i', $page) == 1 ? 'oui' : 'non');
    14         $raccourcis_recherche_existe = strpos($page,'id="raccourci_recherche"');
    15        
    16         //si raccourci recherche existe et recherche aussi -> pas la peine de se fatiguer
    17         if($recherche_existe == "oui" and $raccourcis_recherche_existe)
    18            return $page;
    19    
    2011        if (!function_exists('recuperer_fond')) include_spip('public/assembler');
    21        
     12        $recherche_existe = (preg_match(',<input.*?name[ ]*=[ ]*"recherche".*?>,i', $page) == 1 ? 'oui' : 'non');
    2213        $raccourcis = recuperer_fond('raccourcis', array('lang'=>$GLOBALS['spip_lang'], 'recherche'=>$recherche_existe));
    2314        preg_match(',<body\b.*?>,i', $page, $regs);
    2415 
    25         if ($regs and !strpos($page,'id="raccourcis"'))#pas deux fois au cas ou
     16        if ($regs)
    2617                $page = substr_replace($page, $raccourcis, (strpos($page, $regs[0]) + strlen($regs[0])), 0);
    2718        $remonter = recuperer_fond('remonter', array('lang'=>$GLOBALS['spip_lang']));
     
    3021        if ($regs)
    3122                $page = substr_replace($page, $ancre_basse, strpos($page, $regs[0]), 0);
    32    
    33     if($recherche_existe == "oui" and !$raccourcis_recherche_existe) // au cas où qu1 aurait mis la noisette sans preciser qu'il veut la recherche alors même qu'il l'a.
    34         $raccourcis_erronnes = recuperer_fond('raccourcis', array('lang'=>$GLOBALS['spip_lang'], 'recherche'=>'non'));
    35         $page = str_replace($raccourcis_erronnes,$raccourcis,$page);
    3623
    3724        return $page;
  • _plugins_/_stable_/skiplink/raccourcis.html

    r28406 r28525  
    11<INCLURE{fond=ancre_haute}>
    2 <p class="skiplink" id="raccourcis" style="z-index: 100;"><a href="#contenu" accesskey="s"><:skiplink:go_main:></a> | <a href="#navigation" accesskey="n"><:skiplink:go_sidebar:></a>[(#ENV{recherche}|=={'oui'}|?{' '}) | <a id="raccourci_recherche" href="#recherche" accesskey="4"><:skiplink:go_search:></a>]</p>
     2<p class="skiplink" id="raccourcis" style="z-index: 100;"><a href="#contenu" accesskey="s"><:skiplink:go_main:></a> | <a href="#navigation" accesskey="n"><:skiplink:go_sidebar:></a>[(#ENV{recherche}|=={'oui'}|?{' '}) | <a href="#recherche" accesskey="4"><:skiplink:go_search:></a>]</p>
Note: See TracChangeset for help on using the changeset viewer.