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

Quiz: Question bank - select/deselect all shows strange behaviour (IE7/8)

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.2
          • Fix Version/s: 2.1.6, 2.2.3
          • Component/s: Questions, Quiz
          • Labels:
          • Testing Instructions:
            Hide

            1) create at least two questions.
            2) create a quiz (this is not necessary, but it will demo the functionality on "Editing quiz" page where question bank is displayed)
            3) go to question bank page and click on the checkbox in the table header using following browsers (It works in Firefox, chrome and IE9, and it is cnfusing on IE7 / IE8. For details see Description section in this bug.

            Show
            1) create at least two questions. 2) create a quiz (this is not necessary, but it will demo the functionality on "Editing quiz" page where question bank is displayed) 3) go to question bank page and click on the checkbox in the table header using following browsers (It works in Firefox, chrome and IE9, and it is cnfusing on IE7 / IE8. For details see Description section in this bug.
          • Workaround:
            Hide

            in questions/qbank.js

            YAHOO.util.Event.addListener(question_bank.headercheckbox, 'chnage', question_bank.header_checkbox_click);

            needs to be changed to:
            YAHOO.util.Event.addListener(question_bank.headercheckbox, 'click', question_bank.header_checkbox_click);

            Show
            in questions/qbank.js YAHOO.util.Event.addListener(question_bank.headercheckbox, 'chnage', question_bank.header_checkbox_click); needs to be changed to: YAHOO.util.Event.addListener(question_bank.headercheckbox, 'click', question_bank.header_checkbox_click);
          • Affected Branches:
            MOODLE_22_STABLE
          • Fixed Branches:
            MOODLE_21_STABLE, MOODLE_22_STABLE
          • Pull from Repository:

            Description

            At the top of the question bank there is a select all/deselect all tick box next to the 'T' sort option.

            Firefox, Chrome, IE9: Ticking and unticking this box had the expected outcome. It selects or deselects all the questions in the list so the tick boxes for the questions are either all empty or all ticked.

            IE7, IE8: Behaviour is different and not as expected. Ticking or unticking this box initially seemed to have no effect on the individual tick boxes below. What you need to do to get it to work is the following:
            1. Click in the box to either select or deselect all (nothing appears to happen to the individual questions)
            2. Now mouse click somewhere else on the page (you now see the expected reponse - individual question tick boxes are now all ticked or not as appropriate).

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  mkassaei Mahmoud Kassaei added a comment -

                  here is the diff for review
                  https://github.com/mkassaei/moodle/compare/MOODLE_22_STABLE...MDL-32048

                  Show
                  mkassaei Mahmoud Kassaei added a comment - here is the diff for review https://github.com/mkassaei/moodle/compare/MOODLE_22_STABLE...MDL-32048
                  Hide
                  Permalink
                  timhunt Tim Hunt added a comment -

                  Nice one. Thanks Mahmoud. Submitting for integration.

                  To INTEGRATORS, please cherry-pick this fix to the 21, 22 and master branches. Thanks.

                  Show
                  timhunt Tim Hunt added a comment - Nice one. Thanks Mahmoud. Submitting for integration. To INTEGRATORS, please cherry-pick this fix to the 21, 22 and master branches. Thanks.
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Thanks Mahmoud this has been integrated now.
                  Just as small request for the future, can you please structure your commit messages as laid out in our coding guidelines.

                  MDL-XXXXX component: commit message

                  Cheers
                  Sam

                  Show
                  samhemelryk Sam Hemelryk added a comment - Thanks Mahmoud this has been integrated now. Just as small request for the future, can you please structure your commit messages as laid out in our coding guidelines. MDL-XXXXX component: commit message Cheers Sam
                  Hide
                  Permalink
                  salvetore Michael de Raadt added a comment -

                  Test result: Success.

                  This now works consistenly in IE8 and other browsers. Tested in 2.1, 2.2 and master.

                  Show
                  salvetore Michael de Raadt added a comment - Test result: Success. This now works consistenly in IE8 and other browsers. Tested in 2.1, 2.2 and master.
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Congratulations are in order, you've made it, or at least your code has!
                  It's now part of Moodle and both the git and cvs repositories have been updated.

                  This issue is being marked as fixed and closed.

                  Thank you.

                  Show
                  samhemelryk Sam Hemelryk added a comment - Congratulations are in order, you've made it, or at least your code has! It's now part of Moodle and both the git and cvs repositories have been updated. This issue is being marked as fixed and closed. Thank you.

                    People

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

                      Dates

                      • Created:
                        Updated:
                        Resolved:
                        Fix Release Date:
                        14/May/12