Summary: | Staff search result list shows "other holdings" with AlternateHoldingsField when there are no alternate holdings | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | System Administration | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | RESOLVED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | minor | ||
Priority: | P5 - low | CC: | arthur.suzuki, gmcharlt, lucas, m.de.rooy, michaela.sieber |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.05.00,22.11.05,22.05.13,21.11.21
|
|
Bug Depends on: | 33032 | ||
Bug Blocks: | |||
Attachments: |
Bug 33509: Don't show Other holdings when there are no alternate holdings
Bug 33509: Don't show Other holdings when there are no alternate holdings Bug 33509: Don't show Other holdings when there are no alternate holdings |
Description
Katrin Fischer
2023-04-12 17:10:44 UTC
Created attachment 149575 [details] [review] Bug 33509: Don't show Other holdings when there are no alternate holdings With bug 33509 alternate holdings defined in AlternateHoldingsField are shown in the staff interface result list. This works greatly when there are items or when there is are alternate holdings (the configured field/subfields exist in the record), but not when there are no items and no alternate holdings. To test: * Add 100a to AlternateHoldingsField * Make a search with some records without items Sample data: perl * Make sure the alternate holdings show when there are no items and 100a exists * Change to 100z * Verify the result list now shows "Other holdings:" with nothing below * Apply patch * Repeat * Alternate holdings should still show nicely * Without alternate holdings 'no items' is displayed Created attachment 149616 [details] [review] Bug 33509: Don't show Other holdings when there are no alternate holdings With bug 33509 alternate holdings defined in AlternateHoldingsField are shown in the staff interface result list. This works greatly when there are items or when there is are alternate holdings (the configured field/subfields exist in the record), but not when there are no items and no alternate holdings. To test: * Add 100a to AlternateHoldingsField * Make a search with some records without items Sample data: perl * Make sure the alternate holdings show when there are no items and 100a exists * Change to 100z * Verify the result list now shows "Other holdings:" with nothing below * Apply patch * Repeat * Alternate holdings should still show nicely * Without alternate holdings 'no items' is displayed Signed-off-by: David Nind <david@davidnind.com> Created attachment 149639 [details] [review] Bug 33509: Don't show Other holdings when there are no alternate holdings With bug 33509 alternate holdings defined in AlternateHoldingsField are shown in the staff interface result list. This works greatly when there are items or when there is are alternate holdings (the configured field/subfields exist in the record), but not when there are no items and no alternate holdings. To test: * Add 100a to AlternateHoldingsField * Make a search with some records without items Sample data: perl * Make sure the alternate holdings show when there are no items and 100a exists * Change to 100z * Verify the result list now shows "Other holdings:" with nothing below * Apply patch * Repeat * Alternate holdings should still show nicely * Without alternate holdings 'no items' is displayed Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Backported to 22.05.x for upcoming 22.05.13 applied to 21.11 for 21.11.21 |