|
Description
Laura Escamilla
2023-10-11 15:59:43 UTC
Created attachment 156854 [details] [review] Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. To test: 1. Select two items from the catalog and begin a merge. 2. Notice that the title of the items, their biblionumber and a hyperlink to their MARC record is available. 3. Apply the patch and refresh the page. The biblionumber for each title is now hyperlinked and leads to the item’s bib records. 4. Sign off and have a great day :) Created attachment 156876 [details] [review] Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. To recreate: 1. Select two items from the catalog and begin a merge. 2. Notice that the title of the items, their biblionumber and a hyperlink to their MARC record is available. 3. Apply the patch and refresh the page. The biblionumber for each title is now hyperlinked and leads to the item’s bib records. 4. Sign off and have a great day :) Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Created attachment 156962 [details] [review] Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. To recreate: 1. Select two items from the catalog and begin a merge. 2. Notice that the title of the items, their biblionumber and a hyperlink to their MARC record is available. 3. Apply the patch and refresh the page. The biblionumber for each title is now hyperlinked and leads to the item’s bib records. 4. Sign off and have a great day :) Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> Signed-off-by: Christine <chlee@pascolibraries.org> Hi Laura, your patch is not quite correct HTML: + <a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% record.biblionumber | uri %]"</a> ([% record.biblionumber | uri %]) * Your opening tag is not closing * We don't have a link text within the a tag The browsers guess kinda well here, which is why this was probably not found in testing. But if you take a look at the source, you can see that there is an error. I have another suggestion on how to implement this: To display the title, we have: [% INCLUDE 'biblio-title.inc' biblio=record.data %] You can just add a parameter to make that linked to the record: [% INCLUDE 'biblio-title.inc' biblio=record.data link = 1 % (In reply to Katrin Fischer from comment #4) Thank you, Katrin! I will implement this change. Created attachment 159587 [details] [review] Bug 35034: Add parameter that links titles to their records Same test as above Created attachment 159590 [details] [review] Bug 35034: Add parameter that links titles to their records To recreate: 1. Select two items from the catalog and begin a merge. 2. Notice that the title of the items, their biblionumber and a hyperlink to their MARC record is available. 3. Apply the patch and refresh the page. The biblionumber for each title is now hyperlinked and leads to the item’s bib records. 4. Sign off and have a great day :) Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 161456 [details] [review] Bug 35034: Add parameter that links titles to their records To recreate: 1. Select two items from the catalog and begin a merge. 2. Notice that the title of the items, their biblionumber and a hyperlink to their MARC record is available. 3. Apply the patch and refresh the page. The biblionumber for each title is now hyperlinked and leads to the item’s bib records. 4. Sign off and have a great day :) Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Looks good, QA script is happy. Passing QA Pushed for 24.05! Well done everyone, thank you! Pushed to 23.11.x for 23.11.06 Backported to 23.05.x for upcoming 23.05.12 |