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

"By student name" sort order of quiz manual grading results in error "sql_fullname() not available anymore"

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1, 2.2
    • Fix Version/s: 2.1.2
    • Component/s: Quiz
    • Labels:

      Description

      Fresh installation of 2.1.1+. When manually grading quiz attempts (such as an essay question), sorting attempts using "By student name" sort option results in error "sql_fullname() not available anymore."

      Steps to Reproduce
      Assumption: Quiz with attempts that require manual grading exist.
      Click quiz.
      Click attempts.
      Click manual grading.
      Click grade.
      Select "By student name" as order attempts.
      Click Change options button.

      Expected - Attempts display by student name.
      Actual - Error displays.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            I love stealing easy bug fixes from others, ciao Tim.

            Thanks for the report!

            Show
            skodak Petr Skoda added a comment - I love stealing easy bug fixes from others, ciao Tim. Thanks for the report!
            Hide
            timhunt Tim Hunt added a comment -

            Thanks Petr. Please may I request:

            1. When you steal a quiz/question bug from me, you add me as a watcher.

            2. You give me a chance to peer review the fix before submitting it for integration.

            I have just peer-reviewed this, and +1 from me.

            Show
            timhunt Tim Hunt added a comment - Thanks Petr. Please may I request: 1. When you steal a quiz/question bug from me, you add me as a watcher. 2. You give me a chance to peer review the fix before submitting it for integration. I have just peer-reviewed this, and +1 from me.
            Hide
            skodak Petr Skoda added a comment -

            Right, I am guilty of breaking the proper review process, sorry.

            Show
            skodak Petr Skoda added a comment - Right, I am guilty of breaking the proper review process, sorry.
            Hide
            nebgor Aparup Banerjee added a comment -

            Thanks everyone! This has been integrated!

            Show
            nebgor Aparup Banerjee added a comment - Thanks everyone! This has been integrated!
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Works Great.
            Thanks for fixing this Petr

            Show
            rajeshtaneja Rajesh Taneja added a comment - Works Great. Thanks for fixing this Petr
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Oct/11