Attachment 15998 Details for Bug 8853 – [PATCH] Bug 8853: Suppress links in basketgroups list

[patch] [PATCH] Bug 8853: Suppress links in basketgroups list

0005-patch.patch (text/plain), 2.49 KB, created by Mathieu Saby on 2013-03-09 15:23:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mathieu Saby

Created: 2013-03-09 15:23:06 UTC

Size: 2.49 KB

patch

obsolete

>From ddf869ca3918e2a825925bad91aadc6a630a3d43 Mon Sep 17 00:00:00 2001
>From: Mathieu Saby <mathieu.saby@univ-rennes2.fr>
>Date: Sat, 9 Mar 2013 16:15:19 +0100
>Subject: [PATCH] Bug 8853: Suppress links in basketgroups list
>
>In the table listing all closed basketgroups, clicking the name of a basketgroup reopen it.
>To edit an open basketgroup or reopen a closed basketgroup, there are specific buttons in "Action" column.
>So the link around basketgroup name is an UX problem.
>This patch suppress the link around basketgroup name, in both "closed" and "open" tab.
>
>To test :
>1. create some closed and open basketgroups
>2. check links are not showing anymore
>
>---
> .../prog/en/modules/acqui/basketgroup.tt           |    8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>index d8be02a..b0a7a92 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basketgroup.tt
>@@ -275,11 +275,11 @@ function yuiToolbar() {
> 			[% FOREACH basketgroup IN basketgroups %]
> 				[% UNLESS ( basketgroup.closed ) %]
> 					<tr>
>-						<td><a href="/cgi-bin/koha/acqui/basketgroup.pl?op=add&amp;booksellerid=[% basketgroup.booksellerid %]&amp;basketgroupid=[% basketgroup.id %]">[% IF ( basketgroup.name ) %]
>+						<td>[% IF ( basketgroup.name ) %]
> 													[% basketgroup.name %]
> 												[% ELSE %]
> 													Basket group no. [% basketgroup.id %]
>-												[% END %]</a>
>+												[% END %]
> 						</td>
> 							<td>
>                               <span>
>@@ -306,11 +306,11 @@ function yuiToolbar() {
> 				[% IF ( basketgroup.closed ) %]
> 				<tr>
> 				<td>
>-                    <a href="/cgi-bin/koha/acqui/basketgroup.pl?op=reopen&amp;booksellerid=[% basketgroup.booksellerid %]&amp;basketgroupid=[% basketgroup.id %]">[% IF ( basketgroup.name ) %]
>+                                     [% IF ( basketgroup.name ) %]
> 											[% basketgroup.name %]
> 										[% ELSE %]
> 											Basket group no. [% basketgroup.id %]
>-										[% END %]</a>
>+										[% END %]
> 					</td>
> 					<td>
> 							<form action="/cgi-bin/koha/acqui/basketgroup.pl" method="get"><input type="hidden" name="op" value="reopen" /><input type="hidden" name="booksellerid" value="[% basketgroup.booksellerid %]" /><input type="hidden" name="basketgroupid" value="[% basketgroup.id %]" /><input type="submit" value="Reopen" /></form>
>-- 
>1.7.9.5
>

Actions: View | Diff | Splinter Review

Attachments on bug 8853: 15998 | 16038 | 16040