When there is only one library, the library list is skipped, but since opac-main does not pass a branch code, you won't see branch info.
Created attachment 141824 [details] [review] Bug 31775: Show single library Test plan: Check both cases: one library (public accessible) and more libraries (public accessible). In the latter case click on one of these libraries too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 141825 [details] [review] Bug 31775: Template cleanup Removing unused parts. Test plan: See first test plan. Confirm that it still works. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
I had a go at testing, but struggled to notice any difference - except that for a single library, the library name is linked. No additional information is displayed on the linked page (even if all the library information fields are filling in). If this is what is expected, I'm happy to sign off. Testing notes (using koha-testing-docker): - Needed to batch modify patrons to change their library to the one kept (possibly could have just deleted all patrons). - Needed to batch modify items for all libraries not kept so that their home library and current library changed to the library kept. - Not sure how to make a library not publicly accessible, is there some other system preference to change? . Changing an existing library: get message "An error occurred when updating this library. Perhaps it already exists." . Creating a new library - even if you change public to "No", it saves as public with "Yes".
(In reply to David Nind from comment #3) > I had a go at testing, but struggled to notice any difference - except that > for a single library, the library name is linked. No additional information > is displayed on the linked page (even if all the library information fields > are filling in). > > If this is what is expected, I'm happy to sign off. Thx for testing, David. You are correct: my test plan could be more detailed. What we need to do also, is add the opac_info information for a branch via Additional Contents, html cust., OpacLibraryInfo location. For one library you should see the full info page at once. For more libraries you should see the list first. Deleting a library should not be needed. Just hiding it via the public flag. I will check if you found a bug there too.. Will comment on a few other points still.
(In reply to David Nind from comment #3) > Testing notes (using koha-testing-docker): > > - Needed to batch modify patrons to change their library to the one kept > (possibly could have just deleted all patrons). Should not be needed when you hide a branch. > - Needed to batch modify items for all libraries not kept so that their home > library and current library changed to the library kept. Same. Not really needed. > - Not sure how to make a library not publicly accessible, is there some > other system preference to change? Go to Admin. Libraries. Edit library. Change public. > . Changing an existing library: get message "An error occurred when > updating this library. Perhaps it already exists." Cannot reproduce this one. I toggle various libraries from Yes to No and vv. > . Creating a new library - even if you change public to "No", it saves as > public with "Yes". Yes, good catch. Editing existing libraries works here. But adding a library does not respect the public flag! Opening a new report.
When you have a lot of branches, use SQL: update branches set public=0; And set one branch back to Yes.
(In reply to Marcel de Rooy from comment #5) > > . Creating a new library - even if you change public to "No", it saves as > > public with "Yes". > Yes, good catch. Editing existing libraries works here. But adding a library > does not respect the public flag! Opening a new report. Fixed on bug 31785 now.
Created attachment 141871 [details] [review] Bug 31775: Show single library Test plan: Check both cases: one library (public accessible) and more libraries (public accessible). In the latter case click on one of these libraries too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: David Nind <david@davidnind.com>
Created attachment 141872 [details] [review] Bug 31775: Template cleanup Removing unused parts. Test plan: See first test plan. Confirm that it still works. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: David Nind <david@davidnind.com>
Testing notes (using koha-testing-docker). 1. I applied bug 31785 (so can toggle Public to No) (Alternative is to use SQL: update branches set public=0; and then set two libraries back to Yes.) 2. Add some additional information to the OPAC page for a library (for example, Centerville) at Tools > Additional tools > HTML customisations > New entry > select OpacLibraryInfo 3. Check the OPAC page for the library to make sure this is visible: OPAC Home > Libraries > [List of Libraries] > click link for Centerville: - Note that for the list of libraries, each library is linked to an individual page - On the list of libraries page, the information entered in the Libraries Administration area is displayed - Note that for the individual library page it includes the information on the listing page, as well as any info entered in step 2 4. Make all other libraries except Centerville not public: Administration > Basic parameters > Libraries: - Change public to no for all libraries except Centerville (see step 1 for the SQL way to make this change) 5. Visit the OPAC page for the list of libraries: http://127.0.0.1:8080/cgi-bin/koha/opac-library.pl . Note that there is no link to the individual Centerville page . Note that only the basic library information entered in Administration > Basic parameters > Libraries is displayed 6. Apply the patch and refresh the list of libraries page: - Note that Centerville is now linked - On the individual library page for Centerville you now have the basic information and the information from step 2
Created attachment 141933 [details] [review] Bug 31775: Show single library Test plan: Check both cases: one library (public accessible) and more libraries (public accessible). In the latter case click on one of these libraries too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 141934 [details] [review] Bug 31775: Template cleanup Removing unused parts. Test plan: See first test plan. Confirm that it still works. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 22.11. Nice work everyone, thanks!