Merge pull request grokability#16118 from marcusmoore/fixes/undefined-key-in-asset-observer

snipe · web-flow · commit e53ed2319c07 · 2025-01-22T22:30:55.000Z
diff --git a/app/Observers/AssetObserver.php b/app/Observers/AssetObserver.php
@@ -48,7 +48,7 @@ public function updating(Asset $asset)
             $changed = [];
 
             foreach ($asset->getRawOriginal() as $key => $value) {
-                if ($asset->getRawOriginal()[$key] != $asset->getAttributes()[$key]) {
+                if ((array_key_exists($key, $asset->getAttributes())) && ($asset->getRawOriginal()[$key] != $asset->getAttributes()[$key])) {
                     $changed[$key]['old'] = $asset->getRawOriginal()[$key];
                     $changed[$key]['new'] = $asset->getAttributes()[$key];
                 }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public function updating(Asset $asset)`
`48`	`48`	`$changed = [];`
`49`	`49`
`50`	`50`	`foreach ($asset->getRawOriginal() as $key => $value) {`
`51`		`- if ($asset->getRawOriginal()[$key] != $asset->getAttributes()[$key]) {`
	`51`	`+ if ((array_key_exists($key, $asset->getAttributes())) && ($asset->getRawOriginal()[$key] != $asset->getAttributes()[$key])) {`
`52`	`52`	`$changed[$key]['old'] = $asset->getRawOriginal()[$key];`
`53`	`53`	`$changed[$key]['new'] = $asset->getAttributes()[$key];`
`54`	`54`	`}`