Merge pull request #4 from discoverygarden/ctda9-67

[CTDA9-67] Check update operation in access controller.

src/FixityCheckAccessControlHandler.php

@@ -24,6 +24,7 @@ class FixityCheckAccessControlHandler extends EntityAccessControlHandler {
       case 'view revision':
         return AccessResult::allowedIfHasPermission($account, 'view fixity checks')->cachePerPermissions();
 
+      case 'update':
       case 'delete':
         return AccessResult::allowedIfHasPermission($account, $admin_permission)->cachePerPermissions();
 

src/FixityCheckBatchCheck.php

@@ -179,7 +179,7 @@ class FixityCheckBatchCheck {
     }
 
     $files = $storage->getPeriodic($sandbox['offset'], $batch_size);
-    $end = min($sandbox['total'], $sandbox['offset'] + count($files));
+    $end = min($sandbox['remaining'], $sandbox['offset'] + count($files));
     $context['message'] = \t('Processing @start to @end', [
       '@start' => $sandbox['offset'],
       '@end' => $end,
@@ -219,6 +219,9 @@ class FixityCheckBatchCheck {
           if ($result->passed()) {
             $results['successful']++;
           }
+          else {
+            $results['failed']++;
+          }
         }
         else {
           // The check was not performed as the time elapsed since the last

src/FixityCheckServiceInterface.php

@@ -100,7 +100,7 @@ interface FixityCheckServiceInterface {
    *   elapsed since the last check has not exceed the required threshold.
    *
    * @return \Drupal\dgi_fixity\Entity\FixityCheckInterface|null
-   *   The resulting fixity_check if successful.
+   *   The resulting fixity_check if performed.
    *   NULL if the check was not performed because the time elapsed since the
    *   last check has not exceed the required threshold.
    */