Bug 11222: fix crash that can occur in search for items to add to label batch
authorZeno Tajoli <z.tajoli@cineca.it>
Fri, 8 Nov 2013 17:10:08 +0000 (17:10 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Sat, 23 Nov 2013 19:54:38 +0000 (19:54 +0000)
This patch fixes a regression on label-item-search.pl introduced by
the patch for bug 9239 where searching for items to add to a label
batch fails with an error if QueryParser is not enabled and you use
the "Added on or before date" search option.

Test plan:

0) Set "UseQueryParser" to "do not try"
1) Start with an installation with bibliografic data and items
   information.
2) Check that in items there are values inside Accession Date

Without the patch:
a) Go in Home› Tools› Labels  and click on 'New batch'
b) Click on 'Add Item(s)'
c) Try to select items with option "Added on or before date"
d) You will receive an error

With the patch:
Try steps a),b), and c)
Now you don't receive an error and you can select items

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Added a description of the problem and copied the test plan from the bug
report, with the addition of the information that QueryParser must be
off to confirm and test.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>

labels/label-item-search.pl

index eeb46f8..a9a1805 100755 (executable)
@@ -97,7 +97,7 @@ if ( $op eq "do_search" ) {
             $ccl_query .= "acqdate(-" . $dateto->output("iso") . ')';
         } else {
             $ccl_query .= ' and ' if ( $ccl_textbox || $datefrom );
-            $ccl_query .= "acqdate,st-date-normalized,le=" . $datefrom->output("iso");
+            $ccl_query .= "acqdate,st-date-normalized,le=" . $dateto->output("iso");
         }
     }