Fix findbugs DM_BOXED_PRIMITIVE_FOR_PARSING in BareMetalResourceBase.java

rsafonseca · DaanHoogland · commit dc359c505e07 · 2015-06-22T10:50:12.000+02:00
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com> This closes apache#501
diff --git a/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BareMetalResourceBase.java b/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BareMetalResourceBase.java
@@ -188,7 +188,7 @@ public boolean configure(String name, Map<String, Object> params) throws Configu
 
         try {
             provisionDoneNotificationOn = Boolean.valueOf(configDao.getValue(Config.BaremetalProvisionDoneNotificationEnabled.key()));
-            isProvisionDoneNotificationTimeout = Integer.valueOf(configDao.getValue(Config.BaremetalProvisionDoneNotificationTimeout.key()));
+            isProvisionDoneNotificationTimeout = Integer.parseInt(configDao.getValue(Config.BaremetalProvisionDoneNotificationTimeout.key()));
         } catch (Exception e) {
             s_logger.debug(e.getMessage(), e);
         }

Original file line number	Diff line number	Diff line change
`@@ -188,7 +188,7 @@ public boolean configure(String name, Map<String, Object> params) throws Configu`
`188`	`188`
`189`	`189`	`try {`
`190`	`190`	`provisionDoneNotificationOn = Boolean.valueOf(configDao.getValue(Config.BaremetalProvisionDoneNotificationEnabled.key()));`
`191`		`- isProvisionDoneNotificationTimeout = Integer.valueOf(configDao.getValue(Config.BaremetalProvisionDoneNotificationTimeout.key()));`
	`191`	`+ isProvisionDoneNotificationTimeout = Integer.parseInt(configDao.getValue(Config.BaremetalProvisionDoneNotificationTimeout.key()));`
`192`	`192`	`} catch (Exception e) {`
`193`	`193`	`s_logger.debug(e.getMessage(), e);`
`194`	`194`	`}`