Moodle

If a question's average grade is 0, the quiz report displays an error.

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.1.4, 2.2.1, 2.3
  • Fix Version/s: 2.1.5, 2.2.2
  • Component/s: Quiz
  • Labels:
    None
  • Testing Instructions:
    Hide

    1. Create a quiz with an essay question.

    2. Make an attempt as a student.

    3. As a teacher/admin, go to the quiz overview report, and make sure the average question grade is shown as '-'. and that there are no PHP errors, even with Developer debug.

    Show
    1. Create a quiz with an essay question. 2. Make an attempt as a student. 3. As a teacher/admin, go to the quiz overview report, and make sure the average question grade is shown as '-'. and that there are no PHP errors, even with Developer debug.
  • Affected Branches:
    MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
  • Fixed Branches:
    MOODLE_21_STABLE, MOODLE_22_STABLE
  • Pull from Repository:
  • Pull Master Branch:
  • Pull Master Diff URL:

Description

This is a minor regression caused by MDL-30704

Issue Links

is a regression caused by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-30704 Quiz grades report shows inconsistent averages

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.
is duplicated by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-31187 Warning when viewing quiz resuts.

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Aparup Banerjee added a comment -

looks good to me, integrated and up for testing.

Show
Aparup Banerjee added a comment - looks good to me, integrated and up for testing.
Hide
Permalink
Gerard Caulfield added a comment -

/mod/quiz/report.php?id=X&mode=overview

I observed the pre patch errors and the fixes on 2.1 2.2 and master.

Test passed

Show
Gerard Caulfield added a comment - /mod/quiz/report.php?id=X&mode=overview I observed the pre patch errors and the fixes on 2.1 2.2 and master. Test passed
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

This is now available in the git and cvs repositories.

Consider the responsibility of your fingerprints engraved there for future generations!

Thanks for the work, closing, ciao

Show
Eloy Lafuente (stronk7) added a comment - This is now available in the git and cvs repositories. Consider the responsibility of your fingerprints engraved there for future generations! Thanks for the work, closing, ciao

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: