PEAR is archived and read-only

This mirror preserves historical PEAR package releases and metadata so existing references remain available.

Home » Database » DB_DataObject_FormBuilder » Bug #4450

Request for fieldsRequired and a change to how options are loaded with diff

Details

Request #4450Request for fieldsRequired and a change to how options are loaded with diff
Submitted2005-05-27 03:01 UTC
Fromdanielr at electroteque dot org
Assignedjustinpatrin
StatusClosed
PackageDB_DataObject_FormBuilder
PHP Version5.0.3
OSN/A
Roadmaps(Not assigned)

Comments

[2005-05-27 03:01 UTC] danielr at electroteque dot org

Description:
------------
I have created two patches to add functionality to add
extra fields required say if a date field is null but
its still required to be selected. I have also made a
change to how the options are loaded, in particular the
options array to the create method is loaded after the
DO variables therefore we can override particular
settings on a per page basis rather than a global basis.
This means we can exclude fields from being required to
__ALL__ for one page only rather than every single page.

Handy for search pages where you dont usually need any
fields entered and let the user select their filter.

Diff is here www.electroteque.org/pear/FormBuilder.diff
please let me know if its correct I have checkedout the
latest CVS file and made my changes to it.

[2005-05-27 06:03 UTC] danielr at electroteque dot org

That has now just completely broken my system

Cannot redeclare DB_DataObject::$fieldsRequired

[2005-05-27 06:17 UTC] danielr at electroteque dot org

I uncommented the second declaration, however around
936, there is cvs patch data, maybe its my cvs checking
out wierdly ? Anyway i fixed these and works fine.
excludeFromAutoRules=>'__ALL__' works in the options to
create. $this->do->fb_fieldsRequired = array('thedate');
now works too.