Merge pull request #2525 from adaptlearning/issue/2510

Remove build includes after build

Author: tomgreenfield

lib/outputmanager.js

@@ -266,9 +266,6 @@ OutputPlugin.prototype.sanitizeCourseJSON = function(mode, json, next) {
 
     async.waterfall([
       function(callback) {
-        if (mode === Constants.Modes.Export) {
-          return callback();
-        }
         self.generateIncludesForConfig(configJson, function(error, includes) {
           if (error) {
             return callback(error);
@@ -926,6 +923,18 @@ OutputPlugin.prototype.applyMenu = function(tenantId, courseId, jsonObject, dest
   }
 };
 
+OutputPlugin.prototype.removeBuildIncludes = async (configPath, next) => {
+  try {
+    const config = await fs.readJson(configPath);
+    await fs.writeJson(configPath, config, { spaces: 2, replacer: (key, value) => {
+      if (key !== 'build') return value;
+    }});
+    next(null);
+  } catch (err) {
+    next(err);
+  }
+}
+
 /**
  * extending plugins must implement this
  *

plugins/output/adapt/publish.js

@@ -223,6 +223,10 @@ function publishCourse(courseId, mode, request, response, next) {
         callback(null);
       });
     },
+    function(callback) {
+      const configPath = path.join(BUILD_FOLDER, Constants.Folders.Course, Constants.CourseCollections.config.filename);
+      self.removeBuildIncludes(configPath, err => callback(err));
+    },
     function(callback) {
       if (mode === Constants.Modes.Preview) { // No download required -- skip this step
         return callback();