Attachment 150427 Details for Bug 33596 – Bug 33596: Merge result page is missing page-section

[patch] Bug 33596: Merge result page is missing page-section

Bug-33596-Merge-result-page-is-missing-page-sectio.patch (text/plain), 5.26 KB, created by Nick Clemens (kidclamp) on 2023-04-29 20:12:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2023-04-29 20:12:19 UTC

Size: 5.26 KB

patch

obsolete

>From abecf042759524f6d59b8a5fc2f25df1d83f0f0d Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Mon, 24 Apr 2023 19:09:00 +0000
>Subject: [PATCH] Bug 33596: Merge result page is missing page-section
>
>This patch adds a "page-section" container around the contents of the
>page seen when you complete the process of merging catalog records.
>
>The patch also adds the "merge was successful" message into the standard
>message dialog.
>
>Perform a catalog search which will return more than one record.
>
>- Select two records in your search results and click "Edit" -> "Merge
>  records."
>- Complete the process of merging your selected records.
>- On the last page you should see a "merge was successful" message
>  styled as the standard information box.
>- The "Report" section should be shown inside a "page-section" div.
>
>Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
>Signed-off-by: David Nind <david@davidnind.com>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> .../prog/en/modules/cataloguing/merge.tt      | 76 ++++++++++---------
> 1 file changed, 39 insertions(+), 37 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
>index 9e3176616e..46b8e3b103 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/merge.tt
>@@ -57,48 +57,50 @@ div#result { margin-top: 1em; }
>        [% END %]
> 
>     [% ELSE %]
>-        <p>The merge was successful. <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=[% ref_biblionumber | uri %]">View the merged record.</a></p>
>-        <h3>Report</h3>
>-        <table>
>-            <thead>
>-                <tr>
>-                    <th>Biblionumber</th>
>-                    [% FOREACH key IN report_header.keys.sort %]
>-                        [% tag = key.substr(0, 3) %]
>-                        [% code = key.substr(3, 1) %]
>-                        [% IF code == '@' %]
>-                            [% header = tag %]
>-                        [% ELSE %]
>-                            [% header = tag _ '$' _ code %]
>-                        [% END %]
>-                        <th>[% header | html %]</th>
>-                    [% END %]
>-                </tr>
>-            </thead>
>-            <tbody>
>-                [% FOREACH record IN report_records %]
>+        <div class="dialog message">The merge was successful. <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=[% ref_biblionumber | uri %]">View the merged record.</a></div>
>+        <div class="page-section">
>+            <h3>Report</h3>
>+            <table>
>+                <thead>
>                     <tr>
>-                        <td>
>-                            [% record.biblionumber | html %]
>-                            [% IF loop.first %]
>-                                <span>(record kept)</span>
>-                            [% END %]
>-                        </td>
>+                        <th>Biblionumber</th>
>                         [% FOREACH key IN report_header.keys.sort %]
>+                            [% tag = key.substr(0, 3) %]
>+                            [% code = key.substr(3, 1) %]
>+                            [% IF code == '@' %]
>+                                [% header = tag %]
>+                            [% ELSE %]
>+                                [% header = tag _ '$' _ code %]
>+                            [% END %]
>+                            <th>[% header | html %]</th>
>+                        [% END %]
>+                    </tr>
>+                </thead>
>+                <tbody>
>+                    [% FOREACH record IN report_records %]
>+                        <tr>
>                             <td>
>-                                [% values = record.fields.$key %]
>-                                [% IF values %]
>-                                    [% FOREACH value IN record.fields.$key %]
>-                                        [% value | html %]
>-                                        [% UNLESS loop.last %]<br />[% END %]
>-                                    [% END %]
>+                                [% record.biblionumber | html %]
>+                                [% IF loop.first %]
>+                                    <span>(record kept)</span>
>                                 [% END %]
>                             </td>
>-                        [% END %]
>-                    </tr>
>-                [% END %]
>-            </tbody>
>-        </table>
>+                            [% FOREACH key IN report_header.keys.sort %]
>+                                <td>
>+                                    [% values = record.fields.$key %]
>+                                    [% IF values %]
>+                                        [% FOREACH value IN record.fields.$key %]
>+                                            [% value | html %]
>+                                            [% UNLESS loop.last %]<br />[% END %]
>+                                        [% END %]
>+                                    [% END %]
>+                                </td>
>+                            [% END %]
>+                        </tr>
>+                    [% END %]
>+                </tbody>
>+            </table>
>+        </div> 
>     [% END %]
> 
> [% ELSE %]
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 33596: 150172 | 150175 | 150177 | 150427