Attachment 145369 Details for Bug 32658 – Bug 32658: Use template wrapper in order from staged file template

[patch] Bug 32658: Use template wrapper in order from staged file template

Bug-32658-Use-template-wrapper-in-order-from-stage.patch (text/plain), 5.47 KB, created by Owen Leonard on 2023-01-17 19:38:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2023-01-17 19:38:15 UTC

Size: 5.47 KB

patch

obsolete

>From 2dd0246ac00da006e046c0da3ce3e2d376a1c045 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Tue, 17 Jan 2023 19:28:04 +0000
>Subject: [PATCH] Bug 32658: Use template wrapper in order from staged file
> template
>
>This patch implements the template WRAPPER system (see Bug 32571) for
>building tabs on the page for adding to a basket from a staged
>file.
>
>To test you need an active vendor, at least one basket for that vendor,
>and a staged MARC file.
>
>Apply the patch and go to Acquisitions -> Vendor -> Basket -> Add to
>basket -> From a staged file.
>
>- Click "Add to orders" on one of your staged files.
>- On the "Add orders from..." page you should see three tabs, the first
>  one active.
>- Tabs should look correct and work correctly.
>---
> .../prog/en/modules/acqui/addorderiso2709.tt  | 30 +++++++++----------
> 1 file changed, 15 insertions(+), 15 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt
>index 6151926baf..483ff459ec 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt
>@@ -69,15 +69,15 @@
>                         ([% file_name | html %] staged on [% upload_timestamp | $KohaDates  with_hours => 1 %])
>                     </h1>
>                     <form action="/cgi-bin/koha/acqui/addorderiso2709.pl" method="post" id="Aform">
>-                        <div id="tabs" class="toptabs">
>-                            <ul class="nav nav-tabs" role="tablist">
>-                                <li role="presentation" class="active"><a href="#records_to_import" aria-controls="records_to_import" role="tab" data-toggle="tab">Select to import</a></li>
>-                                <li role="presentation"><a href="#items_info" class="items_info" aria-controls="items_info" role="tab" data-toggle="tab">Item information</a></li>
>-                                <li role="presentation"><a href="#accounting_details" aria-controls="accounting_details" role="tab" data-toggle="tab">Default accounting details</a></li>
>-                            </ul>
>+                        [% WRAPPER tabs id= "tabs" %]
>+                            [% WRAPPER tabs_nav %]
>+                                [% WRAPPER tab_item tabname= "records_to_import" active= 1 %] Select to import [% END %]
>+                                [% WRAPPER tab_item tabname= "items_info" %] Item information [% END %]
>+                                [% WRAPPER tab_item tabname= "accounting_details" %] Default accounting details [% END %]
>+                            [% END %]
> 
>-                            <div class="tab-content">
>-                                <div id="records_to_import" role="tabpanel" class="tab-pane active" >
>+                            [% WRAPPER tab_panels %]
>+                                [% WRAPPER tab_panel tabname= "records_to_import" active= 1 %]
>                                     <div id="searchheader" class="searchheader">
>                                         <div>
>                                             <span class="checkall"><a id="checkAll" href="#">Select all</a></span>
>@@ -366,9 +366,9 @@
>                                             </div> 
>                                         </div> 
>                                     </div> 
>-                                </div> 
>+                                [% END # /#records_to_import %]
> 
>-                                <div id="items_info" role="tabpanel" class="tab-pane">
>+                                [% WRAPPER tab_panel tabname= "items_info" %]
>                                     <h2>Item information</h2>
>                                     <p>Import all the checked items in the basket with the following parameters:</p>
> 
>@@ -423,9 +423,9 @@
>                                                 </div> 
>                                             </fieldset>
>                                     [% END # /IF items %]
>-                                </div> 
>+                                [% END # /#items_info %]
> 
>-                                <div id="accounting_details" role="tabpanel" class="tab-pane">
>+                                [% WRAPPER tab_panel tabname= "accounting_details" %]
>                                     <p>Import all the checked items in the basket with the following accounting details (used only if no information is filled for the item):</p>
>                                     <fieldset class="rows" style="float:none;">
>                                         <legend>Accounting details</legend>
>@@ -489,9 +489,9 @@
>                                             </li>
>                                         </ol>
>                                     </fieldset> 
>-                                </div> 
>-                        </div> 
>-
>+                                [% END # /#accounting_details %]
>+                            [% END #/WRAPPER tab_panels %]
>+                        [% END # /WRAPPER tabs %]
>                         <fieldset class="action">
>                             <input type="submit" id="add_order" class="btn btn-primary" value="Save" /><a class="cancel" href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketno | html %]">Cancel</a>
>                         </fieldset>
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 32658: 145369 | 145537 | 145568 | 145569 | 146289 | 146290 | 147371 | 147372