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

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

    Details

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

      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.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              jk3us 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
              jk3us 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
              skodak Petr Skoda added a comment -

              should be fixed now, thanks for the report

              Show
              skodak Petr Skoda added a comment - should be fixed now, thanks for the report
              Hide
              jk3us 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
              jk3us 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:
                    Fix Release Date:
                    24/Nov/10