Created attachment 182581 [details] [review] Bug 39934: Only fetch attributes from 'Standard' Furthermore, this now only shows metadata present in _get_core_fields, so a list of attributes to ignore is not longer needed (they're not in _get_core_fields).
Created attachment 182582 [details] [review] Bug 39934: Use _get_core_fields as fields to be migrated Instead of ignoring from an arbitrary list
Created attachment 182583 [details] [review] Bug 39934: Add tests prove t/db_dependent/Koha/ILL/Backend/Standard.t Test plan, k-t-d, don't apply patches yet: 1) Enable ILLModule sys pref 2) Install an additional backend, e.g. ReprintsDesk: https://github.com/openfifth/koha-ill-reprintsdesk/releases/tag/v3.0.2 3) Restart plack $ koha-plack --restart kohadev 4) Create a new ReprintsDesk request, visit: <staff_url>/cgi-bin/koha/ill/ill-requests.pl?method=create&backend=ReprintsDesk 5) Enter DOI: '123', EISSN: 'abc', a cardnumber '42' and a library. Click 'Make request' 6) Click 'Switch provider'. Pick 'Standard'. 7) Notice 'Eissn' shows. This is confusing as it means nothing in the 'Standard' context. 'Migrated from' is also shown and is equally as confusing for staff members (This is a remnant of previous behavior when migrating a request would create a new request). 8) Apply patches. Repeat test plan. The attributes from the previous backend still exist in the database, but the UI now only shows attributes relevant to the current backend of the request. To confirm this, run: $ koha-mysql kohadev $ select * from illrequestattributes;