From 3ad2ea698caa0c3d6286274702e75e2bd4267bdd Mon Sep 17 00:00:00 2001
From: Tim Otten <totten@civicrm.org>
Date: Thu, 19 Feb 2015 14:15:13 -0800
Subject: [PATCH] CRM-15979 - crmMailingAB - More symmetry between edit and
 report modes.

The top level file is "main.html" which includes either "edit.html" or
"report.html" depending on whether the test has been submitted.
---
 js/angular-crmMailingAB.js        | 12 +-----------
 partials/crmMailingAB/edit.html   | 17 +++++------------
 partials/crmMailingAB/main.html   | 12 ++++++++++++
 partials/crmMailingAB/report.html |  3 +++
 4 files changed, 21 insertions(+), 23 deletions(-)
 create mode 100644 partials/crmMailingAB/main.html

diff --git a/js/angular-crmMailingAB.js b/js/angular-crmMailingAB.js
index 97251357b9..0dd847c310 100644
--- a/js/angular-crmMailingAB.js
+++ b/js/angular-crmMailingAB.js
@@ -29,7 +29,7 @@
         }
       });
       $routeProvider.when('/abtest/:id', {
-        templateUrl: '~/crmMailingAB/edit.html',
+        templateUrl: '~/crmMailingAB/main.html',
         controller: 'CrmMailingABEditCtrl',
         resolve: {
           abtest: function ($route, CrmMailingAB) {
@@ -38,16 +38,6 @@
           }
         }
       });
-      $routeProvider.when('/abtest/:id/report', {
-        templateUrl: '~/crmMailingAB/report.html',
-        controller: 'CrmMailingABReportCtrl',
-        resolve: {
-          abtest: function ($route, CrmMailingAB) {
-            var abtest = new CrmMailingAB($route.current.params.id);
-            return abtest.load();
-          }
-        }
-      });
     }
   ]);
 
diff --git a/partials/crmMailingAB/edit.html b/partials/crmMailingAB/edit.html
index 3def3ebbea..357d96fcc6 100644
--- a/partials/crmMailingAB/edit.html
+++ b/partials/crmMailingAB/edit.html
@@ -1,20 +1,13 @@
-<div crm-ui-accordion crm-title="ts('Debug')" crm-collapsed="true">
-  <pre>{{abtest.ab|json}}</pre>
-  <pre>{{abtest.mailings|json}}</pre>
-</div>
-
 <!--
+  Implicit Controller: CrmMailingABEditCtrl
+
   An ABTest includes two mailings, but we don't require the user to enter two complete mailings. For
   simplicity, the email composition UI generally displays A (unless we specifically decided to expose an
   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-if="isSubmitted()">
-  <div ng-include="'~/crmMailingAB/report.html'"></div>
-</div>
-
-<form name="crmMailingAB" novalidate>
-  <div class="crm-block crm-form-block crmMailing" ng-if="!isSubmitted()">
+<div ng-form="crmMailingABEdit">
+  <div class="crm-block crm-form-block crmMailing">
     <div crm-ui-wizard>
       <div crm-ui-wizard-step="10" crm-title="ts('Setup')" ng-form="setupForm">
         <div
@@ -180,4 +173,4 @@
         <button crm-icon="disk" ng-disabled="block.check()" ng-click="save().then(leave) ">{{ts('Save Draft')}}</button>
       </span>
     </div>
-</form>
+</div>
diff --git a/partials/crmMailingAB/main.html b/partials/crmMailingAB/main.html
new file mode 100644
index 0000000000..668df76369
--- /dev/null
+++ b/partials/crmMailingAB/main.html
@@ -0,0 +1,12 @@
+<!--
+  Implicit Controller: CrmMailingABEditCtrl
+-->
+<div crm-ui-accordion crm-title="ts('Debug')" crm-collapsed="true">
+  <pre>{{abtest.ab|json}}</pre>
+  <pre>{{abtest.mailings|json}}</pre>
+</div>
+
+<form name="crmMailingAB" novalidate>
+  <div ng-include="'~/crmMailingAB/edit.html'" ng-if="!isSubmitted()"></div>
+  <div ng-include="'~/crmMailingAB/report.html'" ng-if="isSubmitted()"></div>
+</form>
diff --git a/partials/crmMailingAB/report.html b/partials/crmMailingAB/report.html
index 0e6d4cbc38..5cde438664 100644
--- a/partials/crmMailingAB/report.html
+++ b/partials/crmMailingAB/report.html
@@ -1,3 +1,6 @@
+<!--
+  Implicit Controller: CrmMailingABEditCtrl
+-->
 <div ng-controller="CrmMailingABReportCtrl">
   <table>
     <thead>
-- 
2.25.1