Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 16031: sitemap.pl shouldn't append protocol to OPACBaseURL

Browse source 
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>
This commit is contained in:
Tomás Cohen Arazi 2016-03-09 15:41:02 -03:00 committed by Brendan A Gallagher
parent 54214c4016
commit 41657a87f6
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
misc/cronjobs/sitemap.pl
View file

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