#27361 [Opn->Csd]: strip_tags() doesn't fail on incomplete tag if there is non-html output before.

ID: 27361 Updated by: sean@php.net Reported By: stu-php at better dot domain dot name -Status: Open +Status: Closed Bug Type: Documentation problem Operating System: Windows Server 2003 PHP Version: 5.0.0b4 (beta4) -Assigned To: +Assigned To: sean New Comment: This bug has been fixed in the documentation's XML sources. Since the online and downloadable versions of the documentation need some time to get updated, we would like to ask you to be a bit patient. Thank you for the report, and for helping us make our documentation better. removed "It errors on the side of caution in case of incomplete or bogus tags." and added a warning: Because strip_tags does not actually validate the HTML, partial, or broken tags can result in unwanted results. Previous Comments: ------------------------------------------------------------------------ [2004-02-23 18:06:13] iliaa@php.net The fact that strip_tags() does not validate HTML and can strip excess data when encountering incomplete tags should be documented. ------------------------------------------------------------------------ [2004-02-23 16:57:12] stu-php at better dot domain dot name In one line, ok. strip_tags() doesn't error in the way it says it should on the function documentation page. That should be clear and concise enough. Take a look at http://better.domain.name/php/ and tell me if the files there represent expected outcomes of strip_tags() usage. ------------------------------------------------------------------------ [2004-02-23 16:55:25] stu-php at better dot domain dot name The function returns an error if there is text in one place, but NOT if there is text in another set of places, or nowhere at all. How is that NOT a bug? If I were relying on this function to give me accurate error information in an if statement or something then I wouldn't want it erroring if and when it felt like it. If it's supposed to strip html and error on invalid tags and then it goes ahead and fails to error because there is no text before the broken tag then there's a problem. Consider: http://better.domain.name/php/strips.phps The output has to be seen by viewing the source of http://better.domain.name/php/strips.php as it's completely erroneous output at the moment. Is that the expected behaviour? ------------------------------------------------------------------------ [2004-02-23 16:39:53] derick@php.net Explain in one line what the bug is then...I don't see it either. ------------------------------------------------------------------------ [2004-02-23 16:36:36] stu-php at better dot domain dot name This is NOT a bogus bug. Just because the MAIN functionality of the function is there does not mean that there is not a bug here. The function behaves in an unexpected manner. That is a bug. If the documentation contradicts the actual function then something needs to be fixed. Either the documentation or the function code. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/27361 -- Edit this bug report at http://bugs.php.net/?id=27361&edit=1