FluxBB.fr

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

[Mod] FluxBB File Uploader

Bonjour

Un petit mot pour vous dire que j'ai créé et ajouté un plugin sur fluxbb.org permettant d'uploder des fichiers sur son forum. Tout en étant parfaitement intégré au forum, ce plugin ne modifie pas les fichiers de FluxBB, ce qui est très  appréciable. Après upload du fichier ou de l'image, un BBCode est affiché pour intégrer le fichier dans un message.

Une table est créée dans la BDD, elle ne sert pas pour le moment mais est complétée en vue de l'ajouter de futures fonctionnalités. Compatible avec FluxBB 1.4 et 1.5 (non testé avec 1.2).

Vous pouvez télécharger ce module ici : http://fluxbb.org/resources/mods/fluxbb-file-uploader/

N'hésitez pas à me faire part de vos remarques, suggestions ou bugs sur ce sujet.

Dernière modification par MasterGone (09-11-2013 13:51:46)

adaur

Membre

Lieu : Derrière Notepad ++ !

Inscription : 25-09-2008

Messages : 1 937

Site Web

Re : [Mod] FluxBB File Uploader

Salut,

Ne faudrait il pas protéger la variable $upload_file_name dans la requête?

---

"As code is more often read than written it's really important to write clean code." - Franz
Twitter: @adaurfr
Mon fork de FluxBB: FeatherBB

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

Re : [Mod] FluxBB File Uploader

Euh oui, j'ai zappé le $db->escape, je corrige ça

Merci pour l'avoir vu

adaur

Membre

Lieu : Derrière Notepad ++ !

Inscription : 25-09-2008

Messages : 1 937

Site Web

Re : [Mod] FluxBB File Uploader

De rien!

Par contre, je crois aussi qu'il faut que tu échappes le file_dest s'il est est comme le file/name (en fait j'ai pas super bien compris comment tu le gères) et que tu appliques des pun_htmlspecialchars à l'affichage:

	// On crée le BBCode à afficher
	if (in_array($upload_file_ext, $upload_ext_img))
		$upload_file_bbcode = '[img='.$upload_file_name.']'.$pun_config['o_base_url'].'/'.UPLOAD_ROOT.$upload_file_dest.'[/img]';
	else
		$upload_file_bbcode = '[url='.$pun_config['o_base_url'].'/'.UPLOAD_ROOT.$upload_file_dest.']'.$upload_file_name.'[/url]';

Tant qu'à faire, remplace aussi les $pun_config['o_base_url'] par get_base_url()

---

"As code is more often read than written it's really important to write clean code." - Franz
Twitter: @adaurfr
Mon fork de FluxBB: FeatherBB

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

Re : [Mod] FluxBB File Uploader

Je suis parti du (mauvais en fait) principe d'afficher le nom originel du fichier dans ce paramètre (et non le fichier renommé). A la base, je ne voulais rien mettre, mais pour rester dans le cadre du XHTML Strict, je l'ai paramétré comme ça. Je n'ai pas vérifié comment FluxBB gère ce paramètre s'il est absent.

Pour ce qui est de l'échappement, la aussi vulgaire oubli de ma part... Il faut aussi échapper la variable $upload_file_dest dans le cas où le renommage n'est pas activé. Je corrigerai ça à la prochaine mise à jour.

Dernière modification par MasterGone (09-11-2013 19:26:13)

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

Re : [Mod] FluxBB File Uploader

Mise à jour du plugin effectuée (1.0.2). Ajout de contrôles supplémentaires sur les fichiers en plus des corrections mentionnées ci-dessus.

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

Re : [Mod] FluxBB File Uploader

Je viens de faire une mise à jour (1.1.0). Chaque utilisateur peut désormais visualiser et gérer ses fichiers.

Dernière modification par MasterGone (26-11-2013 00:17:30)

sc4t

Membre

Inscription : 21-11-2013

Messages : 14

Re : [Mod] FluxBB File Uploader

Bonjour!

Pourquoi ne pas l'intégrer directement sous la fenêtre de composition des messages ??

MasterGone

Membre

Inscription : 09-11-2013

Messages : 8

Re : [Mod] FluxBB File Uploader

Bonsoir

Le but est de ne pas modifier les fichiers sources de FluxBB.

++