style: `and`->`&&`, `or`->`||`, quelques `==`->`===` + indentation et style autorisations

action/dissocier_encart.php

@@ -45,11 +45,12 @@ function action_dissocier_encart_dist($arg = null) {
 
 	// si l'identifiant de l'objet est négatif, vérifier qu'il correspond à celui du visiteur,
 	// (cas d'un encart lié à un objet pas encore enregistré en base).
+	$id_objet = intval($id_objet);
 	if (
-		$id_objet = intval($id_objet)
-		and (
-			($id_objet < 0 and $id_objet == -$GLOBALS['visiteur_session']['id_auteur'])
-			or autoriser('modifier', $objet, $id_objet)
+		$id_objet
+		&& (
+			($id_objet < 0 && $id_objet == -$GLOBALS['visiteur_session']['id_auteur'])
+			|| autoriser('modifier', $objet, $id_objet)
 		)
 	) {
 		include_spip('action/editer_liens');
@@ -68,9 +69,10 @@ function action_dissocier_encart_dist($arg = null) {
 				}
 				break;
 			default:
+				$id_encart = intval($encart);
 				if (
-					$id_encart = intval($encart)
-					and autoriser('dissocier', 'encart', $id_encart, '', [
+					$id_encart
+					&& autoriser('dissocier', 'encart', $id_encart, '', [
 						'objet' => $objet,
 						'id_objet' => $id_objet
 					])

encarts_autorisations.php

@@ -128,7 +128,7 @@ function autoriser_encart_modifier_dist($faire, $type, $id, $qui, $opt) {
  * @return bool          true s'il a le droit, false sinon
  **/
 function autoriser_encart_supprimer_dist($faire, $type, $id, $qui, $opt) {
-	return $qui['statut'] == '0minirezo' and !$qui['restreint'];
+	return $qui['statut'] === '0minirezo' && !$qui['restreint'];
 }
 
 /**
@@ -153,12 +153,15 @@ function autoriser_encart_supprimer_dist($faire, $type, $id, $qui, $opt) {
  **/
 function autoriser_encart_associer_dist($faire, $type, $id, $qui, $opts) {
 	include_spip('inc/autoriser');
-	$autoriser = (
-		($qui['statut'] == '0minirezo' and !$qui['restreint'])
-		or (autoriser('modifier', $opts['objet'], $opts['id_objet'], $qui))
-	) ? true : false;
-
-	return $autoriser;
+	return (
+		(
+			$qui['statut'] === '0minirezo'
+			&& !$qui['restreint']
+		)
+		|| (
+			autoriser('modifier', $opts['objet'], $opts['id_objet'], $qui)
+		)
+	);
 }
 
 /**
@@ -184,12 +187,16 @@ function autoriser_encart_associer_dist($faire, $type, $id, $qui, $opts) {
 function autoriser_encart_dissocier_dist($faire, $type, $id, $qui, $opts) {
 	include_spip('base/abstract_sql');
 	include_spip('inc/autoriser');
-	$autoriser = (
+	return (
 		autoriser('associer', 'encart', $id, $qui, $opts)
-		and (sql_getfetsel('vu', 'spip_encarts_liens', 'id_encart=' . intval($id) . ' AND objet=' . sql_quote($opts['objet']) . ' AND id_objet=' . intval($opts['id_objet'])) == 'non')
-	) ? true : false;
-
-	return $autoriser;
+		&& (
+			sql_getfetsel(
+				'vu',
+				'spip_encarts_liens',
+				'id_encart=' . intval($id) . ' AND objet=' . sql_quote($opts['objet']) . ' AND id_objet=' . intval($opts['id_objet'])
+			) == 'non'
+		)
+	);
 }
 
 
@@ -204,5 +211,5 @@ function autoriser_encart_dissocier_dist($faire, $type, $id, $qui, $opts) {
  * @return bool          true s'il a le droit, false sinon
  **/
 function autoriser_associerencarts_dist($faire, $type, $id, $qui, $opt) {
-	return $qui['statut'] == '0minirezo' and !$qui['restreint'];
+	return $qui['statut'] == '0minirezo' && !$qui['restreint'];
 }

encarts_pipelines.php

@@ -58,10 +58,10 @@ function encarts_affiche_milieu($flux) {
 	// encarts sur les objets configurés
 	if (
 		$objet
-		and $id_objet
-		and !$e['edition']
-		and is_array($tables_objets)
-		and in_array(table_objet_sql($objet), $tables_objets)
+		&& $id_objet
+		&& !$e['edition']
+		&& is_array($tables_objets)
+		&& in_array(table_objet_sql($objet), $tables_objets)
 	) {
 		$texte .= recuperer_fond('prive/objets/editer/liens', [
 			'table_source' => 'encarts',
@@ -132,10 +132,8 @@ function encarts_post_edition($flux) {
 	include_spip('base/objets');
 
 	if (
-		!empty($flux['args']['action'])
-		and $flux['args']['action'] === 'modifier'
-		and !empty($flux['args']['table'])
-		and !in_array($flux['args']['table'], ['spip_forum', 'spip_signatures'])
+		($flux['args']['action'] ?? '') === 'modifier'
+		&& !in_array($flux['args']['table'] ?? '', ['spip_forum', 'spip_signatures'])
 	) {
 		$serveur = $flux['args']['serveur'] ?? '';
 		$table_sql = $flux['args']['table'];

formulaires/editer_encart.php

@@ -166,10 +166,11 @@ function formulaires_editer_encart_traiter_dist($id_encart = 'new', $retour = ''
 	$res = formulaires_editer_objet_traiter('encart', $id_encart, '', $lier_trad, $retour, $config_fonc, $row, $hidden);
 
 	// Un lien a prendre en compte ?
-	if ($associer_objet and $id_encart = $res['id_encart']) {
+	$id_encart = $res['id_encart'];
+	if ($associer_objet && $id_encart) {
 		list($objet, $id_objet) = explode('|', $associer_objet);
 
-		if ($objet and $id_objet and autoriser('modifier', $objet, $id_objet)) {
+		if ($objet && $id_objet && autoriser('modifier', $objet, $id_objet)) {
 			include_spip('action/editer_liens');
 			objet_associer(['encart' => $id_encart], [$objet => $id_objet]);
 			if (isset($res['redirect'])) {

inc/marquer_doublons_encart.php

@@ -56,7 +56,7 @@ function inc_marquer_doublons_encart_dist($champs, $id_objet, $objet, $primary,
 	if ($load) {
 		$champs[$load] = '';
 		$row = sql_fetsel($load, $table_objet_sql, "$primary=" . sql_quote($id_objet));
-		if ($row and isset($row[$load])) {
+		if (isset($row[$load])) {
 			$champs[$load] = $row[$load];
 		}
 	}