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

Minor question editing issues found by Jamie Pratt

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 2.1
    • 2.1
    • Questions
    • None
    • MOODLE_21_STABLE
    • MOODLE_21_STABLE
    • Hide

      This is really hard to test, because even with the ORDER BY id omitted, databases still normally, but not always, load the question_hint rows in order.

      I just suggest the following test:
      1. Create a new quiz question (e.g. shortanswer) with two hints, hint 1 and hint 2.
      2. Edit the question again, and ensure the hints are still in the right order after saving.

      The extraanswerfields bit does not affect any core question types. It only affects some third-party question types like the ones Jamie is working on. I suggest you assume that we have tested that sufficiently. The tests above that editing a standard qtype is not broken is sufficient.

      Show
      This is really hard to test, because even with the ORDER BY id omitted, databases still normally, but not always, load the question_hint rows in order. I just suggest the following test: 1. Create a new quiz question (e.g. shortanswer) with two hints, hint 1 and hint 2. 2. Edit the question again, and ensure the hints are still in the right order after saving. The extraanswerfields bit does not affect any core question types. It only affects some third-party question types like the ones Jamie is working on. I suggest you assume that we have tested that sufficiently. The tests above that editing a standard qtype is not broken is sufficient.

      The issues are:

      1. The question_type::save_hints method is missing ORDER BY id.

      2. The $extraanswerfields bit of question_type::get_question_options had some SQL errors.

            timhunt Tim Hunt
            timhunt Tim Hunt
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.