Put back DEBUG_DEVELOPER sanity check in has_capability, that makes sure the capabilt...

diff --git a/lib/accesslib.php b/lib/accesslib.php
index bd8e27ab9e5475e14a59886d092f906b97a380f7..3de14bee8c4e5f86f08ae2d72fbf5f7b3cded0d6 100755 (executable)
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -327,6 +327,16 @@ function has_capability($capability, $context, $userid=NULL, $doanything=true) {
         return false;
     }
 
+/// 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.');
+        }
+        if ($doanything != true and $doanything != false) {
+            debugging('Capability parameter "doanything" is wierd ("'.$doanything.'"). This should be fixed in code.');
+        }
+    }
+
     if (empty($userid)) { // we must accept null, 0, '0', '' etc. in $userid
         $userid = $USER->id;
     }