Minor cleanup.

OpenNMS · May 7, 2014 · a073dc8 · a073dc8
1 parent b85d2d5
commit a073dc8
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/...n/api/src/main/java/org/opennms/netmgt/collection/support/AbstractCollectionResource.java b/...n/api/src/main/java/org/opennms/netmgt/collection/support/AbstractCollectionResource.java
@@ -144,7 +144,7 @@ public boolean shouldPersist(ServiceParameters params) {
     }
 
     /**
-     * <p>getLabel</p>
+     * <p>getInterfaceLabel</p>
      *
      * @return a {@link java.lang.String} object.
      */

diff --git a/...ig-jaxb/src/main/java/org/opennms/netmgt/config/internal/collection/ResourceTypeImpl.java b/...ig-jaxb/src/main/java/org/opennms/netmgt/config/internal/collection/ResourceTypeImpl.java
@@ -16,10 +16,10 @@
 /**
  *  &lt;resourceType name="hrStorageIndex" label="Storage (MIB-2 Host Resources)"&gt;
  *    &lt;resourceName&gt;
- *      &lt;m_template&gt;${hrStorageDescr}&lt;/m_template&gt;
+ *      &lt;template&gt;${hrStorageDescr}&lt;/template&gt;
  *    &lt;/resourceName&gt;
- *    &lt;resourceLabel&gt;&lt;m_template&gt;${hrStorageDescr}&lt;/m_template&gt;&lt;/resourceLabel&gt;
- *    &lt;resourceKind&gt;&lt;m_template&gt;${hrStorageType}&lt;/m_template&gt;&lt;/resourceKind&gt;
+ *    &lt;resourceLabel&gt;&lt;template&gt;${hrStorageDescr}&lt;/template&gt;&lt;/resourceLabel&gt;
+ *    &lt;resourceKind&gt;&lt;template&gt;${hrStorageType}&lt;/template&gt;&lt;/resourceKind&gt;
  *    &lt;column oid=".1.3.6.1.2.1.25.2.3.1.2" alias="hrStorageType"  type="string" /&gt;
  *    &lt;column oid=".1.3.6.1.2.1.25.2.3.1.3" alias="hrStorageDescr" type="string" /&gt;
  *  &lt;/resourceType&gt;

diff --git a/opennms-services/src/main/java/org/opennms/netmgt/collectd/IfInfo.java b/opennms-services/src/main/java/org/opennms/netmgt/collectd/IfInfo.java
@@ -221,11 +221,11 @@ private OnmsSnmpCollection getCollection() {
     @Override
     public File getResourceDir(RrdRepository repository) throws FileNotFoundException {
         String label = getInterfaceLabel();
-        File rrdBaseDir = repository.getRrdBaseDir();
-        File dir = new File(rrdBaseDir, getCollectionAgent().getStorageDir().toString());
         if (label == null || "".equals(label)) {
-            throw new FileNotFoundException("Could not construct resource directory because label is null or blank: nodeId: " + getNodeId() + ", rrdRepository: " + repository.toString());
+            throw new FileNotFoundException("Could not construct resource directory because interface label is null or blank: nodeId: " + getNodeId() + ", rrdRepository: " + repository.toString());
         } else {
+            File rrdBaseDir = repository.getRrdBaseDir();
+            File dir = new File(rrdBaseDir, getCollectionAgent().getStorageDir().toString());
             return new File(dir, label);
         }
     }
@@ -241,10 +241,10 @@ public String toString() {
     }
 
     boolean shouldStore(ServiceParameters serviceParameters) {
-        if (serviceParameters.getStoreByNodeID().equals("normal")) {
+        if ("normal".equalsIgnoreCase(serviceParameters.getStoreByNodeID())) {
             return isScheduledForCollection();
         } else {
-            return serviceParameters.getStoreByNodeID().equals("true");
+            return "true".equalsIgnoreCase(serviceParameters.getStoreByNodeID());
         }
     }
 

diff --git a/opennms-services/src/main/java/org/opennms/netmgt/threshd/CollectionResourceWrapper.java b/opennms-services/src/main/java/org/opennms/netmgt/threshd/CollectionResourceWrapper.java
@@ -324,7 +324,7 @@ protected String getIfInfoValue(String attribute) {
      * @return a boolean.
      */
     public boolean isAnInterfaceResource() {
-        return getResourceTypeName() != null && getResourceTypeName().equals("if");
+        return getResourceTypeName() != null && CollectionResource.RESOURCE_TYPE_IF.equals(getResourceTypeName());
     }
 
     /**