@eclipse-scout/core 22.0.12 → 22.0.22

package/dist/file-list

@@ -1,9 +1,9 @@
-eclipse-scout-core-af5ff6abad5f1ff9fd18.min.js
-eclipse-scout-core-af5ff6abad5f1ff9fd18.min.js.map
-eclipse-scout-core-theme-74b63e0d57bed407a729.min.css
-eclipse-scout-core-theme-dark-b82aea152f416e38ce7f.min.css
+eclipse-scout-core-c4cad26b336b02535421.min.js
+eclipse-scout-core-c4cad26b336b02535421.min.js.map
+eclipse-scout-core-theme-dark-fd0e080c10f65e67b68b.min.css
 eclipse-scout-core-theme-dark.css
 eclipse-scout-core-theme-dark.css.map
+eclipse-scout-core-theme-f3a61fbc12acf8e27fcc.min.css
 eclipse-scout-core-theme.css
 eclipse-scout-core-theme.css.map
 eclipse-scout-core.js

package/dist/texts.json

@@ -188,19 +188,19 @@
     "ConfirmApplyChanges": "Voulez-vous appliquer les modifications?",
     "ErrorWhileLoadingData": "Erreur lors du chargement des données",
     "FormEmptyMandatoryFieldsMessage": "Les champs suivants doivent être remplis:",
-    "FormInvalidFieldsMessage": "Les champs suivants ont une valeur non valide:",
-    "FormSaveChangesQuestion": "Voulez-vous enregistrer les modifications?",
+    "FormInvalidFieldsMessage": "Les champs suivants ont une valeur invalide:",
+    "FormSaveChangesQuestion": "Est-ce que vous voulez enregistrer les modifications ?",
     "FormValidationFailedTitle": "Veuillez vérifier les données.",
-    "FormsCannotBeSaved": "Les masques suivants ne peuvent pas être enregistrés, car ils se trouvent dans un statut non valide:",
+    "FormsCannotBeSaved": "Les formulaires suivants ne peuvent pas être enregistrés car ils se trouvent dans un statut non valide:",
     "GroupBy": "Grouper par",
-    "InactiveState": "inactif",
+    "InactiveState": "Inactif",
     "InvalidNumberMessageX": "\"{0}\" est un chiffre non valide,",
     "InvalidValueMessageX": "\"{0}\" est une valeur non valide.",
     "NavigationBackward": "Retour",
     "No": "Non",
     "NoButton": "Non",
     "NoGrouping": "Aucun groupement",
-    "NotAllCheckedFormsCanBeSaved": "Tous les formulaires sélectionnés ne peuvent pas être enregistrés",
+    "NotAllCheckedFormsCanBeSaved": "Tous les formulaires sélectionnés n'ont pas pu être enregistrés",
     "NumberTooLargeMessageX": "La valeur est trop grande; doit être inférieure ou égale à {0}.",
     "NumberTooLargeMessageXY": "La valeur est trop grande; doit être comprise entre {0} et {1}.",
     "NumberTooSmallMessageX": "La valeur est trop petite; doit être supérieure ou égale à {0}.",
@@ -246,7 +246,7 @@
     "ui.CloseAllTabs": "Fermer tous les onglets",
     "ui.CloseOtherTabs": "Fermer les autres onglets",
     "ui.CodeUndefined": "[Inconnu]",
-    "ui.CollapseAll": "fermer tous",
+    "ui.CollapseAll": "tout fermer",
     "ui.Coloring": "Coloration",
     "ui.Column": "Colonne",
     "ui.ConnectionInterrupted": "Connexion interrompue",
@@ -261,7 +261,7 @@
     "ui.Done": "OK",
     "ui.EmptyCell": "-vide-",
     "ui.ErrorCodeX": "Code {0}",
-    "ui.ExpandAll": "déplier tous",
+    "ui.ExpandAll": "tout déplier",
     "ui.FileChooserHint": "Veuillez faire glisser les fichiers souhaités dans cette zone ou les sélectionner avec Parcourir.",
     "ui.FileSizeLimit": "La taille maximale autorisée du fichier a été dépassée ({0} MB}.",
     "ui.FileSizeLimitTitle": "Limitation de la taille des fichier",
