create-profile.sh: make detection of the default_content module more robust
authorAntonio Ospite <ao2@ao2.it>
Fri, 16 Mar 2018 12:31:17 +0000 (13:31 +0100)
committerAntonio Ospite <ao2@ao2.it>
Fri, 16 Mar 2018 12:34:29 +0000 (13:34 +0100)
Do not just grep on "$ENABLED_MODULES" which is a comma-separated list
on which is harder to match for the exact module name.

Use instead the output of pm-list which has one module name for each
line, this way it is easier to match for the exact name of
"default_content" and avoid picking up other modules which may contain
the string "default_content" in their names.

libexec/create-profile.sh

index 62a5b01..370963d 100755 (executable)
@@ -136,7 +136,7 @@ find "${PROFILE_DEST_DIR}/config/install" -type f -exec sed -i -e '/^_core: {  }
 #fdupes -f -1 "${WEB_ROOT}/core/profiles/standard/config/install/" "${PROFILE_DEST_DIR}/config/install/" | xargs rm
 
 # Export the default content if the default_content module is there
-if echo "$ENABLED_MODULES" | grep -q default_content;
+if $DRUSH pm-list --type=module --field=name --status=enabled --pipe | grep -q "^default_content$";
 then
   $DRUSH default-content-export-references --folder="${PROFILE_DEST_DIR}/content" node
 fi