From aa8c40120bdd5a50a4d8fbdae8c54096d1c4a1e3 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 27 Mar 2012 12:01:33 -0400 Subject: [PATCH] Bug 7485 - Cannot edit barcode on Fast Add Simple fix, a special case was added for the fast add framework in the case that an item was being checked out that had not been catalogued. In that case, the barcode was being passed as a parameter through addbiblio.pl to additem.pl. This barcode param was used to overwrite the existing value, so if no barcode param was passed ( which is only the case with fast adds from the circ screen ), the barcode field would appear empty in the items editor. This commit causes that special case code to check for an existing barcode value first, and only use the cgi parameter 'branch' in the case that there is no existing barcode value. Signed-off-by: Liz Rea Editing barcode of fast added item now works. Passes t xt Recommend this for 3.6 release. Signed-off-by: Paul Poulain --- cataloguing/additem.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl index 91dc9e7781..9c8120e1d8 100755 --- a/cataloguing/additem.pl +++ b/cataloguing/additem.pl @@ -147,7 +147,7 @@ sub generate_subfield_form { } } - if ($frameworkcode eq 'FA' && $subfieldlib->{kohafield} eq 'items.barcode'){ + if ($frameworkcode eq 'FA' && $subfieldlib->{kohafield} eq 'items.barcode' && !$value){ my $input = new CGI; $value = $input->param('barcode'); } -- 2.39.5