Mapping Billing Invoice Code field to campaigns, allowing custom QA f…

…unctions to be configured in the sheet, mapping new fields to allow creation of tracking creatives in bulk

Change-Id: Ie680dc42529f745638d62014697d109dbdddc913

.gitignore

@@ -1 +1,2 @@
+clientSpecific*.js
 .clasp.json

BulkdozerQA.html

@@ -19,7 +19,7 @@
   <br />
   <button id='clearFeed' onclick='clearFeed()'>Clear Feed</button><br />
   <button id='loadFromCM' onclick='qaCMLoad()'>Load from CM</button><br />
-  <button id='pushToCM' onclick='pushToCM()'>Push to CM</button><br />
+  <!--<button id='pushToCM' onclick='pushToCM()'>Push to CM</button><br />-->
   <?!= include('runner'); ?>
   <?!= include('sidebar'); ?>
   <?!= include('logger'); ?>

Code.js

@@ -22,6 +22,11 @@
 const DEFAULT_SLEEP = 8 * 1000;
 const DEFAULT_RETRIES = 4;
 
+// Declare context object so we can call functions by name, this enables
+// configuration based functionality, so the tool behaves according to settings
+// defined in the sheet.
+var context = this;
+
 /**
  * onOpen handler to display Bulkdozer menu
  */

Fields.js

@@ -41,6 +41,7 @@ var fields = {
   'campaignName': 'Campaign Name',
   'campaignStartDate': 'Campaign Start Date',
   'campaignEndDate': 'Campaign End Date',
+  'billingInvoiceCode': 'Billing Invoice Code',
 
   // Event Tag
   'eventTagId': 'Event Tag ID',
@@ -89,6 +90,7 @@ var fields = {
   'creativeType': 'Creative Type',
   'thirdPartyUrlType': '3P URL Type',
   'thirdPartyUrl': '3P URL',
+  'creativeActive': 'Creative Active',
 
   // Ad
   'creativeRotation': 'Creative Rotation',

Loaders.js

@@ -19,10 +19,6 @@
 *
 ***************************************************************************/
 
-// Declare context object so we can call constructors by name, this is used so
-// we can define which concrete loader to use based on sheet configurations
-var context = this;
-
 /*
  * Gets the profile id from the Store tab
  *
@@ -786,6 +782,7 @@ var CampaignLoader = function(cmDAO) {
     feedItem[fields.landingPageName] = landingPage.name;
     feedItem[fields.campaignStartDate] = campaign.startDate;
     feedItem[fields.campaignEndDate] = campaign.endDate;
+    feedItem[fields.billingInvoiceCode] = campaign.billingInvoiceCode;
 
     return feedItem;
   };
@@ -1786,17 +1783,31 @@ var CreativeLoader = function(cmDAO) {
     if(feedItem[fields.creativeName]) {
       creative.name = feedItem[fields.creativeName];
     }
+
+    if(feedItem[fields.advertiserId]) {
+      creative.advertiserId = feedItem[fields.advertiserId];
+    }
+
+    if(feedItem[fields.creativeType]) {
+      creative.type = feedItem[fields.creativeType];
+    }
+
+    if(feedItem[fields.creativeActive]) {
+      creative.active = feedItem[fields.creativeActive];
+    }
   }
 
   /**
    * @see CampaignLoader.postProcessPush
    */
   this.postProcessPush = function(job) {
-    var campaign = cmDAO.get('Campaigns', job.feedItem[fields.campaignId]);
+    if(job.feedItem[fields.campaignId]) {
+      var campaign = cmDAO.get('Campaigns', job.feedItem[fields.campaignId]);
 
-    cmDAO.associateCreativeToCampaign(campaign.id, job.cmObject.id);
+      cmDAO.associateCreativeToCampaign(campaign.id, job.cmObject.id);
 
-    job.feedItem[fields.campaignName] = campaign.name;
+      job.feedItem[fields.campaignName] = campaign.name;
+    }
   }
 }
 CreativeLoader.prototype = Object.create(BaseLoader.prototype);

SidebarController.js

@@ -134,15 +134,9 @@ function cmFetch(job) {
 function _qa(job) {
   doBuildHierarchy(job);
 
-  var qaMode = getSheetDAO().getValue('Store', 'B3');
+  var qaFunctionName = getSheetDAO().getValue('Store', 'B3');
 
-  if(qaMode == 'Aggregated Creative Rotation') {
-    qaByAdAggregatedCreativeRotation(job);
-  } else if (qaMode == 'Landing Page') {
-    qaLandingPage(job);
-  } else {
-    qaByCreativeRotation(job);
-  }
+  context[qaFunctionName](job);
 
   // This is needed for large campaigns, it is too much data to transmit to the
   // front end

sidebar.html

@@ -206,11 +206,13 @@
  *  returns false.
  */
 function checkEntityMode(entityConfigs, entity, mode) {
-  if(entityConfigs && entityConfigs[entity].indexOf('WRITE') != -1) {
-      return true;
-  } else if(entityConfigs[entity].indexOf('READ') != -1) {
-    if(mode == 'READ') {
-      return true;
+  if(entityConfigs && entityConfigs[entity]) {
+    if(entityConfigs[entity].indexOf('WRITE') != -1) {
+        return true;
+    } else if(entityConfigs[entity].indexOf('READ') != -1) {
+      if(mode == 'READ') {
+        return true;
+      }
     }
   }