Attachment 136505 Details for Bug 10086 – Bug 10086: Link to go to basket from uncertain prices page

[patch] Bug 10086: Link to go to basket from uncertain prices page

Bug-10086-Link-to-go-to-basket-from-uncertain-pric.patch (text/plain), 3.31 KB, created by Nick Clemens (kidclamp) on 2022-06-24 14:45:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2022-06-24 14:45:18 UTC

Size: 3.31 KB

patch

obsolete

>From 9e81ff8ff9a8b384855dde8b22fad7a247f86842 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Mon, 13 Feb 2017 22:53:10 +0000
>Subject: [PATCH] Bug 10086: Link to go to basket from uncertain prices page
>
>This patch moves the Edit button into an Edit column, styled with
>bootstrap button and font awesome icon, and adds a link on the basket
>name to go to the basket.
>
>To test;
>1) Add an item with uncertain prices to a basket
>2) Go to the vendor of the basket
>3) Click Uncertain prices on the left
>4) Notice 'edit' button does not look like a button, and basket name is
>not a link
>5) Apply patch and refresh page
>6) Basket name is now a link. Click to confirm it takes you to the right
>page
>7) Edit button is now in edit column. Confirm it works as expected
>8) Confirm you cannot sort on Edit column and the button does not wrap
>on a narrower browser
>
>Sponsored-by: Catalyst IT
>---
> .../prog/en/modules/acqui/uncertainprice.tt           | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt
>index 1243a06d13..194d837b10 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/uncertainprice.tt
>@@ -102,6 +102,7 @@
> 	    <tr>
>             <th>Basket</th>
>            <th>[% tp('noun', 'Order') | html %]</th>
>+            <th>Edit</th>
>             <th>By</th>
>             <th>Uncertain</th>
>             <th>Price</th>
>@@ -112,7 +113,7 @@
> 	    [% FOREACH uncertainpriceorder IN uncertainpriceorders %]
> 	    <tr>
>         <td>
>-            [% uncertainpriceorder.basketname | html %]
>+            <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% uncertainpriceorder.basketno %]">[% uncertainpriceorder.basketname %]</a>
>         </td>
> 	    <td>
>             <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% uncertainpriceorder.biblionumber | uri %]">[% uncertainpriceorder.title | html %]</a>
>@@ -125,9 +126,9 @@
>             [% END %]
>             [% IF ( uncertainpriceorder.isbn ) %]<br />[% uncertainpriceorder.isbn | html %][% END %]
>             <br />
>-            <a href="neworderempty.pl?ordernumber=[% uncertainpriceorder.ordernumber | uri %]&amp;booksellerid=[% booksellerid | uri %]&amp;basketno=[% uncertainpriceorder.basketno | uri %]">
>-                Edit
>-            </a>
>+        </td>
>+        <td class="actions">
>+            <a href="/cgi-bin/koha/acqui/neworderempty.pl?ordernumber=[% uncertainpriceorder.ordernumber %]&amp;booksellerid=[% booksellerid %]&amp;basketno=[% uncertainpriceorder.basketno %]" class="btn btn-default btn-xs"><i class="fa fa-pencil"></i> Edit</a>
>         </td>
>         <td>
>             [% uncertainpriceorder.firstname | html %] [% uncertainpriceorder.surname | html %]
>@@ -172,7 +173,7 @@
>         $(document).ready(function() {
>             var uncertainpricet = $("#uncertainpricet").dataTable($.extend(true, {}, dataTablesDefaults, {
>                 "aoColumnDefs": [
>-                    { "aTargets": [ -1, -2, -3 ], "bSortable": false, "bSearchable": false },
>+                    { "aTargets": [ 2, -1, -2, -3 ], "bSortable": false, "bSearchable": false },
>                 ],
>                 "sPaginationType": "full"
>             }));
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 10086: 60167 | 136471 | 136505 | 136506 | 137681 | 137737 | 138854 | 138855