Attachment 91430 Details for Bug 23078 – Bug 23078: (follow-up) Update self checkout help page

[patch] Bug 23078: (follow-up) Update self checkout help page

Bug-23078-follow-up-Update-self-checkout-help-page.patch (text/plain), 2.45 KB, created by Owen Leonard on 2019-07-09 13:56:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2019-07-09 13:56:00 UTC

Size: 2.45 KB

patch

obsolete

>From d3848b8333eb58fc6e5b1787461bec5e4e866d7c Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Tue, 9 Jul 2019 13:51:24 +0000
>Subject: [PATCH] Bug 23078: (follow-up) Update self checkout help page
>
>This patch takes care of some preference instances I missed in the self
>checkout module's help page. It also removes some obsolete lines from
>Auth.pm.
>
>To test, apply the patch and re-test based on the previous test plan.
>Also test in the self-checkout module by logging into self checkout and
>clicking the "Help" link in the upper right of the screen.
>
>The settings of the following preferences should be shown correctly:
>
> - OpacFavicon
> - OPACUserCSS
>---
> C4/Auth.pm                                           | 2 --
> koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/help.tt | 3 ++-
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
>diff --git a/C4/Auth.pm b/C4/Auth.pm
>index ba3c00b5e7..e658c8947c 100644
>--- a/C4/Auth.pm
>+++ b/C4/Auth.pm
>@@ -1237,8 +1237,6 @@ sub checkauth {
>     my $template_name = ( $type eq 'opac' ) ? 'opac-auth.tt' : 'auth.tt';
>     my $template = C4::Templates::gettemplate( $template_name, $type, $query );
>     $template->param(
>-        OpacAdditionalStylesheet                   => C4::Context->preference("OpacAdditionalStylesheet"),
>-        opaclayoutstylesheet                  => C4::Context->preference("opaclayoutstylesheet"),
>         login                                 => 1,
>         INPUTS                                => \@inputs,
>         script_name                           => get_script_name(),
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/help.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/help.tt
>index 185a201aca..ec2bbdfdac 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/help.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/help.tt
>@@ -1,4 +1,5 @@
> [% USE raw %]
>+[% USE Koha %]
> [% USE Asset %]
> [% USE KohaDates %]
> [% INCLUDE 'doc-head-open.inc' %]
>@@ -9,7 +10,7 @@
>  [% Asset.css("lib/bootstrap/css/bootstrap.min.css") | $raw %]
> [% Asset.css("lib/jquery/jquery-ui.css") | $raw %]
> [% Asset.css("css/sco.css") | $raw %]
>-[% IF ( OPACUserCSS ) %]<style>[% OPACUserCSS | $raw %]</style>[% END %]
>+[% IF ( Koha.Preference('OPACUserCSS') ) %]<style>[% Koha.Preference('OPACUserCSS') | $raw %]</style>[% END %]
> [% IF ( SCOUserCSS ) %]<style>[% SCOUserCSS | $raw %]</style>[% END %]
> <!--[if lt IE 9]>
>     <script src="[% interface | html %]/[% theme | html %]/lib/respond.min.js"></script>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23078: 90460 | 90481 | 90936 | 91349 | 91430 | 91431 | 91503 | 91626 | 91627 | 91654 | 91655