Attachment 4064 Details for Bug 6323 – Proposed patch

[patch] Proposed patch

0003-Bug-6323-Attach-move-items-error-handling-clean.patch (text/plain), 1.97 KB, created by Liz Rea on 2011-05-09 15:45:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Liz Rea

Created: 2011-05-09 15:45:34 UTC

Size: 1.97 KB

patch

obsolete

>From d83e411492e30ad8e05b3239fc91b6967c667705 Mon Sep 17 00:00:00 2001
>From: Liz Rea <lrea@nekls.org>
>Date: Mon, 9 May 2011 10:19:35 -0500
>Subject: [PATCH] Bug 6323 - Attach/move items -- error handling cleanup
>Content-Type: text/plain; charset="utf-8"
>
>When an item move fails using Attach Item, the current behavior is to send the user back to the
>MARC detail page only. I believe there should be an option to "try with a
>different barcode." This patch adds that functionality.
>---
> .../prog/en/modules/cataloguing/moveitem.tt        |   12 ++++++++----
> 1 files changed, 8 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/moveitem.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/moveitem.tt
>index 4956f18..80bcfc0 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/moveitem.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/moveitem.tt
>@@ -14,10 +14,14 @@
> 	[% IF ( errornoitem ) %]ERROR: Unable to get the item.[% END %]
> 	[% IF ( errornoitemnumber ) %]ERROR: Unable to get the item number from this barcode.[% END %]
>     </div>
>-    <form action="/cgi-bin/koha/catalogue/MARCdetail.pl" method="post">
>-	<input type="submit" value="OK" />
>-	<input type="hidden" name="biblionumber" value="[% biblionumber %]" />
>-    </form>
>+	<form action="/cgi-bin/koha/catalogue/detail.pl" method="post">
>+	    <input type="submit"  class="approve" value="OK" />
>+	    <input type="hidden" name="biblionumber" value="[% biblionumber %]" />
>+	</form>
>+	<form method="link" action="/cgi-bin/koha/cataloguing/moveitem.pl?biblionumber=[% biblionumber %]"/>
>+	    <input type="submit" class="new" value="Try a different barcode"/>
>+	    <input type="hidden" name="biblionumber" value="[% biblionumber %]" />
>+	</form>
> [% ELSE %]
>     [% IF ( success ) %]
> 	<div class="dialog message">The item has successfully been attached to [% INCLUDE 'biblio-default-view.inc' %]<i>[% bibliotitle |html %]</i></a>.[% IF ( BiblioDefaultViewmarc ) %]
>-- 
>1.5.6.5
>

Actions: View | Diff | Splinter Review

Attachments on bug 6323: 4064 | 4072 | 5628 | 6249 | 6331 | 6332 | 6870 | 6872 | 6874 | 6875 | 6908 | 6927