Merge pull request #2537 from adaptlearning/release/bugpatch

Release 0.10.4

Author: tomgreenfield

CHANGELOG.md

@@ -5,6 +5,30 @@ All notable changes to the Adapt authoring tool are documented in this file.
 **IMPORTANT**: For information on how to **correctly and safely** update your installation, please consult **INSTALL.md**.<br/>
 _Note that we adhere to the [semantic versioning](http://semver.org/) scheme for release numbering._
 
+## [0.10.4] - 2020-08-21
+
+Bugfix release.
+
+### Fixed
+- Copied articles sometimes get pasted into the wrong place ([#1892](https://github.com/adaptlearning/adapt_authoring/issues/1892))
+- Install breaks if any config values contain true/false ([#2133](https://github.com/adaptlearning/adapt_authoring/issues/2133))
+- Upgrade script replaces all core-bundled plugins ([#2316](https://github.com/adaptlearning/adapt_authoring/issues/2316))
+- Copied and pasted pages have out of order articles ([#2404](https://github.com/adaptlearning/adapt_authoring/issues/2404))
+- Tooltip overflow in sidebar 'modal' ([#2414](https://github.com/adaptlearning/adapt_authoring/issues/2414))
+- Error: Failed to determine user's tenant! ([#2504](https://github.com/adaptlearning/adapt_authoring/issues/2504))
+- Export fails as it requires LESS variables for plugins that aren't in the project ([#2510](https://github.com/adaptlearning/adapt_authoring/issues/2510))
+- Top navigation obscures save button on small window sizes ([#2511](https://github.com/adaptlearning/adapt_authoring/issues/2511))
+- Support Node 14 ([#2514](https://github.com/adaptlearning/adapt_authoring/issues/2514))
+- Reset not working ([#2518](https://github.com/adaptlearning/adapt_authoring/issues/2518))
+- Wrong state for display title icon ([#2523](https://github.com/adaptlearning/adapt_authoring/issues/2523))
+- Get 500 error when copying items with assets in Project Settings. Can also break courseassets in other items. ([#2534](https://github.com/adaptlearning/adapt_authoring/issues/2534))
+- default FW -> AT import issues - PLP settings ([adaptlearning/adapt_framework#2841](https://github.com/adaptlearning/adapt_framework/issues/2841))
+
+### Added
+- Option to customise the 'contact support' message with details of who to contact ([#2477](https://github.com/adaptlearning/adapt_authoring/issues/2477))
+- Remove 'field-help' icon if there isn't any 'help' text ([#2509](https://github.com/adaptlearning/adapt_authoring/issues/2509))
+- Customise help link ([#2527](https://github.com/adaptlearning/adapt_authoring/issues/2527))
+
 ## [0.10.3] - 2020-02-21
 
 Bugfix release.
@@ -687,6 +711,7 @@ Initial release.
 - Loading screen of death
 - Session cookie security issues
 
+[0.10.4]: https://github.com/adaptlearning/adapt_authoring/compare/v0.10.3...v0.10.4
 [0.10.3]: https://github.com/adaptlearning/adapt_authoring/compare/v0.10.2...v0.10.3
 [0.10.2]: https://github.com/adaptlearning/adapt_authoring/compare/v0.10.1...v0.10.2
 [0.10.1]: https://github.com/adaptlearning/adapt_authoring/compare/v0.10.0...v0.10.1

frontend/src/core/helpers.js

@@ -326,6 +326,11 @@ define(function(require){
         }
       }
       return flatProperties;
+    },
+
+    importConstants: function() {
+      this.constants = Origin.constants;
+      return '';
     }
   };
 

frontend/src/core/models/blockModel.js

@@ -10,28 +10,14 @@ define(function(require) {
     // Block specific properties
     layoutOptions:  null,
     dragLayoutOptions: null,
-    // These are the only attributes which should be permitted on a save
-    // TODO look into this...
-    whitelistAttributes: [
-      '_id',
-      '_courseId',
-      '_parentId',
-      '_layoutOptions',
-      '_type',
-      '_sortOrder',
-      '_classes',
-      '_isOptional',
-      '_isAvailable',
-      'body',
-      'displayTitle',
-      'title',
-      '_extensions',
-      'themeSettings',
-      '_onScreen',
-      '_isVisible',
-      '_isHidden',
-      'instruction',
-      '_colorLabel'
+    attributeBlacklist: [
+      '_isDeleted',
+      '_tenantId',
+      '_trackingId',
+      'createdBy',
+      'createdAt',
+      'layoutOptions',
+      'updatedAt'
     ]
   });
 

frontend/src/core/models/componentModel.js

@@ -6,31 +6,11 @@ define(function(require) {
     urlRoot: 'api/content/component',
     _parentType: 'block',
     _siblingTypes: 'component',
-    // These are the only attributes which should be permitted on a save
-    // TODO look into this...
-    whitelistAttributes: [
-      '_id',
-      '_componentType',
-      '_courseId',
-      '_layout',
-      '_parentId',
-      '_type',
-      'properties',
-      '_component',
-      '_extensions',
-      '_classes',
-      '_isOptional',
-      '_isAvailable',
-      'body',
-      'displayTitle',
-      'title',
-      'version',
-      'themeSettings',
-      '_onScreen',
-      '_isVisible',
-      '_isHidden',
-      'instruction',
-      '_colorLabel'
+    attributeBlacklist: [
+      '_isDeleted',
+      'createdBy',
+      'createdAt',
+      'updatedAt'
     ]
   });
 

frontend/src/core/models/contentModel.js

@@ -7,7 +7,7 @@ define(function(require) {
 
   var ContentModel = Backbone.Model.extend({
     idAttribute: '_id',
-    whitelistAttributes: null,
+    attributeBlacklist: null,
 
     initialize: function(options) {
       this.on('sync', this.loadedData, this);
@@ -89,17 +89,13 @@ define(function(require) {
       return JSON.stringify(this);
     },
 
-    // Remove any attributes which are not on the whitelist (called before a save)
     pruneAttributes: function() {
-      var self = this;
-      // Ensure that only valid attributes are pushed back on the save
-      if (self.whitelistAttributes) {
-        _.each(_.keys(self.attributes), function(key) {
-          if (!_.contains(self.whitelistAttributes, key)) {
-            self.unset(key);
-          }
-        });
-      }
+      if (!this.attributeBlacklist) return;
+      Object.keys(this.attributes).forEach(function(key) {
+        if (_.contains(this.attributeBlacklist, key)) {
+          this.unset(key);
+        }
+      }, this);
     }
   });
 

frontend/src/modules/editor/contentObject/views/editorPageArticleView.js

@@ -48,7 +48,7 @@ define(function(require){
         var events = {};
         events['editorView:moveBlock:' + id] = this.render;
         events['editorView:deleteArticle:' + id] = this.deletePageArticle;
-        events['editorView:pasted:' + id] = this.onPaste;
+        events['editorView:pasted:' + id] = this.render;
         this.listenTo(Origin, events);
       }
 
@@ -282,20 +282,6 @@ define(function(require){
         duration = 0;
       }
       this.$('.article-content').velocity(shouldCollapse ? 'slideUp' : 'slideDown', duration);
-    },
-
-    onPaste: function(data) {
-      (new BlockModel({ _id: data._id })).fetch({
-        success: _.bind(function(model) {
-          this.addBlockView(model);
-        }, this),
-        error: function(data) {
-          Origin.Notify.alert({
-            type: 'error',
-            text: 'app.errorfetchingdata'
-          });
-        }
-      });
     }
 
   }, {

frontend/src/modules/editor/contentObject/views/editorPageView.js

@@ -27,7 +27,7 @@ define(function(require){
         'pageView:itemAnimated': this.evaluateChildStatus
       };
       originEvents['editorView:moveArticle:' + id] = this.render;
-      originEvents['editorView:pasted:' + id] = this.onPaste;
+      originEvents['editorView:pasted:' + id] = this.render;
       this.listenTo(Origin, originEvents);
 
       Origin.options.addItems([
@@ -109,7 +109,7 @@ define(function(require){
         $.scrollTo(newArticleView.$el, 200);
       }
       // Increment the 'sortOrder' property
-      articleModel.set('_pasteZoneSortOrder', sortOrder++);
+      articleModel.set('_pasteZoneSortOrder', sortOrder + 1);
       // Post-article paste zone - sort order of placeholder will be one greater
       this.$('.page-articles').append(new EditorPasteZoneView({ model: articleModel }).$el);
       return newArticleView;
@@ -159,20 +159,6 @@ define(function(require){
       Origin.trigger('contextMenu:open', fakeView, event);
     },
 
-    onPaste: function(data) {
-      (new ArticleModel({ _id: data._id })).fetch({
-        success: _.bind(function(model) {
-          this.addArticleView(model);
-        }, this),
-        error: function(data) {
-          Origin.Notify.alert({
-            type: 'error',
-            text: 'app.errorfetchingdata'
-          });
-        }
-      });
-    },
-
     onCutArticle: function(view) {
       this.once('pageView:postRender', view.showPasteZones);
       this.render();

frontend/src/modules/help/index.js

@@ -2,7 +2,12 @@ define(function(require) {
   var Origin = require('core/origin');
 
   Origin.on('navigation:help', function() {
-    openWikiLink(getLink());
+    var override = Origin.constants.supportLink;
+    if (override) {
+      window.open(override);
+    } else {
+      openWikiLink(getLink());
+    }
   });
 
   function getLink() {

frontend/src/modules/navigation/less/navigation.less

@@ -46,15 +46,13 @@
 }
 
 .navigation-left {
-	width:50%;
 	float:left;
   .navigation-product-name {
     .no-select;
   }
 }
 
 .navigation-right {
-	width:47%;
   padding-right: 20px;
 	float:right;
 	text-align:right;

frontend/src/modules/scaffold/less/displayTitle.less

@@ -4,7 +4,7 @@
     display: none;
   }
   .field-editor input {
-    width: 81%;
+    width: calc(90% - 80px);
   }
 }