Disable linux target for NetFramework apps

JetBrains · Aug 12, 2024 · 4a3e630 · 4a3e630
1 parent f08f9ea
commit 4a3e630
Show file tree

Hide file tree

Showing 9 changed files with 48 additions and 14 deletions.
diff --git a/...lin/com/microsoft/azure/toolkit/intellij/legacy/appservice/AppServiceInfoAdvancedPanel.kt b/...lin/com/microsoft/azure/toolkit/intellij/legacy/appservice/AppServiceInfoAdvancedPanel.kt
@@ -37,6 +37,7 @@ import javax.swing.JPanel
 
 open class AppServiceInfoAdvancedPanel<T>(
     private val projectName: String,
+    targetProjectOnNetFramework: Boolean = false,
     private val defaultConfigSupplier: Supplier<T>
 ) : JPanel(), Disposable, AzureFormPanel<T> where T : AppServiceConfig {
     companion object {
@@ -104,8 +105,9 @@ open class AppServiceInfoAdvancedPanel<T>(
                 operatingSystemGroup = buttonsGroup {
                     row("Operating System:") {
                         windowsRadioButton = radioButton("Windows", OperatingSystem.WINDOWS)
-                        linuxRadioButton = radioButton("Linux", OperatingSystem.LINUX)
                         windowsRadioButton.component.addItemListener { onOperatingSystemChanged(it) }
+                        linuxRadioButton = radioButton("Linux", OperatingSystem.LINUX)
+                            .enabled(!targetProjectOnNetFramework)
                         dockerRadioButton = radioButton("Docker", OperatingSystem.DOCKER)
                             .visible(false)
                     }

diff --git a/...kotlin/com/microsoft/azure/toolkit/intellij/legacy/appservice/AppServiceInfoBasicPanel.kt b/...kotlin/com/microsoft/azure/toolkit/intellij/legacy/appservice/AppServiceInfoBasicPanel.kt
@@ -23,6 +23,7 @@ import java.util.function.Supplier
 import javax.swing.JPanel
 
 class AppServiceInfoBasicPanel<T>(
+    targetProjectOnNetFramework: Boolean = false,
     private val defaultConfigSupplier: Supplier<T>
 ) : JPanel(), Disposable, AzureFormPanel<T> where T : AppServiceConfig {
 
@@ -54,6 +55,7 @@ class AppServiceInfoBasicPanel<T>(
                     row("Operating System:") {
                         windowsRadioButton = radioButton("Windows", OperatingSystem.WINDOWS)
                         linuxRadioButton = radioButton("Linux", OperatingSystem.LINUX)
+                            .enabled(!targetProjectOnNetFramework)
                         dockerRadioButton = radioButton("Docker", OperatingSystem.DOCKER)
                             .visible(false)
                     }

diff --git a/...com/microsoft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppComboBox.kt b/...com/microsoft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppComboBox.kt
@@ -22,6 +22,8 @@ import java.util.function.Supplier
 import java.util.stream.Collectors
 
 class FunctionAppComboBox(project: Project) : AppServiceComboBox<FunctionAppConfig>(project) {
+    var targetProjectOnNetFramework: Boolean = false
+
     override fun refreshItems() {
         Azure.az(AzureFunctions::class.java).refresh()
         super.refreshItems()
@@ -69,7 +71,7 @@ class FunctionAppComboBox(project: Project) : AppServiceComboBox<FunctionAppConf
     }
 
     override fun createResource() {
-        val dialog = FunctionAppCreationDialog(project)
+        val dialog = FunctionAppCreationDialog(project, targetProjectOnNetFramework)
         Disposer.register(this, dialog)
         dialog.data = FunctionAppConfigProducer.getInstance().generateDefaultConfig()
         val actionId: Action.Id<FunctionAppConfig> = Action.Id.of("user/function.create_app.app")

diff --git a/...crosoft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppCreationDialog.kt b/...crosoft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppCreationDialog.kt
@@ -20,7 +20,10 @@ import com.microsoft.azure.toolkit.lib.auth.IAccountActions
 import com.microsoft.azure.toolkit.lib.common.exception.AzureToolkitRuntimeException
 import javax.swing.JPanel
 
-class FunctionAppCreationDialog(project: Project) : ConfigDialog<FunctionAppConfig>(project), Disposable {
+class FunctionAppCreationDialog(
+    project: Project,
+    targetProjectOnNetFramework: Boolean
+) : ConfigDialog<FunctionAppConfig>(project), Disposable {
     private val basicPanel: AppServiceInfoBasicPanel<FunctionAppConfig>
     private val advancedPanel: FunctionAppInfoAdvancedPanel
     private val panel: JPanel
@@ -36,12 +39,12 @@ class FunctionAppCreationDialog(project: Project) : ConfigDialog<FunctionAppConf
         }
 
         val projectName = removeInvalidCharacters(project.name)
-        basicPanel = AppServiceInfoBasicPanel {
+        basicPanel = AppServiceInfoBasicPanel(targetProjectOnNetFramework) {
             FunctionAppConfigProducer.getInstance().generateDefaultConfig()
         }
         Disposer.register(this, basicPanel)
 
-        advancedPanel = FunctionAppInfoAdvancedPanel(projectName) {
+        advancedPanel = FunctionAppInfoAdvancedPanel(projectName, targetProjectOnNetFramework) {
             FunctionAppConfig()
         }
         Disposer.register(this, advancedPanel)

diff --git a/...soft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppInfoAdvancedPanel.kt b/...soft/azure/toolkit/intellij/legacy/function/runner/deploy/FunctionAppInfoAdvancedPanel.kt
@@ -19,8 +19,9 @@ import java.util.function.Supplier
 
 class FunctionAppInfoAdvancedPanel(
     projectName: String,
+    targetProjectOnNetFramework: Boolean,
     defaultConfigSupplier: Supplier<FunctionAppConfig>
-) : AppServiceInfoAdvancedPanel<FunctionAppConfig>(projectName, defaultConfigSupplier) {
+) : AppServiceInfoAdvancedPanel<FunctionAppConfig>(projectName, targetProjectOnNetFramework, defaultConfigSupplier) {
 
     private lateinit var storageAccountComboBox: Cell<StorageAccountComboBox>
 

diff --git a/.../azure/toolkit/intellij/legacy/function/runner/deploy/FunctionDeploymentSettingsEditor.kt b/.../azure/toolkit/intellij/legacy/function/runner/deploy/FunctionDeploymentSettingsEditor.kt
@@ -11,6 +11,7 @@ import com.intellij.openapi.ui.LabeledComponent
 import com.intellij.openapi.util.Disposer
 import com.intellij.ui.components.JBCheckBox
 import com.intellij.ui.dsl.builder.*
+import com.jetbrains.rider.model.PublishableProjectModel
 import com.jetbrains.rider.model.publishableProjectsModel
 import com.jetbrains.rider.projectView.solution
 import com.jetbrains.rider.run.configurations.publishing.PublishRuntimeSettingsCoreHelper
@@ -70,8 +71,10 @@ class FunctionDeploymentSettingsEditor(private val project: Project) :
             }
         }
 
-        dotnetProjectComboBox.component.reloadItems()
-
+        dotnetProjectComboBox.component.apply {
+            reloadItems()
+            addValueChangedListener(::onSelectProject)
+        }
         functionAppComboBox.component.apply {
             addValueChangedListener(::onSelectFunctionApp)
         }
@@ -80,6 +83,13 @@ class FunctionDeploymentSettingsEditor(private val project: Project) :
         }
     }
 
+    private fun onSelectProject(value: PublishableProjectModel) {
+        if (isLoading) return
+
+        val onNetFramework = !value.isDotNetCore || value.projectOutputs.all { it.tfmInMsbuildFormat == "net48" }
+        functionAppComboBox.component.targetProjectOnNetFramework = onNetFramework
+    }
+
     private fun onSelectFunctionApp(value: FunctionAppConfig?) {
         if (isLoading) return
 

diff --git a/...kotlin/com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppComboBox.kt b/...kotlin/com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppComboBox.kt
@@ -21,6 +21,8 @@ import java.util.function.Supplier
 import java.util.stream.Collectors
 
 open class WebAppComboBox(project: Project) : AppServiceComboBox<AppServiceConfig>(project) {
+    var targetProjectOnNetFramework: Boolean = false
+
     override fun refreshItems() {
         Azure.az(AzureWebApp::class.java).refresh()
         super.refreshItems()
@@ -68,7 +70,7 @@ open class WebAppComboBox(project: Project) : AppServiceComboBox<AppServiceConfi
     }
 
     override fun createResource() {
-        val dialog = WebAppCreationDialog(project)
+        val dialog = WebAppCreationDialog(project, targetProjectOnNetFramework)
         Disposer.register(this, dialog)
         val actionId: Action.Id<AppServiceConfig> = Action.Id.of("user/webapp.create_app.app")
         dialog.setOkAction(

diff --git a/.../com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppCreationDialog.kt b/.../com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppCreationDialog.kt
@@ -22,7 +22,10 @@ import com.microsoft.azure.toolkit.lib.auth.IAccountActions
 import com.microsoft.azure.toolkit.lib.common.exception.AzureToolkitRuntimeException
 import javax.swing.JPanel
 
-class WebAppCreationDialog(project: Project) : ConfigDialog<AppServiceConfig>(project), Disposable {
+class WebAppCreationDialog(
+    project: Project,
+    targetProjectOnNetFramework: Boolean
+) : ConfigDialog<AppServiceConfig>(project), Disposable {
     private val basicPanel: AppServiceInfoBasicPanel<AppServiceConfig>
     private val advancedPanel: AppServiceInfoAdvancedPanel<AppServiceConfig>
     private val panel: JPanel
@@ -38,12 +41,12 @@ class WebAppCreationDialog(project: Project) : ConfigDialog<AppServiceConfig>(pr
         }
 
         val projectName = removeInvalidCharacters(project.name)
-        basicPanel = AppServiceInfoBasicPanel {
+        basicPanel = AppServiceInfoBasicPanel(targetProjectOnNetFramework) {
             WebAppConfigProducer.getInstance().generateDefaultConfig()
         }
         Disposer.register(this, basicPanel)
 
-        advancedPanel = AppServiceInfoAdvancedPanel(projectName) {
+        advancedPanel = AppServiceInfoAdvancedPanel(projectName, targetProjectOnNetFramework) {
             AppServiceConfig()
         }
         Disposer.register(this, advancedPanel)

diff --git a/...n/com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppSettingEditor.kt b/...n/com/microsoft/azure/toolkit/intellij/legacy/webapp/runner/webApp/WebAppSettingEditor.kt
@@ -12,6 +12,7 @@ import com.intellij.openapi.util.Disposer
 import com.intellij.openapi.util.SystemInfo
 import com.intellij.ui.components.JBCheckBox
 import com.intellij.ui.dsl.builder.*
+import com.jetbrains.rider.model.PublishableProjectModel
 import com.jetbrains.rider.model.publishableProjectsModel
 import com.jetbrains.rider.projectView.solution
 import com.jetbrains.rider.run.configurations.publishing.PublishRuntimeSettingsCoreHelper
@@ -70,8 +71,10 @@ class WebAppSettingEditor(private val project: Project) : SettingsEditor<WebAppC
             }
         }
 
-        dotnetProjectComboBox.component.reloadItems()
-
+        dotnetProjectComboBox.component.apply {
+            reloadItems()
+            addValueChangedListener(::onSelectProject)
+        }
         webAppComboBox.component.apply {
             addValueChangedListener(::onSelectWebApp)
         }
@@ -80,6 +83,12 @@ class WebAppSettingEditor(private val project: Project) : SettingsEditor<WebAppC
         }
     }
 
+    private fun onSelectProject(value: PublishableProjectModel) {
+        if (isLoading) return
+
+        webAppComboBox.component.targetProjectOnNetFramework = !value.isDotNetCore
+    }
+
     private fun onSelectWebApp(value: AppServiceConfig?) {
         if (isLoading) return