chore: refactor long fn out

salesforcecli · Dec 5, 2023 · ab5bc89 · ab5bc89
1 parent 30c43e9
commit ab5bc89
Showing 1 changed file with 7 additions and 12 deletions.
diff --git a/src/commands/project/retrieve/start.ts b/src/commands/project/retrieve/start.ts
@@ -373,18 +373,7 @@ const buildRetrieveAndDeleteTargets = async (
     }
     return result;
   } else {
-    // check if we're retrieving metadata based on a pattern ...
-    let retrieveFromOrg: string | undefined;
-    if (flags.metadata) {
-      flags.metadata.some((mdEntry) => {
-        const mdName = mdEntry.split(':')[1];
-        if (mdName?.includes('*') && mdName?.length > 1 && !mdName?.includes('.*')) {
-          retrieveFromOrg = flags['target-org'].getUsername();
-          return true;
-        }
-        return false;
-      });
-    }
+    const retrieveFromOrg = flags.metadata?.some(isWildcard) ? flags['target-org'].getUsername() : undefined;
 
     return {
       componentSetFromNonDeletes: await ComponentSetBuilder.build({
@@ -449,3 +438,9 @@ const buildRetrieveOptions = async (
         output: output ?? (await SfProject.resolve()).getDefaultPackage().fullPath,
       }),
 });
+
+// check if we're retrieving metadata based on a pattern ...
+const isWildcard = (mdEntry: string): boolean => {
+  const mdName = mdEntry.split(':')[1];
+  return mdName?.includes('*') && mdName?.length > 1 && !mdName?.includes('.*');
+};