Changeset 642 for trunk/src/org/expeditee/settings/UserSettings.java

Timestamp:

12/18/13 14:28:51 (10 years ago)

Author:

jts21

Message:

Add more checks to ignore invalid/unset setting values

File:

• trunk/src/org/expeditee/settings/UserSettings.java (modified) (2 diffs)

trunk/src/org/expeditee/settings/UserSettings.java

@@ -243 +243 @@
         }
         public static void setSpelling(Text t) {
-                if(t.hasLink()) {
+                if(t.getChild() != null) {
                 try {
                 JSpellChecker.create(t.getChild());
@@ -253 +253 @@
         
         public static void setGreenstoneSettings(Text t) {
-                SearchGreenstone.init(t.getChild());
+                if(t.getChild() != null) SearchGreenstone.init(t.getChild());
         }
         
         public static void setReminders(Text t) {
-                Reminders.init(t.getChild());
+                if(t.getChild() != null) Reminders.init(t.getChild());
         }
         
         public static void setFormatSpacingMin(Text t) {
-                FrameUtils.MINIMUM_SPACING_RATIO = new AttributeValuePair(t.getText()).getDoubleValue();
+                double value = new AttributeValuePair(t.getText()).getDoubleValue();
+                if(value != Double.NaN) {
+                        FrameUtils.MINIMUM_SPACING_RATIO = value;
+                }
         }
         
         public static void setFormatSpacingMax(Text t) {
-                FrameUtils.MAXIMUM_SPACING_RATIO = new AttributeValuePair(t.getText()).getDoubleValue();
+                double value = new AttributeValuePair(t.getText()).getDoubleValue();
+                if(value != Double.NaN) {
+                        FrameUtils.MAXIMUM_SPACING_RATIO = value;
+                }
         }
         
         public static void setMailSettings(Text t) {
-                MailSession.init(t.getChild());
+                if(t.getChild() != null) MailSession.init(t.getChild());
         }