spip / SPIP

prive/formulaires/configurer_ecran_connexion.php

Braces +4 added lines, -12 removed lines

@@ -32,13 +32,9 @@ 
 
 	if (_request('supprimer_image_fond_login')) {
 		// rien à tester
-	}
-
-	elseif (_request('supprimer_couleur_login')) {
+	} elseif (_request('supprimer_couleur_login')) {
 		// rien à tester
-	}
-
-	elseif (!empty($_FILES['upload_image_fond_login'])) {
+	} elseif (!empty($_FILES['upload_image_fond_login'])) {
 		$file = $_FILES['upload_image_fond_login'];
 		include_spip('inc/documents');
 		$extension = pathinfo((string) $file['name'], PATHINFO_EXTENSION);
@@ -74,14 +70,10 @@ 
 
 	if (_request('supprimer_image_fond_login')) {
 		@unlink($dest);
-	}
-
-	elseif (_request('supprimer_couleur_login')) {
+	} elseif (_request('supprimer_couleur_login')) {
 		effacer_config('couleur_login');
 		set_request('couleur_login', null);
-	}
-
-	elseif (!empty($_FILES['upload_image_fond_login'])) {
+	} elseif (!empty($_FILES['upload_image_fond_login'])) {
 		$file = $_FILES['upload_image_fond_login'];
 		include_spip('inc/documents');
 		deplacer_fichier_upload($file['tmp_name'], $dest);

ecrire/public/phraser_html.php

Braces +4 added lines, -8 removed lines

@@ -798,8 +798,7 @@ 
 function public_compte_ligne($texte, $debut = 0, $fin = null) {
 	if (is_null($fin)) {
 		return substr_count((string) $texte, "\n", $debut);
-	}
-	else {
+	} else {
 		return substr_count((string) $texte, "\n", $debut, $fin - $debut);
 	}
 }
@@ -846,8 +845,7 @@ 
 			erreur_squelette($err_b, $result);
 
 			continue;
-		}
-		else {
+		} else {
 			$boucle = [
 				'id_boucle' => $id_boucle,
 				'id_boucle_err' => $id_boucle,
@@ -987,8 +985,7 @@ 
 	// si c'est un appel pour memoriser une boucle, memorisons la
 	if (is_string($champ) && !empty($boucle_placeholder) && !empty($boucle)) {
 		$boucles_connues[$boucle_placeholder][$champ] = &$boucle;
-	}
-	else {
+	} else {
 		if (!empty($champ->nom_champ) && !empty($boucles_connues[$champ->nom_champ])) {
 			$placeholder = $champ->nom_champ;
 			$id = reset($champ->param[0][1]);
@@ -1131,8 +1128,7 @@ 
 				];
 				erreur_squelette($err_b, $result);
 				$pos_courante += strlen($fin_boucle);
-			}
-			else {
+			} else {
 				// verifier une eventuelle imbrication d'une boucle homonyme
 				// (interdite, generera une erreur plus loin, mais permet de signaler la bonne erreur)
 				$search_debut_boucle = BALISE_BOUCLE . $id_boucle_search . '(';

ecrire/public/criteres.php

Braces +2 added lines, -4 removed lines

@@ -2069,8 +2069,7 @@ 
 						// Champ joker * des iterateurs DATA qui accepte tout
 						if (@array_key_exists('*', $desc['field'])) {
 							$desc['field'][$col_vraie ?: $col] = ''; // on veut pas de cast INT par defaut car le type peut etre n'importe quoi dans les boucles DATA
-						}
-						else {
+						} else {
 							$r = calculer_critere_infixe_externe($boucle, $crit, $op, $desc, $col, $col_alias, $table);
 							if (!$r) {
 								return '';
@@ -2112,8 +2111,7 @@ 
 				. ((isset($r[2]) && $r[2]) ? $r[2] : ",''")
 				. ",'" . addslashes((string) $type_cast_quote) . "'";
 			$val[0] = "sql_quote($r)";
-		}
-		elseif (
+		} elseif (
 			str_contains((string) $val[0], '@@defaultcast@@')
 			&& preg_match("/'@@defaultcast@@'\s*\)\s*\z/ms", (string) $val[0], $r)
 		) {

ecrire/auth/spip.php

Braces +5 added lines, -10 removed lines

@@ -110,14 +110,12 @@ 
 					spip_log('Les cles secretes ont ete restaurées avec le backup du webmestre #' . $row['id_auteur'], 'auth' . _LOG_INFO_IMPORTANTE);
 					if ($cles->save()) {
 						$secret = $cles->getSecretAuth();
-					}
-					else {
+					} else {
 						spip_log("Echec restauration des cles : verifier les droits d'ecriture ?", 'auth' . _LOG_ERREUR);
 						// et on echoue car on ne veut pas que la situation reste telle quelle
 						raler_fichier(_DIR_ETC . 'cles.php');
 					}
-				}
-				else {
+				} else {
 					spip_log('Pas de cle secrete disponible (fichier config/cle.php absent ?) mais le backup du webmestre #' . $row['id_auteur'] . " n'est pas valide", 'auth' . _LOG_ERREUR);
 					sql_updateq('spip_auteurs', ['backup_cles' => ''], 'id_auteur=' . (int) $row['id_auteur']);
 				}
@@ -125,8 +123,7 @@ 
 
 			if (!$secret || !Password::verifier($pass, $row['pass'], $secret)) {
 				unset($row);
-			}
-			else {
+			} else {
 				spip_log("validation du mot de passe pour l'auteur #" . $row['id_auteur'] . " $login via Password::verifier", 'auth' . _LOG_DEBUG);
 			}
 			break;
@@ -234,8 +231,7 @@ 
 		spip_log("Echec generation d'une nouvelle cle : verifier les droits d'ecriture ?", 'auth' . _LOG_ERREUR);
 		// et on echoue car on ne veut pas que la situation reste telle quelle
 		raler_fichier(_DIR_ETC . 'cles.php');
-	}
-	else {
+	} else {
 		spip_log('Pas de cle secrete disponible (fichier config/cle.php absent ?) un des webmestres #' . implode(', #', $has_backup) . ' doit se connecter pour restaurer son backup des cles', 'auth' . _LOG_ERREUR);
 	}
 	return false;
@@ -468,8 +464,7 @@ 
 	if (!$secret) {
 		if (auth_spip_initialiser_secret()) {
 			$secret = $cles->getSecretAuth();
-		}
-		else {
+		} else {
 			return false;
 		}
 	}

ecrire/inc/exporter_csv.php

Braces +1 added lines, -2 removed lines

@@ -182,8 +182,7 @@
 	// sinon on ecrit directement sur stdout
 	if ($options['envoyer'] && $options['envoyer'] !== 'attachment') {
 		$fichier = 'php://output';
-	}
-	else {
+	} else {
 		$fichier = sous_repertoire(_DIR_CACHE, 'export') . $basename;
 	}
 

ecrire/inc/editer.php

Braces +1 added lines, -2 removed lines

@@ -336,8 +336,7 @@
 		if (!autoriser('modifier', $type, (int) $id)) {
 			$contexte['editable'] = '';
 		}
-	}
-	else {
+	} else {
 		if (!autoriser('creer', $type, 0, null, ['id_parent' => $id_parent])) {
 			$contexte['editable'] = '';
 		}

ecrire/inc/distant.php

Braces +2 added lines, -4 removed lines

@@ -122,8 +122,7 @@ 
 		if (!$res || !$res['length'] && $res['status'] != 304) {
 			spip_log("copie_locale : Echec recuperation $source sur $localrac_tmp status : " . ($res ? $res['status'] : '-'), 'distant' . _LOG_INFO_IMPORTANTE);
 			@unlink($localrac_tmp);
-		}
-		else {
+		} else {
 			spip_log("copie_locale : recuperation $source sur $localrac_tmp OK | taille " . $res['length'] . ' status ' . $res['status'], 'distant');
 		}
 		if (!$res || !$res['length']) {
@@ -242,8 +241,7 @@ 
 						break;
 					}
 				}
-			}
-			else {
+			} else {
 				$ip = false;
 			}
 		}

ecrire/inc/autoriser.php

Braces +3 added lines, -6 removed lines

@@ -267,8 +267,7 @@ 
 		if ($id === '*') {
 			unset($GLOBALS['autoriser_exception'][$faire][$type]);
 			unset($autorisation[$faire][$type]);
-		}
-		else {
+		} else {
 			unset($GLOBALS['autoriser_exception'][$faire][$type][$id]);
 			unset($autorisation[$faire][$type][$id]);
 		}
@@ -635,8 +634,7 @@ 
 function autoriser_rubrique_creer_dist(string $faire, string $type, $id, array $qui, array $opt): bool {
 	if (!empty($opt['id_parent'])) {
 		return autoriser('creerrubriquedans', 'rubrique', $opt['id_parent'], $qui);
-	}
-	else {
+	} else {
 		return autoriser('defaut', null, 0, $qui, $opt);
 	}
 }
@@ -791,8 +789,7 @@ 
 	if (!empty($opt['id_parent'])) {
 		// creerarticledans rappelle autoriser(creer,article) sans id, donc on verifiera condition du else aussi
 		return autoriser('creerarticledans', 'rubrique', $opt['id_parent'], $qui);
-	}
-	else {
+	} else {
 		return (sql_countsel('spip_rubriques') > 0 && in_array($qui['statut'], ['0minirezo', '1comite']));
 	}
 }

