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

Categories with random questions sometimes are not backup'ed

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.9.8, 1.9.9, 2.2
    • Fix Version/s: None
    • Component/s: Backup, Questions, Quiz
    • Labels:
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_22_STABLE

      Description

      Sometimes, when importing quizes with random questions from one course to other, Moodle copies only random question in categoriy, but not all this category. This makes copied test unworkable.
      The problem was because cat_id for such categories were not saved correctly (See patch).

      Please commit to cvs.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            vadimon Vadim Dvorovenko added a comment -

            Testing showed that the problem has another roots.
            The problem was that there were backuped not only question used by selected quizes, but used by all quizes in course.
            Error while trying to backup unused categories limited backuped questions list.
            Use second patch. First file is not needed.

            Please add to cvs

            Show
            vadimon Vadim Dvorovenko added a comment - Testing showed that the problem has another roots. The problem was that there were backuped not only question used by selected quizes, but used by all quizes in course. Error while trying to backup unused categories limited backuped questions list. Use second patch. First file is not needed. Please add to cvs
            Hide
            vadimon Vadim Dvorovenko added a comment -

            0021 is right file. 0023 is not needed

            Show
            vadimon Vadim Dvorovenko added a comment - 0021 is right file. 0023 is not needed
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Hi Tim,

            does that change have sense? Can you take a look to it... TIA!

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Hi Tim, does that change have sense? Can you take a look to it... TIA! Ciao
            Hide
            timhunt Tim Hunt added a comment -

            It is plausible. I think the change is right, but I cannot be 100% sure what is correct here. The code is too tricky.

            Show
            timhunt Tim Hunt added a comment - It is plausible. I think the change is right, but I cannot be 100% sure what is correct here. The code is too tricky.
            Hide
            timhunt Tim Hunt added a comment -

            Just assigning this to you, since it looks like you are in the process of fixing it.

            Show
            timhunt Tim Hunt added a comment - Just assigning this to you, since it looks like you are in the process of fixing it.
            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting this issue.

            We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

            If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

            Michael d;

            lqjjLKA0p6

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6
            Hide
            timhunt Tim Hunt added a comment -

            Sadly, this is still an issue, although it is a complete nightmare to solve.

            Show
            timhunt Tim Hunt added a comment - Sadly, this is still an issue, although it is a complete nightmare to solve.
            Hide
            vadimon Vadim Dvorovenko added a comment -

            issue is outdated and unreproducable, please close

            Show
            vadimon Vadim Dvorovenko added a comment - issue is outdated and unreproducable, please close
            Hide
            timhunt Tim Hunt added a comment -

            Thanks for reporting back. Closing the issue as requested.

            Show
            timhunt Tim Hunt added a comment - Thanks for reporting back. Closing the issue as requested.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: