reporting objectional material with no message doesn't get sent to admin with digest emails
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Fix Released
|
Low
|
Ghada El-Zoghbi | ||
1.6 |
Fix Released
|
Low
|
Unassigned | ||
1.7 |
Fix Released
|
Low
|
Son Nguyen | ||
1.8 |
Fix Released
|
Low
|
Unassigned |
Bug Description
This results in an email sent to all admins saying that the notification failed.
The admin with digest on will not receive the empty message, but those without digest will.
Reason is because it tries to store the data in a temporary table to digest them all.
A couple of solutions:
- take off the non null constraint on the temporary table (will fix similar issues if any)
- set the message to be required (won't fix similar issues if any)
If we do the second option, we should have a hunt around other notification types to see if they can have null message and fix them as well
Subject: User notification error was probably caused by your server
configuration.
exception 'SQLException' with message 'Failed to get a recordset: postgres7
error: [-1: ERROR: null value in column "message" violates not-null
constraint] in adodb_throw(INSERT INTO "notification_
("type", "usr", "ctime", "url") VALUES (?, ?, ?, ?), Array)
Command was: INSERT INTO "notification_
"ctime", "url") VALUES (?, ?, ?, ?) and values was (7,2,2012-10-23
15:41:44,
/var/www/
Stack trace:
#0
/var/www/
insert_
#1 [internal function]:
PluginNotificat
#2
/var/www/
call_user_
#3
/var/www/
call_static_
Object(stdClass))
#4
/var/www/
ActivityType-
#5
/var/www/
ActivityType-
#6
/var/www/
handle_
#7
/var/www/
activity_
#8 {main}
Changed in mahara: | |
milestone: | 1.7.0 → 1.8.0 |
Changed in mahara: | |
milestone: | 1.8rc1 → 1.8.0 |
Changed in mahara: | |
milestone: | 1.8.0 → 1.8.1 |
Changed in mahara: | |
assignee: | nobody → Ghada El-Zoghbi (ghada-z) |
Changed in mahara: | |
status: | In Progress → Fix Committed |
no longer affects: | mahara/1.5 |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
no longer affects: | mahara/1.9 |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
This is related to the Alfred Health bug - WR215749.
Applying the same fix plus making the message mandatory for objectionalble content.