|
|
| Participants: |
Anthony Borrow and Petr Skoda
|
| Security Level: |
None
|
| Resolved date: |
17/Mar/08
|
| Affected Branches: |
MOODLE_19_STABLE
|
| Fixed Branches: |
MOODLE_19_STABLE
|
|
When posting a rating with debugging turned to ALL I received the following PHP notice
Notice: Undefined property: stdClass::$cmidnumber in /home/arborrow/Moodle/code/19stable/mod/forum/lib.php on line 1168
This is caused because the forum_grade_item_update function in /mod/forum/lib.php expects the $forum variable to have a field $cmidnumber ( * @param object $forum object with extra cmidnumber). I traced this back to /mod/forum/rate.php and found that the extra cmidnumber field was not being added on prior to passing the variable to the forum_grade_item_update function.
I'll attach one possible patch that I used to get around this.
|
|
Description
|
When posting a rating with debugging turned to ALL I received the following PHP notice
Notice: Undefined property: stdClass::$cmidnumber in /home/arborrow/Moodle/code/19stable/mod/forum/lib.php on line 1168
This is caused because the forum_grade_item_update function in /mod/forum/lib.php expects the $forum variable to have a field $cmidnumber ( * @param object $forum object with extra cmidnumber). I traced this back to /mod/forum/rate.php and found that the extra cmidnumber field was not being added on prior to passing the variable to the forum_grade_item_update function.
I'll attach one possible patch that I used to get around this.
|
Show » |
made changes - 13/Jan/08 12:56 PM
| Field |
Original Value |
New Value |
|
Attachment
|
|
MDL-12961.diff
[ 12813
]
|
made changes - 17/Mar/08 08:25 AM
|
Assignee
|
Martin Dougiamas
[ dougiamas
]
|
Petr Skoda
[ skodak
]
|
made changes - 17/Mar/08 08:29 AM
|
Fix Version/s
|
|
1.9
[ 10190
]
|
|
Status
|
Open
[ 1
]
|
Resolved
[ 5
]
|
|
Resolution
|
|
Fixed
[ 1
]
|
|