MDL-74317 output: Remove capability checks in edit mode WS

- Removed validation on the change edit mode WS. Permission should
be handled where the edit button is rendered in order to show or not
the button.
This commit is contained in:
David Matamoros
2022-04-26 14:33:34 +02:00
parent e36fb751e5
commit f551072f9e
2 changed files with 7 additions and 33 deletions
+1 -9
View File
@@ -34,14 +34,6 @@ require_login();
$context = \context_helper::instance_by_id($contextid);
$PAGE->set_context($context);
if ($context->id === \context_user::instance($USER->id)->id) {
$PAGE->set_blocks_editing_capability('moodle/my:manageblocks');
}
if ($PAGE->user_allowed_editing()) {
$USER->editing = $setmode;
} else {
\core\notification::add(get_string('cannotswitcheditmodeon', 'error'), \core\notification::ERROR);
}
$USER->editing = $setmode;
redirect($pageurl);
+6 -24
View File
@@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
/**
* A web service to load the mapping of moodle pix names to fontawesome icon names.
*
* @package core
* @category external
* @copyright 2021 Bas Brands <[email protected]>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
namespace core\external;
use external_api;
@@ -54,14 +45,14 @@ class editmode extends external_api {
}
/**
* Save the image and return any warnings and the new image url
* Set the given edit mode
*
* @param bool $setmode the current edit mode
* @param bool $setmode the new edit mode
* @param int $contextid the current page context id
* @return array the new edit mode.
* @return array
*/
public static function change_editmode(bool $setmode, int $contextid): array {
global $USER, $PAGE;
global $USER;
$params = self::validate_parameters(
self::change_editmode_parameters(),
@@ -73,19 +64,10 @@ class editmode extends external_api {
$context = \context_helper::instance_by_id($params['context']);
self::validate_context($context);
$PAGE->set_context($context);
if ($context->id === \context_user::instance($USER->id)->id) {
$PAGE->set_blocks_editing_capability('moodle/my:manageblocks');
}
$USER->editing = $params['setmode'];
$success = false;
if ($PAGE->user_allowed_editing()) {
$USER->editing = $setmode;
$success = true;
}
return ['success' => $success];
return ['success' => true];
}
/**