Moodle
  1. Moodle
  2. MDL-34129

course summary doesn't show on enrolment page in 2.2 and 2.3

    Details

    • Testing Instructions:
      Hide

      1/ set up password protected course guest access
      2/ try to access course
      3/ verify the course information box is displayed

      (make sure to test independently in 2.3 and master because the code is different thanks to new localised role names)

      Show
      1/ set up password protected course guest access 2/ try to access course 3/ verify the course information box is displayed (make sure to test independently in 2.3 and master because the code is different thanks to new localised role names)
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w30_MDL-34129_m24_enrolinfo
    • Rank:
      42446

      Description

      Course Summary doesn't show on the enrol page. So unless someone approaches the course from a very specific route, they don't read the Course Summary (which can contain essential information about the course and how to enrol). Even if they approach it from the Course list, the summary doesn't show unless the "i" is clicked. You can see the issue in this image: http://moodle.org/pluginfile.php/143/mod_forum/attachment/898069/course-summary-missing.jpg and see it discussed in this thread: http://moodle.org/mod/forum/discuss.php?d=205546

        Activity

        Hide
        Petr Škoda added a comment -

        Hi, yes it should be there, sorry for the trouble.

        Show
        Petr Škoda added a comment - Hi, yes it should be there, sorry for the trouble.
        Hide
        Petr Škoda added a comment -

        to integrators: I would be probably better to create a centralised renderer, but I guess we can do it later together with other course and category cleanup in master only.

        Show
        Petr Škoda added a comment - to integrators: I would be probably better to create a centralised renderer, but I guess we can do it later together with other course and category cleanup in master only.
        Hide
        Petr Škoda added a comment -

        I have migrated used the course render in master

        Show
        Petr Škoda added a comment - I have migrated used the course render in master
        Hide
        Dan Poltawski added a comment -

        Hi Petr,

        Can this be cherry-picked to 2.2?

        Show
        Dan Poltawski added a comment - Hi Petr, Can this be cherry-picked to 2.2?
        Hide
        Petr Škoda added a comment -

        yep, the 2.3 code should be safe for backporting

        Show
        Petr Škoda added a comment - yep, the 2.3 code should be safe for backporting
        Hide
        Dan Poltawski added a comment -

        Thanks Petr i've integrated this now (22, 23 and master)

        Show
        Dan Poltawski added a comment - Thanks Petr i've integrated this now (22, 23 and master)
        Hide
        Ankit Agarwal added a comment -

        should this be showing the Course name as well? not just the description?

        Show
        Ankit Agarwal added a comment - should this be showing the Course name as well? not just the description?
        Hide
        Ankit Agarwal added a comment -

        Had a talk with Dan, and since we have course name in navigation, it is no more required in the enrollment page.
        Passing.
        Thanks

        Show
        Ankit Agarwal added a comment - Had a talk with Dan, and since we have course name in navigation, it is no more required in the enrollment page. Passing. Thanks
        Hide
        Aparup Banerjee added a comment -

        yay, it works!

        This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

        Thank you all for taking the time to get us here.

        cheers!

        Show
        Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!
        Hide
        Sue M. added a comment -

        Thank you all for working on this, one last question - is there a way to install a 'fix' for this without having to do a complete reinstall of the latest updated version?

        Show
        Sue M. added a comment - Thank you all for working on this, one last question - is there a way to install a 'fix' for this without having to do a complete reinstall of the latest updated version?
        Hide
        Brian Merritt added a comment -

        Is this now in 2.41 and also will this address "my moodle" and the lack of course summary there?

        Show
        Brian Merritt added a comment - Is this now in 2.41 and also will this address "my moodle" and the lack of course summary there?

          People

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

            Dates

            • Created:
              Updated:
              Resolved: