NinjaForge/Ninjaboard — Issue #39: Forum Emails Strip Out BBCode - 1.1 Rev 2150
Author: stipsan
Created: 2011-12-29 17:04:22+00:00
State: open
Labels: pending audit

Reported by @CaptainH00K:

This is something that has bothered me for some time but i keep forgetting to make a ticket for! I was reminded to do it today when snooping at/stealing some code from Ninjaboard and saw why it happens.

Is it intended to work this way? It can be confusing to read the message in an email especially containing quotes.

I think it's because in /components/com_ninjaboard/controllers/watch.php on line 113 we have the text being parsed for BBCode which is great, but then on line 120 it's all stripped away with KFactory::get('lib.koowa.filter.string')->sanitize($text).

Perhaps an option for plain text emails versus Html ones if some would rather not have BBCode in emails? Will make this a feature release ticket just incase.

(p.s When i make a request for a feature that is "optionally" set by the user, i'm thinking in the new "config" way that will eventually happen. I don't mean to clutter up the admin section with more and more options. ;) )