Re: API-Addition

On Wed, 2005-04-20 at 03:13, Martin Preuss wrote:
> On Wednesday 20 April 2005 11:38, Martin Preuss wrote:
> > Therefore I propose to add some fields to OfxAccountData which represent at
> > least the account number (we could call it just that: "account_number").
> > We should also add a field "bank_id" and maybe "branch_id".
> And while we're at it: Maybe we should add a field for the brokerId, too.
>
> I see no reason why we shouldn't make all valuable information from the OFX 
> file available to applications... 

Oh, right, I see.  You're not talking about changing the OfxAccountInfo
struct, right?  This is the OfxAccountData struct that comes BACK.  Yes,
I agree with you, it should match the actual data found in the OFX file,
not be some amalgamation thereof that you have to parse out yourself.

We should call the fields EXACTLY what the are called in OFX, IMO.  So
that's account id, not number.

However, we also need to consider compatibility.  Existing users of
libofx use the fields as they are now.  So I would say leave name and id
alone.  Maybe add branch, broker, and bank id, and then add a "RAW
account ID"?

While you're at it, you could move AccountType out of OfxAccountData for
me if you want.

</Ace>



-------------------------------------------------------
This SF.Net email is sponsored by: New Crystal Reports XI.
Version 11 adds new functionality designed to reduce time involved in
creating, integrating, and deploying reporting solutions. Free runtime info,
new features, or free trial, at: http://www.businessobjects.com/devxi/728