Give warning when deleting the last build target. Fix the missing-all build targets...

[mirror_edk2.git] / Tools / Java / Source / FrameworkWizard / src / org / tianocore / frameworkwizard / platform / ui / FpdPlatformDefs.java

diff --git a/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/platform/ui/FpdPlatformDefs.java b/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/platform/ui/FpdPlatformDefs.java
index b76d8b3822ae7ea9a19944b3ead20eeb54453559..f7d8b01b9aa8e81ec668d0b63c530a0338bbff66 100644 (file)
--- a/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/platform/ui/FpdPlatformDefs.java
+++ b/Tools/Java/Source/FrameworkWizard/src/org/tianocore/frameworkwizard/platform/ui/FpdPlatformDefs.java
@@ -730,13 +730,17 @@ public class FpdPlatformDefs extends IInternalFrame {
 \r
             jButtonDelBuildTarget.addActionListener(new java.awt.event.ActionListener() {\r
                 public void actionPerformed(java.awt.event.ActionEvent e) {\r
-                    if (jTableBuildTargets.getSelectedRow() < 0 || jTableBuildTargets.getSelectedRow() < 0) {\r
+                    if (jTableBuildTargets.getSelectedRow() < 0) {\r
+                        return;\r
+                    }\r
+                    if (jTableBuildTargets.getRowCount() == 1) {\r
+                        JOptionPane.showMessageDialog(frame, "At least one build target should be set for this platform.");\r
                         return;\r
                     }\r
                     buildTargetTableModel.removeRow(jTableBuildTargets.getSelectedRow());\r
                     Vector<Object> v = new Vector<Object>();\r
                     for (int i = 0; i < jTableBuildTargets.getRowCount(); ++i) {\r
-                        v.remove(buildTargetTableModel.getValueAt(i, 0));\r
+                        v.add(buildTargetTableModel.getValueAt(i, 0));\r
                     }\r
                     docConsole.setSaved(false);\r
                     ffc.setPlatformDefsBuildTargets(v);\r