LP#1745462: guard against scanning entire authority table
authorGalen Charlton <gmc@equinoxinitiative.org>
Thu, 25 Jan 2018 19:40:47 +0000 (14:40 -0500)
committerChris Sharp <csharp@georgialibraries.org>
Wed, 11 Jul 2018 18:28:58 +0000 (14:28 -0400)
commit97422ed57cdab35625e4274ebfb305180c06736c
treec0c9e3c43c1f2cd6e14ae3212f49095aabe544bc
parent0a9e00ed0ada77c89b4b0a10600a49e1989ccf10
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>
Open-ILS/src/perlmods/lib/OpenILS/Application/Search/Authority.pm