https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42842
--- Comment #1 from Tomás Cohen Arazi (tcohen) <[email protected]> --- Created attachment 200388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200388&action=edit Bug 42842: Remove materialized patron data from tmp_holdsqueue Since bug 42503, the holds queue page is API-driven and patron data is embedded via the borrowernumber FK. The materialized surname, firstname, phone, and cardnumber columns are no longer read by any code path. Removing them because: - Dead code: nothing reads these columns - Privacy: patron PII was stored in an unaudited table not covered by GDPR/anonymization cleanup tools - Data integrity: values could become stale between queue rebuilds Changes: - Atomicupdate drops the four columns - C4::HoldsQueue::CreatePicklistFromItemMap no longer fetches/inserts patron fields - Koha::Hold::HoldsQueueItem to_api_mapping cleaned up - Tests adjusted to use borrowernumber instead of cardnumber Test plan: 1. Apply all patches 2. Run updatedatabase 3. Verify tmp_holdsqueue no longer has surname/firstname/phone/cardnumber 4. Run: perl misc/cronjobs/holds/build_holds_queue.pl 5. Verify the holds queue page still shows patron info (via API embed) 6. prove t/db_dependent/HoldsQueue/materialized_descriptions.t 7. prove t/db_dependent/api/v1/holds_queue.t -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
