Bug #12252 Updated: URL rewriter outputs a spurious " /"

From: Date: Thu, 19 Jul 2001 14:37:54 +0000
Subject: Bug #12252 Updated: URL rewriter outputs a spurious " /"
References: 1  Groups: php.dev 
Request: Send a blank email to php-dev+get-60302@lists.php.net to get a copy of this message
ID: 12252 Updated by: swm Reported By: dans@free.fr Status: Open Bug Type: Session related Operating System: linux suse 6.4 PHP Version: 4.0.6 New Comment: I don't think it is. Here is the code: static inline void handle_form(STD_PARA) { if (ctx->tag.len == 4 && strncasecmp(ctx->tag.c, "form", 4) == 0) { smart_str_appends(&ctx->result, "<input type=\"hidden\" name=\""); smart_str_append(&ctx->result, &ctx->q_name); smart_str_appends(&ctx->result, "\" value=\""); smart_str_append(&ctx->result, &ctx->q_value); smart_str_appends(&ctx->result, "\" />"); } } You'll notice that it is not a <form> tag we are adding. Rather, it is <input/> Previous Comments: ------------------------------------------------------------------------ [2001-07-19 10:34:02] cynic@php.net actually that _is_ a bug. form is _not_ an empty element. ------------------------------------------------------------------------ [2001-07-19 10:08:38] swm@php.net This is not a spurious slash. This slash is present to make PHP XHTML 1.0 compliant. See more at: http://www.w3.org/TR/xhtml1/ swm ------------------------------------------------------------------------ [2001-07-19 09:29:26] dans@free.fr the rewriter adds a " /" befor the closing bracket in the <form> rewriting. fallowing a diff to patch it. sincerely Daniel S. *** ext/standard/url_scanner_ex.c Thu Jun 21 08:29:04 2001 --- ext/standard/url_scanner_ex_modif.c Thu Jul 19 14:14:34 2001 *************** static inline void handle_form(STD_PARA) *** 178,182 **** smart_str_appends(&ctx->result, "\" value=\""); smart_str_append(&ctx->result, &ctx->q_value); ! smart_str_appends(&ctx->result, "\" />"); } } --- 178,182 ---- smart_str_appends(&ctx->result, "\" value=\""); smart_str_append(&ctx->result, &ctx->q_value); ! smart_str_appends(&ctx->result, "\">"); } } ------------------------------------------------------------------------ Edit this bug report at http://bugs.php.net/?id=12252&edit=1

« previous php.dev (#60302) next »