Merge pull request #121 from amtrack/fix/empty-container-types

fix: omit pure container types for --children flag
amtrack · Apr 6, 2022 · a35bff5 · a35bff5
2 parents f12ac13 + 0065875
commit a35bff5
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 6 deletions.
diff --git a/src/commands/force/mdapi/listallmetadata.ts b/src/commands/force/mdapi/listallmetadata.ts
@@ -175,6 +175,10 @@ export default class MdapiListAllMetadataCommand extends SfdxCommand {
         }
       }
     }
+    if (this.flags.children) {
+      // omit parent types which don't have own properties
+      ignoreFunctions.push(filters.isPureContainerType);
+    }
     let fileProperties = await listAllMetadata(
       conn,
       this.flags,

diff --git a/src/commands/package.xml/generate.ts b/src/commands/package.xml/generate.ts
@@ -89,12 +89,6 @@ export default class PackageXmlGenerateCommand extends SfdxCommand {
     if (this.flags.defaultignore) {
       ignorePatterns.push(...this.flags.defaultignore);
     }
-    // don't list 'CustomLabels:CustomLabels' if specific 'CustomLabel:XXX' are given
-    if (fileProperties.find((fp) => fp.type === 'CustomLabel') !== null) {
-      fileProperties = fileProperties.filter(
-        (fp) => fp.type !== 'CustomLabels'
-      );
-    }
     const [, keep] = match(
       fileProperties,
       ignorePatterns,

diff --git a/src/filters.ts b/src/filters.ts
@@ -21,6 +21,18 @@ const MANAGED_READONLY_TYPES = [
   'ApexPage'
 ];
 
+// parent types which don't have own properties
+const PURE_CONTAINER_TYPES = [
+  'AssignmentRules',
+  'AutoResponseRules',
+  'CustomLabels',
+  'EscalationRules',
+  'ManagedTopics',
+  'MatchingRules',
+  'SharingRules',
+  'Workflow'
+];
+
 const STANDARD_USERNAMES = ['Automated Process', 'salesforce.com'];
 
 export function isManaged(fileProperties: FileProperties): boolean {
@@ -76,3 +88,7 @@ export function isStandard(fileProperties: FileProperties): boolean {
     fileProperties.namespacePrefix === 'standard'
   );
 }
+
+export function isPureContainerType(fileProperties: FileProperties): boolean {
+  return PURE_CONTAINER_TYPES.includes(fileProperties.type);
+}