Bugzilla – Attachment 18327 Details for
Bug 8952
Add http:// to url in social media links on opac-detail
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8952 - Add http:// to url in social media links on opac-detail
Bug-8952---Add-http-to-url-in-social-media-links-o.patch (text/plain), 3.49 KB, created by
Marcel de Rooy
on 2013-05-23 07:43:59 UTC
(
hide
)
Description:
Bug 8952 - Add http:// to url in social media links on opac-detail
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2013-05-23 07:43:59 UTC
Size:
3.49 KB
patch
obsolete
>From a310cb763e050ecc10c973915e89c78a5917f58d Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Wed, 24 Oct 2012 12:30:33 -0400 >Subject: [PATCH] Bug 8952 - Add http:// to url in social media links on > opac-detail >Content-Type: text/plain; charset=utf-8 > >This patch adds a variable to the template, "protocol" based on >whether "using_https" is set. The appropriate protocol is >inserted before social media links. > >To test, confirm that OPACBaseURL is set and lacks "http(s)://" >Social media links should include correct URLs. > >A follow-up patch could copy use of this [% protocol %] variable >in other parts of this template which duplicate large chunks of >HTML based on [% using_https %]. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 13 +++++++++---- > 1 files changed, 9 insertions(+), 4 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >index ca6fb76..ad7e21f 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -16,6 +16,11 @@ > [% END %] > [% END %] > >+[% IF ( using_https ) %] >+ [% SET protocol = "https://" %] >+[% ELSE %] >+ [% SET protocol = "http://" %] >+[% END %] > [% INCLUDE 'doc-head-open.inc' %][% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog › Details for: [% title |html %][% FOREACH subtitl IN subtitle %], [% subtitl.subfield |html %][% END %] > [% INCLUDE 'doc-head-close.inc' %] > <script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.tablesorter.min.js"></script> >@@ -1429,10 +1434,10 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% IF ( SocialNetworks ) %] > <div id="social_networks"> > <span>Share</span> >- <div><a id="facebook" href="http://www.facebook.com/sharer.php?u=[% current_url |url %]&t=[% title |url %]" title="Share on Facebook">Facebook</a></div> >- <div><a id="linkedin" href="http://www.linkedin.com/shareArticle?mini=true&url=[% current_url |url %]&title=[% title |url %]" title="Share on LinkedIn">LinkedIn</a></div> >- <div><a id="delicious" href="http://www.delicious.com/save?url=[% current_url |url %]&title=[% title |url %]" title="Share on Delicious">Delicious</a></div> >- <div><a id="email" href="mailto:?subject=[% title |url %]&body=[% title |url %] ([% current_url |url %])" title="Share by email">Email</a></div> >+ <div><a id="facebook" href="http://www.facebook.com/sharer.php?u=[% protocol %][% current_url |url %]&t=[% title |url %]" title="Share on Facebook">Facebook</a></div> >+ <div><a id="linkedin" href="http://www.linkedin.com/shareArticle?mini=true&url=[% protocol %][% current_url |url %]&title=[% title |url %]" title="Share on LinkedIn">LinkedIn</a></div> >+ <div><a id="delicious" href="http://www.delicious.com/save?url=[% protocol %][% current_url |url %]&title=[% title |url %]" title="Share on Delicious">Delicious</a></div> >+ <div><a id="email" href="mailto:?subject=[% title |url %]&body=[% title |url %] ([% protocol %][% current_url |url %])" title="Share by email">Email</a></div> > <div><g:plusone size="small" annotation="none"></g:plusone></div> > <div><a id="twitter" href="https://twitter.com/share" class="twitter-share-button" data-count="none" data-text="[% title %]" data-lang="[% lang %]">Tweet</a></div> > </div> >-- >1.7.7.6
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 8952
:
13023
|
13033
|
13034
|
14291
|
14292
|
14614
|
16019
|
16020
|
18327
|
18328
|
18353
|
18369
|
21359
|
21360
|
21361