ZF-11620: formRadio ignores doctype when selecting default list separator
Description
public function formRadio($name, $value = null, $attribs = null,
$options = null, $listsep = "
\n")
{
// ...
// done!
$xhtml .= implode($listsep, $list);
}
My proposal:
public function formRadio($name, $value = null, $attribs = null,
$options = null, $listsep = "
\n")
{
// ...
// XHTML or HTML for standard list separator?
if (!$this->_isXhtml() && false !== strpos($listsep, '
')) {
$listsep = str_replace('
', '
', $listsep);
}
// done!
$xhtml .= implode($listsep, $list);
return $xhtml;
}
Comments
Posted by Kai Uwe (kaiuwe) on 2011-07-28T14:24:18.000+0000
Fix and unit test added.
Posted by Adam Lundrigan (adamlundrigan) on 2011-11-07T15:39:49.000+0000
I've tested the patch by applying against trunk, and everything appears to be OK. Unit tests pass individually, and entire {{Zend_View_Helper}} suite passes as well.
Posted by Adam Lundrigan (adamlundrigan) on 2012-06-02T01:02:55.000+0000
Fixed in trunk (1.12.0): r24865