MDL-67707 course: add support for PARAM_ALPHANUM module names
authorFerran Recio <ferran@moodle.com>
Wed, 5 Feb 2020 15:57:11 +0000 (16:57 +0100)
committerFerran Recio <ferran@moodle.com>
Mon, 16 Mar 2020 16:52:08 +0000 (17:52 +0100)
course/mod.php
course/modedit.php
course/moodleform_mod.php
course/upgrade.txt

index 54f6f9c..67719a7 100644 (file)
@@ -27,7 +27,7 @@ require("../config.php");
 require_once("lib.php");
 
 $sectionreturn = optional_param('sr', null, PARAM_INT);
-$add           = optional_param('add', '', PARAM_ALPHA);
+$add           = optional_param('add', '', PARAM_ALPHANUM);
 $type          = optional_param('type', '', PARAM_ALPHA);
 $indent        = optional_param('indent', 0, PARAM_INT);
 $update        = optional_param('update', 0, PARAM_INT);
index f048bcd..0388708 100644 (file)
@@ -31,7 +31,7 @@ require_once($CFG->libdir.'/completionlib.php');
 require_once($CFG->libdir.'/plagiarismlib.php');
 require_once($CFG->dirroot . '/course/modlib.php');
 
-$add    = optional_param('add', '', PARAM_ALPHA);     // module name
+$add    = optional_param('add', '', PARAM_ALPHANUM);     // Module name.
 $update = optional_param('update', 0, PARAM_INT);
 $return = optional_param('return', 0, PARAM_BOOL);    //return to course/view.php if false or mod/modname/view.php if true
 $type   = optional_param('type', '', PARAM_ALPHANUM); //TODO: hopefully will be removed in 2.0
index 34c25f0..2bbdac0 100644 (file)
@@ -942,7 +942,7 @@ abstract class moodleform_mod extends moodleform {
         $mform->setType('instance', PARAM_INT);
 
         $mform->addElement('hidden', 'add', 0);
-        $mform->setType('add', PARAM_ALPHA);
+        $mform->setType('add', PARAM_ALPHANUM);
 
         $mform->addElement('hidden', 'update', 0);
         $mform->setType('update', PARAM_INT);
index f88a58b..195a6eb 100644 (file)
@@ -4,6 +4,7 @@ information provided here is intended especially for developers.
 === 3.9 ===
 
 * The function get_module_metadata is now deprecated. Please use \core_course\local\service\content_item_service instead.
+* Activity module names are now PARAM_ALPHANUM instead of PARAM_ALPHA so integers can be used in activity module names
 
 === 3.8 ===