spip / SPIP

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);

ecrire/base/objets.php

Braces +1 added lines, -2 removed lines

@@ -1391,8 +1391,7 @@
 							'champ' 	=> $parent_methode['champ'],
 							'table'    => $table,
 						];
-					}
-					elseif (isset($parent_methode['champ_type'])) {
+					} elseif (isset($parent_methode['champ_type'])) {
 						$parent = [
 							'objet' 	 => $ligne[$parent_methode['champ_type']],
 							'id_objet' 	 => (int) $ligne[$parent_methode['champ']],

ecrire/action/editer_liens.php

Braces +1 added lines, -2 removed lines

@@ -390,8 +390,7 @@
 			if (lien_triables($table_lien)) {
 				if (isset($qualif['rang_lien'])) {
 					$rang = $qualif['rang_lien'];
-				}
-				else {
+				} else {
 					$where = lien_where($primary, $id, $objet, $id_objet);
 					// si il y a deja un lien pour ce couple (avec un autre role?) on reprend le meme rang si non nul
 					if (!$rang = (int) sql_getfetsel('rang_lien', $table_lien, $where)) {