Remove Obsolete methods for 2.0

OrchardCMS · May 1, 2024 · e242491 · e242491
1 parent d336703
commit e242491
Show file tree

Hide file tree

Showing 48 changed files with 18 additions and 857 deletions.
diff --git a/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IContentDefinitionService.cs b/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IContentDefinitionService.cs
@@ -57,104 +57,4 @@ public interface IContentDefinitionService
     Task AlterTypePartsOrderAsync(ContentTypeDefinition typeDefinition, string[] partNames);
 
     Task AlterPartFieldsOrderAsync(ContentPartDefinition partDefinition, string[] fieldNames);
-
-    [Obsolete($"Instead, utilize the {nameof(LoadTypesAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<EditTypeViewModel> LoadTypes()
-=> LoadTypesAsync().GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GetTypesAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<EditTypeViewModel> GetTypes()
-        => GetTypesAsync().GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(LoadTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    EditTypeViewModel LoadType(string name)
-        => LoadTypeAsync(name).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GetTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    EditTypeViewModel GetType(string name)
-        => GetTypeAsync(name).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    ContentTypeDefinition AddType(string name, string displayName)
-        => AddTypeAsync(name, displayName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(RemoveTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void RemoveType(string name, bool deleteContent)
-        => RemoveTypeAsync(name, deleteContent).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddPartToTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AddPartToType(string partName, string typeName)
-        => AddPartToTypeAsync(partName, typeName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddReusablePartToTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AddReusablePartToType(string name, string displayName, string description, string partName, string typeName)
-        => AddReusablePartToTypeAsync(name, displayName, description, partName, typeName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(RemovePartFromTypeAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void RemovePartFromType(string partName, string typeName)
-        => RemovePartFromTypeAsync(partName, typeName);
-
-    [Obsolete($"Instead, utilize the {nameof(GenerateContentTypeNameFromDisplayNameAsync)} method. This current method is slated for removal in upcoming releases.")]
-    string GenerateContentTypeNameFromDisplayName(string displayName)
-        => GenerateContentTypeNameFromDisplayNameAsync(displayName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GenerateFieldNameFromDisplayNameAsync)} method. This current method is slated for removal in upcoming releases.")]
-    string GenerateFieldNameFromDisplayName(string partName, string displayName)
-        => GenerateFieldNameFromDisplayNameAsync(partName, displayName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(LoadPartsAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<EditPartViewModel> LoadParts(bool metadataPartsOnly)
-        => LoadPartsAsync(metadataPartsOnly).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GetPartsAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<EditPartViewModel> GetParts(bool metadataPartsOnly)
-        => GetPartsAsync(metadataPartsOnly).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(LoadPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    EditPartViewModel LoadPart(string name)
-        => LoadPartAsync(name).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GetPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    EditPartViewModel GetPart(string name)
-        => GetPartAsync(name).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    EditPartViewModel AddPart(CreatePartViewModel partViewModel)
-        => AddPartAsync(partViewModel).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(RemovePartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void RemovePart(string name)
-        => RemovePartAsync(name).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(GetFieldsAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<Type> GetFields()
-        => GetFieldsAsync().GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddFieldToPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AddFieldToPart(string fieldName, string fieldTypeName, string partName)
-        => AddFieldToPartAsync(fieldName, fieldTypeName, partName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AddFieldToPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AddFieldToPart(string fieldName, string displayName, string fieldTypeName, string partName)
-        => AddFieldToPartAsync(fieldName, displayName, fieldTypeName, partName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(RemoveFieldFromPartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void RemoveFieldFromPart(string fieldName, string partName)
-        => RemoveFieldFromPartAsync(fieldName, partName).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AlterFieldAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AlterField(EditPartViewModel partViewModel, EditFieldViewModel fieldViewModel)
-        => AlterFieldAsync(partViewModel, fieldViewModel).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AlterTypePartAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AlterTypePart(EditTypePartViewModel partViewModel)
-        => AlterTypePartAsync(partViewModel).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AlterTypePartsOrderAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AlterTypePartsOrder(ContentTypeDefinition typeDefinition, string[] partNames)
-        => AlterTypePartsOrderAsync(typeDefinition, partNames).GetAwaiter().GetResult();
-
-    [Obsolete($"Instead, utilize the {nameof(AlterPartFieldsOrderAsync)} method. This current method is slated for removal in upcoming releases.")]
-    void AlterPartFieldsOrder(ContentPartDefinition partDefinition, string[] fieldNames)
-        => AlterPartFieldsOrderAsync(partDefinition, fieldNames).GetAwaiter().GetResult();
 }
diff --git a/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IStereotypeService.cs b/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IStereotypeService.cs
@@ -1,4 +1,3 @@
-using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
 
@@ -7,8 +6,4 @@ namespace OrchardCore.ContentTypes.Services;
 public interface IStereotypeService
 {
     Task<IEnumerable<StereotypeDescription>> GetStereotypesAsync();
-
-    [Obsolete($"Instead, utilize the {nameof(GetStereotypesAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<StereotypeDescription> GetStereotypes()
-    => GetStereotypesAsync().GetAwaiter().GetResult();
 }
diff --git a/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IStereotypesProvider.cs b/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/IStereotypesProvider.cs
@@ -1,4 +1,3 @@
-using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
 
@@ -7,8 +6,4 @@ namespace OrchardCore.ContentTypes.Services;
 public interface IStereotypesProvider
 {
     Task<IEnumerable<StereotypeDescription>> GetStereotypesAsync();
-
-    [Obsolete($"Instead, utilize the {nameof(GetStereotypesAsync)} method. This current method is slated for removal in upcoming releases.")]
-    IEnumerable<StereotypeDescription> GetStereotypes()
-    => GetStereotypesAsync().GetAwaiter().GetResult();
 }
diff --git a/src/OrchardCore.Modules/OrchardCore.Contents/ViewModels/SelectContentTypesViewModel.cs b/src/OrchardCore.Modules/OrchardCore.Contents/ViewModels/SelectContentTypesViewModel.cs
@@ -18,10 +18,6 @@ public class ContentTypeSelection
         public bool IsSelected { get; set; }
         public ContentTypeDefinition ContentTypeDefinition { get; set; }
 
-        [Obsolete($"Instead, utilize the {nameof(BuildAsync)} method. This current method is slated for removal in upcoming releases.")]
-        public static ContentTypeSelection[] Build(IContentDefinitionManager contentDefinitionManager, IEnumerable<string> selectedContentTypes)
-            => BuildAsync(contentDefinitionManager, selectedContentTypes).GetAwaiter().GetResult();
-
         public static async Task<ContentTypeSelection[]> BuildAsync(IContentDefinitionManager contentDefinitionManager, IEnumerable<string> selectedContentTypes)
         {
             var contentTypes = (await contentDefinitionManager.ListTypeDefinitionsAsync())

diff --git a/src/OrchardCore.Modules/OrchardCore.Contents/Workflows/Drivers/ContentEventDisplayDriver.cs b/src/OrchardCore.Modules/OrchardCore.Contents/Workflows/Drivers/ContentEventDisplayDriver.cs
@@ -1,4 +1,3 @@
-using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
@@ -31,7 +30,7 @@ public async override Task<IDisplayResult> UpdateAsync(TActivity model, IUpdateM
             await updater.TryUpdateModelAsync(viewModel, Prefix, x => x.SelectedContentTypeNames);
 
             model.ContentTypeFilter = (await FilterContentTypesQueryAsync(viewModel.SelectedContentTypeNames)).ToList();
-            
+
             return Edit(model);
         }
 
@@ -55,13 +54,6 @@ public override IDisplayResult Display(TActivity activity)
             );
         }
 
-        /// <summary>
-        /// Filters out any content type that doesn't exist.
-        /// </summary>
-        [Obsolete($"Instead, utilize the {nameof(FilterContentTypesQueryAsync)} method. This current method is slated for removal in upcoming releases.")]
-        protected IEnumerable<string> FilterContentTypesQuery(IEnumerable<string> contentTypeNames)
-            => FilterContentTypesQueryAsync(contentTypeNames).GetAwaiter().GetResult();
-
         protected async Task<IEnumerable<string>> FilterContentTypesQueryAsync(IEnumerable<string> contentTypeNames)
         {
             var contentTypeDefinitions = (await ContentDefinitionManager.ListTypeDefinitionsAsync()).ToDictionary(x => x.Name);

diff --git a/src/OrchardCore.Modules/OrchardCore.Forms/FormShapeTableProvider.cs b/src/OrchardCore.Modules/OrchardCore.Forms/FormShapeTableProvider.cs
@@ -1,3 +1,4 @@
+using System.Threading.Tasks;
 using OrchardCore.ContentManagement;
 using OrchardCore.DisplayManagement;
 using OrchardCore.DisplayManagement.Descriptors;
@@ -8,17 +9,19 @@ namespace OrchardCore.Forms;
 
 public class FormShapeTableProvider : IShapeTableProvider
 {
-    public void Discover(ShapeTableBuilder builder)
+    public ValueTask DiscoverAsync(ShapeTableBuilder builder)
     {
         builder.Describe("Widget__Form")
             .OnDisplaying(context =>
             {
-                if (context.Shape.TryGetProperty<ContentItem>("ContentItem", out var contentItem) 
-                    && contentItem.TryGet<FormElementPart>(out var elementPart) 
+                if (context.Shape.TryGetProperty<ContentItem>("ContentItem", out var contentItem)
+                    && contentItem.TryGet<FormElementPart>(out var elementPart)
                     && !string.IsNullOrEmpty(elementPart.Id))
                 {
                     context.Shape.Metadata.Alternates.Add($"Widget__{contentItem.ContentType}_{elementPart.Id.EncodeAlternateElement()}");
                 }
             });
+
+        return ValueTask.CompletedTask;
     }
 }
diff --git a/src/OrchardCore.Modules/OrchardCore.Layers/GraphQL/LayerQueryObjectType.cs b/src/OrchardCore.Modules/OrchardCore.Layers/GraphQL/LayerQueryObjectType.cs
@@ -23,9 +23,6 @@ public LayerQueryObjectType()
             Name = "Layer";
 
             Field(layer => layer.Name).Description("The name of the layer.");
-#pragma warning disable 0618
-            Field(layer => layer.Rule).Description("Deprecated. The rule that activates the layer.");
-#pragma warning restore 0618
             Field<ListGraphType<StringGraphType>, IEnumerable<Condition>>("layerrule")
                 .Description("The rule that activates the layer.")
                 .Resolve(ctx => ctx.Source.LayerRule.Conditions);

diff --git a/src/OrchardCore.Modules/OrchardCore.Layers/Migrations.cs b/src/OrchardCore.Modules/OrchardCore.Layers/Migrations.cs
@@ -60,11 +60,7 @@ public async Task<int> UpdateFrom2Async()
                 layer.LayerRule = new Rule();
                 _conditionIdGenerator.GenerateUniqueId(layer.LayerRule);
 
-#pragma warning disable 0618
-                _ruleMigrator.Migrate(layer.Rule, layer.LayerRule);
-
-                layer.Rule = string.Empty;
-#pragma warning restore 0618
+                _ruleMigrator.Migrate(layer.LayerRule.Name, layer.LayerRule);
             }
 
             await _layerService.UpdateAsync(layers);

diff --git a/src/OrchardCore.Modules/OrchardCore.Layers/Models/Layer.cs b/src/OrchardCore.Modules/OrchardCore.Layers/Models/Layer.cs
@@ -1,4 +1,3 @@
-using System;
 using OrchardCore.Rules;
 
 namespace OrchardCore.Layers.Models
@@ -7,12 +6,6 @@ public class Layer
     {
         public string Name { get; set; }
 
-        /// <summary>
-        /// This property is obsolete and layer rules should be used instead.
-        /// It can be removed in a future version.
-        /// </summary>
-        [Obsolete("The rule property is obsolete and LayerRule should be used instead.")]
-        public string Rule { get; set; }
         public string Description { get; set; }
 
         public Rule LayerRule { get; set; }

diff --git a/src/OrchardCore.Modules/OrchardCore.Liquid/Services/SlugService.cs b/src/OrchardCore.Modules/OrchardCore.Liquid/Services/SlugService.cs
diff --git a/src/OrchardCore.Modules/OrchardCore.Menu/Drivers/ContentMenuItemPartDisplayDriver.cs b/src/OrchardCore.Modules/OrchardCore.Menu/Drivers/ContentMenuItemPartDisplayDriver.cs
@@ -42,10 +42,6 @@ public override async Task<IDisplayResult> UpdateAsync(ContentMenuItemPart part,
             await updater.TryUpdateModelAsync(model, Prefix);
 
             part.ContentItem.DisplayText = model.Name;
-            // This code can be removed in a later release.
-#pragma warning disable 0618
-            part.Name = model.Name;
-#pragma warning restore 0618
 
             return Edit(part);
         }

diff --git a/src/OrchardCore.Modules/OrchardCore.Menu/Drivers/LinkMenuItemPartDisplayDriver.cs b/src/OrchardCore.Modules/OrchardCore.Menu/Drivers/LinkMenuItemPartDisplayDriver.cs
@@ -72,11 +72,6 @@ public override async Task<IDisplayResult> UpdateAsync(LinkMenuItemPart part, IU
             part.Url = model.Url;
             part.ContentItem.DisplayText = model.Name;
 
-            // This code can be removed in a later release.
-#pragma warning disable 0618
-            part.Name = model.Name;
-#pragma warning restore 0618
-
             var urlToValidate = part.Url;
 
             if (!string.IsNullOrEmpty(urlToValidate))

diff --git a/src/OrchardCore.Modules/OrchardCore.Menu/GraphQL/LinkMenuItemQueryObjectType.cs b/src/OrchardCore.Modules/OrchardCore.Menu/GraphQL/LinkMenuItemQueryObjectType.cs
@@ -9,10 +9,6 @@ public LinkMenuItemQueryObjectType()
         {
             Name = "LinkMenuItemPart";
 
-            // This code can be removed in a later release.
-#pragma warning disable 0618
-            Field(x => x.Name, nullable: true).Description("Deprecated. Use displayText.");
-#pragma warning restore 0618
             Field(x => x.Url, nullable: true);
         }
     }

diff --git a/src/OrchardCore.Modules/OrchardCore.Menu/Migrations.cs b/src/OrchardCore.Modules/OrchardCore.Menu/Migrations.cs
@@ -70,23 +70,6 @@ private static void MigrateMenuItems(List<ContentItem> menuItems)
         {
             foreach (var menuItem in menuItems)
             {
-                var linkMenuItemPart = menuItem.As<LinkMenuItemPart>();
-                if (linkMenuItemPart != null)
-                {
-                    // This code can be removed in a later release.
-#pragma warning disable 0618
-                    menuItem.DisplayText = linkMenuItemPart.Name;
-#pragma warning restore 0618
-                }
-
-                var contentMenuItemPart = menuItem.As<ContentMenuItemPart>();
-                if (contentMenuItemPart != null)
-                {
-#pragma warning disable 0618
-                    menuItem.DisplayText = contentMenuItemPart.Name;
-#pragma warning restore 0618
-                }
-
                 var menuItemsListPart = menuItem.As<MenuItemsListPart>();
                 if (menuItemsListPart != null)
                 {