Moodle
  1. Moodle
  2. MDL-23253

"Enrol me in this course" link should be hidden for non-enrolable courses

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Enrolments
    • Labels:
      None
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      26909

      Description

      If a course is set to not allow self enrollments, but allow guest access, a guest user still sees the "Enrol me in this course" link in the Setting block, under Course Administration. If they click the link, they are greeted with a very disappointing "This course is not enrollable at this time" message. Instead that useless link should not show up. Screenshots are attached.

        Issue Links

          Activity

          Hide
          Jay Knight added a comment -

          So, I guess you can't just hide it based on that option, because there may be other interactive enrollment methods/plugins? But it seems like you could check that as well?

          Show
          Jay Knight added a comment - So, I guess you can't just hide it based on that option, because there may be other interactive enrollment methods/plugins? But it seems like you could check that as well?
          Hide
          Petr Škoda added a comment -

          should be fixed now, thanks for the report

          Show
          Petr Škoda added a comment - should be fixed now, thanks for the report
          Hide
          Jay Knight added a comment -

          I've attached a wannabe patch to MDL-23548 that uses this code to hide some courses altogether if you can't do anything with them. It probably needs some more thought but it portrays the idea. Thoughts?

          Show
          Jay Knight added a comment - I've attached a wannabe patch to MDL-23548 that uses this code to hide some courses altogether if you can't do anything with them. It probably needs some more thought but it portrays the idea. Thoughts?

            People

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

              Dates

              • Created:
                Updated:
                Resolved: