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

Ajax service should clean the return values of the external functions

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Navigate through the template library, and confirm that it works as it used to (view templates, filter templates, etc...)
      2. Edit the file admin/tool/templatelibrary/classes/external.php
      3. Change the return structure of list_templates_returns, and set it to PARAM_INT
      4. Confirm that when you try to list templates you get "Invalid response value detected"
      Show
      Navigate through the template library, and confirm that it works as it used to (view templates, filter templates, etc...) Edit the file admin/tool/templatelibrary/classes/external.php Change the return structure of list_templates_returns , and set it to PARAM_INT Confirm that when you try to list templates you get "Invalid response value detected"
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-51700-master
    • Pull Master Diff URL:

      Description

      I noticed that because my Javascript was strictly checking for an int but the return value was a string. I could fix my external function but it seems appropriate to force Ajax to clean the value, just the same way REST does it.

        Attachments

          Issue Links

          blocks

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. MDL-49458 Competency Based Education

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-52418 Invalid responses in learning plan templates and course competencies pages

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  23/May/16