Moodle
  1. Moodle
  2. MDL-22945

Categories with random questions sometimes are not backup'ed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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
    • Rank:
      2653

      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.

        Activity

        Hide
        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
        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
        Vadim Dvorovenko added a comment -

        0021 is right file. 0023 is not needed

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

        Hi Tim,

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

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Hi Tim, does that change have sense? Can you take a look to it... TIA! Ciao
        Hide
        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
        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
        Tim Hunt added a comment -

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

        Show
        Tim Hunt added a comment - Just assigning this to you, since it looks like you are in the process of fixing it.
        Hide
        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
        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
        Tim Hunt added a comment -

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

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

        issue is outdated and unreproducable, please close

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

        Thanks for reporting back. Closing the issue as requested.

        Show
        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: