SQL ANDs mus be, always, "AND" :-P

diff --git a/lib/accesslib.php b/lib/accesslib.php
index 229abd015025ad339039fcdac4d12c9d3b3465b5..407033a6525e8d7ff6d006fc553ba9d9e76bb972 100755 (executable)
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -561,7 +561,7 @@ function load_user_capability($capability='', $context ='', $userid='') {
 
 /// Set up SQL fragments for timestart, timeend etc
     $now = time();
-    $timesql = "AND ((ra.timestart = 0 OR ra.timestart < $now) && (ra.timeend = 0 OR ra.timeend > $now))";
+    $timesql = "AND ((ra.timestart = 0 OR ra.timestart < $now) AND (ra.timeend = 0 OR ra.timeend > $now))";
 
 /// Then we use 1 giant SQL to bring out all relevant capabilities.
 /// The first part gets the capabilities of orginal role.