Bug #13472 Updated: input type=hidden should be in a fieldset if there is one

ID: 13472 Updated by: sander Reported By: mat@dioxine.com Status: Open Old Bug Type: Feature/Change Request Bug Type: Session related Operating System: Any PHP Version: 4.0.6 New Comment: Reclassified back to session-related because Yasuo persuaded me to call it a bug ;) Previous Comments: ------------------------------------------------------------------------ [2001-12-05 13:22:53] mat@dioxine.com hum... not a bug ? PHP is not rewriting html code well, so I'd call it a bug :-) Anyway... any chance to get it fixed soon ? That shouldnt be /that/ hard to do, since you just have to write the input after the first fieldset if there is one, or jst after the form is there isnt any... ------------------------------------------------------------------------ [2001-12-05 10:22:58] sander@php.net Not a bug. Making it a feature request. ------------------------------------------------------------------------ [2001-09-28 04:12:48] mat@dioxine.com When enabling trans-id, php rewrites the forms to add an input type=hidden. the problem is that, in xhtml strict, inputs have to be in a fieldset element. when a fieldset element is already here, php still writes the input type=hidden before the fieldset, causing the page to be invalid html. reproducing is easy : <?xml version="1.0" encoding="iso-8859-1"?> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "DTD/xhtml1-strict.dtd"> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> <head> <title>PHP Bug with forms and sessions</title> </head> <body> <form action="blabla.php"><fieldset><input type="submit" value="test" /></fieldset></form></body></html> its valid html, but php will, with transid, make it invalid by putting this input between the form and the fieldset... ------------------------------------------------------------------------ Edit this bug report at http://bugs.php.net/?id=13472&edit=1