Bug 10227

Summary: GetMessagingPreferences does not return correct Digest preferences
Product: Koha Reporter: Sophie MEYNIEUX <sophie.meynieux>
Component: PatronsAssignee: Sophie MEYNIEUX <sophie.meynieux>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, kyle
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10227 : Fix GetMessagingPreferences when digest is possible
Bug 10227 : Fix GetMessagingPreferences when digest is possible
Bug 10227 : Fix GetMessagingPreferences when digest is possible

Description Sophie MEYNIEUX 2013-05-10 15:40:50 UTC
SQL request in GetMessaginPreferences does not join on digest value, so, for a given patron, if you request his messaging preferences for a given message, it  returns 2 rows, one with digest=On and the other with digest=Off rather than juste one corresponding to the Digest preference of the patron.
Comment 1 Sophie MEYNIEUX 2016-08-24 13:01:15 UTC Comment hidden (obsolete)
Comment 2 Sophie MEYNIEUX 2016-08-24 13:22:16 UTC
This bug is a bit tricky to test, as the result is somehow  random, depending on the order of rows returned by sql request. You've got to change the preferences several times to see bad results.

Enable EnhancedMessaging preferences
Set preferences for a borrower or a category. 
run misc/cronjobs/advance_notices.pl and look in message_queue table the letter_code of the generated messages.
Change digest selection and do the test again several times 

Without patch, for message where digest is allowed (Advance_Notice, Item_Due), for a borrower or a category with 'wants_digest' selected, GetBorrowerMessagingPreferences returns sometimes letter_code with DGST and sometimes not.

With the patch, letter_code returned is always correct depending on digest selected or not
Comment 3 Chris Cormack 2016-09-09 01:06:32 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2016-09-12 17:01:37 UTC
Created attachment 55513 [details] [review]
Bug 10227 : Fix GetMessagingPreferences when digest is possible

This bug is a bit tricky to test, as the result is somehow  random, depending on the order of rows returned by sql request. You've got to change the preferences several times to see bad results.

Enable EnhancedMessaging preferences
Set preferences for a borrower or a category.
run misc/cronjobs/advance_notices.pl and look in message_queue table the letter_code of the generated messages.
Change digest selection and do the test again several times

Without patch, for message where digest is allowed (Advance_Notice, Item_Due), for a borrower or a category with 'wants_digest' selected, GetBorrowerMessagingPreferences returns sometimes letter_code with DGST and sometimes not.

With the patch, letter_code returned is always correct depending on digest selected or not

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Kyle M Hall 2016-09-13 18:17:19 UTC
Pushed to master for 16.11, thanks Sophie!