MDL-44148 portfolio: set_user_config usage fix

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:19:36 +08:00
parent c0e88129d1
commit 93075bf073

View File

@ -77,13 +77,10 @@ if (!empty($config)) {
print_error('confirmsesskeybad', '', $baseurl);
}
// This branch is where you process validated data.
$success = $instance->set_user_config($fromform, $USER->id);
if ($success) {
core_plugin_manager::reset_caches();
redirect($baseurl, get_string('instancesaved', 'portfolio'), 3);
} else {
print_error('instancenotsaved', 'portfolio', $baseurl);
}
$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'));