Proposal for Abstraction Layer for Modifications in Bug.pm

Maxwell Kanat-Alexander mkanat at kerio.com
Tue Jul 6 19:01:04 UTC 2004


> > 3. set_defaults_for_missing_fields() (or some shorter name) that sets default
> >    values for things like assigned_to if they have not been specified in the
> >    form.
> > 4. commit() - updates the database, i.e. allocates a new bug number and writes
> >    everything into the correct tables.

	Oh, one other comment -- this structure depends on a "the programmer
must do this" sort of rule. That is, "You must always call Y and then
Z." Designs that involve this tend to cause problems down the road,
since not all programmers read API docs. At least, not until they get an
error. :-)

	Either make sure it's enforced programmatically, or just roll step 3
into step 4.

	-Max
-- 
Maxwell Kanat-Alexander
2nd Level Tech Support Engineer, USA
Kerio Technologies, Inc.
2041 Mission College Blvd. Suite 100
Santa Clara, CA 95054
Phone: (408) 496-4500
Fax: (408) 496-6902
Web: http://www.kerio.com/support.html





More information about the developers mailing list