Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.2

Request #5992 Support for defining field label as array
Submitted: 2005-11-17 07:12 UTC
From: alex at cgi-central dot net Assigned:
Status: Open Package: DB_DataObject_FormBuilder
PHP Version: 4.3.11 OS: any
Roadmaps: (Not assigned)    
Subscription  
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes. If this is not your bug, you can add a comment by following this link. If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: alex at cgi-central dot net
New email:
PHP Version: Package Version: OS:

 

 [2005-11-17 07:12 UTC] alex at cgi-central dot net
Description: ------------ HTML_QuickForm supports defining field labels as array. Have a look to pear/docs/HTML_QuickForm/docs/renderers/multiple-labels.php Howevr, when you define field label as array in $fb_FieldLabels array, it is handled incorrectly ("Array" is passed instead of field title to addRule function). Below patch will fix this situation. I hope this is self-explanatory Test script: --------------- diff -ur DataObject-orig/FormBuilder/QuickForm.php DataObject/FormBuilder/QuickForm.php --- DataObject-orig/FormBuilder/QuickForm.php Thu Oct 20 13:01:36 2005 +++ DataObject/FormBuilder/QuickForm.php Mon Oct 24 10:21:33 2005 @@ -787,13 +787,13 @@ } if ($rule['rule'] === false) { $this->_form->$ruleFunction($realFieldName, - sprintf($rule['message'], $fieldLabel), + sprintf($rule['message'], is_array($fieldLabel)? $fieldLabel[0] : $fieldLabel), $rule['validator'], '', $ruleSide); } else { $this->_form->$ruleFunction($realFieldName, - sprintf($rule['message'], $fieldLabel), + sprintf($rule['message'], is_array($fieldLabel)? $fieldLabel[0] : $fieldLabel), $rule['validator'], $rule['rule'], $ruleSide);

Comments