Skip to content
Extraits de code Groupes Projets
Valider 8fa1a87a rédigé par marcimat@rezo.net's avatar marcimat@rezo.net
Parcourir les fichiers

Utiliser array_column plutôt que array_map + reset / array_shift

parent d892ac1f
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
......@@ -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);
......
......@@ -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'));
......
......@@ -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'
);
}
}
......
......@@ -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) {
......
<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"
......
......@@ -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();
}
0% Chargement en cours ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter