Uploaded image for project: '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

      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

        Gliffy Diagrams

          Activity

          Hide
          skodak Petr Skoda added a comment -

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

          Show
          skodak Petr Skoda added a comment - Hi, yes it should be there, sorry for the trouble.
          Hide
          skodak Petr Skoda 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
          skodak Petr Skoda 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
          skodak Petr Skoda added a comment -

          I have migrated used the course render in master

          Show
          skodak Petr Skoda added a comment - I have migrated used the course render in master
          Hide
          poltawski Dan Poltawski added a comment -

          Hi Petr,

          Can this be cherry-picked to 2.2?

          Show
          poltawski Dan Poltawski added a comment - Hi Petr, Can this be cherry-picked to 2.2?
          Hide
          skodak Petr Skoda added a comment -

          yep, the 2.3 code should be safe for backporting

          Show
          skodak Petr Skoda added a comment - yep, the 2.3 code should be safe for backporting
          Hide
          poltawski Dan Poltawski added a comment -

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

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

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

          Show
          ankit_frenz Ankit Agarwal added a comment - should this be showing the Course name as well? not just the description?
          Hide
          ankit_frenz 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_frenz 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
          nebgor 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
          nebgor 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
          ggsinc 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
          ggsinc 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
          brianlmerritt 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
          brianlmerritt 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:
                Fix Release Date:
                10/Sep/12