Bug 24254: Compare itemlost with 0

[koha-ffzg.git] / Koha / Items.pm

diff --git a/Koha/Items.pm b/Koha/Items.pm
index 057a361..f123a61 100644 (file)
--- a/Koha/Items.pm
+++ b/Koha/Items.pm
@@ -33,7 +33,7 @@ Koha::Items - Koha Item object set class
 
 =head1 API
 
-=head2 Class Methods
+=head2 Class methods
 
 =cut
 
@@ -50,7 +50,46 @@ sub filter_by_for_loan {
     return $self->search( { notforloan => [ 0, undef ] } );
 }
 
-=head3 type
+=head3 filter_by_visible_in_opac
+
+    my $filered_items = $items->filter_by_visible_in_opac({ rules => $rules });
+
+Returns a new resultset, containing those items that are not expected to be hidden in OPAC.
+If no I<rules> are passed, it returns the whole resultset, with the only caveat that the
+I<hidelostitems> system preference is honoured.
+
+=cut
+
+sub filter_by_visible_in_opac {
+    my ($self, $params) = @_;
+
+    my $rules = $params->{rules} // {};
+
+    my $rules_params;
+    foreach my $field (keys %$rules){
+        $rules_params->{$field}->{'-not_in'} = $rules->{$field};
+    }
+
+    my $itemlost_params;
+    $itemlost_params = { itemlost => 0 }
+        if C4::Context->preference('hidelostitems');
+
+    my $search_params;
+    if ( $rules_params and $itemlost_params ) {
+        $search_params = {
+            '-and' => [ $rules_params, $itemlost_params ]
+        };
+    }
+    else {
+        $search_params = $rules_params // $itemlost_params;
+    }
+
+    return $self->search( $search_params );
+}
+
+=head2 Internal methods
+
+=head3 _type
 
 =cut