From 466e4b29bcf023941c2e05d5ba0df76e21863c2d Mon Sep 17 00:00:00 2001
From: Tim Otten <totten@civicrm.org>
Date: Fri, 31 Mar 2017 00:36:35 -0700
Subject: [PATCH] CRM-20600 - Civi\Angular - Generate modules via AssetBuilder

In the typical use-case, AssetBuilder can avoid an extra CMS+Civi bootstrap
along with any build steps.
---
 Civi/Angular/Page/Main.php    |  5 ++++-
 Civi/Angular/Page/Modules.php | 39 ++++++++++++++++++++++++++++++++++-
 Civi/Core/Container.php       |  1 +
 js/angular-crmResource/all.js |  2 +-
 4 files changed, 44 insertions(+), 3 deletions(-)

diff --git a/Civi/Angular/Page/Main.php b/Civi/Angular/Page/Main.php
index 2f945ebae7..81b5536ecb 100644
--- a/Civi/Angular/Page/Main.php
+++ b/Civi/Angular/Page/Main.php
@@ -81,6 +81,7 @@ class Main extends \CRM_Core_Page {
         'angular' => array(
           'modules' => array_merge(array('ngRoute'), array_keys($modules)),
           'cacheCode' => $page->res->getCacheCode(),
+          'bundleUrl' => \Civi::service('asset_builder')->getUrl('angular-modules.json'),
         ),
       ));
     });
@@ -104,11 +105,13 @@ class Main extends \CRM_Core_Page {
     else {
       // Note: addScriptUrl() bypasses the normal string-localization of addScriptFile(),
       // but that's OK because all Angular strings (JS+HTML) will load via crmResource.
-      $aggScriptUrl = \CRM_Utils_System::url('civicrm/ajax/angular-modules', 'format=js&r=' . $page->res->getCacheCode(), FALSE, NULL, FALSE);
+      // $aggScriptUrl = \CRM_Utils_System::url('civicrm/ajax/angular-modules', 'format=js&r=' . $page->res->getCacheCode(), FALSE, NULL, FALSE);
+      $aggScriptUrl = \Civi::service('asset_builder')->getUrl('angular-modules.js');
       $this->res->addScriptUrl($aggScriptUrl, 120, $this->region);
 
       // FIXME: The following CSS aggregator doesn't currently handle path-adjustments - which can break icons.
       //$aggStyleUrl = \CRM_Utils_System::url('civicrm/ajax/angular-modules', 'format=css&r=' . $page->res->getCacheCode(), FALSE, NULL, FALSE);
+      //$aggStyleUrl = \Civi::service('asset_builder')->getUrl('angular-modules.css');
       //$this->res->addStyleUrl($aggStyleUrl, 120, $this->region);
 
       foreach ($this->angular->getResources(array_keys($modules), 'css', 'cacheUrl') as $url) {
diff --git a/Civi/Angular/Page/Modules.php b/Civi/Angular/Page/Modules.php
index 2fc3d86368..663a8f6ea1 100644
--- a/Civi/Angular/Page/Modules.php
+++ b/Civi/Angular/Page/Modules.php
@@ -20,7 +20,10 @@ namespace Civi\Angular\Page;
 class Modules extends \CRM_Core_Page {
 
   /**
-   * See class description.
+   * Generate asset content (when accessed via older, custom
+   * "civicrm/ajax/anulgar-modules" route).
+   *
+   * @deprecated
    */
   public function run() {
     /**
@@ -59,6 +62,40 @@ class Modules extends \CRM_Core_Page {
     \CRM_Utils_System::civiExit();
   }
 
+  /**
+   * Generate asset content (when accessed via AssetBuilder).
+   *
+   * @param \Civi\Core\Event\GenericHookEvent $event
+   * @see CRM_Utils_hook::buildAsset()
+   * @see \Civi\Core\AssetBuilder
+   */
+  public static function buildAngularModules($event) {
+    $page = new Modules();
+    $angular = \Civi::service('angular');
+
+    switch ($event->asset) {
+      case 'angular-modules.json':
+        $moduleNames = $page->parseModuleNames(\CRM_Utils_Array::value('modules', $event->params), $angular);
+        $event->mimeType = 'application/json';
+        $event->content = json_encode($page->getMetadata($moduleNames, $angular));
+        break;
+
+      case 'angular-modules.js':
+        $moduleNames = $page->parseModuleNames(\CRM_Utils_Array::value('modules', $event->params), $angular);
+        $event->mimeType = 'application/javascript';
+        $event->content = $page->digestJs($angular->getResources($moduleNames, 'js', 'path'));
+        break;
+
+      case 'angular-modules.css':
+        $moduleNames = $page->parseModuleNames(\CRM_Utils_Array::value('modules', $event->params), $angular);
+        $event->mimeType = 'text/css';
+        $event->content = \CRM_Utils_File::concat($angular->getResources($moduleNames, 'css', 'path'), "\n");
+
+      default:
+        // Not our problem.
+    }
+  }
+
   /**
    * @param array $files
    *   File paths.
diff --git a/Civi/Core/Container.php b/Civi/Core/Container.php
index 24bf7e0f8c..efe6a656c5 100644
--- a/Civi/Core/Container.php
+++ b/Civi/Core/Container.php
@@ -259,6 +259,7 @@ class Container {
     // TODO We need a better code-convention for metadata about non-hook events.
     $dispatcher->addListener('hook_civicrm_eventDefs', array('\Civi\API\Events', 'hookEventDefs'));
     $dispatcher->addListener('hook_civicrm_eventDefs', array('\Civi\Core\Event\SystemInstallEvent', 'hookEventDefs'));
+    $dispatcher->addListener('hook_civicrm_buildAsset', array('\Civi\Angular\Page\Modules', 'buildAngularModules'));
     $dispatcher->addListener('civi.dao.postInsert', array('\CRM_Core_BAO_RecurringEntity', 'triggerInsert'));
     $dispatcher->addListener('civi.dao.postUpdate', array('\CRM_Core_BAO_RecurringEntity', 'triggerUpdate'));
     $dispatcher->addListener('civi.dao.postDelete', array('\CRM_Core_BAO_RecurringEntity', 'triggerDelete'));
diff --git a/js/angular-crmResource/all.js b/js/angular-crmResource/all.js
index 7c3632fc2a..e8eca721b5 100644
--- a/js/angular-crmResource/all.js
+++ b/js/angular-crmResource/all.js
@@ -32,7 +32,7 @@
       });
     };
 
-    var moduleUrl = CRM.url('civicrm/ajax/angular-modules', {l: CRM.config.lcMessages, r: CRM.angular.cacheCode});
+    var moduleUrl = CRM.angular.bundleUrl;
     $http.get(moduleUrl)
       .success(function httpSuccess(data) {
         templates = [];
-- 
2.25.1