Bug 16031: sitemap.pl shouldn't append protocol to OPACBaseURL
authorTomas Cohen Arazi <tomascohen@theke.io>
Wed, 9 Mar 2016 18:41:02 +0000 (15:41 -0300)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Fri, 11 Mar 2016 15:28:49 +0000 (15:28 +0000)
Since bug 5010 was pushed, OPACBaseURL already contains the protocol. The
sitemap.pl script was written before this was pushed, and thus still concatenates
http:// in front of OPACBaseURL.

This patch removes this behaviour.

To test:
- Have OPACBaseURL set to (say) http://myopac.com
- Run the sitemap.pl script without specifying the --url param
=> FAIL: Notice URLs look like http://http://myopac.com/bib... in the sitemap files.
- Apply the patch
- Run the sitemap.pl script without specifying the --url param
=> SUCCESS: Notice URLs look correctly like http://myopac.com/bib...
- Sign off :-D

Regards

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
misc/cronjobs/sitemap.pl

index 0d61531..bc1bca7 100755 (executable)
@@ -49,7 +49,6 @@ unless ($url) {
         say "OPACBaseURL syspref isn't defined. You can use --url parameter.";
         exit;
     }
-    $url = 'http://' . $url;
 }
 $url =~ s/\/*$//g;