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

[Romain MARIADASSOU]

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.'"');


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);


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 :
  else {
   $content.= "
     
    <td class=\"documentFile\" style=\"text-align:left;\">
     <a  
href=\"document_index.php?action=detailconsult&amp;document_id=$id\" >

      <img src=\"$ico_file\" alt=\"\" />
     </a>
     <a class=\"documentFile\"
      
href=\"document_index.php?action=accessfile&amp;document_id=$id&amp;popup=1\" 

      target=\"_blank\" >
      ".$info["name"]."
     </a>
  }";
  }

Par :
  else {
   $content.= "
     
    <td class=\"documentFile\" style=\"text-align:left;\">
     <a  
href=\"document_index.php?action=detailconsult&amp;document_id=$id\" >

      <img src=\"$ico_file\" alt=\"\" />
     </a>
     <a class=\"documentFile\"
      
href=\"document_index.php?action=accessfile&amp;document_id=$id&amp;popup=1\" 

      target=\"_blank\" >
      ".$info["name"]."
     </a>
  ";
  }


Cordialement,
MARIADASSOU Romain