Bug 28673

Summary: An encoded ampersand missing the ampersand
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: ListsAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Nick Clemens (kidclamp) <nick>
Severity: normal    
Priority: P5 - low CC: m.de.rooy, nick
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:
21.11.00
Circulation function:
Attachments: Bug 28673: Add missing ampersand to amp; construction
Bug 28673: Add missing ampersand to amp; construction

Description Marcel de Rooy 2021-07-07 12:38:22 UTC

    
Comment 1 Marcel de Rooy 2021-07-07 12:43:28 UTC
Created attachment 122649 [details] [review]
Bug 28673: Add missing ampersand to amp; construction

Super trivial one character fix ;)

Test plan:
Open Your Lists from the button on main page.
Click again on Your lists under Lists in the middle of the form.
Verify that the URL does no longer show something like:
    op=listamp;category=1

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 2 Marcel de Rooy 2021-07-07 12:44:26 UTC
Self signoff one character fix
Comment 3 Nick Clemens (kidclamp) 2021-07-09 12:50:35 UTC
Created attachment 122741 [details] [review]
Bug 28673: Add missing ampersand to amp; construction

Super trivial one character fix ;)

Test plan:
Open Your Lists from the button on main page.
Click again on Your lists under Lists in the middle of the form.
Verify that the URL does no longer show something like:
    op=listamp;category=1

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 4 Jonathan Druart 2021-07-12 13:50:03 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 5 Marcel de Rooy 2021-09-29 11:53:33 UTC
Applies on 20.11 with me.