Bug 30740

Summary: Link to authorities 'used in' should not use equal
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: SearchingAssignee: Fridolin Somers <fridolin.somers>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: david
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:
22.05.00
Bug Depends on:    
Bug Blocks: 29048    
Attachments: Bug 30740: Link to authorities 'used in' should not use equal
Bug 30740: Link to authorities 'used in' should not use equal
Bug 30740: Link to authorities 'used in' should not use equal

Description Fridolin Somers 2022-05-12 07:11:11 UTC
When showing an authority, there is a link to search biblio records it is used in.
Actually this link uses 'q=an=$auth_id', this is wrong for an URL, and other places use 'idx:value'.
It does not break search with Zebra nor Elasticsearch but it is better to use ':'
Comment 1 Fridolin Somers 2022-05-12 07:26:24 UTC
Created attachment 134916 [details] [review]
Bug 30740: Link to authorities 'used in' should not use equal

When showing an authority, there is a link to search biblio records it is used in.
Actually this link uses 'q=an=$auth_id', this is wrong for an URL, and other places use 'idx:value'.
It does not break search with Zebra nor Elasticsearch but it is better to use ':'

Test plan :
Check changed links return same results with and without patch
Comment 2 David Nind 2022-05-12 23:08:17 UTC
Created attachment 134938 [details] [review]
Bug 30740: Link to authorities 'used in' should not use equal

When showing an authority, there is a link to search biblio records it is used in.
Actually this link uses 'q=an=$auth_id', this is wrong for an URL, and other places use 'idx:value'.
It does not break search with Zebra nor Elasticsearch but it is better to use ':'

Test plan :
Check changed links return same results with and without patch

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2022-05-12 23:10:32 UTC
Testing notes using koha-testing-docker.

I used the authority term "Agile software development" (authority id = 133) - this is used in two records in the sample data.

The search results were the same before and after the patch was applied.

I only tested with the Zebra search engine.

Links are changed/fixed on:
- the OPAC search results page - number of records link: changes an=133 to an:133
- the staff interface details page - "Used on X record(s)" link: changes an=133 to an:133

Details
=======

OPAC
~~~~

- Before:

  . Search results page - number of records link: http://127.0.0.1:8080/cgi-bin/koha/opac-search.pl?type=opac&op=do_search&q=an=133
  
  . Authority details page - "Number of records used in: 2": http://127.0.0.1:8080/cgi-bin/koha/opac-search.pl?type=opac&q=133&idx=an,phr

- After:
  . Search results page - number of records link: http://127.0.0.1:8080/cgi-bin/koha/opac-search.pl?type=opac&op=do_search&q=an:133
  
  . Authority details page - "Number of records used in: 2":  http://127.0.0.1:8080/cgi-bin/koha/opac-search.pl?type=opac&q=133&idx=an,phr

Staff interface
~~~~~~~~~~~~~~~

- Before:

  . Search results page - number of records link: http://127.0.0.1:8081/cgi-bin/koha/catalogue/search.pl?type=intranet&op=do_search&idx=an,phr&q=133
  
  . Authority details page - "Used in 2 record(s)": http://127.0.0.1:8081/cgi-bin/koha/catalogue/search.pl?type=intranet&op=do_search&q=an=133

- After:

  . Search results page - number of records link: http://127.0.0.1:8081/cgi-bin/koha/catalogue/search.pl?type=intranet&op=do_search&idx=an,phr&q=133
  
  . Authority details page - "Used in 2 record(s)": http://127.0.0.1:8081/cgi-bin/koha/catalogue/search.pl?type=intranet&op=do_search&q=an:133
Comment 4 Fridolin Somers 2022-05-13 02:30:17 UTC
Ahhh link is different between results and detail page :
q=133&idx=an,phr
q=an=133

I will open an other bug report for that
Comment 5 Katrin Fischer 2022-05-13 07:22:20 UTC
(In reply to Fridolin Somers from comment #4)
> Ahhh link is different between results and detail page :
> q=133&idx=an,phr
> q=an=133
> 
> I will open an other bug report for that

No need to - I did and fixed it, has been waiting for sign-off:
Bug 29048 - Incorrect search for linked authority records from authority search result list in OPAC
Comment 6 Katrin Fischer 2022-05-13 07:26:05 UTC
Created attachment 134945 [details] [review]
Bug 30740: Link to authorities 'used in' should not use equal

When showing an authority, there is a link to search biblio records it is used in.
Actually this link uses 'q=an=$auth_id', this is wrong for an URL, and other places use 'idx:value'.
It does not break search with Zebra nor Elasticsearch but it is better to use ':'

Test plan :
Check changed links return same results with and without patch

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Comment 7 Fridolin Somers 2022-05-16 21:21:04 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