Bug 35034 - Add link to the bibliographic records when they are selected for merging
Summary: Add link to the bibliographic records when they are selected for merging
Status: Passed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Laura Escamilla
QA Contact: Emily Lamancusa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-11 15:59 UTC by Laura Escamilla
Modified: 2024-01-25 19:46 UTC (History)
3 users (show)

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


Attachments
Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. (1.67 KB, patch)
2023-10-11 16:59 UTC, Laura Escamilla
Details | Diff | Splinter Review
Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. (1.73 KB, patch)
2023-10-11 18:50 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35034: Added a hyperlink of the bib record to the biblionumber on the merge record form. (1.78 KB, patch)
2023-10-12 13:27 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 35034: Add parameter that links titles to their records (1.41 KB, patch)
2023-12-05 15:07 UTC, Laura Escamilla
Details | Diff | Splinter Review
Bug 35034: Add parameter that links titles to their records (1.89 KB, patch)
2023-12-05 15:58 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 35034: Add parameter that links titles to their records (1.96 KB, patch)
2024-01-25 18:21 UTC, Emily Lamancusa
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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