Skip to content
Extraits de code Groupes Projets
Valider d96d4bfc rédigé par esj's avatar esj
Parcourir les fichiers

Symétrique de [7876]: ne pas utiliser une variable X s'il existe un... 

Symétrique de [7876]: ne pas utiliser une variable X s'il existe un charger_fonction(X). Et corrections de qq oublis.
parent 1e7ec0fc
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
Masquer les modifications d'espaces
En ligne Côte à côte
ecrire/exec/articles.php
+ 3
− 5
Voir le fichier @ d96d4bfc
......@@ -244,17 +244,15 @@ function boites_de_config_articles($id_article)
{
if (autoriser('moderer_forum', 'article', $id_article)) {
$regler_moderation = charger_fonction('regler_moderation', 'inc');
$regler_moderation =
$regler_moderation($id_article,"articles","id_article=$id_article");
$regler = $regler_moderation($id_article,"articles","id_article=$id_article");
}
if (autoriser('moderer_petition', 'article', $id_article)) {
$petitionner = charger_fonction('petitionner', 'inc');
$petitionner =
$petitionner($id_article,"articles","id_article=$id_article");
$petition = $petitionner($id_article,"articles","id_article=$id_article");
}
$masque = $regler_moderation . $petitionner;
$masque = $regler . $petition;
if (!$masque)
return '';
......
ecrire/exec/auteur_infos.php
+ 3
− 3
Voir le fichier @ d96d4bfc
......@@ -45,7 +45,7 @@ function exec_auteur_infos_dist()
}
$legender_auteur = charger_fonction('legender_auteur', 'inc');
$legender_auteur = $legender_auteur($id_auteur, $auteur, $initial, $echec, $redirect);
$legender_auteur_r = $legender_auteur($id_auteur, $auteur, $initial, $echec, $redirect);
if (_request('var_ajaxcharset')) ajax_retour($legender_auteur);
......@@ -71,7 +71,7 @@ function exec_auteur_infos_dist()
// charger ça tout de suite pour diposer de la fonction ci-dessous
$instituer_auteur = charger_fonction('instituer_auteur', 'inc');
$instituer_auteur = $instituer_auteur($id_auteur, $auteur['statut'], "auteurs_edit");
$instituer_auteur_r = $instituer_auteur($id_auteur, $auteur['statut'], "auteurs_edit");
if (statut_modifiable_auteur($id_auteur, $auteur) AND ($spip_display != 4)) {
$iconifier = charger_fonction('iconifier', 'inc');
......@@ -92,7 +92,7 @@ function exec_auteur_infos_dist()
echo
debut_cadre_relief("redacteurs-24.gif", true),
$legender_auteur, $instituer_auteur;
$legender_auteur_r, $instituer_auteur_r;
echo pipeline('affiche_milieu',
array('args' => array(
......
ecrire/inc/editer_auteurs.php
+ 4
− 4
Voir le fichier @ d96d4bfc
......@@ -52,10 +52,10 @@ function editer_auteurs_article($id_article, $flag, $cherche_auteur, $ids, $les_
if ($bouton_creer_auteur) {
$legender = redirige_action_auteur("legender_auteur", $arg . rawurlencode($cherche_auteur), "articles","id_article=$id_article");
$legende = redirige_action_auteur("legender_auteur", $arg . rawurlencode($cherche_auteur), "articles","id_article=$id_article");
$reponse .="<div style='width: 200px;'>"
. icone_horizontale($clic, $legender, "redacteurs-24.gif", "creer.gif", false)
. icone_horizontale($clic, $legende, "redacteurs-24.gif", "creer.gif", false)
. "</div> ";
$bouton_creer_auteur = false;
......@@ -76,11 +76,11 @@ function editer_auteurs_article($id_article, $flag, $cherche_auteur, $ids, $les_
if ($bouton_creer_auteur) {
$legender = redirige_action_auteur("legender_auteur",$arg, "articles","id_article=$id_article");
$legende = redirige_action_auteur("legender_auteur",$arg, "articles","id_article=$id_article");
$clic = "<span class='verdana1'><b>$clic</b></span>";
$res = "<div style='width:170px;'>"
. icone_horizontale($clic, $legender, "redacteurs-24.gif", "creer.gif", false)
. icone_horizontale($clic, $legende, "redacteurs-24.gif", "creer.gif", false)
. "</div>\n";
}
......
ecrire/inc/presentation.php
+ 9
− 7
Voir le fichier @ d96d4bfc
......@@ -646,7 +646,7 @@ function afficher_script_statut($id, $type, $n, $img, $statut, $title, $act)
// Afficher tableau d'articles
//
// http://doc.spip.org/@afficher_articles
function afficher_articles($titre, $requete, $formater_article='') {
function afficher_articles($titre, $requete, $formater='') {
if (!isset($requete['FROM'])) $requete['FROM'] = 'spip_articles AS articles';
......@@ -674,7 +674,7 @@ function afficher_articles($titre, $requete, $formater_article='') {
$id_ajax = $row["id_ajax_fonc"];
} else {
if (isset($requete['LIMIT'])) $cpt = min($requete['LIMIT'], $cpt);
$v = serialize(array($titre, $requete, $tmp_var, $formater_article));
$v = serialize(array($titre, $requete, $tmp_var, $formater));
include_spip ('base/abstract_sql');
$id_ajax = spip_abstract_insert("spip_ajax_fonc", "(variables, hash, date)", "(" . _q($v) . ", 0x$hash, NOW())");
......@@ -684,20 +684,22 @@ function afficher_articles($titre, $requete, $formater_article='') {
$requete['SELECT'] .= ", petitions.id_article AS petition ";
return afficher_articles_trad($titre, $requete, $formater_article, $tmp_var, $id_ajax, $cpt);
return afficher_articles_trad($titre, $requete, $formater, $tmp_var, $id_ajax, $cpt);
}
// http://doc.spip.org/@afficher_articles_trad
function afficher_articles_trad($titre_table, $requete, $formater_article, $tmp_var, $id_ajax, $cpt, $trad=0) {
function afficher_articles_trad($titre_table, $requete, $formater, $tmp_var, $id_ajax, $cpt, $trad=0) {
global $options, $spip_lang_right;
if ($trad) {
$formater_article = 'afficher_articles_trad_boucle';
$formater = 'afficher_articles_trad_boucle';
$icone = "langues-off-12.gif";
} else {
if (!$formater_article)
if (!$formater) {
$formater_article = charger_fonction('formater_article', 'inc');
$formater = $formater_article;
}
$icone = 'langues-12.gif';
}
......@@ -706,7 +708,7 @@ function afficher_articles_trad($titre_table, $requete, $formater_article, $tmp_
$q = spip_query("SELECT " . $requete['SELECT'] . " FROM " . $requete['FROM'] . ($requete['WHERE'] ? (' WHERE ' . $requete['WHERE']) : '') . ($requete['GROUP BY'] ? (' GROUP BY ' . $requete['GROUP BY']) : '') . ($requete['ORDER BY'] ? (' ORDER BY ' . $requete['ORDER BY']) : '') . " LIMIT " . ($deb_aff >= 0 ? "$deb_aff, $nb_aff" : ($requete['LIMIT'] ? $requete['LIMIT'] : "99999")));
$t = '';
while ($r = spip_fetch_array($q)) $t .= $formater_article($r);
while ($r = spip_fetch_array($q)) $t .= $formater($r);
spip_free_result($q);
$style = "style='visibility: hidden; float: $spip_lang_right'";
......
ecrire/inc/selectionner.php
+ 2
− 2
Voir le fichier @ d96d4bfc
......@@ -50,11 +50,11 @@ function inc_selectionner_dist ($sel, $idom="",$fonction="", $exclus=0, $aff_rac
}
$plonger = charger_fonction('plonger', 'inc');
$plonger = $plonger($sel, $idom, $recur, 1, $exclus);
$plonger_r = $plonger($sel, $idom, $recur, 1, $exclus);
// url completee par la fonction JS onkeypress_rechercher
$url = generer_url_ecrire('rechercher', "exclus=$exclus&rac=$idom&type=");
return construire_selectionner_hierarchie($idom, $plonger, $aff_racine, $url, 'id_parent', $url_init);
return construire_selectionner_hierarchie($idom, $plonger_r, $aff_racine, $url, 'id_parent', $url_init);
}
// http://doc.spip.org/@construire_selectionner_hierarchie
......
ecrire/index.php
+ 6
− 6
Voir le fichier @ d96d4bfc
......@@ -82,11 +82,11 @@ if (autoriser_sans_cookie($exec)) {
if (!isset($reinstall)) $reinstall = 'non';
$var_auth = true;
} else {
$var_auth = charger_fonction('auth', 'inc');
$var_auth = $var_auth();
if ($var_auth) {
$auth = charger_fonction('auth', 'inc');
$auth = $auth();
if ($auth) {
include_spip('inc/headers');
redirige_par_entete($var_auth);
redirige_par_entete($auth);
}
}
......@@ -146,8 +146,8 @@ if (isset($GLOBALS['_COOKIE']['spip_lang_ecrire'])) {
AND changer_langue($spip_lang_ecrire)) {
spip_query("UPDATE spip_auteurs SET lang = " . _q($spip_lang_ecrire) . " WHERE id_auteur = " . intval($GLOBALS['auteur_session']['id_auteur']));
$GLOBALS['auteur_session']['lang'] = $var_lang_ecrire;
$var_f = charger_fonction('session', 'inc');
$var_f($GLOBALS['auteur_session']);
$session = charger_fonction('session', 'inc');
$session($GLOBALS['auteur_session']);
}
}
}
......
ecrire/prive.php
+ 6
− 3
Voir le fichier @ d96d4bfc
......@@ -30,9 +30,12 @@ if (autoriser_sans_cookie($exec)) {
if (!isset($reinstall)) $reinstall = 'non';
$var_auth = true;
} else {
$var_auth = charger_fonction('auth', 'inc');
$var_auth = $var_auth();
if ($var_auth) redirige_par_entete($var_auth);
$auth = charger_fonction('auth', 'inc');
$auth = $auth();
if ($auth) {
include_spip('inc/headers');
redirige_par_entete($auth);
}
}
......
ecrire/public.php
+ 4
− 4
Voir le fichier @ d96d4bfc
......@@ -89,8 +89,8 @@ if (defined('_INC_PUBLIC')) {
//
$tableau_des_erreurs = array();
$f = charger_fonction('assembler', 'public');
$page = $f($fond);
$assembler = charger_fonction('assembler', 'public');
$page = $assembler($fond);
if (isset($page['status'])) {
include_spip('inc/headers');
......@@ -200,8 +200,8 @@ if (defined('_INC_PUBLIC')) {
// Gestion des statistiques du site public
if ($GLOBALS['meta']["activer_statistiques"] != "non") {
$f = charger_fonction('stats', 'public');
$f();
$stats = charger_fonction('stats', 'public');
$stats();
}
// Effectuer une tache de fond ?
......
0% ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter

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