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

Refactor the graded_users_iterator class

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.3
    • Component/s: Gradebook
    • Labels:
      None
    • Testing Instructions:
      Hide

      Testing is just a matter of making sure I didnt break anything. Do the following and check that it all still works.

      Do a grade export in each format. txt, xls, xml, ods.

      Go to the user report in the gradebook. Select and user from the drop down and check that displays fine.

      Show
      Testing is just a matter of making sure I didnt break anything. Do the following and check that it all still works. Do a grade export in each format. txt, xls, xml, ods. Go to the user report in the gradebook. Select and user from the drop down and check that displays fine.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-32860_user_iterator

      Description

      Member variables have no php docs. Check all php docs while we're at it.

      All public member variables. Is that really necessary?

      There are some methods whose names start with and underscore. Are they meant to be private or protected?

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12