Merge branch 'MDL-70893-master' of git://github.com/marinaglancy/moodle
This commit is contained in:
+1
-1
@@ -2880,7 +2880,7 @@ function calendar_add_subscription($sub) {
|
||||
* @throws dml_exception A DML specific exception is thrown for invalid subscriptionids.
|
||||
* @return int Code: CALENDAR_IMPORT_EVENT_UPDATED = updated, CALENDAR_IMPORT_EVENT_INSERTED = inserted, 0 = error
|
||||
*/
|
||||
function calendar_add_icalendar_event($event, $unused = null, $subscriptionid, $timezone='UTC') {
|
||||
function calendar_add_icalendar_event($event, $unused, $subscriptionid, $timezone='UTC') {
|
||||
global $DB;
|
||||
|
||||
// Probably an unsupported X-MICROSOFT-CDO-BUSYSTATUS event.
|
||||
|
||||
@@ -316,7 +316,7 @@ class core_calendar_renderer extends plugin_renderer_base {
|
||||
* @param string $importresults
|
||||
* @return string
|
||||
*/
|
||||
public function subscription_details($unused = null, $subscriptions, $importresults = '') {
|
||||
public function subscription_details($unused, $subscriptions, $importresults = '') {
|
||||
$table = new html_table();
|
||||
$table->head = array(
|
||||
get_string('colcalendar', 'calendar'),
|
||||
|
||||
+1
-1
@@ -3074,7 +3074,7 @@ class course_request {
|
||||
* @param string $message
|
||||
* @param int|null $courseid
|
||||
*/
|
||||
protected function notify($touser, $fromuser, $name='courserequested', $subject, $message, $courseid = null) {
|
||||
protected function notify($touser, $fromuser, $name, $subject, $message, $courseid = null) {
|
||||
$eventdata = new \core\message\message();
|
||||
$eventdata->courseid = empty($courseid) ? SITEID : $courseid;
|
||||
$eventdata->component = 'moodle';
|
||||
|
||||
@@ -57,7 +57,7 @@ class grade_edit_tree {
|
||||
/**
|
||||
* Constructor
|
||||
*/
|
||||
public function __construct($gtree, $moving=false, $gpr) {
|
||||
public function __construct($gtree, $moving, $gpr) {
|
||||
global $USER, $OUTPUT, $COURSE;
|
||||
|
||||
$systemdefault = get_config('moodle', 'grade_report_showcalculations');
|
||||
|
||||
@@ -3285,7 +3285,7 @@ function user_get_total_participants($courseid, $groupid = 0, $accesssince = 0,
|
||||
* @param int $limitnum return a subset comprising this many records (optional, required if $limitfrom is set).
|
||||
* @return moodle_recordset
|
||||
*/
|
||||
function user_get_participants($courseid, $groupid = 0, $accesssince, $roleid, $enrolid = 0, $statusid, $search,
|
||||
function user_get_participants($courseid, $groupid, $accesssince, $roleid, $enrolid, $statusid, $search,
|
||||
$additionalwhere = '', $additionalparams = array(), $sort = '', $limitfrom = 0, $limitnum = 0) {
|
||||
global $DB;
|
||||
|
||||
|
||||
@@ -913,7 +913,7 @@ class phpunit_util extends testing_util {
|
||||
* or the name of the static class when calling a static method.
|
||||
* @return mixed the respective return value of the method.
|
||||
*/
|
||||
public static function call_internal_method($object, $methodname, array $params = array(), $classname) {
|
||||
public static function call_internal_method($object, $methodname, array $params, $classname) {
|
||||
$reflection = new \ReflectionClass($classname);
|
||||
$method = $reflection->getMethod($methodname);
|
||||
$method->setAccessible(true);
|
||||
|
||||
+1
-1
@@ -488,7 +488,7 @@ function rss_end_tag($tag,$level=0,$endline=true) {
|
||||
* @param array $attributes the attributes of the xml tag
|
||||
* @return string the whole xml element
|
||||
*/
|
||||
function rss_full_tag($tag,$level=0,$endline=true,$content,$attributes=null) {
|
||||
function rss_full_tag($tag, $level, $endline, $content, $attributes = null) {
|
||||
$st = rss_start_tag($tag,$level,$endline,$attributes);
|
||||
$co="";
|
||||
$co = preg_replace("/\r\n|\r/", "\n", htmlspecialchars($content));
|
||||
|
||||
@@ -2920,7 +2920,7 @@ class core_message_external extends external_api {
|
||||
* @throws moodle_exception
|
||||
* @since 3.2
|
||||
*/
|
||||
public static function get_message_processor($userid = 0, $name) {
|
||||
public static function get_message_processor($userid, $name) {
|
||||
global $USER, $PAGE, $CFG;
|
||||
|
||||
// Check if messaging is enabled.
|
||||
|
||||
+1
-1
@@ -49,7 +49,7 @@ class mnet_peer {
|
||||
* @param int $application - table id - what kind of peer are we talking to
|
||||
* @return bool - indication of success or failure
|
||||
*/
|
||||
function bootstrap($wwwroot, $pubkey = null, $application) {
|
||||
function bootstrap($wwwroot, $pubkey, $application) {
|
||||
global $DB;
|
||||
|
||||
if (substr($wwwroot, -1, 1) == '/') {
|
||||
|
||||
@@ -2080,7 +2080,7 @@ class mod_assign_external extends external_api {
|
||||
* @return null
|
||||
* @since Moodle 2.7
|
||||
*/
|
||||
public static function save_grades($assignmentid, $applytoall = false, $grades) {
|
||||
public static function save_grades($assignmentid, $applytoall, $grades) {
|
||||
global $CFG, $USER;
|
||||
|
||||
$params = self::validate_parameters(self::save_grades_parameters(),
|
||||
|
||||
@@ -395,7 +395,7 @@ class pdf extends TcpdfFpdi {
|
||||
* @param string $imagefolder - Folder containing stamp images.
|
||||
* @return bool true if successful (always)
|
||||
*/
|
||||
public function add_annotation($sx, $sy, $ex, $ey, $colour = 'yellow', $type = 'line', $path, $imagefolder) {
|
||||
public function add_annotation($sx, $sy, $ex, $ey, $colour, $type, $path, $imagefolder) {
|
||||
global $CFG;
|
||||
if (!$this->filename) {
|
||||
return false;
|
||||
|
||||
@@ -1016,7 +1016,7 @@ class mod_data_search_test extends advanced_testcase {
|
||||
* @param mod_data $data
|
||||
* @return void
|
||||
*/
|
||||
protected function create_default_data_fields($fieldtypes = array(), $data) {
|
||||
protected function create_default_data_fields($fieldtypes, $data) {
|
||||
$count = 1;
|
||||
|
||||
// Creating test Fields with default parameter values.
|
||||
|
||||
@@ -2179,7 +2179,7 @@ class mod_forum_external extends external_api {
|
||||
* @param string $sortdirection
|
||||
* @return array
|
||||
*/
|
||||
public static function get_discussion_posts_by_userid(int $userid = 0, int $cmid, ?string $sortby, ?string $sortdirection) {
|
||||
public static function get_discussion_posts_by_userid(int $userid, int $cmid, ?string $sortby, ?string $sortdirection) {
|
||||
global $USER, $DB;
|
||||
// Validate the parameter.
|
||||
$params = self::validate_parameters(self::get_discussion_posts_by_userid_parameters(), [
|
||||
|
||||
+1
-1
@@ -1162,7 +1162,7 @@ function forum_get_readable_forums($userid, $courseid=0) {
|
||||
* @param string $extrasql
|
||||
* @return array|bool Array of posts found or false
|
||||
*/
|
||||
function forum_search_posts($searchterms, $courseid=0, $limitfrom=0, $limitnum=50,
|
||||
function forum_search_posts($searchterms, $courseid, $limitfrom, $limitnum,
|
||||
&$totalcount, $extrasql='') {
|
||||
global $CFG, $DB, $USER;
|
||||
require_once($CFG->libdir.'/searchlib.php');
|
||||
|
||||
@@ -2470,7 +2470,7 @@ function glossary_end_tag($tag,$level=0,$endline=true) {
|
||||
* @param string $content
|
||||
* @return string
|
||||
*/
|
||||
function glossary_full_tag($tag,$level=0,$endline=true,$content) {
|
||||
function glossary_full_tag($tag, $level, $endline, $content) {
|
||||
global $CFG;
|
||||
|
||||
$st = glossary_start_tag($tag,$level,$endline);
|
||||
|
||||
@@ -812,7 +812,7 @@ function scorm_get_available_children($sco) {
|
||||
}
|
||||
}
|
||||
|
||||
function scorm_get_available_descendent($descend = array(), $sco) {
|
||||
function scorm_get_available_descendent($descend, $sco) {
|
||||
if ($sco == null) {
|
||||
return $descend;
|
||||
} else {
|
||||
|
||||
@@ -277,7 +277,7 @@ class content_writer implements \core_privacy\local\request\content_writer {
|
||||
* @param boolean $valueonly Whether to fetch only the value, rather than the value + description.
|
||||
* @return array The metadata as a series of keys to value + descrition objects.
|
||||
*/
|
||||
public function get_metadata(array $subcontext = [], $key, $valueonly = true) {
|
||||
public function get_metadata(array $subcontext, $key, $valueonly = true) {
|
||||
$keys = $this->get_all_metadata($subcontext);
|
||||
|
||||
if (isset($keys[$key])) {
|
||||
|
||||
@@ -66,7 +66,7 @@ class category_condition extends condition {
|
||||
* @param \stdClass $course Course record
|
||||
* @param integer $maxinfolength The maximum displayed length of the category info.
|
||||
*/
|
||||
public function __construct($cat = null, $recurse = false, $contexts, $baseurl, $course, $maxinfolength = null) {
|
||||
public function __construct($cat, $recurse, $contexts, $baseurl, $course, $maxinfolength = null) {
|
||||
$this->cat = $cat;
|
||||
$this->recurse = $recurse;
|
||||
$this->contexts = $contexts;
|
||||
|
||||
@@ -64,7 +64,7 @@ class insight implements \renderable, \templatable {
|
||||
* @param \context $context
|
||||
* @return void
|
||||
*/
|
||||
public function __construct(\core_analytics\prediction $prediction, \core_analytics\model $model, $includedetailsaction = false,
|
||||
public function __construct(\core_analytics\prediction $prediction, \core_analytics\model $model, $includedetailsaction,
|
||||
\context $context) {
|
||||
|
||||
$this->prediction = $prediction;
|
||||
|
||||
@@ -341,7 +341,7 @@ class repository_flickr_public extends repository {
|
||||
* @param int $page
|
||||
* @return array
|
||||
*/
|
||||
private function build_list($photos, $page = 1, &$ret) {
|
||||
private function build_list($photos, $page, &$ret) {
|
||||
global $OUTPUT;
|
||||
|
||||
if (!empty($this->nsid)) {
|
||||
|
||||
Reference in New Issue
Block a user