new simple SQL search field family ID - genomics db only

adapted Fluidx vials simple SQL search to use combined request_storage & request_lab_test_results tables, using new view request_storage_view - for Genomics only

added request notes entry form to screen page; moved notes methods from Result model & controller classes to Request classes

syntax bugfix; new seachable field Patient Notes

retired 'Outreach' screening term and replaced with separate outreach categories (ready to switch off auto-screening Outreach); stopped retrieveing outreach data in C::Report::outreach as it's (re)fetched in downstream load() function; defined pnh, hiv, outreach & molecular screening terms as global vars in Model::Audit - easier to amend categories; new simple SQL search field Request Notes

replaced 'email_from' config var with 'service_email' globally (except within unused functions)

bugfix to report notifications - leading 'C' now optional

auto-inserted request details for Outreach reports; removed commitment in email alerts to provide full written report as we are moving away from paper copies

updated report_notification table definition; updated tests to cope with clinical_details now required field (unless confif override); inserted error_clinical_details in tt

added ability for user to update report status (new/all) for report notifications; request_details (clinical details field) is an optional or required field delegated to config; mail_rports.pl now replaced by email_reports for UCLH & Leeds; preserve line-breaks for patient notes display

report_available_notification.pl deployment ready - using tt for uclh message body; rearranged order of cols in emailcontacts tt for clarity

changed report_available_notification.pl to determine whether notice is new or an update from time of authorisation not request status; commented dump_query in cml_rq_pcr_monitoring.pl

removed skip_redirect param from password_reset() and from cgiapp_prerun(), also removed redirect url on 1st login from cgiapp_prerun() - both seem to be superfluous, only need to override default /login/hello rm if 'destination' set in login(), otherwise app handles all redirects correctly; 15 new 01login.t tests for testing behaviour of redirects; 3529 tests, all pass

LIMS::login - replaced global destination_url (which was never used after re-authentication) with whitelisted GET urls which are safe to use as re-direct in either initial login or re-authentication after session timeout; enabled email_reports.pl to replace mail_reports.pl; report_available_notification.pl - added is_active to contact address query

bugfix to genomics_xml.pl, pdf's need to be transferred in binary mode, ascii_mode flag now only passed to ftp_file() for xml files; collecting destination_url info in login function to see if it's safe to use for idle timeout where it's not currently used; report notification only loads GMC number text box if user has nhs.net email address