commit ab5b6dd5f544789a4d034f72f0298200cfe97a56
author Jarek Gawor <gawor@apache.org> Thu May 02 15:02:51 2013 +0000
committer Jarek Gawor <gawor@apache.org> Thu May 02 15:02:51 2013 +0000
tree c30fe19d86cb984c779a4377b09b424e1d43b844
parent d632ad3f3c73341406ca3e39b2df025d2bb82dbe

GERONIMODEVTOOLS-795: Minor update - try to avoid NumberFormatExceptions

git-svn-id: https://svn.apache.org/repos/asf/geronimo/devtools/eclipse-plugin/trunk@1478399 13f79535-47bb-0310-9956-ffa450edef68

plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/internal/GeronimoServerWizardFragment.java

diff --git a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/internal/GeronimoServerWizardFragment.java b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/internal/GeronimoServerWizardFragment.java
index 4b12858..5518c4f 100644
--- a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/internal/GeronimoServerWizardFragment.java
+++ b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/internal/GeronimoServerWizardFragment.java
@@ -136,7 +136,8 @@
 
         @Override
         protected void setPortOffset(Text portOffset) {
-            getGeronimoServer().setPortOffset(Integer.parseInt(portOffset.getText()));
+            int value = toInt(portOffset.getText(), 0);
+            getGeronimoServer().setPortOffset(value);
         }
 
         @Override

plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/PortEditor.java

diff --git a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/PortEditor.java b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/PortEditor.java
index c9143c1..952327e 100644
--- a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/PortEditor.java
+++ b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/PortEditor.java
@@ -146,4 +146,11 @@
         return gs;
     }
 
+    protected int toInt(String value, int defaultValue) {
+        try {
+            return Integer.parseInt(value);
+        } catch (NumberFormatException e) {
+            return defaultValue;
+        }
+    }
 }   

plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/ServerEditorPortsSection.java

diff --git a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/ServerEditorPortsSection.java b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/ServerEditorPortsSection.java
index 76d56a4..931aeb4 100644
--- a/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/ServerEditorPortsSection.java
+++ b/plugins/org.apache.geronimo.st.v30.ui/src/main/java/org/apache/geronimo/st/v30/ui/sections/ServerEditorPortsSection.java
@@ -89,7 +89,7 @@
         
         @Override
         protected void setPortOffset(Text portOffset) {
-            int value = Integer.parseInt(portOffset.getText());
+            int value = toInt(portOffset.getText(), 0);
             execute(new TextSetPropertyCommand(server, "PortOffset", int.class, value, portOffset));
         }