Bug 35034

Summary: Add link to the bibliographic records when they are selected for merging
Product: Koha Reporter: Laura Escamilla <Laura.escamilla>
Component: CatalogingAssignee: Laura Escamilla <Laura.escamilla>
Status: Pushed to oldstable --- QA Contact: Emily Lamancusa <emily.lamancusa>
Severity: enhancement    
Priority: P5 - low CC: andrewfh, emily.lamancusa, fridolin.somers, lucas, m.de.rooy
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:
24.05.00,23.11.06,23.05.12
Circulation function:
Attachments: Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form.
Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form.
Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form.
Bug 35034: Add parameter that links titles to their records
Bug 35034: Add parameter that links titles to their records
Bug 35034: Add parameter that links titles to their records

Description Laura Escamilla 2023-10-11 15:59:43 UTC
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. 

It would be helpful to have the biblionumber hyperlinked to the bib record so that staff can easily view the record if needed.
Comment 1 Laura Escamilla 2023-10-11 16:59:41 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 :)
Comment 2 ByWater Sandboxes 2023-10-11 18:50:30 UTC
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>
Comment 3 ByWater Sandboxes 2023-10-12 13:27:28 UTC
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>
Comment 4 Katrin Fischer 2023-11-04 18:51:58 UTC
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 %
Comment 5 Laura Escamilla 2023-12-05 14:51:29 UTC
(In reply to Katrin Fischer from comment #4)

Thank you, Katrin! I will implement this change.
Comment 6 Laura Escamilla 2023-12-05 15:07:27 UTC
Created attachment 159587 [details] [review]
Bug 35034: Add parameter that links titles to their records

Same test as above
Comment 7 Owen Leonard 2023-12-05 15:58:26 UTC
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>
Comment 8 Emily Lamancusa 2024-01-25 18:21:14 UTC
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>
Comment 9 Emily Lamancusa 2024-01-25 18:22:40 UTC
Looks good, QA script is happy. Passing QA
Comment 10 Katrin Fischer 2024-03-22 14:48:12 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 11 Fridolin Somers 2024-05-23 08:10:42 UTC
Pushed to 23.11.x for 23.11.06
Comment 12 Lucas Gass 2024-05-28 15:56:18 UTC
Backported to 23.05.x for upcoming 23.05.12