Skip to content

Commit d450fd6

Browse files
rezeaumchurchward
authored andcommitted
Fix for issue sectiontext qtype (label) should NOT support feedback
Signed-off-by: Joseph Rézeau <[email protected]>
1 parent e8607f8 commit d450fd6

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

classes/question/sectiontext.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ public function required() {
5656
* True if question type supports feedback options. False by default.
5757
*/
5858
public function supports_feedback() {
59-
return true;
59+
return false;
6060
}
6161

6262
/**
@@ -70,7 +70,7 @@ public function supports_feedback_scores() {
7070
* True if the question supports feedback and has valid settings for feedback. Override if the default logic is not enough.
7171
*/
7272
public function valid_feedback() {
73-
return true;
73+
return false;
7474
}
7575

7676
/**

tests/behat/no_feedback_question_types.feature

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ Feature: In questionnaire, certain questions will not activate feedback options.
2323
And I log in as "teacher1"
2424
And I am on "Course 1" course homepage
2525
And I follow "Test questionnaire"
26-
And I navigate to "Advanced settings" in current page administration
27-
Then I should not see "Feedback options"
26+
And I navigate to "Feedback" in current page administration
27+
Then I should not see "Display Scores"
2828
And I follow "Questions"
2929
Then I should see "Add questions"
3030
And I add a "Check Boxes" question and I fill the form with:
@@ -62,6 +62,6 @@ Feature: In questionnaire, certain questions will not activate feedback options.
6262
| Max. text length | 15 |
6363
| Question Text | Enter some text |
6464
Then I should see "[Text Box] (Q8)"
65-
And I follow "Advanced settings"
66-
And I should not see "Feedback options"
65+
And I follow "Feedback"
66+
And I should not see "Display Scores"
6767
And I log out

0 commit comments

Comments
 (0)