Inspection of "Éviter une notice PHP en envoyant un logo." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 94b890...c7bfc4 )

by cam

created 2018-03-22 21:00 UTC

Status

Indentation +151 added lines, -151 removed lines patch added patch discarded remove patch

@@ -30,7 +30,7 @@  discard block
 block discarded – undo
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 global $logo_libelles;
@@ -49,89 +49,89 @@  discard block
 block discarded – undo
  * @return array               Variables d'environnement pour le fond
  */
 function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour = '', $options = array()) {
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	$objet = objet_type($objet);
-	$_id_objet = id_table_objet($objet);
+    $objet = objet_type($objet);
+    $_id_objet = id_table_objet($objet);
 
-	if (!is_array($options)) {
-		$options = unserialize($options);
-	}
+    if (!is_array($options)) {
+        $options = unserialize($options);
+    }
 
-	if (!isset($options['titre'])) {
-		$balise_img = chercher_filtre('balise_img');
-		$img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
-		$libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
-		$libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
-		if (isset($libelles[$libelle])) {
-			$libelle = $libelles[$libelle];
-		} elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
-			$libelle = _T($libelle);
-		} else {
-			$libelle = _L('Logo');
-		}
-		switch ($objet) {
-			case 'article':
-				$libelle .= ' ' . aider('logoart');
-				break;
-			case 'breve':
-				$libelle .= ' ' . aider('breveslogo');
-				break;
-			case 'rubrique':
-				$libelle .= ' ' . aider('rublogo');
-				break;
-			default:
-				break;
-		}
+    if (!isset($options['titre'])) {
+        $balise_img = chercher_filtre('balise_img');
+        $img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
+        $libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
+        $libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
+        if (isset($libelles[$libelle])) {
+            $libelle = $libelles[$libelle];
+        } elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
+            $libelle = _T($libelle);
+        } else {
+            $libelle = _L('Logo');
+        }
+        switch ($objet) {
+            case 'article':
+                $libelle .= ' ' . aider('logoart');
+                break;
+            case 'breve':
+                $libelle .= ' ' . aider('breveslogo');
+                break;
+            case 'rubrique':
+                $libelle .= ' ' . aider('rublogo');
+                break;
+            default:
+                break;
+        }
 
-		$options['titre'] = $img . $libelle;
-	}
-	if (!isset($options['editable'])) {
-		include_spip('inc/autoriser');
-		$options['editable'] = autoriser('iconifier', $objet, $id_objet);
-	}
+        $options['titre'] = $img . $libelle;
+    }
+    if (!isset($options['editable'])) {
+        include_spip('inc/autoriser');
+        $options['editable'] = autoriser('iconifier', $objet, $id_objet);
+    }
 
-	$res = array(
-		'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
-		'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
-		'objet' => $objet,
-		'id_objet' => $id_objet,
-		'_options' => $options,
-		'_show_upload_off' => '',
-	);
+    $res = array(
+        'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
+        'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
+        'objet' => $objet,
+        'id_objet' => $id_objet,
+        '_options' => $options,
+        '_show_upload_off' => '',
+    );
 
-	// rechercher le logo de l'objet
-	// la fonction prend un parametre '_id_objet' etrange :
-	// le nom de la cle primaire (et non le nom de la table)
-	// ou directement le nom du raccourcis a chercher
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	$etats = $res['logo_survol'] ? array('on', 'off') : array('on');
-	foreach ($etats as $etat) {
-		$logo = $chercher_logo($id_objet, $_id_objet, $etat);
-		if ($logo) {
-			$res['logo_' . $etat] = $logo[0];
-		}
-	}
-	// pas de logo_on -> pas de formulaire pour le survol
-	if (!isset($res['logo_on'])) {
-		$res['logo_survol'] = '';
-	} elseif (!isset($res['logo_off']) and _request('logo_up')) {
-		$res['_show_upload_off'] = ' ';
-	}
+    // rechercher le logo de l'objet
+    // la fonction prend un parametre '_id_objet' etrange :
+    // le nom de la cle primaire (et non le nom de la table)
+    // ou directement le nom du raccourcis a chercher
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    $etats = $res['logo_survol'] ? array('on', 'off') : array('on');
+    foreach ($etats as $etat) {
+        $logo = $chercher_logo($id_objet, $_id_objet, $etat);
+        if ($logo) {
+            $res['logo_' . $etat] = $logo[0];
+        }
+    }
+    // pas de logo_on -> pas de formulaire pour le survol
+    if (!isset($res['logo_on'])) {
+        $res['logo_survol'] = '';
+    } elseif (!isset($res['logo_off']) and _request('logo_up')) {
+        $res['_show_upload_off'] = ' ';
+    }
 
-	// si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
-	if (!$res['editable']
-		and !isset($res['logo_off'])
-		and !isset($res['logo_on'])
-	) {
-		return false;
-	}
+    // si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
+    if (!$res['editable']
+        and !isset($res['logo_off'])
+        and !isset($res['logo_on'])
+    ) {
+        return false;
+    }
 
-	return $res;
+    return $res;
 }
 
 /**
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
  * @return string              Hash du formulaire
  */
 function formulaires_editer_logo_identifier_dist($objet, $id_objet, $retour = '', $options = array()) {
-	return serialize(array($objet, $id_objet));
+    return serialize(array($objet, $id_objet));
 }
 
 /**
@@ -161,24 +161,24 @@  discard block
 block discarded – undo
  * @return array               Erreurs du formulaire
  */
 function formulaires_editer_logo_verifier_dist($objet, $id_objet, $retour = '', $options = array()) {
-	$erreurs = array();
-	// verifier les extensions
-	$sources = formulaire_editer_logo_get_sources();
-	foreach ($sources as $etat => $file) {
-		// seulement si une reception correcte a eu lieu
-		if ($file and $file['error'] == 0) {
-			if (!in_array(
-				strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
-				array('jpg', 'png', 'gif', 'jpeg')
-			)) {
-				$erreurs['logo_' . $etat] = _L('Extension non reconnue');
-			}
-		} elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
-			$erreurs['message_erreur'] = $file['msg'];
-		}
-	}
+    $erreurs = array();
+    // verifier les extensions
+    $sources = formulaire_editer_logo_get_sources();
+    foreach ($sources as $etat => $file) {
+        // seulement si une reception correcte a eu lieu
+        if ($file and $file['error'] == 0) {
+            if (!in_array(
+                strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
+                array('jpg', 'png', 'gif', 'jpeg')
+            )) {
+                $erreurs['logo_' . $etat] = _L('Extension non reconnue');
+            }
+        } elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
+            $erreurs['message_erreur'] = $file['msg'];
+        }
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 /**
@@ -194,47 +194,47 @@  discard block
 block discarded – undo
  * @return array               Retour des traitements
  */
 function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour = '', $options = array()) {
-	$res = array('editable' => ' ');
+    $res = array('editable' => ' ');
 
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	include_spip('action/editer_logo');
+    include_spip('action/editer_logo');
 
-	// effectuer la suppression si demandee d'un logo
-	$on = _request('supprimer_logo_on');
-	if ($on or _request('supprimer_logo_off')) {
-		logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
-		$res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
-		set_request('logo_up', ' ');
-	} // sinon supprimer ancien logo puis copier le nouveau
-	else {
-		$sources = formulaire_editer_logo_get_sources();
-		foreach ($sources as $etat => $file) {
-			if ($file and $file['error'] == 0) {
-				if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
-					$res['message_erreur'] = $err;
-				} else {
-					$res['message_ok'] = '';
-				} // pas besoin de message : la validation est visuelle
-				set_request('logo_up', ' ');
-			}
-		}
-	}
+    // effectuer la suppression si demandee d'un logo
+    $on = _request('supprimer_logo_on');
+    if ($on or _request('supprimer_logo_off')) {
+        logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
+        $res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
+        set_request('logo_up', ' ');
+    } // sinon supprimer ancien logo puis copier le nouveau
+    else {
+        $sources = formulaire_editer_logo_get_sources();
+        foreach ($sources as $etat => $file) {
+            if ($file and $file['error'] == 0) {
+                if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
+                    $res['message_erreur'] = $err;
+                } else {
+                    $res['message_ok'] = '';
+                } // pas besoin de message : la validation est visuelle
+                set_request('logo_up', ' ');
+            }
+        }
+    }
 
