spip / SPIP

ecrire/public/sandbox.php

Spacing +8 added lines, -8 removed lines

@@ -41,7 +41,7 @@ 
  *     Texte
  */
 function sandbox_composer_texte($texte, &$p) {
-	$code = "'" . str_replace(array("\\", "'"), array("\\\\", "\\'"), $texte) . "'";
+	$code = "'".str_replace(array("\\", "'"), array("\\\\", "\\'"), $texte)."'";
 
 	return $code;
 }
@@ -58,7 +58,7 @@ 
  *     Balise qui appelle ce filtre
  * @return string
  */
-function sandbox_composer_filtre($fonc, $code, $arglist, &$p, $nb_arg_droite=1000) : string {
+function sandbox_composer_filtre($fonc, $code, $arglist, &$p, $nb_arg_droite = 1000) : string {
 	if (isset($GLOBALS['spip_matrice'][$fonc])) {
 		$code = "filtrer('$fonc',$code$arglist)";
 	}
@@ -82,11 +82,11 @@ 
 			$code = "$f($code$arglist)";
 			$nb_arg_gauche = 1; // la balise à laquelle s'applique le filtre
 		}
-		$nb_args_f = $nb_arg_gauche+$nb_arg_droite;
+		$nb_args_f = $nb_arg_gauche + $nb_arg_droite;
 		$min_f = $refl->getNumberOfRequiredParameters();
 		if (($nb_args_f < $min_f)) {
-			$msg_args = ['filtre' => texte_script ($fonc), 'nb'=> $min_f - $nb_args_f];
-			erreur_squelette ([ 'zbug_erreur_filtre_nbarg_min', $msg_args], $p);
+			$msg_args = ['filtre' => texte_script($fonc), 'nb'=> $min_f - $nb_args_f];
+			erreur_squelette(['zbug_erreur_filtre_nbarg_min', $msg_args], $p);
 		}
 	}
 	// le filtre n'existe pas,
