Bug 6026 : export.pl : removing useless condition

Some conditions added AND items.biblionumber=biblio/biblionumber
where JOIN USING (biblionumber) already added that.

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
Henri-Damien LAURENT 2011-04-01 10:34:51 +02:00 committed by Chris Cormack
parent 1c89ef1af4
commit 63f1ea0c2d
1 changed files with 15 additions and 15 deletions

View File

@ -94,28 +94,28 @@ if ($op eq "export") {
$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');
}