Skip to content

Commit 76bddd4

Browse files
vietlx426vietlx426
andauthored
Questionnaire: Make summary page even more intuitive (#640)
Co-authored-by: vietlx426 <vt2944@open.ac.uk>
1 parent e8c5758 commit 76bddd4

File tree

4 files changed

+21
-3
lines changed

4 files changed

+21
-3
lines changed

lang/en/questionnaire.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -570,6 +570,7 @@
570570
$string['responseformat'] = 'Response format';
571571
$string['responseoptions'] = 'Response options';
572572
$string['responses'] = 'Responses';
573+
$string['responses_breakdown'] = '(Submissions: {$a->responses} | In progress: {$a->incomplete})';
573574
$string['responseview'] = 'Students can view ALL responses';
574575
$string['responseview_help'] = 'You can specify who can see the responses of all respondents to submitted questionnaires (general statistics tables).';
575576
$string['responseview_link'] = 'mod/questionnaire/mod#Response_viewing';

questionnaire.class.php

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2891,6 +2891,23 @@ public function survey_results($rid = '', $uid=false, $pdf = false, $currentgrou
28912891
if ($userview === 'y') {
28922892
$respondentstring = get_string('submissions', 'questionnaire');
28932893
}
2894+
if (!$userview) {
2895+
$completedcount = 0;
2896+
$inprogresscount = 0;
2897+
2898+
foreach ($rows as $row) {
2899+
if ($row->complete === 'y') {
2900+
$completedcount++;
2901+
} else if ($row->complete === 'n') {
2902+
$inprogresscount++;
2903+
}
2904+
}
2905+
$numresps .= ' ' . get_string('responses_breakdown', 'questionnaire',
2906+
[
2907+
'responses' => $completedcount,
2908+
'incomplete' => $inprogresscount,
2909+
]);
2910+
}
28942911
$this->page->add_to_page('respondentinfo',
28952912
' ' . $respondentstring . ': <strong>' . $numresps . '</strong>');
28962913
if (empty($rows)) {

tests/behat/check_responses.feature

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ Feature: Review responses
3535
Then I should see "View all responses"
3636
And I navigate to "View all responses" in current page administration
3737
And I should see "View Default order"
38-
And I should see "Responses: 7"
38+
And I should see "Responses: 7 (Submissions: 6 | In progress: 1)"
3939
And I set the field "View" to "Full submissions"
4040
And I should see "Submissions: 6"
4141
And I set the field "View" to "Responses not submitted"
@@ -70,7 +70,7 @@ Feature: Review responses
7070
Then I should see "1 / 6"
7171
And I follow "Summary"
7272
And I should see "View Default order"
73-
And I should see "Responses: 6"
73+
And I should see "Responses: 6 (Submissions: 6 | In progress: 0)"
7474
And I follow "Delete ALL Responses"
7575
Then I should see "Are you sure you want to delete ALL the responses in this questionnaire?"
7676
And I press "Delete"

tests/behat/check_responses_capabilities.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ Feature: Review responses with different capabilities
3333
And I navigate to "View all responses" in current page administration
3434
Then I should see "All responses"
3535
And I should see "View Default order"
36-
And I should see "Responses: 7"
36+
And I should see "Responses: 7 (Submissions: 6 | In progress: 1)"
3737
And I log out
3838

3939
@javascript

0 commit comments

Comments
 (0)