Merge pull request #895 from seth-shaw-asu/issue-2152

Issue-2152: Throw error instead of returning null data
Islandora · Aug 11, 2022 · 62fbc6d · 62fbc6d
2 parents 7bca3d5 + d405a2f
commit 62fbc6d
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/Plugin/Action/AbstractGenerateDerivativeMediaFile.php b/src/Plugin/Action/AbstractGenerateDerivativeMediaFile.php
@@ -40,7 +40,7 @@ public function defaultConfiguration() {
   protected function generateData(EntityInterface $entity) {
     $data = parent::generateData($entity);
     if (get_class($entity) != 'Drupal\media\Entity\Media') {
-      return;
+      throw new \RuntimeException("Entity {$entity->getEntityTypeId()} {$entity->id()} is not a media", 500);
     }
     $source_file = $this->mediaSource->getSourceFile($entity);
     if (!$source_file) {