Attachment 158453 Details for Bug 35048 – Bug 35048: Convert SCOMainUserBlock system preference to HTML customization

[patch] Bug 35048: Convert SCOMainUserBlock system preference to HTML customization

Bug-35048-Convert-SCOMainUserBlock-system-preferen.patch (text/plain), 10.21 KB, created by Katrin Fischer on 2023-11-05 11:17:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2023-11-05 11:17:05 UTC

Size: 10.21 KB

patch

obsolete

>From b50e67eeb8615784eb6c994c30123252529e943f Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 12 Oct 2023 18:01:04 +0000
>Subject: [PATCH] Bug 35048: Convert SCOMainUserBlock system preference to HTML
> customization
>
>This patch moves the SCOMainUserBlock system preference into HTML
>customizations, making it possible to have language- and
>library-specific content.
>
>The patch also makes a minor fix to CSS in order to get the "Back to
>top" link to show up correctly in self-checkout and self-checkin.
>
>To test you should have some content in the SCOMainUserBlock
>system preference before applying the patch. Apply the patch, run the
>database update process, and rebuild the OPAC CSS.
>
>- In the staff client, go to Tools -> HTML customizations and verify
>  that the content from SCOMainUserBlock is now stored there.
>- The HTML customization entry form should offer SCOMainUserBlock
>  as a choice under "Display location."
>- Update and reinstall active translations (for instance fr-FR):
>  - perl misc/translator/translate update fr-FR
>  - perl misc/translator/translate install fr-FR
>- Enable the translation if necessary under Administration -> System
>  preferences -> language.
>- Enable the "opaclanguagesdisplay" preference if necessary.
>- Edit the SCOMainUserBlock HTML customization and add unique
>  content to the "fr-FR" tab.
>
>- Log into the self checkout system and confirm that the
>  SCOMainUserBlock content is shown there.
>- Switch to your updated translation and confirm that the content you
>  added for your translation shows up correctly.
>- Go to Administration -> System preferences and search for
>  "SCOMainUserBlock." It should return no results.
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> ...SCOMainUserBlock-to-additional-contents.pl | 38 +++++++++++++++++++
> installer/data/mysql/mandatory/sysprefs.sql   |  1 -
> .../admin/preferences/circulation.pref        |  6 ---
> .../en/modules/tools/additional-contents.tt   |  2 +-
> .../opac-tmpl/bootstrap/css/src/_common.scss  | 10 +++++
> .../opac-tmpl/bootstrap/css/src/opac.scss     | 10 -----
> .../bootstrap/en/modules/sco/sco-main.tt      |  8 +++-
> 7 files changed, 56 insertions(+), 19 deletions(-)
> create mode 100755 installer/data/mysql/atomicupdate/bug-35048-move-SCOMainUserBlock-to-additional-contents.pl
>
>diff --git a/installer/data/mysql/atomicupdate/bug-35048-move-SCOMainUserBlock-to-additional-contents.pl b/installer/data/mysql/atomicupdate/bug-35048-move-SCOMainUserBlock-to-additional-contents.pl
>new file mode 100755
>index 0000000000..faed6465e8
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/bug-35048-move-SCOMainUserBlock-to-additional-contents.pl
>@@ -0,0 +1,38 @@
>+use Modern::Perl;
>+
>+return {
>+    bug_number  => "35048",
>+    description => "Move SCOMainUserBlock to additional contents",
>+    up          => sub {
>+        my ($args) = @_;
>+        my ( $dbh, $out ) = @$args{qw(dbh out)};
>+
>+        # Get any existing value from the SCOMainUserBlock system preference
>+        my ($scomainuserblock) = $dbh->selectrow_array(
>+            q|
>+            SELECT value FROM systempreferences WHERE variable='SCOMainUserBlock';
>+        |
>+        );
>+        if ($scomainuserblock) {
>+
>+            # Insert any values found from system preference into additional_contents
>+            $dbh->do(
>+                "INSERT INTO additional_contents ( category, code, location, branchcode, published_on ) VALUES ('html_customizations', 'SCOMainUserBlock', 'SCOMainUserBlock', NULL, CAST(NOW() AS date) )"
>+            );
>+
>+            my ($insert_id) = $dbh->selectrow_array(
>+                "SELECT id FROM additional_contents WHERE category = 'html_customizations' AND code = 'SCOMainUserBlock' AND location = 'SCOMainUserBlock' LIMIT 1",
>+                {}
>+            );
>+
>+            $dbh->do(
>+                "INSERT INTO additional_contents_localizations ( additional_content_id, title, content, lang ) VALUES ( ?, 'SCOMainUserBlock default', ?, 'default' )",
>+                undef, $insert_id, $scomainuserblock
>+            );
>+
>+            # Remove old system preference
>+            $dbh->do("DELETE FROM systempreferences WHERE variable='SCOMainUserBlock'");
>+            say $out "Bug 35048 update done";
>+        }
>+    }
>+    }
>diff --git a/installer/data/mysql/mandatory/sysprefs.sql b/installer/data/mysql/mandatory/sysprefs.sql
>index 06755e89d5..f13e46c4e7 100644
>--- a/installer/data/mysql/mandatory/sysprefs.sql
>+++ b/installer/data/mysql/mandatory/sysprefs.sql
>@@ -649,7 +649,6 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `
> ('RoutingSerials','1',NULL,'If ON, serials routing is enabled','YesNo'),
> ('SavedSearchFilters', '0', NULL, 'Allow staff with permission to create/edit custom search filters', 'YesNo'),
> ('SCOAllowCheckin','0','','If enabled, patrons may return items through the Web-based Self Checkout','YesNo'),
>-('SCOMainUserBlock','','70|10','Add a block of HTML that will display on the self checkout screen','Textarea'),
> ('SCOUserCSS','',NULL,'Add CSS to be included in the SCO module in an embedded <style> tag.','free'),
> ('SCOUserJS','',NULL,'Define custom javascript for inclusion in the SCO module','free'),
> ('SearchEngine','Zebra','Elasticsearch|Zebra','Search Engine','Choice'),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>index 5cb207d3f0..8141b0539a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref
>@@ -1127,12 +1127,6 @@ Circulation:
>               type: textarea
>               syntax: javascript
>               class: code
>-        -
>-            - "Include the following HTML on the the web-based self checkout screen:"
>-            - pref: SCOMainUserBlock
>-              type: textarea
>-              syntax: text/html
>-              class: code
>         -
>             - "Include the following CSS on all pages in the web-based self checkout:"
>             - pref: SCOUserCSS
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/additional-contents.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/additional-contents.tt
>index 3f151c18cb..d2e5b591d2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/additional-contents.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/additional-contents.tt
>@@ -524,7 +524,7 @@
>             [% END %]
>         [% END %]
>     [% ELSE %]
>-        [% SET opac_available_options = [ 'OpacNavRight', 'opacheader', 'OpacCustomSearch', 'OpacMainUserBlock', 'opaccredits', 'OpacLoginInstructions', 'OpacNav', 'OpacNavBottom', 'OpacSuggestionInstructions', 'ArticleRequestsDisclaimerText', 'OpacMoreSearches', 'OpacMySummaryNote', 'OpacLibraryInfo', 'OpacMaintenanceNotice', 'OPACResultsSidebar', 'OpacSuppressionMessage', 'CatalogConcernHelp', 'CatalogConcernTemplate', 'CookieConsentBar', 'CookieConsentPopup', 'PatronSelfRegistrationAdditionalInstructions' ] %]
>+        [% SET opac_available_options = [ 'OpacNavRight', 'opacheader', 'OpacCustomSearch', 'OpacMainUserBlock', 'opaccredits', 'OpacLoginInstructions', 'OpacNav', 'OpacNavBottom', 'OpacSuggestionInstructions', 'ArticleRequestsDisclaimerText', 'OpacMoreSearches', 'OpacMySummaryNote', 'OpacLibraryInfo', 'OpacMaintenanceNotice', 'OPACResultsSidebar', 'OpacSuppressionMessage', 'SCOMainUserBlock', 'CatalogConcernHelp', 'CatalogConcernTemplate', 'CookieConsentBar', 'CookieConsentPopup', 'PatronSelfRegistrationAdditionalInstructions' ] %]
>         <optgroup label="OPAC">
>             [% FOREACH l IN opac_available_options.sort %]
>                 [% IF l == location %]
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss b/koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss
>index 8c318e25ab..6dbc6cc393 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss
>+++ b/koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss
>@@ -848,6 +848,16 @@ button {
>     }
> }
> 
>+#backtotop {
>+    bottom: 40px;
>+    display: inline-block;
>+    font-size: 120%;
>+    position: fixed;
>+    right: 40px;
>+    text-align: center;
>+    z-index: 1000;
>+}
>+
> #logo {
>     background: transparent url( "../images/koha-green-logo.svg" ) no-repeat scroll 0%;
>     border: 0;
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>index ae849a8af7..2b5becef5c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>+++ b/koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss
>@@ -245,16 +245,6 @@ th {
>     }
> }
> 
>-#backtotop {
>-    bottom: 40px;
>-    display: inline-block;
>-    font-size: 120%;
>-    position: fixed;
>-    right: 40px;
>-    text-align: center;
>-    z-index: 1000;
>-}
>-
> #booleansearch {
>     width: 80%;
> }
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>index dff5a86237..a54d047a17 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>@@ -5,6 +5,8 @@
> [% USE AudioAlerts %]
> [% USE To %]
> [% USE Price %]
>+[% USE AdditionalContents %]
>+[% SET SCOMainUserBlock = AdditionalContents.get( location => "SCOMainUserBlock", lang => lang, library => logged_in_user.branchcode || default_branch, blocktitle => 0 ) %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>Self checkout &rsaquo; [% IF ( LibraryNameTitle ) %][% LibraryNameTitle | html %][% ELSE %]Koha [% END %]</title>
> <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
>@@ -449,7 +451,11 @@
>                             </div> 
>                         [% END # / IF validuser %]
>                     [% END # / UNLESS ( hide_main %]
>-                    [% IF ( Koha.Preference('SCOMainUserBlock' ) ) %]<div id="scomainuserblock">[% Koha.Preference('SCOMainUserBlock' ) | $raw %]</div>[% END %]
>+                    [% IF ( SCOMainUserBlock ) %]
>+                        <div id="scomainuserblock">
>+                            [% PROCESS koha_news_block news => SCOMainUserBlock %]
>+                        </div>
>+                    [% END %]
>                 </div> 
>             </div> 
>         </div> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35048: 157104 | 157105 | 157559 | 157831 | 157832 | 157918 | 157919 | 158453 | 158454 | 158722 | 158723 | 158724