From 7406ee9f57ed0fd7624d471e8acd05b871d75f34 Mon Sep 17 00:00:00 2001 From: Dan Poltawski Date: Fri, 1 Jun 2012 10:55:18 +0800 Subject: [PATCH] MDL-32117 equella: Switch get_all_editing_roles to get_users_with_capability This is the core function to do the same thing --- repository/equella/lib.php | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/repository/equella/lib.php b/repository/equella/lib.php index f4018032ff3..b65f7ad674f 100644 --- a/repository/equella/lib.php +++ b/repository/equella/lib.php @@ -278,13 +278,7 @@ class repository_equella extends repository { } private static function get_all_editing_roles() { - global $DB; - $sql = "SELECT r.* FROM {role_capabilities} rc - INNER JOIN {role} r - ON rc.roleid = r.id - WHERE capability = :capability AND permission = 1 - ORDER BY r.shortname"; - return $DB->get_records_sql($sql, array('capability' => 'moodle/course:manageactivities')); + return get_roles_with_capability('moodle/course:manageactivities', CAP_ALLOW); } /**