Bug 6861: Fast Add returns to circ/circulation.pl in all cases
authorIan Walls <ian.walls@bywatersolutions.com>
Fri, 9 Sep 2011 10:55:59 +0000 (06:55 -0400)
committerChris Cormack <chrisc@catalyst.net.nz>
Sun, 11 Sep 2011 09:32:30 +0000 (21:32 +1200)
If Fast Add is accessed from either the circulation homepage or cataloguing,
it should stay on the additem.pl page after saving the first item.  Only when
a Fast Add is done in the middle of a circulation transaction should it redirect
back to circ/circulation.pl to complete that transaction.

This patch adds a conditional check for a borrowernumber input param; the presense
of such implies we're in the middle of a checkout.  It's absense implies we should
remain on additem.pl

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
cataloguing/additem.pl

index 275e236..0db2d2d 100755 (executable)
@@ -422,7 +422,7 @@ if ($op eq "additem") {
            undef($itemrecord);
        }
     }  
-    if ($frameworkcode eq 'FA'){
+    if ($frameworkcode eq 'FA' && $input->param('borrowernumber')){
        my $redirect_string = 'borrowernumber=' . uri_escape($input->param('borrowernumber')) .
          '&barcode=' . uri_escape($input->param('barcode'));
        $redirect_string .= '&duedatespec=' . uri_escape($input->param('duedatespec')) .