[Koha-bugs] [Bug 19134] New: C4:: SMS does not handle drivers with more than two names well

[bugzilla-daemon at bugs.koha-community.org]

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

            Bug ID: 19134
           Summary: C4::SMS does not handle drivers with more than two
                    names well
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5 - low
         Component: Notices
          Assignee: koha-bugs at lists.koha-community.org
          Reporter: magnus at libriotech.no
        QA Contact: testopia at bugs.koha-community.org

This code:

 94     my $subpath = $driver;
 95     $subpath =~ s|::|/|;

is supposed to turn the "A::B" part of e.g. SMS::Send::A::B into "A/B", so it
can be used as part of the path to the extra config file introduced in bug
13029. But it will only change the first occurence of :: into a /, so a driver
with a name like SMS::Send::A::B::C will result in "A/B::C" being used as part
of the path, which fails. 

A "g" modifier needs to be added to the regex, to do the substitution gloablly,
thusly: 

 95     $subpath =~ s|::|/|g;

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.