Fix the error exception message (#14889)

OrchardCMS · Dec 12, 2023 · 8f0c371 · 8f0c371
1 parent dc7cc46
commit 8f0c371
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/ContentDefinitionService.cs b/src/OrchardCore.Modules/OrchardCore.ContentTypes/Services/ContentDefinitionService.cs
@@ -97,7 +97,7 @@ public async Task<ContentTypeDefinition> AddTypeAsync(string name, string displa
         {
             if (string.IsNullOrWhiteSpace(displayName))
             {
-                throw new ArgumentException("The 'displayName' can't be null or empty.", nameof(displayName));
+                throw new ArgumentException($"The '{nameof(displayName)}' can't be null or empty.", nameof(displayName));
             }
 
             if (string.IsNullOrWhiteSpace(name))
@@ -322,7 +322,7 @@ public async Task AddFieldToPartAsync(string fieldName, string displayName, stri
         {
             if (string.IsNullOrEmpty(fieldName))
             {
-                throw new ArgumentException("The '{fieldName}' can't be null or empty.", nameof(fieldName));
+                throw new ArgumentException($"The '{nameof(fieldName)}' can't be null or empty.", nameof(fieldName));
             }
 
             var partDefinition = await _contentDefinitionManager.LoadPartDefinitionAsync(partName);
@@ -398,7 +398,7 @@ public Task AlterTypePartsOrderAsync(ContentTypeDefinition typeDefinition, strin
                 for (var i = 0; i < partNames.Length; i++)
                 {
                     var partDefinition = typeDefinition.Parts?.FirstOrDefault(x => x.Name == partNames[i]);
-                    
+
                     if (partDefinition == null)
                     {
                         continue;

diff --git a/src/OrchardCore.Modules/OrchardCore.Roles/ViewModels/PermissionGroupKey.cs b/src/OrchardCore.Modules/OrchardCore.Roles/ViewModels/PermissionGroupKey.cs
@@ -14,7 +14,7 @@ public PermissionGroupKey(string key)
     {
         if (string.IsNullOrWhiteSpace(key))
         {
-            throw new ArgumentException("The 'key' cannot be null or empty", nameof(key));
+            throw new ArgumentException($"The '{nameof(key)}' cannot be null or empty", nameof(key));
         }
 
         Key = key;

diff --git a/src/OrchardCore/OrchardCore.Abstractions/Extensions/Utility/DependencyOrdering.cs b/src/OrchardCore/OrchardCore.Abstractions/Extensions/Utility/DependencyOrdering.cs
@@ -77,7 +77,7 @@ private static void MoveUp<T>(List<T> list, int index, int lowerIndex)
         {
             if (index < lowerIndex)
             {
-                throw new ArgumentException("Should be higher or equal to 'lowerIndex'.", nameof(index));
+                throw new ArgumentException($"Should be higher or equal to '{nameof(lowerIndex)}'.", nameof(index));
             }
 
             if (index != lowerIndex)

diff --git a/...ardCore.Abstractions/Modules/Overrides/Configuration/TenantJsonConfigurationExtensions.cs b/...ardCore.Abstractions/Modules/Overrides/Configuration/TenantJsonConfigurationExtensions.cs
@@ -72,7 +72,7 @@ public static IConfigurationBuilder AddTenantJsonFile(this IConfigurationBuilder
             if (string.IsNullOrEmpty(path))
             {
                 // throw new ArgumentException(SR.Error_InvalidFilePath, nameof(path));
-                throw new ArgumentException($"File path must be a non-empty string.", nameof(path));
+                throw new ArgumentException("File path must be a non-empty string.", nameof(path));
             }
 
             return builder.AddTenantJsonFile(s =>

diff --git a/src/OrchardCore/OrchardCore.Mvc.Core/Utilities/ControllerTypeExtensions.cs b/src/OrchardCore/OrchardCore.Mvc.Core/Utilities/ControllerTypeExtensions.cs
@@ -9,7 +9,7 @@ public static string ControllerName(this Type controllerType)
         {
             if (!typeof(Controller).IsAssignableFrom(controllerType))
             {
-                throw new ArgumentException("The specified type must inherit from " + nameof(Controller), nameof(controllerType));
+                throw new ArgumentException($"The specified type must inherit from '{nameof(Controller)}'", nameof(controllerType));
             }
 
             return controllerType.Name.EndsWith(nameof(Controller), StringComparison.OrdinalIgnoreCase)