REF - Remove unneeded return from private fn, add comments

diff --git a/Civi/Angular/AngularLoader.php b/Civi/Angular/AngularLoader.php
index ecfb4797fbb52f59e5064113d33dd63d9921d5e1..086756a3b407eac39d65cf8f646309c0c7f942fa 100644 (file)
--- a/Civi/Angular/AngularLoader.php
+++ b/Civi/Angular/AngularLoader.php
@@ -80,6 +80,7 @@ class AngularLoader {
     $this->region = \CRM_Utils_Request::retrieve('snippet', 'String') ? 'ajax-snippet' : 'html-header';
     $this->pageName = \CRM_Utils_System::currentPath();
     $this->modules = [];
+    // List of already-present modules may be provided by crmSnippet (see crm.ajax.js)
     if ($this->region === 'ajax-snippet' && !empty($_GET['crmAngularModules'])) {
       $this->modulesAlreadyLoaded = explode(',', $_GET['crmAngularModules']);
     }
@@ -97,13 +98,13 @@ class AngularLoader {
    */
   public function load() {
     \CRM_Core_Error::deprecatedFunctionWarning('angularjs.loader service');
-    return $this->loadAngularResources();
+    $this->loadAngularResources();
+    return $this;
   }
 
   /**
    * Load scripts, styles & settings for the active modules.
    *
-   * @return $this
    * @throws \CRM_Core_Exception
    */
   private function loadAngularResources() {
@@ -132,7 +133,7 @@ class AngularLoader {
 
     if (!$moduleNames && $this->modulesAlreadyLoaded) {
       // No modules to load
-      return $this;
+      return;
     }
     if (!$this->isAllModules($moduleNames)) {
       $assetParams = ['modules' => implode(',', $moduleNames)];
@@ -214,8 +215,6 @@ class AngularLoader {
         $res->addBundle($bundles);
       }
     }
-
-    return $this;
   }
 
   /**

diff --git a/js/crm.ajax.js b/js/crm.ajax.js
index 007c7f93464d8a88a5c6dd224ad358b646d131f3..82a8025a7942191cb0e322a29c506bb4eaf51bc9 100644 (file)
--- a/js/crm.ajax.js
+++ b/js/crm.ajax.js
@@ -273,6 +273,7 @@
         } else {
           url = url.replace(/snippet=[^&]*/, 'snippet=' + snippetType);
         }
+        // See Civi\Angular\AngularLoader
         if (snippetType === 'json' && CRM.angular) {
           url += '&crmAngularModules=' + CRM.angular.modules.join();
         }