]> git.koha-community.org Git - koha.git/commit
Bug 19134: C4::SMS falils on long driver name
authorMagnus Enger <magnus@libriotech.no>
Thu, 17 Aug 2017 20:48:07 +0000 (22:48 +0200)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 29 Sep 2017 11:28:13 +0000 (13:28 +0200)
commit9f8cdead6e29f029e01038ecb4fbd6216544e996
treed3070712c545f1780495a1026c3c1b465eb4d345
parentb3e2933f5c36101784a1780444dd619655ee7ea2
Bug 19134: C4::SMS falils on long driver name

Code in C4::SMS takes the part of the SMS::Send-driver that comes after
SMS::Send and tries to turn it into part of a path to a YAML file
that can contain additional parameters to SMS::Send. The current
code works for e.g. SMS::Send::A::B, but if there is one or more
extra names, it fails to turn :: into /. So we have:
SMS::Send::A::B    -> SMS/Send/A/B
SMS::Send::A::B::C -> SMS/Send/A/B::C
This patch makes sure all occurrences of :: are turned into /, by
adding a "g" modifier at the end of the regex.

Testing:
Testing this preperly would take a whole lot of setup for a very
small change. I would suggest that the following two oneliners
are enough to demonstrate that the change makes sense:
$ perl -e '$x = "a::b::c"; $x =~ s|::|/|; print $x, "\n";'
$ perl -e '$x = "a::b::c"; $x =~ s|::|/|g; print $x, "\n";'

So:
- Check that the output of these oneliners make sense
- Check that the patch changes the code in a similar way to the
  change from the first oneliner to the second.

Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit a4237785d9a2bf2bbf4cf988bc98ac4935842e91)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
C4/SMS.pm