Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29019

Should use 'enrol/self:config', not 'enrol/manual:config'

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Enrolments
    • Labels:

      Description

      In enrol/self/lib.php, function get_newinstance_link() is using 'enrol/manual:config' to judge capability. I think it should be 'enrol/self:config'.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            Sorry and big thanks!

            Show
            skodak Petr Skoda added a comment - Sorry and big thanks!
            Hide
            skodak Petr Skoda added a comment -

            to integrators: please cherry pick to all 2.x branches

            Show
            skodak Petr Skoda added a comment - to integrators: please cherry pick to all 2.x branches
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks (by cherry picking the target commit, the branch was really out of sync).

            Backported to 21 and 20 stable.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks (by cherry picking the target commit, the branch was really out of sync). Backported to 21 and 20 stable.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Passing. No test needed.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Passing. No test needed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Oct/11