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

"Include questions from sub-categories too" has incorrect default behaviour

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.1.2
    • Component/s: Questions
    • Labels:
    • Testing Instructions:
      Hide

      Go to question bank, where there are some subcategories with some quuestions in them. Then chose the top level category (parent of the mentiond subcategories) from "Select a category", click on checkbox " Also show questions from sub-categories" the questions in subcats should display when this is ticked and not display when this is unticked.

      Show
      Go to question bank, where there are some subcategories with some quuestions in them. Then chose the top level category (parent of the mentiond subcategories) from "Select a category", click on checkbox " Also show questions from sub-categories" the questions in subcats should display when this is ticked and not display when this is unticked.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      When viewing a question bank, the "Include questions from sub-categories too" checkbox is unchecked. However, sub-category items are still displayed.

      Checking and unchecking the checkbox fixes the issue temporarily, but these sub-category elements should not be shown when the category page is first viewed.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            mkassaei Mahmoud Kassaei added a comment -
            Show
            mkassaei Mahmoud Kassaei added a comment - This has been fixed and put on https://github.com/mkassaei/moodle/commit/92e6e128ff76ee6677c6c5724a9d451587658af0 to be reviewed
            Hide
            timhunt Tim Hunt added a comment -

            Looks good to me. Thanks Mahmoud. Submitting for integration.

            Show
            timhunt Tim Hunt added a comment - Looks good to me. Thanks Mahmoud. Submitting for integration.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Note: Guess it's intentional not to have the "showquestiontext" checkbox in quiz, correct?

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks! Note: Guess it's intentional not to have the "showquestiontext" checkbox in quiz, correct?
            Hide
            nebgor Aparup Banerjee added a comment -

            Thanks, this works for me!

            Show
            nebgor Aparup Banerjee added a comment - Thanks, this works for me!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Sent upstream and closing, many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!

              People

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

                Dates

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