resolve comments.

Author: wangmingliang-ms

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-plugin-springcloud/src/main/java/com/microsoft/azure/toolkit/intellij/springcloud/properties/SpringCloudAppPropertiesEditor.java

@@ -66,7 +66,12 @@ protected void rerender() {
         tm.runLater(() -> {
             this.setEnabled(false);
             tm.runOnPooledThread(() -> {
-                this.deploymentDraft = this.draft.updateOrCreateActiveDeployment();
+                final SpringCloudDeployment deployment = this.draft.getActiveDeployment();
+                if (deployment == null) {
+                    AzureMessager.getMessager().warning(AzureString.format("No active deployment found for app(%s)", this.draft.getName()));
+                    return;
+                }
+                this.deploymentDraft = (SpringCloudDeploymentDraft) (deployment.isDraft() ? deployment : deployment.update());
                 tm.runLater(() -> {
                     this.formConfig.updateForm(this.draft);
                     this.panelInstances.setApp(this.draft);

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-resource-connector-lib/src/main/java/com/microsoft/azure/toolkit/intellij/connector/dotazure/AzureModule.java

@@ -78,7 +78,7 @@ public AzureModule(@Nonnull final Module module, @Nullable VirtualFile dotAzure)
 
     @SneakyThrows(value = {IOException.class, JDOMException.class})
     private void loadProfiles(@Nonnull final VirtualFile profilesXmlFile) {
-        if (profilesXmlFile.contentsToByteArray().length < 1 && !profilesXmlFile.exists()) {
+        if (!profilesXmlFile.exists() || profilesXmlFile.contentsToByteArray().length < 1) {
             return;
         }
         final Element profilesEle = JDOMUtil.load(Objects.requireNonNull(profilesXmlFile).toNioPath());

PluginsAndFeatures/azure-toolkit-for-intellij/azure-intellij-resource-connector-lib/src/main/java/com/microsoft/azure/toolkit/intellij/connector/projectexplorer/ResourceNode.java

@@ -93,7 +93,7 @@ public void onClicked(Object event) {
     @Nullable
     public IActionGroup getActionGroup() {
         final IActionGroup originalGroup = Optional.ofNullable(getValue()).map(Node::getActions).orElse(null);
-        final Object value = this.getValue().getValue();
+        final Object value = Optional.ofNullable(getValue()).map(Node::getValue).orElse(null);
         if (this.parent instanceof DeploymentTargetsNode targets && value instanceof AbstractAzResource<?, ?, ?> resource) {
             final AzureTaskManager tm = AzureTaskManager.getInstance();
             final Action<Object> removeTarget = new Action<>(Action.Id.of("user/connector.remove_target.app"))