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 Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1, 2.2
    • Fix Version/s: 2.1.2
    • Component/s: Quiz
    • Labels:
    • Rank:
      18673

      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.

        Activity

        Hide
        Petr Škoda added a comment -

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

        Thanks for the report!

        Show
        Petr Škoda added a comment - I love stealing easy bug fixes from others, ciao Tim. Thanks for the report!
        Hide
        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
        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
        Petr Škoda added a comment -

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

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

        Thanks everyone! This has been integrated!

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

        Works Great.
        Thanks for fixing this Petr

        Show
        Rajesh Taneja added a comment - Works Great. Thanks for fixing this Petr
        Hide
        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
        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: