https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18200

Jonathan Druart <jonathan.dru...@bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #60808|0                           |1
        is obsolete|                            |

--- Comment #5 from Jonathan Druart <jonathan.dru...@bugs.koha-community.org> 
---
Created attachment 61036
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=61036&action=edit
Bug 18200: Fix a potential issue with preceding space in GetMarcUrls

Trims the URL in order prevent prefixing a space with http://
Normally you won't have a preceding space here, but I saw it happening
one day and it does not cost much to resolve it.

Bonus: Adding few simple tests in t/db_dependent/Biblio.t.

Test plan:
[1] Run t/db_dependent/Biblio.t
[2] Add a 856$u with preceding space (MARC21)
[3] Check opac-detail, Online access with OPACXSLTDetailsDisplay empty.

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

Followed test plan, works as expected
Signed-off-by: Marc VĂ©ron <ve...@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.dru...@bugs.koha-community.org>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to