Details

    • Type: Sub-task Sub-task
    • Status: 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
    • Rank:
      32932

      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

          • Assignee:
            Penny Leach
            Reporter:
            Penny Leach
            Participants:
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: