From 8fa1a87a75e106da80548e65b49a66d88e79d386 Mon Sep 17 00:00:00 2001
From: "marcimat@rezo.net" <>
Date: Thu, 18 Oct 2018 07:46:25 +0000
Subject: [PATCH] =?UTF-8?q?Utiliser=20array=5Fcolumn=20plut=C3=B4t=20que?=
 =?UTF-8?q?=20array=5Fmap=20+=20reset=20/=20array=5Fshift?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 action/supprimer_document.php                          |  6 +++---
 inc/marquer_doublons_doc.php                           |  2 +-
 medias_administrations.php                             | 10 ++++++++--
 medias_pipelines.php                                   |  6 +++---
 paquet.xml                                             |  2 +-
 .../inclure/mediatheque-navigation_fonctions.php       |  6 +++---
 6 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/action/supprimer_document.php b/action/supprimer_document.php
index 7b8765e2..08fa1659 100644
--- a/action/supprimer_document.php
+++ b/action/supprimer_document.php
@@ -50,9 +50,9 @@ 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_column(
+		sql_allfetsel('id_document', 'spip_documents_liens', "objet='document' AND id_objet=" . intval($id_document)),
+		'id_document'
 	);
 	foreach ($annexes as $id) {
 		action_supprimer_document_dist($id);
diff --git a/inc/marquer_doublons_doc.php b/inc/marquer_doublons_doc.php
index a62de464..b9fb4ed6 100644
--- a/inc/marquer_doublons_doc.php
+++ b/inc/marquer_doublons_doc.php
@@ -142,7 +142,7 @@ function inc_marquer_doublons_doc_dist(
 	if ($nouveaux) {
 		// on vérifie que les documents indiqués vus existent réellement tout de même (en cas d'erreur de saisie)
 		$ids = sql_allfetsel('id_document', 'spip_documents', sql_in('id_document', $nouveaux));
-		$ids = array_map('reset', $ids);
+		$ids = array_column($ids, 'id_document');
 		if ($ids) {
 			// Creer le lien s'il n'existe pas déjà
 			objet_associer(array('document' => $ids), array($type => $id), array('vu' => 'oui'));
diff --git a/medias_administrations.php b/medias_administrations.php
index 5db1174b..91ee027d 100644
--- a/medias_administrations.php
+++ b/medias_administrations.php
@@ -30,7 +30,10 @@ function medias_check_statuts($affiche = false) {
 
 	// utiliser sql_allfetsel pour clore la requete avant la mise a jour en base sur chaque doc (sqlite)
 	// iterer par groupe de 100 pour ne pas exploser sur les grosses bases
-	$docs = array_map('reset', sql_allfetsel('id_document', 'spip_documents', "statut='0'", '', '', '0,100'));
+	$docs = array_column(
+		sql_allfetsel('id_document', 'spip_documents', "statut='0'", '', '', '0,100'),
+		'id_document'
+	);
 	while (count($docs)) {
 		include_spip('action/editer_document');
 		foreach ($docs as $id_document) {
@@ -40,7 +43,10 @@ function medias_check_statuts($affiche = false) {
 		if ($affiche) {
 			echo ' .';
 		}
-		$docs = array_map('reset', sql_allfetsel('id_document', 'spip_documents', "statut='0'", '', '', '0,100'));
+		$docs = array_column(
+			sql_allfetsel('id_document', 'spip_documents', "statut='0'", '', '', '0,100'),
+			'id_document'
+		);
 	}
 }
 
diff --git a/medias_pipelines.php b/medias_pipelines.php
index cf97e1fd..9e6e64bd 100644
--- a/medias_pipelines.php
+++ b/medias_pipelines.php
@@ -153,9 +153,9 @@ function medias_post_edition($flux) {
 		if (($flux['args']['action'] and $flux['args']['action'] == 'instituer') or isset($flux['data']['statut'])) {
 			include_spip('base/abstract_sql');
 			$id = $flux['args']['id_objet'];
-			$docs = array_map(
-				'reset',
-				sql_allfetsel('id_document', 'spip_documents_liens', 'id_objet=' . intval($id) . ' AND objet=' . sql_quote($type))
+			$docs = array_column(
+				sql_allfetsel('id_document', 'spip_documents_liens', 'id_objet=' . intval($id) . ' AND objet=' . sql_quote($type)),
+				'id_document'
 			);
 			include_spip('action/editer_document');
 			foreach ($docs as $id_document) {
diff --git a/paquet.xml b/paquet.xml
index a0d74209..46b5c13f 100644
--- a/paquet.xml
+++ b/paquet.xml
@@ -1,7 +1,7 @@
 <paquet
 	prefix="medias"
 	categorie="multimedia"
-	version="2.21.15"
+	version="2.21.16"
 	etat="stable"
 	compatibilite="[3.3.0-dev;3.3.*]"
 	logo="prive/themes/spip/images/portfolio-32.png"
diff --git a/prive/squelettes/inclure/mediatheque-navigation_fonctions.php b/prive/squelettes/inclure/mediatheque-navigation_fonctions.php
index 86b36e5e..3d89ceab 100644
--- a/prive/squelettes/inclure/mediatheque-navigation_fonctions.php
+++ b/prive/squelettes/inclure/mediatheque-navigation_fonctions.php
@@ -17,12 +17,12 @@ if (!defined('_ECRIRE_INC_VERSION')) {
 
 
 function liste_extensions_utilisees($media) {
-	return $media ? array_map(
-		'reset',
+	return $media ? array_column(
 		sql_allfetsel(
 			'extension',
 			'spip_documents',
 			'media=' . sql_quote($media)
-		)
+		),
+		'extension'
 	) : array();
 }
-- 
GitLab