ZF-9689: Zend_Form_Decorator_ViewHelper does not forward listseparator to viewhelper

Issue Type: Bug Created: 2010-04-16T02:46:15.000+0000 Last Updated: 2012-06-02T02:54:59.000+0000 Status: Resolved Fix version(s): - 1.12.0 (27/Aug/12)

Reporter: Marcel Koonstra (abstractor) Assignee: Frank Brückner (frosch) Tags: - Zend_Form

  • FixForZF1.12
  • zf-crteam-priority
  • zf-crteam-review

Related issues: Attachments: - ViewHelper.php.patch


The Zend_Form_Decorator_ViewHelper does not forward the $listsep option to Radio or MultiCheckbox fields.

On line 246 it reads: $elementContent = $view->$helper($name, $value, $attribs, $element->options);

which should be: $listsep = $attribs['listsep']; unset($attribs['listsep']);

$elementContent = $view->$helper($name, $value, $attribs, $element->options, $listsep);

The listsep is gathered in $attribs in Zend_Form_Decorator_ViewHelper::getElementAttribs() on lines 156-160, but nothing is done with it. Instead it just ends up as an attribute in the tag.


Posted by Frank Brückner (frosch) on 2012-03-12T21:29:56.000+0000

Patch and unit tests added.

Posted by Adam Lundrigan (adamlundrigan) on 2012-06-02T02:54:59.000+0000

Fixed in trunk (1.12.0): r24873

Have you found an issue?

See the Overview section for more details.


© 2006-2018 by Zend, a Rogue Wave Company. Made with by awesome contributors.

This website is built using zend-expressive and it runs on PHP 7.