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

Not using the proper editor element for manually grading questions

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0.1
    • 2.1
    • Questions
    • None
    • MOODLE_20_STABLE
    • MOODLE_21_STABLE
    • Moderate
    • Hide

      1. As teacher, go to a quiz with student attempts
      2. Review an attempt, and manually grade a response using the Make comment & override grade.
      3. Go to Results -> Manual grading in the navigtion, and manually grade using that.
      In both cases, verify the grade and comment are saved, and can be re-edited.

      Ideally verify this with the Use HTML editor option in your profile both on and off.

      Show
      1. As teacher, go to a quiz with student attempts 2. Review an attempt, and manually grade a response using the Make comment & override grade. 3. Go to Results -> Manual grading in the navigtion, and manually grade using that. In both cases, verify the grade and comment are saved, and can be re-edited. Ideally verify this with the Use HTML editor option in your profile both on and off.

    Description

      While playing with quizzes/questions to detect all the file_areas I have detected that the question_sessions->manualcomment is lacking support for files (and that's ok, as commented by Tim in HQ chat) but, at the same time, it seems that the question_sessions->manualcommentformat field isn't being handled at all.

      I get the html editor (without showing any "format"), then save the comment and, the manualcommentformat continues being 0 (FORMAT_MOODLE) should be 1 (FORMAT_HTML).

      So I guess something is missing there (mod/quiz/comment.php)

      Ciao

      Attachments

        Issue Links

          Activity

            People

              timhunt Tim Hunt
              stronk7 Eloy Lafuente (stronk7)
              Sam Hemelryk Sam Hemelryk
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Safat Shahin, 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:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                1/Jul/11