Moodle
  1. Moodle
  2. MDL-27790 Temporary course remains after restore
  3. MDL-28453

temporary courses which are 'restoration in progress' shouldn't be visible at all.

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.0.4, 2.1.1
    • Component/s: Backup
    • Labels:
      None
    • Testing Instructions:
      Hide

      1) restore a course to a halfway stage and cancel
      2) there should be no temporary courses visible to guests.

      Show
      1) restore a course to a halfway stage and cancel 2) there should be no temporary courses visible to guests.
    • Workaround:
      Hide

      (1) go to the course setting, hide the course.
      or better
      (1) create a category for restored courses. hide this category. only restore courses into this category. (you can move the course after)

      Show
      (1) go to the course setting, hide the course. or better (1) create a category for restored courses. hide this category. only restore courses into this category. (you can move the course after)
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
      git@github.com:nebgor/moodle.git
    • Pull Master Branch:
    • Rank:
      17452

      Description

      This is a follow up on the parent issue.
      The fix, as a quick response temporary measure to clean up the cosmetics, is to make the course hidden.
      This in no means deals with the issue of the existence of a skeleton course. living skeletons are a no-no. The parent bug should deal with this.

        Activity

        Hide
        Aparup Banerjee added a comment -

        one liner for 2.x branches to set temporary restoration courses to hidden (cosmetic fix until parent bug is resolved)

        Show
        Aparup Banerjee added a comment - one liner for 2.x branches to set temporary restoration courses to hidden (cosmetic fix until parent bug is resolved)
        Hide
        Aparup Banerjee added a comment -

        up for integration (simple one liner)

        Show
        Aparup Banerjee added a comment - up for integration (simple one liner)
        Hide
        Petr Škoda added a comment -

        I agree this is a good temporary workaround, better to have course hidden then letting in students accidentally.

        The only small problem could if user restoring the course does not have capability to view/enter hidden courses (moodle/course:viewhiddencourses), but the teacher and editing teacher have it by default, so it should not be a big deal...

        Integrated, thanks.

        Show
        Petr Škoda added a comment - I agree this is a good temporary workaround, better to have course hidden then letting in students accidentally. The only small problem could if user restoring the course does not have capability to view/enter hidden courses (moodle/course:viewhiddencourses), but the teacher and editing teacher have it by default, so it should not be a big deal... Integrated, thanks.
        Hide
        Dongsheng Cai added a comment -

        Thanks.

        Show
        Dongsheng Cai added a comment - Thanks.
        Hide
        Petr Škoda added a comment -

        Thanks everybody, this is now part of the weekly build.

        Show
        Petr Škoda added a comment - Thanks everybody, this is now part of the weekly build.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: