Bug 17420 - record export fails when itemtype on biblio
authorFridolin Somers <fridolin.somers@biblibre.com>
Mon, 10 Oct 2016 14:46:04 +0000 (16:46 +0200)
committerKyle M Hall <kyle@bywatersolutions.com>
Sat, 12 Nov 2016 02:32:41 +0000 (02:32 +0000)
In Tools, Export data, you can export with a filter on item type.
When item type is on biblio record, you get the error :
export.pl: DBIx::Class::ResultSet::next(): Unknown column 'biblioitems.itemtype' in 'where clause' at /home/koha/src/Koha/Objects.pm line 150

Looks like its because this code use to be a SQL Select and is now a DBIx call :
                          C4::Context->preference('item-level_itypes')
                            ? ( 'items.itype' => $itemtype )
                            : ( 'me.itemtype' => $itemtype )

This patch corrects by using "me.itemtype"

Test plan :
- Use syspref item-level_itypes = biblio record
- Go to Tools > Export data : /cgi-bin/koha/tools/export.pl
- Select an item type
- Click on "Export biblio records"
=> Without patch you get an error 500
=> With patch you get a correct export file

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Fix the issue, no errors.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
tools/export.pl

index c585d90..03792c3 100755 (executable)
@@ -139,7 +139,7 @@ if ( $op eq "export" ) {
                         ?
                           C4::Context->preference('item-level_itypes')
                             ? ( 'items.itype' => $itemtype )
-                            : ( 'biblioitems.itemtype' => $itemtype )
+                            : ( 'me.itemtype' => $itemtype )
                         : ()
                     ),