Change #5498
Category | None |
Changed by | Galen Charlton <gmc | @equinoxinitiative.org>
Changed at | Wed 11 Jul 2018 14:29:20 |
Repository | git://git.evergreen-ils.org/Evergreen.git |
Project | Evergreen |
Branch | master |
Revision | 91cd25fbcab912aabfc0df204b46b5a67ab6d8b8 |
Comments
LP#1745462: guard against scanning entire authority table This patch ensures that if, for whatever reason, a MARC editor headings validation action includes a field whose subfield values are empty, it ignores any cases where the normalized heading works out to NULL or the empty string. Otherwise, the database can be asked to fetch the IDs of most/all records in the database, and open-ils.cstore backend can be asked to store the entire result set in memory. To test ------- [0] Ensure that statement logging is turned on in the PostgreSQL database. [1] In the web staff client, create a new bib. Ensure that at least one of the authority-controlled fields has no subfield values. [2] Hit the Validate button. [3] Note that the following query is logged by the database: SELECT "are".id AS "id" FROM authority.record_entry AS "are" WHERE "are".control_set = '1' AND "are".deleted = 'f' AND "are".simple_heading IS NOT NULL; [4] Apply the patch and repeat steps 1 and 2. This time, note that no such query is recorded. [5] Verify that validating headings that are not empty does continue to work. Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org> Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Changed files
- Open-ILS/src/perlmods/lib/OpenILS/Application/Search/Authority.pm