@@ -171,8 +171,8 @@ 
  */
 function sandbox_filtrer_squelette($skel, $corps, $filtres) {
 	$series_filtres = func_get_args();
-	array_shift($series_filtres);// skel
-	array_shift($series_filtres);// corps
+	array_shift($series_filtres); // skel
+	array_shift($series_filtres); // corps
 
 	// proteger les <INCLUDE> et tous les morceaux de php licites
 	if ($skel['process_ins'] == 'php') {
@@ -220,7 +220,7 @@ 
 	if (is_array($r)) {
 		$dst[] = $r[0];
 
-		return $src[] = '___' . md5($r[0]) . '___';
+		return $src[] = '___'.md5($r[0]).'___';
 	}
 
 	// si on recoit pas un tableau, on renvoit les couples de substitution

ecrire/inc/filtres_alertes.php

Spacing +10 added lines, -10 removed lines

@@ -48,11 +48,11 @@ 
 	$_class = interprete_argument_balise(3, $p);
 	$_role  = interprete_argument_balise(4, $p);
 	$_id    = interprete_argument_balise(5, $p);
-	$_texte = ($_texte ? $_texte     : "''");
+	$_texte = ($_texte ? $_texte : "''");
 	$_titre = ($_titre ? ", $_titre" : ', null');
 	$_class = ($_class ? ", $_class" : ', null');
-	$_role  = ($_role  ? ", $_role"  : ', null');
-	$_id    = ($_id    ? ", $_id"    : ', null');
+	$_role  = ($_role ? ", $_role" : ', null');
+	$_id    = ($_id ? ", $_id" : ', null');
 
 	$f = chercher_filtre('message_alerte');
 	$p->code = "$f($_texte$_titre$_class$_role$_id)";
@@ -90,10 +90,10 @@ 
 	$_class = interprete_argument_balise(2, $p);
 	$_role  = interprete_argument_balise(3, $p);
 	$_id    = interprete_argument_balise(4, $p);
-	$_titre = ($_titre ? "$_titre"   : 'null');
+	$_titre = ($_titre ? "$_titre" : 'null');
 	$_class = ($_class ? ", $_class" : ', null');
-	$_role  = ($_role  ? ", $_role"  : ', null');
-	$_id    = ($_id    ? ", $_id"    : ', null');
+	$_role  = ($_role ? ", $_role" : ', null');
+	$_id    = ($_id ? ", $_id" : ', null');
 
 	$f = chercher_filtre('message_alerte_ouvrir');
 	$p->code = "$f($_titre$_class$_role$_id)";
@@ -163,8 +163,8 @@ 
 	$message_alerte_ouvrir = chercher_filtre('message_alerte_ouvrir');
 	$message_alerte_fermer = chercher_filtre('message_alerte_fermer');
 	$message =
-		$message_alerte_ouvrir($titre, $class, $role, $id) .
-		$texte .
+		$message_alerte_ouvrir($titre, $class, $role, $id).
+		$texte.
 		$message_alerte_fermer();
 
 	return $message;
@@ -213,7 +213,7 @@ 
 	];
 	$type  = array_intersect(explode(' ', $class), $types);
 	$type  = reset($type);
-	$class = trim(str_replace($types, '', $class) . " $type");
+	$class = trim(str_replace($types, '', $class)." $type");
 
 	// Classes
 	$class_racine = 'msg-alert';
@@ -238,7 +238,7 @@ 
 
 	// Attributs
 	$attr_role = ($role ? "role=\"$role\"" : '');
-	$attr_id   = ($id   ? "id=\"$id\"" : '');
+	$attr_id   = ($id ? "id=\"$id\"" : '');
 	$attr_data = ($type ? "data-alert=\"$type\"" : '');
 
 	$message =

ecrire/inc/plonger.php

Spacing +11 added lines, -11 removed lines

@@ -34,9 +34,9 @@ 
 
 	$res = sql_select("rub1.id_rubrique, rub1.titre, rub1.id_parent, rub1.lang, rub1.langue_choisie, rub2.id_rubrique AS id_enfant",
 		"spip_rubriques AS rub1 LEFT JOIN spip_rubriques AS rub2 ON (rub1.id_rubrique = rub2.id_parent)",
-		"rub1.id_parent = " . sql_quote($id_rubrique) . "
-			AND rub1.id_rubrique!=" . sql_quote($exclu) . "
-			AND (rub2.id_rubrique IS NULL OR rub2.id_rubrique!=" . sql_quote($exclu) . ")", "", "0+rub1.titre,rub1.titre");
+		"rub1.id_parent = ".sql_quote($id_rubrique)."
+			AND rub1.id_rubrique!=" . sql_quote($exclu)."
+			AND (rub2.id_rubrique IS NULL OR rub2.id_rubrique!=" . sql_quote($exclu).")", "", "0+rub1.titre,rub1.titre");
 
 	while ($row = sql_fetch($res)) {
 		if (autoriser('voir', 'rubrique', $row['id_rubrique'])) {
@@ -46,7 +46,7 @@ 
 				if ($row['langue_choisie'] != 'oui') {
 					$t .= ' <small title="'
 						. traduire_nom_langue($row['lang'])
-						. '">[' . $row['lang'] . ']</small>';
+						. '">['.$row['lang'].']</small>';
 				}
 				$ordre[$row['id_rubrique']] = $t;
 			}
@@ -54,14 +54,14 @@ 
 	}
 	$next = isset($list[$col]) ? $list[$col] : 0;
 	if ($ordre) {
-		$rec = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclu&do=$do&col=" . ($col + 1));
+		$rec = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclu&do=$do&col=".($col + 1));
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$args = "'$idom',this,$col,'" . $GLOBALS['spip_lang_left'] . "','$info',event";
+		$args = "'$idom',this,$col,'".$GLOBALS['spip_lang_left']."','$info',event";
 
 		foreach ($ordre as $id => $titrebrut) {
 			$titre = supprimer_numero($titrebrut);
 
-			$classe1 = 'petit-item ' . ($id_rubrique ? 'petite-rubrique' : "petit-secteur");
+			$classe1 = 'petit-item '.($id_rubrique ? 'petite-rubrique' : "petit-secteur");
 			if (isset($rub[$id]["enfants"])) {
 				$classe2 = " class='rub-ouverte'";
 				$url = "\nhref='$rec&amp;id=$id'";
@@ -70,7 +70,7 @@ 
 				$url = "\nhref='javascript:void(0)'";
 			}
 
-			$js_func = $do . '_selection_titre';
+			$js_func = $do.'_selection_titre';
 			$click = "\nonclick=\"changerhighlight(this.parentNode.parentNode.parentNode);\nreturn "
 				. (!is_array($list) ? ' false'
 					: "aff_selection_provisoire($id,$args)")
@@ -97,14 +97,14 @@ 
 		}
 	}
 
-	$idom2 = $idom . "_col_" . ($col + 1);
+	$idom2 = $idom."_col_".($col + 1);
 	$left = ($col * 250);
 
 	return http_img_pack("loader.svg", "",
-		"class='loader' style='visibility: hidden; position: absolute; " . $GLOBALS['spip_lang_left'] . ": "
+		"class='loader' style='visibility: hidden; position: absolute; ".$GLOBALS['spip_lang_left'].": "
 		. ($left - 30)
 		. "px; top: 2px; z-index: 2;' id='img_$idom2'")
-	. "<div style='width: 250px; height: 100%; overflow: auto; position: absolute; top: 0px; " . $GLOBALS['spip_lang_left'] . ": "
+	. "<div style='width: 250px; height: 100%; overflow: auto; position: absolute; top: 0px; ".$GLOBALS['spip_lang_left'].": "
 	. ($left - 250)
 	. "px;'>"
 	. $ret

ecrire/inc/selectionner.php

Spacing +12 added lines, -12 removed lines

@@ -54,7 +54,7 @@ 
 
 	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;";
 
@@ -63,7 +63,7 @@ 
 				textebrut(_T('info_racine_site')))),
 			"\n\r", "  ");
 
