Valider 88bca44c rédigé par esj's avatar esj
Parcourir les fichiers

Déclarations de globales incohérentes: soit nécessaires mais oubliés, soit inutiles. 

Si on compare avec #877, voila où on en est en matière de variable globales, avec le nouveau script suivant tenant compte de fichiers php dorénavant aussi dans dist et prive en plus de ecrire:

{{{
grep  "^[[:space:]]*global" [dep]*/*.php [dep]*/[a-km-z]*/*.php 2>/dev/null |tr ',; \t' '\n'|grep '$.'|sort  |wc -l
grep  "^[[:space:]]*global" [dep]*/*.php [dep]*/[a-km-z]*/*.php 2>/dev/null  |wc -l
cat [dep]*/*php [dep]*/[a-km-z]*/*php|tr "];,.(" "\n"|grep GLOBALS |wc -l
}}}

Ca donne 594, 375 et 1594 contre  1195, 557 et 1305 pour la 1.9.2. Autrement dit les 2500 utilisations de globales de la 1.9.2 ont été ramenées à 2200, et alors que l'intégralité du source est passé de moins 9Mo à près de 11Mo.
parent ba913c1b

ecrire/exec/articles.php

+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -55,7 +55,7 @@ function exec_articles_args($id_article) 
// http://doc.spip.org/@articles_affiche

function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot,  $select_groupe, $trad_err, $debut_forum=0, $statut_forum='prive')

{

	global $spip_display, $spip_lang_left, $spip_lang_right, $connect_id_auteur,$dir_lang;

	global $spip_lang_right, $dir_lang;



	$id_rubrique = $row['id_rubrique'];

	$id_secteur = $row['id_secteur'];

ecrire/exec/breves_voir.php

+0 −2
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -25,8 +25,6 @@ function exec_breves_voir_dist() 
// http://doc.spip.org/@exec_breves_voir_args

function exec_breves_voir_args($id_breve, $cherche_mot, $select_groupe)

{

	global $les_notes, $spip_display, $spip_lang_left, $spip_lang_right;



	$row = sql_fetsel("*", "spip_breves", "id_breve=$id_breve");

	if (!$row OR !autoriser('voir','breve',$id_breve)){

		include_spip('inc/minipres');

ecrire/exec/configuration.php

+0 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -16,7 +16,6 @@ include_spip('inc/presentation'); 


// http://doc.spip.org/@exec_configuration_dist

function exec_configuration_dist(){

	global $spip_display;



	if (!autoriser('configurer', 'configuration')) {

		include_spip('inc/minipres');

ecrire/exec/menu_rubriques.php

+3 −2
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -72,8 +72,9 @@ function exec_menu_rubriques_dist() { 


// http://doc.spip.org/@bandeau_rubrique

function bandeau_rubrique($id_rubrique, $titre_rubrique, $zdecal, $largeur, $image='') {

	global $spip_ecran, $spip_display;

	global $spip_lang, $spip_lang_rtl, $spip_lang_left, $spip_lang_right;



	global $spip_lang_left;



	static $zmax = 6;



	$nav = "<a href='"

ecrire/exec/mots_edit.php

+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -35,7 +35,7 @@ function exec_mots_edit_dist() 
// http://doc.spip.org/@exec_mots_edit_args

function exec_mots_edit_args($id_mot, $id_groupe, $new, $table='', $table_id='', $titre='', $redirect='', $ajouter_id_article='')

{

	global $spip_lang_right, $connect_statut, $spip_display, $les_notes;

	global $spip_lang_right, $connect_statut, $les_notes;



	$autoriser_editer = $editer = false;

	if ($new=='oui')

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site