At our library we need to be able to marc exported records for deletion depending on a marc conditions. This patch adds a new argument that provides that functionality. The tests t/db_dependent/Exporter/Record.t fails in current master, which is an issue when running the included tests, but it seems like an unrelated bug. I will look into it and create a separate issue regarding this.
Created attachment 90189 [details] [review] Bug 23009: Add -deleted_marc_conditions to export_records Add -deleted_marc_conditions argument to export_records script for setting deleted flag on records if condition matches.
Regarding the bug causing the tests to fail it was just a mismatch of my current schema version and code base, so forget about that part :)
Created attachment 133121 [details] [review] Bug 23009: Add -set_deleted_marc_conditions to export_records Add -set_deleted_marc_conditions argument to export_records script for setting deleted flag on records if condition matches. To test: 1) Run tests in t/db_dependent/Exporter/Record.t Sponsored-by: Gothenburg University Library
Rebased. Also renamed option to something more descriptive, the previous was pretty awful as you would probably expect the conditions be applied the same way as for normal records but for deleted ones. -set_deleted_marc_conditions will set the deleted flag for matching records, and are not conditions applied to filter records out as the previous parameter name implied.
Created attachment 177550 [details] [review] Bug 23009: Add -set_deleted_marc_conditions to export_records Add -set_deleted_marc_conditions argument to export_records script for setting deleted flag on records if condition matches. To test: 1) Run tests in t/db_dependent/Exporter/Record.t Sponsored-by: Gothenburg University Library
Rebased against master.