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

        Penny Leach created issue -
        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.
        Penny Leach made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Martin Dougiamas made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Martin Dougiamas made changes -
        Workflow jira [ 27582 ] MDL Workflow [ 60366 ]
        Martin Dougiamas made changes -
        Workflow MDL Workflow [ 60366 ] MDL Full Workflow [ 89542 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: