Bug 38762 - compare_es_to_db.pl should provide links to the staff interface
Summary: compare_es_to_db.pl should provide links to the staff interface
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Tomás Cohen Arazi (tcohen)
QA Contact: Nick Clemens (kidclamp)
URL:
Keywords:
Depends on: 22831
Blocks:
  Show dependency treegraph
 
Reported: 2024-12-20 13:03 UTC by Tomás Cohen Arazi (tcohen)
Modified: 2025-01-29 15:55 UTC (History)
5 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:
Circulation function:


Attachments
Bug 38762: Make compare_es_to_db.pl provide links to staff interface (2.37 KB, patch)
2024-12-20 13:13 UTC, Tomás Cohen Arazi (tcohen)
Details | Diff | Splinter Review
Bug 38762: Make compare_es_to_db.pl provide links to staff interface (2.42 KB, patch)
2024-12-20 13:52 UTC, Roman Dolny
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Tomás Cohen Arazi (tcohen) 2024-12-20 13:03:04 UTC
The script currently generates a link to the OPAC URLs for the affected records. 

It should generate links we can copy and paste to access the records for fixing them.
Comment 1 Tomás Cohen Arazi (tcohen) 2024-12-20 13:13:14 UTC
Created attachment 175839 [details] [review]
Bug 38762: Make compare_es_to_db.pl provide links to staff interface

This patch makes the script provide links to the staff interface instead
of the OPAC. This way, the links can just be picked to access the
problematic records for fixing them.

To test:
1. Run:
   $ ktd --shell
  k$ perl misc/maintenance/compare_es_to_db.pl
=> FAIL: Generated links point to the OPAC
2. Apply this patch
3. Repeat 1
=> SUCCESS: Generated links are useful as they point to the staff
interface
4. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 2 Roman Dolny 2024-12-20 13:52:04 UTC
Created attachment 175845 [details] [review]
Bug 38762: Make compare_es_to_db.pl provide links to staff interface

This patch makes the script provide links to the staff interface instead
of the OPAC. This way, the links can just be picked to access the
problematic records for fixing them.

To test:
1. Run:
   $ ktd --shell
  k$ perl misc/maintenance/compare_es_to_db.pl
=> FAIL: Generated links point to the OPAC
2. Apply this patch
3. Repeat 1
=> SUCCESS: Generated links are useful as they point to the staff
interface
4. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 3 Jonathan Druart 2025-01-16 13:42:46 UTC
Nick, do you agree with replacing the OPAC link with a staff interface link?
Comment 4 Lucas Gass (lukeg) 2025-01-29 15:55:21 UTC
(In reply to Jonathan Druart from comment #3)
> Nick, do you agree with replacing the OPAC link with a staff interface link?

Not Nick, but I do agree. It should point to the staff interface so staff can go fix it, I think. 

Also, I can't see patrons needing to ever view this.