Merge pull request #6010 from abpframework/dev-crudpage-fixes

Dev crudpage fixes

framework/src/Volo.Abp.BlazoriseUI/AbpCrudPageBase.cs

@@ -281,9 +281,7 @@ namespace Volo.Abp.BlazoriseUI
 
         protected virtual async Task OpenCreateModalAsync()
         {
-            await OnOpeningCreateModalAsync();
+            CreateValidationsRef?.ClearAll();
-
-            CreateValidationsRef.ClearAll();
 
             await CheckCreatePolicyAsync();
 
@@ -296,11 +294,6 @@ namespace Volo.Abp.BlazoriseUI
             CreateModal.Show();
         }
 
-        protected virtual Task OnOpeningCreateModalAsync()
-        {
-            return Task.CompletedTask;
-        }
-
         protected virtual Task CloseCreateModalAsync()
         {
             CreateModal.Hide();
@@ -309,9 +302,7 @@ namespace Volo.Abp.BlazoriseUI
 
         protected virtual async Task OpenEditModalAsync(TKey id)
         {
-            await OnOpeningEditModalAsync(id);
+            EditValidationsRef?.ClearAll();
-
-            EditValidationsRef.ClearAll();
 
             await CheckUpdatePolicyAsync();
 
@@ -325,11 +316,6 @@ namespace Volo.Abp.BlazoriseUI
             EditModal.Show();
         }
 
-        protected virtual Task OnOpeningEditModalAsync(TKey id)
-        {
-            return Task.CompletedTask;
-        }
-
         protected virtual TUpdateViewModel MapToEditingEntity(TGetOutputDto entityDto)
         {
             return ObjectMapper.Map<TGetOutputDto, TUpdateViewModel>(entityDto);
@@ -363,7 +349,7 @@ namespace Volo.Abp.BlazoriseUI
 
         protected virtual async Task CreateEntityAsync()
         {
-            if (CreateValidationsRef.ValidateAll())
+            if (CreateValidationsRef?.ValidateAll() ?? false)
             {
                 await OnCreatingEntityAsync();
 
@@ -390,7 +376,7 @@ namespace Volo.Abp.BlazoriseUI
 
         protected virtual async Task UpdateEntityAsync()
         {
-            if (EditValidationsRef.ValidateAll())
+            if (EditValidationsRef?.ValidateAll() ?? false)
             {
                 await OnUpdatingEntityAsync();
 

modules/identity/src/Volo.Abp.Identity.Blazor/Pages/Identity/UserManagement.razor.cs

@@ -61,7 +61,7 @@ namespace Volo.Abp.Identity.Blazor.Pages.Identity
                                       HasManagePermissionsPermission;
         }
 
-        protected override async Task OnOpeningCreateModalAsync()
+        protected override Task OpenCreateModalAsync()
         {
             CreateModalSelectedTab = DefaultSelectedTab;
 
@@ -71,7 +71,7 @@ namespace Volo.Abp.Identity.Blazor.Pages.Identity
                 IsAssigned = x.IsDefault
             }).ToArray();
 
-            await base.OnOpeningCreateModalAsync();
+            return base.OpenCreateModalAsync();
         }
 
         protected override Task OnCreatingEntityAsync()
@@ -94,7 +94,7 @@ namespace Volo.Abp.Identity.Blazor.Pages.Identity
                 IsAssigned = userRoleNames.Contains(x.Name)
             }).ToArray();
 
-            await base.OnOpeningEditModalAsync(id);
+            await base.OpenEditModalAsync(id);
         }
 
         protected override Task OnUpdatingEntityAsync()