Completed
Push — master ( 68e811...0ba822 )
by cam
01:41
created
ecrire/inc/autoriser.php 1 patch
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 // mais apres la fonction autoriser()
120 120
 if ($f = find_in_path('mes_fonctions.php')) {
121 121
 	global $dossier_squelettes;
122
-	include_once(_ROOT_CWD . $f);
122
+	include_once(_ROOT_CWD.$f);
123 123
 }
124 124
 
125 125
 
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 		$qui = $GLOBALS['visiteur_session'] ?: [];
161 161
 		$qui = array_merge(['statut' => '', 'id_auteur' => 0, 'webmestre' => 'non'], $qui);
162 162
 	} elseif (is_numeric($qui)) {
163
-		$qui = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . $qui);
163
+		$qui = sql_fetsel('*', 'spip_auteurs', 'id_auteur='.$qui);
164 164
 		if (!$qui) {
165 165
 			return false;
166 166
 		}
@@ -173,8 +173,8 @@  discard block
 block discarded – undo
173 173
 	}
174 174
 
175 175
 	spip_log(
176
-		"autoriser $faire $type $id (" . ($qui['nom'] ?? '') . ') ?',
177
-		'autoriser' . _LOG_DEBUG
176
+		"autoriser $faire $type $id (".($qui['nom'] ?? '').') ?',
177
+		'autoriser'._LOG_DEBUG
178 178
 	);
179 179
 
180 180
 	$type = autoriser_type($type);
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 		isset($GLOBALS['autoriser_exception'][$faire][$type][$id])
185 185
 		&& autoriser_exception($faire, $type, $id, 'verifier') || isset($GLOBALS['autoriser_exception'][$faire][$type]['*']) && autoriser_exception($faire, $type, '*', 'verifier')
186 186
 	) {
187
-		spip_log("autoriser ($faire, $type, $id, " . ($qui['nom'] ?? '') . ') : OK Exception', 'autoriser' . _LOG_DEBUG);
187
+		spip_log("autoriser ($faire, $type, $id, ".($qui['nom'] ?? '').') : OK Exception', 'autoriser'._LOG_DEBUG);
188 188
 		return true;
189 189
 	}
190 190
 
@@ -193,18 +193,18 @@  discard block
 block discarded – undo
193 193
 	// autoriser_faire[_dist], autoriser_defaut[_dist]
194 194
 	$fonctions = $type
195 195
 		? [
196
-			'autoriser_' . $type . '_' . $faire,
197
-			'autoriser_' . $type . '_' . $faire . '_dist',
198
-			'autoriser_' . $type,
199
-			'autoriser_' . $type . '_dist',
200
-			'autoriser_' . $faire,
201
-			'autoriser_' . $faire . '_dist',
196
+			'autoriser_'.$type.'_'.$faire,
197
+			'autoriser_'.$type.'_'.$faire.'_dist',
198
+			'autoriser_'.$type,
199
+			'autoriser_'.$type.'_dist',
200
+			'autoriser_'.$faire,
201
+			'autoriser_'.$faire.'_dist',
202 202
 			'autoriser_defaut',
203 203
 			'autoriser_defaut_dist'
204 204
 		]
205 205
 		: [
206
-			'autoriser_' . $faire,
207
-			'autoriser_' . $faire . '_dist',
206
+			'autoriser_'.$faire,
207
+			'autoriser_'.$faire.'_dist',
208 208
 			'autoriser_defaut',
209 209
 			'autoriser_defaut_dist'
210 210
 		];
@@ -218,8 +218,8 @@  discard block
 block discarded – undo
218 218
 	}
219 219
 
220 220
 	spip_log(
221
-		"$f($faire, $type, $id, " . ($qui['nom'] ?? '') . ') : ' . ($a ? 'OK' : 'niet'),
222
-		'autoriser' . _LOG_DEBUG
221
+		"$f($faire, $type, $id, ".($qui['nom'] ?? '').') : '.($a ? 'OK' : 'niet'),
222
+		'autoriser'._LOG_DEBUG
223 223
 	);
224 224
 
225 225
 	if (!is_bool($a)) {
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
 function autoriser_previsualiser_dist(string $faire, string $type, $id, array $qui, array $opt): bool {
386 386
 
387 387
 	// Le visiteur a-t-il un statut prevu par la config ?
388
-	if (str_contains((string) $GLOBALS['meta']['preview'], ',' . $qui['statut'] . ',')) {
388
+	if (str_contains((string) $GLOBALS['meta']['preview'], ','.$qui['statut'].',')) {
389 389
 		return test_previsualiser_objet_champ($type, $id, $qui, $opt);
390 390
 	}
391 391
 
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
439 439
 				} // pas de champ passe a la demande => NIET
440 440
 				$previsu = explode(',', (string) $c['previsu']);
441 441
 				// regarder si ce statut est autorise pour l'auteur
442
-				if (in_array($opt[$champ] . '/auteur', $previsu)) {
442
+				if (in_array($opt[$champ].'/auteur', $previsu)) {
443 443
 					// retrouver l’id_auteur qui a filé un lien de prévisu éventuellement,
444 444
 					// sinon l’auteur en session
445 445
 					include_spip('inc/securiser_action');
@@ -453,12 +453,12 @@  discard block
 block discarded – undo
453 453
 
454 454
 					if (!$id_auteur) {
455 455
 						return false;
456
-					} elseif (autoriser('previsualiser' . $opt[$champ], $type, 0, $id_auteur)) {
456
+					} elseif (autoriser('previsualiser'.$opt[$champ], $type, 0, $id_auteur)) {
457 457
 						// dans ce cas (admin en general), pas de filtrage sur ce statut
458 458
 					} elseif (
459 459
 						!sql_countsel(
460 460
 							'spip_auteurs_liens',
461
-							'id_auteur=' . (int) $id_auteur . ' AND objet=' . sql_quote($type) . ' AND id_objet=' . (int) $id
461
+							'id_auteur='.(int) $id_auteur.' AND objet='.sql_quote($type).' AND id_objet='.(int) $id
462 462
 						)
463 463
 					) {
464 464
 						return false;
@@ -503,14 +503,14 @@  discard block
 block discarded – undo
503 503
 			// multilinguisme par secteur et objet rattaché à une rubrique
504 504
 			$primary = id_table_objet($type);
505 505
 			$id_rubrique = $table != 'spip_rubriques'
506
-				? sql_getfetsel('id_rubrique', "$table", "$primary=" . (int) $id)
506
+				? sql_getfetsel('id_rubrique', "$table", "$primary=".(int) $id)
507 507
 				: $id;
508
-			$id_secteur = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
508
+			$id_secteur = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
509 509
 			if (!$id_secteur > 0) {
510 510
 				$id_secteur = $id_rubrique;
511 511
 			}
512
-			$langue_secteur = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_secteur);
513
-			$langue_objet = sql_getfetsel('lang', "$table", "$primary=" . (int) $id);
512
+			$langue_secteur = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique='.(int) $id_secteur);
513
+			$langue_objet = sql_getfetsel('lang', "$table", "$primary=".(int) $id);
514 514
 			if ($langue_secteur != $langue_objet) {
515 515
 				// configuration incohérente, on laisse l'utilisateur corriger la situation
516 516
 				return true;
@@ -518,7 +518,7 @@  discard block
 block discarded – undo
518 518
 			if ($table != 'spip_rubriques') { // le choix de la langue se fait seulement sur les rubriques
519 519
 				return false;
520 520
 			} else {
521
-				$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . (int) $id);
521
+				$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique='.(int) $id);
522 522
 				if ($id_parent != 0) {
523 523
 					// sous-rubriques : pas de choix de langue
524 524
 					return false;
@@ -570,7 +570,7 @@  discard block
 block discarded – undo
570 570
 
571 571
 	if (!isset($opt['statut'])) {
572 572
 		if (isset($desc['field']['statut'])) {
573
-			$statut = sql_getfetsel('statut', $desc['table'], id_table_objet($type) . '=' . (int) $id);
573
+			$statut = sql_getfetsel('statut', $desc['table'], id_table_objet($type).'='.(int) $id);
574 574
 		} else {
575 575
 			$statut = 'publie';
576 576
 		} // pas de statut => publie
@@ -735,11 +735,11 @@  discard block
 block discarded – undo
735 735
 		return false;
736 736
 	}
737 737
 
738
-	if (sql_countsel('spip_rubriques', 'id_parent=' . (int) $id)) {
738
+	if (sql_countsel('spip_rubriques', 'id_parent='.(int) $id)) {
739 739
 		return false;
740 740
 	}
741 741
 
742
-	if (sql_countsel('spip_articles', 'id_rubrique=' . (int) $id . " AND (statut<>'poubelle')")) {
742
+	if (sql_countsel('spip_articles', 'id_rubrique='.(int) $id." AND (statut<>'poubelle')")) {
743 743
 		return false;
744 744
 	}
745 745
 
@@ -776,7 +776,7 @@  discard block
 block discarded – undo
776 776
 	if (!$id) {
777 777
 		return false;
778 778
 	}
779
-	$r = sql_fetsel('id_rubrique,statut', 'spip_articles', 'id_article=' . sql_quote($id));
779
+	$r = sql_fetsel('id_rubrique,statut', 'spip_articles', 'id_article='.sql_quote($id));
780 780
 
781 781
 	return $r && (
782 782
 		autoriser('publierdans', 'rubrique', $r['id_rubrique'], $qui, $opt)
@@ -784,7 +784,7 @@  discard block
 block discarded – undo
784 784
 			(!isset($opt['statut']) || !in_array($opt['statut'], ['publie', 'refuse'], true))
785 785
 			&& in_array($qui['statut'], ['0minirezo', '1comite'])
786 786
 			&& in_array($r['statut'], ['prop', 'prepa', 'poubelle'])
787
-			&& auteurs_objet('article', $id, 'id_auteur=' . $qui['id_auteur'])
787
+			&& auteurs_objet('article', $id, 'id_auteur='.$qui['id_auteur'])
788 788
 		)
789 789
 	);
790 790
 }
@@ -842,7 +842,7 @@  discard block
 block discarded – undo
842 842
 		if (!$id) {
843 843
 			return false;
844 844
 		}
845
-		$statut = sql_getfetsel('statut', 'spip_articles', 'id_article=' . (int) $id);
845
+		$statut = sql_getfetsel('statut', 'spip_articles', 'id_article='.(int) $id);
846 846
 	}
847 847
 
848 848
 	return
@@ -851,7 +851,7 @@  discard block
 block discarded – undo
851 851
 		in_array($statut, ['prop', 'publie'])
852 852
 		// sinon si on est auteur, on a le droit de le voir, evidemment !
853 853
 
854
-		|| $id && $qui['id_auteur'] && auteurs_objet('article', $id, 'id_auteur=' . $qui['id_auteur']);
854
+		|| $id && $qui['id_auteur'] && auteurs_objet('article', $id, 'id_auteur='.$qui['id_auteur']);
855 855
 }
856 856
 
857 857
 
@@ -872,8 +872,8 @@  discard block
 block discarded – undo
872 872
 function autoriser_voir_dist(string $faire, string $type, $id, array $qui, array $opt): bool {
873 873
 	# securite, mais on aurait pas du arriver ici !
874 874
 	if (
875
-		function_exists($f = 'autoriser_' . $type . '_voir')
876
-		|| function_exists($f = 'autoriser_' . $type . '_voir_dist')
875
+		function_exists($f = 'autoriser_'.$type.'_voir')
876
+		|| function_exists($f = 'autoriser_'.$type.'_voir_dist')
877 877
 	) {
878 878
 		return $f($faire, $type, $id, $qui, $opt);
879 879
 	}
@@ -998,7 +998,7 @@  discard block
 block discarded – undo
998 998
 	$n = sql_fetsel(
999 999
 		'A.id_article',
1000 1000
 		'spip_auteurs_liens AS L LEFT JOIN spip_articles AS A ON (L.objet=\'article\' AND L.id_objet=A.id_article)',
1001
-		"A.statut='publie' AND L.id_auteur=" . sql_quote($id)
1001
+		"A.statut='publie' AND L.id_auteur=".sql_quote($id)
1002 1002
 	);
1003 1003
 
1004 1004
 	return (bool) $n;
@@ -1196,7 +1196,7 @@  discard block
 block discarded – undo
1196 1196
 		&& ($r = sql_allfetsel(
1197 1197
 			'id_objet',
1198 1198
 			'spip_auteurs_liens',
1199
-			'id_auteur=' . (int) $id_auteur . " AND objet='rubrique' AND id_objet!=0"
1199
+			'id_auteur='.(int) $id_auteur." AND objet='rubrique' AND id_objet!=0"
1200 1200
 		))
1201 1201
 		&& (is_countable($r) ? count($r) : 0)
1202 1202
 	) {
@@ -1873,8 +1873,8 @@  discard block
 block discarded – undo
1873 1873
 function auteurs_objet($objet, $id_objet, $cond = '') {
1874 1874
 	$objet = objet_type($objet);
1875 1875
 	$where = [
1876
-		'objet=' . sql_quote($objet),
1877
-		'id_objet=' . (int) $id_objet
1876
+		'objet='.sql_quote($objet),
1877
+		'id_objet='.(int) $id_objet
1878 1878
 	];
1879 1879
 	if (!empty($cond)) {
1880 1880
 		if (is_array($cond)) {
Please login to merge, or discard this patch.