Moodle
  1. Moodle
  2. MDL-29019

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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:
    • Rank:
      18715

      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'.

        Activity

        Hide
        Petr Škoda added a comment -

        Sorry and big thanks!

        Show
        Petr Škoda added a comment - Sorry and big thanks!
        Hide
        Petr Škoda added a comment -

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

        Show
        Petr Škoda added a comment - to integrators: please cherry pick to all 2.x branches
        Hide
        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
        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
        Eloy Lafuente (stronk7) added a comment -

        Passing. No test needed.

        Show
        Eloy Lafuente (stronk7) added a comment - Passing. No test needed.
        Hide
        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
        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: