Bug 17852

Summary: Multiple URLs (856) in list email are broken
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: ListsAssignee: Caitlin Goodger <caitlingoodger.student>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: caitlingoodger.student, crazy.mental.onion, julian.maurice, kyle, m.de.rooy, mtj
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14614
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 17852; Multiple URLs (856) in list email are broken.
Bug 17852 Follow up.Multiple URLs (856) in list email are broken
[SIGNED-OFF] Bug 17852; Multiple URLs (856) in list email are broken.
[SIGNED-OFF] Bug 17852 Follow up.Multiple URLs (856) in list email are broken
Bug 17852; Multiple URLs (856) in list email are broken.
Bug 17852 Follow up.Multiple URLs (856) in list email are broken

Description Katrin Fischer 2017-01-05 14:26:52 UTC
Same bug as bug 14614, but for list emails send from staff and intranet.

If there are multiple URLs (865 fields), the URLs get squashed together into 'one URL' in the email sent and are broken therefore.

Same fix as for the other bugs should work nicely here.
Comment 1 Caitlin Goodger 2017-01-17 21:52:52 UTC
Created attachment 59111 [details] [review]
Bug 17852; Multiple URLs (856) in list email are broken.

Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it.
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work
Comment 2 Katrin Fischer 2017-01-18 02:29:42 UTC
Hi Caitlin, this is good, but can you please take a look at the corresponding file for the OPAC (koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-sendshelf.tt)? It needs the same fix and I could then test and sign-off on both fixes together. Thx!
Comment 3 Caitlin Goodger 2017-01-19 00:57:29 UTC
Created attachment 59197 [details] [review]
Bug 17852 Follow up.Multiple URLs (856) in list email are broken

This is a follow-up to my previous patch. This fixes it on the OPAC and
the previous one was from the intranet
Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it from the OPAC
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work
Comment 4 Owen Leonard 2017-01-24 19:16:36 UTC
Created attachment 59527 [details] [review]
[SIGNED-OFF] Bug 17852; Multiple URLs (856) in list email are broken.

Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it.
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Owen Leonard 2017-01-24 19:16:43 UTC
Created attachment 59528 [details] [review]
[SIGNED-OFF] Bug 17852 Follow up.Multiple URLs (856) in list email are broken

This is a follow-up to my previous patch. This fixes it on the OPAC and
the previous one was from the intranet
Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it from the OPAC
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 6 Jonathan Druart 2017-02-20 13:21:51 UTC
Created attachment 60462 [details] [review]
Bug 17852; Multiple URLs (856) in list email are broken.

Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it.
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 7 Jonathan Druart 2017-02-20 13:21:56 UTC
Created attachment 60463 [details] [review]
Bug 17852 Follow up.Multiple URLs (856) in list email are broken

This is a follow-up to my previous patch. This fixes it on the OPAC and
the previous one was from the intranet
Test Plan
1: Add multiple urls to a biblographic record
2: Add the biblographic record to a list and then email it from the OPAC
3: See that the links are broken and have been merged into one.
4: Apply patch
5: Send the list again and see that the links work

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Kyle M Hall 2017-02-21 19:56:00 UTC
Pushed to master for 17.05, thanks Caitlin!
Comment 9 Katrin Fischer 2017-02-21 21:05:45 UTC
These patches have been pushed to 16.11.x and will be in 16.11.04.

Thx Caitlin!
Comment 10 Katrin Fischer 2017-02-21 21:43:33 UTC
Bug, not enhancement :)
Comment 11 Mason James 2017-02-23 20:38:49 UTC
Pushed to 16.05.x, for 16.05.10 release
Comment 12 Julian Maurice 2017-03-15 11:16:11 UTC
Pushed to 3.22.x, will be in 3.22.18