[Hyperic-hq-discuss] hq r13946 - branches/HQ_4_1/src/org/hyperic/hq/product/jmx

[<do...@hy...>]

Author: dougm
Date: 2009-11-13 16:14:09 -0800 (Fri, 13 Nov 2009)
New Revision: 13946
URL: http://svn.hyperic.org/?view=rev&root=Hyperic+HQ&revision=13946

Modified:
   branches/HQ_4_1/src/org/hyperic/hq/product/jmx/MxQuery.java
Log:
[HHQ-3370] If a JMX attribute was found, but returned null, set the custom
property to 'Attribute was null'.  Previously the value would not be reported
or shown in the UI. (merged r13707 from branches/HQ_4_2)



Modified: branches/HQ_4_1/src/org/hyperic/hq/product/jmx/MxQuery.java
===================================================================
--- branches/HQ_4_1/src/org/hyperic/hq/product/jmx/MxQuery.java	2009-11-13 20:26:13 UTC (rev 13945)
+++ branches/HQ_4_1/src/org/hyperic/hq/product/jmx/MxQuery.java	2009-11-14 00:14:09 UTC (rev 13946)
@@ -133,6 +133,10 @@
                     stringValue = value.toString();
                 }
                 setAttribute(names[i], stringValue);
+            } else {
+                // Show 'Attribute was null' in UI.
+                log.debug("getAttribute(" + objName + ", " + name + ") returned null");
+                setAttribute(names[i], "Attribute was null");
             }
         }