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:

Description

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.

Comments

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