FusionForge de l'ADULLACT: openMairie - Framework : [#8387] Upload - Récupération incorrecte du type de fichier sous Firefox

Date :
24/06/2016 17:30

Priorité :
3

État :
Closed

Proposé par :
Nicolas Meucci (nmeucci)

Confié à :
Nicolas Meucci (nmeucci)

Sévérité :
normal

Version cible :
none

Version concernée :
none

Résumé :
Upload - Récupération incorrecte du type de fichier sous Firefox

Description détaillée
Le framework récupère le type du fichier par la variable superglobale $_FILES. C'est le navigateur qui poste cette information en HTTP (cf. doc PHP). Or Firefox gère mal le type PDF. Le bug a été signalé en 2007 mais jamais résolu : https://bugzilla.mozilla.org/show_bug.cgi?id=373621 Ainsi lorsque l'on souhaite accéder à un document PDF déposé via Firefox, openMairie nous propose uniquement de le télécharger. S'il était reconnu comme tel on aurait la possibilité de le lire dans la lecteur intégré (visionneuse native de Firefox). Solution proposée, dans la classe Upload du fichier /core/upload.class.php : 1) Créer une nouvelle fonction qui récupère le type de fichier pour un chemin donné function getFileType($path) { $finfo = new finfo(FILEINFO_MIME); preg_match('/(.*);/', $finfo->file($path), $type); if (isset($type[1]) === true) { return $this->_type = $type[1]; } return $this->_type = $type[0]; } 2) L'appeler dans la méthode CheckUpload() - remplacer : $this->_type = $_FILES['userfile']['type'][$i]; - par : $this->_type = $this->getFileType($this->_temp);
Le framework récupère le type du fichier par la variable superglobale $_FILES. C'est le navigateur qui poste cette information en HTTP (cf. doc PHP). Or Firefox gère mal le type PDF. Le bug a été signalé en 2007 mais jamais résolu : https://bugzilla.mozilla.org/show_bug.cgi?id=373621 Ainsi lorsque l'on souhaite accéder à un document PDF déposé via Firefox, openMairie nous propose uniquement de le télécharger. S'il était reconnu comme tel on aurait la possibilité de le lire dans la lecteur intégré (visionneuse native de Firefox). Solution proposée, dans la classe Upload du fichier /core/upload.class.php : 1) Créer une nouvelle fonction qui récupère le type de fichier pour un chemin donné function getFileType($path) { $finfo = new finfo(FILEINFO_MIME); preg_match('/(.*);/', $finfo->file($path), $type); if (isset($type[1]) === true) { return $this->_type = $type[1]; } return $this->_type = $type[0]; } 2) L'appeler dans la méthode CheckUpload() - remplacer : $this->_type = $_FILES['userfile']['type'][$i]; - par : $this->_type = $this->getFileType($this->_temp);

Commentaires (1)
Pièces jointes
Modifications (3)

Message
Date : 28/07/2016 17:38 Expéditeur : Sofien TIMEZOUAGHT Bonjour, Désormais le mimetype des fichiers sous le navigateur Firefox est correctement récupéré grâce à la classe finfo. Les documents PDF pourront être consultés depuis le navigateur pour plus de confort grâce à l'action de visualisation. Liste des commits de correction : - https://adullact.net/scm/viewvc.php?revision=3529&root=openmairie&view=revision - https://adullact.net/scm/viewvc.php?revision=3531&root=openmairie&view=revision - https://adullact.net/scm/viewvc.php?revision=3533&root=openmairie&view=revision Cordialement. -- Sofien TIMEZOUAGHT http://www.atreal.fr

Pas de documents joints

Champ	Ancienne valeur	Date	Par
status_id	Open	28/07/2016 17:38	Sofien TIMEZOUAGHT
close_date	Aucun(e)	28/07/2016 17:38	Sofien TIMEZOUAGHT
summary	Upload - Récuparation incorrecte du type de fichier sous Firefox	24/06/2016 17:53	Nicolas Meucci

Powered By FusionForge Collaborative Development Environment

Charte d'utilisation / Nous contacter / Mentions légales

Haut de page