ecrire/inc/filtres.php

Braces +7 added lines, -14 removed lines

@@ -615,8 +615,7 @@ 
 			if (!$srcSize) {
 				$poids_img[$src] = filesize($src);
 			}
-		}
-		elseif (str_contains($src, '<svg')) {
+		} elseif (str_contains($src, '<svg')) {
 			include_spip('inc/svg');
 			if ($attrs = svg_lire_attributs($src)) {
 				[$width, $height, $viewbox] = svg_getimagesize_from_attr($attrs);
@@ -3421,8 +3420,7 @@ 
 	}
 	if (!isset($options['chemin_image']) || $options['chemin_image'] == true) {
 		$img_file = chemin_image($img);
-	}
-	else {
+	} else {
 		if (!isset($options['variante_svg_si_possible']) || $options['variante_svg_si_possible'] == true) {
 			$img_file = http_img_variante_svg_si_possible($img_file);
 		}
@@ -3449,11 +3447,9 @@ 
 	}
 	if ($alt === false) {
 		$alt = '';
-	}
-	elseif ($alt || $alt === '') {
+	} elseif ($alt || $alt === '') {
 		$alt = " alt='" . attribut_html($alt) . "'";
-	}
-	else {
+	} else {
 		$alt = " alt='" . attribut_html($title) . "'";
 	}
 	return "<img src='" . attribut_html($img_file) . "'$alt"
@@ -3580,8 +3576,7 @@ 
 		if (!is_null($class)) {
 			$img = strlen((string) $class) ? inserer_attribut($img, 'class', $class) : vider_attribut($img, 'class');
 		}
-	}
-	else {
+	} else {
 		$img = http_img_pack(
 			$img,
 			$alt,
@@ -3689,8 +3684,7 @@ 
 		$balise_svg = inserer_attribut($balise_svg, 'aria-labelledby', $id);
 		$title = "<title id=\"$id\">" . entites_html($alt) . "</title>\n";
 		$balise_svg .= $title;
-	}
-	else {
+	} else {
 		$balise_svg = inserer_attribut($balise_svg, 'aria-hidden', 'true');
 	}
 
@@ -5081,8 +5075,7 @@ 
 	// reduit la variabilite du nom et donc le nombre de css concatenees possibles in fine
 	if (isset($options['hash_on_content']) && $options['hash_on_content']) {
 		$hash = md5((string) ($contexte_implicite['host'] . '::' . $cache));
-	}
-	else {
+	} else {
 		unset($contexte_implicite['notes']); // pas pertinent pour signaler un changeemnt de contenu pour des css/js
 		ksort($contexte);
 		$hash = md5($fond . json_encode($contexte_implicite, JSON_THROW_ON_ERROR) . json_encode($contexte, JSON_THROW_ON_ERROR) . $connect);