Force refresh service plan in combo box loadItems to avoid http request in event listener

Flanker32 · Flanker32 · commit a2cbe756e03d · 2021-08-04T11:01:49.000+08:00
diff --git a/PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azure/toolkit/intellij/appservice/serviceplan/ServicePlanComboBox.java b/PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azure/toolkit/intellij/appservice/serviceplan/ServicePlanComboBox.java
@@ -55,8 +55,6 @@ public void setSubscription(Subscription subscription) {
             this.clear();
             return;
         }
-        // force refresh service plan when switch subscription
-        Azure.az(AzureAppService.class).subscription(subscription.getId()).appServicePlans(true);
         this.refreshItems();
     }
 
@@ -114,7 +112,7 @@ protected List<? extends AppServicePlanEntity> loadItems() throws Exception {
                     .collect(Collectors.toList()));
             }
             final List<AppServicePlanEntity> remotePlans = Azure.az(AzureAppService.class)
-                .subscription(subscription.getId()).appServicePlans().stream().map(IAppServicePlan::entity)
+                .subscription(subscription.getId()).appServicePlans(true).stream().map(IAppServicePlan::entity)
                 .collect(Collectors.toList());
             plans.addAll(remotePlans);
             Stream<AppServicePlanEntity> stream = plans.stream();

Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,6 @@ public void setSubscription(Subscription subscription) {`
`55`	`55`	`this.clear();`
`56`	`56`	`return;`
`57`	`57`	`}`
`58`		`- // force refresh service plan when switch subscription`
`59`		`- Azure.az(AzureAppService.class).subscription(subscription.getId()).appServicePlans(true);`
`60`	`58`	`this.refreshItems();`
`61`	`59`	`}`
`62`	`60`
`@@ -114,7 +112,7 @@ protected List<? extends AppServicePlanEntity> loadItems() throws Exception {`
`114`	`112`	`.collect(Collectors.toList()));`
`115`	`113`	`}`
`116`	`114`	`final List<AppServicePlanEntity> remotePlans = Azure.az(AzureAppService.class)`
`117`		`- .subscription(subscription.getId()).appServicePlans().stream().map(IAppServicePlan::entity)`
	`115`	`+ .subscription(subscription.getId()).appServicePlans(true).stream().map(IAppServicePlan::entity)`
`118`	`116`	`.collect(Collectors.toList());`
`119`	`117`	`plans.addAll(remotePlans);`
`120`	`118`	`Stream<AppServicePlanEntity> stream = plans.stream();`