refactor

src/TimeBasedFieldUpdater.php

@@ -1,7 +1,6 @@
 <?php declare(strict_types = 1);
 
 namespace Drupal\opencase;
-use Drupal;
 use Drupal\Core\Entity\EntityTypeManagerInterface;
 use Drupal\opencase\Utils;
 
@@ -9,9 +8,10 @@ final class TimeBasedFieldUpdater {
 
     private EntityTypeManagerInterface $entityTypeManager;
     private string $date_field;
+    private Utils $utils;
     private string $entity_type;
-    private array $conditions;
     private string $date_format;
+    private string $bundle;
 
     final public function __construct(
         EntityTypeManagerInterface $entityTypeManager, 
@@ -33,11 +33,14 @@ final class TimeBasedFieldUpdater {
         $conditions[] = ['type', $this->bundle, '='];
         $this->utils->addConditionsToQuery($query, $conditions);
         foreach($query->execute() as $id) {
-            $entity = $this->entityTypeManager->getStorage($this->entity_type)->load($id);
-            foreach($new_values as $new_field=>$new_value) {
-                $entity->$new_field = $new_value; 
-            }
-            $entity->save();
+            $this->updateEntity($id, $new_values);
         }      
     }
+    private function updateEntity(int $entity_id, array $new_values): void {
+        $entity = $this->entityTypeManager->getStorage($this->entity_type)->load($entity_id);
+        foreach($new_values as $new_field=>$new_value) {
+            $entity->$new_field = $new_value; 
+        }
+        $entity->save();
+    }
 }

src/Utils.php

@@ -50,4 +50,4 @@ class Utils {
         $term = reset($terms);
         return (int)(!empty($term) ? $term->id() : 0);
     }
-}
+}