Moodle

review roles usage in portfolio

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0
  • Component/s: Portfolio API
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

might be just as simple as require_cap in portfolio/add.php

Activity

Hide
Penny Leach added a comment -

after discussion with MD the plan is to:

  • the calling code can check has_capability before calling portfolio_add_button
  • add a new method in the caller class to check permissions
  • name the capabilities consistently, eg mod/forum:export-post, mod/forum:export-attachment
  • figure out a way to disable (or enable) plugin instances per role.
Show
Penny Leach added a comment - after discussion with MD the plan is to:
  • the calling code can check has_capability before calling portfolio_add_button
  • add a new method in the caller class to check permissions
  • name the capabilities consistently, eg mod/forum:export-post, mod/forum:export-attachment
  • figure out a way to disable (or enable) plugin instances per role.
Hide
Penny Leach added a comment -

MD said:
10:53 <moodler_> hmm plugins, we did something similar for grade import/expot
(relating to fourth point above)

Show
Penny Leach added a comment - MD said: 10:53 <moodler_> hmm plugins, we did something similar for grade import/expot (relating to fourth point above)
Hide
Penny Leach added a comment -

closing this issue as resolved.

the outstanding thing about trying to figure out a way to enable or disable instances per role is trickier and not necessarily needed right now so i will open a separate bug for it.

Show
Penny Leach added a comment - closing this issue as resolved. the outstanding thing about trying to figure out a way to enable or disable instances per role is trickier and not necessarily needed right now so i will open a separate bug for it.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: