Moodle
  1. Moodle
  2. MDL-12874

Should be an option to allow student to see all course grades from their profile at the site level

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.9
    • Fix Version/s: None
    • Component/s: Gradebook
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Rank:
      4050

      Description

      Currently, when a student clicks on their profile from the front page, then Activity Reports -> Grades, they get a list of grades for the front page, if any.

      Need to have an option in the gradebook settings for the administrator to set, enabling display of ALL courses, rather than just the front page items.

      See discussion at http://moodle.org/mod/forum/discuss.php?d=82574 - includes code for the display of all courses, but not the option!

      1. mdl-12874_2010_0513.patch
        2 kB
        Evan Donovan
      2. MDL-12874.diff
        3 kB
        Anthony Borrow
      1. MDL-12874.png
        109 kB

        Issue Links

          Activity

          Hide
          Anthony Borrow added a comment -

          Here is a diff file against 19STABLE

          Show
          Anthony Borrow added a comment - Here is a diff file against 19STABLE
          Hide
          Anthony Borrow added a comment -

          Attaching a screenshot to give folks an idea of how this would look. My +1. Peace - Anthony

          Show
          Anthony Borrow added a comment - Attaching a screenshot to give folks an idea of how this would look. My +1. Peace - Anthony
          Hide
          Anthony Borrow added a comment -

          Petr - You seem to be doing most of the work on the gradebook but please feel free to re-assign. I just wanted this issue to be seen and was afraid with it being assigned to Yu it might easily get overlooked. To facilitate things, I've created a diff and attached a screenshot to show Matt Campbell's code in action. Peace - Anthony

          Show
          Anthony Borrow added a comment - Petr - You seem to be doing most of the work on the gradebook but please feel free to re-assign. I just wanted this issue to be seen and was afraid with it being assigned to Yu it might easily get overlooked. To facilitate things, I've created a diff and attached a screenshot to show Matt Campbell's code in action. Peace - Anthony
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Sounds interesting IMO... although we should consider what happens with other tabs, should all them present the same "list of courses" interface at SITE level? And, in other courses, should we show those lists? Uhm... not really sure.

          Shouldn't the "my moodle" page be used to provide that quick navigation instead of the SITE course?

          Moving to improvement and adding MD to decide about it.

          Show
          Eloy Lafuente (stronk7) added a comment - Sounds interesting IMO... although we should consider what happens with other tabs, should all them present the same "list of courses" interface at SITE level? And, in other courses, should we show those lists? Uhm... not really sure. Shouldn't the "my moodle" page be used to provide that quick navigation instead of the SITE course? Moving to improvement and adding MD to decide about it.
          Hide
          Jamie Tinley added a comment -

          This seems like the same thing as MDL-12916 so please vote for that one too. Perhaps they can be consolidated into one request.

          Show
          Jamie Tinley added a comment - This seems like the same thing as MDL-12916 so please vote for that one too. Perhaps they can be consolidated into one request.
          Hide
          Martin Dougiamas added a comment -

          I'll keep this separate from MDL-12916 because I see it as more of an enhancement for the current user viewing their own profile at site level. Hmm

          Show
          Martin Dougiamas added a comment - I'll keep this separate from MDL-12916 because I see it as more of an enhancement for the current user viewing their own profile at site level. Hmm
          Hide
          Felix Berger added a comment -

          I acutally urgently need the feature of a global gradebook on user profile pages. It would be appreciated, if this found it's way into core!

          Show
          Felix Berger added a comment - I acutally urgently need the feature of a global gradebook on user profile pages. It would be appreciated, if this found it's way into core!
          Hide
          Gary Prosser added a comment -

          I would like to see this feature included in the core. Its particularly useful for tutors (also parents ?) who have a need to get at a students grades across all courses. I am using the code from Matt so that from Mentees Block its possible to select a student > goto profile > activity reports > grades and then see all course grades on a single page.

          Matt's code would ideally be supplemented with an export to spreadsheet facility !

          Show
          Gary Prosser added a comment - I would like to see this feature included in the core. Its particularly useful for tutors (also parents ?) who have a need to get at a students grades across all courses. I am using the code from Matt so that from Mentees Block its possible to select a student > goto profile > activity reports > grades and then see all course grades on a single page. Matt's code would ideally be supplemented with an export to spreadsheet facility !
          Hide
          N Hansen added a comment -

          Does the code posted above allow anyone OTHER than the user whose grades they are to see all their grades? We need the ability to allow an administrator in our program to be able to get an overall view of a student's progress and having this feature available to admins would be helpful.

          Show
          N Hansen added a comment - Does the code posted above allow anyone OTHER than the user whose grades they are to see all their grades? We need the ability to allow an administrator in our program to be able to get an overall view of a student's progress and having this feature available to admins would be helpful.
          Hide
          Anthony Borrow added a comment -

          Unfortunately, I do not recall; however, it is worth checking and I would say as long as the person trying to view the record has the view grade capability then they should be able to see it but more testing is needed to confirm that the roles and permissions are respected. I did not do much testing and relied on the existing grade code to do the checking. So my answer would be a cautious I think so but if you can test it and confirm that would be great. In particular I would be interested in how the report displays when one teacher views a student's grade report. I believe this is related to Eloy's questions about. We need to make sure that folks who are not entitled or have no need of seeing do in fact see what they should but not more. Peace - Anthony

          Show
          Anthony Borrow added a comment - Unfortunately, I do not recall; however, it is worth checking and I would say as long as the person trying to view the record has the view grade capability then they should be able to see it but more testing is needed to confirm that the roles and permissions are respected. I did not do much testing and relied on the existing grade code to do the checking. So my answer would be a cautious I think so but if you can test it and confirm that would be great. In particular I would be interested in how the report displays when one teacher views a student's grade report. I believe this is related to Eloy's questions about. We need to make sure that folks who are not entitled or have no need of seeing do in fact see what they should but not more. Peace - Anthony
          Hide
          N Hansen added a comment -

          I'll ask the client I am setting this up for if they want to test it and if we do I'll let you know the results for admins.

          Show
          N Hansen added a comment - I'll ask the client I am setting this up for if they want to test it and if we do I'll let you know the results for admins.
          Hide
          Anthony Borrow added a comment -

          Thanks, it would be great if we could confirm the behavior with some testing. It shouldn't take too much with a test server with some production data. Peace - Anthony

          Show
          Anthony Borrow added a comment - Thanks, it would be great if we could confirm the behavior with some testing. It shouldn't take too much with a test server with some production data. Peace - Anthony
          Hide
          Evan Donovan added a comment -

          Patch no longer applies to version "1.9.8+ (Build: 20100331)". Attached is a re-rolled version of the patch.

          However, I don't quite understand the best way to use this patch: It appeared from my testing that it only worked when users had the "see user activity reports" permission. However, that permission seemed to let them see the activity reports for everyone.

          Please correct me if I am missed something in how it should work. I do not regularly use Moodle, but needed a transcript-like feature for my organization's online college, live at www.cityvision.edu.

          Show
          Evan Donovan added a comment - Patch no longer applies to version "1.9.8+ (Build: 20100331)". Attached is a re-rolled version of the patch. However, I don't quite understand the best way to use this patch: It appeared from my testing that it only worked when users had the "see user activity reports" permission. However, that permission seemed to let them see the activity reports for everyone. Please correct me if I am missed something in how it should work. I do not regularly use Moodle, but needed a transcript-like feature for my organization's online college, live at www.cityvision.edu.
          Hide
          Evan Donovan added a comment -

          Note that I did not have Moodle from CVS, so I simply rolled the patch from inside the "course" directory.

          Show
          Evan Donovan added a comment - Note that I did not have Moodle from CVS, so I simply rolled the patch from inside the "course" directory.
          Hide
          Jamie Tinley added a comment - - edited

          Hi Mat and Anthony,

          I used your code solution in 1.9 and loved it for years! Any chance we can get this working for 2.5? thanks again for your last work! (fyi anyone else go here: https://moodle.org/mod/forum/discuss.php?d=82574 )

          I see it relates to MDL-30054 so please vote for that too!

          Show
          Jamie Tinley added a comment - - edited Hi Mat and Anthony, I used your code solution in 1.9 and loved it for years! Any chance we can get this working for 2.5? thanks again for your last work! (fyi anyone else go here: https://moodle.org/mod/forum/discuss.php?d=82574 ) I see it relates to MDL-30054 so please vote for that too!
          Hide
          Jamie Tinley added a comment -

          I tried to implement your fix from the link below on 2.5 and get this error Fatal error: Call to undefined function get_record() in C:\Program Files (x86)\Apache Software Foundation\Apache2.2\htdocs\moodle\course\user.php on line 120
          link: https://moodle.org/mod/forum/discuss.php?d=82574

          Show
          Jamie Tinley added a comment - I tried to implement your fix from the link below on 2.5 and get this error Fatal error: Call to undefined function get_record() in C:\Program Files (x86)\Apache Software Foundation\Apache2.2\htdocs\moodle\course\user.php on line 120 link: https://moodle.org/mod/forum/discuss.php?d=82574
          Hide
          Matt Campbell added a comment -

          Sorry I won't be able to provide much help, Jamie. Code was written for 1.9x, looks like it needed some work to get it back up working with later versions of 1.9. Definitely won't work for anything in 2.x.

          I haven't been in Moodle code since 1.9. I can check with staff to see if this is still an issue for us, how we are handling it if we do need this. However, best advise would be to keep following this in MDL-30054, perhaps even get that update to inlcude 2.5.

          Show
          Matt Campbell added a comment - Sorry I won't be able to provide much help, Jamie. Code was written for 1.9x, looks like it needed some work to get it back up working with later versions of 1.9. Definitely won't work for anything in 2.x. I haven't been in Moodle code since 1.9. I can check with staff to see if this is still an issue for us, how we are handling it if we do need this. However, best advise would be to keep following this in MDL-30054 , perhaps even get that update to inlcude 2.5.
          Hide
          Jamie Tinley added a comment -

          Thanks Matt. Any chance I could pay you to code it? It seems like I should be able to just make the overview report show on the course id=1. Thanks JT

          Show
          Jamie Tinley added a comment - Thanks Matt. Any chance I could pay you to code it? It seems like I should be able to just make the overview report show on the course id=1. Thanks JT

            People

            • Votes:
              49 Vote for this issue
              Watchers:
              23 Start watching this issue

              Dates

              • Created:
                Updated: