Inspection of "Nom, nom_site et bio etant des champs librement mo..." - spip/SPIP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1da3a1...d54839 )

by cam

created 2021-10-21 08:38 UTC

Status

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

@@ -102,7 +102,7 @@  discard block
 block discarded – undo
 	return $r ? $r : traiter_lien_explicite($ref, $texte, $pour, $connect, $echappe_typo);
 }
 
-define('_EXTRAIRE_LIEN', ',^\s*(?:' . _PROTOCOLES_STD . '):?/?/?\s*$,iS');
+define('_EXTRAIRE_LIEN', ',^\s*(?:'._PROTOCOLES_STD.'):?/?/?\s*$,iS');
 
 // https://code.spip.net/@traiter_lien_explicite
 function traiter_lien_explicite($ref, $texte = '', $pour = 'url', $connect = '', $echappe_typo = true) {
@@ -119,19 +119,19 @@  discard block
 block discarded – undo
 		$lien_court = charger_fonction('lien_court', 'inc');
 		$texte = $lien_court($texte);
 		if ($echappe_typo) {
-			$texte = '<html>' . quote_amp($texte) . '</html>';
+			$texte = '<html>'.quote_amp($texte).'</html>';
 		}
 	}
 
 	// petites corrections d'URL
 	if (preg_match('/^www\.[^@]+$/S', $lien)) {
-		$lien = 'http://' . $lien;
+		$lien = 'http://'.$lien;
 	} else {
 		if (strpos($lien, '@') && email_valide($lien)) {
 			if (!$texte) {
 				$texte = $lien;
 			}
-			$lien = 'mailto:' . $lien;
+			$lien = 'mailto:'.$lien;
 		}
 	}
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
 }
 
 function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, $connect = '') {
-	if (function_exists($f = 'glossaire_' . $ancre)) {
+	if (function_exists($f = 'glossaire_'.$ancre)) {
 		$url = $f($texte, $id);
 	} else {
 		$url = glossaire_std($texte);
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
 	if (!($match = typer_raccourci($ref))) {
 		return false;
 	}
-	@list($type, , $id, , $args, , $ancre) = $match;
+	@list($type,, $id,, $args,, $ancre) = $match;
 	// attention dans le cas des sites le lien doit pointer non pas sur
 	// la page locale du site, mais directement sur le site lui-meme
 	if ($f = charger_fonction("implicite_$type", 'liens', true)) {
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
 		$r['titre'] = $texte;
 	}
 	if (!@$r['titre']) {
-		$r['titre'] = _T($type) . " $id";
+		$r['titre'] = _T($type)." $id";
 	}
 	if ($pour == 'titre') {
 		return $r['titre'];
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
 		and $mime = sql_getfetsel(
 			'mime_type',
 			'spip_types_documents',
-			'extension IN (' . sql_get_select('extension', 'spip_documents', 'id_document=' . sql_quote($id)) . ')',
+			'extension IN ('.sql_get_select('extension', 'spip_documents', 'id_document='.sql_quote($id)).')',
 			'',
 			'',
 			'',
@@ -304,11 +304,11 @@  discard block
 block discarded – undo
 	'(<([a-z_-]{3,})' # <modele
 	. '\s*([0-9]*)\s*' # id
 	. '([|](?:<[^<>]*>|[^>])*?)?' # |arguments (y compris des tags <...>)
-	. '\s*/?' . '>)' # fin du modele >
+	. '\s*/?'.'>)' # fin du modele >
 	. '\s*(<\/a>)?' # eventuel </a>
 );
 
-define('_RACCOURCI_MODELE_DEBUT', '@^' . _RACCOURCI_MODELE . '@isS');
+define('_RACCOURCI_MODELE_DEBUT', '@^'._RACCOURCI_MODELE.'@isS');
 
 // https://code.spip.net/@traiter_modeles
 function traiter_modeles($texte, $doublons = false, $echap = '', $connect = '', $liens = null, $env = []) {
@@ -388,7 +388,7 @@  discard block
 block discarded – undo
 						if (test_espace_prive()) {
 							$modele = entites_html(substr($texte, $a, $cherche));
 							if (!is_null($liens)) {
-								$modele = '<pre>' . str_replace($liens[0], $liens[1], $modele) . '</pre>';
+								$modele = '<pre>'.str_replace($liens[0], $liens[1], $modele).'</pre>';
 							}
 						}
 					}

Please login to merge, or discard this patch.

ecrire/inc/config.php 1 patch

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

@@ -335,7 +335,7 @@  discard block
 block discarded – undo
 			if (strncmp($url, 'configurer_', 11) == 0) {
 				$deja[$url] = $b;
 			} elseif ($b['url'] == 'configurer' and preg_match(',cfg=([a-z0-9_]+),i', $b['args'], $match)) {
-				$deja['configurer_' . $match[1]] = $b;
+				$deja['configurer_'.$match[1]] = $b;
 			}
 		}
 	}
@@ -348,10 +348,10 @@  discard block
 block discarded – undo
 
 	// trouver toutes les pages configurer_xxx de l'espace prive
 	// et construire un tableau des entrees qui ne sont pas dans $deja
-	$pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
 
 	foreach ($pages as $page) {
-		$configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, '.'._EXTENSION_SQUELETTES);
 		if (!isset($exclure[$configurer])) {
 			$liste[$configurer] = [
 				'parent' => 'bando_configuration',
@@ -371,9 +371,9 @@  discard block
 block discarded – undo
 
 	// trouver tous les formulaires/configurer_
 	// et construire un tableau des entrees
-	$pages = find_all_in_path('formulaires/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
+	$pages = find_all_in_path('formulaires/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
 	foreach ($pages as $page) {
-		$configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
+		$configurer = basename($page, '.'._EXTENSION_SQUELETTES);
 		if (
 			!isset($forms[$configurer])
 			and !isset($liste[$configurer])
@@ -382,7 +382,7 @@  discard block
 block discarded – undo
 			$liste[$configurer] = [
 				'parent' => 'bando_configuration',
 				'url' => 'configurer',
-				'args' => 'cfg=' . substr($configurer, 11),
+				'args' => 'cfg='.substr($configurer, 11),
 				'titre' => _T("configurer:{$configurer}_titre"),
 				'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
 			];
@@ -414,8 +414,8 @@  discard block
 block discarded – undo
 
 	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
 	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
-	$config = basename(substr($file, 0, -strlen('.' . _EXTENSION_SQUELETTES)));
-	spip_log('Calcul de ' . "prive/squelettes/contenu/$config");
+	$config = basename(substr($file, 0, -strlen('.'._EXTENSION_SQUELETTES)));
+	spip_log('Calcul de '."prive/squelettes/contenu/$config");
 	$fond = recuperer_fond("prive/squelettes/contenu/$config", ['exec' => $config]);
 
 	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...

Please login to merge, or discard this patch.

ecrire/inc/invalideur.php 1 patch

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

@@ -45,7 +45,7 @@  discard block
 block discarded – undo
 	$n = 0;
 	$time = isset($GLOBALS['meta']['cache_mark']) ? $GLOBALS['meta']['cache_mark'] : 0;
 	for ($i = 0; $i < 256; $i++) {
-		$dir = _DIR_CACHE . sprintf('%02s', dechex($i));
+		$dir = _DIR_CACHE.sprintf('%02s', dechex($i));
 		if (@is_dir($dir) and is_readable($dir) and $d = opendir($dir)) {
 			while (($f = readdir($d)) !== false) {
 				if (preg_match(',^[[0-9a-f]+\.cache$,S', $f) and $a = stat("$dir/$f")) {
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
 
 	// stocker la date_modif_$objet (ne sert a rien pour le moment)
 	if (isset($objet)) {
-		ecrire_meta('derniere_modif_' . $objet, time());
+		ecrire_meta('derniere_modif_'.$objet, time());
 	}
 
 	// si $derniere_modif_invalide est un array('article', 'rubrique')
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
 		)
 	) {
 		// supprimer le fichier (de facon propre)
-		supprimer_fichier(_DIR_CACHE . $cache);
+		supprimer_fichier(_DIR_CACHE.$cache);
 	} else {
 		spip_log("Nom de fichier cache incorrect : $cache");
 	}

Please login to merge, or discard this patch.

ecrire/inc/securiser_action.php 1 patch

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

@@ -91,7 +91,7 @@  discard block
 block discarded – undo
 	}
 
 	$arg = parametre_url($url_action, 'arg');
-	$confirm = md5("$action:$arg:" . realpath(__FILE__));
+	$confirm = md5("$action:$arg:".realpath(__FILE__));
 	if (_request('confirm_action') === $confirm) {
 		return true;
 	}
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
 		} else {
 			return generer_url_action(
 				$action,
-				'arg=' . rawurlencode($arg) . "&hash=$hash" . (!$r ? '' : "&redirect=$r"),
+				'arg='.rawurlencode($arg)."&hash=$hash".(!$r ? '' : "&redirect=$r"),
 				$mode,
 				$public
 			);
@@ -156,9 +156,9 @@  discard block
 block discarded – undo
 	$hash = calculer_action_auteur("$action-$arg");
 	$att .= " style='margin: 0px; border: 0px'";
 	if ($redirect) {
-		$redirect = "\n\t\t<input name='redirect' type='hidden' value='" . str_replace("'", '&#39;', $redirect) . "' />";
+		$redirect = "\n\t\t<input name='redirect' type='hidden' value='".str_replace("'", '&#39;', $redirect)."' />";
 	}
-	$mode .= $redirect . "
+	$mode .= $redirect."
 <input name='hash' type='hidden' value='$hash' />
 <input name='arg' type='hidden' value='$arg' />";
 
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
  */
 function _action_auteur($action, $id_auteur, $pass, $alea) {
 	static $sha = [];
-	if (!isset($sha[$id_auteur . $pass . $alea])) {
+	if (!isset($sha[$id_auteur.$pass.$alea])) {
 		if (!isset($GLOBALS['meta'][$alea])) {
 			if (!$exec = _request('exec') or !autoriser_sans_cookie($exec)) {
 				include_spip('inc/acces');
@@ -243,12 +243,12 @@  discard block
 block discarded – undo
 			}
 		}
 		include_spip('auth/sha256.inc');
-		$sha[$id_auteur . $pass . $alea] = spip_sha256($id_auteur . $pass . @$GLOBALS['meta'][$alea]);
+		$sha[$id_auteur.$pass.$alea] = spip_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
 	}
 	if (function_exists('sha1')) {
-		return sha1($action . $sha[$id_auteur . $pass . $alea]);
+		return sha1($action.$sha[$id_auteur.$pass.$alea]);
 	} else {
-		return md5($action . $sha[$id_auteur . $pass . $alea]);
+		return md5($action.$sha[$id_auteur.$pass.$alea]);
 	}
 }
 
@@ -332,9 +332,9 @@  discard block
 block discarded – undo
  */
 function calculer_cle_action($action) {
 	if (function_exists('sha1')) {
-		return sha1($action . secret_du_site());
+		return sha1($action.secret_du_site());
 	} else {
-		return md5($action . secret_du_site());
+		return md5($action.secret_du_site());
 	}
 }
 
@@ -374,7 +374,7 @@  discard block
 block discarded – undo
 	// On nettoie l’URL de tous les var_.
 	$url = nettoyer_uri_var($url);
 
-	$token = _action_auteur('previsualiser-' . $url, $id_auteur, null, $alea);
+	$token = _action_auteur('previsualiser-'.$url, $id_auteur, null, $alea);
 	return "$id_auteur-$token";
 }
 

Please login to merge, or discard this patch.

ecrire/inc/filtres_ecrire.php 1 patch

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

@@ -116,7 +116,7 @@  discard block
 block discarded – undo
 	if ($objet == 'rubrique') {
 		// si c'est une rubrique-secteur contenant des breves, demander la
 		// confirmation du deplacement
-		$contient_breves = sql_countsel('spip_breves', 'id_rubrique=' . intval($id_objet));
+		$contient_breves = sql_countsel('spip_breves', 'id_rubrique='.intval($id_objet));
 
 		if ($contient_breves > 0) {
 			$scb = ($contient_breves > 1 ? 's' : '');
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
 			);
 			$confirm .= "\n<div class='confirmer_deplacement verdana2'>"
 				. "<div class='choix'><input type='checkbox' name='confirme_deplace' value='oui' id='confirme-deplace' /><label for='confirme-deplace'>"
-				. $scb .
+				. $scb.
 				"</label></div></div>\n";
 		} else {
 			$confirm .= "<input type='hidden' name='confirme_deplace' value='oui' />\n";
@@ -138,11 +138,11 @@  discard block
 block discarded – undo
 	$form .= $confirm;
 	if ($actionable) {
 		if (strpos($form, '<select') !== false) {
-			$form .= "<div style='text-align: " . $GLOBALS['spip_lang_right'] . ";'>"
-				. '<input class="fondo submit btn" type="submit" value="' . _T('bouton_choisir') . '"/>'
+			$form .= "<div style='text-align: ".$GLOBALS['spip_lang_right'].";'>"
+				. '<input class="fondo submit btn" type="submit" value="'._T('bouton_choisir').'"/>'
 				. '</div>';
 		}
-		$form = "<input type='hidden' name='editer_$objet' value='oui' />\n" . $form;
+		$form = "<input type='hidden' name='editer_$objet' value='oui' />\n".$form;
 		if ($action = charger_fonction("editer_$objet", 'action', true)) {
 			$form = generer_action_auteur(
 				"editer_$objet",
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
 
 	include_spip('inc/presentation');
 
-	return debut_cadre_couleur($logo, true, '', $titre) . $form . fin_cadre_couleur(true);
+	return debut_cadre_couleur($logo, true, '', $titre).$form.fin_cadre_couleur(true);
 }
 
 
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
 	if ($statut == 'nouveau') {
 		if ($attente) {
 			$statut = $attente;
-			$plus = ' (' . _T('info_statut_auteur_a_confirmer') . ')';
+			$plus = ' ('._T('info_statut_auteur_a_confirmer').')';
 		} else {
 			return _T('info_statut_auteur_a_confirmer');
 		}
@@ -257,16 +257,16 @@  discard block
 block discarded – undo
 		'5poubelle' => _T('texte_statut_poubelle'), // bouh
 	];
 	if (isset($recom[$statut])) {
-		return $recom[$statut] . $plus;
+		return $recom[$statut].$plus;
 	}
 
 	// retrouver directement par le statut sinon
 	if ($t = array_search($statut, $GLOBALS['liste_des_statuts'])) {
 		if (isset($recom[$t])) {
-			return $recom[$t] . $plus;
+			return $recom[$t].$plus;
 		}
 
-		return _T($t) . $plus;
+		return _T($t).$plus;
 	}
 
 	// si on a pas reussi a le traduire, retournons la chaine telle quelle
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
 	if (!$id_rubrique and defined('_CHOIX_RUBRIQUE_PAR_DEFAUT') and _CHOIX_RUBRIQUE_PAR_DEFAUT) {
 		$in = !count($GLOBALS['connect_id_rubrique'])
 			? ''
-			: (' AND ' . sql_in('id_rubrique', $GLOBALS['connect_id_rubrique']));
+			: (' AND '.sql_in('id_rubrique', $GLOBALS['connect_id_rubrique']));
 
 		// on tente d'abord l'ecriture a la racine dans le cas des rubriques uniquement
 		if ($objet == 'rubrique') {
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
 		return '';
 	}
 
-	return propre('[->' . $virtuel . ']');
+	return propre('[->'.$virtuel.']');
 }
 
 
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
 	$args = param_low_sec($op, $args, $lang, 'rss');
 	$url = generer_url_public('rss', $args);
 
-	return "<a style='float: " . $GLOBALS['spip_lang_right'] . ";' href='$url'>$clic</a>";
+	return "<a style='float: ".$GLOBALS['spip_lang_right'].";' href='$url'>$clic</a>";
 }
 
 
@@ -528,7 +528,7 @@  discard block
 block discarded – undo
 	);
 
 	if ($alertes = array_filter($alertes)) {
-		return "<div class='wrap-messages-alertes'><div class='messages-alertes'>" .
+		return "<div class='wrap-messages-alertes'><div class='messages-alertes'>".
 		join(' | ', $alertes)
 		. '</div></div>';
 	}
@@ -562,13 +562,13 @@  discard block
 block discarded – undo
  */
 function afficher_plus_info($lien, $titre = '+', $titre_lien = '') {
 	$titre = attribut_html($titre);
-	$icone = "\n<a href='$lien' title='$titre' class='plus_info'>" .
-		http_img_pack('information-16.png', $titre) . '</a>';
+	$icone = "\n<a href='$lien' title='$titre' class='plus_info'>".
+		http_img_pack('information-16.png', $titre).'</a>';
 
 	if (!$titre_lien) {
 		return $icone;
 	} else {
-		return $icone . "\n<a href='$lien'>$titre_lien</a>";
+		return $icone."\n<a href='$lien'>$titre_lien</a>";
 	}
 }
 

Please login to merge, or discard this patch.

ecrire/inc/selectionner.php 1 patch

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

@@ -54,7 +54,7 @@  discard block
 block discarded – undo
 
 	if ($aff_racine) {
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . '_selection';
+		$idom3 = $idom.'_selection';
 
 		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
 
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
 			'  '
 		);
 
-		$js_func = $do . '_selection_titre';
+		$js_func = $do.'_selection_titre';
 		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
 
 		$aff_racine = "<div class='petit-item petite-racine item'>"
@@ -113,12 +113,12 @@  discard block
 block discarded – undo
  **/
 function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '') {
 
-	$idom1 = $idom . '_champ_recherche';
-	$idom2 = $idom . '_principal';
-	$idom3 = $idom . '_selection';
-	$idom4 = $idom . '_col_1';
-	$idom5 = 'img_' . $idom4;
-	$idom6 = $idom . '_fonc';
+	$idom1 = $idom.'_champ_recherche';
+	$idom2 = $idom.'_principal';
+	$idom3 = $idom.'_selection';
+	$idom4 = $idom.'_col_1';
+	$idom5 = 'img_'.$idom4;
+	$idom6 = $idom.'_fonc';
 
 	return "<div id='$idom'>"
 	. "<a id='$idom6' style='visibility: hidden;'"
@@ -128,10 +128,10 @@  discard block
 block discarded – undo
 	. http_img_pack(
 		'loader.svg',
 		'',
-		"class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
+		"class='loader' style='visibility: hidden;float:".$GLOBALS['spip_lang_right']."' id='$idom5'"
 	)
 	. ''
-	. "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
+	. "<input style='width: 10em;float:".$GLOBALS['spip_lang_right'].";' type='text' class='text search' id='$idom1' placeholder='"._T('info_rechercher')."'"
 	// eliminer Return car il provoque la soumission (balise unique)
 	// et eliminer Tab pour la navigation au clavier
 	// ce serait encore mieux de ne le faire que s'il y a encore plusieurs
@@ -178,8 +178,8 @@  discard block
 block discarded – undo
 
 	$liste = $id_rubrique;
 	$id_rubrique = intval($id_rubrique);
-	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
-		$liste = $id_rubrique . ",$liste";
+	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = '.$id_rubrique)) {
+		$liste = $id_rubrique.",$liste";
 	}
 
 	return explode(',', "0,$liste");

Please login to merge, or discard this patch.

ecrire/inc/notifications.php 1 patch

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

@@ -39,7 +39,7 @@  discard block
 block discarded – undo
 
 	if ($notification = charger_fonction($quoi, 'notifications', true)) {
 		spip_log("$notification($quoi,$id"
-			. ($options ? ',' . serialize($options) : '')
+			. ($options ? ','.serialize($options) : '')
 			. ')', 'notifications');
 		$notification($quoi, $id, $options);
 	}
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
 
 		job_queue_add(
 			'envoyer_mail',
-			">$email : " . $envoi['sujet'],
+			">$email : ".$envoi['sujet'],
 			[$email, $envoi['sujet'], $envoi['texte'], $envoi['from'], $envoi['headers']],
 			'inc/'
 		);

Please login to merge, or discard this patch.

ecrire/inc/precharger_objet.php 1 patch

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

@@ -50,7 +50,7 @@  discard block
 block discarded – undo
 
 	// si l'objet existe deja, on retourne simplement ses valeurs
 	if (is_numeric($id_objet)) {
-		return sql_fetsel('*', $table, "$_id_objet=" . intval($id_objet));
+		return sql_fetsel('*', $table, "$_id_objet=".intval($id_objet));
 	}
 
 	// ici, on demande une creation.
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
 	// si demande de traduction
 	// on recupere les valeurs de la traduction
 	if ($lier_trad) {
-		if ($select = charger_fonction('precharger_traduction_' . $type, 'inc', true)) {
+		if ($select = charger_fonction('precharger_traduction_'.$type, 'inc', true)) {
 			$row = $select($id_objet, $id_rubrique, $lier_trad);
 		} else {
 			$row = precharger_traduction_objet($type, $id_objet, $id_rubrique, $lier_trad, $champ_titre);
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
 	// recuperer le secteur, pour affecter les bons champs extras
 	if ($id_rubrique and $is_secteur) {
 		if (!$row['id_secteur']) {
-			$row_rub = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . sql_quote($id_rubrique));
+			$row_rub = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.sql_quote($id_rubrique));
 			$row['id_secteur'] = $row_rub;
 		}
 	}
@@ -133,10 +133,10 @@  discard block
 block discarded – undo
 	$_id_objet = id_table_objet($table);
 
 	// Recuperer les donnees de l'objet original
-	$row = sql_fetsel('*', $table, "$_id_objet=" . intval($lier_trad));
+	$row = sql_fetsel('*', $table, "$_id_objet=".intval($lier_trad));
 	if ($row) {
 		include_spip('inc/filtres');
-		$row[$champ_titre] = filtrer_entites(objet_T($type, 'info_nouvelle_traduction')) . ' ' . $row[$champ_titre];
+		$row[$champ_titre] = filtrer_entites(objet_T($type, 'info_nouvelle_traduction')).' '.$row[$champ_titre];
 	} else {
 		$row = [];
 	}
@@ -186,14 +186,14 @@  discard block
 block discarded – undo
 					$id_parent = 0;
 				} else {
 					// on cherche une rubrique soeur dans la bonne langue
-					$row_rub = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
+					$row_rub = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique));
 					$id_parent = $row_rub['id_parent'];
 				}
 
 				$row_rub = sql_fetsel(
 					'id_rubrique',
 					'spip_rubriques',
-					"lang='" . $GLOBALS['spip_lang'] . "' AND id_parent=" . intval($id_parent)
+					"lang='".$GLOBALS['spip_lang']."' AND id_parent=".intval($id_parent)
 				);
 				if ($row_rub) {
 					$row['id_rubrique'] = $row_rub['id_rubrique'];

Please login to merge, or discard this patch.

ecrire/inc/chercher_logo.php 1 patch

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

@@ -40,9 +40,9 @@  discard block
 block discarded – undo
 	$mode = preg_replace(',\W,', '', $mode);
 	if ($mode) {
 		// chercher dans la base
-		$mode_document = 'logo' . $mode;
+		$mode_document = 'logo'.$mode;
 		$objet = objet_type($_id_objet);
-		$doc = sql_fetsel('D.*', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode=' . sql_quote($mode_document) . ' AND L.objet=' . sql_quote($objet) . ' AND id_objet=' . intval($id));
+		$doc = sql_fetsel('D.*', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode='.sql_quote($mode_document).' AND L.objet='.sql_quote($objet).' AND id_objet='.intval($id));
 		if ($doc) {
 			include_spip('inc/documents');
 			$d = get_spip_doc($doc['fichier']);
@@ -53,10 +53,10 @@  discard block
 block discarded – undo
 		if ($compat_old_logos) {
 			# attention au cas $id = '0' pour LOGO_SITE_SPIP : utiliser intval()
 			$type = type_du_logo($_id_objet);
-			$nom = $type . $mode . intval($id);
+			$nom = $type.$mode.intval($id);
 
 			foreach ($GLOBALS['formats_logos'] as $format) {
-				if (@file_exists($d = (_DIR_LOGOS . $nom . '.' . $format))) {
+				if (@file_exists($d = (_DIR_LOGOS.$nom.'.'.$format))) {
 					return [$d, _DIR_LOGOS, $nom, $format, @filemtime($d)];
 				}
 			}

Please login to merge, or discard this patch.

		@@ -102,7 +102,7 @@ discard block
		block discarded – undo
102	102	return $r ? $r : traiter_lien_explicite($ref, $texte, $pour, $connect, $echappe_typo);
103	103	}
104	104
105		-define('_EXTRAIRE_LIEN', ',^\s(?:' . _PROTOCOLES_STD . '):?/?/?\s$,iS');
	105	+define('_EXTRAIRE_LIEN', ',^\s(?:'._PROTOCOLES_STD.'):?/?/?\s$,iS');
106	106
107	107	// https://code.spip.net/@traiter_lien_explicite
108	108	function traiter_lien_explicite($ref, $texte = '', $pour = 'url', $connect = '', $echappe_typo = true) {
		@@ -119,19 +119,19 @@ discard block
		block discarded – undo
119	119	$lien_court = charger_fonction('lien_court', 'inc');
120	120	$texte = $lien_court($texte);
121	121	if ($echappe_typo) {
122		- $texte = '<html>' . quote_amp($texte) . '</html>';
	122	+ $texte = '<html>'.quote_amp($texte).'</html>';
123	123	}
124	124	}
125	125
126	126	// petites corrections d'URL
127	127	if (preg_match('/^www\.[^@]+$/S', $lien)) {
128		- $lien = 'http://' . $lien;
	128	+ $lien = 'http://'.$lien;
129	129	} else {
130	130	if (strpos($lien, '@') && email_valide($lien)) {
131	131	if (!$texte) {
132	132	$texte = $lien;
133	133	}
134		- $lien = 'mailto:' . $lien;
	134	+ $lien = 'mailto:'.$lien;
135	135	}
136	136	}
137	137
		@@ -147,7 +147,7 @@ discard block
		block discarded – undo
147	147	}
148	148
149	149	function liens_implicite_glose_dist($texte, $id, $type, $args, $ancre, $connect = '') {
150		- if (function_exists($f = 'glossaire_' . $ancre)) {
	150	+ if (function_exists($f = 'glossaire_'.$ancre)) {
151	151	$url = $f($texte, $id);
152	152	} else {
153	153	$url = glossaire_std($texte);
		@@ -161,7 +161,7 @@ discard block
		block discarded – undo
161	161	if (!($match = typer_raccourci($ref))) {
162	162	return false;
163	163	}
164		- @list($type, , $id, , $args, , $ancre) = $match;
	164	+ @list($type,, $id,, $args,, $ancre) = $match;
165	165	// attention dans le cas des sites le lien doit pointer non pas sur
166	166	// la page locale du site, mais directement sur le site lui-meme
167	167	if ($f = charger_fonction("implicite_$type", 'liens', true)) {
		@@ -188,7 +188,7 @@ discard block
		block discarded – undo
188	188	$r['titre'] = $texte;
189	189	}
190	190	if (!@$r['titre']) {
191		- $r['titre'] = _T($type) . " $id";
	191	+ $r['titre'] = _T($type)." $id";
192	192	}
193	193	if ($pour == 'titre') {
194	194	return $r['titre'];
		@@ -201,7 +201,7 @@ discard block
		block discarded – undo
201	201	and $mime = sql_getfetsel(
202	202	'mime_type',
203	203	'spip_types_documents',
204		- 'extension IN (' . sql_get_select('extension', 'spip_documents', 'id_document=' . sql_quote($id)) . ')',
	204	+ 'extension IN ('.sql_get_select('extension', 'spip_documents', 'id_document='.sql_quote($id)).')',
205	205	'',
206	206	'',
207	207	'',
		@@ -304,11 +304,11 @@ discard block
		block discarded – undo
304	304	'(<([a-z_-]{3,})' # <modele
305	305	. '\s([0-9])\s*' # id
306	306	. '([\|](?:<[^<>]>\|[^>])?)?' # \|arguments (y compris des tags <...>)
307		- . '\s*/?' . '>)' # fin du modele >
	307	+ . '\s*/?'.'>)' # fin du modele >
308	308	. '\s*(<\/a>)?' # eventuel </a>
309	309	);
310	310
311		-define('_RACCOURCI_MODELE_DEBUT', '@^' . _RACCOURCI_MODELE . '@isS');
	311	+define('_RACCOURCI_MODELE_DEBUT', '@^'._RACCOURCI_MODELE.'@isS');
312	312
313	313	// https://code.spip.net/@traiter_modeles
314	314	function traiter_modeles($texte, $doublons = false, $echap = '', $connect = '', $liens = null, $env = []) {
		@@ -388,7 +388,7 @@ discard block
		block discarded – undo
388	388	if (test_espace_prive()) {
389	389	$modele = entites_html(substr($texte, $a, $cherche));
390	390	if (!is_null($liens)) {
391		- $modele = '<pre>' . str_replace($liens[0], $liens[1], $modele) . '</pre>';
	391	+ $modele = '<pre>'.str_replace($liens[0], $liens[1], $modele).'</pre>';
392	392	}
393	393	}
394	394	}

		@@ -335,7 +335,7 @@ discard block
		block discarded – undo
335	335	if (strncmp($url, 'configurer_', 11) == 0) {
336	336	$deja[$url] = $b;
337	337	} elseif ($b['url'] == 'configurer' and preg_match(',cfg=([a-z0-9_]+),i', $b['args'], $match)) {
338		- $deja['configurer_' . $match[1]] = $b;
	338	+ $deja['configurer_'.$match[1]] = $b;
339	339	}
340	340	}
341	341	}
		@@ -348,10 +348,10 @@ discard block
		block discarded – undo
348	348
349	349	// trouver toutes les pages configurer_xxx de l'espace prive
350	350	// et construire un tableau des entrees qui ne sont pas dans $deja
351		- $pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
	351	+ $pages = find_all_in_path('prive/squelettes/contenu/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
352	352
353	353	foreach ($pages as $page) {
354		- $configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
	354	+ $configurer = basename($page, '.'._EXTENSION_SQUELETTES);
355	355	if (!isset($exclure[$configurer])) {
356	356	$liste[$configurer] = [
357	357	'parent' => 'bando_configuration',
		@@ -371,9 +371,9 @@ discard block
		block discarded – undo
371	371
372	372	// trouver tous les formulaires/configurer_
373	373	// et construire un tableau des entrees
374		- $pages = find_all_in_path('formulaires/', 'configurer_.*[.]' . _EXTENSION_SQUELETTES . '$');
	374	+ $pages = find_all_in_path('formulaires/', 'configurer_.*[.]'._EXTENSION_SQUELETTES.'$');
375	375	foreach ($pages as $page) {
376		- $configurer = basename($page, '.' . _EXTENSION_SQUELETTES);
	376	+ $configurer = basename($page, '.'._EXTENSION_SQUELETTES);
377	377	if (
378	378	!isset($forms[$configurer])
379	379	and !isset($liste[$configurer])
		@@ -382,7 +382,7 @@ discard block
		block discarded – undo
382	382	$liste[$configurer] = [
383	383	'parent' => 'bando_configuration',
384	384	'url' => 'configurer',
385		- 'args' => 'cfg=' . substr($configurer, 11),
	385	+ 'args' => 'cfg='.substr($configurer, 11),
386	386	'titre' => _T("configurer:{$configurer}_titre"),
387	387	'icone' => find_in_theme($i = "images/{$configurer}-16.png") ? $i : $icone_defaut,
388	388	];
		@@ -414,8 +414,8 @@ discard block
		block discarded – undo
414	414
415	415	// evaluer le fond en lui passant un exec coherent pour que les pipelines le reconnaissent
416	416	// et reperer les formulaires CVT configurer_xx insereres par les plugins via pipeline
417		- $config = basename(substr($file, 0, -strlen('.' . _EXTENSION_SQUELETTES)));
418		- spip_log('Calcul de ' . "prive/squelettes/contenu/$config");
	417	+ $config = basename(substr($file, 0, -strlen('.'._EXTENSION_SQUELETTES)));
	418	+ spip_log('Calcul de '."prive/squelettes/contenu/$config");
419	419	$fond = recuperer_fond("prive/squelettes/contenu/$config", ['exec' => $config]);
420	420
421	421	// passer dans le pipeline affiche_milieu pour que les plugins puissent ajouter leur formulaires...

		@@ -91,7 +91,7 @@ discard block
		block discarded – undo
91	91	}
92	92
93	93	$arg = parametre_url($url_action, 'arg');
94		- $confirm = md5("$action:$arg:" . realpath(__FILE__));
	94	+ $confirm = md5("$action:$arg:".realpath(__FILE__));
95	95	if (_request('confirm_action') === $confirm) {
96	96	return true;
97	97	}
		@@ -145,7 +145,7 @@ discard block
		block discarded – undo
145	145	} else {
146	146	return generer_url_action(
147	147	$action,
148		- 'arg=' . rawurlencode($arg) . "&hash=$hash" . (!$r ? '' : "&redirect=$r"),
	148	+ 'arg='.rawurlencode($arg)."&hash=$hash".(!$r ? '' : "&redirect=$r"),
149	149	$mode,
150	150	$public
151	151	);
		@@ -156,9 +156,9 @@ discard block
		block discarded – undo
156	156	$hash = calculer_action_auteur("$action-$arg");
157	157	$att .= " style='margin: 0px; border: 0px'";
158	158	if ($redirect) {
159		- $redirect = "\n\t\t<input name='redirect' type='hidden' value='" . str_replace("'", ''', $redirect) . "' />";
	159	+ $redirect = "\n\t\t<input name='redirect' type='hidden' value='".str_replace("'", ''', $redirect)."' />";
160	160	}
161		- $mode .= $redirect . "
	161	+ $mode .= $redirect."
162	162	<input name='hash' type='hidden' value='$hash' />
163	163	<input name='arg' type='hidden' value='$arg' />";
164	164
		@@ -229,7 +229,7 @@ discard block
		block discarded – undo
229	229	*/
230	230	function _action_auteur($action, $id_auteur, $pass, $alea) {
231	231	static $sha = [];
232		- if (!isset($sha[$id_auteur . $pass . $alea])) {
	232	+ if (!isset($sha[$id_auteur.$pass.$alea])) {
233	233	if (!isset($GLOBALS['meta'][$alea])) {
234	234	if (!$exec = _request('exec') or !autoriser_sans_cookie($exec)) {
235	235	include_spip('inc/acces');
		@@ -243,12 +243,12 @@ discard block
		block discarded – undo
243	243	}
244	244	}
245	245	include_spip('auth/sha256.inc');
246		- $sha[$id_auteur . $pass . $alea] = spip_sha256($id_auteur . $pass . @$GLOBALS['meta'][$alea]);
	246	+ $sha[$id_auteur.$pass.$alea] = spip_sha256($id_auteur.$pass.@$GLOBALS['meta'][$alea]);
247	247	}
248	248	if (function_exists('sha1')) {
249		- return sha1($action . $sha[$id_auteur . $pass . $alea]);
	249	+ return sha1($action.$sha[$id_auteur.$pass.$alea]);
250	250	} else {
251		- return md5($action . $sha[$id_auteur . $pass . $alea]);
	251	+ return md5($action.$sha[$id_auteur.$pass.$alea]);
252	252	}
253	253	}
254	254
		@@ -332,9 +332,9 @@ discard block
		block discarded – undo
332	332	*/
333	333	function calculer_cle_action($action) {
334	334	if (function_exists('sha1')) {
335		- return sha1($action . secret_du_site());
	335	+ return sha1($action.secret_du_site());
336	336	} else {
337		- return md5($action . secret_du_site());
	337	+ return md5($action.secret_du_site());
338	338	}
339	339	}
340	340
		@@ -374,7 +374,7 @@ discard block
		block discarded – undo
374	374	// On nettoie l’URL de tous les var_.
375	375	$url = nettoyer_uri_var($url);
376	376
377		- $token = _action_auteur('previsualiser-' . $url, $id_auteur, null, $alea);
	377	+ $token = _action_auteur('previsualiser-'.$url, $id_auteur, null, $alea);
378	378	return "$id_auteur-$token";
379	379	}
380	380

		@@ -116,7 +116,7 @@ discard block
		block discarded – undo
116	116	if ($objet == 'rubrique') {
117	117	// si c'est une rubrique-secteur contenant des breves, demander la
118	118	// confirmation du deplacement
119		- $contient_breves = sql_countsel('spip_breves', 'id_rubrique=' . intval($id_objet));
	119	+ $contient_breves = sql_countsel('spip_breves', 'id_rubrique='.intval($id_objet));
120	120
121	121	if ($contient_breves > 0) {
122	122	$scb = ($contient_breves > 1 ? 's' : '');
		@@ -129,7 +129,7 @@ discard block
		block discarded – undo
129	129	);
130	130	$confirm .= "\n<div class='confirmer_deplacement verdana2'>"
131	131	. "<div class='choix'><input type='checkbox' name='confirme_deplace' value='oui' id='confirme-deplace' /><label for='confirme-deplace'>"
132		- . $scb .
	132	+ . $scb.
133	133	"</label></div></div>\n";
134	134	} else {
135	135	$confirm .= "<input type='hidden' name='confirme_deplace' value='oui' />\n";
		@@ -138,11 +138,11 @@ discard block
		block discarded – undo
138	138	$form .= $confirm;
139	139	if ($actionable) {
140	140	if (strpos($form, '<select') !== false) {
141		- $form .= "<div style='text-align: " . $GLOBALS['spip_lang_right'] . ";'>"
142		- . '<input class="fondo submit btn" type="submit" value="' . _T('bouton_choisir') . '"/>'
	141	+ $form .= "<div style='text-align: ".$GLOBALS['spip_lang_right'].";'>"
	142	+ . '<input class="fondo submit btn" type="submit" value="'._T('bouton_choisir').'"/>'
143	143	. '</div>';
144	144	}
145		- $form = "<input type='hidden' name='editer_$objet' value='oui' />\n" . $form;
	145	+ $form = "<input type='hidden' name='editer_$objet' value='oui' />\n".$form;
146	146	if ($action = charger_fonction("editer_$objet", 'action', true)) {
147	147	$form = generer_action_auteur(
148	148	"editer_$objet",
		@@ -168,7 +168,7 @@ discard block
		block discarded – undo
168	168
169	169	include_spip('inc/presentation');
170	170
171		- return debut_cadre_couleur($logo, true, '', $titre) . $form . fin_cadre_couleur(true);
	171	+ return debut_cadre_couleur($logo, true, '', $titre).$form.fin_cadre_couleur(true);
172	172	}
173	173
174	174
		@@ -244,7 +244,7 @@ discard block
		block discarded – undo
244	244	if ($statut == 'nouveau') {
245	245	if ($attente) {
246	246	$statut = $attente;
247		- $plus = ' (' . _T('info_statut_auteur_a_confirmer') . ')';
	247	+ $plus = ' ('._T('info_statut_auteur_a_confirmer').')';
248	248	} else {
249	249	return _T('info_statut_auteur_a_confirmer');
250	250	}
		@@ -257,16 +257,16 @@ discard block
		block discarded – undo
257	257	'5poubelle' => _T('texte_statut_poubelle'), // bouh
258	258	];
259	259	if (isset($recom[$statut])) {
260		- return $recom[$statut] . $plus;
	260	+ return $recom[$statut].$plus;
261	261	}
262	262
263	263	// retrouver directement par le statut sinon
264	264	if ($t = array_search($statut, $GLOBALS['liste_des_statuts'])) {
265	265	if (isset($recom[$t])) {
266		- return $recom[$t] . $plus;
	266	+ return $recom[$t].$plus;
267	267	}
268	268
269		- return _T($t) . $plus;
	269	+ return _T($t).$plus;
270	270	}
271	271
272	272	// si on a pas reussi a le traduire, retournons la chaine telle quelle
		@@ -390,7 +390,7 @@ discard block
		block discarded – undo
390	390	if (!$id_rubrique and defined('_CHOIX_RUBRIQUE_PAR_DEFAUT') and _CHOIX_RUBRIQUE_PAR_DEFAUT) {
391	391	$in = !count($GLOBALS['connect_id_rubrique'])
392	392	? ''
393		- : (' AND ' . sql_in('id_rubrique', $GLOBALS['connect_id_rubrique']));
	393	+ : (' AND '.sql_in('id_rubrique', $GLOBALS['connect_id_rubrique']));
394	394
395	395	// on tente d'abord l'ecriture a la racine dans le cas des rubriques uniquement
396	396	if ($objet == 'rubrique') {
		@@ -424,7 +424,7 @@ discard block
		block discarded – undo
424	424	return '';
425	425	}
426	426
427		- return propre('[->' . $virtuel . ']');
	427	+ return propre('[->'.$virtuel.']');
428	428	}
429	429
430	430
		@@ -452,7 +452,7 @@ discard block
		block discarded – undo
452	452	$args = param_low_sec($op, $args, $lang, 'rss');
453	453	$url = generer_url_public('rss', $args);
454	454
455		- return "<a style='float: " . $GLOBALS['spip_lang_right'] . ";' href='$url'>$clic</a>";
	455	+ return "<a style='float: ".$GLOBALS['spip_lang_right'].";' href='$url'>$clic</a>";
456	456	}
457	457
458	458
		@@ -528,7 +528,7 @@ discard block
		block discarded – undo
528	528	);
529	529
530	530	if ($alertes = array_filter($alertes)) {
531		- return "<div class='wrap-messages-alertes'><div class='messages-alertes'>" .
	531	+ return "<div class='wrap-messages-alertes'><div class='messages-alertes'>".
532	532	join(' \| ', $alertes)
533	533	. '</div></div>';
534	534	}
		@@ -562,13 +562,13 @@ discard block
		block discarded – undo
562	562	*/
563	563	function afficher_plus_info($lien, $titre = '+', $titre_lien = '') {
564	564	$titre = attribut_html($titre);
565		- $icone = "\n<a href='$lien' title='$titre' class='plus_info'>" .
566		- http_img_pack('information-16.png', $titre) . '</a>';
	565	+ $icone = "\n<a href='$lien' title='$titre' class='plus_info'>".
	566	+ http_img_pack('information-16.png', $titre).'</a>';
567	567
568	568	if (!$titre_lien) {
569	569	return $icone;
570	570	} else {
571		- return $icone . "\n<a href='$lien'>$titre_lien</a>";
	571	+ return $icone."\n<a href='$lien'>$titre_lien</a>";
572	572	}
573	573	}
574	574

		@@ -45,7 +45,7 @@ discard block
		block discarded – undo
45	45	$n = 0;
46	46	$time = isset($GLOBALS['meta']['cache_mark']) ? $GLOBALS['meta']['cache_mark'] : 0;
47	47	for ($i = 0; $i < 256; $i++) {
48		- $dir = _DIR_CACHE . sprintf('%02s', dechex($i));
	48	+ $dir = _DIR_CACHE.sprintf('%02s', dechex($i));
49	49	if (@is_dir($dir) and is_readable($dir) and $d = opendir($dir)) {
50	50	while (($f = readdir($d)) !== false) {
51	51	if (preg_match(',^[[0-9a-f]+\.cache$,S', $f) and $a = stat("$dir/$f")) {
		@@ -102,7 +102,7 @@ discard block
		block discarded – undo
102	102
103	103	// stocker la date_modif_$objet (ne sert a rien pour le moment)
104	104	if (isset($objet)) {
105		- ecrire_meta('derniere_modif_' . $objet, time());
	105	+ ecrire_meta('derniere_modif_'.$objet, time());
106	106	}
107	107
108	108	// si $derniere_modif_invalide est un array('article', 'rubrique')
		@@ -201,7 +201,7 @@ discard block
		block discarded – undo
201	201	)
202	202	) {
203	203	// supprimer le fichier (de facon propre)
204		- supprimer_fichier(_DIR_CACHE . $cache);
	204	+ supprimer_fichier(_DIR_CACHE.$cache);
205	205	} else {
206	206	spip_log("Nom de fichier cache incorrect : $cache");
207	207	}

		@@ -54,7 +54,7 @@ discard block
		block discarded – undo
54	54
55	55	if ($aff_racine) {
56	56	$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
57		- $idom3 = $idom . '_selection';
	57	+ $idom3 = $idom.'_selection';
58	58
59	59	$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
60	60
		@@ -72,7 +72,7 @@ discard block
		block discarded – undo
72	72	' '
73	73	);
74	74
75		- $js_func = $do . '_selection_titre';
	75	+ $js_func = $do.'_selection_titre';
76	76	$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
77	77
78	78	$aff_racine = "<div class='petit-item petite-racine item'>"
		@@ -113,12 +113,12 @@ discard block
		block discarded – undo
113	113	**/
114	114	function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '') {
115	115
116		- $idom1 = $idom . '_champ_recherche';
117		- $idom2 = $idom . '_principal';
118		- $idom3 = $idom . '_selection';
119		- $idom4 = $idom . '_col_1';
120		- $idom5 = 'img_' . $idom4;
121		- $idom6 = $idom . '_fonc';
	116	+ $idom1 = $idom.'_champ_recherche';
	117	+ $idom2 = $idom.'_principal';
	118	+ $idom3 = $idom.'_selection';
	119	+ $idom4 = $idom.'_col_1';
	120	+ $idom5 = 'img_'.$idom4;
	121	+ $idom6 = $idom.'_fonc';
122	122
123	123	return "<div id='$idom'>"
124	124	. "<a id='$idom6' style='visibility: hidden;'"
		@@ -128,10 +128,10 @@ discard block
		block discarded – undo
128	128	. http_img_pack(
129	129	'loader.svg',
130	130	'',
131		- "class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
	131	+ "class='loader' style='visibility: hidden;float:".$GLOBALS['spip_lang_right']."' id='$idom5'"
132	132	)
133	133	. ''
134		- . "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
	134	+ . "<input style='width: 10em;float:".$GLOBALS['spip_lang_right'].";' type='text' class='text search' id='$idom1' placeholder='"._T('info_rechercher')."'"
135	135	// eliminer Return car il provoque la soumission (balise unique)
136	136	// et eliminer Tab pour la navigation au clavier
137	137	// ce serait encore mieux de ne le faire que s'il y a encore plusieurs
		@@ -178,8 +178,8 @@ discard block
		block discarded – undo
178	178
179	179	$liste = $id_rubrique;
180	180	$id_rubrique = intval($id_rubrique);
181		- while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
182		- $liste = $id_rubrique . ",$liste";
	181	+ while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = '.$id_rubrique)) {
	182	+ $liste = $id_rubrique.",$liste";
183	183	}
184	184
185	185	return explode(',', "0,$liste");

		@@ -39,7 +39,7 @@ discard block
		block discarded – undo
39	39
40	40	if ($notification = charger_fonction($quoi, 'notifications', true)) {
41	41	spip_log("$notification($quoi,$id"
42		- . ($options ? ',' . serialize($options) : '')
	42	+ . ($options ? ','.serialize($options) : '')
43	43	. ')', 'notifications');
44	44	$notification($quoi, $id, $options);
45	45	}
		@@ -157,7 +157,7 @@ discard block
		block discarded – undo
157	157
158	158	job_queue_add(
159	159	'envoyer_mail',
160		- ">$email : " . $envoi['sujet'],
	160	+ ">$email : ".$envoi['sujet'],
161	161	[$email, $envoi['sujet'], $envoi['texte'], $envoi['from'], $envoi['headers']],
162	162	'inc/'
163	163	);

		@@ -50,7 +50,7 @@ discard block
		block discarded – undo
50	50
51	51	// si l'objet existe deja, on retourne simplement ses valeurs
52	52	if (is_numeric($id_objet)) {
53		- return sql_fetsel('*', $table, "$_id_objet=" . intval($id_objet));
	53	+ return sql_fetsel('*', $table, "$_id_objet=".intval($id_objet));
54	54	}
55	55
56	56	// ici, on demande une creation.
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	// si demande de traduction
65	65	// on recupere les valeurs de la traduction
66	66	if ($lier_trad) {
67		- if ($select = charger_fonction('precharger_traduction_' . $type, 'inc', true)) {
	67	+ if ($select = charger_fonction('precharger_traduction_'.$type, 'inc', true)) {
68	68	$row = $select($id_objet, $id_rubrique, $lier_trad);
69	69	} else {
70	70	$row = precharger_traduction_objet($type, $id_objet, $id_rubrique, $lier_trad, $champ_titre);
		@@ -102,7 +102,7 @@ discard block
		block discarded – undo
102	102	// recuperer le secteur, pour affecter les bons champs extras
103	103	if ($id_rubrique and $is_secteur) {
104	104	if (!$row['id_secteur']) {
105		- $row_rub = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . sql_quote($id_rubrique));
	105	+ $row_rub = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.sql_quote($id_rubrique));
106	106	$row['id_secteur'] = $row_rub;
107	107	}
108	108	}
		@@ -133,10 +133,10 @@ discard block
		block discarded – undo
133	133	$_id_objet = id_table_objet($table);
134	134
135	135	// Recuperer les donnees de l'objet original
136		- $row = sql_fetsel('*', $table, "$_id_objet=" . intval($lier_trad));
	136	+ $row = sql_fetsel('*', $table, "$_id_objet=".intval($lier_trad));
137	137	if ($row) {
138	138	include_spip('inc/filtres');
139		- $row[$champ_titre] = filtrer_entites(objet_T($type, 'info_nouvelle_traduction')) . ' ' . $row[$champ_titre];
	139	+ $row[$champ_titre] = filtrer_entites(objet_T($type, 'info_nouvelle_traduction')).' '.$row[$champ_titre];
140	140	} else {
141	141	$row = [];
142	142	}
		@@ -186,14 +186,14 @@ discard block
		block discarded – undo
186	186	$id_parent = 0;
187	187	} else {
188	188	// on cherche une rubrique soeur dans la bonne langue
189		- $row_rub = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . intval($id_rubrique));
	189	+ $row_rub = sql_fetsel('id_parent', 'spip_rubriques', 'id_rubrique='.intval($id_rubrique));
190	190	$id_parent = $row_rub['id_parent'];
191	191	}
192	192
193	193	$row_rub = sql_fetsel(
194	194	'id_rubrique',
195	195	'spip_rubriques',
196		- "lang='" . $GLOBALS['spip_lang'] . "' AND id_parent=" . intval($id_parent)
	196	+ "lang='".$GLOBALS['spip_lang']."' AND id_parent=".intval($id_parent)
197	197	);
198	198	if ($row_rub) {
199	199	$row['id_rubrique'] = $row_rub['id_rubrique'];

		@@ -40,9 +40,9 @@ discard block
		block discarded – undo
40	40	$mode = preg_replace(',\W,', '', $mode);
41	41	if ($mode) {
42	42	// chercher dans la base
43		- $mode_document = 'logo' . $mode;
	43	+ $mode_document = 'logo'.$mode;
44	44	$objet = objet_type($_id_objet);
45		- $doc = sql_fetsel('D.*', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode=' . sql_quote($mode_document) . ' AND L.objet=' . sql_quote($objet) . ' AND id_objet=' . intval($id));
	45	+ $doc = sql_fetsel('D.*', 'spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document', 'D.mode='.sql_quote($mode_document).' AND L.objet='.sql_quote($objet).' AND id_objet='.intval($id));
46	46	if ($doc) {
47	47	include_spip('inc/documents');
48	48	$d = get_spip_doc($doc['fichier']);
		@@ -53,10 +53,10 @@ discard block
		block discarded – undo
53	53	if ($compat_old_logos) {
54	54	# attention au cas $id = '0' pour LOGO_SITE_SPIP : utiliser intval()
55	55	$type = type_du_logo($_id_objet);
56		- $nom = $type . $mode . intval($id);
	56	+ $nom = $type.$mode.intval($id);
57	57
58	58	foreach ($GLOBALS['formats_logos'] as $format) {
59		- if (@file_exists($d = (_DIR_LOGOS . $nom . '.' . $format))) {
	59	+ if (@file_exists($d = (_DIR_LOGOS.$nom.'.'.$format))) {
60	60	return [$d, _DIR_LOGOS, $nom, $format, @filemtime($d)];
61	61	}
62	62	}

spip / SPIP

Push — master ( 1da3a1...d54839 )

Status

Category

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

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

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

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

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

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

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

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

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