Closed (fixed)
Project:
jGrowl
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Feb 2010 at 20:35 UTC
Updated:
5 Mar 2010 at 09:40 UTC
If you uncheck the 'Override System Messages' option, jgrowl does not pass control back to the regular system message theme function.
I added a check against JGROWL_SYSTEM_MESSAGES before overriding the status_message theme function and that fixed the problem:
function jgrowl_theme_registry_alter(&$theme_registry) {
if ((user_access('access jgrowl'))) {
if (!empty($theme_registry['status_messages']) && JGROWL_SYSTEM_MESSAGES) {
$theme_registry['status_messages']['function'] = 'jgrowl_system_messages';
}
}
}
Comments
Comment #1
btully commentedGood catch bfcam! We're in the middle of converting a site from 5.x to 6.x and encountered this very same issue.
Hopefully this can be fixed in the next release.
Comment #2
ilo commentedSorry for being so late with with simple one, just got busy. I've committed the change in the devel branch. Thank you both!