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

AlexisWilke’s picture

Status: Active » Closed (works as designed)

This 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