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

Feedback images when reviewing a quiz attempt not showing

    XMLWordPrintable

Details

    • MOODLE_32_STABLE
    • MOODLE_31_STABLE, MOODLE_32_STABLE
    • Hide

      For each of the following question types:

      • drag and drop onto image
      • drag and drop markers
      • drag and drop into text
      • select missing words
      • matching
      • mutlitple choice
      1. Ensure you have developer debug on.
      2. Create a question of this type, with an image in the 'Feedback for any correct response'
      3. Add the question to a quiz. (Ensure the review options say to show feedback.)
      4. Attempt the quiz as a student, and get the question right.
      5. Verify that the feedback is displayed, and that the image appears.
      Show
      For each of the following question types: drag and drop onto image drag and drop markers drag and drop into text select missing words matching mutlitple choice Ensure you have developer debug on. Create a question of this type, with an image in the 'Feedback for any correct response' Add the question to a quiz. (Ensure the review options say to show feedback.) Attempt the quiz as a student, and get the question right. Verify that the feedback is displayed, and that the image appears.

    Description

      Commit cf0b943 for MDL-53744 modified the check_combined_feedback_file_access function of question/type/questionbase.php to do a check against $args, but $args is not passed through as a parameter.

      Consequently no feedback graphics (for general correct/incorrect/partiallycorrect feedback) are rendered when reviewing a quiz in gapselect, multichoice and match question types.

      Attachments

        Issue Links

          Activity

            People

              timhunt Tim Hunt
              rbkcapdm Richard Kirby
              Eric Merrill Eric Merrill
              Dan Poltawski Dan Poltawski
              Ankit Agarwal Ankit Agarwal
              Tim Hunt, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Kevin Percy, Laurent David, Mathew May, Mihail Geshoski, Raquel Ortega, Sabina Abellan, Sander Bangma, Sara Arjona (@sarjona), Shamim Rezaie
              Votes:
              6 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                13/Mar/17