Issues

ZF-10491: addDisplayGroup with variable

Issue Type: Improvement Created: 2010-09-23T02:38:24.000+0000 Last Updated: 2010-11-23T13:39:19.000+0000 Status: Resolved Fix version(s): - 1.11.1 (30/Nov/10)

Reporter: Martin Keckeis (thadafinser) Assignee: Richard Tuin (richardtuin) Tags: - Zend_Form

Related issues: - ZF-10731

Attachments: - ZF-10491-rtuin.patch

Description

Hello,

i use Zend_Form myself in this form:

$phone = new Zend_Form_Element_Text('phoneNr'); $phone->setLabel('phoneNr'); $phone->setFilters(array('StringTrim')); $phone->setValue($this->_data->phoneNr); $this->addElement($phone);

When i want to add such a form element to a display group, i have to type the "id" of the element, instead of the variable:

$this->addDisplayGroup(array('phoneNr'), 'name', ...);

Better would be: $this->addDisplayGroup(array($phone), 'name', ...);

Comments

Posted by Richard Tuin (richardtuin) on 2010-11-19T10:29:53.000+0000

I think it would make sense to implement this functionality. Precondition should be that the element defined in the variable is already added to the Zend_Form instance to be consistent with the current functionality.

I've uploaded a patch that enables this functionality, and a unit test.

Posted by Richard Tuin (richardtuin) on 2010-11-20T02:36:09.000+0000

[SVN:r23412:bittarman] ZF-10491: Applying patch from Richard Tuin (See: http://tinyurl.com/2853nos)

Posted by Ramon Henrique Ornelas (ramon) on 2010-11-21T05:23:02.000+0000

Merged to branch release 1.11 r23422.

Have you found an issue?

See the Overview section for more details.

Copyright

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

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

Contacts