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

Incorrect return types definition in get_chats_by_courses get_forums_by_courses and get_databases_by_courses

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.9.2
    • Fix Version/s: 2.8.9, 2.9.3
    • Component/s: Web Services
    • Labels:
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-51565-master
    • Testing Instructions:
      Hide

      This test is mostly to check that nothing has broken:

      1. Enable Mobile services in Moodle via Plugins -> Web Services -> Mobile
      2. Access to a course (that have a chat, forum and database activity) using the latest version of the Mobile app
      3. Confirm that once inside the activities you see the name correct
      Show
      This test is mostly to check that nothing has broken: Enable Mobile services in Moodle via Plugins -> Web Services -> Mobile Access to a course (that have a chat, forum and database activity) using the latest version of the Mobile app Confirm that once inside the activities you see the name correct

      Description

      Course is defined as PARAM_TEXT

      'course' => new external_value(PARAM_TEXT, 'Course id'),

      Timemodified is incorrect in some components too.

      The name is not formatted using external_format_string, and the returned param type is also incorrect.

        Attachments

          Activity

            People

            Assignee:
            jleyva Juan Leyva
            Reporter:
            jleyva Juan Leyva
            Peer reviewer:
            Jun Pataleta Jun Pataleta
            Integrator:
            Andrew Lyons Andrew Lyons
            Tester:
            cameron1729 cameron1729
            Participants:
            Component watchers:
            Juan Leyva, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              9/Nov/15