From 7d5dc65867a55045fa32189d177379e70ecf5725 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Mon, 28 Aug 2023 10:59:29 +0000
Subject: [PATCH] Bug 34627: CMS pages do not correctly display the opaccredits
 footer

This patch corrects the page structure of CMS pages in the OPAC so that
the contents of the footer, including opaccredits, OpacKohaUrl, and the
language selection bar, are displayed correctly.

This patch contains indentation changes, so ignore whitespace when
viewing the diff.

To test you should have these settings:

- OpacKohaUrl enabled in system preferences
- Some content in the opaccredits HTML customization region
- Some content in the OpacNav HTML customization region
- At least one translation installed and enabled
- The opaclanguagesdisplay preference enabled
- The OpacLangSelectorMode set to "only footer" or "both top and footer"

- Apply the patch and go to Tools -> Pages and create a page with the
  display location "OPAC."
- Open the OPAC link for your newly-created page and confirm that the
  page looks correct:
  - The "main" region with the white background should contain the
    breadcrumb navigation, the sidebar OpacNav content, and the contents
    of your CMS page.
  - The opaccredits and OpacKohaUrl content should appear below that
    region.
  - The language selection footer should be at the bottom of the page.

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 .../bootstrap/en/modules/opac-page.tt         | 42 +++++++++----------
 1 file changed, 19 insertions(+), 23 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-page.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-page.tt
index e9d0c89c70..5b4742da93 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-page.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-page.tt
@@ -32,40 +32,36 @@
 
     <div class="container-fluid">
         <div class="row">
-            [% IF ( OpacNav || OpacNavBottom ) %]
-                <div class="col-12 col-lg-2 order-3 order-lg-1">
+            [% IF ( OpacNav ||  OpacNavBottom ) %]
+                <div class="col-lg-2">
                     <div id="navigation">
-                        [% IF ( OpacNav || OpacNavBottom ) %]
-                            [% INCLUDE 'navigation.inc' %]
-                        [% END %]
+                        [% INCLUDE 'navigation.inc' %]
                     </div>
                 </div>
-                <div class="col-12 col-lg-10 order-md-1 maincontent">
+                <div class="col-10 order-first order-md-first order-lg-2">
             [% ELSE %]
-                <div class="col order-md-1 maincontent">
+                <div class="col order-first order-md-first order-lg-2">
             [% END %]
-                <div id="page_[% page.idnew | html %]" class="maincontent">
 
             [% IF page %]
+                <div class="maincontent">
+                    <h1>[% page.title | html %]</h1>
 
-            <h1>[% page.title | html %]</h1>
+                        <div class="page_content">
+                            [% page.content | $raw %]
+                        </div>
 
-            <div class="page_content">
-                [% page.content | $raw %]
-            </div>
+                    [% ELSE %]
 
-            [% ELSE %]
-
-            <div class="alert alert-error">
-                This page does not exist.
-            </div>
-
-            [% END %]
-
-        </div> <!-- / .col 6/8 -->
+                        <div class="alert alert-error">
+                            This page does not exist.
+                        </div>
 
-        </div> <!-- /.container-fluid -->
-    </div> <!-- /.row -->
+                    [% END %]
+                </div> <!-- /.maincontent -->
+            </div> <!-- / .col -->
+        </div> <!-- /.row -->
+    </div> <!-- /.container-fluid -->
 </div> <!-- /.main -->
 
 [% INCLUDE 'opac-bottom.inc' %]
-- 
2.39.5