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

Regression in core_course_search_courses

    XMLWordPrintable

Details

    • MOODLE_31_STABLE
    • MOODLE_31_STABLE
    • MDL-54820-master
    • Hide
      1. Download the latest version of the Moodle Mobile app for Android or iOs
      2. Connect to your local site with this fix applied
      3. Go to the My Courses option
      4. Go to the magnifier glass icon (top right)
      5. Search for an existing course by name, you should find it
      6. In Moodle, enable the Course meta link enrolment plugin
      7. Go to a course -> enrolment methods
      8. Add the Course meta link as an enrolment method
      9. Check that the Course search (ajax) works (Link course field)
      Show
      Download the latest version of the Moodle Mobile app for Android or iOs Connect to your local site with this fix applied Go to the My Courses option Go to the magnifier glass icon (top right) Search for an existing course by name, you should find it In Moodle, enable the Course meta link enrolment plugin Go to a course -> enrolment methods Add the Course meta link as an enrolment method Check that the Course search (ajax) works (Link course field)

    Description

      Caused by changes in MDL-51324

      Problem is here:
      https://github.com/moodle/moodle/blame/master/course/externallib.php#L2142

      Second parameter for external_multiple_structure is a description, VALUE_OPTIONAL is used instead when it should be the 3rd parameter.

      Attachments

        Issue Links

          Activity

            People

              jleyva Juan Leyva
              jleyva Juan Leyva
              Damyon Wiese Damyon Wiese
              Dan Poltawski Dan Poltawski
              John Okely John Okely
              Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona), Juan Leyva, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Jul/16