use strict;
-require Exporter;
+use warnings;
use C4::Auth;
use C4::Output; # contains gettemplate
use C4::Biblio; # GetMarcBiblio GetXmlBiblio
use C4::Branch; # GetBranches
my $query = new CGI;
-my $op=$query->param("op");
+my $op=$query->param("op") || '';
my $filename=$query->param("filename");
my $dbh=C4::Context->dbh;
my $marcflavour = C4::Context->preference("marcflavour");
my $limit_ind_branch=(C4::Context->preference('IndependantBranches') &&
C4::Context->userenv &&
- C4::Context->userenv->{flags} !=1 &&
+ C4::Context->userenv->{flags} % 2 !=1 &&
C4::Context->userenv->{branch}?1:0);
my $branches = GetBranches($limit_ind_branch);
- my $branch = $query->param("branch");
+ my $branch = $query->param("branch") || '';
if ( C4::Context->preference("IndependantBranches") ) {
$branch = C4::Context->userenv->{'branch'};
}
my $strip_nonlocal_items = $query->param("strip_nonlocal_items");
my $dont_export_fields = $query->param("dont_export_fields");
my @sql_params;
- my $query = " SELECT DISTINCT biblioitems.biblionumber
- FROM biblioitems,items
- WHERE biblioitems.biblionumber=items.biblionumber ";
+
+ my $items_filter =
+ $branch || $start_callnumber || $end_callnumber ||
+ $start_accession || $end_accession ||
+ ($itemtype && C4::Context->preference('item-level_itypes'));
+ my $query = $items_filter ?
+ "SELECT DISTINCT biblioitems.biblionumber
+ FROM biblioitems JOIN items
+ USING (biblionumber) WHERE 1"
+ :
+ "SELECT biblioitems.biblionumber FROM biblioitems WHERE biblionumber >0 ";
if ( $StartingBiblionumber ) {
$query .= " AND biblioitems.biblionumber >= ? ";
$query .= " AND biblioitems.biblionumber <= ? ";
push @sql_params, $EndingBiblionumber;
}
-
- if ( $branch ) {
- $query .= " AND biblioitems.biblionumber = items.biblionumber AND homebranch = ? ";
+
+ if ($branch) {
+ $query .= " AND homebranch = ? ";
push @sql_params, $branch;
}
-
- if ( $start_callnumber ) {
- $query .= " AND biblioitems.biblionumber = items.biblionumber AND itemcallnumber <= ? ";
+
+ if ($start_callnumber) {
+ $query .= " AND itemcallnumber <= ? ";
push @sql_params, $start_callnumber;
}
-
- if ( $end_callnumber ) {
- $query .= " AND biblioitems.biblionumber = items.biblionumber AND itemcallnumber >= ? ";
+
+ if ($end_callnumber) {
+ $query .= " AND itemcallnumber >= ? ";
push @sql_params, $end_callnumber;
}
- if ( $start_accession ) {
- $query .= " AND biblioitems.biblionumber = items.biblionumber AND dateaccessioned >= ? ";
- push @sql_params,$start_accession->output('iso');
+ if ($start_accession) {
+ $query .= " AND dateaccessioned >= ? ";
+ push @sql_params, $start_accession->output('iso');
}
-
- if ( $end_accession ) {
- $query .= " AND biblioitems.biblionumber = items.biblionumber AND dateaccessioned <= ? ";
+
+ if ($end_accession) {
+ $query .= " AND dateaccessioned <= ? ";
push @sql_params, $end_accession->output('iso');
}
$sth->execute(@sql_params);
while (my ($biblionumber) = $sth->fetchrow) {
- my $record = GetMarcBiblio($biblionumber);
- my ( $homebranchfield, $homebranchsubfield ) = GetMarcFromKohaField( 'items.homebranch', '' );
- if ( $dont_export_items || $strip_nonlocal_items || $limit_ind_branch) {
+ my $record = eval{ GetMarcBiblio($biblionumber); };
+ # FIXME: decide how to handle records GetMarcBiblio can't parse or retrieve
+ if ($@) {
+ next;
+ }
+ next if not defined $record;
+ C4::Biblio::EmbedItemsInMarcBiblio($record, $biblionumber) unless $dont_export_items;
+ if ($strip_nonlocal_items || $limit_ind_branch) {
+ my ( $homebranchfield, $homebranchsubfield ) =
+ GetMarcFromKohaField( 'items.homebranch', '' );
for my $itemfield ($record->field($homebranchfield)){
# if stripping nonlocal items, use loggedinuser's branch if they didn't select one
$branch = C4::Context->userenv->{'branch'} unless $branch;
- $record->delete_field($itemfield) if($dont_export_items || ($itemfield->subfield($homebranchsubfield) ne $branch) ) ;
+ $record->delete_field($itemfield) if($itemfield->subfield($homebranchsubfield) ne $branch) ;
}
}
/^(\d*)(\w)?$/;
my $field = $1;
my $subfield = $2;
+ # skip if this record doesn't have this field
+ next if not defined $record->field($field);
if( $subfield ) {
$record->field($field)->delete_subfields($subfield);
}
push @itemtypesloop, \%row;
}
my @branchloop;
- for my $thisbranch (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} } keys %$branches) {
- my $selected = 1 if $thisbranch eq $branch;
- my %row = (
- value => $thisbranch,
- selected => $selected,
+ for my $thisbranch (
+ sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} }
+ keys %{$branches}
+ ) {
+ push @branchloop,
+ { value => $thisbranch,
+ selected => $thisbranch eq $branch,
branchname => $branches->{$thisbranch}->{'branchname'},
- );
- push @branchloop, \%row;
+ };
}
-
+
$template->param(
branchloop => \@branchloop,
itemtypeloop => \@itemtypesloop,