-	// Invalider les caches de l'objet
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='$objet/$id_objet'");
+    // Invalider les caches de l'objet
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='$objet/$id_objet'");
 
 
-	if ($retour) {
-		$res['redirect'] = $retour;
-	}
+    if ($retour) {
+        $res['redirect'] = $retour;
+    }
 
-	return $res;
+    return $res;
 }
 
 
@@ -246,29 +246,29 @@  discard block
 block discarded – undo
  *     Sources des fichiers dans les clés `on` ou `off`
  */
 function formulaire_editer_logo_get_sources() {
-	if (!$_FILES) {
-		$_FILES = isset($GLOBALS['HTTP_POST_FILES']) ? $GLOBALS['HTTP_POST_FILES'] : array();
-	}
-	if (!is_array($_FILES)) {
-		return array();
-	}
+    if (!$_FILES) {
+        $_FILES = isset($GLOBALS['HTTP_POST_FILES']) ? $GLOBALS['HTTP_POST_FILES'] : array();
+    }
+    if (!is_array($_FILES)) {
+        return array();
+    }
 
-	include_spip('inc/documents');
-	$sources = array();
-	foreach (array('on', 'off') as $etat) {
-		$logo = 'logo_' . $etat;
-		if (isset($_FILES[$logo])) {
-			if ($_FILES[$logo]['error'] == 0) {
-				$sources[$etat] = $_FILES[$logo];
-			} elseif ($_FILES[$logo]['error'] != 0) {
-				$msg = check_upload_error($_FILES[$logo]['error'], false, true);
-				if ($msg and is_string($msg)) {
-					$sources[$etat] = $_FILES[$logo];
-					$sources[$etat]['msg'] = $msg;
-				}
-			}
-		}
-	}
+    include_spip('inc/documents');
+    $sources = array();
+    foreach (array('on', 'off') as $etat) {
+        $logo = 'logo_' . $etat;
+        if (isset($_FILES[$logo])) {
+            if ($_FILES[$logo]['error'] == 0) {
+                $sources[$etat] = $_FILES[$logo];
+            } elseif ($_FILES[$logo]['error'] != 0) {
+                $msg = check_upload_error($_FILES[$logo]['error'], false, true);
+                if ($msg and is_string($msg)) {
+                    $sources[$etat] = $_FILES[$logo];
+                    $sources[$etat]['msg'] = $msg;
+                }
+            }
+        }
+    }
 
-	return $sources;
+    return $sources;
 }

