Rename CheckPropertyBelongsToType to CheckContains (#32888)

Per #32813 (review)
dotnet · Jan 23, 2024 · 7003680 · 7003680
1 parent 4380d3e
commit 7003680
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/src/EFCore/ChangeTracking/Internal/ArrayPropertyValues.cs b/src/EFCore/ChangeTracking/Internal/ArrayPropertyValues.cs
@@ -132,8 +132,8 @@ public override object? this[string propertyName]
  /// </summary>
  public override object? this[IProperty property]
  {
- get => _values[EntityType.CheckPropertyBelongsToType(property).GetIndex()];
- set => SetValue(EntityType.CheckPropertyBelongsToType(property).GetIndex(), value);
+ get => _values[EntityType.CheckContains(property).GetIndex()];
+ set => SetValue(EntityType.CheckContains(property).GetIndex(), value);
  }
 
  /// <summary>

diff --git a/src/EFCore/ChangeTracking/Internal/CurrentPropertyValues.cs b/src/EFCore/ChangeTracking/Internal/CurrentPropertyValues.cs
@@ -40,7 +40,7 @@ public override TValue GetValue<TValue>(string propertyName)
  /// doing so can result in application failures when updating to a new Entity Framework Core release.
  /// </summary>
  public override TValue GetValue<TValue>(IProperty property)
- => InternalEntry.GetCurrentValue<TValue>(InternalEntry.EntityType.CheckPropertyBelongsToType(property));
+ => InternalEntry.GetCurrentValue<TValue>(InternalEntry.EntityType.CheckContains(property));
 
  /// <summary>
  /// This is an internal API that supports the Entity Framework Core infrastructure and not subject to

diff --git a/src/EFCore/ChangeTracking/Internal/EntryPropertyValues.cs b/src/EFCore/ChangeTracking/Internal/EntryPropertyValues.cs
@@ -130,8 +130,8 @@ public override object? this[string propertyName]
  /// </summary>
  public override object? this[IProperty property]
  {
- get => GetValueInternal(EntityType.CheckPropertyBelongsToType(property));
- set => SetValueInternal(EntityType.CheckPropertyBelongsToType(property), value);
+ get => GetValueInternal(EntityType.CheckContains(property));
+ set => SetValueInternal(EntityType.CheckContains(property), value);
  }
 
  /// <summary>

diff --git a/src/EFCore/ChangeTracking/Internal/OriginalPropertyValues.cs b/src/EFCore/ChangeTracking/Internal/OriginalPropertyValues.cs
@@ -40,7 +40,7 @@ public override TValue GetValue<TValue>(string propertyName)
  /// doing so can result in application failures when updating to a new Entity Framework Core release.
  /// </summary>
  public override TValue GetValue<TValue>(IProperty property)
- => InternalEntry.GetOriginalValue<TValue>(InternalEntry.EntityType.CheckPropertyBelongsToType(property));
+ => InternalEntry.GetOriginalValue<TValue>(InternalEntry.EntityType.CheckContains(property));
 
  /// <summary>
  /// This is an internal API that supports the Entity Framework Core infrastructure and not subject to

diff --git a/src/EFCore/Metadata/Internal/EntityTypeExtensions.cs b/src/EFCore/Metadata/Internal/EntityTypeExtensions.cs
@@ -303,7 +303,7 @@ public static IEnumerable<IReadOnlyNavigation> FindDerivedNavigations(
  /// any release. You should only use it directly in your code with extreme caution and knowing that
  /// doing so can result in application failures when updating to a new Entity Framework Core release.
  /// </summary>
- public static IProperty CheckPropertyBelongsToType(
+ public static IProperty CheckContains(
  this IEntityType entityType,
  IProperty property)
  {