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
    • 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

        Gliffy Diagrams

          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: