Re: [PHP4BETA] Small patch to string.c

On Sun, 19 Mar 2000 rasmus@php.net wrote: > > While this isn't extensively tested, this patch modifies strip_tags to > > convert > characters encountered in state 0 (the output state) into the > > string "&gt;". Whether or not this is entirely useful is another story. > > Hrm.. This doesn't seem like a good idea to me. If you want to convert < > to &lt; feed the output of strip_tags() through htmlspecialchars() I don't want to convert < to &lt; (otherwise I would just use htmlspecialchars like you suggest), I want to convert the > character to &gt; under certain circumstances (i.e. when it isn't part of a tag) Right now strip_tags removes these characters from its output. > strip_tags() takes an optional argument of allowable tags. Your patch > would render this arg useless. I just tested this and it seems to work fine: I passed the following input through strip_tags using "<a><b>" as the "Allowable tags" argument. <a href="http://www.knowpost.com/">KnowPost</a> <b>This is bold</b> > this has a carat in front of it. <i>this isn't italics</i> And it exhibits the correct behavior. I'm not saying this is necessarily desirable behavior, I added it because I need "lone" > characters to appear. Peter -------------------------------------------------- Peter D. Kovacs KnowPost.com LLC Lead Engineer peter@knowpost.com --------------------------------------------------