Issue Details (XML | Word | Printable)

Key: MDL-17251
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Petr Skoda
Reporter: Gordon Bateson
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

admin\modules.php: Argument 4 passed to moodle_database::set_field() must be an array

Created: 17/Nov/08 09:54 AM   Updated: 18/Nov/08 10:13 AM
Return to search
Component/s: Administration
Affects Version/s: 2.0
Fix Version/s: 2.0

Environment: Moodle 2.0

Participants: Gordon Bateson and Petr Skoda
Security Level: None
Resolved date: 17/Nov/08
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
Catchable fatal error:
 - Argument 4 passed to moodle_database::set_field() must be an array, string given,
 - called in admin\modules.php on line 62
 - and defined in lib\dml\moodle_database.php on line 1165

lines 61 and 62 in "admin\modules.php" appear not to be necessary.
so one workaround is just to remove the offending lines
        // REMOVE 61: $DB->set_field('course_modules', 'visible', '1', 'visibleold',
        // REMOVE 62: '1', array('module'=>$module->id)); // Get the previous saved visible state for the course module.


 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Skoda made changes - 17/Nov/08 04:54 PM
Field Original Value New Value
Assignee moodle.com [ moodle.com ] Petr ?koda [ skodak ]
Petr Skoda committed 1 file to 'Moodle CVS' - 17/Nov/08 04:55 PM
MDL-17251 foxed incorrect set_field
MODIFY admin/modules.php   Rev. 1.60    (+2 -3 lines)
Petr Skoda added a comment - 17/Nov/08 04:55 PM
fixed thanks

Petr Skoda made changes - 17/Nov/08 04:55 PM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Fix Version/s 2.0 [ 10122 ]
Gordon Bateson added a comment - 18/Nov/08 10:13 AM
thank you

Gordon Bateson made changes - 18/Nov/08 10:13 AM
Status Resolved [ 5 ] Closed [ 6 ]