Attachment 54417 Details for Bug 17103 – Bug 17103 - Google API Loader jsapi called over http

[patch] Bug 17103 - Google API Loader jsapi called over http

Bug-17103---Google-API-Loader-jsapi-called-over-ht.patch (text/plain), 1.77 KB, created by Indranil Das Gupta on 2016-08-13 00:29:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Indranil Das Gupta

Created: 2016-08-13 00:29:39 UTC

Size: 1.77 KB

patch

obsolete

>From 55a254c7a297437e59b7c4bcf02c0ce5eb75834b Mon Sep 17 00:00:00 2001
>From: "Indranil Das Gupta (L2C2 Technologies)" <indradg@l2c2.co.in>
>Date: Sat, 13 Aug 2016 05:55:14 +0530
>Subject: [PATCH] Bug 17103 - Google API Loader jsapi called over http
>
>This patch removes the hardcoded protocol ("http") from the URI,
>leaving the browser to negotiate the correct http/https context.
>Prevents MIXED CONTENT failure, allowing GoogleIndicTransliteration
>to function correctly when used in SSLized OPACs.
>
>Test plan
>---------
>
>1/ Make sure your OPAC is being served both over HTTP and HTTPS
>2/ Set GoogleIndicTransliteration syspref to "Show"
>3/ Access the OPAC over http, Google transliteration element will
>   show up in masthead searchbox.
>4/ Access the OPAC over https. The element will not show. Console
>   will show MIXED CONTENT error and failure of google loader.
>5/ Apply patch. Repeat steps #3 and #4 again. Now in both cases
>   GoogleIndicTransliteration will work.
>---
> koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>index a22be69..3348c21 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
>@@ -209,7 +209,7 @@ $.widget.bridge('uitooltip', $.ui.tooltip);
>     </script>
> [% END %]
> [% IF ( GoogleIndicTransliteration ) %]
>-    <script type="text/javascript" src="//www.google.com/jsapi"></script>
>+    <script type="text/javascript" src="https://www.google.com/jsapi"></script>
>     <script type="text/javascript" src="[% interface %]/[% theme %]/js/googleindictransliteration.js"></script>
> [% END %]
> [% IF persona %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 17103: 54321 | 54417 | 54418 | 54420 | 54427