[Obm] Bug : module document - nom de fichier avec espace

[Mehdi Rande]

Le mercredi 28 mai 2008 à 00:42 +0200, Romain MARIADASSOU a écrit :
> Bonjour
> 

Bonjour,

> Il y a quelques problèmes avec le module "document" et la gestion des 
> fichiers avec espace.
> 
> Lors du téléchargement des fichiers, il faut mettre dans le fichier 
> [OBM_INSTALL]/php/document/document_display.inc à la ligne 1127 (version 
> trunk) mais cela vaut aussi dans la version 2.1.9 (la ligne change peut 
> être), dans la fonction dis_document_file :
> 
> Remplacer :
> header("Content-Disposition: inline; filename=$name");
> 
> Par :
> header('Content-Disposition: inline; filename="'.$name.'"');
> 

Oui en effet c'est intégré

> 
> De plus, pour éviter une trop grosse consommation de mémoire lors de 
> download de fichier, il faudrait envoyer le fichier avec le code suivant 
> qui permet de vider le buffer php (toujours dans la fonction 
> dis_document_file) :
> 
>     $handle = fopen ($disk_fullname, "r");
>     header('Content-Type: '.$mime.'; name="'.$name.'"');
>     header('Content-Disposition: inline; filename="'.$name.'"');
> 
>     while ( !feof($handle) ) {
>             print fread($handle, 4096);
>             ob_flush();
>     }
>     fclose ($handle);
> 

Oui l'idée est bonne, c'est également intégré


> 
> Modif supplémentaire :
> Toujours dans le même fichier à la ligne 993 pour la fonction 
> display_document_tree, il y a une fermeture d'accolade qui ne sert pas 
> mais qui s'affiche dans le navigateur (à moins que cela ne soit voulu) :
> 
> Remplacer :
>   [...]

Intégré pour la 2.1.10

Merci beaucoup pour ces retours et ces patches!

Mehdi Rande

> 
> 
> Cordialement,
> MARIADASSOU Romain
> 
> _______________________________________________
> Obm mailing list
> Obm at list.aliasource.fr
> http://www.list.aliasource.fr/mailman/listinfo/obm
-- 
Mehdi Rande
Aliasource - Groupe LINAGORA
20, rue Hermès, Parc Technologique du Canal 31520 RAMONVILLE SAINT AGNE
Téléphone : +33 (0)5 62 19 24 91