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

Gradebook: Previous link to view individual student grades from "Grader Report" gone in 1.9

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 1.9.7
    • Component/s: Gradebook
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      From the grader report there seems to be a change in behavior.

      In Moodle 1.8, when viewing the grades (/grade/index.php), clicking on the link of the user's name linked to:
      /grade/index.php?id=16&action=view_student_grades&user=826 (a grade report for the individual student)

      In Moodle 1.9, when viewing the grades via the grade report (/grade/report/grader/index.php?id=1819), clicking on the link of the user's name links to:
      /user/view.php?id=520&course=1819 (the user's profile)

      Previously links for both the user's name and the picture were available. The link for the user's pic would link to the user's profile. The link for the user's name would link to the grade report for the individual student. This seems to be a change in behavior.

      What I would expect to see is that the user's name link take us to the grades for the individual student (perhaps that user's gradebook "Users report"). I'm not sure of the best way to handle this with the new gradebook UI; however, I want to try to preserve the behavior. I found it very convenient to pull up the gradebook and be able to link to the individual grades of a particular student especially during parent teacher conferences as it prevented me from showing the parents information about other students.

      Peace - Anthony

        Gliffy Diagrams

          Attachments

          1. chrome.png
            chrome.png
            119 kB
          2. firefox35.png
            firefox35.png
            88 kB
          3. firefox35big.png
            firefox35big.png
            138 kB
          4. grader_index.html
            79 kB
          5. ie6.png
            ie6.png
            267 kB
          6. ie7.png
            ie7.png
            266 kB
          7. MDL-12972.patch
            7 kB
          8. mdl12972.png
            mdl12972.png
            124 kB
          9. MDL-12972.png
            MDL-12972.png
            31 kB
          10. standardtheme.png
            standardtheme.png
            144 kB

            Activity

            Hide
            aborrow Anthony Borrow added a comment -

            Eloy suggested assigning this to you while Yu is out. IMHO, it is not urgent but something I happened across while working on a possible set of patches for MDL-3942. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Eloy suggested assigning this to you while Yu is out. IMHO, it is not urgent but something I happened across while working on a possible set of patches for MDL-3942 . Peace - Anthony
            Hide
            nicolasconnault Nicolas Connault added a comment -

            This seems sensible, I suggest we address this in 1.9.6

            Show
            nicolasconnault Nicolas Connault added a comment - This seems sensible, I suggest we address this in 1.9.6
            Hide
            quineto Camilo Rivera added a comment -

            Any Ideas where in the code to address this issue?? , I would like to get started on this.

            Show
            quineto Camilo Rivera added a comment - Any Ideas where in the code to address this issue?? , I would like to get started on this.
            Hide
            aborrow Anthony Borrow added a comment -

            I confess to having mixed feelings about this. On the one hand, I like the idea of being consistent with previous versions; however, from a usability standpoint when in other places of Moodle we see the user name as a link it takes us to the user profile. How can we be consistent. Might it be better to have another way of indicating that we just want to see a particular row? I'm not sure from a UI perspective what might be best but figured I would throw the idea out there. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - I confess to having mixed feelings about this. On the one hand, I like the idea of being consistent with previous versions; however, from a usability standpoint when in other places of Moodle we see the user name as a link it takes us to the user profile. How can we be consistent. Might it be better to have another way of indicating that we just want to see a particular row? I'm not sure from a UI perspective what might be best but figured I would throw the idea out there. Peace - Anthony
            Hide
            jrily Sharon Goodson added a comment -

            This is far out of my area, but could a direct link to course/user grade mode be specified only in the grader/index page?

            Show
            jrily Sharon Goodson added a comment - This is far out of my area, but could a direct link to course/user grade mode be specified only in the grader/index page?
            Hide
            nicolasconnault Nicolas Connault added a comment -

            Attached is a patch that adds a "User grades" icon next to the user name. The screenshot shows the result.

            Show
            nicolasconnault Nicolas Connault added a comment - Attached is a patch that adds a "User grades" icon next to the user name. The screenshot shows the result.
            Hide
            aborrow Anthony Borrow added a comment -

            Nicolas - Here is what I noticed when I applied. Not sure the cause but figured I would report it:

            Incorrect scale id: 2

            • line 798 of lib/grade/grade_item.php: call to debugging()
            • line 705 of lib/gradelib.php: call to grade_item->load_scale()
            • line 667 of lib/gradelib.php: call to grade_format_gradevalue_real()
            • line 1948 of lib/grade/grade_item.php: call to grade_format_gradevalue()
            • line 329 of grade/report/user/lib.php: call to grade_item->get_formatted_range()
            • line 354 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive()
            • line 183 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive()
            • line 127 of grade/report/user/index.php: call to grade_report_user->fill_table()

            Incorrect scale id: 2

            • line 798 of lib/grade/grade_item.php: call to debugging()
            • line 705 of lib/gradelib.php: call to grade_item->load_scale()
            • line 667 of lib/gradelib.php: call to grade_format_gradevalue_real()
            • line 1949 of lib/grade/grade_item.php: call to grade_format_gradevalue()
            • line 329 of grade/report/user/lib.php: call to grade_item->get_formatted_range()
            • line 354 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive()
            • line 183 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive()
            • line 127 of grade/report/user/index.php: call to grade_report_user->fill_table()

            Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Nicolas - Here is what I noticed when I applied. Not sure the cause but figured I would report it: Incorrect scale id: 2 line 798 of lib/grade/grade_item.php: call to debugging() line 705 of lib/gradelib.php: call to grade_item->load_scale() line 667 of lib/gradelib.php: call to grade_format_gradevalue_real() line 1948 of lib/grade/grade_item.php: call to grade_format_gradevalue() line 329 of grade/report/user/lib.php: call to grade_item->get_formatted_range() line 354 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive() line 183 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive() line 127 of grade/report/user/index.php: call to grade_report_user->fill_table() Incorrect scale id: 2 line 798 of lib/grade/grade_item.php: call to debugging() line 705 of lib/gradelib.php: call to grade_item->load_scale() line 667 of lib/gradelib.php: call to grade_format_gradevalue_real() line 1949 of lib/grade/grade_item.php: call to grade_format_gradevalue() line 329 of grade/report/user/lib.php: call to grade_item->get_formatted_range() line 354 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive() line 183 of grade/report/user/lib.php: call to grade_report_user->fill_table_recursive() line 127 of grade/report/user/index.php: call to grade_report_user->fill_table() Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            Nicolas - I think you can safely ignore the above - I reversed the patch and seeing the same behavior before the patch so it is likely from some testing or other error already in the database. Patch looks OK but I found the image to be a bit crowded but I suspect this could be fixed with a little CSS magic. I'll attach a screen shot. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Nicolas - I think you can safely ignore the above - I reversed the patch and seeing the same behavior before the patch so it is likely from some testing or other error already in the database. Patch looks OK but I found the image to be a bit crowded but I suspect this could be fixed with a little CSS magic. I'll attach a screen shot. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            I think the image for the new link crowds the name a bit too much. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - I think the image for the new link crowds the name a bit too much. Peace - Anthony
            Hide
            nicolasconnault Nicolas Connault added a comment -

            Anthony, I'm not sure how you obtained that screenshot, because the new icon is actually in a separate cell from the student name... Can you post a dump of the output HTML for one of these cells please?

            Thanks

            Show
            nicolasconnault Nicolas Connault added a comment - Anthony, I'm not sure how you obtained that screenshot, because the new icon is actually in a separate cell from the student name... Can you post a dump of the output HTML for one of these cells please? Thanks
            Hide
            nicolasconnault Nicolas Connault added a comment -

            Looks like your student cells are a fixed width, and their text content overflows into the next cell... What theme do you use?

            Show
            nicolasconnault Nicolas Connault added a comment - Looks like your student cells are a fixed width, and their text content overflows into the next cell... What theme do you use?
            Hide
            aborrow Anthony Borrow added a comment -

            Nicolas - I was using the formal_white theme and am attaching the html of the file as requested. It looks like something happens when there are numerous columns that extend beyond a single page - some type of reducing the width of the name column to a minimum setting. When everything is visible on one screen width wise it looks great. I am also attaching a screenshot using the standard theme which shows similar behavior. Let me know if you have any questions. I did not see anything in the css that seemed to be calling attention to itself. I'll poke around the html file and see if I can find something note worthy. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Nicolas - I was using the formal_white theme and am attaching the html of the file as requested. It looks like something happens when there are numerous columns that extend beyond a single page - some type of reducing the width of the name column to a minimum setting. When everything is visible on one screen width wise it looks great. I am also attaching a screenshot using the standard theme which shows similar behavior. Let me know if you have any questions. I did not see anything in the css that seemed to be calling attention to itself. I'll poke around the html file and see if I can find something note worthy. Peace - Anthony
            Hide
            nicolasconnault Nicolas Connault added a comment -

            Anthony this is only happening when you use the "screenreader" version of this report (without the fixed students column). The main reason is explained in MDL-20418 and is separate from this issue.

            Show
            nicolasconnault Nicolas Connault added a comment - Anthony this is only happening when you use the "screenreader" version of this report (without the fixed students column). The main reason is explained in MDL-20418 and is separate from this issue.
            Hide
            aborrow Anthony Borrow added a comment -

            FYI - I am looking at it using FF 3.0.14 running under Ubuntu. It appears to be something peculiar to FF. I tried it under a Virtual WinXP with FF 3.5 and saw the same behavior but Chrome looked fine. IMO it is really minor and cosmetic. I would not see this as a show stopper but it is strange. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - FYI - I am looking at it using FF 3.0.14 running under Ubuntu. It appears to be something peculiar to FF. I tried it under a Virtual WinXP with FF 3.5 and saw the same behavior but Chrome looked fine. IMO it is really minor and cosmetic. I would not see this as a show stopper but it is strange. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            Here are screen shots from Chrome on WinXP virtual box, FF 3.5 on WinXP with small text and larger text.

            Show
            aborrow Anthony Borrow added a comment - Here are screen shots from Chrome on WinXP virtual box, FF 3.5 on WinXP with small text and larger text.
            Hide
            aborrow Anthony Borrow added a comment -

            Thanks Nicolas - as long as it is a separate issue. The work on this looks good to me. I play with little things like this while I am in class between taking notes. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Thanks Nicolas - as long as it is a separate issue. The work on this looks good to me. I play with little things like this while I am in class between taking notes. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            p.s. fyi - I did not have screenreader on but it does behave much nicer with static width of the student column.

            Show
            aborrow Anthony Borrow added a comment - p.s. fyi - I did not have screenreader on but it does behave much nicer with static width of the student column.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            I tested the patch with ie6 and 7 on windows 7 and took screenshots for both of them.

            Show
            rwijaya Rossiani Wijaya added a comment - I tested the patch with ie6 and 7 on windows 7 and took screenshots for both of them.
            Hide
            timhunt Tim Hunt added a comment -

            Note, the patch committed here was buggy. See MDL-21159.

            Show
            timhunt Tim Hunt added a comment - Note, the patch committed here was buggy. See MDL-21159 .
            Hide
            andyjdavis Andrew Davis added a comment -

            Closing. Any remaining display problems can be dealt with in separate issues

            Show
            andyjdavis Andrew Davis added a comment - Closing. Any remaining display problems can be dealt with in separate issues

              People

              • Votes:
                7 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Nov/09