Bug 27526: Fix encoding issue on subfield
[koha.git] / cataloguing / additem.pl
1 #!/usr/bin/perl
2
3 # Copyright 2000-2002 Katipo Communications
4 # Copyright 2004-2010 BibLibre
5 # Parts Copyright Catalyst IT 2011
6 #
7 # This file is part of Koha.
8 #
9 # Koha is free software; you can redistribute it and/or modify it
10 # under the terms of the GNU General Public License as published by
11 # the Free Software Foundation; either version 3 of the License, or
12 # (at your option) any later version.
13 #
14 # Koha is distributed in the hope that it will be useful, but
15 # WITHOUT ANY WARRANTY; without even the implied warranty of
16 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
17 # GNU General Public License for more details.
18 #
19 # You should have received a copy of the GNU General Public License
20 # along with Koha; if not, see <http://www.gnu.org/licenses>.
21
22 use Modern::Perl;
23
24 use CGI qw ( -utf8 );
25 use C4::Auth qw( get_template_and_user haspermission );
26 use C4::Output qw( output_and_exit_if_error output_and_exit output_html_with_http_headers );
27 use C4::Biblio qw(
28     GetFrameworkCode
29     GetMarcBiblio
30     GetMarcFromKohaField
31     GetMarcStructure
32     IsMarcStructureInternal
33     ModBiblio
34 );
35 use C4::Context;
36 use C4::Circulation qw( LostItem );
37 use C4::Koha qw( GetAuthorisedValues );
38 use C4::ClassSource qw( GetClassSources GetClassSource );
39 use C4::Barcodes;
40 use C4::Barcodes::ValueBuilder;
41 use Koha::DateUtils qw( dt_from_string );
42 use Koha::Items;
43 use Koha::ItemTypes;
44 use Koha::Libraries;
45 use Koha::Patrons;
46 use Koha::SearchEngine::Indexer;
47 use C4::Search qw( enabled_staff_search_views );
48 use Storable qw( freeze thaw );
49 use URI::Escape qw( uri_escape_utf8 );
50 use C4::Members;
51
52 use MARC::File::XML;
53 use URI::Escape qw( uri_escape_utf8 );
54 use Encode qw( encode_utf8 );
55 use MIME::Base64 qw( decode_base64url encode_base64url );
56 use List::Util qw( first );
57 use List::MoreUtils qw( any uniq );
58
59 our $dbh = C4::Context->dbh;
60
61 sub generate_subfield_form {
62         my ($tag, $subfieldtag, $value, $tagslib,$subfieldlib, $branches, $biblionumber, $temp, $subfields, $i, $restrictededition, $item) = @_;
63   
64         my $frameworkcode = &GetFrameworkCode($biblionumber);
65
66         $item //= {};
67
68         my %subfield_data;
69         my $dbh = C4::Context->dbh;
70         
71         my $index_subfield = int(rand(1000000)); 
72         if ($subfieldtag eq '@'){
73             $subfield_data{id} = "tag_".$tag."_subfield_00_".$index_subfield;
74         } else {
75             $subfield_data{id} = "tag_".$tag."_subfield_".$subfieldtag."_".$index_subfield;
76         }
77         
78         $subfield_data{tag}        = $tag;
79         $subfield_data{subfield}   = $subfieldtag;
80         $subfield_data{marc_lib}   ="<span id=\"error$i\" title=\"".$subfieldlib->{lib}."\">".$subfieldlib->{lib}."</span>";
81         $subfield_data{mandatory}  = $subfieldlib->{mandatory};
82         $subfield_data{important}  = $subfieldlib->{important};
83         $subfield_data{repeatable} = $subfieldlib->{repeatable};
84         $subfield_data{maxlength}  = $subfieldlib->{maxlength};
85         $subfield_data{display_order} = $subfieldlib->{display_order};
86         $subfield_data{kohafield}  = $subfieldlib->{kohafield} || 'items.more_subfields_xml';
87         
88         if ( ! defined( $value ) || $value eq '')  {
89             $value = $subfieldlib->{defaultvalue};
90             if ( $value ) {
91                 # get today date & replace <<YYYY>>, <<YY>>, <<MM>>, <<DD>> if provided in the default value
92                 my $today_dt = dt_from_string;
93                 my $year = $today_dt->strftime('%Y');
94                 my $shortyear = $today_dt->strftime('%y');
95                 my $month = $today_dt->strftime('%m');
96                 my $day = $today_dt->strftime('%d');
97                 $value =~ s/<<YYYY>>/$year/g;
98                 $value =~ s/<<YY>>/$shortyear/g;
99                 $value =~ s/<<MM>>/$month/g;
100                 $value =~ s/<<DD>>/$day/g;
101                 # And <<USER>> with surname (?)
102                 my $username=(C4::Context->userenv?C4::Context->userenv->{'surname'}:"superlibrarian");
103                 $value=~s/<<USER>>/$username/g;
104             }
105         }
106
107         $subfield_data{visibility} = "display:none;" if (($subfieldlib->{hidden} > 4) || ($subfieldlib->{hidden} <= -4));
108
109         my $pref_itemcallnumber = C4::Context->preference('itemcallnumber');
110         if (!$value && $subfieldlib->{kohafield} eq 'items.itemcallnumber' && $pref_itemcallnumber) {
111             foreach my $pref_itemcallnumber_part (split(/,/, $pref_itemcallnumber)){
112                 my $CNtag       = substr( $pref_itemcallnumber_part, 0, 3 ); # 3-digit tag number
113                 my $CNsubfields = substr( $pref_itemcallnumber_part, 3 ); # Any and all subfields
114                 $CNsubfields = undef if $CNsubfields eq '';
115                 my $temp2 = $temp->field($CNtag);
116
117                 next unless $temp2;
118                 $value = $temp2->as_string( $CNsubfields, ' ' );
119                 last if $value;
120             }
121         }
122
123         my $default_location = C4::Context->preference('NewItemsDefaultLocation');
124         if ( !$value && $subfieldlib->{kohafield} eq 'items.location' && $default_location ) {
125             $value = $default_location;
126         }
127
128         if ($frameworkcode eq 'FA' && $subfieldlib->{kohafield} eq 'items.barcode' && !$value){
129             my $input = CGI->new;
130             $value = $input->param('barcode');
131         }
132
133         if ( $subfieldlib->{authorised_value} ) {
134             my @authorised_values;
135             my %authorised_lib;
136             # builds list, depending on authorised value...
137             if ( $subfieldlib->{authorised_value} eq "LOST" ) {
138                 my $ClaimReturnedLostValue = C4::Context->preference('ClaimReturnedLostValue');
139                 my $item_is_return_claim = $ClaimReturnedLostValue && exists $item->{itemlost} && $ClaimReturnedLostValue eq $item->{itemlost};
140                 $subfield_data{IS_RETURN_CLAIM} = $item_is_return_claim;
141
142                 $subfield_data{IS_LOST_AV} = 1;
143
144                 push @authorised_values, qq{};
145                 my $av = GetAuthorisedValues( $subfieldlib->{authorised_value} );
146                 for my $r ( @$av ) {
147                     push @authorised_values, $r->{authorised_value};
148                     $authorised_lib{$r->{authorised_value}} = $r->{lib};
149                 }
150             }
151             elsif ( $subfieldlib->{authorised_value} eq "branches" ) {
152                 foreach my $thisbranch (@$branches) {
153                     push @authorised_values, $thisbranch->{branchcode};
154                     $authorised_lib{$thisbranch->{branchcode}} = $thisbranch->{branchname};
155                     $value = $thisbranch->{branchcode} if $thisbranch->{selected} && !$value;
156                 }
157             }
158             elsif ( $subfieldlib->{authorised_value} eq "itemtypes" ) {
159                   push @authorised_values, "";
160                   my $branch_limit = C4::Context->userenv && C4::Context->userenv->{"branch"};
161                   my $itemtypes;
162                   if($branch_limit) {
163                       $itemtypes = Koha::ItemTypes->search_with_localization({branchcode => $branch_limit});
164                   } else {
165                       $itemtypes = Koha::ItemTypes->search_with_localization;
166                   }
167                   while ( my $itemtype = $itemtypes->next ) {
168                       push @authorised_values, $itemtype->itemtype;
169                       $authorised_lib{$itemtype->itemtype} = $itemtype->translated_description;
170                   }
171
172                   unless ( $value ) {
173                       my $itype_sth = $dbh->prepare("SELECT itemtype FROM biblioitems WHERE biblionumber = ?");
174                       $itype_sth->execute( $biblionumber );
175                       ( $value ) = $itype_sth->fetchrow_array;
176                   }
177           
178                   #---- class_sources
179             }
180             elsif ( $subfieldlib->{authorised_value} eq "cn_source" ) {
181                   push @authorised_values, "";
182                     
183                   my $class_sources = GetClassSources();
184                   my $default_source = C4::Context->preference("DefaultClassificationSource");
185                   
186                   foreach my $class_source (sort keys %$class_sources) {
187                       next unless $class_sources->{$class_source}->{'used'} or
188                                   ($value and $class_source eq $value)      or
189                                   ($class_source eq $default_source);
190                       push @authorised_values, $class_source;
191                       $authorised_lib{$class_source} = $class_sources->{$class_source}->{'description'};
192                   }
193                           $value = $default_source unless ($value);
194         
195                   #---- "true" authorised value
196             }
197             else {
198                   push @authorised_values, qq{};
199                   my $av = GetAuthorisedValues( $subfieldlib->{authorised_value} );
200                   for my $r ( @$av ) {
201                       push @authorised_values, $r->{authorised_value};
202                       $authorised_lib{$r->{authorised_value}} = $r->{lib};
203                   }
204             }
205
206             if ( $subfieldlib->{hidden} > 4 or $subfieldlib->{hidden} <= -4 ) {
207                 $subfield_data{marc_value} = {
208                     type        => 'hidden',
209                     id          => $subfield_data{id},
210                     maxlength   => $subfield_data{maxlength},
211                     value       => $value,
212                     ( ( grep { $_ eq $subfieldlib->{authorised_value}} ( qw(branches itemtypes cn_source) ) ) ? () : ( category => $subfieldlib->{authorised_value}) ),
213                 };
214             }
215             else {
216                 $subfield_data{marc_value} = {
217                     type     => 'select',
218                     id       => "tag_".$tag."_subfield_".$subfieldtag."_".$index_subfield,
219                     values   => \@authorised_values,
220                     labels   => \%authorised_lib,
221                     default  => $value,
222                     ( ( grep { $_ eq $subfieldlib->{authorised_value}} ( qw(branches itemtypes cn_source) ) ) ? () : ( category => $subfieldlib->{authorised_value}) ),
223                 };
224             }
225         }
226             # it's a thesaurus / authority field
227         elsif ( $subfieldlib->{authtypecode} ) {
228                 $subfield_data{marc_value} = {
229                     type         => 'text_auth',
230                     id           => $subfield_data{id},
231                     maxlength    => $subfield_data{maxlength},
232                     value        => $value,
233                     authtypecode => $subfieldlib->{authtypecode},
234                 };
235         }
236             # it's a plugin field
237         elsif ( $subfieldlib->{value_builder} ) { # plugin
238             require Koha::FrameworkPlugin;
239             my $plugin = Koha::FrameworkPlugin->new({
240                 name => $subfieldlib->{'value_builder'},
241                 item_style => 1,
242             });
243             my $pars=  { dbh => $dbh, record => $temp, tagslib =>$tagslib,
244                 id => $subfield_data{id}, tabloop => $subfields };
245             $plugin->build( $pars );
246             if( !$plugin->errstr ) {
247                 my $class= 'buttonDot'. ( $plugin->noclick? ' disabled': '' );
248                 $subfield_data{marc_value} = {
249                     type        => 'text_plugin',
250                     id          => $subfield_data{id},
251                     maxlength   => $subfield_data{maxlength},
252                     value       => $value,
253                     class       => $class,
254                     nopopup     => $plugin->noclick,
255                     javascript  => $plugin->javascript,
256                 };
257             } else {
258                 warn $plugin->errstr;
259                 $subfield_data{marc_value} = {
260                     type        => 'text',
261                     id          => $subfield_data{id},
262                     maxlength   => $subfield_data{maxlength},
263                     value       => $value,
264                 }; # supply default input form
265             }
266         }
267         elsif ( $tag eq '' ) {       # it's an hidden field
268             $subfield_data{marc_value} = {
269                 type        => 'hidden',
270                 id          => $subfield_data{id},
271                 maxlength   => $subfield_data{maxlength},
272                 value       => $value,
273             };
274         }
275         elsif ( $subfieldlib->{'hidden'} ) {   # FIXME: shouldn't input type be "hidden" ?
276             $subfield_data{marc_value} = {
277                 type        => 'text',
278                 id          => $subfield_data{id},
279                 maxlength   => $subfield_data{maxlength},
280                 value       => $value,
281             };
282         }
283         elsif (
284                 (
285                     $value and length($value) > 100
286                 )
287                 or (
288                     C4::Context->preference("marcflavour") eq "UNIMARC"
289                     and 300 <= $tag && $tag < 400 && $subfieldtag eq 'a'
290                 )
291                 or (
292                     C4::Context->preference("marcflavour") eq "MARC21"
293                     and 500 <= $tag && $tag < 600
294                 )
295               ) {
296             # oversize field (textarea)
297             $subfield_data{marc_value} = {
298                 type        => 'textarea',
299                 id          => $subfield_data{id},
300                 value       => $value,
301             };
302         } else {
303             # it's a standard field
304             $subfield_data{marc_value} = {
305                 type        => 'text',
306                 id          => $subfield_data{id},
307                 maxlength   => $subfield_data{maxlength},
308                 value       => $value,
309             };
310         }
311
312         # Getting list of subfields to keep when restricted editing is enabled
313         my $subfieldsToAllowForRestrictedEditing = C4::Context->preference('SubfieldsToAllowForRestrictedEditing');
314         my $allowAllSubfields = (
315             not defined $subfieldsToAllowForRestrictedEditing
316               or $subfieldsToAllowForRestrictedEditing eq q||
317         ) ? 1 : 0;
318         my @subfieldsToAllow = split(/ /, $subfieldsToAllowForRestrictedEditing);
319
320         # If we're on restricted editing, and our field is not in the list of subfields to allow,
321         # then it is read-only
322         $subfield_data{marc_value}->{readonly} = (
323             not $allowAllSubfields
324             and $restrictededition
325             and !grep { $tag . '$' . $subfieldtag  eq $_ } @subfieldsToAllow
326         ) ? 1: 0;
327
328         return \%subfield_data;
329 }
330
331 sub get_item_from_cookie {
332     my ( $input ) = @_;
333
334     my $item_from_cookie;
335     my $lastitemcookie = $input->cookie('LastCreatedItem');
336     if ($lastitemcookie) {
337         $lastitemcookie = decode_base64url($lastitemcookie);
338         eval {
339             if ( thaw($lastitemcookie) ) {
340                 $item_from_cookie = thaw($lastitemcookie);
341             }
342         };
343         if ($@) {
344             $lastitemcookie ||= 'undef';
345             warn "Storable::thaw failed to thaw LastCreatedItem-cookie. Cookie value '".encode_base64url($lastitemcookie)."'. Caught error follows: '$@'";
346         }
347     }
348     return $item_from_cookie;
349 }
350
351 my $input        = CGI->new;
352 my $error        = $input->param('error');
353
354 my $biblionumber;
355 my $itemnumber;
356 if( $input->param('itemnumber') && !$input->param('biblionumber') ){
357     $itemnumber = $input->param('itemnumber');
358     my $item = Koha::Items->find( $itemnumber );
359     $biblionumber = $item->biblionumber;
360 } else {
361     $biblionumber = $input->param('biblionumber');
362     $itemnumber = $input->param('itemnumber');
363 }
364
365 my $biblio = Koha::Biblios->find($biblionumber);
366
367 my $op           = $input->param('op') || q{};
368 my $hostitemnumber = $input->param('hostitemnumber');
369 my $marcflavour  = C4::Context->preference("marcflavour");
370 my $searchid     = $input->param('searchid');
371 # fast cataloguing datas
372 my $fa_circborrowernumber = $input->param('circborrowernumber');
373 my $fa_barcode            = $input->param('barcode');
374 my $fa_branch             = $input->param('branch');
375 my $fa_stickyduedate      = $input->param('stickyduedate');
376 my $fa_duedatespec        = $input->param('duedatespec');
377
378 our $frameworkcode = &GetFrameworkCode($biblionumber);
379
380 # Defining which userflag is needing according to the framework currently used
381 my $userflags;
382 if (defined $input->param('frameworkcode')) {
383     $userflags = ($input->param('frameworkcode') eq 'FA') ? "fast_cataloging" : "edit_items";
384 }
385
386 if (not defined $userflags) {
387     $userflags = ($frameworkcode eq 'FA') ? "fast_cataloging" : "edit_items";
388 }
389
390 my ($template, $loggedinuser, $cookie)
391     = get_template_and_user({template_name => "cataloguing/additem.tt",
392                  query => $input,
393                  type => "intranet",
394                  flagsrequired => {editcatalogue => $userflags},
395                  });
396
397
398 # Does the user have a restricted item editing permission?
399 my $uid = Koha::Patrons->find( $loggedinuser )->userid;
400 my $restrictededition = $uid ? haspermission($uid,  {'editcatalogue' => 'edit_items_restricted'}) : undef;
401 # In case user is a superlibrarian, editing is not restricted
402 $restrictededition = 0 if ($restrictededition != 0 &&  C4::Context->IsSuperLibrarian());
403 # In case user has fast cataloging permission (and we're in fast cataloging), editing is not restricted
404 $restrictededition = 0 if ($restrictededition != 0 && $frameworkcode eq 'FA' && haspermission($uid, {'editcatalogue' => 'fast_cataloging'}));
405
406 our $tagslib = &GetMarcStructure(1,$frameworkcode);
407 my $record = GetMarcBiblio({ biblionumber => $biblionumber });
408
409 output_and_exit_if_error( $input, $cookie, $template,
410     { module => 'cataloguing', record => $record } );
411
412 my $current_item;
413 my $nextop="additem";
414 my @errors; # store errors found while checking data BEFORE saving item.
415
416 # Getting last created item cookie
417 my $prefillitem = C4::Context->preference('PrefillItem');
418
419 #-------------------------------------------------------------------------------
420 if ($op eq "additem") {
421
422     my $add_submit                 = $input->param('add_submit');
423     my $add_duplicate_submit       = $input->param('add_duplicate_submit');
424     my $add_multiple_copies_submit = $input->param('add_multiple_copies_submit');
425     my $number_of_copies           = $input->param('number_of_copies');
426
427     my @columns = Koha::Items->columns;
428     my $item = Koha::Item->new;
429     $item->biblionumber($biblio->biblionumber);
430     for my $c ( @columns ) {
431         if ( $c eq 'more_subfields_xml' ) {
432             my @more_subfields_xml = $input->multi_param("items.more_subfields_xml");
433             my @unlinked_item_subfields;
434             for my $subfield ( @more_subfields_xml ) {
435                 my $v = $input->param('items.more_subfields_xml_' . $subfield);
436                 push @unlinked_item_subfields, $subfield, $v;
437             }
438             if ( @unlinked_item_subfields ) {
439                 my $marc = MARC::Record->new();
440                 # use of tag 999 is arbitrary, and doesn't need to match the item tag
441                 # used in the framework
442                 $marc->append_fields(MARC::Field->new('999', ' ', ' ', @unlinked_item_subfields));
443                 $marc->encoding("UTF-8");
444                 $item->more_subfields_xml($marc->as_xml("USMARC"));
445                 next;
446             }
447             $item->more_subfields_xml(undef);
448         } else {
449             my @v = grep { $_ ne "" }
450                 uniq $input->multi_param( "items." . $c );
451
452             next if !@v
453                 && $c ne 'permanent_location'; # See 27837
454
455             $item->$c(join ' | ', @v);
456         }
457     }
458
459     # if autoBarcode is set to 'incremental', calculate barcode...
460     if ( ! defined $item->barcode && C4::Context->preference('autoBarcode') eq 'incremental' ) {
461         my ( $barcode ) = C4::Barcodes::ValueBuilder::incremental::get_barcode;
462         $item->barcode($barcode);
463     }
464
465     # If we have to add or add & duplicate, we add the item
466     if ( $add_submit || $prefillitem) {
467
468         # check for item barcode # being unique
469         if ( defined $item->barcode
470             && Koha::Items->search( { barcode => $item->barcode } )->count )
471         {
472             # if barcode exists, don't create, but report The problem.
473             push @errors, "barcode_not_unique";
474
475             $current_item = $item->unblessed; # Restore edit form for the same item
476         }
477         else {
478             $item->store->discard_changes;
479
480             # This is a bit tricky : if there is a cookie for the last created item and
481             # we just added an item, the cookie value is not correct yet (it will be updated
482             # next page). To prevent the form from being filled with outdated values, we
483             # force the use of "add and duplicate" feature, so the form will be filled with
484             # correct values.
485
486             # Pushing the last created item cookie back
487             if ( $prefillitem ) {
488                 my $last_created_item_cookie = $input->cookie(
489                     -name => 'LastCreatedItem',
490                     # We encode_base64url the whole freezed structure so we're sure we won't have any encoding problems
491                     -value   => encode_base64url( freeze( { %{$item->unblessed}, itemnumber => undef } ) ),
492                     -HttpOnly => 1,
493                     -expires => ''
494                 );
495
496                 $cookie = [ $cookie, $last_created_item_cookie ];
497             }
498
499         }
500         $nextop = "additem";
501
502     }
503
504     # If we have to add & duplicate
505     if ($prefillitem || $add_duplicate_submit) {
506
507         $current_item = $item->unblessed;
508
509         if (C4::Context->preference('autoBarcode') eq 'incremental') {
510             my ( $barcode ) = C4::Barcodes::ValueBuilder::incremental::get_barcode;
511             $current_item->{barcode} = $barcode;
512         }
513         else {
514             # we have to clear the barcode field in the duplicate item record to make way for the new one generated by the javascript plugin
515             $current_item->{barcode} = undef; # FIXME or delete?
516         }
517
518         # Don't use the "prefill" feature if we want to generate the form with all the info from this item
519         # It will remove subfields that are not in SubfieldsToUseWhenPrefill.
520         $prefillitem = 0 if $add_duplicate_submit;
521     }
522
523     # If we have to add multiple copies
524     if ($add_multiple_copies_submit) {
525
526         $current_item = $item->unblessed;
527
528         my $copynumber = $current_item->{copynumber};
529         my $oldbarcode = $current_item->{barcode};
530
531         # If there is a barcode and we can't find their new values, we can't add multiple copies
532         my $testbarcode;
533         my $barcodeobj = C4::Barcodes->new;
534         $testbarcode = $barcodeobj->next_value($oldbarcode) if $barcodeobj;
535         if ( $oldbarcode && !$testbarcode ) {
536
537             push @errors, "no_next_barcode";
538
539         }
540         else {
541             # We add each item
542
543             # For the first iteration
544             my $barcodevalue = $oldbarcode;
545             my $exist_itemnumber;
546
547             for ( my $i = 0 ; $i < $number_of_copies ; ) {
548
549                 # If there is a barcode
550                 if ($barcodevalue) {
551
552 # Getting a new barcode (if it is not the first iteration or the barcode we tried already exists)
553                     $barcodevalue = $barcodeobj->next_value($oldbarcode)
554                       if ( $i > 0 || $exist_itemnumber );
555
556                     # Putting it into the record
557                     if ($barcodevalue) {
558                         if ( C4::Context->preference("autoBarcode") eq
559                             'hbyymmincr' && $i > 0 )
560                         { # The first copy already contains the homebranch prefix
561                              # This is terribly hacky but the easiest way to fix the way hbyymmincr is working
562                              # Contrary to what one might think, the barcode plugin does not prefix the returned string with the homebranch
563                              # For a single item, it is handled with some JS code (see cataloguing/value_builder/barcode.pl)
564                              # But when adding multiple copies we need to prefix it here,
565                              # so we retrieve the homebranch from the item and prefix the barcode with it.
566                             my $homebranch = $current_item->{homebranch};
567                             $barcodevalue = $homebranch . $barcodevalue;
568                         }
569                         $current_item->{barcode} = $barcodevalue;
570                     }
571
572                     # Checking if the barcode already exists
573                     $exist_itemnumber = Koha::Items->search({ barcode => $barcodevalue })->count;
574                 }
575
576                 # Updating record with the new copynumber
577                 if ($copynumber) {
578                     $current_item->{copynumber} = $copynumber;
579                 }
580
581                 # Adding the item
582                 if ( !$exist_itemnumber ) {
583                     delete $current_item->{itemnumber};
584                     $current_item = Koha::Item->new($current_item)->store(
585                         { skip_record_index => 1 } );
586                     $current_item->discard_changes; # Cannot chain discard_changes
587                     $current_item = $current_item->unblessed;
588
589 # We count the item only if it was really added
590 # That way, all items are added, even if there was some already existing barcodes
591 # FIXME : Please note that there is a risk of infinite loop here if we never find a suitable barcode
592                     $i++;
593
594                     # Only increment copynumber if item was really added
595                     $copynumber++ if ( $copynumber && $copynumber =~ m/^\d+$/ );
596                 }
597
598                 # Preparing the next iteration
599                 $oldbarcode = $barcodevalue;
600             }
601
602             my $indexer = Koha::SearchEngine::Indexer->new(
603                 { index => $Koha::SearchEngine::BIBLIOS_INDEX } );
604             $indexer->index_records( $biblionumber, "specialUpdate",
605                 "biblioserver" );
606
607             undef($current_item);
608         }
609     }
610     if ($frameworkcode eq 'FA' && $fa_circborrowernumber){
611         print $input->redirect(
612            '/cgi-bin/koha/circ/circulation.pl?'
613            .'borrowernumber='.$fa_circborrowernumber
614            .'&barcode='.uri_escape_utf8($fa_barcode)
615            .'&duedatespec='.$fa_duedatespec
616            .'&stickyduedate='.$fa_stickyduedate
617         );
618         exit;
619     }
620
621
622 #-------------------------------------------------------------------------------
623 } elsif ($op eq "edititem") {
624 #-------------------------------------------------------------------------------
625 # retrieve item if exist => then, it's a modif
626     $current_item = Koha::Items->find($itemnumber)->unblessed;
627     # FIXME Handle non existent item
628     $nextop = "saveitem";
629 #-------------------------------------------------------------------------------
630 } elsif ($op eq "dupeitem") {
631 #-------------------------------------------------------------------------------
632 # retrieve item if exist => then, it's a modif
633     my $item = Koha::Items->find($itemnumber);
634     # FIXME Handle non existent item
635     if (C4::Context->preference('autoBarcode') eq 'incremental') {
636         my ( $barcode ) = C4::Barcodes::ValueBuilder::incremental::get_barcode;
637         $item->barcode($barcode);
638     }
639     else {
640         $item->barcode(undef); # Don't save it!
641     }
642
643     $nextop = "additem";
644 #-------------------------------------------------------------------------------
645 } elsif ($op eq "delitem") {
646 #-------------------------------------------------------------------------------
647     # check that there is no issue on this item before deletion.
648     my $item = Koha::Items->find($itemnumber);
649     $error = $item->safe_delete;
650     if(ref($error) eq 'Koha::Item'){
651         print $input->redirect("additem.pl?biblionumber=$biblionumber&frameworkcode=$frameworkcode&searchid=$searchid");
652     }else{
653         push @errors,$error;
654         $nextop="additem";
655     }
656 #-------------------------------------------------------------------------------
657 } elsif ($op eq "delallitems") {
658 #-------------------------------------------------------------------------------
659     my $items = Koha::Items->search({ biblionumber => $biblionumber });
660     while ( my $item = $items->next ) {
661         $error = $item->safe_delete({ skip_record_index => 1 });
662         next if ref $error eq 'Koha::Item'; # Deleted item is returned if deletion successful
663         push @errors,$error;
664     }
665     my $indexer = Koha::SearchEngine::Indexer->new({ index => $Koha::SearchEngine::BIBLIOS_INDEX });
666     $indexer->index_records( $biblionumber, "specialUpdate", "biblioserver" );
667     if ( @errors ) {
668         $nextop="additem";
669     } else {
670         my $defaultview = C4::Context->preference('IntranetBiblioDefaultView');
671         my $views = { C4::Search::enabled_staff_search_views };
672         if ($defaultview eq 'isbd' && $views->{can_view_ISBD}) {
673             print $input->redirect("/cgi-bin/koha/catalogue/ISBDdetail.pl?biblionumber=$biblionumber&searchid=$searchid");
674         } elsif  ($defaultview eq 'marc' && $views->{can_view_MARC}) {
675             print $input->redirect("/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=$biblionumber&searchid=$searchid");
676         } elsif  ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC}) {
677             print $input->redirect("/cgi-bin/koha/catalogue/labeledMARCdetail.pl?biblionumber=$biblionumber&searchid=$searchid");
678         } else {
679             print $input->redirect("/cgi-bin/koha/catalogue/detail.pl?biblionumber=$biblionumber&searchid=$searchid");
680         }
681         exit;
682     }
683 #-------------------------------------------------------------------------------
684 } elsif ($op eq "saveitem") {
685 #-------------------------------------------------------------------------------
686
687     my $itemnumber = $input->param('itemnumber');
688     my $item = Koha::Items->find($itemnumber);
689     # FIXME Handle non existent item
690     my $olditemlost = $item->itemlost;
691     my @columns = Koha::Items->columns;
692     for my $c ( @columns ) {
693         if ( $c eq 'more_subfields_xml' ) {
694             my @more_subfields_xml = $input->multi_param("items.more_subfields_xml");
695             my @unlinked_item_subfields;
696             for my $subfield ( uniq @more_subfields_xml ) {
697                 my @v = $input->multi_param('items.more_subfields_xml_' . encode_utf8($subfield));
698                 push @unlinked_item_subfields, $subfield, $_ for @v;
699             }
700             if ( @unlinked_item_subfields ) {
701                 my $marc = MARC::Record->new();
702                 # use of tag 999 is arbitrary, and doesn't need to match the item tag
703                 # used in the framework
704                 $marc->append_fields(MARC::Field->new('999', ' ', ' ', @unlinked_item_subfields));
705                 $marc->encoding("UTF-8");
706                 $item->more_subfields_xml($marc->as_xml("USMARC"));
707                 next;
708             }
709             $item->more_subfields_xml(undef);
710         } else {
711             my @v = $input->multi_param("items.".$c);
712             next unless @v;
713             $item->$c(join ' | ', uniq @v);
714         }
715     }
716
717     # check that the barcode don't exist already
718     if (
719         defined $item->barcode
720         && Koha::Items->search(
721             {
722                 barcode    => $item->barcode,
723                 itemnumber => { '!=' => $item->itemnumber }
724             }
725         )->count
726       )
727     {
728         # FIXME We shouldn't need that, ->store would explode as there is a unique constraint on items.barcode
729         push @errors,"barcode_not_unique";
730         $current_item = $item->unblessed; # Restore edit form for the same item
731     } else {
732         my $newitemlost = $item->itemlost;
733         if ( $newitemlost && $newitemlost ge '1' && !$olditemlost ) {
734             LostItem( $item->itemnumber, 'additem' );
735         }
736         $item->store;
737     }
738
739     $nextop="additem";
740 } elsif ($op eq "delinkitem"){
741
742     my $analyticfield = '773';
743         if ($marcflavour  eq 'MARC21' || $marcflavour eq 'NORMARC'){
744         $analyticfield = '773';
745     } elsif ($marcflavour eq 'UNIMARC') {
746         $analyticfield = '461';
747     }
748     foreach my $field ($record->field($analyticfield)){
749         if ($field->subfield('9') eq $hostitemnumber){
750             $record->delete_field($field);
751             last;
752         }
753     }
754         my $modbibresult = ModBiblio($record, $biblionumber,'');
755 }
756
757 # update OAI-PMH sets
758 if ($op) {
759     if (C4::Context->preference("OAI-PMH:AutoUpdateSets")) {
760         C4::OAI::Sets::UpdateOAISetsBiblio($biblionumber, $record);
761     }
762 }
763
764 #
765 #-------------------------------------------------------------------------------
766 # build screen with existing items. and "new" one
767 #-------------------------------------------------------------------------------
768
769 # now, build existiing item list
770
771 my @items;
772 for my $item ( $biblio->items->as_list, $biblio->host_items->as_list ) {
773     push @items, $item->columns_to_str;
774 }
775
776 my @witness_attributes = uniq map {
777     my $item = $_;
778     map { defined $item->{$_} && $item->{$_} ne "" ? $_ : () } keys %$item
779 } @items;
780
781 our ( $itemtagfield, $itemtagsubfield ) = &GetMarcFromKohaField("items.itemnumber");
782
783 my $subfieldcode_attribute_mappings;
784 for my $subfield_code ( keys %{ $tagslib->{$itemtagfield} } ) {
785
786     my $subfield = $tagslib->{$itemtagfield}->{$subfield_code};
787
788     next if IsMarcStructureInternal( $subfield );
789     next unless $subfield->{tab} eq 10; # Is this really needed?
790
791     my $attribute;
792     if ( $subfield->{kohafield} ) {
793         ( $attribute = $subfield->{kohafield} ) =~ s|^items\.||;
794     } else {
795         $attribute = $subfield_code; # It's in more_subfields_xml
796     }
797     next unless grep { $attribute eq $_ } @witness_attributes;
798     $subfieldcode_attribute_mappings->{$subfield_code} = $attribute;
799 }
800
801 my @header_value_loop = map {
802     {
803         header_value  => $tagslib->{$itemtagfield}->{$_}->{lib},
804         attribute     => $subfieldcode_attribute_mappings->{$_},
805         subfield_code => $_,
806     }
807 } sort keys %$subfieldcode_attribute_mappings;
808
809 # now, build the item form for entering a new item
810 my $branch = $input->param('branch') || C4::Context->userenv->{branch};
811 my $libraries = Koha::Libraries->search({}, { order_by => ['branchname'] })->unblessed;# build once ahead of time, instead of multiple times later.
812 for my $library ( @$libraries ) {
813     $library->{selected} = 1 if $library->{branchcode} eq $branch
814 }
815
816
817 # Using last created item if it exists
818 if (   $prefillitem
819     && $op ne "additem"
820     && $op ne "edititem" )
821 {
822     my $item_from_cookie = get_item_from_cookie($input);
823     $current_item = $item_from_cookie if $item_from_cookie;
824 }
825
826 my @subfields_to_prefill = split ' ', C4::Context->preference('SubfieldsToUseWhenPrefill');
827
828 if ( $current_item->{more_subfields_xml} ) {
829     $current_item->{marc_more_subfields_xml} = MARC::Record->new_from_xml($current_item->{more_subfields_xml}, 'UTF-8');
830 }
831
832 # We generate form, and fill with values if defined
833 my $temp = GetMarcBiblio({ biblionumber => $biblionumber });
834 my $i = 0;
835 my @subfields;
836 foreach my $tag ( keys %{$tagslib} ) {
837     foreach my $subtag ( keys %{ $tagslib->{$tag} } ) {
838
839         my $subfield = $tagslib->{$tag}{$subtag};
840
841         next if IsMarcStructureInternal( $subfield );
842         next if ( $subfield->{tab} ne "10" );
843
844         my @values = ();
845
846         my $subfield_data;
847
848         # If we are not adding a new item
849         # OR
850         # If the subfield must be prefilled with last catalogued item
851         if (
852             $nextop ne 'additem'
853             || (
854                 !$prefillitem
855                 || ( $prefillitem && grep { $_ eq $subtag }
856                     @subfields_to_prefill )
857             )
858           )
859         {
860             my $kohafield = $subfield->{kohafield};
861             if ($kohafield) {
862
863                 # This is a mapped field
864                 ( my $attribute = $kohafield ) =~ s|^items\.||;
865                 push @values, $subfield->{repeatable}
866                     ? split '\s\|\s', $current_item->{$attribute}
867                     : $current_item->{$attribute}
868                   if defined $current_item->{$attribute};
869             } else {
870                 # Not mapped, picked the values from more_subfields_xml's MARC
871                 my $marc_more = $current_item->{marc_more_subfields_xml};
872                 if ( $marc_more ) {
873                     for my $f ( $marc_more->fields($tag) ) {
874                         push @values, $f->subfield($subtag);
875                     }
876                 }
877             }
878         }
879
880         @values = ('') unless @values;
881
882         for my $value (@values) {
883             my $subfield_data = generate_subfield_form(
884                 $tag,                        $subtag,
885                 $value,                      $tagslib,
886                 $subfield,                   $libraries,
887                 $biblionumber,               $temp,
888                 \@subfields,                 $i,
889                 $restrictededition,          $current_item,
890             );
891             push @subfields, $subfield_data;
892             $i++;
893         }
894     }
895 }
896 @subfields = sort { $a->{display_order} <=> $b->{display_order} || $a->{subfield} cmp $b->{subfield} } @subfields;
897
898 # what's the next op ? it's what we are not in : an add if we're editing, otherwise, and edit.
899 $template->param(
900     biblio       => $biblio,
901     items        => \@items,
902     item_header_loop => \@header_value_loop,
903     subfields    => \@subfields,
904     itemnumber       => $itemnumber,
905     barcode          => $current_item->{barcode},
906     itemtagfield     => $itemtagfield,
907     itemtagsubfield  => $itemtagsubfield,
908     op      => $nextop,
909     popup => scalar $input->param('popup') ? 1: 0,
910     C4::Search::enabled_staff_search_views,
911 );
912 $template->{'VARS'}->{'searchid'} = $searchid;
913
914 if ($frameworkcode eq 'FA'){
915     # fast cataloguing datas
916     $template->param(
917         'circborrowernumber' => $fa_circborrowernumber,
918         'barcode'            => $fa_barcode,
919         'branch'             => $fa_branch,
920         'stickyduedate'      => $fa_stickyduedate,
921         'duedatespec'        => $fa_duedatespec,
922     );
923 }
924
925 foreach my $error (@errors) {
926     $template->param($error => 1);
927 }
928 output_html_with_http_headers $input, $cookie, $template->output;