Please login to merge, or discard this patch.

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -64,7 +64,7 @@  discard block
 block discarded – undo
 
 	if (!isset($options['titre'])) {
 		$balise_img = chercher_filtre('balise_img');
-		$img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
+		$img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
 		$libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
 		$libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
 		if (isset($libelles[$libelle])) {
@@ -76,19 +76,19 @@  discard block
 block discarded – undo
 		}
 		switch ($objet) {
 			case 'article':
-				$libelle .= ' ' . aider('logoart');
+				$libelle .= ' '.aider('logoart');
 				break;
 			case 'breve':
-				$libelle .= ' ' . aider('breveslogo');
+				$libelle .= ' '.aider('breveslogo');
 				break;
 			case 'rubrique':
-				$libelle .= ' ' . aider('rublogo');
+				$libelle .= ' '.aider('rublogo');
 				break;
 			default:
 				break;
 		}
 
-		$options['titre'] = $img . $libelle;
+		$options['titre'] = $img.$libelle;
 	}
 	if (!isset($options['editable'])) {
 		include_spip('inc/autoriser');
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
 	foreach ($etats as $etat) {
 		$logo = $chercher_logo($id_objet, $_id_objet, $etat);
 		if ($logo) {
-			$res['logo_' . $etat] = $logo[0];
+			$res['logo_'.$etat] = $logo[0];
 		}
 	}
 	// pas de logo_on -> pas de formulaire pour le survol
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
 				strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
 				array('jpg', 'png', 'gif', 'jpeg')
 			)) {
-				$erreurs['logo_' . $etat] = _L('Extension non reconnue');
+				$erreurs['logo_'.$etat] = _L('Extension non reconnue');
 			}
 		} elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
 			$erreurs['message_erreur'] = $file['msg'];
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
 	include_spip('inc/documents');
 	$sources = array();
 	foreach (array('on', 'off') as $etat) {
-		$logo = 'logo_' . $etat;
+		$logo = 'logo_'.$etat;
 		if (isset($_FILES[$logo])) {
 			if ($_FILES[$logo]['error'] == 0) {
 				$sources[$etat] = $_FILES[$logo];

Please login to merge, or discard this patch.

		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64
65	65	if (!isset($options['titre'])) {
66	66	$balise_img = chercher_filtre('balise_img');
67		- $img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
	67	+ $img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
68	68	$libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
69	69	$libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
70	70	if (isset($libelles[$libelle])) {
		@@ -76,19 +76,19 @@ discard block
		block discarded – undo
76	76	}
77	77	switch ($objet) {
78	78	case 'article':
79		- $libelle .= ' ' . aider('logoart');
	79	+ $libelle .= ' '.aider('logoart');
80	80	break;
81	81	case 'breve':
82		- $libelle .= ' ' . aider('breveslogo');
	82	+ $libelle .= ' '.aider('breveslogo');
83	83	break;
84	84	case 'rubrique':
85		- $libelle .= ' ' . aider('rublogo');
	85	+ $libelle .= ' '.aider('rublogo');
86	86	break;
87	87	default:
88	88	break;
89	89	}
90	90
91		- $options['titre'] = $img . $libelle;
	91	+ $options['titre'] = $img.$libelle;
92	92	}
93	93	if (!isset($options['editable'])) {
94	94	include_spip('inc/autoriser');
		@@ -113,7 +113,7 @@ discard block
		block discarded – undo
113	113	foreach ($etats as $etat) {
114	114	$logo = $chercher_logo($id_objet, $_id_objet, $etat);
115	115	if ($logo) {
116		- $res['logo_' . $etat] = $logo[0];
	116	+ $res['logo_'.$etat] = $logo[0];
117	117	}
118	118	}
119	119	// pas de logo_on -> pas de formulaire pour le survol
		@@ -171,7 +171,7 @@ discard block
		block discarded – undo
171	171	strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
172	172	array('jpg', 'png', 'gif', 'jpeg')
173	173	)) {
174		- $erreurs['logo_' . $etat] = _L('Extension non reconnue');
	174	+ $erreurs['logo_'.$etat] = _L('Extension non reconnue');
175	175	}
176	176	} elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
177	177	$erreurs['message_erreur'] = $file['msg'];
		@@ -256,7 +256,7 @@ discard block
		block discarded – undo
256	256	include_spip('inc/documents');
257	257	$sources = array();
258	258	foreach (array('on', 'off') as $etat) {
259		- $logo = 'logo_' . $etat;
	259	+ $logo = 'logo_'.$etat;
260	260	if (isset($_FILES[$logo])) {
261	261	if ($_FILES[$logo]['error'] == 0) {
262	262	$sources[$etat] = $_FILES[$logo];

		@@ -30,7 +30,7 @@ discard block
		block discarded – undo
30	30	**/
31	31
32	32	if (!defined('_ECRIRE_INC_VERSION')) {
33		- return;
	33	+ return;
34	34	}
35	35
36	36	global $logo_libelles;
		@@ -49,89 +49,89 @@ discard block
		block discarded – undo
49	49	* @return array Variables d'environnement pour le fond
50	50	*/
51	51	function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour = '', $options = array()) {
52		- // pas dans une boucle ? formulaire pour le logo du site
53		- // dans ce cas, il faut chercher un 'siteon0.ext'
54		- if (!$objet) {
55		- $objet = 'site';
56		- }
	52	+ // pas dans une boucle ? formulaire pour le logo du site
	53	+ // dans ce cas, il faut chercher un 'siteon0.ext'
	54	+ if (!$objet) {
	55	+ $objet = 'site';
	56	+ }
57	57
58		- $objet = objet_type($objet);
59		- $_id_objet = id_table_objet($objet);
	58	+ $objet = objet_type($objet);
	59	+ $_id_objet = id_table_objet($objet);
60	60
61		- if (!is_array($options)) {
62		- $options = unserialize($options);
63		- }
	61	+ if (!is_array($options)) {
	62	+ $options = unserialize($options);
	63	+ }
64	64
65		- if (!isset($options['titre'])) {
66		- $balise_img = chercher_filtre('balise_img');
67		- $img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
68		- $libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
69		- $libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
70		- if (isset($libelles[$libelle])) {
71		- $libelle = $libelles[$libelle];
72		- } elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
73		- $libelle = _T($libelle);
74		- } else {
75		- $libelle = _L('Logo');
76		- }
77		- switch ($objet) {
78		- case 'article':
79		- $libelle .= ' ' . aider('logoart');
80		- break;
81		- case 'breve':
82		- $libelle .= ' ' . aider('breveslogo');
83		- break;
84		- case 'rubrique':
85		- $libelle .= ' ' . aider('rublogo');
86		- break;
87		- default:
88		- break;
89		- }
	65	+ if (!isset($options['titre'])) {
	66	+ $balise_img = chercher_filtre('balise_img');
	67	+ $img = $balise_img(chemin_image('image-24.png'),'', 'cadre-icone');
	68	+ $libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
	69	+ $libelle = (($id_objet or $objet != 'rubrique') ? $objet : 'racine');
	70	+ if (isset($libelles[$libelle])) {
	71	+ $libelle = $libelles[$libelle];
	72	+ } elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
	73	+ $libelle = _T($libelle);
	74	+ } else {
	75	+ $libelle = _L('Logo');
	76	+ }
	77	+ switch ($objet) {
	78	+ case 'article':
	79	+ $libelle .= ' ' . aider('logoart');
	80	+ break;
	81	+ case 'breve':
	82	+ $libelle .= ' ' . aider('breveslogo');
	83	+ break;
	84	+ case 'rubrique':
	85	+ $libelle .= ' ' . aider('rublogo');
	86	+ break;
	87	+ default:
	88	+ break;
	89	+ }
90	90
91		- $options['titre'] = $img . $libelle;
92		- }
93		- if (!isset($options['editable'])) {
94		- include_spip('inc/autoriser');
95		- $options['editable'] = autoriser('iconifier', $objet, $id_objet);
96		- }
	91	+ $options['titre'] = $img . $libelle;
	92	+ }
	93	+ if (!isset($options['editable'])) {
	94	+ include_spip('inc/autoriser');
	95	+ $options['editable'] = autoriser('iconifier', $objet, $id_objet);
	96	+ }
97	97
98		- $res = array(
99		- 'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
100		- 'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
101		- 'objet' => $objet,
102		- 'id_objet' => $id_objet,
103		- '_options' => $options,
104		- '_show_upload_off' => '',
105		- );
	98	+ $res = array(
	99	+ 'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) or $options['editable']),
	100	+ 'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
	101	+ 'objet' => $objet,
	102	+ 'id_objet' => $id_objet,
	103	+ '_options' => $options,
	104	+ '_show_upload_off' => '',
	105	+ );
106	106
107		- // rechercher le logo de l'objet
108		- // la fonction prend un parametre '_id_objet' etrange :
109		- // le nom de la cle primaire (et non le nom de la table)
110		- // ou directement le nom du raccourcis a chercher
111		- $chercher_logo = charger_fonction('chercher_logo', 'inc');
112		- $etats = $res['logo_survol'] ? array('on', 'off') : array('on');
113		- foreach ($etats as $etat) {
114		- $logo = $chercher_logo($id_objet, $_id_objet, $etat);
115		- if ($logo) {
116		- $res['logo_' . $etat] = $logo[0];
117		- }
118		- }
119		- // pas de logo_on -> pas de formulaire pour le survol
120		- if (!isset($res['logo_on'])) {
121		- $res['logo_survol'] = '';
122		- } elseif (!isset($res['logo_off']) and _request('logo_up')) {
123		- $res['_show_upload_off'] = ' ';
124		- }
	107	+ // rechercher le logo de l'objet
	108	+ // la fonction prend un parametre '_id_objet' etrange :
	109	+ // le nom de la cle primaire (et non le nom de la table)
	110	+ // ou directement le nom du raccourcis a chercher
	111	+ $chercher_logo = charger_fonction('chercher_logo', 'inc');
	112	+ $etats = $res['logo_survol'] ? array('on', 'off') : array('on');
	113	+ foreach ($etats as $etat) {
	114	+ $logo = $chercher_logo($id_objet, $_id_objet, $etat);
	115	+ if ($logo) {
	116	+ $res['logo_' . $etat] = $logo[0];
	117	+ }
	118	+ }
	119	+ // pas de logo_on -> pas de formulaire pour le survol
	120	+ if (!isset($res['logo_on'])) {
	121	+ $res['logo_survol'] = '';
	122	+ } elseif (!isset($res['logo_off']) and _request('logo_up')) {
	123	+ $res['_show_upload_off'] = ' ';
	124	+ }
125	125
126		- // si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
127		- if (!$res['editable']
128		- and !isset($res['logo_off'])
129		- and !isset($res['logo_on'])
130		- ) {
131		- return false;
132		- }
	126	+ // si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
	127	+ if (!$res['editable']
	128	+ and !isset($res['logo_off'])
	129	+ and !isset($res['logo_on'])
	130	+ ) {
	131	+ return false;
	132	+ }
133	133
134		- return $res;
	134	+ return $res;
135	135	}
136	136
137	137	/**
		@@ -145,7 +145,7 @@ discard block
		block discarded – undo
145	145	* @return string Hash du formulaire
146	146	*/
147	147	function formulaires_editer_logo_identifier_dist($objet, $id_objet, $retour = '', $options = array()) {
148		- return serialize(array($objet, $id_objet));
	148	+ return serialize(array($objet, $id_objet));
149	149	}
150	150
151	151	/**
		@@ -161,24 +161,24 @@ discard block
		block discarded – undo
161	161	* @return array Erreurs du formulaire
162	162	*/
163	163	function formulaires_editer_logo_verifier_dist($objet, $id_objet, $retour = '', $options = array()) {
164		- $erreurs = array();
165		- // verifier les extensions
166		- $sources = formulaire_editer_logo_get_sources();
167		- foreach ($sources as $etat => $file) {
168		- // seulement si une reception correcte a eu lieu
169		- if ($file and $file['error'] == 0) {
170		- if (!in_array(
171		- strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
172		- array('jpg', 'png', 'gif', 'jpeg')
173		- )) {
174		- $erreurs['logo_' . $etat] = _L('Extension non reconnue');
175		- }
176		- } elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
177		- $erreurs['message_erreur'] = $file['msg'];
178		- }
179		- }
	164	+ $erreurs = array();
	165	+ // verifier les extensions
	166	+ $sources = formulaire_editer_logo_get_sources();
	167	+ foreach ($sources as $etat => $file) {
	168	+ // seulement si une reception correcte a eu lieu
	169	+ if ($file and $file['error'] == 0) {
	170	+ if (!in_array(
	171	+ strtolower(pathinfo($file['name'], PATHINFO_EXTENSION)),
	172	+ array('jpg', 'png', 'gif', 'jpeg')
	173	+ )) {
	174	+ $erreurs['logo_' . $etat] = _L('Extension non reconnue');
	175	+ }
	176	+ } elseif ($file and $file['error'] != 0 and isset($file['msg'])) {
	177	+ $erreurs['message_erreur'] = $file['msg'];
	178	+ }
	179	+ }
180	180
181		- return $erreurs;
	181	+ return $erreurs;
182	182	}
183	183
184	184	/**
		@@ -194,47 +194,47 @@ discard block
		block discarded – undo
194	194	* @return array Retour des traitements
195	195	*/
196	196	function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour = '', $options = array()) {
197		- $res = array('editable' => ' ');
	197	+ $res = array('editable' => ' ');
198	198
199		- // pas dans une boucle ? formulaire pour le logo du site
200		- // dans ce cas, il faut chercher un 'siteon0.ext'
201		- if (!$objet) {
202		- $objet = 'site';
203		- }
	199	+ // pas dans une boucle ? formulaire pour le logo du site
	200	+ // dans ce cas, il faut chercher un 'siteon0.ext'
	201	+ if (!$objet) {
	202	+ $objet = 'site';
	203	+ }
204	204
205		- include_spip('action/editer_logo');
	205	+ include_spip('action/editer_logo');
206	206
207		- // effectuer la suppression si demandee d'un logo
208		- $on = _request('supprimer_logo_on');
209		- if ($on or _request('supprimer_logo_off')) {
210		- logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
211		- $res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
212		- set_request('logo_up', ' ');
213		- } // sinon supprimer ancien logo puis copier le nouveau
214		- else {
215		- $sources = formulaire_editer_logo_get_sources();
216		- foreach ($sources as $etat => $file) {
217		- if ($file and $file['error'] == 0) {
218		- if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
219		- $res['message_erreur'] = $err;
220		- } else {
221		- $res['message_ok'] = '';
222		- } // pas besoin de message : la validation est visuelle
223		- set_request('logo_up', ' ');
224		- }
225		- }
226		- }
	207	+ // effectuer la suppression si demandee d'un logo
	208	+ $on = _request('supprimer_logo_on');
	209	+ if ($on or _request('supprimer_logo_off')) {
	210	+ logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
	211	+ $res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
	212	+ set_request('logo_up', ' ');
	213	+ } // sinon supprimer ancien logo puis copier le nouveau
	214	+ else {
	215	+ $sources = formulaire_editer_logo_get_sources();
	216	+ foreach ($sources as $etat => $file) {
	217	+ if ($file and $file['error'] == 0) {
	218	+ if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
	219	+ $res['message_erreur'] = $err;
	220	+ } else {
	221	+ $res['message_ok'] = '';
	222	+ } // pas besoin de message : la validation est visuelle
	223	+ set_request('logo_up', ' ');
	224	+ }
	225	+ }
	226	+ }
