Lp#1411422: Clean up the code a little bit.
authorJason Stephenson <jason@sigio.com>
Thu, 8 Mar 2018 19:11:31 +0000 (14:11 -0500)
committerJason Stephenson <jason@sigio.com>
Tue, 13 Mar 2018 15:39:42 +0000 (11:39 -0400)
We don't need to left join when we flesh fields.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/src/perlmods/lib/OpenILS/Application/Cat.pm

index 4bc9a43..f88cc45 100644 (file)
@@ -855,14 +855,6 @@ sub transfer_copies_to_volume {
         $copy = $editor->search_asset_copy([
             { id => $copy_id , deleted => 'f' },
             {
-                join => {
-                    acpm => {
-                        type => 'left',
-                        join => {
-                            bmp => { type => 'left' }
-                        }
-                    }
-                },
                 flesh => 1,
                 flesh_fields => { acp => ['parts'] }
             }
@@ -1401,14 +1393,6 @@ sub batch_volume_transfer {
         my $copies = $e->search_asset_copy([
             { call_number => $vol->id , deleted => 'f' },
             {
-                join => {
-                    acpm => {
-                        type => 'left',
-                        join => {
-                            bmp => { type => 'left' }
-                        }
-                    }
-                },
                 flesh => 1,
                 flesh_fields => { acp => ['parts'] }
             }