Fixed #2764

craftcms · Apr 18, 2018 · 5332468 · 5332468
1 parent 05b6d07
commit 5332468
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG-v3.md b/CHANGELOG-v3.md
@@ -4,6 +4,7 @@
 
 ### Fixed
 - Fixed an error that occurred when editing an entry if any of the entry’s revisions were created with an entry type that no longer exists.
+- Fixed an error that could occur when saving an asset. ([#2764](https://github.com/craftcms/cms/issues/2764))
 
 ## 3.0.3 - 2018-04-17
 

diff --git a/src/elements/Asset.php b/src/elements/Asset.php
@@ -1212,9 +1212,12 @@ public function afterSave(bool $isNew)
     {
         // If this is just an element being propagated, there's absolutely no need for re-saving this.
         if (!$this->propagating) {
-            if (AssetsHelper::getFileKindByExtension($this->tempFilePath) === static::KIND_IMAGE &&
+            if (
+                $this->tempFilePath !== null &&
+                AssetsHelper::getFileKindByExtension($this->tempFilePath) === static::KIND_IMAGE &&
                 \in_array($this->getScenario(), [self::SCENARIO_REPLACE, self::SCENARIO_CREATE], true) &&
-                null !== $this->tempFilePath) {
+                null !== $this->tempFilePath
+            ) {
                 Image::cleanImageByPath($this->tempFilePath);
             }