Merge pull request #5046 from abpframework/maliming/Obsolete-pr-4893

Make the method that use propertyBuildAction obsolete.

framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/ObjectExtending/EfCoreObjectExtensionInfoExtensions.cs

@@ -6,6 +6,7 @@ namespace Volo.Abp.ObjectExtending
 {
     public static class EfCoreObjectExtensionInfoExtensions
     {
+        [Obsolete("Use MapEfCoreProperty with EntityTypeAndPropertyBuildAction parameters.")]
         public static ObjectExtensionInfo MapEfCoreProperty<TProperty>(
             [NotNull] this ObjectExtensionInfo objectExtensionInfo,
             [NotNull] string propertyName,
@@ -18,6 +19,7 @@ namespace Volo.Abp.ObjectExtending
             );
         }
 
+        [Obsolete("Use MapEfCoreProperty with EntityTypeAndPropertyBuildAction parameters.")]
         public static ObjectExtensionInfo MapEfCoreProperty(
             [NotNull] this ObjectExtensionInfo objectExtensionInfo,
             [NotNull] Type propertyType,

framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/ObjectExtending/EfCoreObjectExtensionManagerExtensions.cs

@@ -39,6 +39,7 @@ namespace Volo.Abp.ObjectExtending
             );
         }
 
+        [Obsolete("Use MapEfCoreProperty with EntityTypeAndPropertyBuildAction parameters.")]
         public static ObjectExtensionManager MapEfCoreProperty<TEntity, TProperty>(
             [NotNull] this ObjectExtensionManager objectExtensionManager,
             [NotNull] string propertyName,
@@ -53,6 +54,7 @@ namespace Volo.Abp.ObjectExtending
             );
         }
 
+        [Obsolete("Use MapEfCoreProperty with EntityTypeAndPropertyBuildAction parameters.")]
         public static ObjectExtensionManager MapEfCoreProperty(
             [NotNull] this ObjectExtensionManager objectExtensionManager,
             [NotNull] Type entityType,

framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/ObjectExtending/EfCoreObjectExtensionPropertyInfoExtensions.cs

@@ -23,6 +23,7 @@ namespace Volo.Abp.ObjectExtending
             return propertyExtension;
         }
 
+        [Obsolete("Use MapEfCore with EntityTypeAndPropertyBuildAction parameters.")]
         [NotNull]
         public static ObjectExtensionPropertyInfo MapEfCore(
             [NotNull] this ObjectExtensionPropertyInfo propertyExtension,
@@ -49,7 +50,7 @@ namespace Volo.Abp.ObjectExtending
             propertyExtension.Configuration[EfCorePropertyConfigurationName] =
                 new ObjectExtensionPropertyInfoEfCoreMappingOptions(
                     propertyExtension,
-                    entityTypeAndPropertyBuildAction: entityTypeAndPropertyBuildAction
+                    entityTypeAndPropertyBuildAction
                 );
 
             return propertyExtension;

framework/src/Volo.Abp.EntityFrameworkCore/Volo/Abp/ObjectExtending/ObjectExtensionPropertyInfoEfCoreMappingOptions.cs

@@ -12,12 +12,14 @@ namespace Volo.Abp.ObjectExtending
         [NotNull]
         public ObjectExtensionInfo ObjectExtension => ExtensionProperty.ObjectExtension;
 
+        [Obsolete("Use EntityTypeAndPropertyBuildAction property.")]
         [CanBeNull]
         public Action<PropertyBuilder> PropertyBuildAction { get; set; }
 
         [CanBeNull]
         public Action<EntityTypeBuilder, PropertyBuilder> EntityTypeAndPropertyBuildAction { get; set; }
 
+        [Obsolete("Use other constructors.")]
         public ObjectExtensionPropertyInfoEfCoreMappingOptions(
             [NotNull] ObjectExtensionPropertyInfo extensionProperty,
             [CanBeNull] Action<PropertyBuilder> propertyBuildAction = null,
@@ -28,5 +30,20 @@ namespace Volo.Abp.ObjectExtending
             PropertyBuildAction = propertyBuildAction;
             EntityTypeAndPropertyBuildAction = entityTypeAndPropertyBuildAction;
         }
+
+        public ObjectExtensionPropertyInfoEfCoreMappingOptions(
+            [NotNull] ObjectExtensionPropertyInfo extensionProperty)
+        {
+            ExtensionProperty = Check.NotNull(extensionProperty, nameof(extensionProperty));
+        }
+
+        public ObjectExtensionPropertyInfoEfCoreMappingOptions(
+            [NotNull] ObjectExtensionPropertyInfo extensionProperty,
+            [CanBeNull] Action<EntityTypeBuilder, PropertyBuilder> entityTypeAndPropertyBuildAction)
+        {
+            ExtensionProperty = Check.NotNull(extensionProperty, nameof(extensionProperty));
+
+            EntityTypeAndPropertyBuildAction = entityTypeAndPropertyBuildAction;
+        }
     }
 }