From d1256c5608e8ac799d91dc2526e0b0ddf5c38896 Mon Sep 17 00:00:00 2001 From: "jack.lewis" Date: Wed, 15 Jan 2025 09:21:53 +0000 Subject: [PATCH] simplify continue in canvasPainting --- .../BackgroundHandler/Helpers/ManifestMerger.cs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/IIIFPresentation/BackgroundHandler/Helpers/ManifestMerger.cs b/src/IIIFPresentation/BackgroundHandler/Helpers/ManifestMerger.cs index 4a847c0e..ea158532 100644 --- a/src/IIIFPresentation/BackgroundHandler/Helpers/ManifestMerger.cs +++ b/src/IIIFPresentation/BackgroundHandler/Helpers/ManifestMerger.cs @@ -22,9 +22,7 @@ public static Manifest Merge(Manifest baseManifest, List? canvas // We want to use the canvas order set when creating assets, rather than the foreach (var canvasPainting in orderedCanvasPaintings) { - itemDictionary.TryGetValue(canvasPainting.AssetId!, out var generatedItem); - - if (generatedItem == null) continue; + if (!itemDictionary.TryGetValue(canvasPainting.AssetId!, out var generatedItem)) continue; var existingItem = indexedBaseManifest.FirstOrDefault(bm => bm.item.Id == generatedItem.Id);