Summary: | Link to host item doesn't work in analytical records if 773$a is present | ||
---|---|---|---|
Product: | Koha | Reporter: | verolencinas <verolencinas> |
Component: | OPAC | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fridolin.somers, kyle, martin.renvoize, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11175 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
21.11.00,21.05.05,20.11.11
|
|
Circulation function: | |||
Attachments: |
Bug 20277: Improve 773 link when $a is present
Bug 20277: Improve 773 link when $a is present Bug 20277: Improve 773 link when $a is present |
Description
verolencinas
2018-02-22 21:36:54 UTC
Is this still an issue.. changing to enhancement request as it's not received any attention. Created attachment 124061 [details] [review] Bug 20277: Improve 773 link when $a is present At the moment $t (Title) and $a (Main heading - author) are both searched in ti,phr, which sometimes may work, as ti contains the full 245 field, including $c with author information. But often $c will not match what is in 100$a, so the search fails. This splits the search string so we search for $t in with ti,phr and for $a in au. To test: - Make sure UseControlnumber is deactivated - Search for a record with an author in 100 in your catalog Example from sample data: Programming Perl / Tom Christiansen, Brian D. Foy & Larry Wall. - Add a child record using New > Add child record - Verify that in 773 $a and $t are filled in - Complete mandatory fields and save the record . In OPAC and staff interface: - Verify the link in the detail page is doing a search with ti,phr - Verify the link works/doesn't work (more likely the latter) - Apply patch - Verify the link has changed and (now) works Hmm, would it be possible to also get this added to the corresponding search which sets whether the 'Show analytics' link it visible or not (or depend on 11175 and update the search string method there?) I'd love to DRY this search string out a bit honestly.. constructing it within the XSLT and also having an accessor for it in the Koha::Biblio objects isn't great practice.. perhaps we should pass it as an XSLT variable from the Biblio object and thus only have it built in one place.? Created attachment 125990 [details] [review] Bug 20277: Improve 773 link when $a is present At the moment $t (Title) and $a (Main heading - author) are both searched in ti,phr, which sometimes may work, as ti contains the full 245 field, including $c with author information. But often $c will not match what is in 100$a, so the search fails. This splits the search string so we search for $t in with ti,phr and for $a in au. To test: - Make sure UseControlnumber is deactivated - Search for a record with an author in 100 in your catalog Example from sample data: Programming Perl / Tom Christiansen, Brian D. Foy & Larry Wall. - Add a child record using New > Add child record - Verify that in 773 $a and $t are filled in - Complete mandatory fields and save the record . In OPAC and staff interface: - Verify the link in the detail page is doing a search with ti,phr - Verify the link works/doesn't work (more likely the latter) - Apply patch - Verify the link has changed and (now) works Signed-off-by: David Nind <david@davidnind.com> Created attachment 126444 [details] [review] Bug 20277: Improve 773 link when $a is present At the moment $t (Title) and $a (Main heading - author) are both searched in ti,phr, which sometimes may work, as ti contains the full 245 field, including $c with author information. But often $c will not match what is in 100$a, so the search fails. This splits the search string so we search for $t in with ti,phr and for $a in au. To test: - Make sure UseControlnumber is deactivated - Search for a record with an author in 100 in your catalog Example from sample data: Programming Perl / Tom Christiansen, Brian D. Foy & Larry Wall. - Add a child record using New > Add child record - Verify that in 773 $a and $t are filled in - Complete mandatory fields and save the record . In OPAC and staff interface: - Verify the link in the detail page is doing a search with ti,phr - Verify the link works/doesn't work (more likely the latter) - Apply patch - Verify the link has changed and (now) works Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Passing QA.. this works well.. I read it backwards... whilst it would be nice to DRY this code out too.. it's not this particular query I was thinking about it was the reverse one for looking up component parts. Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.05 Pushed to 20.11.x for 20.11.11 Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. |