From 57194571216ee0074cbdc928bcd84f175aee9abb Mon Sep 17 00:00:00 2001
From: Tim Otten <totten@civicrm.org>
Date: Wed, 18 Feb 2015 10:15:27 -0800
Subject: [PATCH] CRM-15979 - crmMailingAB - Merge the compose and report URLs

---
 js/angular-crmMailingAB.js        | 18 ++++++++----------
 partials/crmMailingAB/edit.html   |  8 ++++----
 partials/crmMailingAB/list.html   |  2 +-
 partials/crmMailingAB/report.html |  7 +------
 4 files changed, 14 insertions(+), 21 deletions(-)

diff --git a/js/angular-crmMailingAB.js b/js/angular-crmMailingAB.js
index 0cacb6c974..9379970845 100644
--- a/js/angular-crmMailingAB.js
+++ b/js/angular-crmMailingAB.js
@@ -172,20 +172,18 @@
     });
   });
 
-  angular.module('crmMailingAB').controller('CrmMailingABReportCtrl', function ($scope, abtest, crmApi, crmMailingPreviewMgr, dialogService) {
+  angular.module('crmMailingAB').controller('CrmMailingABReportCtrl', function ($scope, crmApi, crmMailingPreviewMgr, dialogService) {
     var ts = $scope.ts = CRM.ts(null);
 
-    $scope.abtest = abtest;
-
     $scope.stats = {};
-    crmApi('Mailing', 'stats', {mailing_id: abtest.ab.mailing_id_a}).then(function(data){
-      $scope.stats.a = data.values[abtest.ab.mailing_id_a];
+    crmApi('Mailing', 'stats', {mailing_id: $scope.abtest.ab.mailing_id_a}).then(function(data){
+      $scope.stats.a = data.values[$scope.abtest.ab.mailing_id_a];
     });
-    crmApi('Mailing', 'stats', {mailing_id: abtest.ab.mailing_id_b}).then(function(data){
-      $scope.stats.b = data.values[abtest.ab.mailing_id_b];
+    crmApi('Mailing', 'stats', {mailing_id: $scope.abtest.ab.mailing_id_b}).then(function(data){
+      $scope.stats.b = data.values[$scope.abtest.ab.mailing_id_b];
     });
-    crmApi('Mailing', 'stats', {mailing_id: abtest.ab.mailing_id_c}).then(function(data){
-      $scope.stats.c = data.values[abtest.ab.mailing_id_c];
+    crmApi('Mailing', 'stats', {mailing_id: $scope.abtest.ab.mailing_id_c}).then(function(data){
+      $scope.stats.c = data.values[$scope.abtest.ab.mailing_id_c];
     });
 
     $scope.previewMailing = function previewMailing(mailingName, mode) {
@@ -193,7 +191,7 @@
     };
     $scope.selectWinner = function selectWinner(mailingName) {
       var model = {
-        abtest: abtest,
+        abtest: $scope.abtest,
         mailingName: mailingName
       };
       var options = CRM.utils.adjustDialogDefaults({
diff --git a/partials/crmMailingAB/edit.html b/partials/crmMailingAB/edit.html
index aafb88fb6c..74c1fc3b95 100644
--- a/partials/crmMailingAB/edit.html
+++ b/partials/crmMailingAB/edit.html
@@ -9,12 +9,12 @@
   individual field from B). At the end of the composition process, the controller's "sync" operation will
   merge shared settings from "A" into "B".
 -->
-<div ng-show="isSubmitted()">
-  {{ts('This mailing has been submitted.')}}
+<div ng-if="isSubmitted()">
+  <div ng-include="'~/crmMailingAB/report.html'"></div>
 </div>
 
-<form name="crmMailingAB" novalidate ng-hide="isSubmitted()" crm-autosave="save()" crm-autosave-model="abtest.getAutosaveSignature()" crm-autosave-if="sync()">
-  <div class="crm-block crm-form-block crmMailing">
+<form name="crmMailingAB" novalidate crm-autosave="save()" crm-autosave-model="abtest.getAutosaveSignature()" crm-autosave-if="sync()">
+  <div class="crm-block crm-form-block crmMailing" ng-if="!isSubmitted()">
     <div crm-ui-wizard>
       <div crm-ui-wizard-step="10" crm-title="ts('Setup')" ng-form="setupForm">
         <div
diff --git a/partials/crmMailingAB/list.html b/partials/crmMailingAB/list.html
index 66b35b6996..361f9c867c 100644
--- a/partials/crmMailingAB/list.html
+++ b/partials/crmMailingAB/list.html
@@ -44,7 +44,7 @@ Required vars: mailingABList
       <td>{{mailingAB.created_date}}</td>
       <td>
         <a class="action-item crm-hover-button" ng-href="#/abtest/{{mailingAB.id}}" ng-show="mailingAB.status == 'Draft'">{{ts('Continue')}}</a>
-        <a class="action-item crm-hover-button" ng-href="#/abtest/{{mailingAB.id}}/report" ng-show="mailingAB.status != 'Draft'">{{ts('Results')}}</a>
+        <a class="action-item crm-hover-button" ng-href="#/abtest/{{mailingAB.id}}" ng-show="mailingAB.status != 'Draft'">{{ts('Results')}}</a>
       </td>
     </tr>
     </tbody>
diff --git a/partials/crmMailingAB/report.html b/partials/crmMailingAB/report.html
index 2b70a987a8..4913630e07 100644
--- a/partials/crmMailingAB/report.html
+++ b/partials/crmMailingAB/report.html
@@ -1,9 +1,4 @@
-<div crm-ui-accordion crm-title="ts('Debug')" crm-collapsed="true">
-  <pre>{{abtest.ab|json}}</pre>
-  <pre>{{abtest.mailings|json}}</pre>
-</div>
-
-<div>
+<div ng-controller="CrmMailingABReportCtrl">
   <table>
     <thead>
     <tr>
-- 
2.25.1