Uploaded image for project: '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
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.9
    • Fix Version/s: None
    • Component/s: Gradebook
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE

      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!

        Gliffy Diagrams

        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
            aborrow Anthony Borrow added a comment -

            Here is a diff file against 19STABLE

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

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

            Show
            aborrow Anthony Borrow added a comment - Attaching a screenshot to give folks an idea of how this would look. My +1. Peace - Anthony
            Hide
            aborrow 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
            aborrow 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
            stronk7 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
            stronk7 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
            jtinley 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
            jtinley 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
            dougiamas 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
            dougiamas 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
            fberger 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
            fberger 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
            garyprosser 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
            garyprosser 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
            hatshepsut 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
            hatshepsut 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
            aborrow 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
            aborrow 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
            hatshepsut 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
            hatshepsut 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
            aborrow 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
            aborrow 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
            evandonovan 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
            evandonovan 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
            evandonovan 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
            evandonovan 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
            jtinley 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
            jtinley 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
            jtinley 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
            jtinley 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
            mcampbell 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
            mcampbell 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
            jtinley 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
            jtinley 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:
                50 Vote for this issue
                Watchers:
                23 Start watching this issue

                Dates

                • Created:
                  Updated: