Remove TenantSwitchViewComponent. Refactor tenant selection

framework/src/Volo.Abp.AspNetCore.MultiTenancy/Volo/Abp/AspNetCore/MultiTenancy/DomainTenantResolveContributor.cs

@@ -9,23 +9,29 @@ namespace Volo.Abp.AspNetCore.MultiTenancy
 
     public class DomainTenantResolveContributor : HttpTenantResolveContributorBase
     {
+        private static readonly string[] ProtocolPrefixes = { "http://", "https://" };
+
         private readonly string _domainFormat;
 
         public DomainTenantResolveContributor(string domainFormat)
         {
-            _domainFormat = domainFormat.RemovePreFix("http://", "https://");
+            _domainFormat = domainFormat.RemovePreFix(ProtocolPrefixes);
         }
 
-        protected override string GetTenantIdOrNameFromHttpContextOrNull(ITenantResolveContext context, HttpContext httpContext)
+        protected override string GetTenantIdOrNameFromHttpContextOrNull(
+            ITenantResolveContext context, 
+            HttpContext httpContext)
         {
             if (httpContext.Request?.Host == null)
             {
                 return null;
             }
 
-            var hostName = httpContext.Request.Host.Host.RemovePreFix("http://", "https://");
+            var hostName = httpContext.Request.Host.Host.RemovePreFix(ProtocolPrefixes);
             var extractResult = FormattedStringValueExtracter.Extract(hostName, _domainFormat, ignoreCase: true);
 
+            context.Handled = true;
+
             if (!extractResult.IsMatch)
             {
                 return null;

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Pages/Abp/MultiTenancy/tenant-switch.js

@@ -3,7 +3,7 @@
     var tenantSwitchModal = new abp.ModalManager(abp.appPath + 'Abp/MultiTenancy/TenantSwitchModal');
 
     $(function() {
-        $('#TenantSwitchToolbarLink').click(function(e) {
+        $('#AbpTenantSwitchLink').click(function(e) {
             e.preventDefault();
             tenantSwitchModal.open();
         });

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy.csproj

@@ -18,11 +18,9 @@
   <ItemGroup>
     <EmbeddedResource Include="Pages\**\*.cshtml" />
     <EmbeddedResource Include="Pages\**\*.js" />
-    <EmbeddedResource Include="Volo\Abp\AspNetCore\Mvc\UI\MultiTenancy\Components\**\*.cshtml" />
     <EmbeddedResource Include="Volo\Abp\AspNetCore\Mvc\UI\MultiTenancy\Localization\*.json" />
     <Content Remove="Pages\**\*.cshtml" />
     <Content Remove="Pages\**\*.js" />
-    <Content Remove="Volo\Abp\AspNetCore\Mvc\UI\MultiTenancy\Components\**\*.cshtml" />
     <Content Remove="Volo\Abp\AspNetCore\Mvc\UI\MultiTenancy\Localization\*.json" />
   </ItemGroup>
 

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/AbpAspNetCoreMvcUiMultiTenancyModule.cs

@@ -1,11 +1,9 @@
-using Microsoft.Extensions.DependencyInjection;
+using Volo.Abp.AspNetCore.MultiTenancy;
-using Volo.Abp.AspNetCore.MultiTenancy;
 using Volo.Abp.AspNetCore.Mvc.Localization;
 using Volo.Abp.AspNetCore.Mvc.UI.Bundling;
 using Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy.Localization;
 using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared;
 using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared.Bundling;
-using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared.Toolbars;
 using Volo.Abp.Localization;
 using Volo.Abp.Modularity;
 using Volo.Abp.VirtualFileSystem;
@@ -36,11 +34,6 @@ namespace Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy
                 options.FileSets.AddEmbedded<AbpAspNetCoreMvcUiMultiTenancyModule>();
             });
 
-            Configure<ToolbarOptions>(options =>
-            {
-                options.Contributors.Add(new MultiTenancyToolbarContributor());
-            });
-
             Configure<AbpLocalizationOptions>(options =>
             {
                 options.Resources

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Components/TenantSwitch/Default.cshtml

@@ -1,17 +0,0 @@
-@using Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy.Components.TenantSwitch
-@model TenantSwitchViewComponent.TenantSwitchViewModel
-@if (!Model.CurrentUser.IsAuthenticated)
-{
-    <li class="nav-item">
-        <a abp-button="Link" id="TenantSwitchToolbarLink" href="#">
-            @if (Model.Tenant == null)
-            {
-                <text>@@host</text>
-            }
-            else
-            {
-                <text>@@@Model.Tenant.Name</text>
-            }
-        </a>
-    </li>
-}

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Components/TenantSwitch/TenantSwitchViewComponent.cs

@@ -1,51 +0,0 @@
-using System.Threading.Tasks;
-using Microsoft.AspNetCore.Mvc;
-using Volo.Abp.MultiTenancy;
-using Volo.Abp.Users;
-
-namespace Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy.Components.TenantSwitch
-{
-    public class TenantSwitchViewComponent : AbpViewComponent
-    {
-        /// <summary>
-        /// -1,000,000
-        /// </summary>
-        public const int Order = -1_000_000;
-
-        protected ITenantStore TenantStore { get; }
-        protected ICurrentTenant CurrentTenant { get; }
-        protected ICurrentUser CurrentUser { get; }
-
-        public TenantSwitchViewComponent(
-            ITenantStore tenantStore, 
-            ICurrentTenant currentTenant,
-            ICurrentUser currentUser)
-        {
-            TenantStore = tenantStore;
-            CurrentTenant = currentTenant;
-            CurrentUser = currentUser;
-        }
-
-        public async Task<IViewComponentResult> InvokeAsync()
-        {
-            var model = new TenantSwitchViewModel
-            {
-                CurrentUser = CurrentUser
-            };
-
-            if (CurrentTenant.Id.HasValue)
-            {
-                model.Tenant = await TenantStore.FindAsync(CurrentTenant.GetId());
-            }
-
-            return View("~/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Components/TenantSwitch/Default.cshtml", model);
-        }
-
-        public class TenantSwitchViewModel
-        {
-            public TenantConfiguration Tenant { get; set; }
-
-            public ICurrentUser CurrentUser { get; set; }
-        }
-    }
-}

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Components/_ViewImports.cshtml

@@ -1,3 +0,0 @@
-@addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers
-@addTagHelper *, Volo.Abp.AspNetCore.Mvc.UI.Bootstrap
-@addTagHelper *, Volo.Abp.AspNetCore.Mvc.UI.Bundling

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Localization/en.json

@@ -1,9 +1,11 @@
-{
+{
   "culture": "en",
   "texts": {
     "GivenTenantIsNotAvailable": "Given tenant is not available: {0}",
-    "SwitchTenant": "Switch tenant",
+    "Tenant": "Tenant",
+    "Switch": "Switch",
     "Name": "Name",
-    "SwitchTenantHint": "Leave the name field blank to switch to the host side."
+    "SwitchTenantHint": "Leave the name field blank to switch to the host side.",
+    "NotSelected": "Seçili değil"
   }
 }

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/Localization/tr.json

@@ -2,8 +2,10 @@
   "culture": "tr",
   "texts": {
     "GivenTenantIsNotAvailable": "İstenilen müşteri bulunamadı: {0}",
-    "SwitchTenant": "Müşteri değiştir",
+    "Tenant": "Müşteri",
+    "Switch": "Değiştir",
     "Name": "İsim",
-    "SwitchTenantHint": "Host tarafına geçmek için isim alanını boş bırakın."
+    "SwitchTenantHint": "Host tarafına geçmek için isim alanını boş bırakın.",
+    "NotSelected": "Not selected"
   }
 }

framework/src/Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy/Volo/Abp/AspNetCore/Mvc/UI/MultiTenancy/MultiTenancyToolbarContributor.cs

@@ -1,21 +0,0 @@
-using System.Threading.Tasks;
-using Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy.Components.TenantSwitch;
-using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared.Toolbars;
-
-namespace Volo.Abp.AspNetCore.Mvc.UI.MultiTenancy
-{
-    public class MultiTenancyToolbarContributor : IToolbarContributor
-    {
-        public Task ConfigureToolbarAsync(IToolbarConfigurationContext context)
-        {
-            if (context.Toolbar.Name != StandardToolbars.Main)
-            {
-                return Task.CompletedTask;
-            }
-
-            context.Toolbar.Items.Add(new ToolbarItem(typeof(TenantSwitchViewComponent), TenantSwitchViewComponent.Order));
-
-            return Task.CompletedTask;
-        }
-    }
-}

framework/src/Volo.Abp.MultiTenancy/Volo/Abp/MultiTenancy/CurrentClaimsPrincipalTenantResolveContributor.cs

@@ -14,8 +14,11 @@ namespace Volo.Abp.MultiTenancy
                 return;
             }
 
-            context.TenantIdOrName = principal.Claims.FirstOrDefault(c => c.Type == AbpClaimTypes.TenantId)?.Value;
             context.Handled = true;
+            context.TenantIdOrName = principal
+                .Claims
+                .FirstOrDefault(c => c.Type == AbpClaimTypes.TenantId)
+                ?.Value;
         }
     }
 }