Before fetching the biblio, we can check if agerestriction is mapped to a field. The structure is cached, so we can save on lookup when not using this feature
Created attachment 127788 [details] [review] Bug 29517: Check if agerestriction field is mapped before fetching biblio This patch simply adds a check of cached values to see if agerestriction enabled before fetching the biblio object To test: 1 - prove -v t/db_dependent/Holds.t
Created attachment 131541 [details] [review] Bug 29517: Check if agerestriction field is mapped before fetching biblio This patch simply adds a check of cached values to see if agerestriction enabled before fetching the biblio object To test: 1 - prove -v t/db_dependent/Holds.t Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Nice performance catch
Created attachment 132218 [details] [review] Bug 29517: Check if agerestriction field is mapped before fetching biblio This patch simply adds a check of cached values to see if agerestriction enabled before fetching the biblio object To test: 1 - prove -v t/db_dependent/Holds.t Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed to master for 22.05, thanks to everybody involved [U+1F984]
Pushed to 21.11.x for 21.11.05