@@ -315,7 +315,7 @@
     "ui.ReloadData": "Recharger les données",
     "ui.RemoveFilter": "Supprimer le filtre",
     "ui.SearchFor_": "Rechercher par…",
-    "ui.SelectAll": "Sélectionner tous",
+    "ui.SelectAll": "Tout sélectionner",
     "ui.SelectAllFilter": "Tous",
     "ui.SelectNone": "Sélectionner aucun",
     "ui.SelectNoneFilter": "Aucun",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eclipse-scout/core",
-  "version": "22.0.12",
+  "version": "22.0.22",
   "description": "Eclipse Scout runtime",
   "author": "BSI Business Systems Integration AG",
   "homepage": "https://www.eclipse.org/scout",
@@ -26,7 +26,7 @@
     "src"
   ],
   "devDependencies": {
-    "@eclipse-scout/cli": "22.0.12",
+    "@eclipse-scout/cli": "22.0.22",
     "@eclipse-scout/releng": "^22.0.0",
     "jasmine-core": "3.10.1",
     "jasmine-ajax": "4.0.0",

package/src/App.js

@@ -62,6 +62,7 @@ export default class App {
    * @param {string|string[]} [options.bootstrap.textsUrl] URL or multiple URLs pointing to a json resource containing texts that will be available through texts.js.
    * @param {string} [options.bootstrap.localesUrl] URL pointing to a json resource containing locale information processed by locales.js
    * @param {string} [options.bootstrap.codesUrl] URL pointing to a json resources containing codes that will be available through codes.js
+   * @param {object} [options.session] Object to configure the session, see {@link Session.init} for the available options.
    */
   init(options) {
     options = options || {};

package/src/desktop/DesktopDense.less

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -50,7 +50,8 @@
 
   .menubar:not(.main-menubar) {
     & > .menubar-box {
-      & > .menu-item {
+      & > .menu-item,
+      & > .combo-menu > .menu-item {
         padding-top: 3px; /* @menubar-item-padding-y; */
         padding-bottom: 3px; /* @menubar-item-padding-y; */
 
@@ -58,10 +59,22 @@
         min-height: @logical-grid-row-height-dense; /* @logical-grid-row-height; */
       }
 
-      & > .menu-button {
+      & > .menu-button,
+      & > .combo-menu > .menu-button {
         padding-top: 2px;
         padding-bottom: 2px;
       }
+
+      & > .combo-menu {
+        padding: 0;
+      }
+
+      & > .form-field-menu {
+        padding-top: 0;
+        padding-bottom: 0;
+        margin-top: 0;
+        margin-bottom: 0;
+      }
     }
   }
 
@@ -155,14 +168,18 @@
     }
   }
 
-
   /* ContextMenuPopup.less */
 
   .context-menu {
-    & > .menu-item {
+    & > .menu-item,
+    & > .combo-menu > .menu-item {
       padding-top: 6px; /* padding: 11px 15px; */
       padding-bottom: 6px;
     }
+
+    & > .combo-menu {
+      padding: 0;
+    }
   }
 
   /* CheckBoxField.less */
@@ -279,6 +296,7 @@
   }
 
   /* BreadcrumbItem.less */
+
   .breadcrumb-item {
     font-size: @font-size-normal;
   }

package/src/desktop/OpenUriHandler.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2014-2018 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -50,7 +50,7 @@ export default class OpenUriHandler {
       } else {
         this.openUriAsNewWindow(uri);
       }
-    } else if (Device.get().browser === Device.Browser.CHROME) {
+    } else if (Device.get().browser === Device.Browser.CHROME && this.isUriWithExternallyHandledProtocol(uri)) {
       // "Hidden iframe"-solution is not working in Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=663325)
       this.openUriInSameWindow(uri);
     } else {
@@ -59,7 +59,7 @@ export default class OpenUriHandler {
   }
 
   isUriWithExternallyHandledProtocol(uri) {
-    return /^(callto|facetime|fax|geo|mailto|maps|notes|sip|skype|tel|google.navigation):/.test(uri);
+    return /^(callto|facetime|fax|geo|mailto|maps|notes|sip|skype|tel|google.navigation|sms|msteams):/.test(uri);
   }
 
   handleUriActionOpen(uri) {
@@ -69,7 +69,7 @@ export default class OpenUriHandler {
       // Additionally, some url types require to be opened in the same window like tel or mailto, at least on mobile devices
       this.openUriInSameWindow(uri);
     } else if (this.isUriWithExternallyHandledProtocol(uri)) {
-      if (Device.get().browser === Device.Browser.CHROME) {
+      if (Device.get().browser === Device.Browser.CHROME || Device.get().isAndroid()) {
         // "Hidden iframe"-solution is not working in Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=663325)
         this.openUriInSameWindow(uri);
       } else {

package/src/desktop/outline/pages/Page.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -198,6 +198,11 @@ export default class Page extends TreeNode {
     if (form instanceof Form) {
       form.setModal(false);
       form.setClosable(false);
+
+      form.setDisplayHint(Form.DisplayHint.VIEW);
+      form.setDisplayViewId('C');
+
+      form.setShowOnOpen(false);
     }
     if (form instanceof TileOverviewForm) {
       form.setPage(this);

package/src/form/Form.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -21,6 +21,7 @@ export default class Form extends Widget {
     this.askIfNeedSave = true;
     this.askIfNeedSaveText = null; // if not set, a default text is used (see Lifecycle.js)
     this.data = {};
+    this.displayViewId = null;
     this.displayHint = Form.DisplayHint.DIALOG;
     this.displayParent = null; // only relevant if form is opened, not relevant if form is just rendered into another widget (not managed by a form controller)
     this.maximized = false;
@@ -44,6 +45,7 @@ export default class Form extends Widget {
     this.messageBoxController = null;
     this.fileChooserController = null;
     this.closeKeyStroke = null;
+    this.showOnOpen = true;
     this._glassPaneRenderer = null;
     this._preMaximizedBounds = null;
     this._resizeHandler = this._onResize.bind(this);
@@ -239,7 +241,9 @@ export default class Form extends Widget {
           // If form has been closed right after it was opened don't try to show it
           return;
         }
-        this.show();
+        if (this.showOnOpen) {
+          this.show();
+        }
       });
   }
 
@@ -754,6 +758,10 @@ export default class Form extends Widget {
     }
   }
 
+  setDisplayViewId(displayViewId) {
+    this.setProperty('displayViewId', displayViewId);
+  }
+
   setDisplayHint(displayHint) {
     this.setProperty('displayHint', displayHint);
   }
@@ -820,6 +828,10 @@ export default class Form extends Widget {
     return $prevIcon;
   }
 
+  setShowOnOpen(showOnOpen) {
+    this.setProperty('showOnOpen', showOnOpen);
+  }
+
   _updateTitleForWindow() {
     let formTitle = strings.join(' - ', this.title, this.subTitle),
       applicationTitle = this.session.desktop.title;

package/src/form/fields/FormField.js

@@ -1134,7 +1134,7 @@ export default class FormField extends Widget {
     if (!this.fieldStatus) {
       return;
     }
-    this.fieldStatus.remove();
+    this.fieldStatus.destroy();
     this.$status = null;
     this.fieldStatus = null;
   }

package/src/form/fields/datefield/DateField.js

@@ -205,6 +205,7 @@ export default class DateField extends ValueField {
       this.htmlDateTimeComposite.invalidateLayoutTree();
     }
     this._renderDateClearable();
+    this.$container.toggleClass('has-date', this.hasDate);
   }
 
   setHasTime(hasTime) {
@@ -257,6 +258,7 @@ export default class DateField extends ValueField {
       this.htmlDateTimeComposite.invalidateLayoutTree();
     }
     this._renderTimeClearable();
+    this.$container.toggleClass('has-time', this.hasTime);
   }
 
   setTimePickerResolution(timePickerResolution) {

package/src/form/fields/sequencebox/SequenceBox.js

@@ -1,9 +1,9 @@
 /*
- * Copyright (c) 2010-2021 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -19,6 +19,8 @@ export default class SequenceBox extends CompositeField {
     this.logicalGrid = scout.create('scout.HorizontalGrid');
     this.layoutConfig = null;
     this.fields = [];
+    this._fieldPropertyChangeHandler = this._onFieldPropertyChange.bind(this);
+    this._lastVisibleFieldSuppressStatusHandler = this._onLastVisibleFieldSuppressStatusChange.bind(this);
   }
 
   _init(model) {
@@ -65,7 +67,7 @@ export default class SequenceBox extends CompositeField {
     for (i = 0; i < this.fields.length; i++) {
       field = this.fields[i];
       field.labelUseUiWidth = true;
-      field.on('propertyChange', this._onFieldPropertyChange.bind(this));
+      field.on('propertyChange', this._fieldPropertyChangeHandler);
       field.render(this.$field);
       this._modifyLabel(field);
 
@@ -83,6 +85,14 @@ export default class SequenceBox extends CompositeField {
     return new SequenceBoxLayout(this, this.layoutConfig);
   }
 
+  _remove() {
+    this.fields.forEach(f => f.off('propertyChange', this._fieldPropertyChangeHandler));
+    if (this._lastVisibleField) {
+      this._lastVisibleField.off('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
+    }
+    super._remove();
+  }
+
   /**
    * @override
    */
@@ -137,6 +147,7 @@ export default class SequenceBox extends CompositeField {
   _handleStatus(visibilityChanged) {
     if (visibilityChanged && this._lastVisibleField) {
       // if there is a new last visible field, make sure the status is shown on the previously last one
+      this._lastVisibleField.off('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
       this._lastVisibleField.setSuppressStatus(null);
       if (this._lastVisibleField.rendered) {
         this._lastVisibleField._renderErrorStatus();
@@ -184,6 +195,7 @@ export default class SequenceBox extends CompositeField {
 
     // Make sure the last field won't display a status (but shows status CSS class)
     this._lastVisibleField.setSuppressStatus(FormField.SuppressStatus.ICON);
+    this._lastVisibleField.on('propertyChange:suppressStatus', this._lastVisibleFieldSuppressStatusHandler);
     if (visibilityChanged) {
       // If the last field got invisible, make sure the new last field does not display a status anymore (now done by the seq box)
       if (this._lastVisibleField.rendered) {
@@ -194,6 +206,11 @@ export default class SequenceBox extends CompositeField {
     }
   }
 
+  _onLastVisibleFieldSuppressStatusChange(e) {
+    // do not change suppressStatus
+    e.preventDefault();
+  }
+
   setErrorStatus(errorStatus) {
     if (this._isOverwritingStatusFromField && !this._isErrorStatusOverwritten) {
       // was not overwritten, will be overwritten now -> backup old value

package/src/form/fields/stringfield/StringField.js

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014-2018 BSI Business Systems Integration AG.
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -255,6 +255,14 @@ export default class StringField extends BasicField {
     this.revalidateLayout();
   }
 
+  /**
+   * @override
+   */
+  _renderEnabled() {
+    super._renderEnabled();
+    this.revalidateLayout();
+  }
+
   setFormatUpper(formatUpper) {
     if (formatUpper) {
       this.setFormat(StringField.Format.UPPER);

package/src/form/js/JsFormAdapter.js

@@ -0,0 +1,88 @@
+/*
+ * Copyright (c) 2010-2022 BSI Business Systems Integration AG.
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * which accompanies this distribution, and is available at
+ * https://www.eclipse.org/legal/epl-v10.html
+ *
+ * Contributors:
+ *     BSI Business Systems Integration AG - initial API and implementation
+ */
+import {FormAdapter} from '../../index';
+
+/**
+ * @typedef JsFormModel
+ * @property {object} parent
+ * @property {object} owner
+ * @property {object} displayParent
+ * @property {object} inputData
+ * @property {string} jsFormObjectType
+ * @property {object} jsFormModel
+ */
+
+export default class JsFormAdapter extends FormAdapter {
+
+  constructor() {
+    super();
+  }
+
+  /**
+   * @param {JsFormModel} model
+   */
+  _initModel(model, parent) {
+    model = super._initModel(model, parent);
+
+    if (!model.jsFormObjectType || !model.jsFormObjectType.length) {
+      throw new Error('jsFormObjectType not set');
+    }
+
+    let jsFormModel = {
+      parent: model.parent,
+      owner: model.owner,
+      objectType: model.jsFormObjectType,
+      displayParent: model.displayParent,
+      data: model.inputData
+    };
+
+    if (model.jsFormModel) {
+      jsFormModel = $.extend(true, {}, model.jsFormModel, jsFormModel);
+    }
+
+    return jsFormModel;
+  }
+
+  _createWidget(model) {
+    let widget = super._createWidget(model);
+
+    widget.showOnOpen = false;
+    widget.open();
+
+    return widget;
+  }
+
+  _onWidgetEvent(event) {
+    if (event.type === 'save') {
+      this._onWidgetSave(event);
+    } else {
+      super._onWidgetEvent(event);
+    }
+  }
+
+  _onWidgetSave(event) {
+    this._send('save', {
+      outputData: this.widget.data
+    });
+  }
+
+  _onWidgetClose(event) {
+    // marks the end of the js lifecycle
+    // prevent remove/destroy of the widget as it will be done by the UI server
+    event.preventDefault();
+    // fromClosing will trigger a 'formHide' event on the desktop which then removes the widget
+    this._send('formClosing');
+  }
+
+  _onWidgetAbort(event) {
+    // completely handled by the js lifecycle -> no need to notify the UI server
+  }
+}

package/src/index.js

@@ -3,7 +3,7 @@
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
- * http://www.eclipse.org/legal/epl-v10.html
+ * https://www.eclipse.org/legal/epl-v10.html
  *
  * Contributors:
  *     BSI Business Systems Integration AG - initial API and implementation
@@ -294,6 +294,7 @@ export {default as GroupBoxResponsiveHandler} from './form/GroupBoxResponsiveHan
 export {default as DesktopResponsiveHandler} from './form/DesktopResponsiveHandler';
 export {default as TileOverviewForm} from './form/TileOverviewForm';
 export {default as TileOverviewFormAdapter} from './form/TileOverviewFormAdapter';
+export {default as JsFormAdapter} from './form/js/JsFormAdapter';
 export {default as Table} from './table/Table';
 export {default as TableAdapter} from './table/TableAdapter';
 export {default as TableCompactHandler} from './table/TableCompactHandler';

package/src/scrollbar/scrollbars.js

@@ -152,7 +152,7 @@ export function installScrollShadow($container, session, options) {
   $container.data('scroll-shadow', $shadow);
   $container.data('scroll-shadow-style', scrollShadowStyle);
   $container.data('scroll-shadow-customizer', options.scrollShadowCustomizer);
-  let handler = () => updateScrollShadow($container);
+  let handler = () => updateScrollShadowWhileScrolling($container);
   $container.data('scroll-shadow-handler', handler);
   $container.on('scroll', handler);
   updateScrollShadow($container);
@@ -236,6 +236,17 @@ function _computeScrollShadowStyle(options) {
   return scrollShadow;
 }
 
+export function updateScrollShadowWhileScrolling($container) {
+  let $animatingParent = $container.findUp($elem => $elem.hasAnimationClass());
+  if ($animatingParent.length > 0) {
+    // If the container is scrolled while being animated, the shadow will likely get the wrong size and/or position if the animation changes the bounds.
+    // The scroll event is mostly probably not triggered by the user directly but by the scrollable container itself, e.g. to reveal a focused / selected / checked element.
+    $animatingParent.oneAnimationEnd(() => updateScrollShadow($container));
+    return;
+  }
+  updateScrollShadow($container);
+}
+
 export function updateScrollShadow($container) {
   let $shadow = $container.data('scroll-shadow');
   if (!$shadow) {

package/src/session/Session.js

@@ -100,36 +100,39 @@ export default class Session {
   static EMPTY_UPLOAD_FILENAME = '*empty*';
 
   /**
-   * $entryPoint is required to create a new session.
-   *
-   * @param model The 'model' argument holds all optional values that may be used during
-   * initialization (it is the same object passed to the scout.init() function).
-   * @param [model.portletPartId]
-   *     Optional, default is 0. Necessary when multiple UI sessions are managed
-   *     by the same window (portlet support). Each session's partId must be unique.
-   * @param model.clientSessionId
+   * @param {$} model.$entryPoint
+   *     The HTML element that is used by the {@link Desktop} to render its content.
+   * @param {string} [model.portletPartId]
+   *     Necessary when multiple UI sessions are managed by the same window (portlet support).
+   *     Each session's partId must be unique. Default is 0.
+   * @param {string} [model.clientSessionId]
    *     Identifies the 'client instance' on the UI server. If the property is not set
    *     (which is the default case), the clientSessionId is taken from the browser's
    *     session storage (per browser window, survives F5 refresh of page). If no
    *     clientSessionId can be found, a new one is generated on the server.
-   * @param [model.userAgent]
-   *     Default: DESKTOP
-   * @param [model.backgroundJobPollingEnabled]
+   * @param {boolean} [model.forceNewClientSession]
+   *     If set to true, the stored or passed clientSessionId will be ignored
+   *     and a new one generated by the server.
+   * @param {UserAgent} [model.userAgent]
+   *     By default the user agent for the running platform is used. Use this option if you want to set a custom one.
+   * @param {Locale|object} [model.locale]
+   *     If not specified, {@link Locale.DEFAULT} is used.
+   * @param {boolean} [model.backgroundJobPollingEnabled]
    *     Unless websockets is used, this property turns on (default) or off background
    *     polling using an async ajax call together with setTimeout()
-   * @param [model.suppressErrors]
+   * @param {boolean} [model.suppressErrors]
    *     Basically added because of Jasmine-tests. When working with async tests that
    *     use setTimeout(), sometimes the Jasmine-Maven plug-in fails and aborts the
    *     build because there were console errors. These errors always happen in this
    *     class. That's why we can skip suppress error handling with this flag.
-   * @param [model.focusManagerActive]
+   * @param {boolean} [model.focusManagerActive]
    *     Forces the focus manager to be active or not. If undefined, the value is
    *     auto detected by Device.js.
-   * @param [model.reconnectorOptions]
-   *     Optional, properties of this object are copied to the Session's reconnector
-   *     instance (see Reconnector.js).
-   * @param [model.ajaxCallOptions]
-   *     Optional, properties of this object are copied to all instances of AjaxCall.js.
+   * @param {object} [model.reconnectorOptions]
+   *     Properties of this object are copied to the Session's reconnector
+   *     instance (see {@link Reconnector}).
+   * @param {object} [model.ajaxCallOptions]
+   *     Properties of this object are copied to all instances of {@link AjaxCall}.
    */
   init(model) {
     let options = model || {};

package/src/table/Table.js

@@ -5271,15 +5271,14 @@ export default class Table extends Widget {
 
     for (i = 0; i < columns.length; i++) {
       column = columns[i];
-      currentPosition = this.columns.indexOf(column);
+      currentPosition = arrays.findIndex(this.columns, element => element.id === column.id);
       if (currentPosition < 0) {
         throw new Error('Column with id ' + column.id + 'not found.');
       }
 
       if (currentPosition !== i) {
         // Update model
-        arrays.remove(this.columns, column);
-        arrays.insert(this.columns, column, i);
+        arrays.move(this.columns, currentPosition, i);
       }
     }