Calling toggle modules within setup and setup:update targets.

Author: grasmash

phing/tasks/ci.xml

@@ -14,6 +14,8 @@
     <phingcall target="setup">
       <property name="drush.alias" value="${drush.aliases.ci}"/>
       <property name="create_alias" value="n"/>
+      <property name="enable_property" value="modules.ci.enable"/>
+      <property name="uninstall_property" value="modules.ci.uninstall"/>
     </phingcall>
   </target>
 
@@ -37,14 +39,8 @@
   <target name="ci:update" description="Update current database to reflect the state of the Drupal file system; uses ci drush alias." hidden="true">
     <phingcall target="setup:update">
       <property name="drush.alias" value="${drush.aliases.ci}"/>
-    </phingcall>
-    <phingcall target="ci:toggle-modules"/>
-  </target>
-
-  <target name="ci:toggle-modules" description="Enables modules.ci.enable projects and uninstalls modules.ci.uninstall projects." hidden="true">
-    <phingcall target="setup:toggle-modules">
-      <param name="enable_property" value="modules.ci.enable"/>
-      <param name="uninstall_property" value="modules.ci.uninstall"/>
+      <property name="enable_property" value="modules.ci.enable"/>
+      <property name="uninstall_property" value="modules.ci.uninstall"/>
     </phingcall>
   </target>
 </project>

phing/tasks/deploy.xml

@@ -157,19 +157,14 @@
     </delete>
   </target>
 
-  <target name="deploy:update" description="Update current database to reflect the state of the Drupal file system; uses local drush alias." depends="deploy:toggle-modules">
+  <target name="deploy:update" description="Update current database to reflect the state of the Drupal file system; uses local drush alias.">
     <phingcall target="setup:update">
+      <param name="enable_property" value="modules.deploy.enable"/>
+      <param name="uninstall_property" value="modules.deploy.uninstall"/>
       <property name="drush.alias" value="self"/>
       <!-- Most sites store their version-controlled configuration in /config/default. -->
       <!-- ACE internally sets the vcs configuration directory to /config/default, so we use that. -->
       <property name="cm.core.config-dir" value="vcs"/>
     </phingcall>
   </target>
-
-  <target name="deploy:toggle-modules" description="Enables modules.deploy.enable projects and uninstalls modules.deploy.uninstall projects.">
-    <phingcall target="setup:toggle-modules">
-      <param name="enable_property" value="modules.deploy.enable"/>
-      <param name="uninstall_property" value="modules.deploy.uninstall"/>
-    </phingcall>
-  </target>
 </project>

phing/tasks/local-sync.xml

@@ -8,9 +8,11 @@
 
   <target name="local:refresh" description="Refreshes local environment from upstream testing database." depends="setup:build, local:sync, local:update"/>
 
-  <target name="local:setup" description="Install dependencies, builds docroot, installs Drupal; uses local drush alias." depends="local:toggle-modules">
+  <target name="local:setup" description="Install dependencies, builds docroot, installs Drupal; uses local drush alias.">
     <phingcall target="setup">
       <property name="drush.alias" value="${drush.aliases.local}" />
+      <param name="enable_property" value="modules.local.enable"/>
+      <param name="uninstall_property" value="modules.local.uninstall"/>
     </phingcall>
   </target>
 
@@ -30,18 +32,12 @@
     <drush command="cache-rebuild"/>
   </target>
 
-  <target name="local:toggle-modules" description="Enables modules.local.enable projects and uninstalls modules.local.uninstall projects.">
-    <phingcall target="setup:toggle-modules">
-      <param name="enable_property" value="modules.local.enable"/>
-      <param name="uninstall_property" value="modules.local.uninstall"/>
-    </phingcall>
-  </target>
-
   <target name="local:update" description="Update current database to reflect the state of the Drupal file system; uses local drush alias.">
     <phingcall target="setup:update">
       <property name="drush.alias" value="${drush.aliases.local}"/>
+      <property name="enable_property" value="modules.local.enable"/>
+      <property name="uninstall_property" value="modules.local.uninstall"/>
     </phingcall>
-    <phingcall target="local:toggle-modules"/>
   </target>
 
 </project>

phing/tasks/setup.xml

@@ -158,6 +158,9 @@
 
     <phingcall target="setup:config-import"/>
 
+    <!-- enable_property and uninstall_property must be set at this time. -->
+    <phingcall target="setup:update"/>
+
     <!-- Set sites directory file permissions. -->
     <echo>Making ${docroot}/sites/default writable.</echo>
     <chmod mode="0755" failonerror="false">
@@ -225,6 +228,10 @@
   </target>
 
   <target name="setup:update" description="Update current database to reflect the state of the Drupal file system.">
+
+    <!-- enable_property and uninstall_property must be set at this time. -->
+    <phingcall target="setup:toggle-modules"/>
+
     <!-- Import configuration before executing updates, in case any db updates are dependent on new configs to be imported. -->
     <drush command="config-import" assume="yes" alias="${drush.alias}">
       <option name="partial"></option>