Moodle

groupmode + default role in get_users_by_capability() trouble

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Affects Version/s: 1.9.5
  • Fix Version/s: None
  • Component/s: Roles / Access
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE

Description

The problem is that group mode does work when the capability used in get_users_by_capability() is allowed in frontpage or default role.

see http://moodle.org/mod/forum/discuss.php?d=135649

Eloy proposed a solution to use u.id instead which makes sense imo, needs a lot more testing, will not be in 1.9.6

Activity

Hide
Tim Hunt added a comment -

By the way, get_users_by_capability has reasonably good unit tests in 2.0. Might be worth doing the development there and backporting.

Show
Tim Hunt added a comment - By the way, get_users_by_capability has reasonably good unit tests in 2.0. Might be worth doing the development there and backporting.
Hide
Michael de Raadt added a comment -

Thanks for reporting this issue.

We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

Michael d;

lqjjLKA0p6

Show
Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6
Hide
Michael de Raadt added a comment -

I'm closing this issue as it appears to have become inactive and is probably not relevant to a current supported version. If you are encountering this problem or one similar, please launch a new issue.

Show
Michael de Raadt added a comment - I'm closing this issue as it appears to have become inactive and is probably not relevant to a current supported version. If you are encountering this problem or one similar, please launch a new issue.

People

Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: