Fixed: crash when using obfuscated IDs

Bart Koelman · Bart Koelman · commit fef483d8f38e · 2020-04-13T17:22:28.000+02:00
diff --git a/src/JsonApiDotNetCore/Services/DefaultResourceService.cs b/src/JsonApiDotNetCore/Services/DefaultResourceService.cs
@@ -213,7 +213,7 @@ public virtual async Task<TResource> UpdateAsync(TId id, TResource requestEntity
             }
 
             _repository.FlushFromCache(databaseEntity);
-            TResource afterEntity = await _repository.Get(databaseEntity.Id).FirstOrDefaultAsync();
+            TResource afterEntity = await _repository.Get(id).FirstOrDefaultAsync();
             _resourceChangeTracker.SetFinallyStoredAttributeValues(afterEntity);
 
             bool hasImplicitChanges = _resourceChangeTracker.HasImplicitChanges();

Original file line number	Diff line number	Diff line change
`@@ -213,7 +213,7 @@ public virtual async Task<TResource> UpdateAsync(TId id, TResource requestEntity`
`213`	`213`	`}`
`214`	`214`
`215`	`215`	`_repository.FlushFromCache(databaseEntity);`
`216`		`- TResource afterEntity = await _repository.Get(databaseEntity.Id).FirstOrDefaultAsync();`
	`216`	`+ TResource afterEntity = await _repository.Get(id).FirstOrDefaultAsync();`
`217`	`217`	`_resourceChangeTracker.SetFinallyStoredAttributeValues(afterEntity);`
`218`	`218`
`219`	`219`	`bool hasImplicitChanges = _resourceChangeTracker.HasImplicitChanges();`