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

Course summary exporter lists properties as 'other' when they are not

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: None
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a bunch of courses with full names that differ from their short names.
      2. Enrol a student in all these courses.
      3. Log in as the student.
      4. Go to your dashboard.
      5. Click on 'Timeline' with 'Sort by courses' selected.
      6. Confirm that the courses' details are shown.
      7. Click on 'Courses'.
      8. Confirm that the courses' details are shown.
      Show
      Create a bunch of courses with full names that differ from their short names. Enrol a student in all these courses. Log in as the student. Go to your dashboard. Click on 'Timeline' with 'Sort by courses' selected. Confirm that the courses' details are shown. Click on 'Courses'. Confirm that the courses' details are shown.
    • Affected Branches:
      MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-58222_master

      Description

      In the course_summary_exporter.php the function define_other_properties() lists 'summary', 'startdate' and 'enddate' but these are not preset in 'get_other_values' - meaning no conversion is necessary. These values are present in the course table (which is passed to the constructor) so they actually belong in define_properties.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              markn Mark Nelson
              Reporter:
              markn Mark Nelson
              Peer reviewer:
              cameron1729
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: