Bug 36623

Summary: Remove localhost reference from counter logs page
Product: Koha Reporter: Matt Blenkinsop <matt.blenkinsop>
Component: ERMAssignee: Matt Blenkinsop <matt.blenkinsop>
Status: RESOLVED FIXED QA Contact:
Severity: minor    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, jonathan.field, lucas, martin.renvoize, matt.blenkinsop, pedro.amorim, tomascohen
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
Circulation function:
Bug Depends on:    
Bug Blocks: 36627    
Attachments: Bug 36623: Remove localhost reference
Bug 36623: Fix translation
Bug 36623:(follow-up) Fix cypress test

Description Matt Blenkinsop 2024-04-17 15:03:28 UTC
There is a reference to localhost on the counter logs page that shouldn't be there and needs removing
Comment 1 Matt Blenkinsop 2024-04-17 15:12:27 UTC
Created attachment 165023 [details] [review]
Bug 36623: Remove localhost reference

This patch removes a reference to localhost from a url within ERM

Test plan:
This is quite hard to test without sushi credentials and is quite an easy fix. Check the patch diff to verify that the localhost reference has now been removed from the url
Comment 2 Martin Renvoize (ashimema) 2024-04-17 18:26:25 UTC
Very obvious and clear correction. Going straight for a PQA here.
Comment 3 Katrin Fischer 2024-04-18 14:24:36 UTC
I believe this highlights an untranslatable string (Borrowernumber):

? `<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=${row.borrowernumber}">Borrowernumber ${row.borrowernumber}</a>`

Please fix in a follow-up or on a separate new bug report.
Comment 4 Matt Blenkinsop 2024-04-18 14:29:52 UTC
That is addressed in bug 36627 which embeds the patron and uses the patron name instead
Comment 5 Katrin Fischer 2024-04-18 14:36:30 UTC
Can you please double check the file for other occurences? I think there is another: "Cronjob"
Comment 6 Matt Blenkinsop 2024-04-18 14:39:19 UTC
Created attachment 165114 [details] [review]
Bug 36623: Fix translation
Comment 7 Katrin Fischer 2024-04-18 14:52:20 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 8 Katrin Fischer 2024-04-19 16:07:42 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 9 Matt Blenkinsop 2024-05-15 10:42:42 UTC
Created attachment 166767 [details] [review]
Bug 36623:(follow-up) Fix cypress test
Comment 10 Tomás Cohen Arazi (tcohen) 2024-05-15 20:45:36 UTC
Follow up pushed to main. Thanks!
Comment 11 Fridolin Somers 2024-05-23 13:34:43 UTC
Pushed to 23.11.x for 23.11.06
Comment 12 Lucas Gass 2024-05-28 19:38:50 UTC
Doesn't apply cleanly to 23.05.x, no backport.