Moodle

gradebook reports and plugins do not validate

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: 1.9.5
  • Component/s: Gradebook
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

  • alt attribute
  • div closing

Activity

Hide
Petr Škoda (skodak) added a comment -

both issues fixed

Show
Petr Škoda (skodak) added a comment - both issues fixed
Hide
Helen Foster added a comment -

Petr, thanks for your fixes, which I tested by enabling xmlstrictheaders then viewing a grader report. I obtained the following error message (using Moodle 1.9.4+ build 20090505):

XML Parsing Error: mismatched tag. Expected: </td>.
Location: http://localhost/19/moodle/grade/report/grader/index.php?id=3
Line Number 205, Column 10003:

Show
Helen Foster added a comment - Petr, thanks for your fixes, which I tested by enabling xmlstrictheaders then viewing a grader report. I obtained the following error message (using Moodle 1.9.4+ build 20090505): XML Parsing Error: mismatched tag. Expected: </td>. Location: http://localhost/19/moodle/grade/report/grader/index.php?id=3 Line Number 205, Column 10003:
Hide
Petr Škoda (skodak) added a comment -

Fixed some more strict issues - works fine for me now, please retest and reopen if needed.
I did not find any </td> problem though, please add gradebook settings or course backup too, thanks a lot.

Show
Petr Škoda (skodak) added a comment - Fixed some more strict issues - works fine for me now, please retest and reopen if needed. I did not find any </td> problem though, please add gradebook settings or course backup too, thanks a lot.
Hide
Petr Škoda (skodak) added a comment -

grrrrrr

Show
Petr Škoda (skodak) added a comment - grrrrrr
Hide
Helen Foster added a comment -

Petr, thanks for looking at this issue again.

Unfortunately I'm still getting the </td> error. It's also shown on http://test.moodle.org/1.9/grade/report/grader/index.php?id=16 (and for the grader report for other courses on the test site).

Show
Helen Foster added a comment - Petr, thanks for looking at this issue again. Unfortunately I'm still getting the </td> error. It's also shown on http://test.moodle.org/1.9/grade/report/grader/index.php?id=16 (and for the grader report for other courses on the test site).
Hide
Petr Škoda (skodak) added a comment -

found and hopefully fixed, thanks a lot

Show
Petr Škoda (skodak) added a comment - found and hopefully fixed, thanks a lot
Hide
Petr Škoda (skodak) added a comment -

going to update test server in a few minutes

Show
Petr Škoda (skodak) added a comment - going to update test server in a few minutes
Hide
Petr Škoda (skodak) added a comment -

reopening, found other issues elsewhere - grrrrrrrrr

Show
Petr Škoda (skodak) added a comment - reopening, found other issues elsewhere - grrrrrrrrr
Hide
Petr Škoda (skodak) added a comment -

please file separate issues if more problems found
thanks a lot

Show
Petr Škoda (skodak) added a comment - please file separate issues if more problems found thanks a lot
Hide
Helen Foster added a comment -

Petr, thanks for fixing everything. All gradebook pages now validate

Show
Helen Foster added a comment - Petr, thanks for fixing everything. All gradebook pages now validate

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: