L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-44148 portfolio: set_user_config usage fix

Browse Source 
Amended user/portfolio.php's use of portfolio_plugin_base::set_user_config to be consistent with the API.
This commit is contained in:
Jetha Chan
2014-03-18 14:27:02 +08:00
parent 872f4e2899
commit 3a1237997a
1 changed files with 5 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
user/portfolio.php
+5 -9
View File
@@ -77,15 +77,11 @@ if (!empty($config)) {
if (!confirm_sesskey()) {
print_error('confirmsesskeybad', '', $baseurl);
}
//this branch is where you process validated data.
$success = $instance->set_user_config($fromform, $USER->id);
//$success = $success && $instance->save();
if ($success) {
core_plugin_manager::reset_caches();
redirect($baseurl, get_string('instancesaved', 'portfolio'), 3);
} else {
print_error('instancenotsaved', 'portfolio', $baseurl);
}
// This branch is where you process validated data.
$instance->set_user_config($fromform, $USER->id);
core_plugin_manager::reset_caches();
redirect($baseurl, get_string('instancesaved', 'portfolio'), 3);
exit;
} else {
echo $OUTPUT->heading(get_string('configplugin', 'portfolio'));