Zend Framework

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.0
• Fix Version/s: 1.10.6
• Component/s: Zend_Form
• Labels:

  None

When using multiple decorators of the same type (a specific name is given to each of them, as described in the documentation), getting decorators and setting them back on the object make all except one of them disappear. It occurs for Zend_Form, Zend_Form_DisplayGroup and Zend_Form_Element instances.

An example with a form element :

$element = new Zend_Form_Element_Text('foo');
$element->addDecorator(array('TdTag' => 'HtmlTag'), array('tag' => 'td'));
$element->addDecorator(array('TrTag' => 'HtmlTag'), array('tag' => 'tr'));

var_export($element->getDecorators());

$element->setDecorators($element->getDecorators()); // This should do nothing
var_export($element->getDecorators());

The first var_export will show the following HtmlTag decorators :

• HtmlTag : the default one which renders the element surrounded with "td"
• TrTag
• TdTag

At the second var_export, only the TrTag (the last HtmlTag decorator) remains among the HtmlTag decorators (and furthermore, it has lost its name, getting a mere 'Zend_Form_Decorator_HtmlTag' name).

This is due to the fact that keys of the array provided to the addDecorators method are not used.

Proposed patch :

Index: Zend/Form.php
===================================================================
--- Zend/Form.php	(revision 17501)
+++ Zend/Form.php	(working copy)
@@ -2394,11 +2394,11 @@
      */
     public function addDecorators(array $decorators)
     {
-        foreach ($decorators as $decoratorInfo) {
+        foreach ($decorators as $decoratorName => $decoratorInfo) {
             if (is_string($decoratorInfo)) {
                 $this->addDecorator($decoratorInfo);
             } elseif ($decoratorInfo instanceof Zend_Form_Decorator_Interface) {
-                $this->addDecorator($decoratorInfo);
+                $this->addDecorator(array($decoratorName => $decoratorInfo));
             } elseif (is_array($decoratorInfo)) {
                 $argc    = count($decoratorInfo);
                 $options = array();
Index: Zend/Form/Element.php
===================================================================
--- Zend/Form/Element.php	(revision 17501)
+++ Zend/Form/Element.php	(working copy)
@@ -1761,11 +1763,11 @@
      */
     public function addDecorators(array $decorators)
     {
-        foreach ($decorators as $decoratorInfo) {
+        foreach ($decorators as $decoratorName => $decoratorInfo) {
             if (is_string($decoratorInfo)) {
                 $this->addDecorator($decoratorInfo);
             } elseif ($decoratorInfo instanceof Zend_Form_Decorator_Interface) {
-                $this->addDecorator($decoratorInfo);
+                $this->addDecorator(array($decoratorName => $decoratorInfo));
             } elseif (is_array($decoratorInfo)) {
                 $argc    = count($decoratorInfo);
                 $options = array();
Index: Zend/Form/DisplayGroup.php
===================================================================
--- Zend/Form/DisplayGroup.php	(revision 17501)
+++ Zend/Form/DisplayGroup.php	(working copy)
@@ -702,11 +702,11 @@
      */
     public function addDecorators(array $decorators)
     {
-        foreach ($decorators as $decoratorInfo) {
+        foreach ($decorators as $decoratorName => $decoratorInfo) {
             if (is_string($decoratorInfo)) {
                 $this->addDecorator($decoratorInfo);
             } elseif ($decoratorInfo instanceof Zend_Form_Decorator_Interface) {
-                $this->addDecorator($decoratorInfo);
+                $this->addDecorator(array($decoratorName => $decoratorInfo));
             } elseif (is_array($decoratorInfo)) {
                 $argc    = count($decoratorInfo);
                 $options = array();

1. 

   ZF7552.patch

   25/Jan/10 6:12 AM

   3 kB

   Hendri Smit

• Assignee:

  Christian Albrecht

  Reporter:

  Michaël Perrin

• Created:

  10/Aug/09 8:54 AM

  Updated:

  19/Jun/10 10:42 AM

  Resolved:

  19/Jun/10 10:42 AM