refactor

Author: chrisgorgo

validators/nii.js

@@ -1,4 +1,3 @@
-var async = require('async');
 var utils = require('../utils');
 var Issue = utils.issues.Issue;
 
@@ -219,32 +218,32 @@ module.exports = function NIFTI (header, file, jsonContentsDict, bContentsDict,
         }
 
         if (path.includes("_phasediff.nii") || path.includes("_phase1.nii") ||
-            path.includes("_phase2.nii") || path.includes("_fieldmap.nii") || path.includes("_epi.nii")) {
-            if (mergedDictionary.hasOwnProperty('IntendedFor')) {
-                var intendedFor = typeof mergedDictionary['IntendedFor'] == "string" ? [mergedDictionary['IntendedFor']] : mergedDictionary['IntendedFor'];
-
-                for (var key = 0; key < intendedFor.length; key++) {
-                    var intendedForFile = intendedFor[key];
-                    var intendedForFileFull = "/" + path.split("/")[1] + "/" + intendedForFile;
-                    var onTheList = false;
-
-                    for (var key2 in fileList) {
-                        var filePath = fileList[key2].relativePath;
-                        if (filePath === intendedForFileFull) {
-                            onTheList = true;
-                        }
-                    }
-                    if (!onTheList) {
-                        issues.push(new Issue({
-                            file: file,
-                            code: 37,
-                            reason: "'IntendedFor' property of this fieldmap  ('" + path +
-                            "') does not point to an existing file('" + intendedForFile + "'). Please mind that this value should not include subject level directory " +
-                            "('/" + path.split("/")[1] + "/').",
-                            evidence: intendedForFile
-                        }));
+            path.includes("_phase2.nii") || path.includes("_fieldmap.nii") || path.includes("_epi.nii") &&
+            mergedDictionary.hasOwnProperty('IntendedFor')) {
+            var intendedFor = typeof mergedDictionary['IntendedFor'] == "string" ? [mergedDictionary['IntendedFor']] : mergedDictionary['IntendedFor'];
+
+            for (var key = 0; key < intendedFor.length; key++) {
+                var intendedForFile = intendedFor[key];
+                var intendedForFileFull = "/" + path.split("/")[1] + "/" + intendedForFile;
+                var onTheList = false;
+
+                for (var key2 in fileList) {
+                    var filePath = fileList[key2].relativePath;
+                    if (filePath === intendedForFileFull) {
+                        onTheList = true;
                     }
                 }
+                if (!onTheList) {
+                    issues.push(new Issue({
+                        file: file,
+                        code: 37,
+                        reason: "'IntendedFor' property of this fieldmap  ('" + path +
+                        "') does not point to an existing file('" + intendedForFile + "'). Please mind that this value should not include subject level directory " +
+                        "('/" + path.split("/")[1] + "/').",
+                        evidence: intendedForFile
+                    }));
+                }
+            }
             }
         }
     }