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

Error in manually grading 'Essay' type quiz questions in a quiz

    Details

    • Testing Instructions:
      Hide
      1. Login as the admin.
      2. Turn on debugging and show on screen.
      3. Create a course
      4. Enrol a student in the course
      5. Create quiz with unlimited attempts and an essay question.
      6. Login as the student.
      7. Attempt the quiz 20 times. (fewer may be okay)
      8. Login as the admin.
      9. Goto manually grading in the quiz.
      10. Click on grade link under To grade.
      11. The page will display correctly
      12. Change shown question per page to 20.
      13. The page will display correctly
      Show
      Login as the admin. Turn on debugging and show on screen. Create a course Enrol a student in the course Create quiz with unlimited attempts and an essay question. Login as the student. Attempt the quiz 20 times. (fewer may be okay) Login as the admin. Goto manually grading in the quiz. Click on grade link under To grade. The page will display correctly Change shown question per page to 20. The page will display correctly
    • Workaround:
      Hide

      Turn debugging down or off.

      Show
      Turn debugging down or off.
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      mdl41908-master

      Description

      PHP Unknown Error: Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context in /var/www/lib/pear/HTML/QuickForm.php on line 60

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            timhunt Tim Hunt added a comment -

            Thanks for finding and fixing it. I am surprised I have never seen that error.

            I assume this is ready for peer review.

            1. First line of the commit comment is too long.

            2. I would be surprised if the integrators accept the 2.3 branch, since that is now security fixes only. Still, no harm in having the branch.

            3. You could usefully edit the testing instructions. They currently explain how to reproduce the bug, not how to verify that it is fixed.

            So, almost ready for integration, but not quite.

            Show
            timhunt Tim Hunt added a comment - Thanks for finding and fixing it. I am surprised I have never seen that error. I assume this is ready for peer review. 1. First line of the commit comment is too long. 2. I would be surprised if the integrators accept the 2.3 branch, since that is now security fixes only. Still, no harm in having the branch. 3. You could usefully edit the testing instructions. They currently explain how to reproduce the bug, not how to verify that it is fixed. So, almost ready for integration, but not quite.
            Hide
            tlock Tim Lock added a comment -

            Hi Tim,

            Thanks for you comments, please advise where I can read up on the commit message guidelines?

            Show
            tlock Tim Lock added a comment - Hi Tim, Thanks for you comments, please advise where I can read up on the commit message guidelines?
            Show
            aolley Adam Olley added a comment - http://docs.moodle.org/dev/Commit_cheat_sheet
            Hide
            tlock Tim Lock added a comment -

            Updated commit msg and github links

            Show
            tlock Tim Lock added a comment - Updated commit msg and github links
            Hide
            timhunt Tim Hunt added a comment -

            The Master diff URL now points to an unrelated commit.

            Show
            timhunt Tim Hunt added a comment - The Master diff URL now points to an unrelated commit.
            Hide
            tlock Tim Lock added a comment -

            Fixed

            Show
            tlock Tim Lock added a comment - Fixed
            Hide
            timhunt Tim Hunt added a comment -

            I don't really like this. In fact, I really don't like this.

            But given the horrible state of the formslib code, what you have done is probably the least bad solution. Thanks.

            Note that, if you want to get depressed search for isError, and you will find other places calling it statically.

            Anyway, submitting for integration.

            Show
            timhunt Tim Hunt added a comment - I don't really like this. In fact, I really don't like this. But given the horrible state of the formslib code, what you have done is probably the least bad solution. Thanks. Note that, if you want to get depressed search for isError, and you will find other places calling it statically. Anyway, submitting for integration.
            Hide
            poltawski Dan Poltawski added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            poltawski Dan Poltawski added a comment - - edited

            Thanks Tim, integrated to master, 25 and 24.

            Tim your 25 branch contained many more commits so I cherry-picked there, it'd be good if that branch was clean for future.

            It looks like your MOODLE_25_STABLE branch has lots of commits which are different from upstream: https://github.com/tlock/moodle/compare/moodle:MOODLE_25_STABLE...MOODLE_25_STABLE

            Show
            poltawski Dan Poltawski added a comment - - edited Thanks Tim, integrated to master, 25 and 24. Tim your 25 branch contained many more commits so I cherry-picked there, it'd be good if that branch was clean for future. It looks like your MOODLE_25_STABLE branch has lots of commits which are different from upstream: https://github.com/tlock/moodle/compare/moodle:MOODLE_25_STABLE...MOODLE_25_STABLE
            Hide
            fred Frédéric Massart added a comment -

            Passing, thanks.

            I just noticed the following:

            Did you remember to call setType() for 'id'? Defaulting to PARAM_RAW cleaning.
            line 1334 of /lib/formslib.php: call to debugging()
            line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
            line 202 of /lib/formslib.php: call to moodleform->_process_submission()
            line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform()
            line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct()
            line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface()
            line 100 of /mod/quiz/report.php: call to quiz_grading_report->display()
            Did you remember to call setType() for 'mode'? Defaulting to PARAM_RAW cleaning.
            line 1334 of /lib/formslib.php: call to debugging()
            line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
            line 202 of /lib/formslib.php: call to moodleform->_process_submission()
            line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform()
            line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct()
            line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface()
            line 100 of /mod/quiz/report.php: call to quiz_grading_report->display()
            Did you remember to call setType() for 'slot'? Defaulting to PARAM_RAW cleaning.
            line 1334 of /lib/formslib.php: call to debugging()
            line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
            line 202 of /lib/formslib.php: call to moodleform->_process_submission()
            line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform()
            line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct()
            line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface()
            line 100 of /mod/quiz/report.php: call to quiz_grading_report->display()
            Did you remember to call setType() for 'qid'? Defaulting to PARAM_RAW cleaning.
            line 1334 of /lib/formslib.php: call to debugging()
            line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
            line 202 of /lib/formslib.php: call to moodleform->_process_submission()
            line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform()
            line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct()
            line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface()
            line 100 of /mod/quiz/report.php: call to quiz_grading_report->display()
            Did you remember to call setType() for 'page'? Defaulting to PARAM_RAW cleaning.
            line 1334 of /lib/formslib.php: call to debugging()
            line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
            line 202 of /lib/formslib.php: call to moodleform->_process_submission()
            line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform()
            line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct()
            line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface()
            line 100 of /mod/quiz/report.php: call to quiz_grading_report->display()
            

            Is there an issue for it?

            Show
            fred Frédéric Massart added a comment - Passing, thanks. I just noticed the following: Did you remember to call setType() for 'id'? Defaulting to PARAM_RAW cleaning. line 1334 of /lib/formslib.php: call to debugging() line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType() line 202 of /lib/formslib.php: call to moodleform->_process_submission() line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform() line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct() line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface() line 100 of /mod/quiz/report.php: call to quiz_grading_report->display() Did you remember to call setType() for 'mode'? Defaulting to PARAM_RAW cleaning. line 1334 of /lib/formslib.php: call to debugging() line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType() line 202 of /lib/formslib.php: call to moodleform->_process_submission() line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform() line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct() line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface() line 100 of /mod/quiz/report.php: call to quiz_grading_report->display() Did you remember to call setType() for 'slot'? Defaulting to PARAM_RAW cleaning. line 1334 of /lib/formslib.php: call to debugging() line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType() line 202 of /lib/formslib.php: call to moodleform->_process_submission() line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform() line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct() line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface() line 100 of /mod/quiz/report.php: call to quiz_grading_report->display() Did you remember to call setType() for 'qid'? Defaulting to PARAM_RAW cleaning. line 1334 of /lib/formslib.php: call to debugging() line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType() line 202 of /lib/formslib.php: call to moodleform->_process_submission() line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform() line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct() line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface() line 100 of /mod/quiz/report.php: call to quiz_grading_report->display() Did you remember to call setType() for 'page'? Defaulting to PARAM_RAW cleaning. line 1334 of /lib/formslib.php: call to debugging() line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType() line 202 of /lib/formslib.php: call to moodleform->_process_submission() line 51 of /mod/quiz/report/grading/gradingsettings_form.php: call to moodleform->moodleform() line 361 of /mod/quiz/report/grading/report.php: call to quiz_grading_settings_form->__construct() line 155 of /mod/quiz/report/grading/report.php: call to quiz_grading_report->display_grading_interface() line 100 of /mod/quiz/report.php: call to quiz_grading_report->display() Is there an issue for it?
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Yes, it's happening (somewhere in the French Polynesia, right now). And you did it, raising Moodle to new excellency levels.

            Or, if you prefer, yes, you fixed that boring issue.

            Thanks anyway! Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Yes, it's happening (somewhere in the French Polynesia, right now). And you did it, raising Moodle to new excellency levels. Or, if you prefer, yes, you fixed that boring issue. Thanks anyway! Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Nov/13