| Summary: | borrowers table should be better normalized | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Barton Chittenden <barton> |
| Component: | Architecture, internals, and plumbing | Assignee: | Galen Charlton <gmcharlt> |
| Status: | NEW --- | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | fridolin.somers |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
|
Description
Barton Chittenden
2016-08-31 14:14:43 UTC
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. *** |