MDL-54987 mod_choice: Fix notices and behat failures
authorSimey Lameze <simey@moodle.com>
Tue, 26 Jul 2016 07:51:45 +0000 (15:51 +0800)
committerSimey Lameze <simey@moodle.com>
Tue, 26 Jul 2016 08:24:35 +0000 (16:24 +0800)
mod/choice/renderer.php
mod/choice/tests/behat/publish_results.feature
mod/choice/tests/behat/publish_results_anonymously.feature

index 7d78435..7bd4a9c 100644 (file)
@@ -316,16 +316,20 @@ class mod_choice_renderer extends plugin_renderer_base {
         global $OUTPUT;
         $count = 0;
         $data = [];
+        $numberofuser = 0;
         $percentageamount = 0;
         foreach ($choices->options as $optionid => $option) {
+            if (!empty($option->user)) {
+                $numberofuser = count($option->user);
+            }
             if($choices->numberofuser > 0) {
-                $percentageamount = ((float)count($option->user) / (float)$choices->numberofuser) * 100.0;
+                $percentageamount = ((float)$numberofuser / (float)$choices->numberofuser) * 100.0;
             }
-
             $data['labels'][$count] = $option->text;
-            $data['series'][$count] = count($option->user);
+            $data['series'][$count] = $numberofuser;
             $data['series_labels'][$count] = '(' . format_float($percentageamount, 1) . '%)';
             $count++;
+            $numberofuser = 0;
         }
 
         $chart = new \core\chart_bar();
index 16faeab..d0f9bc2 100644 (file)
@@ -47,12 +47,10 @@ Feature: A teacher can choose one of 4 options for publishing choice results
     And I follow "Course 1"
     When I follow "Choice 1"
     Then I should not see "Responses"
-    And I should not see "Graph display"
     And I follow "Course 1"
     And I choose "Option 1" from "Choice 1" choice activity
     And I should see "Your selection: Option 1"
     And I should see "Responses"
-    And I should see "Graph display"
 
   Scenario: Show results to students only after the choice is closed
     Given I add a "Choice" to section "1" and I fill the form with:
@@ -66,7 +64,6 @@ Feature: A teacher can choose one of 4 options for publishing choice results
     And I follow "Course 1"
     When I follow "Choice 1"
     Then I should not see "Responses"
-    And I should not see "Graph display"
     And I choose "Option 1" from "Choice 1" choice activity
     And I log out
     And I log in as "teacher1"
@@ -88,7 +85,6 @@ Feature: A teacher can choose one of 4 options for publishing choice results
     And I follow "Course 1"
     And I follow "Choice 1"
     And I should see "Responses"
-    And I should see "Graph display"
 
   Scenario: Always show results to students
     Given I add a "Choice" to section "1" and I fill the form with:
@@ -102,4 +98,3 @@ Feature: A teacher can choose one of 4 options for publishing choice results
     And I follow "Course 1"
     When I follow "Choice 1"
     And I should see "Responses"
-    And I should see "Graph display"
index 374f97d..3b0cefd 100644 (file)
@@ -40,7 +40,6 @@ Feature: A teacher can choose whether to publish choice activity results anonymo
     And I follow "Choice 1"
     Then I should not see "Student 1"
     And I should not see "Users who chose this option"
-    And ".results .graph img" "css_element" should exist
 
   Scenario: Publish full results
     Given I add a "Choice" to section "1" and I fill the form with: