MDL-46481 backup: removed collapsing and better lang string
authorSam Hemelryk <sam@moodle.com>
Fri, 1 Aug 2014 05:28:55 +0000 (17:28 +1200)
committerSam Hemelryk <sam@moodle.com>
Sun, 5 Oct 2014 03:21:38 +0000 (16:21 +1300)
Thanks Rex for the peer-review!

backup/util/ui/backup_moodleform.class.php
backup/util/ui/backup_ui_stage.class.php
backup/util/ui/base_moodleform.class.php
backup/util/ui/tests/behat/backup_courses.feature
backup/util/ui/tests/behat/behat_backup.php
lang/en/backup.php

index f00f0c9..f03888a 100644 (file)
@@ -69,8 +69,6 @@ class backup_initial_form extends backup_moodleform {
         $mform->addElement('submit', 'oneclickbackup', get_string('performoneclickbackup', 'backup'));
 
         parent::definition();
-
-        $mform->setDisableShortforms(false);
     }
 }
 /**
index 2b52880..cde79ff 100644 (file)
@@ -175,7 +175,6 @@ class backup_ui_stage_initial extends backup_ui_stage {
                 // For the initial stage we are only interested in the root settings
                 if ($task instanceof backup_root_task) {
                     $form->add_heading('rootsettings', get_string('rootsettings', 'backup'));
-                    $form->collapse_heading('rootsettings');
                     $settings = $task->get_settings();
                     // First add all settings except the filename setting
                     foreach ($settings as &$setting) {
index b358436..262142e 100644 (file)
@@ -369,13 +369,4 @@ abstract class base_moodleform extends moodleform {
             $this->definition_after_data();
         }
     }
-
-    /**
-     * Can be used to collapse a heading from outside of this form.
-     *
-     * @param string $heading
-     */
-    public function collapse_heading($heading) {
-        $this->_form->setExpanded($heading, false);
-    }
 }
index ee3fea3..de40df1 100644 (file)
@@ -64,4 +64,6 @@ Feature: Backup Moodle courses
   @javascript
   Scenario: Backup a course using the one click backup button
     When I perform a quick backup of course "Course 2"
-    Then I should see "Restore"
\ No newline at end of file
+    Then I should see "Restore course"
+    And I should see "Course backup area"
+    And I should see "backup-moodle2-course-"
\ No newline at end of file
index 064bfac..d26d10e 100644 (file)
@@ -77,9 +77,6 @@ class behat_backup extends behat_base {
         $this->navigate_to_course_settings_link('backup');
         $this->wait();
 
-        // Expand the backup settings section.
-        $this->find_link(get_string('backupsettings', 'backup'))->click();
-
         // Initial settings.
         $this->fill_backup_restore_form($this->get_step_options($options, "Initial"));
         $this->find_button(get_string('backupstage1action', 'backup'))->press();
index 61f6cb6..5025441 100644 (file)
@@ -178,7 +178,7 @@ $string['nomatchingcourses'] = 'There are no courses to display';
 $string['norestoreoptions'] = 'There are no categories or existing courses you can restore to.';
 $string['originalwwwroot'] = 'URL of backup';
 $string['performoneclickbackup'] = 'Perform quick backup';
-$string['performoneclickbackup_desc'] = 'Clicking this will create a backup using the current settings. If you wish to proceed through each stage of the backup individually use the controls at the bottom of the form.';
+$string['performoneclickbackup_desc'] = 'Performing a quick backup will create a backup using the currently selected settings below. If you wish to further make choices regarding what sections or activities that you want to include in your backup, please process onto the "Next" step.';
 $string['previousstage'] = 'Previous';
 $string['preparingui'] = 'Preparing to display page';
 $string['preparingdata'] = 'Preparing data';