From 8935eb8546c4710a52602f0d8f931e03391dafef Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Thu, 19 Mar 2015 18:22:44 -0400
Subject: [PATCH] CRM-16142 - Variable scope fixes

---
 js/angular-crmMailing.js            | 21 ---------------------
 js/angular-crmMailing/directives.js | 19 +++++++++++++++++++
 2 files changed, 19 insertions(+), 21 deletions(-)

diff --git a/js/angular-crmMailing.js b/js/angular-crmMailing.js
index 57f09ba864..256cda54e8 100644
--- a/js/angular-crmMailing.js
+++ b/js/angular-crmMailing.js
@@ -109,27 +109,6 @@
         }));
     };
 
-    $scope.previewTestGroup = function(e) {
-      var $dialog = $(this),
-        // Fixme: how to get this from the scope?
-        gid = $('select[name=preview_test_group]').val();
-      $dialog.html('<div class="crm-loading-element"></div>');
-      CRM.api3('contact', 'get', {group: gid, options: {limit: 0}, return: 'display_name,email'}).done(function(data) {
-        // Fixme: should this be in a template?
-        var count = 0,
-          markup = '<ol>';
-        _.each(data.values, function(row) {
-          // Fixme: contact api doesn't seem capable of filtering out contacts with no email, so we're doing it client-side
-          if (row.email) {
-            count++;
-            markup += '<li>' + row.display_name + ' - ' + row.email + '</li>';
-          }
-        });
-        markup += '</ol>';
-        $dialog.html('<h4>' + ts('A test message will be sent to %1 people:', {1: count}) + '</h4>' + markup).trigger('crmLoad');
-      });
-    };
-
     // @return Promise
     $scope.submit = function submit(options) {
       options = options || {};
diff --git a/js/angular-crmMailing/directives.js b/js/angular-crmMailing/directives.js
index 307df6369e..aa32d86083 100644
--- a/js/angular-crmMailing/directives.js
+++ b/js/angular-crmMailing/directives.js
@@ -61,6 +61,25 @@
             preview: {recipient: recipient}
           });
         };
+
+        scope.previewTestGroup = function(e) {
+          var $dialog = $(this);
+          $dialog.html('<div class="crm-loading-element"></div>');
+          CRM.api3('contact', 'get', {group: scope.testGroup.gid, options: {limit: 0}, return: 'display_name,email'}).done(function(data) {
+            var count = 0,
+              // Fixme: should this be in a template?
+              markup = '<ol>';
+            _.each(data.values, function(row) {
+              // Fixme: contact api doesn't seem capable of filtering out contacts with no email, so we're doing it client-side
+              if (row.email) {
+                count++;
+                markup += '<li>' + row.display_name + ' - ' + row.email + '</li>';
+              }
+            });
+            markup += '</ol>';
+            $dialog.html('<h4>' + ts('A test message will be sent to %1 people:', {1: count}) + '</h4>' + markup).trigger('crmLoad');
+          });
+        };
       }
     };
   });
-- 
2.25.1