From d8e6ba7cac90755c81efcbbb1a97bf0d642fdadf Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 30 Mar 2020 09:39:13 -0300 Subject: [PATCH] Bug 25019: Make the ViewPolicy filter initialization standard This patch makes the ViewPolicy filter use the 'params' accessor instead of relying of ->{options} which has no accessor. This will allow interacting with the filter object be similar through all the filters in the chain. To test, we just need to verify no behaviour change takes place: 1. Run: $ kshell k$ prove t/db_dependent/Filter_MARC_ViewPolicy.t => SUCCESS: Tests pass 2. Apply this patch 3. Repeat 1 => SUCCESS: Tests still pass! 4. Sign off :-D --- Koha/Filter/MARC/ViewPolicy.pm | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/Koha/Filter/MARC/ViewPolicy.pm b/Koha/Filter/MARC/ViewPolicy.pm index e69b7303b7..a963e3f129 100644 --- a/Koha/Filter/MARC/ViewPolicy.pm +++ b/Koha/Filter/MARC/ViewPolicy.pm @@ -78,10 +78,12 @@ sub filter { push @records, $precord; } + my $params = $self->params; + my $interface = $params->{options}->{interface} // 'opac'; + my $frameworkcode = $params->{options}->{frameworkcode} // q{}; + foreach my $current_record (@records) { my $result = $current_record; - my $interface = $self->{options}->{interface} // 'opac'; - my $frameworkcode = $self->{options}->{frameworkcode} // q{}; my $hide = _should_hide_on_interface(); my $marcsubfieldstructure = GetMarcStructure( 0, $frameworkcode, { unsafe => 1 } ); @@ -153,16 +155,6 @@ sub _filter_field { return; } -sub initialize { - my $self = shift; - my $param = shift; - - my $options = $param->{options}; - $self->{options} = $options; - $self->Koha::RecordProcessor::Base::initialize($param); - return; -} - # Copied and modified from 3.10.x help file # marc_subfields_structure.hidden # allows you to select from 19 possible visibility conditions, 17 of which are implemented. They are the following: -- 2.26.0