fixed testAutoRendererConfigurable - pt.2
authorEgor.Ushakov <egor.ushakov@jetbrains.com>
Thu, 3 Sep 2015 13:18:09 +0000 (16:18 +0300)
committerEgor.Ushakov <egor.ushakov@jetbrains.com>
Thu, 3 Sep 2015 13:19:04 +0000 (16:19 +0300)
java/debugger/impl/src/com/intellij/debugger/ui/tree/render/BasicRendererProperties.java

index af38859684dde0b37006a9509e2bbb2e543dbd9c..188f3b51fce90b2f07ebd0feeff8a4c0d2a4fb45 100644 (file)
@@ -34,7 +34,7 @@ public final class BasicRendererProperties implements Cloneable, JDOMExternaliza
   private String myName;
 
   private static final @NonNls String ENABLED_OPTION = "ENABLED";
   private String myName;
 
   private static final @NonNls String ENABLED_OPTION = "ENABLED";
-  private Boolean myEnabled;
+  private boolean myEnabled;
 
   private static final @NonNls String CLASSNAME_OPTION = "QUALIFIED_NAME";
   private String myClassName;
 
   private static final @NonNls String CLASSNAME_OPTION = "QUALIFIED_NAME";
   private String myClassName;
@@ -51,7 +51,7 @@ public final class BasicRendererProperties implements Cloneable, JDOMExternaliza
   }
 
   public boolean isEnabled() {
   }
 
   public boolean isEnabled() {
-    return myEnabled != null && myEnabled.booleanValue();
+    return myEnabled;
   }
 
   public void setEnabled(final boolean enabled) {
   }
 
   public void setEnabled(final boolean enabled) {
@@ -76,7 +76,6 @@ public final class BasicRendererProperties implements Cloneable, JDOMExternaliza
 
   @SuppressWarnings({"HardCodedStringLiteral"}) public void readExternal(Element element) throws InvalidDataException {
     myName = null;
 
   @SuppressWarnings({"HardCodedStringLiteral"}) public void readExternal(Element element) throws InvalidDataException {
     myName = null;
-    myEnabled = null;
     myClassName = null;
     for (Element option : element.getChildren("option")) {
       final String optionName = option.getAttributeValue("name");
     myClassName = null;
     for (Element option : element.getChildren("option")) {
       final String optionName = option.getAttributeValue("name");
@@ -84,6 +83,7 @@ public final class BasicRendererProperties implements Cloneable, JDOMExternaliza
         myName = option.getAttributeValue("value");
       }
       else if (ENABLED_OPTION.equals(optionName)) {
         myName = option.getAttributeValue("value");
       }
       else if (ENABLED_OPTION.equals(optionName)) {
+        // default is false
         myEnabled = Boolean.parseBoolean(option.getAttributeValue("value"));
       }
       else if (CLASSNAME_OPTION.equals(optionName)) {
         myEnabled = Boolean.parseBoolean(option.getAttributeValue("value"));
       }
       else if (CLASSNAME_OPTION.equals(optionName)) {
@@ -100,15 +100,18 @@ public final class BasicRendererProperties implements Cloneable, JDOMExternaliza
     if (myName != null) {
       addOption(element, NAME_OPTION, myName);
     }
     if (myName != null) {
       addOption(element, NAME_OPTION, myName);
     }
-    if (myEnabled != null) {
-      addOption(element, ENABLED_OPTION, myEnabled.toString());
+    if (myEnabled) {
+      // default is false
+      //noinspection ConstantConditions
+      addOption(element, ENABLED_OPTION, Boolean.toString(myEnabled));
     }
     if (myClassName != null) {
       addOption(element, CLASSNAME_OPTION, myClassName);
     }
     if (!myShowType) {
       // default is true
     }
     if (myClassName != null) {
       addOption(element, CLASSNAME_OPTION, myClassName);
     }
     if (!myShowType) {
       // default is true
-      addOption(element, SHOWTYPE_OPTION, "false");
+      //noinspection ConstantConditions
+      addOption(element, SHOWTYPE_OPTION, Boolean.toString(myShowType));
     }
   }
 
     }
   }