Bug 15870: potential follow up to comment #8
authorMark Tompsett <mtompset@hotmail.com>
Tue, 8 Mar 2016 02:34:45 +0000 (21:34 -0500)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Thu, 24 Mar 2016 20:15:40 +0000 (20:15 +0000)
commit4e1289a55d7eee0987216a306278cba8c2d66036
tree8c7688249dd1a7376019f674effc02791d07d6d9
parentc9170233c976b3ac05f8712fdc2c9aa25973f34f
 Bug 15870: potential follow up to comment #8

This patch:
- makes the Koha::RecordProcessor code more clear
  by removing the unnecessary newrecord variable.
- revises the filter to be more clear about the
  expectation that operations are done directly on
  the record parameter.

TEST PLAN
---------
prove t/RecordProcessor.t
prove t/db_dependent/Filter_MARC_ViewPolicy.t
run koha qa test tools

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Works as advertised. NO koha-qa errors
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
Koha/Filter/MARC/ViewPolicy.pm
Koha/RecordProcessor.pm