Closed (works as designed)
Project:
Table of Contents
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
7 Mar 2011 at 04:05 UTC
Updated:
7 Mar 2011 at 04:23 UTC
I recommend changing this:
define('TABLEOFCONTENTS_ALLOWED_TAGS', '<em> <i> <strong> <b> <u> <del> <ins> <sub> <sup> <cite> <strike> <s> <tt> <span> <font> <abbr> <acronym> <dfn> <q> <bdo> <big> <small>');
And moving this into variable_get();
The idea would be to remove configurables out of the code base. I see this can be altered later, in the theme layer but this approach would be even more flexible.
Comments
Comment #1
AlexisWilke commentedThis is the default used on a couple of variable_get() calls. I don't see the point in adding yet another variable. The reason why I used a define() is because it appears in two places and if I were to change the defaults, it would need to occur in both places. This way it will. Automatically.
Thank you.
Alexis Wilke