Skip to content

Commit ec50eda

Browse files
committed
Rename CheckPropertyBelongsToType to CheckContains
Per #32813 (review)
1 parent 29d3b3b commit ec50eda

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

src/EFCore/ChangeTracking/Internal/ArrayPropertyValues.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,8 @@ public override object? this[string propertyName]
132132
/// </summary>
133133
public override object? this[IProperty property]
134134
{
135-
get => _values[EntityType.CheckPropertyBelongsToType(property).GetIndex()];
136-
set => SetValue(EntityType.CheckPropertyBelongsToType(property).GetIndex(), value);
135+
get => _values[EntityType.CheckContains(property).GetIndex()];
136+
set => SetValue(EntityType.CheckContains(property).GetIndex(), value);
137137
}
138138

139139
/// <summary>

src/EFCore/ChangeTracking/Internal/CurrentPropertyValues.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public override TValue GetValue<TValue>(string propertyName)
4040
/// doing so can result in application failures when updating to a new Entity Framework Core release.
4141
/// </summary>
4242
public override TValue GetValue<TValue>(IProperty property)
43-
=> InternalEntry.GetCurrentValue<TValue>(InternalEntry.EntityType.CheckPropertyBelongsToType(property));
43+
=> InternalEntry.GetCurrentValue<TValue>(InternalEntry.EntityType.CheckContains(property));
4444

4545
/// <summary>
4646
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to

src/EFCore/ChangeTracking/Internal/EntryPropertyValues.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,8 +130,8 @@ public override object? this[string propertyName]
130130
/// </summary>
131131
public override object? this[IProperty property]
132132
{
133-
get => GetValueInternal(EntityType.CheckPropertyBelongsToType(property));
134-
set => SetValueInternal(EntityType.CheckPropertyBelongsToType(property), value);
133+
get => GetValueInternal(EntityType.CheckContains(property));
134+
set => SetValueInternal(EntityType.CheckContains(property), value);
135135
}
136136

137137
/// <summary>

src/EFCore/ChangeTracking/Internal/OriginalPropertyValues.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public override TValue GetValue<TValue>(string propertyName)
4040
/// doing so can result in application failures when updating to a new Entity Framework Core release.
4141
/// </summary>
4242
public override TValue GetValue<TValue>(IProperty property)
43-
=> InternalEntry.GetOriginalValue<TValue>(InternalEntry.EntityType.CheckPropertyBelongsToType(property));
43+
=> InternalEntry.GetOriginalValue<TValue>(InternalEntry.EntityType.CheckContains(property));
4444

4545
/// <summary>
4646
/// This is an internal API that supports the Entity Framework Core infrastructure and not subject to

src/EFCore/Metadata/Internal/EntityTypeExtensions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ public static IEnumerable<IReadOnlyNavigation> FindDerivedNavigations(
303303
/// any release. You should only use it directly in your code with extreme caution and knowing that
304304
/// doing so can result in application failures when updating to a new Entity Framework Core release.
305305
/// </summary>
306-
public static IProperty CheckPropertyBelongsToType(
306+
public static IProperty CheckContains(
307307
this IEntityType entityType,
308308
IProperty property)
309309
{

0 commit comments

Comments
 (0)