Skip to content
Browse files

MDL-35198 replace key_exists by array_key_exists

  • Loading branch information...
1 parent f9a8571 commit 71daa4a6d678dab1822ad7d54ade02158e706641 @mouneyrac mouneyrac committed Oct 8, 2012
View
2 admin/webservice/service_users.php
@@ -97,7 +97,7 @@
//add the missing capabilities to the allowed users object to be displayed by renderer
foreach ($allowedusers as &$alloweduser) {
- if (!is_siteadmin($alloweduser->id) and key_exists($alloweduser->id, $usersmissingcaps)) {
+ if (!is_siteadmin($alloweduser->id) and array_key_exists($alloweduser->id, $usersmissingcaps)) {
$alloweduser->missingcapabilities = implode(', ', $usersmissingcaps[$alloweduser->id]);
}
}
View
2 blocks/community/forms.php
@@ -140,7 +140,7 @@ public function definition() {
$options = array();
$firsthub = false;
foreach ($hubs as $hub) {
- if (key_exists('id', $hub)) {
+ if (array_key_exists('id', $hub)) {
$params = array('hubid' => $hub['id'],
'filetype' => HUB_HUBSCREENSHOT_FILE_TYPE);
$imgurl = new moodle_url(HUB_HUBDIRECTORYURL .
View
10 course/externallib.php
@@ -267,7 +267,7 @@ public static function get_courses($options = array()) {
array('options' => $options));
//retrieve courses
- if (!key_exists('ids', $params['options'])
+ if (!array_key_exists('ids', $params['options'])
or empty($params['options']['ids'])) {
$courses = $DB->get_records('course');
} else {
@@ -504,13 +504,13 @@ public static function create_courses($courses) {
require_capability('moodle/course:create', $context);
// Make sure lang is valid
- if (key_exists('lang', $course) and empty($availablelangs[$course['lang']])) {
+ if (array_key_exists('lang', $course) and empty($availablelangs[$course['lang']])) {
throw new moodle_exception(
get_string('errorinvalidparam', 'webservice', 'lang'));
}
// Make sure theme is valid
- if (key_exists('forcetheme', $course)) {
+ if (array_key_exists('forcetheme', $course)) {
if (!empty($CFG->allowcoursethemes)) {
if (empty($availablethemes[$course['forcetheme']])) {
throw new moodle_exception(
@@ -530,10 +530,10 @@ public static function create_courses($courses) {
//set default value for completion
$courseconfig = get_config('moodlecourse');
if (completion_info::is_enabled_for_site()) {
- if (!key_exists('enablecompletion', $course)) {
+ if (!array_key_exists('enablecompletion', $course)) {
$course['enablecompletion'] = $courseconfig->enablecompletion;
}
- if (!key_exists('completionstartonenrol', $course)) {
+ if (!array_key_exists('completionstartonenrol', $course)) {
$course['completionstartonenrol'] = $courseconfig->completionstartonenrol;
}
} else {
View
4 enrol/externallib.php
@@ -371,7 +371,7 @@ public static function assign_roles($assignments) {
// throw an exception if user is not able to assign the role in this context
$roles = get_assignable_roles($context, ROLENAME_SHORT);
- if (!key_exists($assignment['roleid'], $roles)) {
+ if (!array_key_exists($assignment['roleid'], $roles)) {
throw new invalid_parameter_exception('Can not assign roleid='.$assignment['roleid'].' in contextid='.$assignment['contextid']);
}
@@ -433,7 +433,7 @@ public static function unassign_roles($unassignments) {
// throw an exception if user is not able to unassign the role in this context
$roles = get_assignable_roles($context, ROLENAME_SHORT);
- if (!key_exists($unassignment['roleid'], $roles)) {
+ if (!array_key_exists($unassignment['roleid'], $roles)) {
throw new invalid_parameter_exception('Can not unassign roleid='.$unassignment['roleid'].' in contextid='.$unassignment['contextid']);
}
View
2 enrol/manual/externallib.php
@@ -90,7 +90,7 @@ public static function enrol_users($enrolments) {
//throw an exception if user is not able to assign the role
$roles = get_assignable_roles($context);
- if (!key_exists($enrolment['roleid'], $roles)) {
+ if (!array_key_exists($enrolment['roleid'], $roles)) {
$errorparams = new stdClass();
$errorparams->roleid = $enrolment['roleid'];
$errorparams->courseid = $enrolment['courseid'];
View
2 lib/adminlib.php
@@ -7584,7 +7584,7 @@ public function output_html($data, $query='') {
array(array('id' => $token->userid)), $token->serviceid);
if (!is_siteadmin($token->userid) and
- key_exists($token->userid, $usermissingcaps)) {
+ array_key_exists($token->userid, $usermissingcaps)) {
$missingcapabilities = implode(', ',
$usermissingcaps[$token->userid]);
if (!empty($missingcapabilities)) {
View
4 lib/upgradelib.php
@@ -877,7 +877,7 @@ function external_update_descriptions($component) {
$dbfunction->classpath = $function['classpath'];
$update = true;
}
- $functioncapabilities = key_exists('capabilities', $function)?$function['capabilities']:'';
+ $functioncapabilities = array_key_exists('capabilities', $function)?$function['capabilities']:'';
if ($dbfunction->capabilities != $functioncapabilities) {
$dbfunction->capabilities = $functioncapabilities;
$update = true;
@@ -893,7 +893,7 @@ function external_update_descriptions($component) {
$dbfunction->methodname = $function['methodname'];
$dbfunction->classpath = empty($function['classpath']) ? null : $function['classpath'];
$dbfunction->component = $component;
- $dbfunction->capabilities = key_exists('capabilities', $function)?$function['capabilities']:'';
+ $dbfunction->capabilities = array_key_exists('capabilities', $function)?$function['capabilities']:'';
$dbfunction->id = $DB->insert_record('external_functions', $dbfunction);
}
unset($functions);
View
2 webservice/lib.php
@@ -511,7 +511,7 @@ public function get_missing_capabilities_by_users($users, $serviceid) {
//detect the missing capabilities
foreach ($servicecaps as $functioname => $functioncaps) {
foreach ($functioncaps as $functioncap) {
- if (!key_exists($functioncap, $usercaps)) {
+ if (!array_key_exists($functioncap, $usercaps)) {
if (!isset($usersmissingcaps[$user->id])
or array_search($functioncap, $usersmissingcaps[$user->id]) === false) {
$usersmissingcaps[$user->id][] = $functioncap;
View
6 webservice/simpletest/testwebservice.php
@@ -448,15 +448,15 @@ function moodle_course_get_courses($client) {
$dbcourses[$course['id']]->timecreated);
$this->assertEqual($course['timemodified'],
$dbcourses[$course['id']]->timemodified);
- if (key_exists('enablecompletion', $course)) {
+ if (array_key_exists('enablecompletion', $course)) {
$this->assertEqual($course['enablecompletion'],
$dbcourses[$course['id']]->enablecompletion);
}
- if (key_exists('completionstartonenrol', $course)) {
+ if (array_key_exists('completionstartonenrol', $course)) {
$this->assertEqual($course['completionstartonenrol'],
$dbcourses[$course['id']]->completionstartonenrol);
}
- if (key_exists('completionnotify', $course)) {
+ if (array_key_exists('completionnotify', $course)) {
$this->assertEqual($course['completionnotify'],
$dbcourses[$course['id']]->completionnotify);
}

0 comments on commit 71daa4a

Please sign in to comment.
Something went wrong with that request. Please try again.