[spip-dev] Re: <head>

----Message d'origine----

Copie à: "Déesse A." <esj@vertsdesevres.net>,
De: "Déesse A." <esj@vertsdesevres.net>
Sujet: Re: [spip-dev] Re: <head>
Date: Sat, 23 Oct 2004 12:13:40 +0200
A: Nicolas Krebs <nicolas1.krebs2@netcourrier.com>

Ne serait-il pas préférable de nommer cette fonction debut_entete_html
?
Il est vrai qu'il y a une ligne d'en-tête http.

son nom est encore provisoire; je vais voir.

Dans cette même fonction, la balise <html> contient un attribut dir,
mais pas lang. À ajouter.

bien vu. C'est fait.

Cf diff ci-dessous.

Est-il préférable de placer la fonction dans ecrire/inc_version.php3 plutôt
que dans ecrire/inc_presentation.php3 ?

diff -u ecrire/aide_index.php3
--- ecrire/aide_index.php3 Fri Oct 22 20:01:58 2004
+++ ecrire/aide_index.php3 Sun Oct 24 20:56:06 2004
@@ -442,7 +442,7 @@
       install_fin_html();
     }
   } else {
- echo debut_entete(_T('info_aide_en_ligne'));
+ echo debut_entete_html(_T('info_aide_en_ligne'));
     if ($frame == 'menu')
       help_menu($aide, $html);
     else if ($frame == 'body')
diff -u ecrire/inc_debug_sql.php3
--- ecrire/inc_debug_sql.php3 Fri Oct 22 20:01:58 2004
+++ ecrire/inc_debug_sql.php3 Sun Oct 24 20:56:00 2004
@@ -98,7 +98,7 @@
     if ($HTTP_COOKIE_VARS['spip_admin'] OR
     $auteur_session['statut'] == '0minirezo' OR
     $GLOBALS['var_debug']) {
- echo debut_entete('Debug'), '</head><body>';
+ echo debut_entete_html('Debug'), '</head><body>';
       die(affiche_erreurs_page($tableau_des_erreurs));
     }
   }
@@ -146,7 +146,7 @@
   global $debug_objets, $debug_objet, $debug_affiche;
   if (!$GLOBALS['debug_objets']['sourcefile']) return;
   spip_setcookie('spip_debug', 'oui', time()+12*3600);
- echo debut_entete('Debug');
+ echo debut_entete_html('Debug');
         $page = "</head>\n<body>";
         echo calcul_admin_page('', $page),
     "<div id='spip-debug' style='position: absolute; top: 20; z-index: 1000;'><ul>\n";
diff -u ecrire/inc_presentation.php3
--- ecrire/inc_presentation.php3 Sat Oct 23 15:14:58 2004
+++ ecrire/inc_presentation.php3 Sun Oct 24 20:56:08 2004
@@ -1506,7 +1506,7 @@
   @Header("Expires: 0");
   @Header("Cache-Control: no-cache,no-store");
   @Header("Pragma: no-cache");
- echo debut_entete("[$nom_site_spip] $titre");
+ echo debut_entete_html("[$nom_site_spip] $titre");
   if ($spip_display != 4) {
     echo "<link rel='alternate' type='application/rss+xml' title='".addslashes($nom_site_spip)."' href='$adresse_site/backend.php3' />";
     $activer_breves=lire_meta("activer_breves");
@@ -3277,7 +3277,7 @@
   if ($titre=='AUTO')
     $titre=_T('info_installation_systeme_publication');

- echo debut_entete($titre),
+ echo debut_entete_html($titre),
     "<meta http-equiv='Expires' content='0'>
   <meta http-equiv='cache-control' content='no-cache,no-store'>
   <meta http-equiv='pragma' content='no-cache'>
diff -u ecrire/inc_version.php3
--- ecrire/inc_version.php3 Sat Oct 23 12:12:42 2004
+++ ecrire/inc_version.php3 Sun Oct 24 21:05:18 2004
@@ -1168,7 +1168,7 @@

// Envoi des en-tetes

-function debut_entete($title)
+function debut_entete_html($title)
{
// '<html xmlns:m="http://www.w3.org/1998/Math/MathML&quot;&gt;&#39;\.&quot;\\n&quot;\.&#39;&lt;head&gt;&#39;\.&quot;\\n&quot;;

@@ -1176,7 +1176,6 @@
// '<object id="mathplayer" classid="clsid:32F66A20-7614-11D4-BD11-00104BD3F987">'."\n".'</object>'."\n";
// '<'.'?import namespace="m" implementation="#mathplayer"?'.'>'."\n";

- $dir = ($GLOBALS['spip_lang_rtl'] ? 'rtl' : 'ltr');
   $base=lire_meta("adresse_site");
   if (!$base)
     $base = dirname($GLOBALS['HTTP_SERVERS_VARS']['SCRIPT_NAME']);
@@ -1184,9 +1183,9 @@
     $base .= '/' . (_DIR_RESTREINT ? '' : 'ecrire/');
   if (!$charset = lire_meta('charset')) $charset = 'utf-8';
   @Header("Content-Type: text/html; charset=$charset");
- return "<!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN' 'http://www.w3.org/TR/html4/loose.dtd’>\n" .
- "<html dir='$dir'>\n" .
- "<head dir='$dir'>\n" .
+ return '<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd&quot;&gt;&#39;\.
+ "\n<html lang='".$GLOBALS['spip_lang']."' dir='".($GLOBALS['spip_lang_rtl'] ? 'rtl' : 'ltr')."'>\n" .
+ "<head>\n" .
# "<base href='$base' />\n" . # + tard
     "<title>$title</title>\n" .
     "<meta http-equiv='Content-Type' content='text/html; charset=$charset'>\n";
diff -u inc-public.php3
--- inc-public.php3 Fri Oct 22 23:48:04 2004
+++ inc-public.php3 Sun Oct 24 20:56:02 2004
@@ -109,7 +109,7 @@
   // si le squelette est nul se rabattre sur l'entete standard
   if ($page['texte'])
     @header("Content-Type: text/html; charset=".lire_meta('charset'));
- else echo debut_entete($fond);
+ else echo debut_entete_html($fond);
     // Faudra-t-il post-traiter la page ?
   define('spip_active_ob', $flag_ob AND
     ($var_debug OR $var_recherche OR $affiche_boutons_admin));

Dans cette même fonction, la balise <html> contient un attribut dir,
mais pas lang. À ajouter.

bien vu. C'est fait.

Cf diff ci-dessous.

Oui, ok je reporte.

Est-il préférable de placer la fonction dans ecrire/inc_version.php3 plutôt
que dans ecrire/inc_presentation.php3 ?

Il y a des cas où on en a besoin à partir de l'espace public, notamment lorsqu'un squelette est naze:
il faut alors une entete propre par défaut. Cela dit, il faudrait regarder ces qq cas en détail voire
s'ils incluent inc_presentation de toutes façons, auquel cas on pourrait effectivement la mettre là.

      Emmanuel