Inventory: with_objects cleanup
authorSven Schöling <s.schoeling@googlemail.com>
Fri, 16 Oct 2020 16:07:34 +0000 (18:07 +0200)
committerSven Schöling <s.schoeling@googlemail.com>
Fri, 27 Nov 2020 15:27:45 +0000 (16:27 +0100)
SL/Helper/Inventory.pm

index 8e18a6a..6933862 100644 (file)
@@ -116,16 +116,13 @@ sub _get_stock_onhand {
     for my $with_object (listify($params{with_objects})) {
       Carp::croak("unknown with_object $with_object") if !exists $with_objects{$with_object};
 
-      if (my $manager = $with_objects{$with_object}) {
-        my $slot = $slots{$with_object};
-        next if !(my @ids = map { $_->{$slot} } @$results);
-        my $objects = $manager->get_all(query => [ id => \@ids ]);
-        my %objects_by_id = map { $_->id => $_ } @$objects;
-
-        $_->{$with_object} = $objects_by_id{$_->{$slot}} for @$results;
-      } else {
-        # need to fetch all partitions
-      }
+      my $manager = $with_objects{$with_object};
+      my $slot = $slots{$with_object};
+      next if !(my @ids = map { $_->{$slot} } @$results);
+      my $objects = $manager->get_all(query => [ id => \@ids ]);
+      my %objects_by_id = map { $_->id => $_ } @$objects;
+
+      $_->{$with_object} = $objects_by_id{$_->{$slot}} for @$results;
     }
   }