L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tweak has_capability() debug mode to add only one query per page for cap name verification; merged from MOODLE_19_STABLE

Browse Source
This commit is contained in:
skodak
2008-01-16 17:24:08 +00:00
parent 34d04affe3
commit 588cd516e1
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/accesslib.php
+15 -3
View File
@@ -329,10 +329,22 @@ function has_capability($capability, $context, $userid=NULL, $doanything=true) {
/// Some sanity checks
if (debugging('',DEBUG_DEVELOPER)) {
if (!record_exists('capabilities', 'name', $capability)) {
debugging('Capability "'.$capability.'" was not found! This should be fixed in code.');
static $capsnames = null; // one request per page only
if (is_null($capsnames)) {
if ($caps = get_records('capabilities', '', '', '', 'id, name')) {
$capsnames = array();
foreach ($caps as $cap) {
$capsnames[$cap->name] = true;
}
}
}
if ($doanything != true and $doanything != false) {
if ($capsnames) { // ignore if can not fetch caps
if (!isset($capsnames[$capability])) {
debugging('Capability "'.$capability.'" was not found! This should be fixed in code.');
}
}
if (!is_bool($doanything)) {
debugging('Capability parameter "doanything" is wierd ("'.$doanything.'"). This should be fixed in code.');
}
}