Typing corrections

Signed-off-by: Louis Chemineau <louis@chmn.me>

32 files changed, 87 insertions, 90 deletions

diff --git a/apps/federatedfilesharing/js/external.js b/apps/federatedfilesharing/js/external.js
index 40bb935b5c5..577fed21bac 100644
--- a/apps/federatedfilesharing/js/external.js
+++ b/apps/federatedfilesharing/js/external.js
@@ -19,7 +19,7 @@
 	 * @param {String} share.owner owner name
 	 * @param {String} share.name name of the shared folder
 	 * @param {String} share.token authentication token
-	 * @param {bool} passwordProtected true if the share is password protected
+	 * @param {boolean} passwordProtected true if the share is password protected
 	 * @param {Function} callback the callback
 	 */
 	OCA.Sharing.showAddExternalDialog = function(share, passwordProtected, callback) {
diff --git a/apps/files/js/app.js b/apps/files/js/app.js
index 5efa20887e6..ac0763397aa 100644
--- a/apps/files/js/app.js
+++ b/apps/files/js/app.js
@@ -187,7 +187,7 @@
 		 * they have initialized themselves. Therefore the files list needs to call this
 		 * method manually
 		 *
-		 * @param {OCA.Files.FileList} newFileList
+		 * @param {OCA.Files.FileList} newFileList -
 		 */
 		updateCurrentFileList: function(newFileList) {
 			this.currentFileList = newFileList;
diff --git a/apps/files/js/breadcrumb.js b/apps/files/js/breadcrumb.js
index f0a4da2d6d9..383d3debd09 100644
--- a/apps/files/js/breadcrumb.js
+++ b/apps/files/js/breadcrumb.js
@@ -104,7 +104,7 @@
 		 * Returns the full URL to the given directory
 		 *
 		 * @param {Object.<String, String>} part crumb data as map
-		 * @param {int} index crumb index
+		 * @param {number} index crumb index
 		 * @return full URL
 		 */
 		getCrumbUrl: function(part, index) {
diff --git a/apps/files/js/detailtabview.js b/apps/files/js/detailtabview.js
index 1e046f30246..dcb25190003 100644
--- a/apps/files/js/detailtabview.js
+++ b/apps/files/js/detailtabview.js
@@ -126,7 +126,7 @@
 		 * the given file info, for example based on mime type.
 		 *
 		 * @param {OCA.Files.FileInfoModel} fileInfo file info model
-		 * @return {bool} whether to display this tab
+		 * @return {boolean} whether to display this tab
 		 */
 		canDisplay: function(fileInfo) {
 			return true;
diff --git a/apps/files/js/file-upload.js b/apps/files/js/file-upload.js
index 7ab88ed61bd..9f51f05d561 100644
--- a/apps/files/js/file-upload.js
+++ b/apps/files/js/file-upload.js
@@ -163,7 +163,7 @@ OC.FileUpload.prototype = {
 	/**
 	 * Returns conflict resolution mode.
 	 *
-	 * @return {int} conflict mode
+	 * @return {number} conflict mode
 	 */
 	getConflictMode: function() {
 		return this._conflictMode || OC.FileUpload.CONFLICT_MODE_DETECT;
@@ -173,7 +173,7 @@ OC.FileUpload.prototype = {
 	 * Set conflict resolution mode.
 	 * See CONFLICT_MODE_* constants.
 	 *
-	 * @param {int} mode conflict mode
+	 * @param {number} mode conflict mode
 	 */
 	setConflictMode: function(mode) {
 		this._conflictMode = mode;
@@ -383,7 +383,7 @@ OC.FileUpload.prototype = {
 	/**
 	 * Returns the status code from the response
 	 *
-	 * @return {int} status code
+	 * @return {number} status code
 	 */
 	getResponseStatus: function() {
 		if (this.uploader.isXHRUpload()) {
@@ -515,7 +515,7 @@ OC.Uploader.prototype = _.extend({
 	/**
 	 * Returns whether an XHR upload will be used
 	 *
-	 * @return {bool} true if XHR upload will be used,
+	 * @return {boolean} true if XHR upload will be used,
 	 * false for iframe upload
 	 */
 	isXHRUpload: function () {
@@ -656,7 +656,7 @@ OC.Uploader.prototype = _.extend({
 	/**
 	 * Returns an upload by id
 	 *
-	 * @param {int} data uploadId
+	 * @param {number} data uploadId
 	 * @return {OC.FileUpload} file upload
 	 */
 	getUpload: function(data) {
@@ -860,7 +860,7 @@ OC.Uploader.prototype = _.extend({
 	 * Returns whether the given file is known to be a received shared file
 	 *
 	 * @param {Object} file file
-	 * @return {bool} true if the file is a shared file
+	 * @return {boolean} true if the file is a shared file
 	 */
 	_isReceivedSharedFile: function(file) {
 		if (!window.FileList) {
diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js
index d413bf7471b..505ab21c10c 100644
--- a/apps/files/js/fileactions.js
+++ b/apps/files/js/fileactions.js
@@ -191,7 +191,7 @@
 		 *
 		 * @param {string} mime mime type
 		 * @param {string} type "dir" or "file"
-		 * @param {int} permissions permissions
+		 * @param {number} permissions permissions
 		 * @param {string} filename filename
 		 *
 		 * @return {Object.<string,OCA.Files.FileActions~actionHandler>} map of action name to action spec
@@ -210,7 +210,7 @@
 		 *
 		 * @param {string} mime mime type
 		 * @param {string} type "dir" or "file"
-		 * @param {int} permissions permissions
+		 * @param {number} permissions permissions
 		 * @param {string} filename filename
 		 *
 		 * @return {Array.<OCA.Files.FileAction>} array of action specs
@@ -262,7 +262,7 @@
 		 *
 		 * @param {string} mime mime type
 		 * @param {string} type "dir" or "file"
-		 * @param {int} permissions permissions
+		 * @param {number} permissions permissions
 		 *
 		 * @return {OCA.Files.FileActions~actionHandler} action handler
 		 *
@@ -294,7 +294,7 @@
 		 *
 		 * @param {string} mime mime type
 		 * @param {string} type "dir" or "file"
-		 * @param {int} permissions permissions
+		 * @param {number} permissions permissions
 		 *
 		 * @return {OCA.Files.FileActions~actionSpec} action spec
 		 * @since 8.2
@@ -797,7 +797,7 @@
 	 * Defaults to the name given in name property
 	 * @property {String} mime mime type
 	 * @property {String} filename filename
-	 * @property {int} permissions permissions
+	 * @property {number} permissions permissions
 	 * @property {(Function|String)} icon icon path to the icon or function that returns it (deprecated, use iconClass instead)
 	 * @property {(String|OCA.Files.FileActions~iconClassFunction)} iconClass class name of the icon (recommended for theming)
 	 * @property {OCA.Files.FileActions~renderActionFunction} [render] optional rendering function
diff --git a/apps/files/js/fileinfomodel.js b/apps/files/js/fileinfomodel.js
index 2205c7e6477..8e7b399544c 100644
--- a/apps/files/js/fileinfomodel.js
+++ b/apps/files/js/fileinfomodel.js
@@ -15,14 +15,14 @@
 	 * @classdesc File information
 	 *
 	 * @param {Object} attributes file data
-	 * @param {int} attributes.id file id
+	 * @param {number} attributes.id file id
 	 * @param {string} attributes.name file name
 	 * @param {string} attributes.path path leading to the file,
 	 * without the file name and with a leading slash
-	 * @param {int} attributes.size size
+	 * @param {number} attributes.size size
 	 * @param {string} attributes.mimetype mime type
 	 * @param {string} attributes.icon icon URL
-	 * @param {int} attributes.permissions permissions
+	 * @param {number} attributes.permissions permissions
 	 * @param {Date} attributes.mtime modification time
 	 * @param {string} attributes.etag etag
 	 * @param {string} mountType mount type
diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 6fe9895c60e..fec058effd3 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -102,7 +102,7 @@
 		 * Number of files per page
 		 * Always show a minimum of 1
 		 *
-		 * @return {int} page size
+		 * @return {number} page size
 		 */
 		pageSize: function() {
 			var isGridView = this.$showGridView.is(':checked');
@@ -803,7 +803,7 @@
 		 * the internal selection cache.
 		 *
 		 * @param {Object} $tr single file row element
-		 * @param {bool} state true to select, false to deselect
+		 * @param {boolean} state true to select, false to deselect
 		 */
 		_selectFileEl: function($tr, state) {
 			var $checkbox = $tr.find('td.selection>.selectCheckBox');
@@ -1958,7 +1958,7 @@
 		 *
 		 * @param {OC.Files.FileInfo} fileData map of file attributes
 		 * @param {Object} [options] map of attributes
-		 * @param {int} [options.index] index at which to insert the element
+		 * @param {number} [options.index] index at which to insert the element
 		 * @param {boolean} [options.updateSummary] true to update the summary
 		 * after adding (default), false otherwise. Defaults to true.
 		 * @param {boolean} [options.animate] true to animate the thumbnail image after load
@@ -2234,7 +2234,7 @@
 			if (status === 401) {
 				// We are not authentificated, so reload the page so that we get
 				// redirected to the login page while saving the current url.
-				location.reload(); 
+				location.reload();
 			}
 
 			// Firewall Blocked request?
@@ -2348,8 +2348,8 @@
 		/**
 		 * Generates a preview URL based on the URL space.
 		 * @param urlSpec attributes for the URL
-		 * @param {int} urlSpec.x width
-		 * @param {int} urlSpec.y height
+		 * @param {number} urlSpec.x width
+		 * @param {number} urlSpec.y height
 		 * @param {String} urlSpec.file path to the file
 		 * @return preview URL
 		 */
@@ -3166,7 +3166,7 @@
 		 *
 		 * @param {string} file file name
 		 *
-		 * @return {bool} true if the file exists in the list, false otherwise
+		 * @return {boolean} true if the file exists in the list, false otherwise
 		 */
 		inList:function(file) {
 			return this.findFile(file);
@@ -3176,7 +3176,7 @@
 		 * Shows busy state on a given file row or multiple
 		 *
 		 * @param {string|Array.<string>} files file name or array of file names
-		 * @param {bool} [busy=true] busy state, true for busy, false to remove busy state
+		 * @param {boolean} [busy=true] busy state, true for busy, false to remove busy state
 		 *
 		 * @since 8.2
 		 */
@@ -3952,7 +3952,7 @@
 		 *
 		 * @param {OC.Files.FileInfo} fileInfo1 file info
 		 * @param {OC.Files.FileInfo} fileInfo2 file info
-		 * @return {int} -1 if the first file must appear before the second one,
+		 * @return {number} -1 if the first file must appear before the second one,
 		 * 0 if they are identify, 1 otherwise.
 		 */
 		name: function(fileInfo1, fileInfo2) {
@@ -3969,7 +3969,7 @@
 		 *
 		 * @param {OC.Files.FileInfo} fileInfo1 file info
 		 * @param {OC.Files.FileInfo} fileInfo2 file info
-		 * @return {int} -1 if the first file must appear before the second one,
+		 * @return {number} -1 if the first file must appear before the second one,
 		 * 0 if they are identify, 1 otherwise.
 		 */
 		size: function(fileInfo1, fileInfo2) {
@@ -3980,7 +3980,7 @@
 		 *
 		 * @param {OC.Files.FileInfo} fileInfo1 file info
 		 * @param {OC.Files.FileInfo} fileInfo2 file info
-		 * @return {int} -1 if the first file must appear before the second one,
+		 * @return {number} -1 if the first file must appear before the second one,
 		 * 0 if they are identify, 1 otherwise.
 		 */
 		mtime: function(fileInfo1, fileInfo2) {
diff --git a/apps/files/js/files.js b/apps/files/js/files.js
index 4de3fb4352f..03778fb92f9 100644
--- a/apps/files/js/files.js
+++ b/apps/files/js/files.js
@@ -219,7 +219,7 @@
 		 * Returns the download URL of the given file(s)
 		 * @param {string} filename string or array of file names to download
 		 * @param {string} [dir] optional directory in which the file name is, defaults to the current directory
-		 * @param {bool} [isDir=false] whether the given filename is a directory and might need a special URL
+		 * @param {boolean} [isDir=false] whether the given filename is a directory and might need a special URL
 		 */
 		getDownloadUrl: function(filename, dir, isDir) {
 			if (!_.isArray(filename) && !isDir) {
@@ -271,8 +271,8 @@
 		/**
 		 * Generates a preview URL based on the URL space.
 		 * @param urlSpec attributes for the URL
-		 * @param {int} urlSpec.x width
-		 * @param {int} urlSpec.y height
+		 * @param {number} urlSpec.x width
+		 * @param {number} urlSpec.y height
 		 * @param {String} urlSpec.file path to the file
 		 * @return preview URL
 		 * @deprecated used OCA.Files.FileList.generatePreviewUrl instead
diff --git a/apps/files/src/models/Setting.js b/apps/files/src/models/Setting.js
index 128c28d8679..ac2df7875b8 100644
--- a/apps/files/src/models/Setting.js
+++ b/apps/files/src/models/Setting.js
@@ -33,6 +33,7 @@ export default class Setting {
 	 *
 	 * @since 19.0.0
 	 * @param {string} name the name of this setting
+	 * @param {object} component the component
 	 * @param {Function} component.el function that returns an unmounted dom element to be added
 	 * @param {Function} [component.open] callback for when setting is added
 	 * @param {Function} [component.close] callback for when setting is closed
diff --git a/apps/files_external/js/settings.js b/apps/files_external/js/settings.js
index 266b4f96af3..823c9a364a8 100644
--- a/apps/files_external/js/settings.js
+++ b/apps/files_external/js/settings.js
@@ -54,7 +54,7 @@ function highlightInput($input) {
  * Initialize select2 plugin on the given elements
  *
  * @param {Array<Object>} array of jQuery elements
- * @param {int} userListLimit page size for result list
+ * @param {number} userListLimit page size for result list
  */
 function addSelect2 ($elements, userListLimit) {
 	var escapeHTML = function (text) {
@@ -571,7 +571,7 @@ MountOptionsDropdown.prototype = {
  *
  * @param {Object} $el DOM object containing the list
  * @param {Object} [options]
- * @param {int} [options.userListLimit] page size in applicable users dropdown
+ * @param {number} [options.userListLimit] page size in applicable users dropdown
  */
 var MountConfigListView = function($el, options) {
 	this.initialize($el, options);
@@ -642,7 +642,7 @@ MountConfigListView.prototype = _.extend({
 	/**
 	 * @param {Object} $el DOM object containing the list
 	 * @param {Object} [options]
-	 * @param {int} [options.userListLimit] page size in applicable users dropdown
+	 * @param {number} [options.userListLimit] page size in applicable users dropdown
 	 */
 	initialize: function($el, options) {
 		var self = this;
@@ -1227,7 +1227,7 @@ MountConfigListView.prototype = _.extend({
 	 * Update status display
 	 *
 	 * @param {jQuery} $tr
-	 * @param {int} status
+	 * @param {number} status
 	 * @param {string} message
 	 */
 	updateStatus: function($tr, status, message) {
diff --git a/apps/files_sharing/js/files_drop.js b/apps/files_sharing/js/files_drop.js
index 1360ffe42d7..e2830aeea9e 100644
--- a/apps/files_sharing/js/files_drop.js
+++ b/apps/files_sharing/js/files_drop.js
@@ -14,7 +14,7 @@
 		/** @type {Function} **/
 		_template: undefined,
 
-		/** @type {bool} */
+		/** @type {boolean} */
 		_uploading: false,
 
 		addFileToUpload: function(e, data) {
diff --git a/apps/files_sharing/js/sharedfilelist.js b/apps/files_sharing/js/sharedfilelist.js
index 4e364f92077..e8420113ad9 100644
--- a/apps/files_sharing/js/sharedfilelist.js
+++ b/apps/files_sharing/js/sharedfilelist.js
@@ -374,7 +374,7 @@
 		 * Converts the OCS API share response data to a file info
 		 * list
 		 * @param {Array} data OCS API share array
-		 * @param {bool} sharedWithUser
+		 * @param {boolean} sharedWithUser
 		 * @returns {Array.<OCA.Sharing.SharedFileInfo>} array of shared file info
 		 */
 			_makeFilesFromShares: function(data, sharedWithUser) {
@@ -511,10 +511,10 @@
 	 *
 	 * @typedef {Object} OCA.Sharing.ShareInfo
 	 *
-	 * @property {int} id share ID
-	 * @property {int} type share type
+	 * @property {number} id share ID
+	 * @property {number} type share type
 	 * @property {String} target share target, either user name or group name
-	 * @property {int} stime share timestamp in milliseconds
+	 * @property {number} stime share timestamp in milliseconds
 	 * @property {String} [targetDisplayName] display name of the recipient
 	 * (only when shared with others)
 	 * @property {String} [targetShareWithId] id of the recipient
@@ -536,7 +536,7 @@
 	 *
 	 * @property {Array.<OCA.Sharing.ShareInfo>} shares array of shares for
 	 * this file
-	 * @property {int} mtime most recent share time (if multiple shares)
+	 * @property {number} mtime most recent share time (if multiple shares)
 	 * @property {String} shareOwner name of the share owner
 	 * @property {Array.<String>} recipients name of the first 4 recipients
 	 * (this is mostly for display purposes)
diff --git a/apps/files_sharing/src/components/SharingEntry.vue b/apps/files_sharing/src/components/SharingEntry.vue
index 8d610333834..d5d7f62a832 100644
--- a/apps/files_sharing/src/components/SharingEntry.vue
+++ b/apps/files_sharing/src/components/SharingEntry.vue
@@ -373,7 +373,7 @@ export default {
 		},
 
 		/**
-		 * @return {bool}
+		 * @return {boolean}
 		 */
 		hasStatus() {
 			if (this.share.type !== this.SHARE_TYPES.SHARE_TYPE_USER) {
diff --git a/apps/files_sharing/src/mixins/SharesMixin.js b/apps/files_sharing/src/mixins/SharesMixin.js
index 97eea53391f..3844cef042b 100644
--- a/apps/files_sharing/src/mixins/SharesMixin.js
+++ b/apps/files_sharing/src/mixins/SharesMixin.js
@@ -27,7 +27,6 @@
 
 // eslint-disable-next-line import/no-unresolved, node/no-missing-import
 import PQueue from 'p-queue'
-// import PQueue from 'p-queue/dist/index'
 import debounce from 'debounce'
 
 import Share from '../models/Share'
@@ -200,7 +199,6 @@ export default {
 		/**
 		 * When the note change, we trim, save and dispatch
 		 *
-		 * @param {string} note the note
 		 */
 		onNoteSubmit() {
 			if (this.share.newNote) {
@@ -231,7 +229,7 @@ export default {
 		/**
 		 * Send an update of the share to the queue
 		 *
-		 * @param {string} propertyNames the properties to sync
+		 * @param {Array<string>} propertyNames the properties to sync
 		 */
 		queueUpdate(...propertyNames) {
 			if (propertyNames.length === 0) {
diff --git a/apps/files_sharing/src/models/Share.js b/apps/files_sharing/src/models/Share.js
index 239eb8c947e..3f8b5ed9ae7 100644
--- a/apps/files_sharing/src/models/Share.js
+++ b/apps/files_sharing/src/models/Share.js
@@ -65,7 +65,7 @@ export default class Share {
 	/**
 	 * get the share id
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
@@ -76,7 +76,7 @@ export default class Share {
 	/**
 	 * Get the share type
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
@@ -88,7 +88,7 @@ export default class Share {
 	 * Get the share permissions
 	 * See OC.PERMISSION_* variables
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
@@ -100,7 +100,7 @@ export default class Share {
 	 * Set the share permissions
 	 * See OC.PERMISSION_* variables
 	 *
-	 * @param {int} permissions valid permission, See OC.PERMISSION_* variables
+	 * @param {number} permissions valid permission, See OC.PERMISSION_* variables
 	 * @memberof Share
 	 */
 	set permissions(permissions) {
@@ -219,7 +219,7 @@ export default class Share {
 	/**
 	 * Get the share creation timestamp
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
@@ -417,7 +417,7 @@ export default class Share {
 	/**
 	 * Get the shared item id
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
@@ -441,7 +441,7 @@ export default class Share {
 	/**
 	 * Get the parent folder id if any
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Share
 	 */
diff --git a/apps/files_sharing/src/services/ConfigService.js b/apps/files_sharing/src/services/ConfigService.js
index 919c0db7b36..0c371633994 100644
--- a/apps/files_sharing/src/services/ConfigService.js
+++ b/apps/files_sharing/src/services/ConfigService.js
@@ -219,7 +219,7 @@ export default class Config {
 	/**
 	 * Get the default days to link shares expiration
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Config
 	 */
@@ -230,7 +230,7 @@ export default class Config {
 	/**
 	 * Get the default days to internal shares expiration
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Config
 	 */
@@ -241,7 +241,7 @@ export default class Config {
 	/**
 	 * Get the default days to remote shares expiration
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Config
 	 */
@@ -294,7 +294,7 @@ export default class Config {
 	/**
 	 * Get the maximum results of a share search
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Config
 	 */
@@ -306,7 +306,7 @@ export default class Config {
 	 * Get the minimal string length
 	 * to initiate a share search
 	 *
-	 * @return {int}
+	 * @return {number}
 	 * @readonly
 	 * @memberof Config
 	 */
diff --git a/apps/files_versions/src/versionstabview.js b/apps/files_versions/src/versionstabview.js
index 6adc3e61a18..38e78a1b154 100644
--- a/apps/files_versions/src/versionstabview.js
+++ b/apps/files_versions/src/versionstabview.js
@@ -215,7 +215,7 @@ import Template from './templates/template.handlebars';
 		 * Returns true for files, false for folders.
 		 *
 		 * @param {FileInfo} fileInfo fileInfo
-		 * @return {bool} true for files, false for folders
+		 * @return {boolean} true for files, false for folders
 		 */
 		canDisplay(fileInfo) {
 			if (!fileInfo) {
diff --git a/apps/settings/js/federationsettingsview.js b/apps/settings/js/federationsettingsview.js
index ca56992b1ed..9eb0b930733 100644
--- a/apps/settings/js/federationsettingsview.js
+++ b/apps/settings/js/federationsettingsview.js
@@ -15,9 +15,9 @@
 	 * @constructs FederationScopeMenu
 	 * @memberof OC.Settings
 	 * @param {object} options
-	 * @param {bool} [options.showFederatedScope=false] whether show the
+	 * @param {boolean} [options.showFederatedScope=false] whether show the
 	 *        "v2-federated" scope or not
-	 * @param {bool} [options.showPublishedScope=false] whether show the
+	 * @param {boolean} [options.showPublishedScope=false] whether show the
 	 *        "v2-published" scope or not
 	 */
 	var FederationSettingsView = OC.Backbone.View.extend({
diff --git a/apps/settings/src/mixins/UserRowMixin.js b/apps/settings/src/mixins/UserRowMixin.js
index 603f7a87837..23c039357be 100644
--- a/apps/settings/src/mixins/UserRowMixin.js
+++ b/apps/settings/src/mixins/UserRowMixin.js
@@ -158,7 +158,7 @@ export default {
 		 * Generate avatar url
 		 *
 		 * @param {string} user The user name
-		 * @param {int} size Size integer, default 32
+		 * @param {number} size Size integer, default 32
 		 * @return {string}
 		 */
 		generateAvatar(user, size = 32) {
diff --git a/apps/settings/src/store/admin-security.js b/apps/settings/src/store/admin-security.js
index b2806096989..c8e9dc5ad08 100644
--- a/apps/settings/src/store/admin-security.js
+++ b/apps/settings/src/store/admin-security.js
@@ -22,7 +22,7 @@
  */
 
 import Vue from 'vue'
-import Vuex from 'vuex'
+import Vuex, { Store } from 'vuex'
 
 Vue.use(Vuex)
 
@@ -44,7 +44,7 @@ const mutations = {
 	},
 }
 
-export default new Vuex.Store({
+export default new Store({
 	strict: process.env.NODE_ENV !== 'production',
 	state,
 	mutations,
diff --git a/apps/settings/src/store/index.js b/apps/settings/src/store/index.js
index 9f4313413eb..e9e58513ca9 100644
--- a/apps/settings/src/store/index.js
+++ b/apps/settings/src/store/index.js
@@ -22,7 +22,7 @@
  */
 
 import Vue from 'vue'
-import Vuex from 'vuex'
+import Vuex, { Store } from 'vuex'
 import users from './users'
 import apps from './apps'
 import settings from './settings'
@@ -44,7 +44,7 @@ const mutations = {
 	},
 }
 
-export default new Vuex.Store({
+export default new Store({
 	modules: {
 		users,
 		apps,
diff --git a/apps/settings/src/store/oc.js b/apps/settings/src/store/oc.js
index 35fbfcd34d1..7e34641deff 100644
--- a/apps/settings/src/store/oc.js
+++ b/apps/settings/src/store/oc.js
@@ -36,7 +36,7 @@ const actions = {
 	 * @param {string} options.app Application name
 	 * @param {boolean} options.key Config key
 	 * @param {boolean} options.value Value to set
-	 * @returns{Promise}
+	 * @return {Promise}
 	 */
 	setAppConfig(context, { app, key, value }) {
 		return api.requireAdmin().then((response) => {
diff --git a/apps/settings/src/store/users.js b/apps/settings/src/store/users.js
index 7bc04e25d9f..d898e647d32 100644
--- a/apps/settings/src/store/users.js
+++ b/apps/settings/src/store/users.js
@@ -207,8 +207,8 @@ const actions = {
 	 *
 	 * @param {object} context store context
 	 * @param {object} options destructuring object
-	 * @param {int} options.offset List offset to request
-	 * @param {int} options.limit List number to return from offset
+	 * @param {number} options.offset List offset to request
+	 * @param {number} options.limit List number to return from offset
 	 * @param {string} options.search Search amongst users
 	 * @param {string} options.group Get users from group
 	 * @return {Promise}
@@ -276,8 +276,8 @@ const actions = {
 	 *
 	 * @param {object} context store context
 	 * @param {object} options destructuring object
-	 * @param {int} options.offset List offset to request
-	 * @param {int} options.limit List number to return from offset
+	 * @param {number} options.offset List offset to request
+	 * @param {number} options.limit List number to return from offset
 	 * @param options.search
 	 * @return {Promise}
 	 */
@@ -299,8 +299,8 @@ const actions = {
 	 *
 	 * @param {object} context store context
 	 * @param {object} options destructuring object
-	 * @param {int} options.offset List offset to request
-	 * @param {int} options.limit List number to return from offset
+	 * @param {number} options.offset List offset to request
+	 * @param {number} options.limit List number to return from offset
 	 * @param options.groupid
 	 * @return {Promise}
 	 */
@@ -462,6 +462,8 @@ const actions = {
 	 * Add a user
 	 *
 	 * @param {object} context store context
+	 * @param {Function} context.commit
+	 * @param {Function} context.dispatch
 	 * @param {object} options destructuring object
 	 * @param {string} options.userid User id
 	 * @param {string} options.password User password
@@ -470,8 +472,7 @@ const actions = {
 	 * @param {string} options.groups User groups
 	 * @param {string} options.subadmin User subadmin groups
 	 * @param {string} options.quota User email
-	 * @param context.commit
-	 * @param context.dispatch
+	 * @param {string} options.language User language
 	 * @param options.language
 	 * @return {Promise}
 	 */
diff --git a/apps/systemtags/js/admin.js b/apps/systemtags/js/admin.js
index 2409cb488de..b8c08c8b8db 100644
--- a/apps/systemtags/js/admin.js
+++ b/apps/systemtags/js/admin.js
@@ -122,7 +122,7 @@
 		/**
 		 * Prepare the form for create/update
 		 *
-		 * @param {int} tagId
+		 * @param {number} tagId
 		 */
 		_prepareForm: function (tagId) {
 			if (tagId > 0) {
diff --git a/apps/systemtags/src/systemtagsinfoview.js b/apps/systemtags/src/systemtagsinfoview.js
index b2a11fbc959..353f8d555f9 100644
--- a/apps/systemtags/src/systemtagsinfoview.js
+++ b/apps/systemtags/src/systemtagsinfoview.js
@@ -56,7 +56,7 @@
 			id: 'systemTagsInfoView',
 
 			/**
-			 * @type OC.SystemTags.SystemTagsInputField
+			 * @type {OC.SystemTags.SystemTagsInputField}
 			 */
 			_inputView: null,
 
diff --git a/apps/twofactor_backupcodes/src/store.js b/apps/twofactor_backupcodes/src/store.js
index 762d6050e8f..9b34c9c6092 100644
--- a/apps/twofactor_backupcodes/src/store.js
+++ b/apps/twofactor_backupcodes/src/store.js
@@ -22,7 +22,7 @@
  */
 
 import Vue from 'vue'
-import Vuex from 'vuex'
+import Vuex, { Store } from 'vuex'
 import { generateCodes } from './service/BackupCodesService'
 
 Vue.use(Vuex)
@@ -63,7 +63,7 @@ const actions = {
 	},
 }
 
-export default new Vuex.Store({
+export default new Store({
 	strict: process.env.NODE_ENV !== 'production',
 	state,
 	mutations,
diff --git a/apps/user_ldap/js/wizard/wizardTabGeneric.js b/apps/user_ldap/js/wizard/wizardTabGeneric.js
index edb099f99cd..b68c399f992 100644
--- a/apps/user_ldap/js/wizard/wizardTabGeneric.js
+++ b/apps/user_ldap/js/wizard/wizardTabGeneric.js
@@ -29,7 +29,7 @@ OCA = OCA || {};
 		bjQuiButtonClass: 'ui-button',
 
 		/**
-		 * @property {bool} - indicates whether a filter mode toggle operation
+		 * @property {boolean} - indicates whether a filter mode toggle operation
 		 * is still in progress
 		 */
 		isToggling: false,
diff --git a/apps/user_status/src/store/index.js b/apps/user_status/src/store/index.js
index 6bca839b41f..49b77f92ba8 100644
--- a/apps/user_status/src/store/index.js
+++ b/apps/user_status/src/store/index.js
@@ -21,13 +21,13 @@
  */
 
 import Vue from 'vue'
-import Vuex from 'vuex'
+import Vuex, { Store } from 'vuex'
 import predefinedStatuses from './predefinedStatuses'
 import userStatus from './userStatus'
 
 Vue.use(Vuex)
 
-export default new Vuex.Store({
+export default new Store({
 	modules: {
 		predefinedStatuses,
 		userStatus,
diff --git a/apps/weather_status/src/services/weatherStatusService.js b/apps/weather_status/src/services/weatherStatusService.js
index bf9cc43b090..83776c296fc 100644
--- a/apps/weather_status/src/services/weatherStatusService.js
+++ b/apps/weather_status/src/services/weatherStatusService.js
@@ -97,7 +97,6 @@ const getLocation = async () => {
 /**
  * Fetches the weather forecast
  *
- * @param {string} address The location
  * @return {Promise<object>}
  */
 const fetchForecast = async () => {
@@ -110,7 +109,6 @@ const fetchForecast = async () => {
 /**
  * Fetches the location favorites
  *
- * @param {string} address The location
  * @return {Promise<object>}
  */
 const getFavorites = async () => {
diff --git a/apps/workflowengine/src/store.js b/apps/workflowengine/src/store.js
index 81735a43638..f1590099948 100644
--- a/apps/workflowengine/src/store.js
+++ b/apps/workflowengine/src/store.js
@@ -25,7 +25,7 @@
  */
 
 import Vue from 'vue'
-import Vuex from 'vuex'
+import Vuex, { Store } from 'vuex'
 import axios from '@nextcloud/axios'
 import { getApiUrl } from './helpers/api'
 import confirmPassword from '@nextcloud/password-confirmation'
@@ -33,7 +33,7 @@ import { loadState } from '@nextcloud/initial-state'
 
 Vue.use(Vuex)
 
-const store = new Vuex.Store({
+const store = new Store({
 	state: {
 		rules: [],
 		scope: loadState('workflowengine', 'scope'),
@@ -161,8 +161,7 @@ const store = new Vuex.Store({
 		 * Return all available checker plugins for a given entity class
 		 *
 		 * @param {object} state the store state
-		 * @param {object} entity the entity class
-		 * @return {Array} the available plugins
+		 * @return {Function} the available plugins
 		 */
 		getChecksForEntity(state) {
 			return (entity) => {
diff --git a/apps/workflowengine/src/workflowengine.js b/apps/workflowengine/src/workflowengine.js
index a3e92c2a997..f66b9e54f6b 100644
--- a/apps/workflowengine/src/workflowengine.js
+++ b/apps/workflowengine/src/workflowengine.js
@@ -37,8 +37,8 @@ import ShippedChecks from './components/Checks'
  *  The component should handle the v-model directive properly,
  *  so it needs a value property to receive data and emit an input
  *  event once the data has changed
- * @property {callable} placeholder - Return a placeholder of no custom component is used
- * @property {callable} validate - validate a check if no custom component is used
+ * @property {Function} placeholder - Return a placeholder of no custom component is used
+ * @property {Function} validate - validate a check if no custom component is used
  */
 
 /**