[freeside-devel] RE: [freeside] Freeside draft checks scheme changes

ivan ivan at 420.am
Tue Nov 6 09:50:44 PST 2001


On Tue, Nov 06, 2001 at 11:33:43AM -0600, Jeff Foster wrote:
> 
> 
> From: ivan [mailto:ivan at 420.am]
> Sent: Tuesday, November 06, 2001 11:07 AM
> 
> > On Tue, Nov 06, 2001 at 10:45:36AM -0600, Jeff Foster wrote:
> > > 
> > > From: ivan [mailto:ivan at 420.am]
> > > Sent: Tuesday, November 06, 2001 10:37 AM
> > > 
> > > > Use the `payinfo' field for the routing number and account number, or
> > > > convince me why that's not appropriate.
> > > 
> > > Thought of that and started that way first. I would join the
> > > routing number and account number with a ':' separater. But I 
> > > didn't want to make too much work for myself and thought that
> > > just adding the new fields would be easier.
> > 
> > Schema changes are a pain and I probably tend toward minimizing them when
> > possible.  I don't think I'm convinced that it's any significant extra
> > work to just put them in the payinfo field; it's not called "cardnumber"
> > for a reason.
> >
> 
> Agreed. However the payinfo is only 16 characters and this isn't big enough,
> the routing number is 9 digits by its self. The paydate and payname fields
> could be re-used, if you don't mind column names that don't quite fit the
> actual data. 

It's too short?  Hmm.  Okay, I'm convinced, adding the new fields is the
best solution.  Changing an existing field is more of a pain.

I agree that reusing the paydate or payname fields for something else
entirely is a bad idea. 

> > > > I could maybe see adding the `checknum' field.  Tell me more about how
> > > > it's supposed to work; does the customer give you a check number in
> some
> > > > starting range and you incremement it by one each time or what?
> > > 
> > > Yes that is correct. When the billing is done the current check
> > > number is sent to the billing program, the value is incremented,
> > > and the cust_main entry is updated.
> > 
> > Okay.  How about an integer instead of a varchar?
> 
> Good idea, if the field is added.

I look forward to your patch :)

-- 
_ivan



More information about the freeside-devel mailing list