Skip to content

Commit

Permalink
MDL-74317 output: Remove capability checks in edit mode WS
Browse files Browse the repository at this point in the history
- 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.
  • Loading branch information
dravek committed Apr 26, 2022
1 parent e36fb75 commit f551072
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 33 deletions.
10 changes: 1 addition & 9 deletions editmode.php
Expand Up @@ -34,14 +34,6 @@
$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);
30 changes: 6 additions & 24 deletions lib/classes/external/editmode.php
Expand Up @@ -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 <bas@sonsbeekmedia.nl>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

namespace core\external;

use external_api;
Expand Down Expand Up @@ -54,14 +45,14 @@ public static function change_editmode_parameters(): external_function_parameter
}

/**
* 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(),
Expand All @@ -73,19 +64,10 @@ public static function change_editmode(bool $setmode, int $contextid): array {

$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');
}

$success = false;
if ($PAGE->user_allowed_editing()) {
$USER->editing = $setmode;
$success = true;
}
$USER->editing = $params['setmode'];

return ['success' => $success];
return ['success' => true];
}

/**
Expand Down

0 comments on commit f551072

Please sign in to comment.