Merge branch 'MDL-61045_34' of git://github.com/timhunt/moodle into M…

…OODLE_34_STABLE
moodle · Dec 19, 2017 · fb8b947 · fb8b947
2 parents b844baf + 3093cc2
commit fb8b947
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/mod/quiz/mod_form.php b/mod/quiz/mod_form.php
@@ -242,6 +242,9 @@ protected function definition() {
                 'neq', 'wontmatch');
         $mform->disabledIf('overallfeedbackduring', 'preferredbehaviour',
                 'neq', 'wontmatch');
+        foreach (self::$reviewfields as $field => $notused) {
+            $mform->disabledIf($field . 'closed', 'timeclose[enabled]');
+        }
 
         // -------------------------------------------------------------------------------
         $mform->addElement('header', 'display', get_string('appearance'));

diff --git a/mod/quiz/tests/behat/settings_form_fields_disableif.feature b/mod/quiz/tests/behat/settings_form_fields_disableif.feature
@@ -95,3 +95,24 @@ Feature: Settings form fields disabled if not required
     And the "Each attempt builds on the last" "field" should be enabled
     And the "id_delay1_enabled" "field" should be enabled
     And the "id_delay2_enabled" "field" should be enabled
+
+  @javascript
+  Scenario: Depending on whether there is a close date, some review options are disabled.
+    When I add a "Quiz" to section "1"
+    And I expand all fieldsets
+    And I set the field "Name" to "Test quiz"
+    Then the "id_attemptclosed" "checkbox" should be disabled
+    Then the "id_correctnessclosed" "checkbox" should be disabled
+    Then the "id_marksclosed" "checkbox" should be disabled
+    Then the "id_specificfeedbackclosed" "checkbox" should be disabled
+    Then the "id_generalfeedbackclosed" "checkbox" should be disabled
+    Then the "id_rightanswerclosed" "checkbox" should be disabled
+    Then the "id_overallfeedbackclosed" "checkbox" should be disabled
+    And I set the field "id_timeclose_enabled" to "1"
+    Then the "id_attemptclosed" "checkbox" should be enabled
+    Then the "id_correctnessclosed" "checkbox" should be enabled
+    Then the "id_marksclosed" "checkbox" should be enabled
+    Then the "id_specificfeedbackclosed" "checkbox" should be enabled
+    Then the "id_generalfeedbackclosed" "checkbox" should be enabled
+    Then the "id_rightanswerclosed" "checkbox" should be enabled
+    Then the "id_overallfeedbackclosed" "checkbox" should be enabled