The original installation notes created by Alex for ILL (https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7317#c20) describe a requirement to create a new patron category with a code ILLLIBS, organisational patrons are then attached to this category and their email address is used for the "Place request with partners" function. However, Koha ships with a patron category called "Inter-Library Loans" (code: IL) which would appear to remove the requirement for an ILLLIBS category to be created. This bug proposes moving away from using ILLLIBS and using the pre-existing IL category instead. Grepping the code for ILLLIBS reveals that it's assumed to exist and will be used as a default code if one is not specified in the <partner_code> element in koha-conf.xml, so modifications will be required.
Hi Andrew, I think using the existing value from the sample data makes sense. But could we make the code configurable? Preferrably moving the current setting out of the conf file and into a pref?
That would certainly make sense Katrin, there's quite a bit of the ILL config that would benefit from moving out of koha-conf.xml into sysprefs.
Bug 19822 suggests to shift the ILL config from koha-conf.xml to sysprefs.
Move to pref and change default to IL are still both great ideas.
Created attachment 146333 [details] [review] 21548: ILLLIBS patron category / partner_code seems extraneous This patch updates the default partner category used by the partner_code config to be in line with sample data in sample_patrons.yml Preparation: Apply patch Enable ILLModule sys pref Install an ILL backend (e.g. FreeForm) Add this change to your koha-conf.xml Flush, restart. Search for patron of category inter-library loan and assign a primary e-mail address to it Test plan: Create an ILL request and click 'place request with partners' Verify that the 'select partner libraries' has the correct patron of IL category Run tests and ensure they pass: prove t/db_dependent/Illrequest/Config.t prove t/Koha/Config.t
Created attachment 147416 [details] [review] Bug 21548: ILLLIBS patron category / partner_code seems extraneous This patch updates the default partner category used by the partner_code config to be in line with sample data in sample_patrons.yml Preparation: Apply patch Enable ILLModule sys pref Install an ILL backend (e.g. FreeForm) Add this change to your koha-conf.xml Flush, restart. Search for patron of category inter-library loan and assign a primary e-mail address to it Test plan: Create an ILL request and click 'place request with partners' Verify that the 'select partner libraries' has the correct patron of IL category Run tests and ensure they pass: prove t/db_dependent/Illrequest/Config.t prove t/Koha/Config.t Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
We should have done this ages ago :)
Created attachment 148977 [details] [review] Bug 33276: Get auth values for location on serials Serial location is linked to a library, if we want to modify serial with users does not belongs to, location is not in select and we could erase it. Test plan: 1) Be sure to have a LOC authorized value limited to one library at least 2) See when editing serial subscription you didn't get the current location 3) Apply this patch 4) Refresh and now you've got the value Signed-off-by: Clemens Tubach <clemens.tubach@kit.edu> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 148978 [details] [review] Bug 21548: ILLLIBS patron category / partner_code seems extraneous This patch updates the default partner category used by the partner_code config to be in line with sample data in sample_patrons.yml Preparation: Apply patch Enable ILLModule sys pref Install an ILL backend (e.g. FreeForm) Add this change to your koha-conf.xml Flush, restart. Search for patron of category inter-library loan and assign a primary e-mail address to it Test plan: Create an ILL request and click 'place request with partners' Verify that the 'select partner libraries' has the correct patron of IL category Run tests and ensure they pass: prove t/db_dependent/Illrequest/Config.t prove t/Koha/Config.t Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed to master for 23.05. Nice work everyone, thanks!
Enhancement - not backporting to 22.11.x Nice work everyone!
Manual will need to be updated.
*** Bug 30820 has been marked as a duplicate of this bug. ***
23.05 Manual updated.