From 9dccd0961914439d33bd50135352111605bd64ce Mon Sep 17 00:00:00 2001 From: Chris Cormack Date: Fri, 30 Sep 2016 09:31:11 +1300 Subject: [PATCH] [SIGNED-OFF] Bug 13029 : Follow up Adding POD, changing the config files to live in a path pointed to by koha-conf.xml This means multiple instances can have their own config Please test the 2 patches together Signed-off-by: Magnus Enger Works as advertised. Arbitrary arguments can be passed to SMS:Send drivers. If an argument is provided that has already been set by SMS::Send or the driver, it will be overwritten by the value from the YAML file. My only suggestion for an improvement would be an example of what the YAML should look like, but that is a minor thing. --- C4/SMS.pm | 25 +++++++++++++++++++++---- debian/templates/koha-conf-site.xml.in | 2 ++ etc/koha-conf.xml | 2 ++ 3 files changed, 25 insertions(+), 4 deletions(-) diff --git a/C4/SMS.pm b/C4/SMS.pm index d011f5f..c603117 100644 --- a/C4/SMS.pm +++ b/C4/SMS.pm @@ -1,6 +1,8 @@ package C4::SMS; # Copyright 2007 Liblime +# Copyright 2015 Biblibre +# Copyright 2016 Catalyst # # This file is part of Koha. # @@ -28,7 +30,23 @@ my $success = C4::SMS->send_sms({ message => 'This is my text message', =head1 DESCRIPTION +A wrapper for SMS::Send. +Can use a yaml file for config, the path to which is in the koha-conf.xml +__KOHA_CONF_DIR__/sms_send/ + +Each file needs to be in the format of +__KOHA_CONF_DIR__/sms_send/.yaml + +For example for SMS::Send::UK::Kapow the config would be + +/etc/koha/sites/instancename/sms_send/UK/Kapow.yaml for package install +or +/etc/koha/sms_send/UK/Kapow.yaml for tarball + +A underscore character is prepended to all parameter names so they are +treated as driver-specific options (leading underscore must not appear +in config file). =cut @@ -77,10 +95,9 @@ sub send_sms { my $subpath = $driver; $subpath =~ s|::|/|; - my $conf_file = File::Spec->catfile( - C4::Context->config('installdir'), - 'etc', 'sms', 'driver', $subpath - ) . q{.yaml}; + my $conf_file = + File::Spec->catfile( C4::Context->config('sms_send_config'), $subpath ) + . q{.yaml}; my %args; if ( -f $conf_file ) { require YAML; diff --git a/debian/templates/koha-conf-site.xml.in b/debian/templates/koha-conf-site.xml.in index 6bfe004..7f13c64 100644 --- a/debian/templates/koha-conf-site.xml.in +++ b/debian/templates/koha-conf-site.xml.in @@ -311,5 +311,7 @@ __END_SRU_PUBLICSERVER__ /usr/share/fonts/truetype/ttf-dejavu/DejaVuSans-BoldOblique.ttf + + __KOHA_CONF_DIR__/sms_send/ diff --git a/etc/koha-conf.xml b/etc/koha-conf.xml index 1ce3c90..2c188f8 100644 --- a/etc/koha-conf.xml +++ b/etc/koha-conf.xml @@ -138,5 +138,7 @@ __PAZPAR2_TOGGLE_XML_POST__ __FONT_DIR__/DejaVuSans-BoldOblique.ttf + + __KOHA_CONF_DIR__/sms_send/ -- 2.1.4