227	227
228		- // Invalider les caches de l'objet
229		- include_spip('inc/invalideur');
230		- suivre_invalideur("id='$objet/$id_objet'");
	228	+ // Invalider les caches de l'objet
	229	+ include_spip('inc/invalideur');
	230	+ suivre_invalideur("id='$objet/$id_objet'");
231	231
232	232
233		- if ($retour) {
234		- $res['redirect'] = $retour;
235		- }
	233	+ if ($retour) {
	234	+ $res['redirect'] = $retour;
	235	+ }
236	236
237		- return $res;
	237	+ return $res;
238	238	}
239	239
240	240
		@@ -246,29 +246,29 @@ discard block
		block discarded – undo
246	246	* Sources des fichiers dans les clés `on` ou `off`
247	247	*/
248	248	function formulaire_editer_logo_get_sources() {
249		- if (!$_FILES) {
250		- $_FILES = isset($GLOBALS['HTTP_POST_FILES']) ? $GLOBALS['HTTP_POST_FILES'] : array();
251		- }
252		- if (!is_array($_FILES)) {
253		- return array();
254		- }
	249	+ if (!$_FILES) {
	250	+ $_FILES = isset($GLOBALS['HTTP_POST_FILES']) ? $GLOBALS['HTTP_POST_FILES'] : array();
	251	+ }
	252	+ if (!is_array($_FILES)) {
	253	+ return array();
	254	+ }
255	255
256		- include_spip('inc/documents');
257		- $sources = array();
258		- foreach (array('on', 'off') as $etat) {
259		- $logo = 'logo_' . $etat;
260		- if (isset($_FILES[$logo])) {
261		- if ($_FILES[$logo]['error'] == 0) {
262		- $sources[$etat] = $_FILES[$logo];
263		- } elseif ($_FILES[$logo]['error'] != 0) {
264		- $msg = check_upload_error($_FILES[$logo]['error'], false, true);
265		- if ($msg and is_string($msg)) {
266		- $sources[$etat] = $_FILES[$logo];
267		- $sources[$etat]['msg'] = $msg;
268		- }
269		- }
270		- }
271		- }
	256	+ include_spip('inc/documents');
	257	+ $sources = array();
	258	+ foreach (array('on', 'off') as $etat) {
	259	+ $logo = 'logo_' . $etat;
	260	+ if (isset($_FILES[$logo])) {
	261	+ if ($_FILES[$logo]['error'] == 0) {
	262	+ $sources[$etat] = $_FILES[$logo];
	263	+ } elseif ($_FILES[$logo]['error'] != 0) {
	264	+ $msg = check_upload_error($_FILES[$logo]['error'], false, true);
	265	+ if ($msg and is_string($msg)) {
	266	+ $sources[$etat] = $_FILES[$logo];
	267	+ $sources[$etat]['msg'] = $msg;
	268	+ }
	269	+ }
	270	+ }
	271	+ }
272	272
273		- return $sources;
	273	+ return $sources;
274	274	}

spip / SPIP

Push — master ( 94b890...c7bfc4 )

Status

Category

Indentation +151 added lines, -151 removed lines patch added patch discarded remove patch

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch