From d4a6e9e5923097a1b175e08d526aca2ac26dddae Mon Sep 17 00:00:00 2001
From: "kent1@arscenic.info" <>
Date: Wed, 2 Nov 2016 07:29:32 +0000
Subject: [PATCH] PSR SPIP

---
 action/acceder_document.php          |  35 ++++----
 action/ajouter_documents.php         | 123 +++++++++++++++------------
 action/changer_mode_document.php     |   4 +-
 action/copier_local.php              |  22 ++---
 action/dissocier_document.php        |  21 +++--
 action/editer_document.php           |  63 ++++++++------
 action/supprimer_document.php        |  15 ++--
 action/supprimer_tous_orphelins.php  |  16 ++--
 action/tourner.php                   |  15 ++--
 action/verifier_documents_brises.php |   2 +-
 action/verifier_documents_liens.php  |   3 +-
 11 files changed, 177 insertions(+), 142 deletions(-)

diff --git a/action/acceder_document.php b/action/acceder_document.php
index 92e12194..d1d09593 100644
--- a/action/acceder_document.php
+++ b/action/acceder_document.php
@@ -30,7 +30,7 @@ function action_acceder_document_dist() {
 	$file = get_spip_doc($f);
 	$arg = rawurldecode(_request('arg'));
 
-	$status = $dcc = false;
+	$status = false;
 	if (strpos($f, '../') !== false
 		or preg_match(',^\w+://,', $f)
 	) {
@@ -39,15 +39,17 @@ function action_acceder_document_dist() {
 		if (!file_exists($file) or !is_readable($file)) {
 			$status = 404;
 		} else {
-			$where = "D.fichier=" . sql_quote(set_spip_doc($file))
-				. ($arg ? " AND D.id_document=" . intval($arg) : '');
-
-			$doc = sql_fetsel("D.id_document, D.titre, D.fichier, T.mime_type, T.inclus, D.extension",
-				"spip_documents AS D LEFT JOIN spip_types_documents AS T ON D.extension=T.extension", $where);
+			$where = 'D.fichier=' . sql_quote(set_spip_doc($file))
+				. ($arg ? ' AND D.id_document=' . intval($arg) : '');
+
+			$doc = sql_fetsel(
+				'D.id_document, D.titre, D.fichier, T.mime_type, T.inclus, D.extension',
+				'spip_documents AS D LEFT JOIN spip_types_documents AS T ON D.extension=T.extension',
+				$where
+			);
 			if (!$doc) {
 				$status = 404;
 			} else {
-
 				// ETag pour gerer le status 304
 				$ETag = md5($file . ': ' . filemtime($file));
 				if (isset($_SERVER['HTTP_IF_NONE_MATCH'])
@@ -74,7 +76,6 @@ function action_acceder_document_dist() {
 	}
 
 	switch ($status) {
-
 		case 403:
 			include_spip('inc/minipres');
 			echo minipres();
@@ -83,19 +84,17 @@ function action_acceder_document_dist() {
 		case 404:
 			http_status(404);
 			include_spip('inc/minipres');
-			echo minipres(_T('erreur') . ' 404',
-				_T('medias:info_document_indisponible'));
+			echo minipres(_T('erreur') . ' 404', _T('medias:info_document_indisponible'));
 			break;
 
 		default:
-			header("Content-Type: " . $doc['mime_type']);
+			header('Content-Type: ' . $doc['mime_type']);
 
 			// pour les images ne pas passer en attachment
 			// sinon, lorsqu'on pointe directement sur leur adresse,
 			// le navigateur les downloade au lieu de les afficher
 
 			if ($doc['inclus'] == 'non') {
-
 				// Si le fichier a un titre avec extension,
 				// ou si c'est un nom bien connu d'Unix, le prendre
 				// sinon l'ignorer car certains navigateurs pataugent
@@ -111,21 +110,19 @@ function action_acceder_document_dist() {
 				header('Content-Type: application/octet-stream');
 
 				header("Content-Disposition: attachment; filename=\"$f\";");
-				header("Content-Transfer-Encoding: binary");
+				header('Content-Transfer-Encoding: binary');
 
 				// fix for IE catching or PHP bug issue
-				header("Pragma: public");
-				header("Expires: 0"); // set expiration time
-				header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
-
+				header('Pragma: public');
+				header('Expires: 0'); // set expiration time
+				header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
 			}
 
 			if ($cl = filesize($file)) {
-				header("Content-Length: " . $cl);
+				header('Content-Length: ' . $cl);
 			}
 
 			readfile($file);
 			break;
 	}
-
 }
diff --git a/action/ajouter_documents.php b/action/ajouter_documents.php
index 6c68bc6f..155531ed 100644
--- a/action/ajouter_documents.php
+++ b/action/ajouter_documents.php
@@ -16,7 +16,7 @@
  * @package SPIP\Medias\Action
  **/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -32,7 +32,7 @@ include_spip('inc/renseigner_document');
  *   Document à remplacer, ou pour une vignette, l'id_document de maman
  *   0 ou 'new' pour une insertion
  * @param array $files
- *   Tableau de taleau de propriété pour chaque document à insérer
+ *   Tableau de tableaux de propriétés pour chaque document à insérer
  * @param string $objet
  *   Objet auquel associer le document
  * @param int $id_objet
@@ -104,8 +104,9 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 	$mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode);
 
 	include_spip('inc/modifier');
-	if (isset($file['distant']) and $file['distant'] and !in_array($mode, array('choix', 'auto', 'image', 'document'))) {
-		spip_log("document distant $source accepte sans verification, mode=$mode","medias"._LOG_INFO_IMPORTANTE);
+	if (isset($file['distant']) and $file['distant']
+		and !in_array($mode, array('choix', 'auto', 'image', 'document'))) {
+		spip_log("document distant $source accepte sans verification, mode=$mode", 'medias'._LOG_INFO_IMPORTANTE);
 		include_spip('inc/distant');
 		$file['tmp_name'] = _DIR_RACINE . copie_locale($source);
 		$source = $file['tmp_name'];
@@ -116,12 +117,11 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 	// via une requete HEAD pour savoir si la ressource existe (non 404), si le
 	// content-type est connu, et si possible recuperer la taille, voire plus.
 	if (isset($file['distant']) and $file['distant']) {
-		if (!tester_url_absolue($source)){
+		if (!tester_url_absolue($source)) {
 			return _T('medias:erreur_chemin_distant', array('nom' => $source));
 		}
 		include_spip('inc/distant');
 		if (is_array($a = renseigner_source_distante($source))) {
-
 			$champs = $a;
 			# NB: dans les bonnes conditions (fichier autorise et pas trop gros)
 			# $a['fichier'] est une copie locale du fichier
@@ -148,13 +148,14 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 		$type_image = ''; // au pire
 		$champs['titre'] = '';
 		if ($titrer) {
-			$titre = substr($nom_envoye, 0, strrpos($nom_envoye, ".")); // Enlever l'extension du nom du fichier
+			$titre = substr($nom_envoye, 0, strrpos($nom_envoye, '.')); // Enlever l'extension du nom du fichier
 			$titre = preg_replace(',[[:punct:][:space:]]+,u', ' ', $titre);
 			$champs['titre'] = preg_replace(',\.([^.]+)$,', '', $titre);
 		}
 
 		if (!is_array($fichier = fixer_fichier_upload($file, $mode))) {
-			return is_string($fichier) ? $fichier : _T("medias:erreur_upload_type_interdit", array('nom' => $file['name']));
+			return is_string($fichier) ?
+				$fichier : _T('medias:erreur_upload_type_interdit', array('nom' => $file['name']));
 		}
 
 		$champs['inclus'] = $fichier['inclus'];
@@ -196,8 +197,11 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 
 	// si le media est pas renseigne, le faire, en fonction de l'extension
 	if (!isset($champs['media'])) {
-		$champs['media'] = sql_getfetsel('media_defaut', 'spip_types_documents',
-			'extension=' . sql_quote($champs['extension']));
+		$champs['media'] = sql_getfetsel(
+			'media_defaut',
+			'spip_types_documents',
+			'extension=' . sql_quote($champs['extension'])
+		);
 	}
 
 	// lier le parent si necessaire
@@ -219,15 +223,19 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 	// Installer le document dans la base
 	if (!$id_document) {
 		if ($id_document = document_inserer()) {
-			spip_log("ajout du document " . $file['tmp_name'] . " " . $file['name'] . "  (M '$mode' T '$objet' L '$id_objet' D '$id_document')",
-				'medias');
+			spip_log(
+				'ajout du document ' . $file['tmp_name'] . ' ' . $file['name'] . "  (M '$mode' T '$objet' L '$id_objet' D '$id_document')",
+				'medias'
+			);
 		} else {
-			spip_log("Echec insert_document() du document " . $file['tmp_name'] . " " . $file['name'] . "  (M '$mode' T '$objet' L '$id_objet' D '$id_document')",
-				'medias' . _LOG_ERREUR);
+			spip_log(
+				'Echec insert_document() du document ' . $file['tmp_name'] . ' ' . $file['name'] . "  (M '$mode' T '$objet' L '$id_objet' D '$id_document')",
+				'medias' . _LOG_ERREUR
+			);
 		}
 	}
 	if (!$id_document) {
-		return _T('medias:erreur_insertion_document_base', array('fichier' => "<em>" . $file['name'] . "</em>"));
+		return _T('medias:erreur_insertion_document_base', array('fichier' => '<em>' . $file['name'] . '</em>'));
 	}
 
 	document_modifier($id_document, $champs);
@@ -235,7 +243,8 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
 	// permettre aux plugins de faire des modifs a l'ajout initial
 	// ex EXIF qui tourne les images si necessaire
 	// Ce plugin ferait quand même mieux de se placer dans metadata/jpg.php
-	pipeline('post_edition',
+	pipeline(
+		'post_edition',
 		array(
 			'args' => array(
 				'table' => 'spip_documents', // compatibilite
@@ -311,28 +320,31 @@ function corriger_extension($ext) {
 function verifier_upload_autorise($source, $mode = '') {
 	$infos = array('fichier' => $source);
 	$res = false;
-	if (preg_match(",\.([a-z0-9]+)(\?.*)?$,i", $source, $match)
+	if (preg_match(',\.([a-z0-9]+)(\?.*)?$,i', $source, $match)
 		and $ext = $match[1]
 	) {
-
 		$ext = corriger_extension(strtolower($ext));
-		if ($res = sql_fetsel("extension,inclus,media_defaut as media", "spip_types_documents",
-			"extension=" . sql_quote($ext) . " AND upload='oui'")
-		) {
+		if ($res = sql_fetsel(
+			'extension,inclus,media_defaut as media',
+			'spip_types_documents',
+			'extension=' . sql_quote($ext) . " AND upload='oui'"
+		)) {
 			$infos = array_merge($infos, $res);
 		}
 	}
 	if (!$res) {
-		if ($res = sql_fetsel("extension,inclus,media_defaut as media", "spip_types_documents",
-			"extension='zip' AND upload='oui'")
-		) {
+		if ($res = sql_fetsel(
+			'extension,inclus,media_defaut as media',
+			'spip_types_documents',
+			"extension='zip' AND upload='oui'"
+		)) {
 			$infos = array_merge($infos, $res);
 			$res['autozip'] = true;
 		}
 	}
 	if ($mode and $res) {
 		// verifier en fonction du mode si une fonction est proposee
-		if ($verifier_document_mode = charger_fonction("verifier_document_mode_" . $mode, "inc", true)) {
+		if ($verifier_document_mode = charger_fonction('verifier_document_mode_' . $mode, 'inc', true)) {
 			$check = $verifier_document_mode($infos); // true ou message d'erreur sous forme de chaine
 			if ($check !== true) {
 				$res = $check;
@@ -376,18 +388,15 @@ function fixer_fichier_upload($file, $mode = '') {
 			 * si non, on le supprime et on affiche une erreur
 			 */
 			if ($row['fichier'] && (!$taille = @intval(filesize(get_spip_doc($row['fichier']))))) {
-				spip_log("Echec copie du fichier " . $file['tmp_name'] . " (taille de fichier indéfinie)");
+				spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)');
 				spip_unlink(get_spip_doc($row['fichier']));
-
 				return _T('medias:erreur_copie_fichier', array('nom' => $file['tmp_name']));
 			} else {
 				return $row;
 			}
-		}
-		// creer un zip comme demande
-		// pour encapsuler un fichier dont l'extension n'est pas supportee
-		else {
-
+		} else {
+			// creer un zip comme demande
+			// pour encapsuler un fichier dont l'extension n'est pas supportee
 			unset($row['autozip']);
 
 			$ext = 'zip';
@@ -401,7 +410,8 @@ function fixer_fichier_upload($file, $mode = '') {
 			include_spip('inc/charsets');
 			$tmp = $tmp_dir . '/' . translitteration($file['name']);
 
-			$file['name'] .= '.' . $ext; # conserver l'extension dans le nom de fichier, par exemple toto.js => toto.js.zip
+			// conserver l'extension dans le nom de fichier, par exemple toto.js => toto.js.zip
+			$file['name'] .= '.' . $ext;
 
 			// deplacer le fichier tmp_name dans le dossier tmp
 			deplacer_fichier_upload($file['tmp_name'], $tmp, true);
@@ -410,14 +420,17 @@ function fixer_fichier_upload($file, $mode = '') {
 			$source = _DIR_TMP . basename($tmp_dir) . '.' . $ext;
 			$archive = new PclZip($source);
 
-			$v_list = $archive->create($tmp,
-				PCLZIP_OPT_REMOVE_PATH, $tmp_dir,
-				PCLZIP_OPT_ADD_PATH, '');
+			$v_list = $archive->create(
+				$tmp,
+				PCLZIP_OPT_REMOVE_PATH,
+				$tmp_dir,
+				PCLZIP_OPT_ADD_PATH,
+				''
+			);
 
 			effacer_repertoire_temporaire($tmp_dir);
 			if (!$v_list) {
-				spip_log("Echec creation du zip ");
-
+				spip_log('Echec creation du zip');
 				return false;
 			}
 
@@ -428,7 +441,7 @@ function fixer_fichier_upload($file, $mode = '') {
 			 * si non, on le supprime et on affiche une erreur
 			 */
 			if ($row['fichier'] && (!$taille = @intval(filesize(get_spip_doc($row['fichier']))))) {
-				spip_log("Echec copie du fichier " . $file['tmp_name'] . " (taille de fichier indéfinie)");
+				spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)');
 				spip_unlink(get_spip_doc($row['fichier']));
 
 				return _T('medias:erreur_copie_fichier', array('nom' => $file['tmp_name']));
@@ -452,7 +465,8 @@ function verifier_taille_document_acceptable(&$infos) {
 	// si ce n'est pas une image
 	if (!$infos['type_image']) {
 		if (defined('_DOC_MAX_SIZE') and _DOC_MAX_SIZE > 0 and $infos['taille'] > _DOC_MAX_SIZE * 1024) {
-			return _T('medias:info_doc_max_poids',
+			return _T(
+				'medias:info_doc_max_poids',
 				array(
 					'maxi' => taille_en_octets(_DOC_MAX_SIZE * 1024),
 					'actuel' => taille_en_octets($infos['taille'])
@@ -461,7 +475,6 @@ function verifier_taille_document_acceptable(&$infos) {
 		}
 	} // si c'est une image
 	else {
-
 		if ((defined('_IMG_MAX_WIDTH') and _IMG_MAX_WIDTH and $infos['largeur'] > _IMG_MAX_WIDTH)
 			or (defined('_IMG_MAX_HEIGHT') and _IMG_MAX_HEIGHT and $infos['hauteur'] > _IMG_MAX_HEIGHT)
 		) {
@@ -488,39 +501,43 @@ function verifier_taille_document_acceptable(&$infos) {
 			if ((defined('_IMG_MAX_WIDTH') and _IMG_MAX_WIDTH and $infos['largeur'] > _IMG_MAX_WIDTH)
 				or (defined('_IMG_MAX_HEIGHT') and _IMG_MAX_HEIGHT and $infos['hauteur'] > _IMG_MAX_HEIGHT)
 			) {
-
-				return _T('medias:info_image_max_taille',
+				return _T(
+					'medias:info_image_max_taille',
 					array(
 						'maxi' =>
-							_T('info_largeur_vignette',
+							_T(
+								'info_largeur_vignette',
 								array(
 									'largeur_vignette' => $max_width,
 									'hauteur_vignette' => $max_height
-								)),
+								)
+							),
 						'actuel' =>
-							_T('info_largeur_vignette',
+							_T(
+								'info_largeur_vignette',
 								array(
 									'largeur_vignette' => $infos['largeur'],
 									'hauteur_vignette' => $infos['hauteur']
-								))
-					));
+								)
+							)
+					)
+				);
 			}
 		}
 
 		if (defined('_IMG_MAX_SIZE') and _IMG_MAX_SIZE > 0 and $infos['taille'] > _IMG_MAX_SIZE * 1024) {
-			return _T('medias:info_image_max_poids',
+			return _T(
+				'medias:info_image_max_poids',
 				array(
 					'maxi' => taille_en_octets(_IMG_MAX_SIZE * 1024),
-					'actuel' => taille_en_octets($infos['taille']
-					)
+					'actuel' => taille_en_octets($infos['taille'])
 				)
 			);
 		}
-
 	}
 
 	// verifier en fonction du mode si une fonction est proposee
-	if ($verifier_document_mode = charger_fonction("verifier_document_mode_" . $infos['mode'], "inc", true)) {
+	if ($verifier_document_mode = charger_fonction('verifier_document_mode_' . $infos['mode'], 'inc', true)) {
 		return $verifier_document_mode($infos);
 	}
 
diff --git a/action/changer_mode_document.php b/action/changer_mode_document.php
index 8e9c5963..66eab81b 100644
--- a/action/changer_mode_document.php
+++ b/action/changer_mode_document.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -29,7 +29,7 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 
-		if (!preg_match(",^(\d+)\W(\w+)$,", $arg, $r)) {
+		if (!preg_match(',^(\d+)\W(\w+)$,', $arg, $r)) {
 			spip_log("action_changer_mode_document $arg pas compris");
 		} else {
 			array_shift($r);
diff --git a/action/copier_local.php b/action/copier_local.php
index 709d3d01..7d22776a 100644
--- a/action/copier_local.php
+++ b/action/copier_local.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -54,21 +54,21 @@ function action_copier_local_dist($id_document = null) {
 function action_copier_local_post($id_document) {
 
 	// Il faut la source du document pour le copier
-	$row = sql_fetsel("mode,fichier, descriptif, credits", "spip_documents", "id_document=$id_document");
+	$row = sql_fetsel('mode,fichier, descriptif, credits', 'spip_documents', 'id_document='.intval($id_document));
 	$source = $row['fichier'];
 
 	// si la source est bien un fichier distant
 	// sinon c'est une donnee moisie, on ne fait rien
-	if (tester_url_absolue($source)){
-
+	if (tester_url_absolue($source)) {
 		include_spip('inc/distant'); // pour 'copie_locale'
 		$fichier = copie_locale($source);
 		if ($fichier
-		  and tester_url_absolue($source)) {
+			and tester_url_absolue($source)) {
 			$fichier = _DIR_RACINE . $fichier;
+			$files = array();
 			$files[] = array('tmp_name' => $fichier, 'name' => basename($fichier));
 			$ajouter_documents = charger_fonction('ajouter_documents', 'action');
-			spip_log("convertit doc $id_document en local: $source => $fichier", "medias");
+			spip_log("convertit doc $id_document en local: $source => $fichier", 'medias');
 			$liste = array();
 			$ajouter_documents($id_document, $files, '', 0, $row['mode'], $liste);
 
@@ -76,15 +76,17 @@ function action_copier_local_post($id_document) {
 
 			// ajouter l'origine du document aux credits
 			include_spip('action/editer_document');
-			document_modifier($id_document, array('credits' => ($row['credits'] ? $row['credits'] . ', ' : '') . $source));
+			document_modifier(
+				$id_document,
+				array('credits' => ($row['credits'] ? $row['credits'] . ', ' : '') . $source)
+			);
 
 			return true;
-
 		} else {
-			spip_log("echec copie locale $source", "medias" . _LOG_ERREUR);
+			spip_log("echec copie locale $source", 'medias' . _LOG_ERREUR);
 		}
 	} else {
-		spip_log("echec copie locale $source n'est pas une URL distante", "medias" . _LOG_ERREUR);
+		spip_log("echec copie locale $source n'est pas une URL distante", 'medias' . _LOG_ERREUR);
 	}
 
 	return _T('medias:erreur_copie_fichier', array('nom' => $source));
diff --git a/action/dissocier_document.php b/action/dissocier_document.php
index 4ee01cb0..4bf0d842 100644
--- a/action/dissocier_document.php
+++ b/action/dissocier_document.php
@@ -16,7 +16,7 @@
  * @package SPIP\Medias\Action
  **/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -64,7 +64,7 @@ function action_dissocier_document_dist($arg = null) {
 	) {
 		dissocier_document($document, $objet, $id_objet, $suppr, $check);
 	} else {
-		spip_log("Interdit de modifier $objet $id_objet", "spip");
+		spip_log("Interdit de modifier $objet $id_objet", 'spip');
 	}
 }
 
@@ -95,7 +95,7 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
 
 	// Si c'est une vignette, l'eliminer du document auquel elle appartient
 	// cas tordu peu probable
-	sql_updateq("spip_documents", array('id_vignette' => 0), "id_vignette=" . $id_document);
+	sql_updateq('spip_documents', array('id_vignette' => 0), 'id_vignette=' . $id_document);
 
 	// verifier son statut apres une suppression de lien
 	include_spip('action/editer_document');
@@ -105,7 +105,8 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
 	include_spip('inc/invalideur');
 	suivre_invalideur("id='id_document/$id_document'");
 
-	pipeline('post_edition',
+	pipeline(
+		'post_edition',
 		array(
 			'args' => array(
 				'operation' => 'delier_document', // compat v<=2
@@ -124,7 +125,7 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
 		$spip_table_objet = table_objet_sql($objet);
 		$table_objet = table_objet($objet);
 		$id_table_objet = id_table_objet($objet, $serveur);
-		$champs = sql_fetsel('*', $spip_table_objet, addslashes($id_table_objet) . "=" . intval($id_objet));
+		$champs = sql_fetsel('*', $spip_table_objet, addslashes($id_table_objet) . '=' . intval($id_objet));
 
 		$marquer_doublons_doc = charger_fonction('marquer_doublons_doc', 'inc');
 		$marquer_doublons_doc($champs, $id_objet, $objet, $id_table_objet, $table_objet, $spip_table_objet, '', $serveur);
@@ -166,11 +167,13 @@ function dissocier_document($document, $objet, $id_objet, $supprime = false, $ch
 		$image = ($image == 'I');
 		$typdoc = sql_in('docs.extension', array('gif', 'jpg', 'png'), $image ? '' : 'NOT');
 
-		$obj = "id_objet=" . intval($id_objet) . " AND objet=" . sql_quote($objet);
+		$obj = 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet);
 
-		$s = sql_select('docs.id_document',
-			"spip_documents AS docs LEFT JOIN spip_documents_liens AS l ON l.id_document=docs.id_document",
-			"$obj AND vu='non' AND docs.mode=" . sql_quote($mode) . " AND $typdoc");
+		$s = sql_select(
+			'docs.id_document',
+			'spip_documents AS docs LEFT JOIN spip_documents_liens AS l ON l.id_document=docs.id_document',
+			"$obj AND vu='non' AND docs.mode=" . sql_quote($mode) . " AND $typdoc"
+		);
 		while ($t = sql_fetch($s)) {
 			supprimer_lien_document($t['id_document'], $objet, $id_objet, $supprime, $check);
 		}
diff --git a/action/editer_document.php b/action/editer_document.php
index ed8c0eda..b5ea8a14 100644
--- a/action/editer_document.php
+++ b/action/editer_document.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -62,7 +62,8 @@ function document_inserer($id_parent = null, $set = null) {
 	}
 
 	// Envoyer aux plugins
-	$champs = pipeline('pre_insertion',
+	$champs = pipeline(
+		'pre_insertion',
 		array(
 			'args' => array(
 				'table' => 'spip_documents',
@@ -70,8 +71,9 @@ function document_inserer($id_parent = null, $set = null) {
 			'data' => $champs
 		)
 	);
-	$id_document = sql_insertq("spip_documents", $champs);
-	pipeline('post_insertion',
+	$id_document = sql_insertq('spip_documents', $champs);
+	pipeline(
+		'post_insertion',
 		array(
 			'args' => array(
 				'table' => 'spip_documents',
@@ -100,7 +102,7 @@ function document_modifier($id_document, $set = null) {
 
 	// champs normaux
 	$champs = collecter_requests(
-	// white list
+		// white list
 		objet_info('document', 'champs_editables'),
 		// black list
 		array('parents', 'ajout_parents'),
@@ -109,30 +111,32 @@ function document_modifier($id_document, $set = null) {
 	);
 
 
-	$invalideur = "";
+	$invalideur = '';
 	$indexation = false;
 
 	// Si le document est publie, invalider les caches et demander sa reindexation
-	$t = sql_getfetsel("statut", "spip_documents", 'id_document=' . intval($id_document));
+	$t = sql_getfetsel('statut', 'spip_documents', 'id_document=' . intval($id_document));
 	if ($t == 'publie') {
 		$invalideur = "id='id_document/$id_document'";
 		$indexation = true;
 	}
 
-	$ancien_fichier = "";
+	$ancien_fichier = '';
 	// si le fichier est modifie, noter le nom de l'ancien pour faire le menage
 	if (isset($champs['fichier'])) {
 		$ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document));
 	}
 
-	if ($err = objet_modifier_champs('document', $id_document,
+	if ($err = objet_modifier_champs(
+		'document',
+		$id_document,
 		array(
 			'data' => $set,
 			'invalideur' => $invalideur,
 			'indexation' => $indexation
 		),
-		$champs)
-	) {
+		$champs
+	)) {
 		return $err;
 	}
 
@@ -151,7 +155,6 @@ function document_modifier($id_document, $set = null) {
 	// le statut n'est jamais fixe manuellement mais decoule de celui des objets lies
 	$champs = collecter_requests(array('parents', 'ajouts_parents'), array(), $set);
 	if (document_instituer($id_document, $champs)) {
-
 		//
 		// Post-modifications
 		//
@@ -160,7 +163,6 @@ function document_modifier($id_document, $set = null) {
 		include_spip('inc/invalideur');
 		suivre_invalideur("id='id_document/$id_document'");
 	}
-
 }
 
 
@@ -183,7 +185,7 @@ function document_instituer($id_document, $champs = array()) {
 		medias_revision_document_parents($id_document, $champs['ajout_parents'], true);
 	}
 
-	$row = sql_fetsel("statut,date_publication", "spip_documents", "id_document=$id_document");
+	$row = sql_fetsel('statut,date_publication', 'spip_documents', 'id_document='.intval($id_document));
 	$statut_ancien = $row['statut'];
 	$date_publication_ancienne = $row['date_publication'];
 
@@ -192,8 +194,11 @@ function document_instituer($id_document, $champs = array()) {
 		$statut = 'prepa';
 
 		$trouver_table = charger_fonction('trouver_table', 'base');
-		$res = sql_select('id_objet,objet', 'spip_documents_liens',
-			"objet!='document' AND id_document=" . intval($id_document));
+		$res = sql_select(
+			'id_objet,objet',
+			'spip_documents_liens',
+			"objet!='document' AND id_document=" . intval($id_document)
+		);
 		// dans 10 ans, ca nous fera un bug a corriger vers 2018
 		// penser a ouvrir un ticket d'ici la :p
 		$date_publication = time() + 10 * 365 * 24 * 3600;
@@ -210,8 +215,11 @@ function document_instituer($id_document, $champs = array()) {
 				continue;
 			} // si pas publie, et article, il faut checker la date de post-publi eventuelle
 			elseif ($row['objet'] == 'article'
-				and $row2 = sql_fetsel('date', 'spip_articles',
-					'id_article=' . intval($row['id_objet']) . " AND statut='publie'")
+				and $row2 = sql_fetsel(
+					'date',
+					'spip_articles',
+					'id_article=' . intval($row['id_objet']) . " AND statut='publie'"
+				)
 			) {
 				$statut = 'publie';
 				$date_publication = min($date_publication, strtotime($row2['date']));
@@ -228,11 +236,17 @@ function document_instituer($id_document, $champs = array()) {
 	if ($statut !== $statut_ancien
 		or $date_publication != $date_publication_ancienne
 	) {
-		sql_updateq('spip_documents', array('statut' => $statut, 'date_publication' => $date_publication),
-			'id_document=' . intval($id_document));
+		sql_updateq(
+			'spip_documents',
+			array('statut' => $statut, 'date_publication' => $date_publication),
+			'id_document=' . intval($id_document)
+		);
 		if ($statut !== $statut_ancien) {
-			$publier_rubriques = sql_allfetsel('id_objet', 'spip_documents_liens',
-				"objet='rubrique' AND id_document=" . intval($id_document));
+			$publier_rubriques = sql_allfetsel(
+				'id_objet',
+				'spip_documents_liens',
+				"objet='rubrique' AND id_document=" . intval($id_document)
+			);
 			if (count($publier_rubriques)) {
 				include_spip('inc/rubriques');
 				foreach ($publier_rubriques as $r) {
@@ -240,10 +254,8 @@ function document_instituer($id_document, $champs = array()) {
 				}
 			}
 		}
-
 		return true;
 	}
-
 	return false;
 }
 
@@ -258,7 +270,7 @@ function document_instituer($id_document, $champs = array()) {
  */
 function medias_revision_document_parents($id_document, $parents = null, $ajout = false) {
 	include_spip('inc/autoriser');
-	
+
 	if (!is_array($parents)) {
 		return;
 	}
@@ -303,7 +315,6 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
 		}
 	}
 	objet_associer(array('document' => $id_document), $objets_parents);
-
 }
 
 
diff --git a/action/supprimer_document.php b/action/supprimer_document.php
index 026b8c96..9c582d95 100644
--- a/action/supprimer_document.php
+++ b/action/supprimer_document.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -30,7 +30,7 @@ function action_supprimer_document_dist($id_document = 0) {
 	// si c'etait une vignette, modifier le document source !
 	if ($source = sql_getfetsel('id_document', 'spip_documents', 'id_vignette=' . intval($id_document))) {
 		include_spip('action/editer_document');
-		document_modifier($source, array("id_vignette" => 0));
+		document_modifier($source, array('id_vignette' => 0));
 	}
 
 	include_spip('inc/documents');
@@ -40,7 +40,7 @@ function action_supprimer_document_dist($id_document = 0) {
 		return false;
 	}
 
-	spip_log("Suppression du document $id_document (" . $doc['fichier'] . ")");
+	spip_log("Suppression du document $id_document (" . $doc['fichier'] . ')');
 	include_spip('action/editer_liens');
 
 	// Si c'est un document ayant une vignette, supprimer aussi la vignette
@@ -50,8 +50,10 @@ function action_supprimer_document_dist($id_document = 0) {
 	}
 	// Si c'est un document ayant des documents annexes (sous-titre, ...)
 	// les supprimer aussi
-	$annexes = array_map('reset',
-		sql_allfetsel("id_document", "spip_documents_liens", "objet='document' AND id_objet=" . intval($id_document)));
+	$annexes = array_map(
+		'reset',
+		sql_allfetsel('id_document', 'spip_documents_liens', "objet='document' AND id_objet=" . intval($id_document))
+	);
 	foreach ($annexes as $id) {
 		action_supprimer_document_dist($id);
 	}
@@ -72,7 +74,8 @@ function action_supprimer_document_dist($id_document = 0) {
 		spip_unlink(get_spip_doc($doc['fichier']));
 	}
 
-	pipeline('post_edition',
+	pipeline(
+		'post_edition',
 		array(
 			'args' => array(
 				'operation' => 'supprimer_document', // compat v<=2
diff --git a/action/supprimer_tous_orphelins.php b/action/supprimer_tous_orphelins.php
index 60138c68..a88e5839 100644
--- a/action/supprimer_tous_orphelins.php
+++ b/action/supprimer_tous_orphelins.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -24,19 +24,20 @@ function action_supprimer_tous_orphelins() {
 	//on recupere le contexte pour ne supprimer les orphelins que de ce dernier
 	list($media, $distant, $statut, $sanstitre) = explode('/', $arg);
 
+	$where = array();
 	//critere sur le media
 	if ($media) {
-		$select = "media=" . sql_quote($media);
+		$select = 'media=' . sql_quote($media);
 	}
 
 	//critere sur le distant
 	if ($distant) {
-		$where[] = "distant=" . sql_quote($distant);
+		$where[] = 'distant=' . sql_quote($distant);
 	}
 
 	//critere sur le statut
 	if ($statut) {
-		$where[] = "statut REGEXP " . sql_quote("($statut)");
+		$where[] = 'statut REGEXP ' . sql_quote("($statut)");
 	}
 
 	//critere sur le sanstitre
@@ -45,14 +46,15 @@ function action_supprimer_tous_orphelins() {
 	}
 
 	//on isole les orphelins
-	$select = sql_get_select("DISTINCT id_document", "spip_documents_liens as oooo");
+	$select = sql_get_select('DISTINCT id_document', 'spip_documents_liens as oooo');
 	$cond = "spip_documents.id_document NOT IN ($select)";
 	$where[] = $cond;
 
-	$ids_doc_orphelins = sql_select("id_document", "spip_documents", $where);
+	$ids_doc_orphelins = sql_select('id_document', 'spip_documents', $where);
 
 	$supprimer_document = charger_fonction('supprimer_document', 'action');
 	while ($row = sql_fetch($ids_doc_orphelins)) {
-		$supprimer_document($row['id_document']); // pour les orphelins du contexte, on traite avec la fonction existante
+		// pour les orphelins du contexte, on traite avec la fonction existante
+		$supprimer_document($row['id_document']);
 	}
 }
diff --git a/action/tourner.php b/action/tourner.php
index 16591095..94a26512 100644
--- a/action/tourner.php
+++ b/action/tourner.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -32,13 +32,12 @@ function action_tourner_dist($id_document = null, $angle = null) {
 		$securiser_action = charger_fonction('securiser_action', 'inc');
 		$arg = $securiser_action();
 
-		if (!preg_match(",^\W*(\d+)\W?(-?\d+)$,", $arg, $r)) {
+		if (!preg_match(',^\W*(\d+)\W?(-?\d+)$,', $arg, $r)) {
 			spip_log("action_tourner_dist $arg pas compris");
 		} else {
 			array_shift($r);
 			list($id_document, $angle) = $r;
 		}
-
 	}
 	if ($id_document and autoriser('modifier', 'document', $id_document)) {
 		action_tourner_post($id_document, $angle);
@@ -56,7 +55,7 @@ function action_tourner_dist($id_document = null, $angle = null) {
  * @return
  */
 function action_tourner_post($id_document, $angle) {
-	$row = sql_fetsel("fichier,extension", "spip_documents", "id_document=" . intval($id_document));
+	$row = sql_fetsel('fichier,extension', 'spip_documents', 'id_document=' . intval($id_document));
 
 	if (!$row) {
 		return;
@@ -109,13 +108,14 @@ function action_tourner_post($id_document, $angle) {
 		if ($taille = @filesize($dest)) {
 			$set['taille'] = $taille;
 		}
-		sql_updateq('spip_documents', $set, "id_document=" . intval($id_document));
+		sql_updateq('spip_documents', $set, 'id_document=' . intval($id_document));
 		if ($effacer) {
 			spip_log("rotation : j'efface $effacer");
 			spip_unlink($effacer);
 		}
 		// pipeline pour les plugins
-		pipeline('post_edition',
+		pipeline(
+			'post_edition',
 			array(
 				'args' => array(
 					'table' => 'spip_documents',
@@ -130,7 +130,6 @@ function action_tourner_post($id_document, $angle) {
 			)
 		);
 	}
-
 }
 
 // Appliquer l'EXIF orientation
@@ -149,8 +148,10 @@ function tourner_selon_exif_orientation($id_document, $fichier) {
 		switch ($ort) {
 			case 3:
 				$rot = 180;
+				// rotation à 180
 			case 6:
 				$rot = 90;
+				// rotation à 90
 			case 8:
 				$rot = -90;
 		}
diff --git a/action/verifier_documents_brises.php b/action/verifier_documents_brises.php
index 8195bcf3..bf5385bb 100644
--- a/action/verifier_documents_brises.php
+++ b/action/verifier_documents_brises.php
@@ -10,7 +10,7 @@
  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
 \***************************************************************************/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
diff --git a/action/verifier_documents_liens.php b/action/verifier_documents_liens.php
index 4e1a6513..12a17f7d 100644
--- a/action/verifier_documents_liens.php
+++ b/action/verifier_documents_liens.php
@@ -16,7 +16,7 @@
  * @package SPIP\Medias\Action
  **/
 
-if (!defined("_ECRIRE_INC_VERSION")) {
+if (!defined('_ECRIRE_INC_VERSION')) {
 	return;
 }
 
@@ -37,5 +37,4 @@ function action_verifier_documents_liens_dist($id_document = null) {
 	$id_document = ($id_document == '*') ? '*' : intval($id_document);
 	include_spip('action/editer_liens');
 	objet_optimiser_liens(array('document' => $id_document), '*');
-
 }
-- 
GitLab