diff --git a/user/lib.php b/user/lib.php
index 560e3112580..02dd4a7c6a2 100644
--- a/user/lib.php
+++ b/user/lib.php
@@ -509,7 +509,7 @@ function user_get_user_details_courses($user) {
     } else {
         // Try through course profile.
         foreach ($courses as $course) {
-            if ($can_view_user_details_cap($user, $course) || ($user->id == $USER->id) || has_coursecontact_role($user->id)) {
+            if (can_view_user_details_cap($user, $course) || ($user->id == $USER->id) || has_coursecontact_role($user->id)) {
                 $userdetails = user_get_user_details($user, $course);
             }
         }
diff --git a/user/tests/externallib_test.php b/user/tests/externallib_test.php
index 42f080ba543..43a6c832583 100644
--- a/user/tests/externallib_test.php
+++ b/user/tests/externallib_test.php
@@ -340,6 +340,16 @@ class core_user_external_testcase extends externallib_advanced_testcase {
 
         // Only the own $USER username should be returned
         $this->assertEquals(1, count($returnedusers));
+
+        // And finally test as one of the enrolled users.
+        $this->setUser($user1);
+
+        // Call the external function.
+        $returnedusers = core_user_external::get_users_by_field('username',
+            array($USER->username, $user1->username, $user2->username));
+
+        // Only the own $USER username should be returned still.
+        $this->assertEquals(1, count($returnedusers));
     }
 
     /**