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

Bug #4562 Request to change group elements like dates to use addGroupRule
Submitted: 2005-06-09 14:10 UTC
From: danielr at electroteque dot org Assigned: justinpatrin
Status: Closed Package: DB_DataObject_FormBuilder
PHP Version: 5.0.3 OS: N/A
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 26 + 27 = ?

 
 [2005-06-09 14:10 UTC] danielr at electroteque dot org
Description: ------------ Ok this is maybe a feature request more than a bug, but it might still be a bug. I made a bug request to QuickForms except, it looks like dates and group elements need to use addGroupRule instead of addRule here is the link http://pear.php.net/bugs/bug.php?id=4495&edit=2

Comments

 [2005-08-02 17:14 UTC] justinpatrin
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. Please check the code in CVS to make sure it fixes this problem for you.
 [2005-08-02 17:37 UTC] justinpatrin
Unfortunately, the change I made broke the rules for some other elements. In some cases you want the rule to be a normal rule as the element you're assing the rule for it within the group. Are you sure that addGroupRule fixes this?
 [2005-08-02 18:52 UTC] danielr at electroteque dot org
I suppose so ? This is what the QuickForm guy said, I guess I can look more into it when I get back from holidays.
 [2005-08-22 04:28 UTC] justinpatrin
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. I've changed my previous code to use addGroupRule only for dates. Please test the code in CVS and let me know if it fixes the problem for you.