From 808f938517e07f0dc75a755c77e0a114edf86ed0 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Wed, 9 Mar 2016 15:41:02 -0300 Subject: [PATCH] Bug 16031: sitemap.pl shouldn't append protocol to OPACBaseURL 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 Signed-off-by: Jonathan Druart Signed-off-by: Brendan A Gallagher (cherry picked from commit 41657a87f6981516affff2f482084c7104aa34cb) Signed-off-by: Julian Maurice --- misc/cronjobs/sitemap.pl | 1 - 1 file changed, 1 deletion(-) diff --git a/misc/cronjobs/sitemap.pl b/misc/cronjobs/sitemap.pl index 0d615310d0..bc1bca79c9 100755 --- a/misc/cronjobs/sitemap.pl +++ b/misc/cronjobs/sitemap.pl @@ -49,7 +49,6 @@ unless ($url) { say "OPACBaseURL syspref isn't defined. You can use --url parameter."; exit; } - $url = 'http://' . $url; } $url =~ s/\/*$//g; -- 2.39.5