spip / SPIP

ecrire/maj/v019_pre193.php

Doc Comments +3 added lines

@@ -12,6 +12,9 @@
 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
+/**
+ * @param integer $version_cible
+ */
 function v019_pre193($version_installee, $version_cible)
 {
 	// Syndication : ajout de l'option resume=oui/non et de la langue

Indentation +232 added lines, -232 removed lines

@@ -14,58 +14,58 @@ 
 
 function v019_pre193($version_installee, $version_cible)
 {
-	// Syndication : ajout de l'option resume=oui/non et de la langue
-	if (upgrade_vers(1.901, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_syndic ADD `resume` VARCHAR(3) DEFAULT 'oui'");
-		spip_query("ALTER TABLE spip_syndic_articles ADD `lang` VARCHAR(10) DEFAULT '' NOT NULL");
-		maj_version(1.901);
-	}
+    // Syndication : ajout de l'option resume=oui/non et de la langue
+    if (upgrade_vers(1.901, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_syndic ADD `resume` VARCHAR(3) DEFAULT 'oui'");
+        spip_query("ALTER TABLE spip_syndic_articles ADD `lang` VARCHAR(10) DEFAULT '' NOT NULL");
+        maj_version(1.901);
+    }
 
-	// Syndication : ajout de source, url_source, tags
-	if (upgrade_vers(1.902, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_syndic_articles ADD `url_source` TINYTEXT DEFAULT '' NOT NULL");
-		spip_query("ALTER TABLE spip_syndic_articles ADD `source` TINYTEXT DEFAULT '' NOT NULL");
-		spip_query("ALTER TABLE spip_syndic_articles ADD `tags` TEXT DEFAULT '' NOT NULL");
-		maj_version(1.902);
-	}
+    // Syndication : ajout de source, url_source, tags
+    if (upgrade_vers(1.902, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_syndic_articles ADD `url_source` TINYTEXT DEFAULT '' NOT NULL");
+        spip_query("ALTER TABLE spip_syndic_articles ADD `source` TINYTEXT DEFAULT '' NOT NULL");
+        spip_query("ALTER TABLE spip_syndic_articles ADD `tags` TEXT DEFAULT '' NOT NULL");
+        maj_version(1.902);
+    }
 
-	// URLs propres des sites (sait-on jamais)
-	// + oubli des KEY url_propre sur les auteurs si installation neuve
-	if (upgrade_vers(1.903, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_syndic ADD `url_propre` VARCHAR(255) NOT NULL");
-		spip_query("ALTER TABLE spip_syndic ADD INDEX `url_propre` (`url_propre`)");
-		spip_query("ALTER TABLE spip_auteurs ADD INDEX `url_propre` (`url_propre`)");
-		maj_version(1.903);
-	}
+    // URLs propres des sites (sait-on jamais)
+    // + oubli des KEY url_propre sur les auteurs si installation neuve
+    if (upgrade_vers(1.903, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_syndic ADD `url_propre` VARCHAR(255) NOT NULL");
+        spip_query("ALTER TABLE spip_syndic ADD INDEX `url_propre` (`url_propre`)");
+        spip_query("ALTER TABLE spip_auteurs ADD INDEX `url_propre` (`url_propre`)");
+        maj_version(1.903);
+    }
 
-	// suppression des anciennes tables temporaires des visites
-	// (maintenant stockees sous forme de fichiers)
-	if (upgrade_vers(1.904, $version_installee, $version_cible)) {
-		spip_query("DROP TABLE IF EXISTS spip_visites_temp");
-		spip_query("DROP TABLE IF EXISTS spip_referers_temp");
-		maj_version(1.904);
-	}
+    // suppression des anciennes tables temporaires des visites
+    // (maintenant stockees sous forme de fichiers)
+    if (upgrade_vers(1.904, $version_installee, $version_cible)) {
+        spip_query("DROP TABLE IF EXISTS spip_visites_temp");
+        spip_query("DROP TABLE IF EXISTS spip_referers_temp");
+        maj_version(1.904);
+    }
 
-	// fusion des 10 tables index en une seule
-	// pour fonctions futures evoluees du moteur de recherche
-	if (upgrade_vers(1.905, $version_installee, $version_cible)) {
-		// agrandir le champ "valeur" de spip_meta pour pouvoir y stocker
-		// des choses plus sympa
-		spip_query("ALTER TABLE spip_meta DROP INDEX `valeur`");
-		spip_query("ALTER TABLE spip_meta CHANGE `valeur` `valeur` TEXT");
-		// table des correspondances table->id_table
-		$liste_tables = array();
-		$liste_tables[1]='spip_articles';
-		$liste_tables[2]='spip_auteurs';
-		$liste_tables[3]='spip_breves';
-		$liste_tables[4]='spip_documents';
-		$liste_tables[5]='spip_forum';
-		$liste_tables[6]='spip_mots';
-		$liste_tables[7]='spip_rubriques';
-		$liste_tables[8]='spip_signatures';
-		$liste_tables[9]='spip_syndic';
+    // fusion des 10 tables index en une seule
+    // pour fonctions futures evoluees du moteur de recherche
+    if (upgrade_vers(1.905, $version_installee, $version_cible)) {
+        // agrandir le champ "valeur" de spip_meta pour pouvoir y stocker
+        // des choses plus sympa
+        spip_query("ALTER TABLE spip_meta DROP INDEX `valeur`");
+        spip_query("ALTER TABLE spip_meta CHANGE `valeur` `valeur` TEXT");
+        // table des correspondances table->id_table
+        $liste_tables = array();
+        $liste_tables[1]='spip_articles';
+        $liste_tables[2]='spip_auteurs';
+        $liste_tables[3]='spip_breves';
+        $liste_tables[4]='spip_documents';
+        $liste_tables[5]='spip_forum';
+        $liste_tables[6]='spip_mots';
+        $liste_tables[7]='spip_rubriques';
+        $liste_tables[8]='spip_signatures';
+        $liste_tables[9]='spip_syndic';
 
-		ecrire_meta('index_table', serialize($liste_tables));
+        ecrire_meta('index_table', serialize($liste_tables));
 
 ## devenu inutile car suppression totale de l'indexation
 /*
@@ -96,207 +96,207 @@ 
 		spip_query("INSERT INTO spip_index (`hash`,`points`,`id_objet`,`id_table`) SELECT `hash`,`points`,`id_syndic` as id_objet,'9' as `id_table FROM spip_index_syndic");
 		spip_query("DROP TABLE IF EXISTS spip_index_syndic");
 */
-		maj_version(1.905);
-	}
+        maj_version(1.905);
+    }
 
 
-	// cette table est desormais geree par le plugin "podcast_client", on la
-	// supprime si le plugin n'est pas active ; risque inherent a l'utilisation
-	// de versions alpha :-)
-	if (upgrade_vers(1.906, $version_installee, $version_cible)) {
-		if (!@in_array('podcast_client', $GLOBALS['plugins'])) {
-			spip_query("DROP TABLE spip_documents_syndic");
-		}
-		maj_version(1.906);
-	}
-	if (upgrade_vers(1.907, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_forum ADD INDEX `idx` (`idx`)");
-		maj_version(1.907);
-	}
-	// Oups ! on stockait les tags de syndication sous la forme rel="category"
-	// au lieu de rel="directory" - http://microformats.org/wiki/rel-directory
-	if (upgrade_vers(1.908, $version_installee, $version_cible)) {
-		spip_query("UPDATE spip_syndic_articles SET `tags` = REPLACE(`tags`, 'rel=\"category\">', 'rel=\"directory\">') WHERE `tags` like '%category%'");
-		maj_version(1.908);
-	}
-	if (upgrade_vers(1.909, $version_installee, $version_cible)) {
-		spip_query("ALTER IGNORE TABLE spip_mots_articles ADD PRIMARY KEY (`id_article`, `id_mot`)");
-		spip_query("ALTER IGNORE TABLE spip_mots_breves ADD PRIMARY KEY (`id_breve`, `id_mot`)");
-		spip_query("ALTER IGNORE TABLE spip_mots_rubriques ADD PRIMARY KEY (`id_rubrique`, `id_mot`)");
-		spip_query("ALTER IGNORE TABLE spip_mots_syndic ADD PRIMARY KEY (`id_syndic`, `id_mot`)");
-		spip_query("ALTER IGNORE TABLE spip_mots_documents ADD PRIMARY KEY (`id_document`, `id_mot`)");
-		spip_query("ALTER IGNORE TABLE spip_mots_forum ADD PRIMARY KEY (`id_forum`, `id_mot`)");
-		maj_version(1.909);
-	}
+    // cette table est desormais geree par le plugin "podcast_client", on la
+    // supprime si le plugin n'est pas active ; risque inherent a l'utilisation
+    // de versions alpha :-)
+    if (upgrade_vers(1.906, $version_installee, $version_cible)) {
+        if (!@in_array('podcast_client', $GLOBALS['plugins'])) {
+            spip_query("DROP TABLE spip_documents_syndic");
+        }
+        maj_version(1.906);
+    }
+    if (upgrade_vers(1.907, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_forum ADD INDEX `idx` (`idx`)");
+        maj_version(1.907);
+    }
+    // Oups ! on stockait les tags de syndication sous la forme rel="category"
+    // au lieu de rel="directory" - http://microformats.org/wiki/rel-directory
+    if (upgrade_vers(1.908, $version_installee, $version_cible)) {
+        spip_query("UPDATE spip_syndic_articles SET `tags` = REPLACE(`tags`, 'rel=\"category\">', 'rel=\"directory\">') WHERE `tags` like '%category%'");
+        maj_version(1.908);
+    }
+    if (upgrade_vers(1.909, $version_installee, $version_cible)) {
+        spip_query("ALTER IGNORE TABLE spip_mots_articles ADD PRIMARY KEY (`id_article`, `id_mot`)");
+        spip_query("ALTER IGNORE TABLE spip_mots_breves ADD PRIMARY KEY (`id_breve`, `id_mot`)");
+        spip_query("ALTER IGNORE TABLE spip_mots_rubriques ADD PRIMARY KEY (`id_rubrique`, `id_mot`)");
+        spip_query("ALTER IGNORE TABLE spip_mots_syndic ADD PRIMARY KEY (`id_syndic`, `id_mot`)");
+        spip_query("ALTER IGNORE TABLE spip_mots_documents ADD PRIMARY KEY (`id_document`, `id_mot`)");
+        spip_query("ALTER IGNORE TABLE spip_mots_forum ADD PRIMARY KEY (`id_forum`, `id_mot`)");
+        maj_version(1.909);
+    }
 
-	if (upgrade_vers(1.910, $version_installee, $version_cible)) {
-		spip_query("ALTER IGNORE TABLE spip_auteurs_articles ADD PRIMARY KEY (`id_auteur`, `id_article`)");
-		spip_query("ALTER IGNORE TABLE spip_auteurs_rubriques ADD PRIMARY KEY (`id_auteur`, `id_rubrique`)");
-		spip_query("ALTER IGNORE TABLE spip_auteurs_messages ADD PRIMARY KEY (`id_auteur`, `id_message`)");
-		maj_version(1.910);
-	}
+    if (upgrade_vers(1.910, $version_installee, $version_cible)) {
+        spip_query("ALTER IGNORE TABLE spip_auteurs_articles ADD PRIMARY KEY (`id_auteur`, `id_article`)");
+        spip_query("ALTER IGNORE TABLE spip_auteurs_rubriques ADD PRIMARY KEY (`id_auteur`, `id_rubrique`)");
+        spip_query("ALTER IGNORE TABLE spip_auteurs_messages ADD PRIMARY KEY (`id_auteur`, `id_message`)");
+        maj_version(1.910);
+    }
 
-	if (upgrade_vers(1.911, $version_installee, $version_cible)) {
+    if (upgrade_vers(1.911, $version_installee, $version_cible)) {
 
-		spip_query("ALTER IGNORE TABLE spip_auteurs_articles DROP INDEX `id_auteur`");
-		spip_query("ALTER IGNORE TABLE spip_auteurs_rubriques DROP INDEX `id_auteur`");
-		spip_query("ALTER IGNORE TABLE spip_auteurs_messages DROP INDEX `id_auteur`");
-		spip_query("ALTER IGNORE TABLE spip_mots_articles DROP INDEX `id_article`");
-		spip_query("ALTER IGNORE TABLE spip_mots_breves DROP INDEX `id_breve`");
-		spip_query("ALTER IGNORE TABLE spip_mots_rubriques DROP INDEX `id_rubrique`");
-		spip_query("ALTER IGNORE TABLE spip_mots_syndic DROP INDEX `id_syndic`");
-		spip_query("ALTER IGNORE TABLE spip_mots_forum DROP INDEX `id_forum`");
-		spip_query("ALTER IGNORE TABLE spip_mots_documents DROP INDEX `id_document`");
+        spip_query("ALTER IGNORE TABLE spip_auteurs_articles DROP INDEX `id_auteur`");
+        spip_query("ALTER IGNORE TABLE spip_auteurs_rubriques DROP INDEX `id_auteur`");
+        spip_query("ALTER IGNORE TABLE spip_auteurs_messages DROP INDEX `id_auteur`");
+        spip_query("ALTER IGNORE TABLE spip_mots_articles DROP INDEX `id_article`");
+        spip_query("ALTER IGNORE TABLE spip_mots_breves DROP INDEX `id_breve`");
+        spip_query("ALTER IGNORE TABLE spip_mots_rubriques DROP INDEX `id_rubrique`");
+        spip_query("ALTER IGNORE TABLE spip_mots_syndic DROP INDEX `id_syndic`");
+        spip_query("ALTER IGNORE TABLE spip_mots_forum DROP INDEX `id_forum`");
+        spip_query("ALTER IGNORE TABLE spip_mots_documents DROP INDEX `id_document`");
 # 18 juillet 2007: table depreciee
 #		spip_query("ALTER IGNORE TABLE spip_caches DROP	INDEX fichier");
-		maj_version(1.911);
-	}
+        maj_version(1.911);
+    }
 
-	// Le logo du site n'est plus le logo par defaut des rubriques
-	// mais pour assurer la compatibilite ascendante, on le duplique
-	if (upgrade_vers(1.912, $version_installee, $version_cible)) {
-		@copy(_DIR_LOGOS.'rubon0.gif', _DIR_LOGOS.'siteon0.gif');
-		@copy(_DIR_LOGOS.'ruboff0.gif', _DIR_LOGOS.'siteoff0.gif');
-		@copy(_DIR_LOGOS.'rubon0.jpg', _DIR_LOGOS.'siteon0.jpg');
-		@copy(_DIR_LOGOS.'ruboff0.jpg', _DIR_LOGOS.'siteoff0.jpg');
-		@copy(_DIR_LOGOS.'rubon0.png', _DIR_LOGOS.'siteon0.png');
-		@copy(_DIR_LOGOS.'ruboff0.png', _DIR_LOGOS.'siteoff0.png');
-		maj_version(1.912);
-	}
+    // Le logo du site n'est plus le logo par defaut des rubriques
+    // mais pour assurer la compatibilite ascendante, on le duplique
+    if (upgrade_vers(1.912, $version_installee, $version_cible)) {
+        @copy(_DIR_LOGOS.'rubon0.gif', _DIR_LOGOS.'siteon0.gif');
+        @copy(_DIR_LOGOS.'ruboff0.gif', _DIR_LOGOS.'siteoff0.gif');
+        @copy(_DIR_LOGOS.'rubon0.jpg', _DIR_LOGOS.'siteon0.jpg');
+        @copy(_DIR_LOGOS.'ruboff0.jpg', _DIR_LOGOS.'siteoff0.jpg');
+        @copy(_DIR_LOGOS.'rubon0.png', _DIR_LOGOS.'siteon0.png');
+        @copy(_DIR_LOGOS.'ruboff0.png', _DIR_LOGOS.'siteoff0.png');
+        maj_version(1.912);
+    }
 
-	// suppression de auteur_modif qui n'est plus utilise nulle part
-	if (upgrade_vers(1.913, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_articles DROP `auteur_modif`");
-		maj_version(1.913);
-	}
+    // suppression de auteur_modif qui n'est plus utilise nulle part
+    if (upgrade_vers(1.913, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_articles DROP `auteur_modif`");
+        maj_version(1.913);
+    }
 
-	// Ajout de SVG
-	if (upgrade_vers(1.914, $version_installee, $version_cible)) {
-		spip_query("INSERT IGNORE INTO spip_types_documents (`extension`, `titre`, `inclus`) VALUES ('svg', 'Scalable Vector Graphics', 'embed')");
-		spip_query("UPDATE spip_types_documents SET `mime_type`='image/svg+xml' WHERE `extension`='svg'");
-		maj_version(1.914);
-	}
+    // Ajout de SVG
+    if (upgrade_vers(1.914, $version_installee, $version_cible)) {
+        spip_query("INSERT IGNORE INTO spip_types_documents (`extension`, `titre`, `inclus`) VALUES ('svg', 'Scalable Vector Graphics', 'embed')");
+        spip_query("UPDATE spip_types_documents SET `mime_type`='image/svg+xml' WHERE `extension`='svg'");
+        maj_version(1.914);
+    }
 
-	// Ajout de plein de type mime
-	if (upgrade_vers(1.915, $version_installee, $version_cible)) {
-		maj_version(1.915);
-	}
-	// refaire l'upgrade 1.905 qui a pu foirer en partie a cause de la requete ALTER sur spip_meta
-	if (upgrade_vers(1.916, $version_installee, $version_cible)) {
-		// agrandir le champ "valeur" de spip_meta pour pouvoir y stocker
-		// des choses plus sympa
-		spip_query("ALTER TABLE spip_meta DROP INDEX `valeur`");
-		spip_query("ALTER TABLE spip_meta CHANGE `valeur` `valeur` TEXT");
+    // Ajout de plein de type mime
+    if (upgrade_vers(1.915, $version_installee, $version_cible)) {
+        maj_version(1.915);
+    }
+    // refaire l'upgrade 1.905 qui a pu foirer en partie a cause de la requete ALTER sur spip_meta
+    if (upgrade_vers(1.916, $version_installee, $version_cible)) {
+        // agrandir le champ "valeur" de spip_meta pour pouvoir y stocker
+        // des choses plus sympa
+        spip_query("ALTER TABLE spip_meta DROP INDEX `valeur`");
+        spip_query("ALTER TABLE spip_meta CHANGE `valeur` `valeur` TEXT");
 #8/08/07  plus d'indexation dans le core
-		//include_spip('inc/indexation'); 
-		//update_index_tables();
-		maj_version(1.916);
-	}
-	if (upgrade_vers(1.917, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_documents DROP `inclus`");
-		maj_version(1.917);
-	}
+        //include_spip('inc/indexation'); 
+        //update_index_tables();
+        maj_version(1.916);
+    }
+    if (upgrade_vers(1.917, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_documents DROP `inclus`");
+        maj_version(1.917);
+    }
 
-	// Permettre d'enregistrer un numero IP dans les revisions d'articles
-	// a la place de l'id_auteur
-	if (upgrade_vers(1.918, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_versions CHANGE `id_auteur` `id_auteur` VARCHAR(23)");
-		maj_version(1.918);
-	}
+    // Permettre d'enregistrer un numero IP dans les revisions d'articles
+    // a la place de l'id_auteur
+    if (upgrade_vers(1.918, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_versions CHANGE `id_auteur` `id_auteur` VARCHAR(23)");
+        maj_version(1.918);
+    }
 
-	if (upgrade_vers(1.919, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_ajax_fonc DROP `id_auteur`");
-		maj_version('1.919');
-	}
+    if (upgrade_vers(1.919, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_ajax_fonc DROP `id_auteur`");
+        maj_version('1.919');
+    }
 
-	if (upgrade_vers(1.920, $version_installee, $version_cible)) {
-		spip_query("ALTER IGNORE TABLE spip_documents_articles ADD PRIMARY KEY (`id_article`, `id_document`)");
-		spip_query("ALTER IGNORE TABLE spip_documents_breves ADD PRIMARY KEY (`id_breve`, `id_document`)");
-		spip_query("ALTER IGNORE TABLE spip_documents_rubriques ADD PRIMARY KEY (`id_rubrique`, `id_document`)");
-		spip_query("ALTER IGNORE TABLE spip_documents_articles DROP INDEX `id_article`");
-		spip_query("ALTER IGNORE TABLE spip_documents_breves DROP INDEX `id_breve`");
-		spip_query("ALTER IGNORE TABLE spip_documents_rubriques DROP INDEX `id_rubrique`");
-		maj_version('1.920');
-	}
-	if (upgrade_vers(1.922, $version_installee, $version_cible)) {
-		spip_query("ALTER TABLE spip_meta ADD `impt` ENUM('non', 'oui') DEFAULT 'oui' NOT NULL AFTER `valeur`");
-		$meta_serveur = array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
-		'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
-		foreach($meta_serveur as $nom)
-			spip_query("UPDATE spip_meta SET `impt`='non' WHERE `nom`="._q($nom));
-		maj_version('1.922');
-	}
-	if (upgrade_vers(1.923, $version_installee, $version_cible)) {
-		if (isset($GLOBALS['meta']['IMPORT_tables_noimport'])){
-			$IMPORT_tables_noimport = unserialize($GLOBALS['meta']['IMPORT_tables_noimport']);
-			foreach ($IMPORT_tables_noimport as $key=>$table)
-				if ($table=='spip_meta') unset($IMPORT_tables_noimport[$key]);
-			ecrire_meta('IMPORT_tables_noimport',serialize($IMPORT_tables_noimport),'non');
-		}
-		maj_version('1.923');
-	}
+    if (upgrade_vers(1.920, $version_installee, $version_cible)) {
+        spip_query("ALTER IGNORE TABLE spip_documents_articles ADD PRIMARY KEY (`id_article`, `id_document`)");
+        spip_query("ALTER IGNORE TABLE spip_documents_breves ADD PRIMARY KEY (`id_breve`, `id_document`)");
+        spip_query("ALTER IGNORE TABLE spip_documents_rubriques ADD PRIMARY KEY (`id_rubrique`, `id_document`)");
+        spip_query("ALTER IGNORE TABLE spip_documents_articles DROP INDEX `id_article`");
+        spip_query("ALTER IGNORE TABLE spip_documents_breves DROP INDEX `id_breve`");
+        spip_query("ALTER IGNORE TABLE spip_documents_rubriques DROP INDEX `id_rubrique`");
+        maj_version('1.920');
+    }
+    if (upgrade_vers(1.922, $version_installee, $version_cible)) {
+        spip_query("ALTER TABLE spip_meta ADD `impt` ENUM('non', 'oui') DEFAULT 'oui' NOT NULL AFTER `valeur`");
+        $meta_serveur = array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
+        'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
+        foreach($meta_serveur as $nom)
+            spip_query("UPDATE spip_meta SET `impt`='non' WHERE `nom`="._q($nom));
+        maj_version('1.922');
+    }
+    if (upgrade_vers(1.923, $version_installee, $version_cible)) {
+        if (isset($GLOBALS['meta']['IMPORT_tables_noimport'])){
+            $IMPORT_tables_noimport = unserialize($GLOBALS['meta']['IMPORT_tables_noimport']);
+            foreach ($IMPORT_tables_noimport as $key=>$table)
+                if ($table=='spip_meta') unset($IMPORT_tables_noimport[$key]);
+            ecrire_meta('IMPORT_tables_noimport',serialize($IMPORT_tables_noimport),'non');
+        }
+        maj_version('1.923');
+    }
 
-	if (upgrade_vers(1.924, $version_installee, $version_cible)) {
-		spip_query('DROP TABLE spip_ajax_fonc');
-		maj_version('1.924');
-	}
+    if (upgrade_vers(1.924, $version_installee, $version_cible)) {
+        spip_query('DROP TABLE spip_ajax_fonc');
+        maj_version('1.924');
+    }
 
-	if (upgrade_vers(1.925, $version_installee, $version_cible)) {
-		include_spip('inc/flock');
-		/* deplacement des sessions */
-		$f_session = preg_files('data', 'session_');
-		$repertoire = _DIR_SESSIONS;
-		if(!@file_exists($repertoire)) {
-			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
-			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
-		}
-		foreach($f_session as $f) {
-			$d = basename($f);
-			@copy($f, $repertoire.$d);
-		}
-		/* deplacement des visites */
-		$f_visites = preg_files('data/visites');
-		$repertoire = sous_repertoire(_DIR_TMP, 'visites');
-		foreach($f_visites as $f) {
-			$d = basename($f);
-			@copy($f, $repertoire.$d);
-		}
-		/* deplacement des upload */
-		$auteurs = array();
-		$req = spip_query("SELECT `login` FROM spip_auteurs WHERE `statut` = '0minirezo'");
-		while($row = sql_fetch($req))
-			$auteurs[] = $row['login']; 
-		$f_upload = preg_files('upload', -1, 10000, $auteurs);
-		$repertoire = _DIR_TRANSFERT;
-		if(!@file_exists($repertoire)) {
-			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
-			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
-		}
-		foreach($auteurs as $login) {
-			if(is_dir('upload/'.$login))
-				$sous_repertoire = sous_repertoire(_DIR_TRANSFERT, $login);
-		}
-		foreach($f_upload as $f) {
-			@copy($f, _DIR_TMP.$f);
-		}
-		/* deplacement des dumps */
-		$f_session = preg_files('data', 'dump');
-		$repertoire = _DIR_DUMP;
-		if(!@file_exists($repertoire)) {
-			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
-			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
-		}
-		foreach($f_session as $f) {
-			$d = basename($f);
-			@copy($f, $repertoire.$d);
-		}
-		maj_version('1.925');
-	}
-	// Ajout de MP4
-	if (upgrade_vers(1.926, $version_installee, $version_cible)) {
-		spip_query("INSERT IGNORE INTO spip_types_documents (`extension`, `titre`, `inclus`) VALUES ('mp4', 'MPEG4', 'embed')");
-		spip_query("UPDATE spip_types_documents SET `mime_type`='application/mp4' WHERE `extension`='mp4'");
-		maj_version('1.926');
-	}
+    if (upgrade_vers(1.925, $version_installee, $version_cible)) {
+        include_spip('inc/flock');
+        /* deplacement des sessions */
+        $f_session = preg_files('data', 'session_');
+        $repertoire = _DIR_SESSIONS;
+        if(!@file_exists($repertoire)) {
+            $repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
+            $repertoire = sous_repertoire(_DIR_TMP, $repertoire);
+        }
+        foreach($f_session as $f) {
+            $d = basename($f);
+            @copy($f, $repertoire.$d);
+        }
+        /* deplacement des visites */
+        $f_visites = preg_files('data/visites');
+        $repertoire = sous_repertoire(_DIR_TMP, 'visites');
+        foreach($f_visites as $f) {
+            $d = basename($f);
+            @copy($f, $repertoire.$d);
+        }
+        /* deplacement des upload */
+        $auteurs = array();
+        $req = spip_query("SELECT `login` FROM spip_auteurs WHERE `statut` = '0minirezo'");
+        while($row = sql_fetch($req))
+            $auteurs[] = $row['login']; 
+        $f_upload = preg_files('upload', -1, 10000, $auteurs);
+        $repertoire = _DIR_TRANSFERT;
+        if(!@file_exists($repertoire)) {
+            $repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
+            $repertoire = sous_repertoire(_DIR_TMP, $repertoire);
+        }
+        foreach($auteurs as $login) {
+            if(is_dir('upload/'.$login))
+                $sous_repertoire = sous_repertoire(_DIR_TRANSFERT, $login);
+        }
+        foreach($f_upload as $f) {
+            @copy($f, _DIR_TMP.$f);
+        }
+        /* deplacement des dumps */
+        $f_session = preg_files('data', 'dump');
+        $repertoire = _DIR_DUMP;
+        if(!@file_exists($repertoire)) {
+            $repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
+            $repertoire = sous_repertoire(_DIR_TMP, $repertoire);
+        }
+        foreach($f_session as $f) {
+            $d = basename($f);
+            @copy($f, $repertoire.$d);
+        }
+        maj_version('1.925');
+    }
+    // Ajout de MP4
+    if (upgrade_vers(1.926, $version_installee, $version_cible)) {
+        spip_query("INSERT IGNORE INTO spip_types_documents (`extension`, `titre`, `inclus`) VALUES ('mp4', 'MPEG4', 'embed')");
+        spip_query("UPDATE spip_types_documents SET `mime_type`='application/mp4' WHERE `extension`='mp4'");
+        maj_version('1.926');
+    }
 }
 ?>

Spacing +25 added lines, -25 removed lines

@@ -55,15 +55,15 @@ 
 		spip_query("ALTER TABLE spip_meta CHANGE `valeur` `valeur` TEXT");
 		// table des correspondances table->id_table
 		$liste_tables = array();
-		$liste_tables[1]='spip_articles';
-		$liste_tables[2]='spip_auteurs';
-		$liste_tables[3]='spip_breves';
-		$liste_tables[4]='spip_documents';
-		$liste_tables[5]='spip_forum';
-		$liste_tables[6]='spip_mots';
-		$liste_tables[7]='spip_rubriques';
-		$liste_tables[8]='spip_signatures';
-		$liste_tables[9]='spip_syndic';
+		$liste_tables[1] = 'spip_articles';
+		$liste_tables[2] = 'spip_auteurs';
+		$liste_tables[3] = 'spip_breves';
+		$liste_tables[4] = 'spip_documents';
+		$liste_tables[5] = 'spip_forum';
+		$liste_tables[6] = 'spip_mots';
+		$liste_tables[7] = 'spip_rubriques';
+		$liste_tables[8] = 'spip_signatures';
+		$liste_tables[9] = 'spip_syndic';
 
 		ecrire_meta('index_table', serialize($liste_tables));
 
@@ -220,18 +220,18 @@ 
 	}
 	if (upgrade_vers(1.922, $version_installee, $version_cible)) {
 		spip_query("ALTER TABLE spip_meta ADD `impt` ENUM('non', 'oui') DEFAULT 'oui' NOT NULL AFTER `valeur`");
-		$meta_serveur = array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
-		'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
-		foreach($meta_serveur as $nom)
+		$meta_serveur = array('version_installee', 'adresse_site', 'alea_ephemere_ancien', 'alea_ephemere', 'alea_ephemere_date', 'langue_site', 'langues_proposees', 'date_calcul_rubriques', 'derniere_modif', 'optimiser_table', 'drapeau_edition', 'creer_preview', 'taille_preview', 'creer_htpasswd', 'creer_htaccess', 'gd_formats_read', 'gd_formats',
+		'netpbm_formats', 'formats_graphiques', 'image_process', 'plugin_header', 'plugin');
+		foreach ($meta_serveur as $nom)
 			spip_query("UPDATE spip_meta SET `impt`='non' WHERE `nom`="._q($nom));
 		maj_version('1.922');
 	}
 	if (upgrade_vers(1.923, $version_installee, $version_cible)) {
-		if (isset($GLOBALS['meta']['IMPORT_tables_noimport'])){
+		if (isset($GLOBALS['meta']['IMPORT_tables_noimport'])) {
 			$IMPORT_tables_noimport = unserialize($GLOBALS['meta']['IMPORT_tables_noimport']);
 			foreach ($IMPORT_tables_noimport as $key=>$table)
-				if ($table=='spip_meta') unset($IMPORT_tables_noimport[$key]);
-			ecrire_meta('IMPORT_tables_noimport',serialize($IMPORT_tables_noimport),'non');
+				if ($table == 'spip_meta') unset($IMPORT_tables_noimport[$key]);
+			ecrire_meta('IMPORT_tables_noimport', serialize($IMPORT_tables_noimport), 'non');
 		}
 		maj_version('1.923');
 	}
@@ -246,47 +246,47 @@ 
 		/* deplacement des sessions */
 		$f_session = preg_files('data', 'session_');
 		$repertoire = _DIR_SESSIONS;
-		if(!@file_exists($repertoire)) {
+		if (!@file_exists($repertoire)) {
 			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
 			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
 		}
-		foreach($f_session as $f) {
+		foreach ($f_session as $f) {
 			$d = basename($f);
 			@copy($f, $repertoire.$d);
 		}
 		/* deplacement des visites */
 		$f_visites = preg_files('data/visites');
 		$repertoire = sous_repertoire(_DIR_TMP, 'visites');
-		foreach($f_visites as $f) {
+		foreach ($f_visites as $f) {
 			$d = basename($f);
 			@copy($f, $repertoire.$d);
 		}
 		/* deplacement des upload */
 		$auteurs = array();
 		$req = spip_query("SELECT `login` FROM spip_auteurs WHERE `statut` = '0minirezo'");
-		while($row = sql_fetch($req))
+		while ($row = sql_fetch($req))
 			$auteurs[] = $row['login']; 
 		$f_upload = preg_files('upload', -1, 10000, $auteurs);
 		$repertoire = _DIR_TRANSFERT;
-		if(!@file_exists($repertoire)) {
+		if (!@file_exists($repertoire)) {
 			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
 			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
 		}
-		foreach($auteurs as $login) {
-			if(is_dir('upload/'.$login))
+		foreach ($auteurs as $login) {
+			if (is_dir('upload/'.$login))
 				$sous_repertoire = sous_repertoire(_DIR_TRANSFERT, $login);
 		}
-		foreach($f_upload as $f) {
+		foreach ($f_upload as $f) {
 			@copy($f, _DIR_TMP.$f);
 		}
 		/* deplacement des dumps */
 		$f_session = preg_files('data', 'dump');
 		$repertoire = _DIR_DUMP;
-		if(!@file_exists($repertoire)) {
+		if (!@file_exists($repertoire)) {
 			$repertoire = preg_replace(','._DIR_TMP.',', '', $repertoire);
 			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
 		}
-		foreach($f_session as $f) {
+		foreach ($f_session as $f) {
 			$d = basename($f);
 			@copy($f, $repertoire.$d);
 		}

Braces +15 added lines, -9 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 function v019_pre193($version_installee, $version_cible)
 {
@@ -222,15 +224,17 @@ 
 		spip_query("ALTER TABLE spip_meta ADD `impt` ENUM('non', 'oui') DEFAULT 'oui' NOT NULL AFTER `valeur`");
 		$meta_serveur = array('version_installee','adresse_site','alea_ephemere_ancien','alea_ephemere','alea_ephemere_date','langue_site','langues_proposees','date_calcul_rubriques','derniere_modif','optimiser_table','drapeau_edition','creer_preview','taille_preview','creer_htpasswd','creer_htaccess','gd_formats_read','gd_formats',
 		'netpbm_formats','formats_graphiques','image_process','plugin_header','plugin');
-		foreach($meta_serveur as $nom)
-			spip_query("UPDATE spip_meta SET `impt`='non' WHERE `nom`="._q($nom));
+		foreach($meta_serveur as $nom) {
+					spip_query("UPDATE spip_meta SET `impt`='non' WHERE `nom`="._q($nom));
+		}
 		maj_version('1.922');
 	}
 	if (upgrade_vers(1.923, $version_installee, $version_cible)) {
 		if (isset($GLOBALS['meta']['IMPORT_tables_noimport'])){
 			$IMPORT_tables_noimport = unserialize($GLOBALS['meta']['IMPORT_tables_noimport']);
-			foreach ($IMPORT_tables_noimport as $key=>$table)
-				if ($table=='spip_meta') unset($IMPORT_tables_noimport[$key]);
+			foreach ($IMPORT_tables_noimport as $key=>$table) {
+							if ($table=='spip_meta') unset($IMPORT_tables_noimport[$key]);
+			}
 			ecrire_meta('IMPORT_tables_noimport',serialize($IMPORT_tables_noimport),'non');
 		}
 		maj_version('1.923');
@@ -264,8 +268,9 @@ 
 		/* deplacement des upload */
 		$auteurs = array();
 		$req = spip_query("SELECT `login` FROM spip_auteurs WHERE `statut` = '0minirezo'");
-		while($row = sql_fetch($req))
-			$auteurs[] = $row['login']; 
+		while($row = sql_fetch($req)) {
+					$auteurs[] = $row['login'];
+		}
 		$f_upload = preg_files('upload', -1, 10000, $auteurs);
 		$repertoire = _DIR_TRANSFERT;
 		if(!@file_exists($repertoire)) {
@@ -273,8 +278,9 @@ 
 			$repertoire = sous_repertoire(_DIR_TMP, $repertoire);
 		}
 		foreach($auteurs as $login) {
-			if(is_dir('upload/'.$login))
-				$sous_repertoire = sous_repertoire(_DIR_TRANSFERT, $login);
+			if(is_dir('upload/'.$login)) {
+							$sous_repertoire = sous_repertoire(_DIR_TRANSFERT, $login);
+			}
 		}
 		foreach($f_upload as $f) {
 			@copy($f, _DIR_TMP.$f);

ecrire/plugins/afficher_liste.php

Doc Comments +4 added lines

@@ -51,6 +51,10 @@
 
 
 // http://doc.spip.org/@affiche_block_initiale
+/**
+ * @param string $initiale
+ * @param string $block
+ */
 function affiche_block_initiale($initiale,$block,$block_actif){
 	if (strlen($block)){
 		return "<li class='item'>"

Indentation +62 added lines, -62 removed lines

@@ -26,78 +26,78 @@
  * @return string
  */
 function plugins_afficher_liste_dist($url_page,$liste_plugins, $liste_plugins_checked, $liste_plugins_actifs, $dir_plugins=_DIR_PLUGINS,$afficher_un = 'afficher_plugin'){
-	$get_infos = charger_fonction('get_infos','plugins');
-	$ligne_plug = charger_fonction($afficher_un,'plugins');
+    $get_infos = charger_fonction('get_infos','plugins');
+    $ligne_plug = charger_fonction($afficher_un,'plugins');
 
-	$all_infos = $get_infos($liste_plugins, false, $dir_plugins);
+    $all_infos = $get_infos($liste_plugins, false, $dir_plugins);
 
-	$all_infos = pipeline('filtrer_liste_plugins',
-	                     array(
-											  'args'=>array(
-		                      'liste_plugins'=>$liste_plugins,
-		                      'liste_plugins_checked'=>$liste_plugins_checked,
-		                      'liste_plugins_actifs'=>$liste_plugins_actifs,
-		                      'dir_plugins'=>$dir_plugins),
-												'data'=>$all_infos
-	                     )
-							);
+    $all_infos = pipeline('filtrer_liste_plugins',
+                            array(
+                                                'args'=>array(
+                                'liste_plugins'=>$liste_plugins,
+                                'liste_plugins_checked'=>$liste_plugins_checked,
+                                'liste_plugins_actifs'=>$liste_plugins_actifs,
+                                'dir_plugins'=>$dir_plugins),
+                                                'data'=>$all_infos
+                            )
+                            );
 
-	$liste_plugins = array_flip($liste_plugins);
-	foreach($liste_plugins as $chemin => $v) {
-		// des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
-		if (isset($all_infos[$chemin])){
-			$liste_plugins[$chemin] = strtoupper(trim(typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
-		}
-		else {
-			unset($liste_plugins[$chemin]);
-		}
-	}
-	asort($liste_plugins);
-	$exposed = urldecode(_request('plugin'));
+    $liste_plugins = array_flip($liste_plugins);
+    foreach($liste_plugins as $chemin => $v) {
+        // des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
+        if (isset($all_infos[$chemin])){
+            $liste_plugins[$chemin] = strtoupper(trim(typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
+        }
+        else {
+            unset($liste_plugins[$chemin]);
+        }
+    }
+    asort($liste_plugins);
+    $exposed = urldecode(_request('plugin'));
 
-	$block_par_lettre = false;//count($liste_plugins)>10;
-	$fast_liste_plugins_actifs = array();
-	$fast_liste_plugins_checked = array();
-	if (is_array($liste_plugins_actifs))
-		$fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
-	if (is_array($liste_plugins_checked))
-		$fast_liste_plugins_checked = array_flip($liste_plugins_checked);
+    $block_par_lettre = false;//count($liste_plugins)>10;
+    $fast_liste_plugins_actifs = array();
+    $fast_liste_plugins_checked = array();
+    if (is_array($liste_plugins_actifs))
+        $fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
+    if (is_array($liste_plugins_checked))
+        $fast_liste_plugins_checked = array_flip($liste_plugins_checked);
 
-	$res = '';
-	$block = '';
-	$initiale = '';
-	$block_actif = false;
-	foreach($liste_plugins as $plug => $nom){
-		if (($i=substr($nom,0,1))!==$initiale){
-			$res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
-			$initiale = $i;
-			$block = '';
-			$block_actif = false;
-		}
-		// le rep suivant
-		$actif = isset($fast_liste_plugins_actifs[$plug]);
-		$checked = isset($fast_liste_plugins_checked[$plug]);
-		$block_actif = $block_actif | $actif;
-		$expose = ($exposed AND ($exposed==$plug OR $exposed==$dir_plugins . $plug OR $exposed==substr($dir_plugins,strlen(_DIR_RACINE)) . $plug));
-		$block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, "item", $dir_plugins)."\n";
-	}
-	$res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
-	$class = basename($dir_plugins);
-	return $res ? "<ul class='liste-items plugins $class'>$res</ul>" : "";
+    $res = '';
+    $block = '';
+    $initiale = '';
+    $block_actif = false;
+    foreach($liste_plugins as $plug => $nom){
+        if (($i=substr($nom,0,1))!==$initiale){
+            $res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
+            $initiale = $i;
+            $block = '';
+            $block_actif = false;
+        }
+        // le rep suivant
+        $actif = isset($fast_liste_plugins_actifs[$plug]);
+        $checked = isset($fast_liste_plugins_checked[$plug]);
+        $block_actif = $block_actif | $actif;
+        $expose = ($exposed AND ($exposed==$plug OR $exposed==$dir_plugins . $plug OR $exposed==substr($dir_plugins,strlen(_DIR_RACINE)) . $plug));
+        $block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, "item", $dir_plugins)."\n";
+    }
+    $res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
+    $class = basename($dir_plugins);
+    return $res ? "<ul class='liste-items plugins $class'>$res</ul>" : "";
 }
 
 
 // http://doc.spip.org/@affiche_block_initiale
 function affiche_block_initiale($initiale,$block,$block_actif){
-	if (strlen($block)){
-		return "<li class='item'>"
-		  . bouton_block_depliable($initiale,$block_actif?true:false)
-		  . debut_block_depliable($block_actif)
-		  . "<ul>$block</ul>"
-		  . fin_block()
-		  . "</li>";
-	}
-	return "";
+    if (strlen($block)){
+        return "<li class='item'>"
+            . bouton_block_depliable($initiale,$block_actif?true:false)
+            . debut_block_depliable($block_actif)
+            . "<ul>$block</ul>"
+            . fin_block()
+            . "</li>";
+    }
+    return "";
 }
 
 ?>

Braces +10 added lines, -7 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 include_spip('inc/charsets');
 
 /**
@@ -47,8 +49,7 @@ 
 		// des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
 		if (isset($all_infos[$chemin])){
 			$liste_plugins[$chemin] = strtoupper(trim(typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
-		}
-		else {
+		} else {
 			unset($liste_plugins[$chemin]);
 		}
 	}
@@ -58,10 +59,12 @@ 
 	$block_par_lettre = false;//count($liste_plugins)>10;
 	$fast_liste_plugins_actifs = array();
 	$fast_liste_plugins_checked = array();
-	if (is_array($liste_plugins_actifs))
-		$fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
-	if (is_array($liste_plugins_checked))
-		$fast_liste_plugins_checked = array_flip($liste_plugins_checked);
+	if (is_array($liste_plugins_actifs)) {
+			$fast_liste_plugins_actifs = array_flip($liste_plugins_actifs);
+	}
+	if (is_array($liste_plugins_checked)) {
+			$fast_liste_plugins_checked = array_flip($liste_plugins_checked);
+	}
 
 	$res = '';
 	$block = '';

Spacing +14 added lines, -14 removed lines

@@ -25,9 +25,9 @@ 
  * @param string $afficher_un
  * @return string
  */
-function plugins_afficher_liste_dist($url_page,$liste_plugins, $liste_plugins_checked, $liste_plugins_actifs, $dir_plugins=_DIR_PLUGINS,$afficher_un = 'afficher_plugin'){
-	$get_infos = charger_fonction('get_infos','plugins');
-	$ligne_plug = charger_fonction($afficher_un,'plugins');
+function plugins_afficher_liste_dist($url_page, $liste_plugins, $liste_plugins_checked, $liste_plugins_actifs, $dir_plugins = _DIR_PLUGINS, $afficher_un = 'afficher_plugin') {
+	$get_infos = charger_fonction('get_infos', 'plugins');
+	$ligne_plug = charger_fonction($afficher_un, 'plugins');
 
 	$all_infos = $get_infos($liste_plugins, false, $dir_plugins);
 
@@ -43,9 +43,9 @@ 
 							);
 
 	$liste_plugins = array_flip($liste_plugins);
-	foreach($liste_plugins as $chemin => $v) {
+	foreach ($liste_plugins as $chemin => $v) {
 		// des plugins ont pu etre enleves de la liste par le pipeline. On en tient compte.
-		if (isset($all_infos[$chemin])){
+		if (isset($all_infos[$chemin])) {
 			$liste_plugins[$chemin] = strtoupper(trim(typo(translitteration(unicode2charset(html2unicode($all_infos[$chemin]['nom']))))));
 		}
 		else {
@@ -55,7 +55,7 @@ 
 	asort($liste_plugins);
 	$exposed = urldecode(_request('plugin'));
 
-	$block_par_lettre = false;//count($liste_plugins)>10;
+	$block_par_lettre = false; //count($liste_plugins)>10;
 	$fast_liste_plugins_actifs = array();
 	$fast_liste_plugins_checked = array();
 	if (is_array($liste_plugins_actifs))
@@ -67,9 +67,9 @@ 
 	$block = '';
 	$initiale = '';
 	$block_actif = false;
-	foreach($liste_plugins as $plug => $nom){
-		if (($i=substr($nom,0,1))!==$initiale){
-			$res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
+	foreach ($liste_plugins as $plug => $nom) {
+		if (($i = substr($nom, 0, 1)) !== $initiale) {
+			$res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
 			$initiale = $i;
 			$block = '';
 			$block_actif = false;
@@ -78,20 +78,20 @@ 
 		$actif = isset($fast_liste_plugins_actifs[$plug]);
 		$checked = isset($fast_liste_plugins_checked[$plug]);
 		$block_actif = $block_actif | $actif;
-		$expose = ($exposed AND ($exposed==$plug OR $exposed==$dir_plugins . $plug OR $exposed==substr($dir_plugins,strlen(_DIR_RACINE)) . $plug));
+		$expose = ($exposed AND ($exposed == $plug OR $exposed == $dir_plugins.$plug OR $exposed == substr($dir_plugins, strlen(_DIR_RACINE)).$plug));
 		$block .= $ligne_plug($url_page, $plug, $checked, $actif, $expose, "item", $dir_plugins)."\n";
 	}
-	$res .= $block_par_lettre ? affiche_block_initiale($initiale,$block,$block_actif): $block;
+	$res .= $block_par_lettre ? affiche_block_initiale($initiale, $block, $block_actif) : $block;
 	$class = basename($dir_plugins);
 	return $res ? "<ul class='liste-items plugins $class'>$res</ul>" : "";
 }
 
 
 // http://doc.spip.org/@affiche_block_initiale
-function affiche_block_initiale($initiale,$block,$block_actif){
-	if (strlen($block)){
+function affiche_block_initiale($initiale, $block, $block_actif) {
+	if (strlen($block)) {
 		return "<li class='item'>"
-		  . bouton_block_depliable($initiale,$block_actif?true:false)
+		  . bouton_block_depliable($initiale, $block_actif ?true:false)
 		  . debut_block_depliable($block_actif)
 		  . "<ul>$block</ul>"
 		  . fin_block()

ecrire/plugins/afficher_repertoires.php

Doc Comments +8 added lines

@@ -69,11 +69,19 @@
 // vraiment n'importe quoi la gestion des chemins des plugins
 // une fonction pour aider...
 // http://doc.spip.org/@chemin_plug
+/**
+ * @param string $racine
+ *
+ * @return string
+ */
 function chemin_plug($racine, $plug) {
 	return preg_replace(',[^/]+/\.\./,', '', "$racine/$plug");
 }
 
 // http://doc.spip.org/@tree_open_close_dir
+/**
+ * @param string $target
+ */
 function tree_open_close_dir(&$current,$target,$deplie=array()){
 	if ($current == $target) return "";
 	$tcur = explode("/",$current);

Indentation +75 added lines, -75 removed lines

@@ -14,55 +14,55 @@ 
 
 // http://doc.spip.org/@affiche_arbre_plugins
 function plugins_afficher_repertoires_dist($url_page, $liste_plugins, $liste_plugins_actifs){
-	$ligne_plug = charger_fonction('afficher_plugin','plugins');
-	$racine = basename(_DIR_PLUGINS);
-	$init_dir = $current_dir = "";
-	// liste des repertoires deplies : construit en remontant l'arbo de chaque plugin actif
-	// des qu'un path est deja note deplie on s'arrete
-	$deplie = array($racine=>true);
-	$fast_liste_plugins_actifs=array();
-	foreach($liste_plugins_actifs as $key=>$plug){
-		$chemin_plug = chemin_plug($racine, $plug);
-		$fast_liste_plugins_actifs[$chemin_plug]=true;
-		$dir = dirname($chemin_plug);$maxiter=100;
-		while(strlen($dir) && !isset($deplie[$dir]) && $dir!=$racine && $maxiter-->0){
-			$deplie[$dir] = true;
-			$dir = dirname($dir);
-		}
-	}
+    $ligne_plug = charger_fonction('afficher_plugin','plugins');
+    $racine = basename(_DIR_PLUGINS);
+    $init_dir = $current_dir = "";
+    // liste des repertoires deplies : construit en remontant l'arbo de chaque plugin actif
+    // des qu'un path est deja note deplie on s'arrete
+    $deplie = array($racine=>true);
+    $fast_liste_plugins_actifs=array();
+    foreach($liste_plugins_actifs as $key=>$plug){
+        $chemin_plug = chemin_plug($racine, $plug);
+        $fast_liste_plugins_actifs[$chemin_plug]=true;
+        $dir = dirname($chemin_plug);$maxiter=100;
+        while(strlen($dir) && !isset($deplie[$dir]) && $dir!=$racine && $maxiter-->0){
+            $deplie[$dir] = true;
+            $dir = dirname($dir);
+        }
+    }
 
-	// index repertoires --> plugin
-	$dir_index=array();
-	foreach($liste_plugins as $key=>$plug){
-		$liste_plugins[$key] = chemin_plug($racine, $plug);
-		$dir_index[dirname($liste_plugins[$key])][] = $key;
-	}
+    // index repertoires --> plugin
+    $dir_index=array();
+    foreach($liste_plugins as $key=>$plug){
+        $liste_plugins[$key] = chemin_plug($racine, $plug);
+        $dir_index[dirname($liste_plugins[$key])][] = $key;
+    }
 
-	$visible = @isset($deplie[$current_dir]);
-	$maxiter=1000;
+    $visible = @isset($deplie[$current_dir]);
+    $maxiter=1000;
 
-	$res = '';
-	while (count($liste_plugins) && $maxiter--){
-		// le rep suivant
-		$dir = dirname(reset($liste_plugins));
-		if ($dir != $current_dir)
-			$res .= tree_open_close_dir($current_dir,$dir,$deplie);
+    $res = '';
+    while (count($liste_plugins) && $maxiter--){
+        // le rep suivant
+        $dir = dirname(reset($liste_plugins));
+        if ($dir != $current_dir)
+            $res .= tree_open_close_dir($current_dir,$dir,$deplie);
 
-		// d'abord tous les plugins du rep courant
-		if (isset($dir_index[$current_dir]))
-			foreach($dir_index[$current_dir] as $key){
-				$plug = $liste_plugins[$key];
-				$actif = @isset($fast_liste_plugins_actifs[$plug]);
-				$id = substr(md5($plug),0,16);
-				$res .= $ligne_plug($url_page, str_replace(_DIR_PLUGINS, '', _DIR_RACINE.$plug), $actif, 'menu-entree') . "\n";
-				unset($liste_plugins[$key]);
-			}
-	}
-	$res .= tree_open_close_dir($current_dir,$init_dir, true);
+        // d'abord tous les plugins du rep courant
+        if (isset($dir_index[$current_dir]))
+            foreach($dir_index[$current_dir] as $key){
+                $plug = $liste_plugins[$key];
+                $actif = @isset($fast_liste_plugins_actifs[$plug]);
+                $id = substr(md5($plug),0,16);
+                $res .= $ligne_plug($url_page, str_replace(_DIR_PLUGINS, '', _DIR_RACINE.$plug), $actif, 'menu-entree') . "\n";
+                unset($liste_plugins[$key]);
+            }
+    }
+    $res .= tree_open_close_dir($current_dir,$init_dir, true);
 
-	return "<ul class='menu-liste plugins'>"
-	. $res
-	. "</ul>";
+    return "<ul class='menu-liste plugins'>"
+    . $res
+    . "</ul>";
 }
 
 
@@ -70,42 +70,42 @@ 
 // une fonction pour aider...
 // http://doc.spip.org/@chemin_plug
 function chemin_plug($racine, $plug) {
-	return preg_replace(',[^/]+/\.\./,', '', "$racine/$plug");
+    return preg_replace(',[^/]+/\.\./,', '', "$racine/$plug");
 }
 
 // http://doc.spip.org/@tree_open_close_dir
 function tree_open_close_dir(&$current,$target,$deplie=array()){
-	if ($current == $target) return "";
-	$tcur = explode("/",$current);
-	$ttarg = explode("/",$target);
-	$tcom = array();
-	$output = "";
-	// la partie commune
-	while (reset($tcur)==reset($ttarg)){
-		$tcom[] = array_shift($tcur);
-		array_shift($ttarg);
-	}
-	// fermer les repertoires courant jusqu'au point de fork
-	while($close = array_pop($tcur)){
-		$output .= "</ul>\n";
-		$output .= fin_block();
-		$output .= "</li>\n";
-	}
-	$chemin = "";
-	if (count($tcom))
-		$chemin .= implode("/",$tcom)."/";
-	// ouvrir les repertoires jusqu'a la cible
-	while($open = array_shift($ttarg)){
-		$visible = @isset($deplie[$chemin.$open]);
-		$chemin .= $open . "/";
-		$output .= "<li>";
-		$output .= bouton_block_depliable($chemin,$visible);
-		$output .= debut_block_depliable($visible);
+    if ($current == $target) return "";
+    $tcur = explode("/",$current);
+    $ttarg = explode("/",$target);
+    $tcom = array();
+    $output = "";
+    // la partie commune
+    while (reset($tcur)==reset($ttarg)){
+        $tcom[] = array_shift($tcur);
+        array_shift($ttarg);
+    }
+    // fermer les repertoires courant jusqu'au point de fork
+    while($close = array_pop($tcur)){
+        $output .= "</ul>\n";
+        $output .= fin_block();
+        $output .= "</li>\n";
+    }
+    $chemin = "";
+    if (count($tcom))
+        $chemin .= implode("/",$tcom)."/";
+    // ouvrir les repertoires jusqu'a la cible
+    while($open = array_shift($ttarg)){
+        $visible = @isset($deplie[$chemin.$open]);
+        $chemin .= $open . "/";
+        $output .= "<li>";
+        $output .= bouton_block_depliable($chemin,$visible);
+        $output .= debut_block_depliable($visible);
 
-		$output .= "<ul>\n";
-	}
-	$current = $target;
-	return $output;
+        $output .= "<ul>\n";
+    }
+    $current = $target;
+    return $output;
 }
 
 ?>

Spacing +25 added lines, -25 removed lines

@@ -13,52 +13,52 @@ 
 if (!defined('_ECRIRE_INC_VERSION')) return;
 
 // http://doc.spip.org/@affiche_arbre_plugins
-function plugins_afficher_repertoires_dist($url_page, $liste_plugins, $liste_plugins_actifs){
-	$ligne_plug = charger_fonction('afficher_plugin','plugins');
+function plugins_afficher_repertoires_dist($url_page, $liste_plugins, $liste_plugins_actifs) {
+	$ligne_plug = charger_fonction('afficher_plugin', 'plugins');
 	$racine = basename(_DIR_PLUGINS);
 	$init_dir = $current_dir = "";
 	// liste des repertoires deplies : construit en remontant l'arbo de chaque plugin actif
 	// des qu'un path est deja note deplie on s'arrete
 	$deplie = array($racine=>true);
-	$fast_liste_plugins_actifs=array();
-	foreach($liste_plugins_actifs as $key=>$plug){
+	$fast_liste_plugins_actifs = array();
+	foreach ($liste_plugins_actifs as $key=>$plug) {
 		$chemin_plug = chemin_plug($racine, $plug);
-		$fast_liste_plugins_actifs[$chemin_plug]=true;
-		$dir = dirname($chemin_plug);$maxiter=100;
-		while(strlen($dir) && !isset($deplie[$dir]) && $dir!=$racine && $maxiter-->0){
+		$fast_liste_plugins_actifs[$chemin_plug] = true;
+		$dir = dirname($chemin_plug); $maxiter = 100;
+		while (strlen($dir) && !isset($deplie[$dir]) && $dir != $racine && $maxiter-- > 0) {
 			$deplie[$dir] = true;
 			$dir = dirname($dir);
 		}
 	}
 
 	// index repertoires --> plugin
-	$dir_index=array();
-	foreach($liste_plugins as $key=>$plug){
+	$dir_index = array();
+	foreach ($liste_plugins as $key=>$plug) {
 		$liste_plugins[$key] = chemin_plug($racine, $plug);
 		$dir_index[dirname($liste_plugins[$key])][] = $key;
 	}
 
 	$visible = @isset($deplie[$current_dir]);
-	$maxiter=1000;
+	$maxiter = 1000;
 
 	$res = '';
-	while (count($liste_plugins) && $maxiter--){
+	while (count($liste_plugins) && $maxiter--) {
 		// le rep suivant
 		$dir = dirname(reset($liste_plugins));
 		if ($dir != $current_dir)
-			$res .= tree_open_close_dir($current_dir,$dir,$deplie);
+			$res .= tree_open_close_dir($current_dir, $dir, $deplie);
 
 		// d'abord tous les plugins du rep courant
 		if (isset($dir_index[$current_dir]))
-			foreach($dir_index[$current_dir] as $key){
+			foreach ($dir_index[$current_dir] as $key) {
 				$plug = $liste_plugins[$key];
 				$actif = @isset($fast_liste_plugins_actifs[$plug]);
-				$id = substr(md5($plug),0,16);
-				$res .= $ligne_plug($url_page, str_replace(_DIR_PLUGINS, '', _DIR_RACINE.$plug), $actif, 'menu-entree') . "\n";
+				$id = substr(md5($plug), 0, 16);
+				$res .= $ligne_plug($url_page, str_replace(_DIR_PLUGINS, '', _DIR_RACINE.$plug), $actif, 'menu-entree')."\n";
 				unset($liste_plugins[$key]);
 			}
 	}
-	$res .= tree_open_close_dir($current_dir,$init_dir, true);
+	$res .= tree_open_close_dir($current_dir, $init_dir, true);
 
 	return "<ul class='menu-liste plugins'>"
 	. $res
@@ -74,32 +74,32 @@ 
 }
 
 // http://doc.spip.org/@tree_open_close_dir
-function tree_open_close_dir(&$current,$target,$deplie=array()){
+function tree_open_close_dir(&$current, $target, $deplie = array()) {
 	if ($current == $target) return "";
-	$tcur = explode("/",$current);
-	$ttarg = explode("/",$target);
+	$tcur = explode("/", $current);
+	$ttarg = explode("/", $target);
 	$tcom = array();
 	$output = "";
 	// la partie commune
-	while (reset($tcur)==reset($ttarg)){
+	while (reset($tcur) == reset($ttarg)) {
 		$tcom[] = array_shift($tcur);
 		array_shift($ttarg);
 	}
 	// fermer les repertoires courant jusqu'au point de fork
-	while($close = array_pop($tcur)){
+	while ($close = array_pop($tcur)) {
 		$output .= "</ul>\n";
 		$output .= fin_block();
 		$output .= "</li>\n";
 	}
 	$chemin = "";
 	if (count($tcom))
-		$chemin .= implode("/",$tcom)."/";
+		$chemin .= implode("/", $tcom)."/";
 	// ouvrir les repertoires jusqu'a la cible
-	while($open = array_shift($ttarg)){
+	while ($open = array_shift($ttarg)) {
 		$visible = @isset($deplie[$chemin.$open]);
-		$chemin .= $open . "/";
+		$chemin .= $open."/";
 		$output .= "<li>";
-		$output .= bouton_block_depliable($chemin,$visible);
+		$output .= bouton_block_depliable($chemin, $visible);
 		$output .= debut_block_depliable($visible);
 
 		$output .= "<ul>\n";

Braces +15 added lines, -8 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // http://doc.spip.org/@affiche_arbre_plugins
 function plugins_afficher_repertoires_dist($url_page, $liste_plugins, $liste_plugins_actifs){
@@ -45,13 +47,15 @@ 
 	while (count($liste_plugins) && $maxiter--){
 		// le rep suivant
 		$dir = dirname(reset($liste_plugins));
-		if ($dir != $current_dir)
-			$res .= tree_open_close_dir($current_dir,$dir,$deplie);
+		if ($dir != $current_dir) {
+					$res .= tree_open_close_dir($current_dir,$dir,$deplie);
+		}
 
 		// d'abord tous les plugins du rep courant
-		if (isset($dir_index[$current_dir]))
-			foreach($dir_index[$current_dir] as $key){
+		if (isset($dir_index[$current_dir])) {
+					foreach($dir_index[$current_dir] as $key){
 				$plug = $liste_plugins[$key];
+		}
 				$actif = @isset($fast_liste_plugins_actifs[$plug]);
 				$id = substr(md5($plug),0,16);
 				$res .= $ligne_plug($url_page, str_replace(_DIR_PLUGINS, '', _DIR_RACINE.$plug), $actif, 'menu-entree') . "\n";
@@ -75,7 +79,9 @@ 
 
 // http://doc.spip.org/@tree_open_close_dir
 function tree_open_close_dir(&$current,$target,$deplie=array()){
-	if ($current == $target) return "";
+	if ($current == $target) {
+	    return "";
+	}
 	$tcur = explode("/",$current);
 	$ttarg = explode("/",$target);
 	$tcom = array();
@@ -92,8 +98,9 @@ 
 		$output .= "</li>\n";
 	}
 	$chemin = "";
-	if (count($tcom))
-		$chemin .= implode("/",$tcom)."/";
+	if (count($tcom)) {
+			$chemin .= implode("/",$tcom)."/";
+	}
 	// ouvrir les repertoires jusqu'a la cible
 	while($open = array_shift($ttarg)){
 		$visible = @isset($deplie[$chemin.$open]);

ecrire/public/admin.php

Doc Comments +6 added lines

@@ -21,6 +21,9 @@ 
 // Compatibilite : on utilise stripos/strripos() qui n'existent pas en php4
 if (!function_exists('strripos')) {
 // http://doc.spip.org/@strripos
+	/**
+	 * @param string $aiguille
+	 */
 	function strripos($botte, $aiguille) {
 		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@is',
 		$botte, $regs)) { 
@@ -31,6 +34,9 @@ 
 }
 if (!function_exists('stripos')) {
 // http://doc.spip.org/@stripos
+	/**
+	 * @param string $aiguille
+	 */
 	function stripos($botte, $aiguille) {
 		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@isU',
 		$botte, $regs)) { 

Indentation +34 added lines, -34 removed lines

@@ -21,54 +21,54 @@
 // Compatibilite : on utilise stripos/strripos() qui n'existent pas en php4
 if (!function_exists('strripos')) {
 // http://doc.spip.org/@strripos
-	function strripos($botte, $aiguille) {
-		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@is',
-		$botte, $regs)) { 
-			return strlen($regs[1]);
-		}
-		return false;
-	}
+    function strripos($botte, $aiguille) {
+        if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@is',
+        $botte, $regs)) { 
+            return strlen($regs[1]);
+        }
+        return false;
+    }
 }
 if (!function_exists('stripos')) {
 // http://doc.spip.org/@stripos
-	function stripos($botte, $aiguille) {
-		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@isU',
-		$botte, $regs)) { 
-			return strlen($regs[1]);
-		}
-		return false;
-	}
+    function stripos($botte, $aiguille) {
+        if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@isU',
+        $botte, $regs)) { 
+            return strlen($regs[1]);
+        }
+        return false;
+    }
 }
 
 // http://doc.spip.org/@affiche_boutons_admin
 function affiche_boutons_admin($contenu) {
-	include_spip('inc/filtres');
+    include_spip('inc/filtres');
 
-	// Inserer le css d'admin
-	$css = "<link rel='stylesheet' href='".url_absolue(direction_css(find_in_path('spip_admin.css')))
-	. "' type='text/css' />\n";
-	if ($f = find_in_path('spip_admin_perso.css'))
-		$css .= "<link rel='stylesheet' href='"
-		. url_absolue(direction_css($f)) . "' type='text/css' />\n";
+    // Inserer le css d'admin
+    $css = "<link rel='stylesheet' href='".url_absolue(direction_css(find_in_path('spip_admin.css')))
+    . "' type='text/css' />\n";
+    if ($f = find_in_path('spip_admin_perso.css'))
+        $css .= "<link rel='stylesheet' href='"
+        . url_absolue(direction_css($f)) . "' type='text/css' />\n";
 
-	($pos = stripos($contenu, '</head>'))
-	    || ($pos = stripos($contenu, '<body>'))
-	    || ($pos = 0);
-	$contenu = substr_replace($contenu, $css, $pos, 0);
+    ($pos = stripos($contenu, '</head>'))
+        || ($pos = stripos($contenu, '<body>'))
+        || ($pos = 0);
+    $contenu = substr_replace($contenu, $css, $pos, 0);
 
 
-	// Inserer la balise #FORMULAIRE_ADMIN, en float
-	$boutons_admin = inclure_balise_dynamique(
-		balise_FORMULAIRE_ADMIN_dyn('spip-admin-float'),
-	false);
+    // Inserer la balise #FORMULAIRE_ADMIN, en float
+    $boutons_admin = inclure_balise_dynamique(
+        balise_FORMULAIRE_ADMIN_dyn('spip-admin-float'),
+    false);
 
-	($pos = strripos($contenu, '</body>'))
-	    || ($pos = strripos($contenu, '</html>'))
-	    || ($pos = strlen($contenu));
-	$contenu = substr_replace($contenu, $boutons_admin, $pos, 0);
+    ($pos = strripos($contenu, '</body>'))
+        || ($pos = strripos($contenu, '</html>'))
+        || ($pos = strlen($contenu));
+    $contenu = substr_replace($contenu, $boutons_admin, $pos, 0);
 
 
-	return $contenu;
+    return $contenu;
 }
 
 ?>

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
 if (!function_exists('strripos')) {
 // http://doc.spip.org/@strripos
 	function strripos($botte, $aiguille) {
-		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@is',
+		if (preg_match('@^(.*)'.preg_quote($aiguille, '@').'@is',
 		$botte, $regs)) { 
 			return strlen($regs[1]);
 		}
@@ -32,7 +32,7 @@ 
 if (!function_exists('stripos')) {
 // http://doc.spip.org/@stripos
 	function stripos($botte, $aiguille) {
-		if (preg_match('@^(.*)' . preg_quote($aiguille, '@') . '@isU',
+		if (preg_match('@^(.*)'.preg_quote($aiguille, '@').'@isU',
 		$botte, $regs)) { 
 			return strlen($regs[1]);
 		}
@@ -49,7 +49,7 @@ 
 	. "' type='text/css' />\n";
 	if ($f = find_in_path('spip_admin_perso.css'))
 		$css .= "<link rel='stylesheet' href='"
-		. url_absolue(direction_css($f)) . "' type='text/css' />\n";
+		. url_absolue(direction_css($f))."' type='text/css' />\n";
 
 	($pos = stripos($contenu, '</head>'))
 	    || ($pos = stripos($contenu, '<body>'))

Braces +6 added lines, -3 removed lines

@@ -11,7 +11,9 @@ 
 \***************************************************************************/
 
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Inserer la feuille de style selon les normes, dans le <head>
 // puis les boutons
@@ -47,9 +49,10 @@ 
 	// Inserer le css d'admin
 	$css = "<link rel='stylesheet' href='".url_absolue(direction_css(find_in_path('spip_admin.css')))
 	. "' type='text/css' />\n";
-	if ($f = find_in_path('spip_admin_perso.css'))
-		$css .= "<link rel='stylesheet' href='"
+	if ($f = find_in_path('spip_admin_perso.css')) {
+			$css .= "<link rel='stylesheet' href='"
 		. url_absolue(direction_css($f)) . "' type='text/css' />\n";
+	}
 
 	($pos = stripos($contenu, '</head>'))
 	    || ($pos = stripos($contenu, '<body>'))

ecrire/public/normaliser.php

Doc Comments +3 added lines

@@ -71,6 +71,9 @@
 }
 
 
+/**
+ * @param string $nom
+ */
 function phraser_logo_faux_filtres($nom)
 {
 	switch($nom) {

Indentation +137 added lines, -137 removed lines

@@ -24,65 +24,65 @@ 
 
 function phraser_vieux_logos(&$p)
 {
-	if ($p->param[0][0])
-		$args = array('');
-	else {
-		$args = array_shift($p->param);
-	}
-
-	foreach($p->param as $couple) {
-		$nom = trim($couple[0]);
-		if ($nom == '')  {array_shift($p->param); break;}
-		$r = phraser_logo_faux_filtres($nom);
-		if ($r === 0) {
-			$c = new Texte;
-			$c->texte = $nom;
-			$args[] = array($c);
-			array_shift($p->param);
-			spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
-		} elseif ($r === 2) {
-				$p->etoile = '**';
-				array_shift($p->param);
-				spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
-		} elseif ($r === 1) {
-				array_shift($p->param);
-				$p->etoile = '*';
-				spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
-
-		} elseif (preg_match("/^".NOM_DE_CHAMP.'(.*)$/sS', $nom, $m)) {
-				$champ = new Champ();
-				$champ->nom_boucle = $m[2];
-				$champ->nom_champ = $m[3];
-				$champ->etoile = $m[5];
-				$champ = array($champ);
-				if ($m[6]) {
-				  $r = new Texte;
-				  $r->texte = $m[6];
-				  $champ[]= $r;
-				}
-				$args[]= $champ;
-				array_shift($p->param);
-				spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
-
-		} // le cas else est la seule incompatibilite
-
-	}
-	array_unshift($p->param, $args);
+    if ($p->param[0][0])
+        $args = array('');
+    else {
+        $args = array_shift($p->param);
+    }
+
+    foreach($p->param as $couple) {
+        $nom = trim($couple[0]);
+        if ($nom == '')  {array_shift($p->param); break;}
+        $r = phraser_logo_faux_filtres($nom);
+        if ($r === 0) {
+            $c = new Texte;
+            $c->texte = $nom;
+            $args[] = array($c);
+            array_shift($p->param);
+            spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
+        } elseif ($r === 2) {
+                $p->etoile = '**';
+                array_shift($p->param);
+                spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
+        } elseif ($r === 1) {
+                array_shift($p->param);
+                $p->etoile = '*';
+                spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
+
+        } elseif (preg_match("/^".NOM_DE_CHAMP.'(.*)$/sS', $nom, $m)) {
+                $champ = new Champ();
+                $champ->nom_boucle = $m[2];
+                $champ->nom_champ = $m[3];
+                $champ->etoile = $m[5];
+                $champ = array($champ);
+                if ($m[6]) {
+                    $r = new Texte;
+                    $r->texte = $m[6];
+                    $champ[]= $r;
+                }
+                $args[]= $champ;
+                array_shift($p->param);
+                spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
+
+        } // le cas else est la seule incompatibilite
+
+    }
+    array_unshift($p->param, $args);
 }
 
 
 function phraser_logo_faux_filtres($nom)
 {
-	switch($nom) {
-	case 'top':
-	case 'left':
-	case 'right':
-	case 'center':
-	case 'bottom':  return 0;
-	case 'lien':    return 1;
-	case 'fichier': return 2;
-	default: return $nom;
-	}
+    switch($nom) {
+    case 'top':
+    case 'left':
+    case 'right':
+    case 'center':
+    case 'bottom':  return 0;
+    case 'lien':    return 1;
+    case 'fichier': return 2;
+    default: return $nom;
+    }
 }
 
 
@@ -90,64 +90,64 @@ 
 
 function phraser_vieux_emb(&$p)
 {
-	if (!is_array($p->param))
-		$p->param=array();
-
-	// Produire le premier argument {emb}
-	$texte = new Texte;
-	$texte->texte='emb';
-	$param = array('', array($texte));
-
-	// Transformer les filtres en arguments
-	for ($i=0; $i<count($p->param); $i++) {
-		if ($p->param[$i][0]) {
-			if (!strstr($p->param[$i][0], '='))
-				break;# on a rencontre un vrai filtre, c'est fini
-			$texte = new Texte;
-			$texte->texte=$p->param[$i][0];
-			$param[] = array($texte);
-		}
-		array_shift($p->param);
-	}
-	array_unshift($p->param, $param);	
-	spip_log('balise EMBED_DOCUMENT obsolete', 'vieilles_defs');
-	$p->nom_champ = 'MODELE';
+    if (!is_array($p->param))
+        $p->param=array();
+
+    // Produire le premier argument {emb}
+    $texte = new Texte;
+    $texte->texte='emb';
+    $param = array('', array($texte));
+
+    // Transformer les filtres en arguments
+    for ($i=0; $i<count($p->param); $i++) {
+        if ($p->param[$i][0]) {
+            if (!strstr($p->param[$i][0], '='))
+                break;# on a rencontre un vrai filtre, c'est fini
+            $texte = new Texte;
+            $texte->texte=$p->param[$i][0];
+            $param[] = array($texte);
+        }
+        array_shift($p->param);
+    }
+    array_unshift($p->param, $param);	
+    spip_log('balise EMBED_DOCUMENT obsolete', 'vieilles_defs');
+    $p->nom_champ = 'MODELE';
 }
 
 // Vieux formulaire de recherch
 
 function phraser_vieux_recherche($p)
 {
-	if ($p->param[0][0]) {
-		$c = new Texte;
-		$c->texte = $p->param[0][0];
-		$p->param[0][1] = array($c);
-		$p->param[0][0] = '';
-		$p->fonctions = array();
-		spip_log('FORMULAIRE_RECHERCHE avec filtre ' . $c->texte, 'vieilles_defs');
-	}
+    if ($p->param[0][0]) {
+        $c = new Texte;
+        $c->texte = $p->param[0][0];
+        $p->param[0][1] = array($c);
+        $p->param[0][0] = '';
+        $p->fonctions = array();
+        spip_log('FORMULAIRE_RECHERCHE avec filtre ' . $c->texte, 'vieilles_defs');
+    }
 }
 
 // Gerer la notation [(#EXPOSER|on,off)]
 function phraser_vieux_exposer($p)
 {
-	if ($a = $p->fonctions) {
-		preg_match("#([^,]*)(,(.*))?#", $a[0][0], $regs);
-		$args = array();
-		if ($regs[1]) {
-			$a = new Texte;
-			$a->texte = $regs[1];
-			$args = array('', array($a));
-			if ($regs[3]) {
-				$a = new Texte;
-				$a->texte = $regs[3];
-				$args[] = array($a);
-			}
-		}
-		$p->param[0] = $args;
-		$p->fonctions = array();
-		$p->nom_champ = 'EXPOSE';
-	}
+    if ($a = $p->fonctions) {
+        preg_match("#([^,]*)(,(.*))?#", $a[0][0], $regs);
+        $args = array();
+        if ($regs[1]) {
+            $a = new Texte;
+            $a->texte = $regs[1];
+            $args = array('', array($a));
+            if ($regs[3]) {
+                $a = new Texte;
+                $a->texte = $regs[3];
+                $args[] = array($a);
+            }
+        }
+        $p->param[0] = $args;
+        $p->fonctions = array();
+        $p->nom_champ = 'EXPOSE';
+    }
 }
 
 function phraser_vieux_modele($p)  {normaliser_args_inclumodel($p);}
@@ -155,46 +155,46 @@ 
 
 function normaliser_args_inclumodel($p)
 {
-	$params = $p->param;
-	if (!$params) return;
-	$args = $params[0];
-	if ($args[0]) return; // filtre immediat
-	array_shift($p->param);
-	foreach ($p->param as $l) {
-		if (!array_shift($l))  {
-			$args = array_merge($args, $l);
-			array_shift($p->param);
-		} else  break; // filtre
-	}
-	array_unshift($p->param, $args);
+    $params = $p->param;
+    if (!$params) return;
+    $args = $params[0];
+    if ($args[0]) return; // filtre immediat
+    array_shift($p->param);
+    foreach ($p->param as $l) {
+        if (!array_shift($l))  {
+            $args = array_merge($args, $l);
+            array_shift($p->param);
+        } else  break; // filtre
+    }
+    array_unshift($p->param, $args);
 }
 
 function normaliser_inclure($champ)
 {
-	normaliser_args_inclumodel($champ);
-	$l = $champ->param[0];
-	if (is_array($l) AND !$l[0]) {
-	  foreach ($l as $k => $p) {
-	    if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte,'=')) {
-	      $p[0]->texte = trim($p[0]->texte);
-	      }
-	  }
-	  foreach ($l as $k => $p) {
-		if (!$p OR $p[0]->type != 'texte' OR 
-		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
-			continue;
-
-		if ($r[1])
-			$p[0]->texte = $r[1];
-		else unset($p[0]);
-		$champ->texte = $p;
-		unset($champ->param[0][$k]);
-		if (count($champ->param[0]) ==1) 
-			array_shift($champ->param);
-		return;
-	  }
-	}
-	spip_log("inclure sans fond ni fichier");
+    normaliser_args_inclumodel($champ);
+    $l = $champ->param[0];
+    if (is_array($l) AND !$l[0]) {
+        foreach ($l as $k => $p) {
+        if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte,'=')) {
+            $p[0]->texte = trim($p[0]->texte);
+            }
+        }
+        foreach ($l as $k => $p) {
+        if (!$p OR $p[0]->type != 'texte' OR 
+            !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
+            continue;
+
+        if ($r[1])
+            $p[0]->texte = $r[1];
+        else unset($p[0]);
+        $champ->texte = $p;
+        unset($champ->param[0][$k]);
+        if (count($champ->param[0]) ==1) 
+            array_shift($champ->param);
+        return;
+        }
+    }
+    spip_log("inclure sans fond ni fichier");
 }
 
 ?>

Switch Indentation +8 added lines, -8 removed lines

@@ -74,14 +74,14 @@
 function phraser_logo_faux_filtres($nom)
 {
 	switch($nom) {
-	case 'top':
-	case 'left':
-	case 'right':
-	case 'center':
-	case 'bottom':  return 0;
-	case 'lien':    return 1;
-	case 'fichier': return 2;
-	default: return $nom;
+	    case 'top':
+	    case 'left':
+	    case 'right':
+	    case 'center':
+	    case 'bottom':  return 0;
+	    case 'lien':    return 1;
+	    case 'fichier': return 2;
+	    default: return $nom;
 	}
 }
 

Spacing +17 added lines, -17 removed lines

@@ -30,9 +30,9 @@ 
 		$args = array_shift($p->param);
 	}
 
-	foreach($p->param as $couple) {
+	foreach ($p->param as $couple) {
 		$nom = trim($couple[0]);
-		if ($nom == '')  {array_shift($p->param); break;}
+		if ($nom == '') {array_shift($p->param); break; }
 		$r = phraser_logo_faux_filtres($nom);
 		if ($r === 0) {
 			$c = new Texte;
@@ -58,9 +58,9 @@ 
 				if ($m[6]) {
 				  $r = new Texte;
 				  $r->texte = $m[6];
-				  $champ[]= $r;
+				  $champ[] = $r;
 				}
-				$args[]= $champ;
+				$args[] = $champ;
 				array_shift($p->param);
 				spip_log("filtre de logo obsolete $nom", 'vieilles_defs');
 
@@ -73,7 +73,7 @@ 
 
 function phraser_logo_faux_filtres($nom)
 {
-	switch($nom) {
+	switch ($nom) {
 	case 'top':
 	case 'left':
 	case 'right':
@@ -91,20 +91,20 @@ 
 function phraser_vieux_emb(&$p)
 {
 	if (!is_array($p->param))
-		$p->param=array();
+		$p->param = array();
 
 	// Produire le premier argument {emb}
 	$texte = new Texte;
-	$texte->texte='emb';
+	$texte->texte = 'emb';
 	$param = array('', array($texte));
 
 	// Transformer les filtres en arguments
-	for ($i=0; $i<count($p->param); $i++) {
+	for ($i = 0; $i < count($p->param); $i++) {
 		if ($p->param[$i][0]) {
 			if (!strstr($p->param[$i][0], '='))
-				break;# on a rencontre un vrai filtre, c'est fini
+				break; # on a rencontre un vrai filtre, c'est fini
 			$texte = new Texte;
-			$texte->texte=$p->param[$i][0];
+			$texte->texte = $p->param[$i][0];
 			$param[] = array($texte);
 		}
 		array_shift($p->param);
@@ -124,7 +124,7 @@ 
 		$p->param[0][1] = array($c);
 		$p->param[0][0] = '';
 		$p->fonctions = array();
-		spip_log('FORMULAIRE_RECHERCHE avec filtre ' . $c->texte, 'vieilles_defs');
+		spip_log('FORMULAIRE_RECHERCHE avec filtre '.$c->texte, 'vieilles_defs');
 	}
 }
 
@@ -150,8 +150,8 @@ 
 	}
 }
 
-function phraser_vieux_modele($p)  {normaliser_args_inclumodel($p);}
-function phraser_vieux_inclu($p) {normaliser_args_inclumodel($p);}
+function phraser_vieux_modele($p) {normaliser_args_inclumodel($p); }
+function phraser_vieux_inclu($p) {normaliser_args_inclumodel($p); }
 
 function normaliser_args_inclumodel($p)
 {
@@ -161,7 +161,7 @@ 
 	if ($args[0]) return; // filtre immediat
 	array_shift($p->param);
 	foreach ($p->param as $l) {
-		if (!array_shift($l))  {
+		if (!array_shift($l)) {
 			$args = array_merge($args, $l);
 			array_shift($p->param);
 		} else  break; // filtre
@@ -175,13 +175,13 @@ 
 	$l = $champ->param[0];
 	if (is_array($l) AND !$l[0]) {
 	  foreach ($l as $k => $p) {
-	    if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte,'=')) {
+	    if ($p AND $p[0]->type == 'texte' AND !strpos($p[0]->texte, '=')) {
 	      $p[0]->texte = trim($p[0]->texte);
 	      }
 	  }
 	  foreach ($l as $k => $p) {
 		if (!$p OR $p[0]->type != 'texte' OR 
-		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
+		    !preg_match('/^fond\s*=\s*(.*)$/', $p[0]->texte, $r))
 			continue;
 
 		if ($r[1])
@@ -189,7 +189,7 @@ 
 		else unset($p[0]);
 		$champ->texte = $p;
 		unset($champ->param[0][$k]);
-		if (count($champ->param[0]) ==1) 
+		if (count($champ->param[0]) == 1) 
 			array_shift($champ->param);
 		return;
 	  }

Braces +35 added lines, -18 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // Les fonctions de ce fichier sont appelees en certains points 
 // de l'analyseur syntaxique afin de normaliser de vieilles syntaxes,
@@ -24,9 +26,9 @@ 
 
 function phraser_vieux_logos(&$p)
 {
-	if ($p->param[0][0])
-		$args = array('');
-	else {
+	if ($p->param[0][0]) {
+			$args = array('');
+	} else {
 		$args = array_shift($p->param);
 	}
 
@@ -90,8 +92,9 @@ 
 
 function phraser_vieux_emb(&$p)
 {
-	if (!is_array($p->param))
-		$p->param=array();
+	if (!is_array($p->param)) {
+			$p->param=array();
+	}
 
 	// Produire le premier argument {emb}
 	$texte = new Texte;
@@ -101,8 +104,10 @@ 
 	// Transformer les filtres en arguments
 	for ($i=0; $i<count($p->param); $i++) {
 		if ($p->param[$i][0]) {
-			if (!strstr($p->param[$i][0], '='))
-				break;# on a rencontre un vrai filtre, c'est fini
+			if (!strstr($p->param[$i][0], '=')) {
+							break;
+			}
+			# on a rencontre un vrai filtre, c'est fini
 			$texte = new Texte;
 			$texte->texte=$p->param[$i][0];
 			$param[] = array($texte);
@@ -156,15 +161,23 @@ 
 function normaliser_args_inclumodel($p)
 {
 	$params = $p->param;
-	if (!$params) return;
+	if (!$params) {
+	    return;
+	}
 	$args = $params[0];
-	if ($args[0]) return; // filtre immediat
+	if ($args[0]) {
+	    return;
+	}
+	// filtre immediat
 	array_shift($p->param);
 	foreach ($p->param as $l) {
 		if (!array_shift($l))  {
 			$args = array_merge($args, $l);
 			array_shift($p->param);
-		} else  break; // filtre
+		} else {
+		    break;
+		}
+		// filtre
 	}
 	array_unshift($p->param, $args);
 }
@@ -181,16 +194,20 @@ 
 	  }
 	  foreach ($l as $k => $p) {
 		if (!$p OR $p[0]->type != 'texte' OR 
-		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r))
-			continue;
+		    !preg_match('/^fond\s*=\s*(.*)$/',$p[0]->texte, $r)) {
+					continue;
+		}
 
-		if ($r[1])
-			$p[0]->texte = $r[1];
-		else unset($p[0]);
+		if ($r[1]) {
+					$p[0]->texte = $r[1];
+		} else {
+		    unset($p[0]);
+		}
 		$champ->texte = $p;
 		unset($champ->param[0][$k]);
-		if (count($champ->param[0]) ==1) 
-			array_shift($champ->param);
+		if (count($champ->param[0]) ==1) {
+					array_shift($champ->param);
+		}
 		return;
 	  }
 	}

ecrire/public/parametrer.php

Doc Comments +1 added lines, -1 removed lines

@@ -183,7 +183,7 @@
  * http://doc.spip.org/@tester_redirection
  *
  * @param string $fond
- * @param array $contexte
+ * @param string $contexte
  * @param string $connect
  * @return array|bool
  */

Indentation +186 added lines, -186 removed lines

@@ -17,16 +17,16 @@ 
 // NB: mes_fonctions peut initialiser $dossier_squelettes (old-style)
 // donc il faut l'inclure "en globals"
 if ($f = find_in_path('mes_fonctions.php')) {
-	global $dossier_squelettes;
-	include_once(_ROOT_CWD . $f);
+    global $dossier_squelettes;
+    include_once(_ROOT_CWD . $f);
 }
 
 if (@is_readable(_CACHE_PLUGINS_FCT)) {
-	// chargement optimise precompile
-	include_once(_CACHE_PLUGINS_FCT);
+    // chargement optimise precompile
+    include_once(_CACHE_PLUGINS_FCT);
 }
 if (test_espace_prive ())
-	include_spip('inc/filtres_ecrire');
+    include_spip('inc/filtres_ecrire');
 
 # Determine le squelette associe a une requete 
 # et l'applique sur le contexte, le nom du cache et le serveur
@@ -43,158 +43,158 @@ 
 
 // http://doc.spip.org/@public_parametrer_dist
 function public_parametrer_dist($fond, $contexte='', $cache='', $connect='')  {
-	static $composer,$styliser,$notes=null;
-	$page = tester_redirection($fond, $contexte, $connect);
-	if ($page) return $page;
-
-	if (isset($contexte['lang']))
-		$lang = $contexte['lang'];
-	elseif (!isset($lang))
-		$lang = $GLOBALS['meta']['langue_site'];
-
-	$select = ((!isset($GLOBALS['forcer_lang']) OR !$GLOBALS['forcer_lang']) AND $lang <> $GLOBALS['spip_lang']);
-	if ($select) $select = lang_select($lang);
-
-	$debug = (defined('_VAR_MODE') && _VAR_MODE == 'debug');
-
-	if (!$styliser)
-		$styliser = charger_fonction('styliser', 'public');
-	list($skel,$mime_type, $gram, $sourcefile) =
-		$styliser($fond, $contexte, $GLOBALS['spip_lang'], $connect);
-
-	if ($skel) {
-
-		// sauver le nom de l'eventuel squelette en cours d'execution
-		// (recursion possible a cause des modeles)
-		if ($debug) {
-			$courant = @$GLOBALS['debug_objets']['courant'];
-			$GLOBALS['debug_objets']['contexte'][$sourcefile] = $contexte;
-		}
-
-		// charger le squelette en specifiant les langages cibles et source
-		// au cas il faudrait le compiler (source posterieure au resultat)
-
-		if (!$composer)
-			$composer = charger_fonction('composer', 'public');
-		$fonc = $composer($skel, $mime_type, $gram, $sourcefile, $connect);
-	}
-	else
-		$fonc = '';
-
-	if (!$fonc) { // squelette inconnu (==='') ou faux (===false)
-		$page = $fonc;
-	}
-	else {
-		// Preparer l'appel de la fonction principale du squelette 
-
-		spip_timer($a = 'calcul page '.rand(0,1000));
-
-		// On cree un marqueur de notes unique lie a cette composition
-		// et on enregistre l'etat courant des globales de notes...
-		if (is_null($notes))
-			$notes = charger_fonction('notes', 'inc', true);
-		if ($notes)
-			$notes('','empiler');
-
-		// Rajouter d'office ces deux parametres
-		// (mais vaudrait mieux que le compilateur sache le simuler
-		// car ca interdit l'usage de criteres conditionnels dessus).
-		if (!isset($contexte['date'])) {
-			$contexte['date'] = date("Y-m-d H:i:s");
-			$contexte['date_default'] = true;
-		} else $contexte['date'] = normaliser_date($contexte['date'], true);
-
-		if (!isset($contexte['date_redac'])) {
-			$contexte['date_redac'] = date("Y-m-d H:i:s");
-			$contexte['date_redac_default'] = true;
-		} else $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
-
-	// Passer le nom du cache pour produire sa destruction automatique
-		$page = $fonc(array('cache' => $cache), array($contexte));
-
-	// Restituer les globales de notes telles qu'elles etaient avant l'appel
-	// Si l'inclus n'a pas affiche ses notes, tant pis (elles *doivent*
-	// etre dans son resultat, autrement elles ne seraient pas prises en
-	// compte a chaque calcul d'un texte contenant un modele, mais seulement
-	// quand le modele serait calcule, et on aurait des resultats incoherents)
-		if ($notes)
-			$notes('','depiler');
-
-		// reinjecter en dynamique la pile des notes
-		// si il y a des inclure dynamiques
-		// si la pile n'est pas vide
-		// la generalisation de cette injection permettrait de corriger le point juste au dessus
-		// en faisant remonter les notes a l'incluant (A tester et valider avant application)
-		if ($notes)
-			$page['notes'] = $notes('','sauver_etat');
-
-		// spip_log: un joli contexte
-		$infos = array();
-		foreach (array_filter($contexte) as $var => $val) {
-			if (is_array($val)) $val = serialize($val);
-			if (strlen("$val") > 30)
-				$val = substr("$val", 0,27).'..';
-			if (strstr($val,' '))
-				$val = "'$val'";
-			$infos[] = $var.'='.$val;
-		}
-		$profile = spip_timer($a);
-		spip_log("calcul ($profile) [$skel] "
-			. join(', ', $infos)
-			.' ('.strlen($page['texte']).' octets)');
-
-		if (defined('_CALCUL_PROFILER') AND intval($profile)>_CALCUL_PROFILER){
-			spip_log("calcul ($profile) [$skel] "
-				. join(', ', $infos)
-				.' ('.strlen($page['texte']).' octets) | '.$_SERVER['REQUEST_URI'],"profiler"._LOG_AVERTISSEMENT);
-		}
-
-		if ($debug) {
-		// si c'est ce que demande le debusqueur, lui passer la main
-			$t = strlen($page['texte']) ? $page['texte'] : " ";
-			$GLOBALS['debug_objets']['resultat'][$fonc . 'tout'] = $t;
-			$GLOBALS['debug_objets']['courant'] = $courant;
-			$GLOBALS['debug_objets']['profile'][$sourcefile] = $profile;
-			if ($GLOBALS['debug_objets']['sourcefile']
-			AND (_request('var_mode_objet') == $fonc)
-			AND (_request('var_mode_affiche') == 'resultat')) {
-				erreur_squelette();
-			}
-		}
-		// Si #CACHE{} n'etait pas la, le mettre a $delais
-		if (!isset($page['entetes']['X-Spip-Cache'])){
-			// Dans l'espace prive ou dans un modeles/ on pose un cache 0 par defaut
-			// si aucun #CACHE{} specifie
-			// le contexte implicite qui conditionne le cache assure qu'on retombe pas sur le meme
-			// entre public et prive
-			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0)
-				$page['entetes']['X-Spip-Cache'] = 0;
-			else
-				$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
-		}
-
-		$page['contexte'] = $contexte;
+    static $composer,$styliser,$notes=null;
+    $page = tester_redirection($fond, $contexte, $connect);
+    if ($page) return $page;
+
+    if (isset($contexte['lang']))
+        $lang = $contexte['lang'];
+    elseif (!isset($lang))
+        $lang = $GLOBALS['meta']['langue_site'];
+
+    $select = ((!isset($GLOBALS['forcer_lang']) OR !$GLOBALS['forcer_lang']) AND $lang <> $GLOBALS['spip_lang']);
+    if ($select) $select = lang_select($lang);
+
+    $debug = (defined('_VAR_MODE') && _VAR_MODE == 'debug');
+
+    if (!$styliser)
+        $styliser = charger_fonction('styliser', 'public');
+    list($skel,$mime_type, $gram, $sourcefile) =
+        $styliser($fond, $contexte, $GLOBALS['spip_lang'], $connect);
+
+    if ($skel) {
+
+        // sauver le nom de l'eventuel squelette en cours d'execution
+        // (recursion possible a cause des modeles)
+        if ($debug) {
+            $courant = @$GLOBALS['debug_objets']['courant'];
+            $GLOBALS['debug_objets']['contexte'][$sourcefile] = $contexte;
+        }
+
+        // charger le squelette en specifiant les langages cibles et source
+        // au cas il faudrait le compiler (source posterieure au resultat)
+
+        if (!$composer)
+            $composer = charger_fonction('composer', 'public');
+        $fonc = $composer($skel, $mime_type, $gram, $sourcefile, $connect);
+    }
+    else
+        $fonc = '';
+
+    if (!$fonc) { // squelette inconnu (==='') ou faux (===false)
+        $page = $fonc;
+    }
+    else {
+        // Preparer l'appel de la fonction principale du squelette 
+
+        spip_timer($a = 'calcul page '.rand(0,1000));
+
+        // On cree un marqueur de notes unique lie a cette composition
+        // et on enregistre l'etat courant des globales de notes...
+        if (is_null($notes))
+            $notes = charger_fonction('notes', 'inc', true);
+        if ($notes)
+            $notes('','empiler');
+
+        // Rajouter d'office ces deux parametres
+        // (mais vaudrait mieux que le compilateur sache le simuler
+        // car ca interdit l'usage de criteres conditionnels dessus).
+        if (!isset($contexte['date'])) {
+            $contexte['date'] = date("Y-m-d H:i:s");
+            $contexte['date_default'] = true;
+        } else $contexte['date'] = normaliser_date($contexte['date'], true);
+
+        if (!isset($contexte['date_redac'])) {
+            $contexte['date_redac'] = date("Y-m-d H:i:s");
+            $contexte['date_redac_default'] = true;
+        } else $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
+
+    // Passer le nom du cache pour produire sa destruction automatique
+        $page = $fonc(array('cache' => $cache), array($contexte));
+
+    // Restituer les globales de notes telles qu'elles etaient avant l'appel
+    // Si l'inclus n'a pas affiche ses notes, tant pis (elles *doivent*
+    // etre dans son resultat, autrement elles ne seraient pas prises en
+    // compte a chaque calcul d'un texte contenant un modele, mais seulement
+    // quand le modele serait calcule, et on aurait des resultats incoherents)
+        if ($notes)
+            $notes('','depiler');
+
+        // reinjecter en dynamique la pile des notes
+        // si il y a des inclure dynamiques
+        // si la pile n'est pas vide
+        // la generalisation de cette injection permettrait de corriger le point juste au dessus
+        // en faisant remonter les notes a l'incluant (A tester et valider avant application)
+        if ($notes)
+            $page['notes'] = $notes('','sauver_etat');
+
+        // spip_log: un joli contexte
+        $infos = array();
+        foreach (array_filter($contexte) as $var => $val) {
+            if (is_array($val)) $val = serialize($val);
+            if (strlen("$val") > 30)
+                $val = substr("$val", 0,27).'..';
+            if (strstr($val,' '))
+                $val = "'$val'";
+            $infos[] = $var.'='.$val;
+        }
+        $profile = spip_timer($a);
+        spip_log("calcul ($profile) [$skel] "
+            . join(', ', $infos)
+            .' ('.strlen($page['texte']).' octets)');
+
+        if (defined('_CALCUL_PROFILER') AND intval($profile)>_CALCUL_PROFILER){
+            spip_log("calcul ($profile) [$skel] "
+                . join(', ', $infos)
+                .' ('.strlen($page['texte']).' octets) | '.$_SERVER['REQUEST_URI'],"profiler"._LOG_AVERTISSEMENT);
+        }
+
+        if ($debug) {
+        // si c'est ce que demande le debusqueur, lui passer la main
+            $t = strlen($page['texte']) ? $page['texte'] : " ";
+            $GLOBALS['debug_objets']['resultat'][$fonc . 'tout'] = $t;
+            $GLOBALS['debug_objets']['courant'] = $courant;
+            $GLOBALS['debug_objets']['profile'][$sourcefile] = $profile;
+            if ($GLOBALS['debug_objets']['sourcefile']
+            AND (_request('var_mode_objet') == $fonc)
+            AND (_request('var_mode_affiche') == 'resultat')) {
+                erreur_squelette();
+            }
+        }
+        // Si #CACHE{} n'etait pas la, le mettre a $delais
+        if (!isset($page['entetes']['X-Spip-Cache'])){
+            // Dans l'espace prive ou dans un modeles/ on pose un cache 0 par defaut
+            // si aucun #CACHE{} specifie
+            // le contexte implicite qui conditionne le cache assure qu'on retombe pas sur le meme
+            // entre public et prive
+            if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0)
+                $page['entetes']['X-Spip-Cache'] = 0;
+            else
+                $page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+        }
+
+        $page['contexte'] = $contexte;
 	  
-		// faire remonter le fichier source
-		static $js_inclus = false;
-		if (defined('_VAR_INCLURE') AND _VAR_INCLURE){
-			$page['sourcefile'] = $sourcefile;
-			$page['texte'] =
-				"<div class='inclure_blocs'><h6>".$page['sourcefile']."</h6>".$page['texte']."</div>"
-				. ($js_inclus?"":"<script type='text/javascript'>jQuery(function(){jQuery('.inclure_blocs > h6:first-child').hover(function(){jQuery(this).parent().addClass('hover')},function(){jQuery(this).parent().removeClass('hover')})});</script>");
-			$js_inclus = true;
-		}
-
-		// Si un modele contenait #SESSION, on note l'info dans $page
-		if (isset($GLOBALS['cache_utilise_session'])) {
-			$page['invalideurs']['session'] = $GLOBALS['cache_utilise_session'];
-			unset($GLOBALS['cache_utilise_session']);
-		}
-	}
-
-	if ($select) lang_select();
-
-	return $page;
+        // faire remonter le fichier source
+        static $js_inclus = false;
+        if (defined('_VAR_INCLURE') AND _VAR_INCLURE){
+            $page['sourcefile'] = $sourcefile;
+            $page['texte'] =
+                "<div class='inclure_blocs'><h6>".$page['sourcefile']."</h6>".$page['texte']."</div>"
+                . ($js_inclus?"":"<script type='text/javascript'>jQuery(function(){jQuery('.inclure_blocs > h6:first-child').hover(function(){jQuery(this).parent().addClass('hover')},function(){jQuery(this).parent().removeClass('hover')})});</script>");
+            $js_inclus = true;
+        }
+
+        // Si un modele contenait #SESSION, on note l'info dans $page
+        if (isset($GLOBALS['cache_utilise_session'])) {
+            $page['invalideurs']['session'] = $GLOBALS['cache_utilise_session'];
+            unset($GLOBALS['cache_utilise_session']);
+        }
+    }
+
+    if ($select) lang_select();
+
+    return $page;
 }
 
 
@@ -212,36 +212,36 @@ 
  */
 function tester_redirection($fond, $contexte, $connect)
 {
-	if ($fond == 'article'
-	  AND $id_article = intval($contexte['id_article'])) {
-		include_spip('public/quete'); // pour quete_virtuel et ses dependances
-		$m = quete_virtuel($id_article, $connect);
-		if (strlen($m)) {
-			include_spip('inc/texte');
-			// les navigateurs pataugent si l'URL est vide
-			if ($url = virtuel_redirige($m, true)){
-				// passer en url absolue car cette redirection pourra
-				// etre utilisee dans un contexte d'url qui change
-				// y compris url arbo
-				$status = 302;
-				if (defined('_STATUS_REDIRECTION_VIRTUEL'))
-					$status=_STATUS_REDIRECTION_VIRTUEL;
-				if (!preg_match(',^\w+:,', $url)) {
-					include_spip('inc/filtres_mini');
-					$url = url_absolue($url);
-				}
-				$url = str_replace('&amp;', '&', $url);
-				return array('texte' => "<"
-				. "?php include_spip('inc/headers');redirige_par_entete('"
-				. texte_script($url)
-				. "','',$status);"
-				. "?" . ">",
-					'process_ins' => 'php',
-				  'status' => $status);
-			}
-		}
-	}
-	return false;
+    if ($fond == 'article'
+      AND $id_article = intval($contexte['id_article'])) {
+        include_spip('public/quete'); // pour quete_virtuel et ses dependances
+        $m = quete_virtuel($id_article, $connect);
+        if (strlen($m)) {
+            include_spip('inc/texte');
+            // les navigateurs pataugent si l'URL est vide
+            if ($url = virtuel_redirige($m, true)){
+                // passer en url absolue car cette redirection pourra
+                // etre utilisee dans un contexte d'url qui change
+                // y compris url arbo
+                $status = 302;
+                if (defined('_STATUS_REDIRECTION_VIRTUEL'))
+                    $status=_STATUS_REDIRECTION_VIRTUEL;
+                if (!preg_match(',^\w+:,', $url)) {
+                    include_spip('inc/filtres_mini');
+                    $url = url_absolue($url);
+                }
+                $url = str_replace('&amp;', '&', $url);
+                return array('texte' => "<"
+                . "?php include_spip('inc/headers');redirige_par_entete('"
+                . texte_script($url)
+                . "','',$status);"
+                . "?" . ">",
+                    'process_ins' => 'php',
+                    'status' => $status);
+            }
+        }
+    }
+    return false;
 }
 
 ?>

Spacing +22 added lines, -22 removed lines

@@ -18,14 +18,14 @@ 
 // donc il faut l'inclure "en globals"
 if ($f = find_in_path('mes_fonctions.php')) {
 	global $dossier_squelettes;
-	include_once(_ROOT_CWD . $f);
+	include_once(_ROOT_CWD.$f);
 }
 
 if (@is_readable(_CACHE_PLUGINS_FCT)) {
 	// chargement optimise precompile
 	include_once(_CACHE_PLUGINS_FCT);
 }
-if (test_espace_prive ())
+if (test_espace_prive())
 	include_spip('inc/filtres_ecrire');
 
 # Determine le squelette associe a une requete 
@@ -42,8 +42,8 @@ 
 # En cas d'erreur process_ins est absent et texte est un tableau de 2 chaines
 
 // http://doc.spip.org/@public_parametrer_dist
-function public_parametrer_dist($fond, $contexte='', $cache='', $connect='')  {
-	static $composer,$styliser,$notes=null;
+function public_parametrer_dist($fond, $contexte = '', $cache = '', $connect = '') {
+	static $composer, $styliser, $notes = null;
 	$page = tester_redirection($fond, $contexte, $connect);
 	if ($page) return $page;
 
@@ -59,7 +59,7 @@ 
 
 	if (!$styliser)
 		$styliser = charger_fonction('styliser', 'public');
-	list($skel,$mime_type, $gram, $sourcefile) =
+	list($skel, $mime_type, $gram, $sourcefile) =
 		$styliser($fond, $contexte, $GLOBALS['spip_lang'], $connect);
 
 	if ($skel) {
@@ -87,14 +87,14 @@ 
 	else {
 		// Preparer l'appel de la fonction principale du squelette 
 
-		spip_timer($a = 'calcul page '.rand(0,1000));
+		spip_timer($a = 'calcul page '.rand(0, 1000));
 
 		// On cree un marqueur de notes unique lie a cette composition
 		// et on enregistre l'etat courant des globales de notes...
 		if (is_null($notes))
 			$notes = charger_fonction('notes', 'inc', true);
 		if ($notes)
-			$notes('','empiler');
+			$notes('', 'empiler');
 
 		// Rajouter d'office ces deux parametres
 		// (mais vaudrait mieux que le compilateur sache le simuler
@@ -118,7 +118,7 @@ 
 	// compte a chaque calcul d'un texte contenant un modele, mais seulement
 	// quand le modele serait calcule, et on aurait des resultats incoherents)
 		if ($notes)
-			$notes('','depiler');
+			$notes('', 'depiler');
 
 		// reinjecter en dynamique la pile des notes
 		// si il y a des inclure dynamiques
@@ -126,15 +126,15 @@ 
 		// la generalisation de cette injection permettrait de corriger le point juste au dessus
 		// en faisant remonter les notes a l'incluant (A tester et valider avant application)
 		if ($notes)
-			$page['notes'] = $notes('','sauver_etat');
+			$page['notes'] = $notes('', 'sauver_etat');
 
 		// spip_log: un joli contexte
 		$infos = array();
 		foreach (array_filter($contexte) as $var => $val) {
 			if (is_array($val)) $val = serialize($val);
 			if (strlen("$val") > 30)
-				$val = substr("$val", 0,27).'..';
-			if (strstr($val,' '))
+				$val = substr("$val", 0, 27).'..';
+			if (strstr($val, ' '))
 				$val = "'$val'";
 			$infos[] = $var.'='.$val;
 		}
@@ -143,16 +143,16 @@ 
 			. join(', ', $infos)
 			.' ('.strlen($page['texte']).' octets)');
 
-		if (defined('_CALCUL_PROFILER') AND intval($profile)>_CALCUL_PROFILER){
+		if (defined('_CALCUL_PROFILER') AND intval($profile) > _CALCUL_PROFILER) {
 			spip_log("calcul ($profile) [$skel] "
 				. join(', ', $infos)
-				.' ('.strlen($page['texte']).' octets) | '.$_SERVER['REQUEST_URI'],"profiler"._LOG_AVERTISSEMENT);
+				.' ('.strlen($page['texte']).' octets) | '.$_SERVER['REQUEST_URI'], "profiler"._LOG_AVERTISSEMENT);
 		}
 
 		if ($debug) {
 		// si c'est ce que demande le debusqueur, lui passer la main
 			$t = strlen($page['texte']) ? $page['texte'] : " ";
-			$GLOBALS['debug_objets']['resultat'][$fonc . 'tout'] = $t;
+			$GLOBALS['debug_objets']['resultat'][$fonc.'tout'] = $t;
 			$GLOBALS['debug_objets']['courant'] = $courant;
 			$GLOBALS['debug_objets']['profile'][$sourcefile] = $profile;
 			if ($GLOBALS['debug_objets']['sourcefile']
@@ -162,26 +162,26 @@ 
 			}
 		}
 		// Si #CACHE{} n'etait pas la, le mettre a $delais
-		if (!isset($page['entetes']['X-Spip-Cache'])){
+		if (!isset($page['entetes']['X-Spip-Cache'])) {
 			// Dans l'espace prive ou dans un modeles/ on pose un cache 0 par defaut
 			// si aucun #CACHE{} specifie
 			// le contexte implicite qui conditionne le cache assure qu'on retombe pas sur le meme
 			// entre public et prive
-			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0)
+			if (test_espace_prive() OR strncmp($fond, 'modeles/', 8) == 0)
 				$page['entetes']['X-Spip-Cache'] = 0;
 			else
-				$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+				$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais']) ? $GLOBALS['delais'] : 36000;
 		}
 
 		$page['contexte'] = $contexte;
 	  
 		// faire remonter le fichier source
 		static $js_inclus = false;
-		if (defined('_VAR_INCLURE') AND _VAR_INCLURE){
+		if (defined('_VAR_INCLURE') AND _VAR_INCLURE) {
 			$page['sourcefile'] = $sourcefile;
 			$page['texte'] =
 				"<div class='inclure_blocs'><h6>".$page['sourcefile']."</h6>".$page['texte']."</div>"
-				. ($js_inclus?"":"<script type='text/javascript'>jQuery(function(){jQuery('.inclure_blocs > h6:first-child').hover(function(){jQuery(this).parent().addClass('hover')},function(){jQuery(this).parent().removeClass('hover')})});</script>");
+				. ($js_inclus ? "" : "<script type='text/javascript'>jQuery(function(){jQuery('.inclure_blocs > h6:first-child').hover(function(){jQuery(this).parent().addClass('hover')},function(){jQuery(this).parent().removeClass('hover')})});</script>");
 			$js_inclus = true;
 		}
 
@@ -219,13 +219,13 @@ 
 		if (strlen($m)) {
 			include_spip('inc/texte');
 			// les navigateurs pataugent si l'URL est vide
-			if ($url = virtuel_redirige($m, true)){
+			if ($url = virtuel_redirige($m, true)) {
 				// passer en url absolue car cette redirection pourra
 				// etre utilisee dans un contexte d'url qui change
 				// y compris url arbo
 				$status = 302;
 				if (defined('_STATUS_REDIRECTION_VIRTUEL'))
-					$status=_STATUS_REDIRECTION_VIRTUEL;
+					$status = _STATUS_REDIRECTION_VIRTUEL;
 				if (!preg_match(',^\w+:,', $url)) {
 					include_spip('inc/filtres_mini');
 					$url = url_absolue($url);
@@ -235,7 +235,7 @@ 
 				. "?php include_spip('inc/headers');redirige_par_entete('"
 				. texte_script($url)
 				. "','',$status);"
-				. "?" . ">",
+				. "?".">",
 					'process_ins' => 'php',
 				  'status' => $status);
 			}

Braces +63 added lines, -38 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('inc/lang');
 
@@ -25,8 +27,9 @@ 
 	// chargement optimise precompile
 	include_once(_CACHE_PLUGINS_FCT);
 }
-if (test_espace_prive ())
+if (test_espace_prive ()) {
 	include_spip('inc/filtres_ecrire');
+}
 
 # Determine le squelette associe a une requete 
 # et l'applique sur le contexte, le nom du cache et le serveur
@@ -45,20 +48,26 @@ 
 function public_parametrer_dist($fond, $contexte='', $cache='', $connect='')  {
 	static $composer,$styliser,$notes=null;
 	$page = tester_redirection($fond, $contexte, $connect);
-	if ($page) return $page;
+	if ($page) {
+	    return $page;
+	}
 
-	if (isset($contexte['lang']))
-		$lang = $contexte['lang'];
-	elseif (!isset($lang))
-		$lang = $GLOBALS['meta']['langue_site'];
+	if (isset($contexte['lang'])) {
+			$lang = $contexte['lang'];
+	} elseif (!isset($lang)) {
+			$lang = $GLOBALS['meta']['langue_site'];
+	}
 
 	$select = ((!isset($GLOBALS['forcer_lang']) OR !$GLOBALS['forcer_lang']) AND $lang <> $GLOBALS['spip_lang']);
-	if ($select) $select = lang_select($lang);
+	if ($select) {
+	    $select = lang_select($lang);
+	}
 
 	$debug = (defined('_VAR_MODE') && _VAR_MODE == 'debug');
 
-	if (!$styliser)
-		$styliser = charger_fonction('styliser', 'public');
+	if (!$styliser) {
+			$styliser = charger_fonction('styliser', 'public');
+	}
 	list($skel,$mime_type, $gram, $sourcefile) =
 		$styliser($fond, $contexte, $GLOBALS['spip_lang'], $connect);
 
@@ -74,27 +83,29 @@ 
 		// charger le squelette en specifiant les langages cibles et source
 		// au cas il faudrait le compiler (source posterieure au resultat)
 
-		if (!$composer)
-			$composer = charger_fonction('composer', 'public');
+		if (!$composer) {
+					$composer = charger_fonction('composer', 'public');
+		}
 		$fonc = $composer($skel, $mime_type, $gram, $sourcefile, $connect);
+	} else {
+			$fonc = '';
 	}
-	else
-		$fonc = '';
 
 	if (!$fonc) { // squelette inconnu (==='') ou faux (===false)
 		$page = $fonc;
-	}
-	else {
+	} else {
 		// Preparer l'appel de la fonction principale du squelette 
 
 		spip_timer($a = 'calcul page '.rand(0,1000));
 
 		// On cree un marqueur de notes unique lie a cette composition
 		// et on enregistre l'etat courant des globales de notes...
-		if (is_null($notes))
-			$notes = charger_fonction('notes', 'inc', true);
-		if ($notes)
-			$notes('','empiler');
+		if (is_null($notes)) {
+					$notes = charger_fonction('notes', 'inc', true);
+		}
+		if ($notes) {
+					$notes('','empiler');
+		}
 
 		// Rajouter d'office ces deux parametres
 		// (mais vaudrait mieux que le compilateur sache le simuler
@@ -102,12 +113,16 @@ 
 		if (!isset($contexte['date'])) {
 			$contexte['date'] = date("Y-m-d H:i:s");
 			$contexte['date_default'] = true;
-		} else $contexte['date'] = normaliser_date($contexte['date'], true);
+		} else {
+		    $contexte['date'] = normaliser_date($contexte['date'], true);
+		}
 
 		if (!isset($contexte['date_redac'])) {
 			$contexte['date_redac'] = date("Y-m-d H:i:s");
 			$contexte['date_redac_default'] = true;
-		} else $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
+		} else {
+		    $contexte['date_redac'] = normaliser_date($contexte['date_redac'], true);
+		}
 
 	// Passer le nom du cache pour produire sa destruction automatique
 		$page = $fonc(array('cache' => $cache), array($contexte));
@@ -117,25 +132,31 @@ 
 	// etre dans son resultat, autrement elles ne seraient pas prises en
 	// compte a chaque calcul d'un texte contenant un modele, mais seulement
 	// quand le modele serait calcule, et on aurait des resultats incoherents)
-		if ($notes)
-			$notes('','depiler');
+		if ($notes) {
+					$notes('','depiler');
+		}
 
 		// reinjecter en dynamique la pile des notes
 		// si il y a des inclure dynamiques
 		// si la pile n'est pas vide
 		// la generalisation de cette injection permettrait de corriger le point juste au dessus
 		// en faisant remonter les notes a l'incluant (A tester et valider avant application)
-		if ($notes)
-			$page['notes'] = $notes('','sauver_etat');
+		if ($notes) {
+					$page['notes'] = $notes('','sauver_etat');
+		}
 
 		// spip_log: un joli contexte
 		$infos = array();
 		foreach (array_filter($contexte) as $var => $val) {
-			if (is_array($val)) $val = serialize($val);
-			if (strlen("$val") > 30)
-				$val = substr("$val", 0,27).'..';
-			if (strstr($val,' '))
-				$val = "'$val'";
+			if (is_array($val)) {
+			    $val = serialize($val);
+			}
+			if (strlen("$val") > 30) {
+							$val = substr("$val", 0,27).'..';
+			}
+			if (strstr($val,' ')) {
+							$val = "'$val'";
+			}
 			$infos[] = $var.'='.$val;
 		}
 		$profile = spip_timer($a);
@@ -167,10 +188,11 @@ 
 			// si aucun #CACHE{} specifie
 			// le contexte implicite qui conditionne le cache assure qu'on retombe pas sur le meme
 			// entre public et prive
-			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0)
-				$page['entetes']['X-Spip-Cache'] = 0;
-			else
-				$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+			if (test_espace_prive() OR strncmp($fond,'modeles/',8)==0) {
+							$page['entetes']['X-Spip-Cache'] = 0;
+			} else {
+							$page['entetes']['X-Spip-Cache'] = isset($GLOBALS['delais'])?$GLOBALS['delais']:36000;
+			}
 		}
 
 		$page['contexte'] = $contexte;
@@ -192,7 +214,9 @@ 
 		}
 	}
 
-	if ($select) lang_select();
+	if ($select) {
+	    lang_select();
+	}
 
 	return $page;
 }
@@ -224,8 +248,9 @@ 
 				// etre utilisee dans un contexte d'url qui change
 				// y compris url arbo
 				$status = 302;
-				if (defined('_STATUS_REDIRECTION_VIRTUEL'))
-					$status=_STATUS_REDIRECTION_VIRTUEL;
+				if (defined('_STATUS_REDIRECTION_VIRTUEL')) {
+									$status=_STATUS_REDIRECTION_VIRTUEL;
+				}
 				if (!preg_match(',^\w+:,', $url)) {
 					include_spip('inc/filtres_mini');
 					$url = url_absolue($url);

ecrire/public/tracer.php

Doc Comments +3 added lines

@@ -36,6 +36,9 @@
 }
 
 // http://doc.spip.org/@trace_query_chrono
+/**
+ * @param string $m2
+ */
 function trace_query_chrono($m1, $m2, $query, $result, $serveur='')
 {
 	static $tt = 0, $nb=0;

Indentation +114 added lines, -114 removed lines

@@ -15,135 +15,135 @@
 // http://doc.spip.org/@trace_query_start
 function trace_query_start()
 {
-	static $trace = '?';
-	if ($trace === '?') {
-		include_spip('inc/autoriser');
-		// gare au bouclage sur calcul de droits au premier appel
-		// A fortiori quand on demande une trace
-		$trace = isset($_GET['var_profile']) AND (autoriser('debug'));
-	}
-	return  $trace ?  microtime() : 0;
+    static $trace = '?';
+    if ($trace === '?') {
+        include_spip('inc/autoriser');
+        // gare au bouclage sur calcul de droits au premier appel
+        // A fortiori quand on demande une trace
+        $trace = isset($_GET['var_profile']) AND (autoriser('debug'));
+    }
+    return  $trace ?  microtime() : 0;
 }
 
 // http://doc.spip.org/@trace_query_end
 function trace_query_end($query, $start, $result, $erreur, $serveur=''){
-	if ($start)
-		trace_query_chrono($start, microtime(), $query, $result, $serveur);
-	// tracer les erreurs, sauf pour select, c'est fait dans abstract_sql
-	if ($erreur AND !preg_match('/^select\b/i', $query))
-		erreur_squelette(array(sql_errno($serveur), $erreur, $query));
-	return $result;
+    if ($start)
+        trace_query_chrono($start, microtime(), $query, $result, $serveur);
+    // tracer les erreurs, sauf pour select, c'est fait dans abstract_sql
+    if ($erreur AND !preg_match('/^select\b/i', $query))
+        erreur_squelette(array(sql_errno($serveur), $erreur, $query));
+    return $result;
 }
 
 // http://doc.spip.org/@trace_query_chrono
 function trace_query_chrono($m1, $m2, $query, $result, $serveur='')
 {
-	static $tt = 0, $nb=0;
-	global $tableau_des_temps;
-
-	include_spip('inc/filtres_mini');
-
-	$x = _request('var_mode_objet');
-	if (isset($GLOBALS['debug']['aucasou'])) {
-		list(, $boucle, $serveur, $contexte) = $GLOBALS['debug']['aucasou'];
-		if ($x AND !preg_match("/$boucle\$/", $x))
-			return;
-		if ($serveur) $boucle .= " ($serveur)";
-		$boucle = "<b>$boucle</b>";
-	} else {
-		if ($x) return;
-		$boucle = $contexte = '';
-	}
-
-	list($usec, $sec) = explode(" ", $m1);
-	list($usec2, $sec2) = explode(" ", $m2);
- 	$dt = $sec2 + $usec2 - $sec - $usec;
-	$tt += $dt;
-	$nb++;
-
-	$q = preg_replace('/([a-z)`])\s+([A-Z])/', "$1\n<br />$2",spip_htmlentities($query));
-	$e =  sql_explain($query, $serveur);
-	$r = str_replace('Resource id ','',(is_object($result)?get_class($result):$result));
-	$tableau_des_temps[] = array($dt, $nb, $boucle, $q, $e, $r, $contexte);
+    static $tt = 0, $nb=0;
+    global $tableau_des_temps;
+
+    include_spip('inc/filtres_mini');
+
+    $x = _request('var_mode_objet');
+    if (isset($GLOBALS['debug']['aucasou'])) {
+        list(, $boucle, $serveur, $contexte) = $GLOBALS['debug']['aucasou'];
+        if ($x AND !preg_match("/$boucle\$/", $x))
+            return;
+        if ($serveur) $boucle .= " ($serveur)";
+        $boucle = "<b>$boucle</b>";
+    } else {
+        if ($x) return;
+        $boucle = $contexte = '';
+    }
+
+    list($usec, $sec) = explode(" ", $m1);
+    list($usec2, $sec2) = explode(" ", $m2);
+        $dt = $sec2 + $usec2 - $sec - $usec;
+    $tt += $dt;
+    $nb++;
+
+    $q = preg_replace('/([a-z)`])\s+([A-Z])/', "$1\n<br />$2",spip_htmlentities($query));
+    $e =  sql_explain($query, $serveur);
+    $r = str_replace('Resource id ','',(is_object($result)?get_class($result):$result));
+    $tableau_des_temps[] = array($dt, $nb, $boucle, $q, $e, $r, $contexte);
 }
 
 
 function chrono_requete($temps)
 {
-	$total = 0;
-	$hors = "<i>" . _T('zbug_hors_compilation') . "</i>";
-	$t = $q = $n = $d = array();
-	// Totaliser les temps et completer le Explain
-	foreach ($temps as $key => $v) {
-		list($dt, $nb, $boucle, $query, $explain, $res, $contexte) = $v;
-		if (is_array($contexte)) {
-			$k = ($contexte[0] . " $boucle");
-			include_spip('public/compiler');
-			$env = reconstruire_contexte_compil($contexte);
-		} else $k = $env = $boucle;
-
-		$total += $dt;
-		$t[$key] = $dt;
-		$q[$key] = $nb;
-		$d[$k]+= $dt;
-		if ($k) @++$n[$k];
-
-		if (!is_array($explain))
-			$explain = array();
-		foreach($explain as $j => $v) {
-			$explain[$j] = "<tr><th>$j</th><td>"
-			  . str_replace(';','<br />',$v)
-			  . "</td></tr>";
-		}
-		$e = "<table class='explain'>"
-		. "<caption>"
-		. $query
-		. "</caption>"
-		. "<tr><th>Time</th><td>$dt</td></tr>"
-		. "<tr><th>Order</th><td>$nb</td></tr>"
-		. "<tr><th>Res</th><td>$res</td></tr>"
-		. join('', $explain)
-		. "</table>";
-
-		$temps[$key] = array($e, $env, $k);
-	}
-	// Trier par temps d'execution decroissant
-	array_multisort($t, SORT_DESC, $q, $temps);
-	arsort($d);
-	$i = 1;
-	$t = array();
-	// Fabriquer les liens de navigations dans le tableau des temps
-	foreach($temps as $k => $v) {
-		$titre = strip_tags($v[2]);
-		$href = quote_amp($GLOBALS['REQUEST_URI'])."#req$i";
-
-		$t[$v[2]][]= "<span class='spip-debug-arg'> "
-		. "<a title='$titre' href='$href'>$i</a>"
-		. '</span>'
-		. ((count($t[$v[2]]) % 10 == 9) ?  "<br />" : '');
-		$i++;
-	}
-
-	if ($d['']) {
-		$d[$hors] = $d[''];
-		$n[$hors] = $n[''];
-		$t[$hors] = $t[''];
-	}
-	unset($d['']);
-	// Fabriquer le tableau des liens de navigation dans le grand tableau
-	foreach ($d as $k => $v) {
-		$d[$k] =  $n[$k] . "</td><td>$k</td><td class='time'>$v</td><td class='liste-reqs'>"
-		  . join('',$t[$k]);
-	}
-
-	$navigation = array(_T('zbug_statistiques'),
-	  "<tr><td>"
-		. join("</td></tr>\n<tr><td>", $d)
-	  . "</td></tr>\n"
-	  .  (# _request('var_mode_objet') ? '' : 
-	     ("<tr><td>" .  count($temps) . "</td><td>" . _T('info_total') . '</td><td class="time">' . $total . "</td><td></td></tr>")));
+    $total = 0;
+    $hors = "<i>" . _T('zbug_hors_compilation') . "</i>";
+    $t = $q = $n = $d = array();
+    // Totaliser les temps et completer le Explain
+    foreach ($temps as $key => $v) {
+        list($dt, $nb, $boucle, $query, $explain, $res, $contexte) = $v;
+        if (is_array($contexte)) {
+            $k = ($contexte[0] . " $boucle");
+            include_spip('public/compiler');
+            $env = reconstruire_contexte_compil($contexte);
+        } else $k = $env = $boucle;
+
+        $total += $dt;
+        $t[$key] = $dt;
+        $q[$key] = $nb;
+        $d[$k]+= $dt;
+        if ($k) @++$n[$k];
+
+        if (!is_array($explain))
+            $explain = array();
+        foreach($explain as $j => $v) {
+            $explain[$j] = "<tr><th>$j</th><td>"
+                . str_replace(';','<br />',$v)
+                . "</td></tr>";
+        }
+        $e = "<table class='explain'>"
+        . "<caption>"
+        . $query
+        . "</caption>"
+        . "<tr><th>Time</th><td>$dt</td></tr>"
+        . "<tr><th>Order</th><td>$nb</td></tr>"
+        . "<tr><th>Res</th><td>$res</td></tr>"
+        . join('', $explain)
+        . "</table>";
+
+        $temps[$key] = array($e, $env, $k);
+    }
+    // Trier par temps d'execution decroissant
+    array_multisort($t, SORT_DESC, $q, $temps);
+    arsort($d);
+    $i = 1;
+    $t = array();
+    // Fabriquer les liens de navigations dans le tableau des temps
+    foreach($temps as $k => $v) {
+        $titre = strip_tags($v[2]);
+        $href = quote_amp($GLOBALS['REQUEST_URI'])."#req$i";
+
+        $t[$v[2]][]= "<span class='spip-debug-arg'> "
+        . "<a title='$titre' href='$href'>$i</a>"
+        . '</span>'
+        . ((count($t[$v[2]]) % 10 == 9) ?  "<br />" : '');
+        $i++;
+    }
+
+    if ($d['']) {
+        $d[$hors] = $d[''];
+        $n[$hors] = $n[''];
+        $t[$hors] = $t[''];
+    }
+    unset($d['']);
+    // Fabriquer le tableau des liens de navigation dans le grand tableau
+    foreach ($d as $k => $v) {
+        $d[$k] =  $n[$k] . "</td><td>$k</td><td class='time'>$v</td><td class='liste-reqs'>"
+            . join('',$t[$k]);
+    }
+
+    $navigation = array(_T('zbug_statistiques'),
+        "<tr><td>"
+        . join("</td></tr>\n<tr><td>", $d)
+        . "</td></tr>\n"
+        .  (# _request('var_mode_objet') ? '' : 
+            ("<tr><td>" .  count($temps) . "</td><td>" . _T('info_total') . '</td><td class="time">' . $total . "</td><td></td></tr>")));
 	  
-	return array($temps, $navigation);
+    return array($temps, $navigation);
 }
 
 ?>

Braces +27 added lines, -13 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 // http://doc.spip.org/@trace_query_start
 function trace_query_start()
@@ -27,11 +29,13 @@ 
 
 // http://doc.spip.org/@trace_query_end
 function trace_query_end($query, $start, $result, $erreur, $serveur=''){
-	if ($start)
-		trace_query_chrono($start, microtime(), $query, $result, $serveur);
+	if ($start) {
+			trace_query_chrono($start, microtime(), $query, $result, $serveur);
+	}
 	// tracer les erreurs, sauf pour select, c'est fait dans abstract_sql
-	if ($erreur AND !preg_match('/^select\b/i', $query))
-		erreur_squelette(array(sql_errno($serveur), $erreur, $query));
+	if ($erreur AND !preg_match('/^select\b/i', $query)) {
+			erreur_squelette(array(sql_errno($serveur), $erreur, $query));
+	}
 	return $result;
 }
 
@@ -46,12 +50,17 @@ 
 	$x = _request('var_mode_objet');
 	if (isset($GLOBALS['debug']['aucasou'])) {
 		list(, $boucle, $serveur, $contexte) = $GLOBALS['debug']['aucasou'];
-		if ($x AND !preg_match("/$boucle\$/", $x))
-			return;
-		if ($serveur) $boucle .= " ($serveur)";
+		if ($x AND !preg_match("/$boucle\$/", $x)) {
+					return;
+		}
+		if ($serveur) {
+		    $boucle .= " ($serveur)";
+		}
 		$boucle = "<b>$boucle</b>";
 	} else {
-		if ($x) return;
+		if ($x) {
+		    return;
+		}
 		$boucle = $contexte = '';
 	}
 
@@ -80,16 +89,21 @@ 
 			$k = ($contexte[0] . " $boucle");
 			include_spip('public/compiler');
 			$env = reconstruire_contexte_compil($contexte);
-		} else $k = $env = $boucle;
+		} else {
+		    $k = $env = $boucle;
+		}
 
 		$total += $dt;
 		$t[$key] = $dt;
 		$q[$key] = $nb;
 		$d[$k]+= $dt;
-		if ($k) @++$n[$k];
+		if ($k) {
+		    @++$n[$k];
+		}
 
-		if (!is_array($explain))
-			$explain = array();
+		if (!is_array($explain)) {
+					$explain = array();
+		}
 		foreach($explain as $j => $v) {
 			$explain[$j] = "<tr><th>$j</th><td>"
 			  . str_replace(';','<br />',$v)

Spacing +17 added lines, -17 removed lines

@@ -26,7 +26,7 @@ 
 }
 
 // http://doc.spip.org/@trace_query_end
-function trace_query_end($query, $start, $result, $erreur, $serveur=''){
+function trace_query_end($query, $start, $result, $erreur, $serveur = '') {
 	if ($start)
 		trace_query_chrono($start, microtime(), $query, $result, $serveur);
 	// tracer les erreurs, sauf pour select, c'est fait dans abstract_sql
@@ -36,9 +36,9 @@ 
 }
 
 // http://doc.spip.org/@trace_query_chrono
-function trace_query_chrono($m1, $m2, $query, $result, $serveur='')
+function trace_query_chrono($m1, $m2, $query, $result, $serveur = '')
 {
-	static $tt = 0, $nb=0;
+	static $tt = 0, $nb = 0;
 	global $tableau_des_temps;
 
 	include_spip('inc/filtres_mini');
@@ -61,9 +61,9 @@ 
 	$tt += $dt;
 	$nb++;
 
-	$q = preg_replace('/([a-z)`])\s+([A-Z])/', "$1\n<br />$2",spip_htmlentities($query));
-	$e =  sql_explain($query, $serveur);
-	$r = str_replace('Resource id ','',(is_object($result)?get_class($result):$result));
+	$q = preg_replace('/([a-z)`])\s+([A-Z])/', "$1\n<br />$2", spip_htmlentities($query));
+	$e = sql_explain($query, $serveur);
+	$r = str_replace('Resource id ', '', (is_object($result) ?get_class($result) : $result));
 	$tableau_des_temps[] = array($dt, $nb, $boucle, $q, $e, $r, $contexte);
 }
 
@@ -71,13 +71,13 @@ 
 function chrono_requete($temps)
 {
 	$total = 0;
-	$hors = "<i>" . _T('zbug_hors_compilation') . "</i>";
+	$hors = "<i>"._T('zbug_hors_compilation')."</i>";
 	$t = $q = $n = $d = array();
 	// Totaliser les temps et completer le Explain
 	foreach ($temps as $key => $v) {
 		list($dt, $nb, $boucle, $query, $explain, $res, $contexte) = $v;
 		if (is_array($contexte)) {
-			$k = ($contexte[0] . " $boucle");
+			$k = ($contexte[0]." $boucle");
 			include_spip('public/compiler');
 			$env = reconstruire_contexte_compil($contexte);
 		} else $k = $env = $boucle;
@@ -85,14 +85,14 @@ 
 		$total += $dt;
 		$t[$key] = $dt;
 		$q[$key] = $nb;
-		$d[$k]+= $dt;
+		$d[$k] += $dt;
 		if ($k) @++$n[$k];
 
 		if (!is_array($explain))
 			$explain = array();
-		foreach($explain as $j => $v) {
+		foreach ($explain as $j => $v) {
 			$explain[$j] = "<tr><th>$j</th><td>"
-			  . str_replace(';','<br />',$v)
+			  . str_replace(';', '<br />', $v)
 			  . "</td></tr>";
 		}
 		$e = "<table class='explain'>"
@@ -113,14 +113,14 @@ 
 	$i = 1;
 	$t = array();
 	// Fabriquer les liens de navigations dans le tableau des temps
-	foreach($temps as $k => $v) {
+	foreach ($temps as $k => $v) {
 		$titre = strip_tags($v[2]);
 		$href = quote_amp($GLOBALS['REQUEST_URI'])."#req$i";
 
-		$t[$v[2]][]= "<span class='spip-debug-arg'> "
+		$t[$v[2]][] = "<span class='spip-debug-arg'> "
 		. "<a title='$titre' href='$href'>$i</a>"
 		. '</span>'
-		. ((count($t[$v[2]]) % 10 == 9) ?  "<br />" : '');
+		. ((count($t[$v[2]]) % 10 == 9) ? "<br />" : '');
 		$i++;
 	}
 
@@ -132,8 +132,8 @@ 
 	unset($d['']);
 	// Fabriquer le tableau des liens de navigation dans le grand tableau
 	foreach ($d as $k => $v) {
-		$d[$k] =  $n[$k] . "</td><td>$k</td><td class='time'>$v</td><td class='liste-reqs'>"
-		  . join('',$t[$k]);
+		$d[$k] = $n[$k]."</td><td>$k</td><td class='time'>$v</td><td class='liste-reqs'>"
+		  . join('', $t[$k]);
 	}
 
 	$navigation = array(_T('zbug_statistiques'),
@@ -141,7 +141,7 @@ 
 		. join("</td></tr>\n<tr><td>", $d)
 	  . "</td></tr>\n"
 	  .  (# _request('var_mode_objet') ? '' : 
-	     ("<tr><td>" .  count($temps) . "</td><td>" . _T('info_total') . '</td><td class="time">' . $total . "</td><td></td></tr>")));
+	     ("<tr><td>".count($temps)."</td><td>"._T('info_total').'</td><td class="time">'.$total."</td><td></td></tr>")));
 	  
 	return array($temps, $navigation);
 }

ecrire/xml/analyser_dtd.php

Doc Comments +24 added lines

@@ -139,6 +139,9 @@ 
 }
 
 // http://doc.spip.org/@analyser_dtd_comment
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_comment($dtd, &$dtc, $grammaire){
 	// ejecter les commentaires, surtout quand ils contiennent du code.
 	// Option /s car sur plusieurs lignes parfois
@@ -149,6 +152,9 @@ 
 }
 
 // http://doc.spip.org/@analyser_dtd_pi
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_pi($dtd, &$dtc, $grammaire){
 	if (!preg_match('/^<\?.*?>\s*(.*)$/s', $dtd, $m))
 		return -10;
@@ -156,6 +162,9 @@ 
 }
 
 // http://doc.spip.org/@analyser_dtd_lexeme
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_lexeme($dtd, &$dtc, $grammaire){
 
 	if (!preg_match(_REGEXP_ENTITY_DEF,$dtd, $m))
@@ -181,6 +190,9 @@ 
 // ca ne depasse pas 3 ici.
 
 // http://doc.spip.org/@analyser_dtd_data
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_data($dtd, &$dtc, $grammaire){
 
 	if (!preg_match(_REGEXP_INCLUDE_USE,$dtd,$m))
@@ -196,6 +208,9 @@ 
 }
 
 // http://doc.spip.org/@analyser_dtd_notation
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_notation($dtd, &$dtc, $grammaire){
 	if (!preg_match('/^<!NOTATION.*?>\s*(.*)$/s',$dtd, $m))
 		return -8;
@@ -204,6 +219,9 @@ 
 }
 
 // http://doc.spip.org/@analyser_dtd_entity
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_entity($dtd, &$dtc, $grammaire)
 {
 	if (!preg_match(_REGEXP_ENTITY_DECL, $dtd, $m))
@@ -253,6 +271,9 @@ 
 // Fin du controle en finElement
 
 // http://doc.spip.org/@analyser_dtd_element
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_element($dtd, &$dtc, $grammaire)
 {
 	if (!preg_match('/^<!ELEMENT\s+([^>\s]+)([^>]*)>\s*(.*)$/s', $dtd, $m))
@@ -295,6 +316,9 @@ 
 
 
 // http://doc.spip.org/@analyser_dtd_attlist
+/**
+ * @param string $dtd
+ */
 function analyser_dtd_attlist($dtd, &$dtc, $grammaire)
 {
 	if (!preg_match('/^<!ATTLIST\s+(\S+)\s+([^>]*)>\s*(.*)/s', $dtd, $m))

prive/formulaires/login.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
 
 include_spip('base/abstract_sql');
 
+/**
+ * @param string $cible
+ */
 function is_url_prive($cible){
 	include_spip('inc/filtres_mini');
 	$path = parse_url(tester_url_absolue($cible)?$cible:url_absolue($cible));

Indentation +175 added lines, -175 removed lines

@@ -15,87 +15,87 @@ 
 include_spip('base/abstract_sql');
 
 function is_url_prive($cible){
-	include_spip('inc/filtres_mini');
-	$path = parse_url(tester_url_absolue($cible)?$cible:url_absolue($cible));
-	$path = (isset($path['path'])?$path['path']:'');
-	return strncmp(substr($path,-strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS))==0;
+    include_spip('inc/filtres_mini');
+    $path = parse_url(tester_url_absolue($cible)?$cible:url_absolue($cible));
+    $path = (isset($path['path'])?$path['path']:'');
+    return strncmp(substr($path,-strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS))==0;
 }
 
 function formulaires_login_charger_dist($cible="",$login="",$prive=null)
 {
-	$erreur = _request('var_erreur');
-
-	if (!$login) $login = strval(_request('var_login'));
-	// si on est deja identifie
-	if (!$login AND isset($GLOBALS['visiteur_session']['login'])) {
-		$login = $GLOBALS['visiteur_session']['login'];
-	}
-	// ou si on a un cookie admin
-	if (!$login) {
-		if (isset($_COOKIE['spip_admin'])
-		AND preg_match(",^@(.*)$,", $_COOKIE['spip_admin'], $regs))
-			$login = $regs[1];
-	}
-
-	include_spip('inc/auth');
-	$row = auth_informer_login($login);
-
-	// Construire l'environnement du squelette
-	// Ne pas proposer de "rester connecte quelques jours"
-	// si la duree de l'alea est inferieure a 12 h (valeur par defaut)
+    $erreur = _request('var_erreur');
+
+    if (!$login) $login = strval(_request('var_login'));
+    // si on est deja identifie
+    if (!$login AND isset($GLOBALS['visiteur_session']['login'])) {
+        $login = $GLOBALS['visiteur_session']['login'];
+    }
+    // ou si on a un cookie admin
+    if (!$login) {
+        if (isset($_COOKIE['spip_admin'])
+        AND preg_match(",^@(.*)$,", $_COOKIE['spip_admin'], $regs))
+            $login = $regs[1];
+    }
+
+    include_spip('inc/auth');
+    $row = auth_informer_login($login);
+
+    // Construire l'environnement du squelette
+    // Ne pas proposer de "rester connecte quelques jours"
+    // si la duree de l'alea est inferieure a 12 h (valeur par defaut)
 		
-	$valeurs = array(
-		'var_login' => $login,
-		'editable' => !$row,
-		'cnx' => isset($row['cnx']) ? $row['cnx'] : '',
-		'auth_http' => login_auth_http(),
-		'rester_connecte' => ((_RENOUVELLE_ALEA < 12*3600)? '' : ' '),
-		'_logo' => isset($row['logo']) ? $row['logo'] : '',
-		'_alea_actuel' => isset($row['alea_actuel'])?$row['alea_actuel']:'',
-		'_alea_futur' => isset($row['alea_futur'])?$row['alea_futur']:'',
-		'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
-		);
-
-	if ($erreur OR !isset($GLOBALS['visiteur_session']['id_auteur']) OR !$GLOBALS['visiteur_session']['id_auteur'])
-		$valeurs['editable'] = true;
-
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		include_spip('inc/autoriser');
-		$loge = autoriser('ecrire');
-	} 
-	else
-		$loge = (isset($GLOBALS['visiteur_session']['auth']) AND $GLOBALS['visiteur_session']['auth'] != '');
-
-	// Si on est connecte, appeler traiter()
-	// et lancer la redirection si besoin
-	if (!$valeurs['editable'] AND $loge) {
-		$traiter = charger_fonction('traiter','formulaires/login');
-		$res = $traiter($cible, $login, $prive);
-		$valeurs = array_merge($valeurs,$res);
-
-		if (isset($res['redirect']) and $res['redirect']){
-			include_spip('inc/headers');
-			# preparer un lien pour quand redirige_formulaire ne fonctionne pas
-			$m = redirige_formulaire($res['redirect']);
-			$valeurs['_deja_loge'] = inserer_attribut(
-				"<a>" . _T('login_par_ici') . "</a>$m",
-				'href', $res['redirect']
-			);
-		}
-	}
-	// en cas d'echec de cookie, inc_auth a renvoye vers le script de
-	// pose de cookie ; s'il n'est pas la, c'est echec cookie
-	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
-	// et pas un echec cookie.
-	if ($erreur == 'cookie') $valeurs['echec_cookie'] = ' ';
-	elseif ($erreur){
-		// une erreur d'un SSO indique dans la redirection vers ici
-		// mais il faut se proteger de toute tentative d'injection malveilante
-		include_spip('inc/texte');
-		$valeurs['message_erreur'] = safehtml($erreur);
-	}
-
-	return $valeurs;
+    $valeurs = array(
+        'var_login' => $login,
+        'editable' => !$row,
+        'cnx' => isset($row['cnx']) ? $row['cnx'] : '',
+        'auth_http' => login_auth_http(),
+        'rester_connecte' => ((_RENOUVELLE_ALEA < 12*3600)? '' : ' '),
+        '_logo' => isset($row['logo']) ? $row['logo'] : '',
+        '_alea_actuel' => isset($row['alea_actuel'])?$row['alea_actuel']:'',
+        '_alea_futur' => isset($row['alea_futur'])?$row['alea_futur']:'',
+        '_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
+        );
+
+    if ($erreur OR !isset($GLOBALS['visiteur_session']['id_auteur']) OR !$GLOBALS['visiteur_session']['id_auteur'])
+        $valeurs['editable'] = true;
+
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        include_spip('inc/autoriser');
+        $loge = autoriser('ecrire');
+    } 
+    else
+        $loge = (isset($GLOBALS['visiteur_session']['auth']) AND $GLOBALS['visiteur_session']['auth'] != '');
+
+    // Si on est connecte, appeler traiter()
+    // et lancer la redirection si besoin
+    if (!$valeurs['editable'] AND $loge) {
+        $traiter = charger_fonction('traiter','formulaires/login');
+        $res = $traiter($cible, $login, $prive);
+        $valeurs = array_merge($valeurs,$res);
+
+        if (isset($res['redirect']) and $res['redirect']){
+            include_spip('inc/headers');
+            # preparer un lien pour quand redirige_formulaire ne fonctionne pas
+            $m = redirige_formulaire($res['redirect']);
+            $valeurs['_deja_loge'] = inserer_attribut(
+                "<a>" . _T('login_par_ici') . "</a>$m",
+                'href', $res['redirect']
+            );
+        }
+    }
+    // en cas d'echec de cookie, inc_auth a renvoye vers le script de
+    // pose de cookie ; s'il n'est pas la, c'est echec cookie
+    // s'il est la, c'est probablement un bookmark sur bonjour=oui,
+    // et pas un echec cookie.
+    if ($erreur == 'cookie') $valeurs['echec_cookie'] = ' ';
+    elseif ($erreur){
+        // une erreur d'un SSO indique dans la redirection vers ici
+        // mais il faut se proteger de toute tentative d'injection malveilante
+        include_spip('inc/texte');
+        $valeurs['message_erreur'] = safehtml($erreur);
+    }
+
+    return $valeurs;
 }
 
 
@@ -104,116 +104,116 @@ 
 
 function login_auth_http()
 {
-	if (!$GLOBALS['ignore_auth_http']
-		AND _request('var_erreur')=='cookie' 
-		AND (!isset($_COOKIE['spip_session']) OR $_COOKIE['spip_session'] != 'test_echec_cookie')
-		AND (($GLOBALS['flag_sapi_name'] AND preg_match(",apache,i", @php_sapi_name()))
-			OR preg_match(",^Apache.* PHP,", $_SERVER['SERVER_SOFTWARE']))
-		// Attention dans le cas 'intranet' la proposition de se loger
-		// par auth_http peut conduire a l'echec.
-		AND !(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])))
-
-		return generer_url_action('cookie',"",false,true);
-	else 	return '';
+    if (!$GLOBALS['ignore_auth_http']
+        AND _request('var_erreur')=='cookie' 
+        AND (!isset($_COOKIE['spip_session']) OR $_COOKIE['spip_session'] != 'test_echec_cookie')
+        AND (($GLOBALS['flag_sapi_name'] AND preg_match(",apache,i", @php_sapi_name()))
+            OR preg_match(",^Apache.* PHP,", $_SERVER['SERVER_SOFTWARE']))
+        // Attention dans le cas 'intranet' la proposition de se loger
+        // par auth_http peut conduire a l'echec.
+        AND !(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])))
+
+        return generer_url_action('cookie',"",false,true);
+    else 	return '';
 }
 
 function formulaires_login_verifier_dist($cible="",$login="",$prive=null){
 	
-	$session_login = _request('var_login');
-	$session_password = _request('password');
-	$session_remember = _request('session_remember');
-
-	if (!$session_login) {
-		# pas de login saisi !
-		return array('var_login' => _T('info_obligatoire'));
-	}
-
-	// appeler auth_identifier_login qui va :
-	// - renvoyer un string si echec (message d'erreur)
-	// - un array decrivant l'auteur identifie si possible
-	// - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
-	include_spip('inc/auth');
-	$auteur = auth_identifier_login($session_login, $session_password);
-	// on arrive ici si on ne s'est pas identifie avec un SSO
-	if (!is_array($auteur)) {
-		$erreurs = array();
-		if (is_string($auteur))
-			$erreurs['var_login'] = $auteur;
-		include_spip('inc/cookie');
-		spip_setcookie("spip_admin", "", time() - 3600);
-		if (strlen($session_password))
-			$erreurs['password'] = _T('login_erreur_pass');
-		// sinon c'est un login en deux passe old style (ou js en panne)
-		// pas de message d'erreur
-		else 
-			$erreurs['password'] = ' ';
-		return
-			$erreurs;
-	}
-	// on a ete authentifie, construire la session
-	// en gerant la duree demandee pour son cookie 
-	if ($session_remember !== NULL)
-		$auteur['cookie'] = $session_remember;
-	auth_loger($auteur);
-
-	return (is_null($prive) ? is_url_prive($cible) : $prive)
-	?  login_autoriser() : array();
+    $session_login = _request('var_login');
+    $session_password = _request('password');
+    $session_remember = _request('session_remember');
+
+    if (!$session_login) {
+        # pas de login saisi !
+        return array('var_login' => _T('info_obligatoire'));
+    }
+
+    // appeler auth_identifier_login qui va :
+    // - renvoyer un string si echec (message d'erreur)
+    // - un array decrivant l'auteur identifie si possible
+    // - rediriger vers un SSO qui renverra in fine sur action/auth qui finira l'authentification
+    include_spip('inc/auth');
+    $auteur = auth_identifier_login($session_login, $session_password);
+    // on arrive ici si on ne s'est pas identifie avec un SSO
+    if (!is_array($auteur)) {
+        $erreurs = array();
+        if (is_string($auteur))
+            $erreurs['var_login'] = $auteur;
+        include_spip('inc/cookie');
+        spip_setcookie("spip_admin", "", time() - 3600);
+        if (strlen($session_password))
+            $erreurs['password'] = _T('login_erreur_pass');
+        // sinon c'est un login en deux passe old style (ou js en panne)
+        // pas de message d'erreur
+        else 
+            $erreurs['password'] = ' ';
+        return
+            $erreurs;
+    }
+    // on a ete authentifie, construire la session
+    // en gerant la duree demandee pour son cookie 
+    if ($session_remember !== NULL)
+        $auteur['cookie'] = $session_remember;
+    auth_loger($auteur);
+
+    return (is_null($prive) ? is_url_prive($cible) : $prive)
+    ?  login_autoriser() : array();
 }
 
 function login_autoriser()
 {
-	include_spip('inc/autoriser');
-	if (!autoriser('ecrire')){
-		$h = generer_url_action('logout','logout=prive&url='.urlencode(self()));
-		return array('message_erreur' => "<h1>"
-				._T('avis_erreur_visiteur')
-				."</h1><p>"
-				._T('texte_erreur_visiteur')
-				."</p><p class='retour'>[<a href='$h'>"
-				._T('icone_deconnecter')."</a>]</p>");
-	}
-	return array();
+    include_spip('inc/autoriser');
+    if (!autoriser('ecrire')){
+        $h = generer_url_action('logout','logout=prive&url='.urlencode(self()));
+        return array('message_erreur' => "<h1>"
+                ._T('avis_erreur_visiteur')
+                ."</h1><p>"
+                ._T('texte_erreur_visiteur')
+                ."</p><p class='retour'>[<a href='$h'>"
+                ._T('icone_deconnecter')."</a>]</p>");
+    }
+    return array();
 }
 
 function formulaires_login_traiter_dist($cible="",$login="",$prive=null){
-	$res = array();
-	// Si on se connecte dans l'espace prive, 
-	// ajouter "bonjour" (repere a peu pres les cookies desactives)
-	if (is_null($prive) ? is_url_prive($cible) : $prive) {
-		$cible = parametre_url($cible, 'bonjour', 'oui', '&');
-	}
-	if ($cible=='@page_auteur')
-		$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'],'auteur');
-
-	if ($cible) {
-		$cible = parametre_url($cible, 'var_login', '', '&');
-
-		// transformer la cible absolue en cible relative
-		// pour pas echouer quand la meta adresse_site est foireuse
-		if (strncmp($cible,$u = url_de_base(),strlen($u))==0){
-			$cible = "./".substr($cible,strlen($u));
-		}
-
-		// si c'est une url absolue, refuser la redirection
-		// sauf si cette securite est levee volontairement par le webmestre
-		elseif (tester_url_absolue($cible) AND !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
-			$cible = "";
-		}
-	}
-
-	// Si on est connecte, envoyer vers la destination
-	if ($cible AND ($cible!=self('&'))  AND ($cible!=self())) {
-		if (!headers_sent() AND !$_GET['var_mode']) {
-			include_spip('inc/headers');
-			$res['redirect'] = $cible;
-		} else {
-			$res['message_ok'] = inserer_attribut(
-				"<a>" . _T('login_par_ici') . "</a>",
-				'href', $cible
-			);
-		}
-	}
-	return $res;
+    $res = array();
+    // Si on se connecte dans l'espace prive, 
+    // ajouter "bonjour" (repere a peu pres les cookies desactives)
+    if (is_null($prive) ? is_url_prive($cible) : $prive) {
+        $cible = parametre_url($cible, 'bonjour', 'oui', '&');
+    }
+    if ($cible=='@page_auteur')
+        $cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'],'auteur');
+
+    if ($cible) {
+        $cible = parametre_url($cible, 'var_login', '', '&');
+
+        // transformer la cible absolue en cible relative
+        // pour pas echouer quand la meta adresse_site est foireuse
+        if (strncmp($cible,$u = url_de_base(),strlen($u))==0){
+            $cible = "./".substr($cible,strlen($u));
+        }
+
+        // si c'est une url absolue, refuser la redirection
+        // sauf si cette securite est levee volontairement par le webmestre
+        elseif (tester_url_absolue($cible) AND !defined('_AUTORISER_LOGIN_ABS_REDIRECT')) {
+            $cible = "";
+        }
+    }
+
+    // Si on est connecte, envoyer vers la destination
+    if ($cible AND ($cible!=self('&'))  AND ($cible!=self())) {
+        if (!headers_sent() AND !$_GET['var_mode']) {
+            include_spip('inc/headers');
+            $res['redirect'] = $cible;
+        } else {
+            $res['message_ok'] = inserer_attribut(
+                "<a>" . _T('login_par_ici') . "</a>",
+                'href', $cible
+            );
+        }
+    }
+    return $res;
 }
 
 ?>

Braces +39 added lines, -25 removed lines

@@ -10,7 +10,9 @@ 
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+    return;
+}
 
 include_spip('base/abstract_sql');
 
@@ -25,7 +27,9 @@ 
 {
 	$erreur = _request('var_erreur');
 
-	if (!$login) $login = strval(_request('var_login'));
+	if (!$login) {
+	    $login = strval(_request('var_login'));
+	}
 	// si on est deja identifie
 	if (!$login AND isset($GLOBALS['visiteur_session']['login'])) {
 		$login = $GLOBALS['visiteur_session']['login'];
@@ -33,8 +37,9 @@ 
 	// ou si on a un cookie admin
 	if (!$login) {
 		if (isset($_COOKIE['spip_admin'])
-		AND preg_match(",^@(.*)$,", $_COOKIE['spip_admin'], $regs))
-			$login = $regs[1];
+		AND preg_match(",^@(.*)$,", $_COOKIE['spip_admin'], $regs)) {
+					$login = $regs[1];
+		}
 	}
 
 	include_spip('inc/auth');
@@ -56,15 +61,16 @@ 
 		'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
 		);
 
-	if ($erreur OR !isset($GLOBALS['visiteur_session']['id_auteur']) OR !$GLOBALS['visiteur_session']['id_auteur'])
-		$valeurs['editable'] = true;
+	if ($erreur OR !isset($GLOBALS['visiteur_session']['id_auteur']) OR !$GLOBALS['visiteur_session']['id_auteur']) {
+			$valeurs['editable'] = true;
+	}
 
 	if (is_null($prive) ? is_url_prive($cible) : $prive) {
 		include_spip('inc/autoriser');
 		$loge = autoriser('ecrire');
-	} 
-	else
-		$loge = (isset($GLOBALS['visiteur_session']['auth']) AND $GLOBALS['visiteur_session']['auth'] != '');
+	} else {
+			$loge = (isset($GLOBALS['visiteur_session']['auth']) AND $GLOBALS['visiteur_session']['auth'] != '');
+	}
 
 	// Si on est connecte, appeler traiter()
 	// et lancer la redirection si besoin
@@ -87,8 +93,9 @@ 
 	// pose de cookie ; s'il n'est pas la, c'est echec cookie
 	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
 	// et pas un echec cookie.
-	if ($erreur == 'cookie') $valeurs['echec_cookie'] = ' ';
-	elseif ($erreur){
+	if ($erreur == 'cookie') {
+	    $valeurs['echec_cookie'] = ' ';
+	} elseif ($erreur){
 		// une erreur d'un SSO indique dans la redirection vers ici
 		// mais il faut se proteger de toute tentative d'injection malveilante
 		include_spip('inc/texte');
@@ -111,11 +118,13 @@ 
 			OR preg_match(",^Apache.* PHP,", $_SERVER['SERVER_SOFTWARE']))
 		// Attention dans le cas 'intranet' la proposition de se loger
 		// par auth_http peut conduire a l'echec.
-		AND !(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])))
-
+		AND !(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW']))) {
+	
 		return generer_url_action('cookie',"",false,true);
-	else 	return '';
-}
+	} else {
+	    return '';
+	}
+	}
 
 function formulaires_login_verifier_dist($cible="",$login="",$prive=null){
 	
@@ -137,23 +146,27 @@ 
 	// on arrive ici si on ne s'est pas identifie avec un SSO
 	if (!is_array($auteur)) {
 		$erreurs = array();
-		if (is_string($auteur))
-			$erreurs['var_login'] = $auteur;
+		if (is_string($auteur)) {
+					$erreurs['var_login'] = $auteur;
+		}
 		include_spip('inc/cookie');
 		spip_setcookie("spip_admin", "", time() - 3600);
-		if (strlen($session_password))
-			$erreurs['password'] = _T('login_erreur_pass');
+		if (strlen($session_password)) {
+					$erreurs['password'] = _T('login_erreur_pass');
+		}
 		// sinon c'est un login en deux passe old style (ou js en panne)
 		// pas de message d'erreur
-		else 
-			$erreurs['password'] = ' ';
+		else {
+					$erreurs['password'] = ' ';
+		}
 		return
 			$erreurs;
 	}
 	// on a ete authentifie, construire la session
 	// en gerant la duree demandee pour son cookie 
-	if ($session_remember !== NULL)
-		$auteur['cookie'] = $session_remember;
+	if ($session_remember !== NULL) {
+			$auteur['cookie'] = $session_remember;
+	}
 	auth_loger($auteur);
 
 	return (is_null($prive) ? is_url_prive($cible) : $prive)
@@ -182,8 +195,9 @@ 
 	if (is_null($prive) ? is_url_prive($cible) : $prive) {
 		$cible = parametre_url($cible, 'bonjour', 'oui', '&');
 	}
-	if ($cible=='@page_auteur')
-		$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'],'auteur');
+	if ($cible=='@page_auteur') {
+			$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'],'auteur');
+	}
 
 	if ($cible) {
 		$cible = parametre_url($cible, 'var_login', '', '&');

Spacing +25 added lines, -25 removed lines

@@ -14,14 +14,14 @@ 
 
 include_spip('base/abstract_sql');
 
-function is_url_prive($cible){
+function is_url_prive($cible) {
 	include_spip('inc/filtres_mini');
-	$path = parse_url(tester_url_absolue($cible)?$cible:url_absolue($cible));
-	$path = (isset($path['path'])?$path['path']:'');
-	return strncmp(substr($path,-strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS))==0;
+	$path = parse_url(tester_url_absolue($cible) ? $cible : url_absolue($cible));
+	$path = (isset($path['path']) ? $path['path'] : '');
+	return strncmp(substr($path, -strlen(_DIR_RESTREINT_ABS)), _DIR_RESTREINT_ABS, strlen(_DIR_RESTREINT_ABS)) == 0;
 }
 
-function formulaires_login_charger_dist($cible="",$login="",$prive=null)
+function formulaires_login_charger_dist($cible = "", $login = "", $prive = null)
 {
 	$erreur = _request('var_erreur');
 
@@ -49,10 +49,10 @@ 
 		'editable' => !$row,
 		'cnx' => isset($row['cnx']) ? $row['cnx'] : '',
 		'auth_http' => login_auth_http(),
-		'rester_connecte' => ((_RENOUVELLE_ALEA < 12*3600)? '' : ' '),
+		'rester_connecte' => ((_RENOUVELLE_ALEA < 12 * 3600) ? '' : ' '),
 		'_logo' => isset($row['logo']) ? $row['logo'] : '',
-		'_alea_actuel' => isset($row['alea_actuel'])?$row['alea_actuel']:'',
-		'_alea_futur' => isset($row['alea_futur'])?$row['alea_futur']:'',
+		'_alea_actuel' => isset($row['alea_actuel']) ? $row['alea_actuel'] : '',
+		'_alea_futur' => isset($row['alea_futur']) ? $row['alea_futur'] : '',
 		'_pipeline' => 'affiche_formulaire_login', // faire passer le formulaire dans un pipe dedie pour les methodes auth
 		);
 
@@ -69,16 +69,16 @@ 
 	// Si on est connecte, appeler traiter()
 	// et lancer la redirection si besoin
 	if (!$valeurs['editable'] AND $loge) {
-		$traiter = charger_fonction('traiter','formulaires/login');
+		$traiter = charger_fonction('traiter', 'formulaires/login');
 		$res = $traiter($cible, $login, $prive);
-		$valeurs = array_merge($valeurs,$res);
+		$valeurs = array_merge($valeurs, $res);
 
-		if (isset($res['redirect']) and $res['redirect']){
+		if (isset($res['redirect']) and $res['redirect']) {
 			include_spip('inc/headers');
 			# preparer un lien pour quand redirige_formulaire ne fonctionne pas
 			$m = redirige_formulaire($res['redirect']);
 			$valeurs['_deja_loge'] = inserer_attribut(
-				"<a>" . _T('login_par_ici') . "</a>$m",
+				"<a>"._T('login_par_ici')."</a>$m",
 				'href', $res['redirect']
 			);
 		}
@@ -88,7 +88,7 @@ 
 	// s'il est la, c'est probablement un bookmark sur bonjour=oui,
 	// et pas un echec cookie.
 	if ($erreur == 'cookie') $valeurs['echec_cookie'] = ' ';
-	elseif ($erreur){
+	elseif ($erreur) {
 		// une erreur d'un SSO indique dans la redirection vers ici
 		// mais il faut se proteger de toute tentative d'injection malveilante
 		include_spip('inc/texte');
@@ -105,7 +105,7 @@ 
 function login_auth_http()
 {
 	if (!$GLOBALS['ignore_auth_http']
-		AND _request('var_erreur')=='cookie' 
+		AND _request('var_erreur') == 'cookie' 
 		AND (!isset($_COOKIE['spip_session']) OR $_COOKIE['spip_session'] != 'test_echec_cookie')
 		AND (($GLOBALS['flag_sapi_name'] AND preg_match(",apache,i", @php_sapi_name()))
 			OR preg_match(",^Apache.* PHP,", $_SERVER['SERVER_SOFTWARE']))
@@ -113,11 +113,11 @@ 
 		// par auth_http peut conduire a l'echec.
 		AND !(isset($_SERVER['PHP_AUTH_USER']) AND isset($_SERVER['PHP_AUTH_PW'])))
 
-		return generer_url_action('cookie',"",false,true);
+		return generer_url_action('cookie', "", false, true);
 	else 	return '';
 }
 
-function formulaires_login_verifier_dist($cible="",$login="",$prive=null){
+function formulaires_login_verifier_dist($cible = "", $login = "", $prive = null) {
 	
 	$session_login = _request('var_login');
 	$session_password = _request('password');
@@ -163,8 +163,8 @@ 
 function login_autoriser()
 {
 	include_spip('inc/autoriser');
-	if (!autoriser('ecrire')){
-		$h = generer_url_action('logout','logout=prive&url='.urlencode(self()));
+	if (!autoriser('ecrire')) {
+		$h = generer_url_action('logout', 'logout=prive&url='.urlencode(self()));
 		return array('message_erreur' => "<h1>"
 				._T('avis_erreur_visiteur')
 				."</h1><p>"
@@ -175,23 +175,23 @@ 
 	return array();
 }
 
-function formulaires_login_traiter_dist($cible="",$login="",$prive=null){
+function formulaires_login_traiter_dist($cible = "", $login = "", $prive = null) {
 	$res = array();
 	// Si on se connecte dans l'espace prive, 
 	// ajouter "bonjour" (repere a peu pres les cookies desactives)
 	if (is_null($prive) ? is_url_prive($cible) : $prive) {
 		$cible = parametre_url($cible, 'bonjour', 'oui', '&');
 	}
-	if ($cible=='@page_auteur')
-		$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'],'auteur');
+	if ($cible == '@page_auteur')
+		$cible = generer_url_entite($GLOBALS['auteur_session']['id_auteur'], 'auteur');
 
 	if ($cible) {
 		$cible = parametre_url($cible, 'var_login', '', '&');
 
 		// transformer la cible absolue en cible relative
 		// pour pas echouer quand la meta adresse_site est foireuse
-		if (strncmp($cible,$u = url_de_base(),strlen($u))==0){
-			$cible = "./".substr($cible,strlen($u));
+		if (strncmp($cible, $u = url_de_base(), strlen($u)) == 0) {
+			$cible = "./".substr($cible, strlen($u));
 		}
 
 		// si c'est une url absolue, refuser la redirection
@@ -202,13 +202,13 @@ 
 	}
 
 	// Si on est connecte, envoyer vers la destination
-	if ($cible AND ($cible!=self('&'))  AND ($cible!=self())) {
+	if ($cible AND ($cible != self('&')) AND ($cible != self())) {
 		if (!headers_sent() AND !$_GET['var_mode']) {
 			include_spip('inc/headers');
 			$res['redirect'] = $cible;
 		} else {
 			$res['message_ok'] = inserer_attribut(
-				"<a>" . _T('login_par_ici') . "</a>",
+				"<a>"._T('login_par_ici')."</a>",
 				'href', $cible
 			);
 		}