Changeset 82881 in spip-zone

Timestamp:

Jun 6, 2014, 2:14:15 PM (5 years ago)

Author:

teddy.spip@…

Message:

On met des sécus avant le foreach pour éviter de gros warnnings php. cf. ​http://contrib.spip.net/Nettoyer-la-mediatheque#forum475697
Ajout de chaîne de langue.
Petit up de z

Location:

_plugins_/medias_nettoyage/trunk

Files:

• lang/medias_nettoyage_fr.php (modified) (1 diff)
• medias_nettoyage_fonctions.php (modified) (14 diffs)
• paquet.xml (modified) (1 diff)
• plugin.xml (modified) (1 diff)
• prive/squelettes/contenu/medias_rep_img.html (modified) (1 diff)

_plugins_/medias_nettoyage/trunk/lang/medias_nettoyage_fr.php

@@ -8 +8 @@
 
     // A
-    'aucun_document_orphelin' => 'Il n\'y a aucun document orphelin',
-    'aucun_logo' => 'Il n\'y a aucun logo',
+    'aucun_document_orphelin' => 'Il n\'y a aucun document orphelin.',
+    'aucun_logo' => 'Il n\'y a aucun logo.',
+    'aucun_repertoire' => 'Il n\'y a aucun répertoire.',
 
     'cfg_titre_parametrages' => 'Paramétrages',

_plugins_/medias_nettoyage/trunk/medias_nettoyage_fonctions.php

@@ -27 +27 @@
     $extensions = array();
     $extensions_cibles = sql_allfetsel('DISTINCT extension', 'spip_documents');
-    foreach ($extensions_cibles as $extension) {
-        $extensions[] = $extension['extension'];
+    // On vérifie bien qu'on reçoit un tableau.
+    if (is_array($extensions_cibles) and count($extensions_cibles) > 0) {
+        foreach ($extensions_cibles as $extension) {
+            $extensions[] = $extension['extension'];
+        }
     }
     // On rajoute le répertoire "vignettes"
@@ -50 +53 @@
     $extensions = medias_lister_extensions_documents();
 
-    foreach ($extensions as $extension) {
-        if (!is_dir($repertoire_img . $extension)) {
-            @mkdir($repertoire_img . $extension, _SPIP_CHMOD);
+    if (is_array($extensions) and count($extensions) > 0) {
+        foreach ($extensions as $extension) {
+            if (!is_dir($repertoire_img . $extension)) {
+                @mkdir($repertoire_img . $extension, _SPIP_CHMOD);
+            }
         }
     }
@@ -88 +93 @@
     // cf. ../IMG/orphelins qui ne serait pas encore créé.
     if (is_dir($repertoire_img)) {
+        // Avec la fonction scandir, on liste le contenu (existant) du répertoire cible.
         $rep_img = array_diff(scandir($repertoire_img), array('..','.','.svn')); // On ne liste pas le répertoire .svn
         foreach ($rep_img as $repertoire) {
+            // On vérifie que c'est un répertoire et non un fichier.
             if (is_dir($repertoire_img . $repertoire)) {
                 $repertoires[] = $repertoire_img . $repertoire;
@@ -112 +119 @@
 
     $docs_bdd = sql_allfetsel('fichier', 'spip_documents', "distant='non' AND fichier!=''");
-    foreach ($docs_bdd as $doc) {
-        /**
-         * On formate par rapport au répertoire ../IMG/
-         * On évite les doubles // qu'il peut y avoir
-         */
-        $docs_fichiers[] = preg_replace("/\/\//", "/", get_spip_doc($doc['fichier']));
-    }
-    // on enlève les url vides issues de la base :
-    $docs_fichiers = array_filter($docs_fichiers);
+    // On vérifie que nous avons au moins un élément dans le tableau
+    if (count($docs_bdd) > 0) {
+        foreach ($docs_bdd as $doc) {
+            /**
+             * On formate par rapport au répertoire ../IMG/
+             * On évite les doubles // qu'il peut y avoir
+             */
+            $docs_fichiers[] = preg_replace("/\/\//", "/", get_spip_doc($doc['fichier']));
+        }
+        // on enlève les url vides issues de la base :
+        $docs_fichiers = array_filter($docs_fichiers);
+    }
 
     // On trie dans l'ordre alphabétique :
@@ -187 +197 @@
 function medias_lister_documents_bdd_orphelins_taille()
 {
-    $documents_orphelins    = medias_lister_documents_bdd_orphelins();
+    $docs_orphelins    = medias_lister_documents_bdd_orphelins();
     $taille         = 0;
     $pattern_img        = "/" . preg_replace("/\//", "\/", _DIR_IMG) . "/";
 
-    if (count($documents_orphelins) > 0) {
-        $documents_bdd = sql_allfetsel(
+    if (count($docs_orphelins) > 0) {
+        $docs_bdd = sql_allfetsel(
             'fichier,taille',
             'spip_documents',
             "fichier IN ('"
-            . join("','", preg_replace($pattern_img, '', $documents_orphelins)) . "')"
+            . join("','", preg_replace($pattern_img, '', $docs_orphelins)) . "')"
         );
-        foreach ($documents_bdd as $document_bdd) {
-            if (!file_exists(get_spip_doc($document_bdd['fichier']))) {
-                $taille = $taille + ($document_bdd['taille']/1000);
-                // On divise par 1000 pour éviter la limite de l'integer php.
+        if (is_array($docs_bdd) and count($docs_bdd) > 0) {
+            foreach ($docs_bdd as $document_bdd) {
+                if (!file_exists(get_spip_doc($document_bdd['fichier']))) {
+                    $taille = $taille + ($document_bdd['taille']/1000);
+                    // On divise par 1000 pour éviter la limite de l'integer php.
+                }
             }
         }
@@ -224 +236 @@
 
     foreach (medias_lister_extensions_documents() as $extension) {
-        // On va chercher dans IMG/$extension/*.*
-        $fichiers = glob($repertoire_img . "$extension/*.*");
-        foreach ($fichiers as $fichier) {
-            $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+        // Par sécurité, on vérifie que l'extension a bel
+        // et bien un répertoire physique
+        if (is_dir($repertoire_img . $extension)) {
+            // On va chercher dans IMG/$extension/*.*
+            $fichiers = glob($repertoire_img . "$extension/*.*");
+            if (is_array($fichiers) and count($fichiers) > 0) {
+                foreach ($fichiers as $fichier) {
+                    $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+                }
+            }
         }
     }
     // On va chercher dans IMG/*.*
     $fichiers = glob($repertoire_img . "*.*");
-    foreach ($fichiers as $fichier) {
-        $docs_fichiers[] = $fichier;
+    // On vérifie que c'est bien un tableau, avec au moins un élément.
+    if (is_array($fichiers) and count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            $docs_fichiers[] = $fichier;
+        }
     }
     $docs_fichiers = array_unique(
@@ -305 +326 @@
         // Exemple : IMG/distant/jpg/nom_fichier.jpg
     $fichiers = glob($repertoire_img . "*/*/*.*");
-    foreach ($fichiers as $fichier) {
-        $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
-        // On évite les doubles slashs '//' qui pourrait arriver comme un cheveu sur la soupe.
+    if (is_array($fichiers) and count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+            // On évite les doubles slashs '//' qui pourrait arriver comme un cheveu sur la soupe.
+        }
     }
 
@@ -313 +336 @@
         // Exemple : IMG/pdf/nom_fichier.pdf
     $fichiers = glob($repertoire_img . "*/*.*");
-    foreach ($fichiers as $fichier) {
-        $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+    if (is_array($fichiers) and count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+        }
     }
 
@@ -320 +345 @@
         // Exemple : IMG/arton4.png
     $fichiers = glob($repertoire_img . "*.*");
-    foreach ($fichiers as $fichier) {
-        $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+    if (is_array($fichiers) and count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            $docs_fichiers[] = preg_replace("/\/\//", "/", $fichier);
+        }
     }
 
@@ -412 +439 @@
     // On force la recherche sur cet élément même si la recherche "classique"
     // devrait gérer cela initialement…
-    if ($logo_site = glob($repertoire_img . "site{on|off}0.*", GLOB_BRACE)
-        and count($logo_site) > 0) {
+    $logo_site = glob($repertoire_img . "site{on|off}0.*", GLOB_BRACE);
+    // On évite d'utiliser la fonction glob() directement dans le if car ça peut créer un bug pour PHP <5.4
+    // cf. http://contrib.spip.net/Nettoyer-la-mediatheque#forum475712
+    if (count($logo_site) > 0) {
         $fichiers = array_merge($fichiers, $logo_site);
     }
@@ -424 +453 @@
         // la regex de GLOB_BRACE est très basique...
         $liste = glob($repertoire_img . "{" . $logo_pattern ."}{on,off}*.*", GLOB_BRACE);
-        $fichiers = array_merge($fichiers, $liste);
-    }
-
-    foreach ($fichiers as $fichier) {
-        // ... Donc on fait une regex plus poussée avec un preg_match
-        if (preg_match("/(" . join("|", $logos_objet) .")on\d+.(" . join("|", $formats_logos) .")$/", $fichier)) {
-            $docs_fichiers_on[] = preg_replace("/\/\//", "/", $fichier);
-        }
-        if (preg_match("/(" . join("|", $logos_objet) .")off\d+.(" . join("|", $formats_logos) .")$/", $fichier)) {
-            $docs_fichiers_off[] = preg_replace("/\/\//", "/", $fichier);
+        if (is_array($liste) and count($liste) > 0) {
+            $fichiers = array_merge($fichiers, $liste);
+        }
+    }
+
+    // il faut avoir au moins un élément dans le tableau de fichiers.
+    if (count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            // ... Donc on fait une regex plus poussée avec un preg_match
+            if (preg_match("/(" . join("|", $logos_objet) .")on\d+.(" . join("|", $formats_logos) .")$/", $fichier)) {
+                $docs_fichiers_on[] = preg_replace("/\/\//", "/", $fichier);
+            }
+            if (preg_match("/(" . join("|", $logos_objet) .")off\d+.(" . join("|", $formats_logos) .")$/", $fichier)) {
+                $docs_fichiers_off[] = preg_replace("/\/\//", "/", $fichier);
+            }
         }
     }
@@ -610 +644 @@
     foreach ($pattern_obsoletes as $pattern) {
         $repertoires = glob($repertoire_img . $pattern . "*");
-        $repertoires_obsoletes = array_merge($repertoires_obsoletes, $repertoires);
+        if (is_array($repertoires) and count($repertoires) > 0) {
+            $repertoires_obsoletes = array_merge($repertoires_obsoletes, $repertoires);
+        }
     }
     // on fusionne avec les fichiers obsolètes
@@ -777 +813 @@
     // On va chercher dans IMG/*.*
     $fichiers = glob($repertoire_img . "*.*");
-    foreach ($fichiers as $fichier) {
-        $docs_fichiers[] = $fichier;
-    }
-    $docs_fichiers = array_filter(
-        array_diff(
-            $docs_fichiers,
-            medias_lister_logos_fichiers()
-        )
-    ); // a voir si on n'a pas de logos ce que ça donne comme ça…
+    if (is_array($fichiers) and count($fichiers) > 0) {
+        foreach ($fichiers as $fichier) {
+            $docs_fichiers[] = $fichier;
+        }
+        $docs_fichiers = array_filter(
+            array_diff(
+                $docs_fichiers,
+                medias_lister_logos_fichiers()
+            )
+        ); // a voir si on n'a pas de logos ce que ça donne comme ça…
+    }
     $docs_fichiers = preg_replace($pattern_img, '', $docs_fichiers);
 
@@ -802 +840 @@
             . ")"
         );
-        foreach ($docs_bdd as $document) {
-            $destination = preg_replace(',^([a-z0-3]+)_([^/]+\.(\1))$,i', '$1/$2', $document['fichier']);
-            // On va vérifier si on est bien sous la forme ../IMG/ext/nom_fichier.ext
-            // Sinon, on le construit manuellement.
-            // (ne pas oublier d'enlever '../IMG/' à notre variable de test
-            // car cette variable sera enresgitrée en BDD)
-            $destination_test = preg_replace($pattern_img, '', $destination);
-            if (count(explode("/", $destination_test)) == 1) {
-                $destination = $document['extension'] . '/' . $destination_test ;
-            }
-            if ($document['fichier'] != $destination
-                and rename($repertoire_img . $document['fichier'], $repertoire_img . $destination)) {
-                sql_updateq(
-                    'spip_documents',
-                    array('fichier' => $destination),
-                    'id_document=' . $document['id_document']
-                );
-                $message_log[] = date_format(date_create(), 'Y-m-d H:i:s')
-                . ' : le fichier '
-                . $repertoire_img
-                . $document['fichier']
-                . ' a été déplacé vers '
-                . $repertoire_img
-                . $destination
-                .'.';
-            } else {
-                $message_log[] = date_format(date_create(), 'Y-m-d H:i:s')
-                . ' : le fichier '
-                . $repertoire_img
-                . $document['fichier']
-                . ' n\'a pu être déplacé vers '
-                . $repertoire_img
-                . $destination
-                . '.';
+
+        if (is_array($docs_bdd) and count($docs_bdd) > 0) {
+            foreach ($docs_bdd as $document) {
+                $destination = preg_replace(',^([a-z0-3]+)_([^/]+\.(\1))$,i', '$1/$2', $document['fichier']);
+                // On va vérifier si on est bien sous la forme ../IMG/ext/nom_fichier.ext
+                // Sinon, on le construit manuellement.
+                // (ne pas oublier d'enlever '../IMG/' à notre variable de test
+                // car cette variable sera enresgitrée en BDD)
+                $destination_test = preg_replace($pattern_img, '', $destination);
+                if (count(explode("/", $destination_test)) == 1) {
+                    $destination = $document['extension'] . '/' . $destination_test ;
+                }
+                if ($document['fichier'] != $destination
+                    and rename($repertoire_img . $document['fichier'], $repertoire_img . $destination)) {
+                    sql_updateq(
+                        'spip_documents',
+                        array('fichier' => $destination),
+                        'id_document=' . $document['id_document']
+                    );
+                    $message_log[] = date_format(date_create(), 'Y-m-d H:i:s')
+                    . ' : le fichier '
+                    . $repertoire_img
+                    . $document['fichier']
+                    . ' a été déplacé vers '
+                    . $repertoire_img
+                    . $destination
+                    .'.';
+                } else {
+                    $message_log[] = date_format(date_create(), 'Y-m-d H:i:s')
+                    . ' : le fichier '
+                    . $repertoire_img
+                    . $document['fichier']
+                    . ' n\'a pu être déplacé vers '
+                    . $repertoire_img
+                    . $destination
+                    . '.';
+                }
             }
         }

_plugins_/medias_nettoyage/trunk/paquet.xml

@@ -2 +2 @@
         prefix="medias_nettoyage"
         categorie="outil"
-        version="0.9.3"
+        version="0.9.4"
         etat="stable"
         compatibilite="[2.1.0;3.0.*]"

_plugins_/medias_nettoyage/trunk/plugin.xml

@@ -5 +5 @@
         <icon>prive/themes/spip/images/medias_nettoyage-64.png</icon>
         <prefix>medias_nettoyage</prefix>
-        <version>0.9.3</version>
+        <version>0.9.4</version>
         <etat>stable</etat>
         <categorie>outil</categorie>

_plugins_/medias_nettoyage/trunk/prive/squelettes/contenu/medias_rep_img.html

@@ -2 +2 @@
 <div class="nettoyeur"></div>
 
-<INCLURE{fond=prive/objets/liste/repertoires,repertoires=#EVAL{medias_lister_repertoires()},env,ajax} />
+<INCLURE{fond=prive/objets/liste/repertoires,repertoires=#EVAL{medias_lister_repertoires()},sinon=<:medias_nettoyage:aucun_repertoire:>,env,ajax} />
 <div class="nettoyeur"></div>