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

Automated backup setting page needs additional validation

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8
    • Fix Version/s: 2.9
    • Component/s: Backup
    • Labels:
    • Testing Instructions:
      Hide

      Before upgrade

      1. Set your automated backup storage to Specified directory...
      2. Leave the directory backup_auto_destination empty
      3. Upgrade.
      4. Go back to the backup settings and confirm that you backup_auto_storage was reset to Course file area

      Validation test

      1. Try different combinations of backup_auto_storage and backup_auto_destination settings
      2. Make sure that if the storage includes an external directory the directory must be specified, valid and writeable.
      Show
      Before upgrade Set your automated backup storage to Specified directory... Leave the directory backup_auto_destination empty Upgrade. Go back to the backup settings and confirm that you backup_auto_storage was reset to Course file area Validation test Try different combinations of backup_auto_storage and backup_auto_destination settings Make sure that if the storage includes an external directory the directory must be specified, valid and writeable.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-48266-master
    • Sprint:
      BACKEND Sprint 18, BACKEND Sprint 19

      Description

      Currently if you set backup_auto_storage to "Specified directory for automated backups", it lets you leave backup_auto_destination empty. This seems extremely confusing.

      There needs to be validation.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              ankit_frenz Ankit Agarwal
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Mark Nelson
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/May/15