Moodle

Tooltip text for grader report cells includes HTML

Details

  • Database:
    Any
  • Difficulty:
    Easy
  • Affected Branches:
    MOODLE_19_STABLE, MOODLE_20_STABLE

Description

Here is an example of the HTML for a cell in the grader report. It's actually including HTML in the tool tip that appears when you hover over the cell:

<td title='<div class="fullname">B***** B******</div><div class="itemname">Informative Speech</div><div class="feedback">I cannot get your CD to work. I<br/ >have tried in 2 DVD players, my<br/ >computer, and my sony camera. Did<br/ >you finalize it.</div>' class="grade cell c1"><span class="gradevalue">0.00</span></td>

Attachments

  1. File
    gradetooltip.diff
    04/Nov/09 10:14 PM
    0.9 kB
    Eric Merrill
  2. Text File
    lib.php.patch
    22/Jul/09 5:08 AM
    2 kB
    Justin Filip
  1. GradeBook2.png.jpg
    37 kB
    23/Jul/09 9:44 PM

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Justin Filip added a comment -

This is a patch against the latest version of /grade/report/grader/lib.php from the MOODLE_19_STABLE branch and it removes the HTML tags from the tool tip text used in the report cell HTML. There are also some whitespace fixes as my editor is configured to strip whitespace on save.

Show
Justin Filip added a comment - This is a patch against the latest version of /grade/report/grader/lib.php from the MOODLE_19_STABLE branch and it removes the HTML tags from the tool tip text used in the report cell HTML. There are also some whitespace fixes as my editor is configured to strip whitespace on save.
Hide
Permalink
Justin Filip added a comment -

Added a screenshot showing the problem this patch fixes.

Show
Justin Filip added a comment - Added a screenshot showing the problem this patch fixes.
Hide
Permalink
Eric Merrill added a comment -

I've attached a cleaner patch - the previous one had some changes that were not pertinent to the bug.

I've also added a - between the user name and the grade item name.

I need to research the code some more to make sure that this doesn't affect anything, but if not, and if there are no objections, I'll commit this in the next few days.

-eric

Show
Eric Merrill added a comment - I've attached a cleaner patch - the previous one had some changes that were not pertinent to the bug. I've also added a - between the user name and the grade item name. I need to research the code some more to make sure that this doesn't affect anything, but if not, and if there are no objections, I'll commit this in the next few days. -eric
Hide
Permalink
Helen Foster added a comment -

Just noting that this issue has also been reported in the gradebook forum: http://moodle.org/mod/forum/discuss.php?d=141908

Show
Helen Foster added a comment - Just noting that this issue has also been reported in the gradebook forum: http://moodle.org/mod/forum/discuss.php?d=141908
Hide
Permalink
Cristian Cordeiro added a comment -

Although this is reported as pertinent to versions 1.9.5 & 1.9.6, it also happens on version 2.0, so I wonder if Eric's fix was not yet committed or if it was committed but it is just not merged into 2.0 yet. I've made a patch for version 2.0 and I would be happy to contribute, just wanted to understand if it is something that was already taken care of or if it is something that I could be contributing on.

Cristian Cordeiro

Show
Cristian Cordeiro added a comment - Although this is reported as pertinent to versions 1.9.5 & 1.9.6, it also happens on version 2.0, so I wonder if Eric's fix was not yet committed or if it was committed but it is just not merged into 2.0 yet. I've made a patch for version 2.0 and I would be happy to contribute, just wanted to understand if it is something that was already taken care of or if it is something that I could be contributing on. Cristian Cordeiro
Hide
Permalink
Helen Foster added a comment -

Thanks for all the patches and apologies for this issue having been neglected for so long. Can anyone confirm that the problem also affects the latest 2.0.2?

Show
Helen Foster added a comment - Thanks for all the patches and apologies for this issue having been neglected for so long. Can anyone confirm that the problem also affects the latest 2.0.2?
Hide
Permalink
Andrew Davis added a comment - - edited

Hi. I'm not able to reproduce this against current 1.9.12, 2.0.3 or 2.1.0. I've also turned up this previously fixed issue, MDL-20179, so I believe this has already been fixed.

Thankyou for the bug report

Show
Andrew Davis added a comment - - edited Hi. I'm not able to reproduce this against current 1.9.12, 2.0.3 or 2.1.0. I've also turned up this previously fixed issue, MDL-20179, so I believe this has already been fixed. Thankyou for the bug report

People

Vote (5)
Watch (6)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.