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

          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:
              1 Start watching this issue

              Dates

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