From aafc090b2d697a9c2fac785b6bc252e8390590e3 Mon Sep 17 00:00:00 2001
From: Tim Otten <totten@civicrm.org>
Date: Sun, 14 Dec 2014 13:49:41 -0800
Subject: [PATCH] CRM-15578 - crmMailingAB2 - Make crmMailingAbBlockMailing
 more consistent+flexible

---
 js/angular-crmMailingAB2-services.js      |  9 ++-
 partials/crmMailingAB2/edit.html          |  3 +-
 partials/crmMailingAB2/joint-mailing.html | 78 ++++++++++++++++++++++-
 3 files changed, 87 insertions(+), 3 deletions(-)

diff --git a/js/angular-crmMailingAB2-services.js b/js/angular-crmMailingAB2-services.js
index a26897100d..33d489c819 100644
--- a/js/angular-crmMailingAB2-services.js
+++ b/js/angular-crmMailingAB2-services.js
@@ -18,7 +18,14 @@
     };
   });
 
-  // CrmMailingAB is a data-model which combines an AB test (APIv3 "MailingAB") and three mailings (APIv3 "Mailing").
+  // CrmMailingAB is a data-model which combines an AB test (APIv3 "MailingAB"), three mailings (APIv3 "Mailing"),
+  // and three sets of attachments (APIv3 "Attachment").
+  //
+  // example:
+  //   var abtest = new CrmMailingAB(123);
+  //   abtest.load().then(function(){
+  //     alert("Mailing A is named "+abtest.mailings.a.name);
+  //   });
   angular.module('crmMailingAB2').factory('CrmMailingAB', function (crmApi, crmMailingMgr, $q, CrmAttachments) {
     function CrmMailingAB(id) {
       this.id = id;
diff --git a/partials/crmMailingAB2/edit.html b/partials/crmMailingAB2/edit.html
index d9eaab3c1f..e2ed4bd3a5 100644
--- a/partials/crmMailingAB2/edit.html
+++ b/partials/crmMailingAB2/edit.html
@@ -43,7 +43,8 @@
                 msg_template_id: 1,
                 fromAddressA: 1,
                 fromAddressB: 1,
-                replyTo: 1,
+                replyToA: 1,
+                replyToB: 1,
                 subject: 1
                 }"
               crm-abtest="abtest"></div>
diff --git a/partials/crmMailingAB2/joint-mailing.html b/partials/crmMailingAB2/joint-mailing.html
index 18be406363..8836c34295 100644
--- a/partials/crmMailingAB2/joint-mailing.html
+++ b/partials/crmMailingAB2/joint-mailing.html
@@ -1,10 +1,20 @@
 <!--
-Required vars: abtest
+Required vars: abtest, fields
+
 Note: Much of this file is duplicated in crmMailing and crmMailingAB with variations on placement/title/binding.
 It could perhaps be thinned by 30-60% by making more directives.
+
+This template follows a basic pattern. For each included field, there are three variants, as in this example:
+ - fromAddress: The default From: address shared by both mailings (representatively mapped to mailing A)
+ - fromAddressA: The From: address for mailing A
+ - fromAddressB: The From: address for mailing B
+Each variant is guarded with "ng-if='fields.fieldName'"; if true, the field will be displayed and
+processed by Angular; if false, the field will be hidden and completely ignored by Angular.
 -->
 <div class="crm-block" ng-form="subform" crm-ui-id-scope>
   <div class="crm-group">
+
+
     <div crm-ui-field="subform.msg_template_id" crm-title="ts('Template')" style="background: #bbf; width:100%; padding: 0.1em;" ng-if="fields.msg_template_id">
       <div ng-controller="MsgTemplateCtrl">
         <select
@@ -20,6 +30,38 @@ It could perhaps be thinned by 30-60% by making more directives.
         <a ng-click="saveTemplate(abtest.mailings.a)" class="crm-hover-button action-item" title="{{ts('Save As')}}"><span class="icon ui-icon-disk"></span></a>
       </div>
     </div>
+    <div crm-ui-field="subform.msg_template_idA" crm-title="ts('Template (A)')" style="background: #bbf; width:100%; padding: 0.1em;" ng-if="fields.msg_template_idA">
+      <div ng-controller="MsgTemplateCtrl">
+        <select
+          crm-ui-id="subform.msg_template_idA"
+          name="msg_template_idA"
+          crm-ui-select="{dropdownAutoWidth : true, allowClear: true, placeholder: ts('Message Template')}"
+          ng-model="abtest.mailings.a.msg_template_id"
+          ng-change="loadTemplate(abtest.mailings.a, abtest.mailings.a.msg_template_id)"
+          >
+          <option value=""></option>
+          <option ng-repeat="frm in crmMsgTemplates.getAll() | orderBy:'msg_title'" ng-value="frm.id">{{frm.msg_title}}</option>
+        </select>
+        <a ng-click="saveTemplate(abtest.mailings.a)" class="crm-hover-button action-item" title="{{ts('Save As')}}"><span class="icon ui-icon-disk"></span></a>
+      </div>
+    </div>
+    <div crm-ui-field="subform.msg_template_idB" crm-title="ts('Template (B)')" style="background: #bbf; width:100%; padding: 0.1em;" ng-if="fields.msg_template_idB">
+      <div ng-controller="MsgTemplateCtrl">
+        <select
+          crm-ui-id="subform.msg_template_idB"
+          name="msg_template_idB"
+          crm-ui-select="{dropdownAutoWidth : true, allowClear: true, placeholder: ts('Message Template')}"
+          ng-model="abtest.mailings.b.msg_template_id"
+          ng-change="loadTemplate(abtest.mailings.b, abtest.mailings.b.msg_template_id)"
+          >
+          <option value=""></option>
+          <option ng-repeat="frm in crmMsgTemplates.getAll() | orderBy:'msg_title'" ng-value="frm.id">{{frm.msg_title}}</option>
+        </select>
+        <a ng-click="saveTemplate(abtest.mailings.b)" class="crm-hover-button action-item" title="{{ts('Save As')}}"><span class="icon ui-icon-disk"></span></a>
+      </div>
+    </div>
+
+
     <div crm-ui-field="subform.fromAddress" crm-title="ts('From')" ng-if="fields.fromAddress">
       <span ng-controller="EmailAddrCtrl" crm-mailing-from-address="fromPlaceholder" crm-mailing="abtest.mailings.a">
         <select
@@ -62,6 +104,8 @@ It could perhaps be thinned by 30-60% by making more directives.
         </select>
       </span>
     </div>
+
+
     <div crm-ui-field="subform.replyTo" crm-title="ts('Reply-To')" ng-show="crmMailingConst.enableReplyTo" ng-if="fields.replyTo">
       <span ng-controller="EmailAddrCtrl">
         <select
@@ -76,6 +120,36 @@ It could perhaps be thinned by 30-60% by making more directives.
         </select>
       </span>
     </div>
+    <div crm-ui-field="subform.replyToA" crm-title="ts('Reply-To (A)')" ng-show="crmMailingConst.enableReplyTo" ng-if="fields.replyToA">
+      <span ng-controller="EmailAddrCtrl">
+        <select
+          crm-ui-id="subform.replyToA"
+          name="replyToA"
+          ui-jq="select2"
+          ui-options="{dropdownAutoWidth : true, allowClear: true, placeholder: ts('Email address')}"
+          ng-model="abtest.mailings.a.replyto_email"
+          ng-options="frm.label as frm.label for frm in crmFromAddresses.getAll() | filter:{is_active:1} | orderBy:'weight'"
+          >
+          <option value=""></option>
+        </select>
+      </span>
+    </div>
+    <div crm-ui-field="subform.replyToB" crm-title="ts('Reply-To (B)')" ng-show="crmMailingConst.enableReplyTo" ng-if="fields.replyToB">
+      <span ng-controller="EmailAddrCtrl">
+        <select
+          crm-ui-id="subform.replyToB"
+          name="replyToB"
+          ui-jq="select2"
+          ui-options="{dropdownAutoWidth : true, allowClear: true, placeholder: ts('Email address')}"
+          ng-model="abtest.mailings.b.replyto_email"
+          ng-options="frm.label as frm.label for frm in crmFromAddresses.getAll() | filter:{is_active:1} | orderBy:'weight'"
+          >
+          <option value=""></option>
+        </select>
+      </span>
+    </div>
+
+
     <!--
     <div crm-ui-field="subform.recipients" crm-title="ts('Recipients')">
       <div ng-controller="EditRecipCtrl">
@@ -102,6 +176,8 @@ It could perhaps be thinned by 30-60% by making more directives.
       </div>
     </div>
     -->
+
+
     <div crm-ui-field="subform.subject" crm-title="ts('Subject')" ng-if="fields.subject">
       <div style="float: right;">
         <input crm-mailing-token crm-for="subject" />
-- 
2.25.1