allowed_tags() more semantic

[dave@…]

• File: template-functions-general.php
• Function: allowed_tags()

The function allowed_tags() gereates a long line of tags which are allowed in comments.

function allowed_tags() {
	global $allowedtags;
	$allowed = '';
	foreach ( $allowedtags as $tag => $attributes ) {
		$allowed .= '<'.$tag;
		if ( 0 < count($attributes) ) {
			foreach ( $attributes as $attribute => $limits ) {
				$allowed .= ' '.$attribute.'=""';
			}
		}
		$allowed .= '> ';
	}
	return htmlentities($allowed);
}

This generates for example: <em> <i> <b> <strong> ...

But imo, this should be an unordered list, so i modified the function:

function allowed_tags() {
	global $allowedtags;
	$allowed = '';
	$pre = '<ul class="allowedtags">';
	$end = '</ul>';
	foreach ( $allowedtags as $tag => $attributes ) {
		$allowed .= '<li>&lt;'.$tag;
		if ( 0 < count($attributes) ) {
			foreach ( $attributes as $attribute => $limits ) {
				$allowed .= ' '.$attribute.'=""';
			}
		}
		$allowed .= '&gt;</li>';
	}
	return $pre.$allowed.$end;
}

Now, it generates this:

<ul class="allowedtags">

<li>&lt;em&gt;</li>

<li>&lt;i&gt;</li>

<li>&lt;strong&gt;</li>

<li>&lt;b&gt;</li>

</ul>

To make the whole thing html valid, change comments.php the allowed_tags part to:

	<p><strong>XHTML:</strong> You can use these tags:</p>
	<?php echo allowed_tags(); ?>

Greetings

[ryan]

I'm concerned about breaking themes that process the return value. Are there any such themes out there? We could always add an optional $format arg that allows theme authors to request a list.

[ptvguy]

Just for comparison, I've put the suggested mod on my test1 blog with the "Flashy" theme in place:

​http://test1.ptvguy.com/?p=1#comments

Here's what the same theme looks like normally:

​http://blog.leenarts.net/2006/06/30/naked-presentations/#comments

Review the allowed comment tags between the website input box and the comment input box.

[ptvguy]

BTW, I didnm't do the extra comments.php change.

[leftjustified]

Updated: because backwards compat. is a Good Thing :)

[leftjustified]

Patch modified to address Ryan's concerns.

[rob1n]

I think it's fine the way we do it, IMO. I think we include it in <code />, which is good enough. Devs?

[rob1n]

Not worth it, really. If you really wanted to, you could write your own function. Or loop through $allowedtags yourself.