diff --git a/lib/Drupal/checklistapi/ChecklistapiChecklist.php b/lib/Drupal/checklistapi/ChecklistapiChecklist.php index 18f71271ce10e9aa63b20d8747c93500930bafe0..0cfd7814bc25a125da768f8aaa8caa1559e201af 100644 --- a/lib/Drupal/checklistapi/ChecklistapiChecklist.php +++ b/lib/Drupal/checklistapi/ChecklistapiChecklist.php @@ -142,7 +142,7 @@ class ChecklistapiChecklist { public function saveProgress(array $values) { global $user; $time = time(); - $changed_items_counter = 0; + $num_changed_items = 0; $progress = array( '#changed' => $time, '#changed_by' => $user->uid, @@ -170,7 +170,7 @@ class ChecklistapiChecklist { '#completed' => $time, '#uid' => $user->uid, ); - $changed_items_counter++; + $num_changed_items++; } } else { @@ -178,7 +178,7 @@ class ChecklistapiChecklist { $new_item = 0; if ($old_item) { // Item was previously checked. - $changed_items_counter++; + $num_changed_items++; } } $progress[$item_key] = $new_item; @@ -192,7 +192,7 @@ class ChecklistapiChecklist { variable_set($this->getSavedProgressVariableName(), $progress); drupal_set_message(format_plural( - $changed_items_counter, + $num_changed_items, 'Checklist %title has been updated. 1 item changed.', 'Checklist %title has been updated. @count items changed.', array('%title' => $this->title) diff --git a/tests/checklistapi.test b/tests/checklistapi.test index 107e92777b9755614492edd62e5ff820e88c0468..816216d670386cae84faef9a14f4cb34abf60857 100644 --- a/tests/checklistapi.test +++ b/tests/checklistapi.test @@ -55,8 +55,7 @@ class ChecklistapiUnitTestCase extends DrupalUnitTestCase { * Test checklistapi_strtolowercamel(). */ public function testChecklistapiStrtolowercamel() { - $output = checklistapi_strtolowercamel('Abc def_ghi'); - $this->assertEqual($output, 'abcDefGhi', 'Converted string to lowerCamel case.'); + $this->assertEqual(checklistapi_strtolowercamel('Abc def_ghi'), 'abcDefGhi', 'Converted string to lowerCamel case.'); } }