Changeset 82885 in spip-zone


Ignore:
Timestamp:
Jun 6, 2014, 7:16:57 PM (5 years ago)
Author:
teddy.spip@…
Message:

On doit toujours vérifier si on a bien un tableau après l'utilisation de la fonction glob(). Si cette fonction ne ramène aucun fichier, son retour est 'false'. On va aussi uniformiser les if sur les count(), on regarde si la variable est un tableau pour seulement après compter les entrées.
Up de z pour que l'archive se fasse. spipfacto, le patch vous sera utile.

Location:
_plugins_/medias_nettoyage/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/medias_nettoyage/trunk/medias_nettoyage_fonctions.php

    r82881 r82885  
    440440    // devrait gérer cela initialement…
    441441    $logo_site = glob($repertoire_img . "site{on|off}0.*", GLOB_BRACE);
    442     // On évite d'utiliser la fonction glob() directement dans le if car ça peut créer un bug pour PHP <5.4
     442    // On évite d'utiliser la fonction `glob()` directement dans le if car ça peut créer un bug pour PHP <5.4
     443    // S'il n'y a pas de siteon0.ext, `glob()` va retourner un `false`. Donc, on regarde si c'est bien un tableau.
    443444    // cf. http://contrib.spip.net/Nettoyer-la-mediatheque#forum475712
    444     if (count($logo_site) > 0) {
     445    if (is_array($logo_site) and count($logo_site) > 0) {
    445446        $fichiers = array_merge($fichiers, $logo_site);
    446447    }
     
    458459    }
    459460
    460     // il faut avoir au moins un élément dans le tableau de fichiers.
    461     if (count($fichiers) > 0) {
     461    // Il faut avoir au moins un élément dans le tableau de fichiers.
     462    if (is_array($fichiers) and count($fichiers) > 0) {
    462463        foreach ($fichiers as $fichier) {
    463464            // ... Donc on fait une regex plus poussée avec un preg_match
  • _plugins_/medias_nettoyage/trunk/paquet.xml

    r82881 r82885  
    22        prefix="medias_nettoyage"
    33        categorie="outil"
    4         version="0.9.4"
     4        version="0.9.5"
    55        etat="stable"
    66        compatibilite="[2.1.0;3.0.*]"
  • _plugins_/medias_nettoyage/trunk/plugin.xml

    r82881 r82885  
    55        <icon>prive/themes/spip/images/medias_nettoyage-64.png</icon>
    66        <prefix>medias_nettoyage</prefix>
    7         <version>0.9.4</version>
     7        <version>0.9.5</version>
    88        <etat>stable</etat>
    99        <categorie>outil</categorie>
Note: See TracChangeset for help on using the changeset viewer.