Bug 25885 - Show list of hits for one search result
Summary: Show list of hits for one search result
Status: RESOLVED DUPLICATE of bug 35728
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Magnus Enger
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-06-26 22:07 UTC by Christopher Brannon
Modified: 2024-01-10 10:04 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 25885 - Show list of hits for one search result (5.26 KB, patch)
2021-05-06 10:49 UTC, Magnus Enger
Details | Diff | Splinter Review
Bug 25885 - Show list of hits for one search result (5.31 KB, patch)
2021-05-10 20:46 UTC, Lucas Gass
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Christopher Brannon 2020-06-26 22:07:36 UTC
I would like to suggest, if you have OverDrive (or other 3rd party databases) enabled on Koha, that Koha NOT push you straight into a record (detail.pl) if there is only one "Koha" result.  You end up not being able to see OverDrive results, or switch over to OverDrive when it does that.  Makes the connectivity useless for those with pin-point searching skills.

When you enable one of these databases, that should be all you need to disable the default Koha behavior.
Comment 1 Magnus Enger 2021-04-26 12:02:27 UTC
I know there are people who dislike the "if one result, go directly to it" functionality, without using OverDrive. So perhaps there could be a setting to disable just that, independently of other settings?
Comment 2 Christopher Brannon 2021-04-26 15:12:43 UTC
Can you think of any reason a library would choose to have a database (that they are probably paying for), and still want it to be bypassed and the search result go straight into the record?  If so, then yes, having a separate control makes sense.  But if not, why add another control?  I can't think of any good reason to add more controls.  It seems to complicate the desired affect, and makes more work for those that setup and manage the system.  I think having this behavior disabled if a database is turned on if it produces a result on the results page.  That is straight forward and logical, and the least amount of work for setup, and for troubleshooting.
Comment 3 Magnus Enger 2021-04-27 06:31:47 UTC
Well, my thought was that are libraries that do not have OverDrive (or similar) who want to turn off the "if one result, go directly to it" functionality.
Comment 4 Christopher Brannon 2021-04-27 14:51:09 UTC
Well, there we have it.  A great reason for a separate pref.  :)  Thanks for that.  I'm on board with a separate pref then.
Comment 5 Magnus Enger 2021-05-03 13:10:39 UTC
A library has asked me to implement this, at least for the OPAC. Any suggestions for what the syspref should be called?
Comment 6 Magnus Enger 2021-05-03 13:21:22 UTC
Would something like OPACShowResultListOfOne work? 

OPACShowResultListOfOne = 0 => Current behaviour, go directly to detail view if there is only one hit

OPACShowResultListOfOne = 1 => New behaviour, show the result list even if there is only one hit. 

And then:

OPACShowResultListOfOne [Show|Don't show] the result list in a bibliographic search, if there is only one hit.

Where 

Show = 1
Don't show = 0
Comment 7 Christopher Brannon 2021-05-04 14:41:57 UTC
(In reply to Magnus Enger from comment #6)
> Would something like OPACShowResultListOfOne work? 
> 
> OPACShowResultListOfOne = 0 => Current behaviour, go directly to detail view
> if there is only one hit
> 
> OPACShowResultListOfOne = 1 => New behaviour, show the result list even if
> there is only one hit. 
> 
> And then:
> 
> OPACShowResultListOfOne [Show|Don't show] the result list in a bibliographic
> search, if there is only one hit.
> 
> Where 
> 
> Show = 1
> Don't show = 0

Looks good.  You could also go with OPACAlwaysShowResults.  Either would work, the latter being less specific.
Comment 8 Magnus Enger 2021-05-06 10:49:50 UTC Comment hidden (obsolete)
Comment 9 Lucas Gass 2021-05-10 20:46:23 UTC
Created attachment 120811 [details] [review]
Bug 25885 - Show list of hits for one search result

Default behaviour in Koha, for a bibliographic search that
returns one hit is to skip the result list and go straight
to the detail view of that one hit. This patch makes this
behaviour configurable.

To test:
- Make sure you can do a search that returns exactly one hit
- Apply this patch
- Do the search and verify the default behaviour of skipping
  the result list and going straight to detail view is in place
- Toggle the OPACShowResultListOfOne syspref and make sure the
  behaviour corresponds to the setting.

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 10 Lucas Gass 2021-05-10 20:47:44 UTC
Everything works good here and I'm signing off. One thing, the QA tool doesn't like the commit message because there is no ':' after 'Bug 25885'.
Comment 11 Marcel de Rooy 2021-07-09 09:15:25 UTC
Your search returned 1 results.
Comment 12 Marcel de Rooy 2021-07-09 09:16:21 UTC
Looks good. Please address comments 10 and 11.
Comment 13 Magnus Enger 2024-01-10 10:04:17 UTC

*** This bug has been marked as a duplicate of bug 35728 ***