Bug 5603 - advance_notices.pl generating "uninitialized value in hash element"
Summary: advance_notices.pl generating "uninitialized value in hash element"
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Notices (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Ian Walls
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-11 03:12 UTC by Ian Walls
Modified: 2012-10-25 23:03 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
patch (1.29 KB, patch)
2011-01-11 15:37 UTC, Nicole C. Engard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Ian Walls 2011-01-11 03:12:16 UTC
When running advance_notices.pl, numerous lines of the following are generated:

Use of uninitialized value in hash element at /home/koha/kohaclone/C4/Members/Messaging.pm line 101.

Investigation shows this is due to the SQL in GetMessagingPreferences returning null values for message_transport_type for advanced messaging types NOT utilized by patrons.

The solution is to change the LEFT JOIN on line 77 to JOIN; this eliminates the NULL entries.  Apparently, the patch for bug 4254 didn't factor in that NULL values would be returned now that the duplicate columns were eliminated.
Comment 1 Nicole C. Engard 2011-01-11 15:37:03 UTC
Created attachment 2972 [details] [review]
patch
Comment 2 Chris Cormack 2011-01-11 20:00:52 UTC
Pushed, please test
Comment 3 MJ Ray (software.coop) 2011-02-06 11:13:57 UTC
This bug is mentioned in:
Bug 5603: advance_notice.pl	uninitialized element in hash http://lists.koha-community.org/pipermail/koha-patches/2011-January/013574.html
Comment 4 Ian Walls 2011-03-10 15:08:31 UTC
Tested, works.