Author: tfennelly Date: 2010-10-07 06:55:47 -0400 (Thu, 07 Oct 2010) New Revision: 25551 Modified: branches/jbosstools-3.2.0.Beta1/smooks/plugins/org.jboss.tools.smooks.ui/src/org/jboss/tools/smooks/graphical/actions/AddTaskNodeAction.java Log: https://jira.jboss.org/browse/JBIDE-7233 Empty selected task details when using Apply Template (XML/XSD) from Input Task Modified: branches/jbosstools-3.2.0.Beta1/smooks/plugins/org.jboss.tools.smooks.ui/src/org/jboss/tools/smooks/graphical/actions/AddTaskNodeAction.java =================================================================== --- branches/jbosstools-3.2.0.Beta1/smooks/plugins/org.jboss.tools.smooks.ui/src/org/jboss/tools/smooks/graphical/actions/AddTaskNodeAction.java 2010-10-07 10:52:58 UTC (rev 25550) +++ branches/jbosstools-3.2.0.Beta1/smooks/plugins/org.jboss.tools.smooks.ui/src/org/jboss/tools/smooks/graphical/actions/AddTaskNodeAction.java 2010-10-07 10:55:47 UTC (rev 25551) @@ -1,5 +1,7 @@ package org.jboss.tools.smooks.graphical.actions; +import java.util.List; + import org.eclipse.ui.IEditorPart; import org.jboss.tools.smooks.editor.ISmooksModelProvider; import org.jboss.tools.smooks.graphical.editors.TaskTypeManager; @@ -35,12 +37,16 @@ @Override public void update() { this.setEnabled(false); - if (this.getCurrentSelectedTask() != null && this.getCurrentSelectedTask().size() == 1) { - TaskType currentTask = this.getCurrentSelectedTask().get(0); - String taskID = currentTask.getId(); - if (taskID != null) { - if (taskID.equals(TaskTypeManager.TASK_ID_INPUT) || taskID.equals(TaskTypeManager.TASK_ID_JAVA_MAPPING)) - this.setEnabled(true); + List<TaskType> currentSelectedTask = this.getCurrentSelectedTask(); + if (currentSelectedTask != null && currentSelectedTask.size() == 1) { + TaskType currentTask = currentSelectedTask.get(0); + + if(currentTask.getTask().isEmpty()) { + String taskID = currentTask.getId(); + if (taskID != null) { + if (taskID.equals(TaskTypeManager.TASK_ID_INPUT) || taskID.equals(TaskTypeManager.TASK_ID_JAVA_MAPPING)) + this.setEnabled(true); + } } } }