From c8f3b293295dc90b63fe832e26de5aad8d922a68 Mon Sep 17 00:00:00 2001 From: TekH Date: Wed, 10 Dec 2025 15:00:44 +0100 Subject: [PATCH] Refactor EnsureEntity for clarity and consistency Renamed local variable from 'cluster' to 'entities' in EnsureEntity and unified logic for selecting entity options. Improved readability by consistently using entities.TryGetValue and clearer variable naming. --- src/ReC.Infrastructure/Options/DbModelOptions.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ReC.Infrastructure/Options/DbModelOptions.cs b/src/ReC.Infrastructure/Options/DbModelOptions.cs index 1a8ac29..9e0445e 100644 --- a/src/ReC.Infrastructure/Options/DbModelOptions.cs +++ b/src/ReC.Infrastructure/Options/DbModelOptions.cs @@ -12,11 +12,11 @@ public record DbModelOptions public void EnsureEntity(bool isVirtual) { - var cluster = isVirtual + var entities = isVirtual ? VirtualEntities.ToDictionary(kvp => kvp.Key, kvp => kvp.Value as EntityBaseOptions) : Entities.ToDictionary(kvp => kvp.Key, kvp => kvp.Value as EntityBaseOptions); - if(cluster.TryGetValue(nameof(T), out var entityOptions)) + if(entities.TryGetValue(nameof(T), out var entityOptions)) entityOptions.EnsureProperties(); else throw new DbModelConfigurationException($"Entity options for type '{typeof(T).FullName}' not found.");