-		$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'>"
@@ -104,12 +104,12 @@ 
  **/
 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;'"
@@ -117,9 +117,9 @@ 
 	. "></a>"
 	. "<div class='recherche_rapide_parent formulaire_recherche'>"
 	. 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
@@ -166,8 +166,8 @@ 
 
 	$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");

ecrire/inc/filtres_dates.php

Spacing +35 added lines, -35 removed lines

@@ -38,7 +38,7 @@ 
 function extraire_date($texte) {
 	// format = 2001-08
 	if (preg_match(",([1-2][0-9]{3})[^0-9]*(1[0-2]|0?[1-9]),", $texte, $regs)) {
-		return $regs[1] . "-" . sprintf("%02d", $regs[2]) . "-01";
+		return $regs[1]."-".sprintf("%02d", $regs[2])."-01";
 	}
 }
 
@@ -68,11 +68,11 @@ 
 		}
 		if (preg_match("#^([12][0-9]{3})([-/]00)?( [-0-9:]+)?$#", $date, $regs)) {
 			$regs = array_pad($regs, 4, null); // eviter notice php
-			$date = $regs[1] . "-00-00" . $regs[3];
+			$date = $regs[1]."-00-00".$regs[3];
 		} else {
 			if (preg_match("#^([12][0-9]{3}[-/][01]?[0-9])([-/]00)?( [-0-9:]+)?$#", $date, $regs)) {
 				$regs = array_pad($regs, 4, null); // eviter notice php
-				$date = preg_replace("@/@", "-", $regs[1]) . "-00" . $regs[3];
+				$date = preg_replace("@/@", "-", $regs[1])."-00".$regs[3];
 			} else {
 				$date = date("Y-m-d H:i:s", strtotime($date));
 			}
@@ -97,7 +97,7 @@ 
  **/
 function vider_date($letexte, $verif_format_date = false) {
 	if (!$verif_format_date
-	  or (in_array(strlen($letexte), [10,19]) and
+	  or (in_array(strlen($letexte), [10, 19]) and
 			  preg_match("/^[0-9]{4}-[0-9]{2}-[0-9]{2}(\s[0-9]{2}:[0-9]{2}:[0-9]{2})?$/", $letexte))) {
 
 		if (strncmp("0000-00-00", $letexte, 10) == 0) {
@@ -208,8 +208,8 @@ 
  *	- si 'abbr' ne précise pas l'unité des minutes : 12h10
  * @return string L'heure formatée dans la langue en cours.
  **/
-function heures_minutes($numdate, $forme='') {
-	if ($forme !='abbr') {
+function heures_minutes($numdate, $forme = '') {
+	if ($forme != 'abbr') {
 		return _T('date_fmt_heures_minutes', array('h' => heures($numdate), 'm' => minutes($numdate)));
 	}
 	else {
@@ -381,17 +381,17 @@ 
 	if ($decal > 3600 * 24 * 30) {
 		$mois = floor($decal / (3600 * 24 * 30));
 		if ($mois < 2) {
-			$delai = "$mois " . _T("date_un_mois");
+			$delai = "$mois "._T("date_un_mois");
 		} else {
-			$delai = "$mois " . _T("date_mois");
+			$delai = "$mois "._T("date_mois");
 		}
 	} else {
 		if ($decal > 3600 * 24 * 7) {
 			$semaines = floor($decal / (3600 * 24 * 7));
 			if ($semaines < 2) {
-				$delai = "$semaines " . _T("date_une_semaine");
+				$delai = "$semaines "._T("date_une_semaine");
 			} else {
-				$delai = "$semaines " . _T("date_semaines");
+				$delai = "$semaines "._T("date_semaines");
 			}
 		} else {
 			if ($decal > 3600 * 24) {
@@ -399,30 +399,30 @@ 
 				if ($jours < 2) {
 					return $il_y_a == "date_dans" ? _T("date_demain") : _T("date_hier");
 				} else {
-					$delai = "$jours " . _T("date_jours");
+					$delai = "$jours "._T("date_jours");
 				}
 			} else {
 				if ($decal >= 3600) {
 					$heures = floor($decal / 3600);
 					if ($heures < 2) {
-						$delai = "$heures " . _T("date_une_heure");
+						$delai = "$heures "._T("date_une_heure");
 					} else {
-						$delai = "$heures " . _T("date_heures");
+						$delai = "$heures "._T("date_heures");
 					}
 				} else {
 					if ($decal >= 60) {
 						$minutes = floor($decal / 60);
 						if ($minutes < 2) {
-							$delai = "$minutes " . _T("date_une_minute");
+							$delai = "$minutes "._T("date_une_minute");
 						} else {
-							$delai = "$minutes " . _T("date_minutes");
+							$delai = "$minutes "._T("date_minutes");
 						}
 					} else {
 						$secondes = ceil($decal);
 						if ($secondes < 2) {
-							$delai = "$secondes " . _T("date_une_seconde");
+							$delai = "$secondes "._T("date_une_seconde");
 						} else {
-							$delai = "$secondes " . _T("date_secondes");
+							$delai = "$secondes "._T("date_secondes");
 						}
 					}
 				}
@@ -517,7 +517,7 @@ 
 		$njour = 0;
 	} else {
 		$njour = intval($jour);
-		if ($jourth = _T('date_jnum' . $jour)) {
+		if ($jourth = _T('date_jnum'.$jour)) {
 			$jour = $jourth;
 		}
 	}
@@ -525,10 +525,10 @@ 
 	$mois = intval($mois);
 	if ($mois > 0 and $mois < 13) {
 		/* Traiter le cas "abbr" pour les noms de mois */
-		$param = ((isset($options['param']) and $options['param'] === "abbr") ? '_' . $options['param'] : '');
-		$nommois = _T('date_mois_' . $mois . $param);
+		$param = ((isset($options['param']) and $options['param'] === "abbr") ? '_'.$options['param'] : '');
+		$nommois = _T('date_mois_'.$mois.$param);
 		if ($jour) {
-			$jourmois = _T('date_de_mois_' . $mois, array('j' => $jour, 'nommois' => $nommois));
+			$jourmois = _T('date_de_mois_'.$mois, array('j' => $jour, 'nommois' => $nommois));
 		} else {
 			$jourmois = $nommois;
 		}
@@ -538,7 +538,7 @@ 
 	}
 
 	if ($annee < 0) {
-		$annee = -$annee . " " . _T('date_avant_jc');
+		$annee = -$annee." "._T('date_avant_jc');
 		$avjc = true;
 	} else {
 		$avjc = false;
@@ -564,10 +564,10 @@ 
 				}
 			}
 			if ($vue == 'saison') {
-				return $saison ? _T('date_saison_' . $saison) : '';
+				return $saison ? _T('date_saison_'.$saison) : '';
 			} else {
 				return $saison ? trim(_T('date_fmt_saison_annee',
-					array('saison' => _T('date_saison_' . $saison), 'annee' => $annee))) : '';
+					array('saison' => _T('date_saison_'.$saison), 'annee' => $annee))) : '';
 			}
 
 		case 'court':
@@ -633,9 +633,9 @@ 
 			}
 			$nom = mktime(1, 1, 1, $mois, $njour, $annee);
 			$nom = 1 + date('w', $nom);
-			$param = ((isset($options['param']) and $options['param']) ? '_' . $options['param'] : '');
+			$param = ((isset($options['param']) and $options['param']) ? '_'.$options['param'] : '');
 
-			return _T('date_jour_' . $nom . $param);
+			return _T('date_jour_'.$nom.$param);
 
 		case 'mois_annee':
 			if ($avjc) {
@@ -1030,8 +1030,8 @@ 
 
 	$dtstart = $dtend = $dtabbr = "";
 	if (strpos($forme, 'hcal') !== false) {
-		$dtstart = "<abbr class='dtstart' title='" . date_iso($date_debut) . "'>";
-		$dtend = "<abbr class='dtend' title='" . date_iso($date_fin) . "'>";
+		$dtstart = "<abbr class='dtstart' title='".date_iso($date_debut)."'>";
+		$dtend = "<abbr class='dtend' title='".date_iso($date_fin)."'>";
 		$dtabbr = "</abbr>";
 	}
 
@@ -1075,7 +1075,7 @@ 
 			}
 		} else {
 			if ($dtabbr && $dtstart) {
-				$s = $dtstart . spip_ucfirst($s) . $dtabbr;
+				$s = $dtstart.spip_ucfirst($s).$dtabbr;
 			} else {
 				$s = spip_ucfirst($s);
 			}
@@ -1098,8 +1098,8 @@ 
 				$date_debut = _T('date_fmt_jour_heure', array('jour' => $date_debut, 'heure' => $hd));
 				$date_fin = _T('date_fmt_jour_heure', array('jour' => $date_fin, 'heure' => $hf));
 			}
-			$date_debut = $dtstart . $date_debut . $dtabbr;
-			$date_fin = $dtend . $date_fin . $dtabbr;
+			$date_debut = $dtstart.$date_debut.$dtabbr;
+			$date_fin = $dtend.$date_fin.$dtabbr;
 
 			$s = _T('date_fmt_periode', array('date_debut' => $date_debut, 'date_fin' => $date_fin));
 		} else {
@@ -1116,8 +1116,8 @@ 
 				$date_fin = _T('date_fmt_jour_heure', array('jour' => $date_fin, 'heure' => $hf));
 			}
 
-			$date_debut = $dtstart . $date_debut . $dtabbr;
-			$date_fin = $dtend . $date_fin . $dtabbr;
+			$date_debut = $dtstart.$date_debut.$dtabbr;
+			$date_fin = $dtend.$date_fin.$dtabbr;
 			$s = _T('date_fmt_periode', array('date_debut' => $date_debut, 'date_fin' => $date_fin));
 
 		}
@@ -1222,7 +1222,7 @@ 
 		$d = date("Y-m-d");
 	}
 
-	return substr($d, 0, 4) . substr($d, 5, 2) . substr($d, 8, 2);
+	return substr($d, 0, 4).substr($d, 5, 2).substr($d, 8, 2);
 }
 
 /**
@@ -1242,7 +1242,7 @@ 
 		$d = date("Y-m-d");
 	}
 
-	return substr($d, 0, 4) . substr($d, 5, 2);
+	return substr($d, 0, 4).substr($d, 5, 2);
 }
 
 /**

prive/formulaires/configurer_ecran_connexion.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 function formulaires_configurer_ecran_connexion_data() : array {
 	return [
 		'couleur_defaut' => "#db1762",
-		'img_fond' => _DIR_IMG . "spip_fond_login.jpg",
+		'img_fond' => _DIR_IMG."spip_fond_login.jpg",
 	];
 }
 

ecrire/balise/formulaire_.php

Spacing +5 added lines, -5 removed lines

@@ -104,7 +104,7 @@ 
 	if (in_array('eval', $trace_fonctions) and in_array('inclure_modele', $trace_fonctions)) {
 		$k = array_search('inclure_modele', $trace_fonctions);
 		// les arguments de recuperer_fond() passes par inclure_modele()
-		return $trace[$k-1]['args'][1]['args'];
+		return $trace[$k - 1]['args'][1]['args'];
 	}
 	return false;
 }
@@ -250,9 +250,9 @@ 
 	if (isset($valeurs['_action'])) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$secu = $securiser_action(reset($valeurs['_action']), end($valeurs['_action']), '', -1);
-		$valeurs['_hidden'] = (isset($valeurs['_hidden']) ? $valeurs['_hidden'] : '') .
-			"<input type='hidden' name='arg' value='" . $secu['arg'] . "' />"
-			. "<input type='hidden' name='hash' value='" . $secu['hash'] . "' />";
+		$valeurs['_hidden'] = (isset($valeurs['_hidden']) ? $valeurs['_hidden'] : '').
+			"<input type='hidden' name='arg' value='".$secu['arg']."' />"
+			. "<input type='hidden' name='hash' value='".$secu['hash']."' />";
 	}
 
 	// empiler la lang en tant que premier argument implicite du CVT
@@ -292,7 +292,7 @@ 
 		// et si $k ne commence pas par un _ (c'est bien une vrai erreur sur un vrai champ)
 		if (html5_permis()) {
 			foreach ($erreurs as $k => $v) {
-				if (is_string($v) and strlen(trim($v)) and strpos($k,'_') !== 0) {
+				if (is_string($v) and strlen(trim($v)) and strpos($k, '_') !== 0) {
 					// on encapsule dans un span car ces messages sont en general simple, juste du texte, et deja dans un span dans le form
 					$valeurs['erreurs'][$k] = "<span role='alert'>".$erreurs[$k]."</span>";
 				}

ecrire/action/supprimer_rubrique.php

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 	return;
 }
 
-include_spip('inc/charsets');  # pour le nom de fichier
+include_spip('inc/charsets'); # pour le nom de fichier
 
 /**
  * Effacer une rubrique
@@ -37,24 +37,24 @@ 
 
 	if (intval($id_rubrique)) {
 
-		sql_delete("spip_rubriques", "id_rubrique=" . intval($id_rubrique));
+		sql_delete("spip_rubriques", "id_rubrique=".intval($id_rubrique));
 		// Les admin restreints qui n'administraient que cette rubrique
 		// deviennent redacteurs
 		// (il y a sans doute moyen de faire ca avec un having)
 
-		$q = sql_select("id_auteur", "spip_auteurs_liens", "objet='rubrique' AND id_objet=" . intval($id_rubrique));
+		$q = sql_select("id_auteur", "spip_auteurs_liens", "objet='rubrique' AND id_objet=".intval($id_rubrique));
 		while ($r = sql_fetch($q)) {
 			$id_auteur = $r['id_auteur'];
 			// degrader avant de supprimer la restriction d'admin
 			// section critique sur les droits
 			$n = sql_countsel("spip_auteurs_liens",
-				"objet='rubrique' AND id_objet!=" . intval($id_rubrique) . " AND id_auteur=" . intval($id_auteur));
+				"objet='rubrique' AND id_objet!=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 			if (!$n) {
 				include_spip('action/editer_auteur');
 				auteur_modifier($id_auteur, array("statut" => '1comite'));
 			}
 			sql_delete("spip_auteurs_liens",
-				"objet='rubrique' AND id_objet=" . intval($id_rubrique) . " AND id_auteur=" . intval($id_auteur));
+				"objet='rubrique' AND id_objet=".intval($id_rubrique)." AND id_auteur=".intval($id_auteur));
 		}
 		// menu_rubriques devra recalculer
 		effacer_meta("date_calcul_rubriques");

ecrire/base/repair.php

Spacing +8 added lines, -8 removed lines

@@ -38,7 +38,7 @@ 
 
 	$res = admin_repair_tables();
 	if (!$res) {
-		$res = "<div class='error'>" . _T('avis_erreur_mysql') . ' ' . sql_errno() . ': ' . sql_error() . "</div>\n";
+		$res = "<div class='error'>"._T('avis_erreur_mysql').' '.sql_errno().': '.sql_error()."</div>\n";
 	} else {
 		include_spip('inc/rubriques');
 		calculer_rubriques();
@@ -47,7 +47,7 @@ 
 	include_spip('inc/minipres');
 	$res .= pipeline('base_admin_repair', $res);
 	echo minipres(_T('texte_tentative_recuperation'),
-		$res . generer_form_ecrire('accueil', '', '', _T('public:accueil_site')));
+		$res.generer_form_ecrire('accueil', '', '', _T('public:accueil_site')));
 }
 
 /**
@@ -90,24 +90,24 @@ 
 		$count = sql_countsel($tab);
 
 		if ($count > 1) {
-			$m .= "(" . _T('texte_compte_elements', array('count' => $count)) . ")\n";
+			$m .= "("._T('texte_compte_elements', array('count' => $count)).")\n";
 		} else {
 			if ($count == 1) {
-				$m .= "(" . _T('texte_compte_element', array('count' => $count)) . ")\n";
+				$m .= "("._T('texte_compte_element', array('count' => $count)).")\n";
 			} else {
-				$m .= "(" . _T('texte_vide') . ")\n";
+				$m .= "("._T('texte_vide').")\n";
 			}
 		}
 
 		if ($result_repair
 			and $msg = join(" ",
-					(is_resource($result_repair) or is_object($result_repair)) ? sql_fetch($result_repair) : $result_repair) . ' '
+					(is_resource($result_repair) or is_object($result_repair)) ? sql_fetch($result_repair) : $result_repair).' '
 			and strpos($msg, ' OK ') === false
 		) {
 			$class = " class='notice'";
-			$m .= "<br /><tt>" . spip_htmlentities($msg) . "</tt>\n";
+			$m .= "<br /><tt>".spip_htmlentities($msg)."</tt>\n";
 		} else {
-			$m .= " " . _T('texte_table_ok');
+			$m .= " "._T('texte_table_ok');
 		}
 
 		$res .= "<div$class>$m</div>";