Bug 11279: Improve how a new quote of the day is selected
authorKatrin Fischer <Katrin.Fischer.83@web.de>
Wed, 11 Dec 2013 23:54:08 +0000 (00:54 +0100)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 19 Dec 2013 15:06:31 +0000 (15:06 +0000)
commit5a3db23362eb076a2f6d5a87f87b29f4a436a14f
tree34960d96b1cfb8a9e9939e82ba96c6c4bbb2ed14
parentf874eec0793f83c3c0f1461fecb878ca57ed701b
Bug 11279: Improve how a new quote of the day is selected

If there is a gap in the id sequence for the quotes table, it
is possible that no new quote will be selected. This will happen
particularly when a lot of the older quotes with low ids have been
deleted.

This patch improves the selection of a new quote.

To test:
- Load sample quotes
- Delete the first half of the quotes.
  Note: With 34 quotes, delete the quotes with ids from 1-17
- Activate the QuoteOfTheDay system preference
- Check if a quote is displayed in OPAC
- Reload the page a few times, no quote should be displayed
  Note: make sure you don't have a quote with the current
  date in your quotes table before running those tests
- Run 'perl t/db_dependent/Koha.t'
  Note: requires sample quotes!
- Apply patch
- Reload the OPAC start page
- Verify a quote was now picked
- Run 'perl t/db/dependent/Koha.t' again - all tests should still pass

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Patch modified to use "LIMIT 1 OFFSET ?" rather than "LIMIT ?, 1"; the
latter construction does not work in PostgreSQL.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/Koha.pm