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

missing ORDER BY clause for the SQL for fetching answer records when answers are joined with the extra answer fields table.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.1
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      This only affects third-party question types, and the OU has been encountering this issue, and this patch fixes it. Therefore, no testing required in Moodle core.

      Show
      This only affects third-party question types, and the OU has been encountering this issue, and this patch fixes it. Therefore, no testing required in Moodle core.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Gliffy Diagrams

        Attachments

          Activity

          Hide
          timhunt Tim Hunt added a comment -

          I just did a quick amend of Jamie's commit (https://github.com/jamiepratt/moodle/compare/moodle...OU-921) to fix the commit comment.

          Jamie, in general, I prefer it if people submit quiz and question issues to me for Peer review, before submitting them for integration. I know I am meant to be on holiday for the next 10 days, and you might not want to wait. I don't know what to suggest in that case ...

          Anyway, thanks for fixing this.

          Show
          timhunt Tim Hunt added a comment - I just did a quick amend of Jamie's commit ( https://github.com/jamiepratt/moodle/compare/moodle...OU-921 ) to fix the commit comment. Jamie, in general, I prefer it if people submit quiz and question issues to me for Peer review, before submitting them for integration. I know I am meant to be on holiday for the next 10 days, and you might not want to wait. I don't know what to suggest in that case ... Anyway, thanks for fixing this.
          Hide
          timhunt Tim Hunt added a comment -

          I should add:

          Dear integrators, please apply this fix to 21_STABLE and master.

          Show
          timhunt Tim Hunt added a comment - I should add: Dear integrators, please apply this fix to 21_STABLE and master.
          Hide
          jamiesensei Jamie Pratt added a comment -

          Sorry Tim. I need to find the document explaining the process for submitting patches for consideration, I had expected that you would review this and didn't mean to bypass you.

          Show
          jamiesensei Jamie Pratt added a comment - Sorry Tim. I need to find the document explaining the process for submitting patches for consideration, I had expected that you would review this and didn't mean to bypass you.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks guys, this has been integrated now.
          It has been merged to master and cherry-picked to 21_STABLE.
          Please in the future create a branch for each version you want to fix.

          Cheers
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks guys, this has been integrated now. It has been merged to master and cherry-picked to 21_STABLE. Please in the future create a branch for each version you want to fix. Cheers Sam
          Hide
          rajeshtaneja Rajesh Taneja added a comment -

          Passing this test, as suggested in testing instruction "No test required on Moodle Core".
          Thanks for providing the patch Jamie

          Show
          rajeshtaneja Rajesh Taneja added a comment - Passing this test, as suggested in testing instruction "No test required on Moodle Core". Thanks for providing the patch Jamie
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Congratulations - this fix has just been released in the weeklies.

          Show
          samhemelryk Sam Hemelryk added a comment - Congratulations - this fix has just been released in the weeklies.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                1/Aug/11