PHP/regex: Script pour créer des noms de fichiers avec des tirets au lieu d'espaces

Question

Je veux modifier un script PHP que j'utilise dans WordPress (Auto Featured Image plugin).
Le problème est que ce script crée des noms de fichiers pour les vignettes en fonction des URL de l'image.

Cela sonne bien jusqu'à ce que vous obtenez un nom de fichier avec des espaces et la vignette est quelque chose comme this%20Thumbnail.jpg et quand le navigateur va http://www.whatever.com/this%20Thumbnail.jpg il convertit le %20 à un espace et il n'y a aucun nom de fichier sur le serveur de ce nom (avec des espaces) .

Pour résoudre ce problème, je pense J'ai besoin de changer la ligne suivante de telle manière que $ imageURL est filtré pour convertir %20 en espaces. Vous avez raison?
Voici le code. Peut-être pouvez-vous me dire si j'aboie le mauvais arbre.
Merci!

<?php 
    static function create_post_attachment_from_url($imageUrl = null) 
    { 
     if(is_null($imageUrl)) return null; 
     // get file name 
     $filename = substr($imageUrl, (strrpos($imageUrl, '/'))+1); 
     if (!(($uploads = wp_upload_dir(current_time('mysql'))) && false === $uploads['error'])) { 
      return null; 
     } 
     // Generate unique file name 
     $filename = wp_unique_filename($uploads['path'], $filename); 
?> 

Answer 1

Edité à une réponse plus appropriée et complète:

static function create_post_attachment_from_url($imageUrl = null) 
{ 
    if(is_null($imageUrl)) return null; 

    // get the original filename from the URL 
    $filename = substr($imageUrl, (strrpos($imageUrl, '/'))+1); 

    // this bit is not relevant to the question, but we'll leave it in 
    if (!(($uploads = wp_upload_dir(current_time('mysql'))) && false === $uploads['error'])) { 
     return null; 
    } 

    // Sanitize the filename we extracted from the URL 
    // Replace any %-escaped character with a dash 
    $filename = preg_replace('/%[a-fA-F0-9]{2}/', '-', $filename); 

    // Let Wordpress further modify the filename if it may clash with 
    // an existing one in the same directory 
    $filename = wp_unique_filename($uploads['path'], $filename); 

    // ... 
} 

Answer 2

Vous feriez mieux de remplacer les espaces dans le nom de l'image ou des underscores hypens en utilisant regexp.

$string = "Google%20%20%20Search%20Amit%20Singhal" 
preg_replace('/%20+/g', ' ', $string); 

Cette expression remplacera plusieurs espaces (% 20) par un seul espace ('').