I've always been bothered by the fact that we have 3 separate sets of contact information in the borrowers table: streetnumber streettype address address2 city state zipcode country email phone B_streetnumber B_streettype B_address B_address2 B_city B_state B_zipcode B_country B_email B_phone altstreetnumber altstreettype altaddress altaddress2 altcity altstate altzipcode altcountry altemail altphone These should be moved to a contacts table, which contains a 'contact_type' field corresponding to 'primary', 'B_' or 'alt' (although these also desperately need to be renamed), as well as a link to the borrower and a timestamp.
Nice idea, more relational database indeed
(In reply to Barton Chittenden from comment #0) > I've always been bothered by the fact that we have 3 separate sets of > contact information in the borrowers table: > > streetnumber > streettype > address > address2 > city > state > zipcode > country > email > phone > > B_streetnumber > B_streettype > B_address > B_address2 > B_city > B_state > B_zipcode > B_country > B_email > B_phone > > altstreetnumber > altstreettype > altaddress > altaddress2 > altcity > altstate > altzipcode > altcountry > altemail > altphone > > These should be moved to a contacts table, which contains a 'contact_type' > field corresponding to 'primary', 'B_' or 'alt' (although these also > desperately need to be renamed), as well as a link to the borrower and a > timestamp. alt* is actually altcontact*. Other borrower contact fields: mobile fax emailpro phonepro smsalertnumber sms_provider_id Contact fields from other tables: aqbooksellers: address1 address2 address3 address4 phone booksellerfax bookselleremail booksellerurl postal url fax branches: branchaddress1 branchaddress2 branchaddress3 branchzip branchcity branchstate branchcountry branchphone branchfax branchemail branchreplyto branchreturnpath branchurl geolocation (?) This will require us to use a contact_id and some linking tables (borrower_contact_link, branch_contact_link, aqbooksellers_contact_link)
*** Bug 18218 has been marked as a duplicate of this bug. ***
Bug 18218 suggested to remodel the aqcontacts table for this.