address PR comments from @sureshanaparti

Author: Glover, Rene (rg9975)

engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategy.java

@@ -935,7 +935,7 @@ private VolumeInfo createTemporaryVolumeCopyOfSnapshotAdaptive(SnapshotInfo snap
         try {
             tempVolumeVO = new VolumeVO(Volume.Type.DATADISK, snapshotInfo.getName() + "_" + System.currentTimeMillis() + ".TMP",
                 snapshotInfo.getDataCenterId(), snapshotInfo.getDomainId(), snapshotInfo.getAccountId(), 0, ProvisioningType.THIN, snapshotInfo.getSize(), 0L, 0L, "");
-                tempVolumeVO.setPoolId(snapshotInfo.getDataStore().getId());
+            tempVolumeVO.setPoolId(snapshotInfo.getDataStore().getId());
             _volumeDao.persist(tempVolumeVO);
             tempVolumeInfo = this._volFactory.getVolume(tempVolumeVO.getId());
 

framework/spring/module/src/main/java/org/apache/cloudstack/spring/module/model/impl/DefaultModuleDefinitionSet.java

@@ -103,7 +103,6 @@ public void with(ModuleDefinition def, Stack<ModuleDefinition> parents) {
                     try {
                         if (context == null) {
                             log.warn(String.format("Application context not found for module definition [%s]", moduleDefinitionName));
-
                         } else if (context.containsBean("moduleStartup")) {
                             Runnable runnable = context.getBean("moduleStartup", Runnable.class);
                             log.info(String.format("Starting module [%s].", moduleDefinitionName));