Add generic type and description when its missing

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

apps/comments/src/services/GetComments.js

@@ -65,8 +65,8 @@ export default async function({ commentsType, ressourceId }, options = {}) {
 
 // https://github.com/perry-mitchell/webdav-client/blob/9de2da4a2599e06bd86c2778145b7ade39fe0b3c/source/interface/directoryContents.js#L32
 /**
- * @param result
- * @param isDetailed
+ * @param {any} result -
+ * @param {any} isDetailed -
  */
 function processMultistatus(result, isDetailed = false) {
 	// Extract the response items (directory contents)
@@ -91,8 +91,8 @@ function processMultistatus(result, isDetailed = false) {
 }
 
 /**
- * @param value
- * @param passes
+ * @param {any} value -
+ * @param {any} passes -
  */
 function decodeHtmlEntities(value, passes = 1) {
 	const parser = new DOMParser()

apps/dav/src/service/CalendarService.js

@@ -108,8 +108,8 @@ export async function findScheduleInboxAvailability() {
 }
 
 /**
- * @param slots
- * @param timezoneId
+ * @param {any} slots -
+ * @param {any} timezoneId -
  */
 export async function saveScheduleInboxAvailability(slots, timezoneId) {
 	const all = [...Object.keys(slots).flatMap(dayId => slots[dayId].map(slot => ({

apps/files/js/filelist.js

@@ -1203,7 +1203,7 @@
 		/**
 		 * Custom code
 		 * Set tag for all selected files
-		 * @param tagModel
+		 * @param {any} tagModel -
 		 * @private
 		 */
 		_onSelectTag: function(tagModel) {
@@ -1221,7 +1221,7 @@
 		},
 		/**
 		 * remove tag from all selected files
-		 * @param tagId
+		 * @param {any} tagId -
 		 * @private
 		 */
 		_onDeselectTag: function(tagId) {
@@ -3348,7 +3348,7 @@
 		},
 		/**
 		 * hide files matching the given filter
-		 * @param filter
+		 * @param {any} filter -
 		 */
 		setFilter:function(filter) {
 			var total = 0;
@@ -3413,7 +3413,7 @@
 		},
 		/**
 		 * get the current filter
-		 * @param filter
+		 * @param {any} filter -
 		 */
 		getFilter:function(filter) {
 			return this._filter;

apps/files/src/services/FileInfo.js

@@ -23,7 +23,7 @@
 import axios from '@nextcloud/axios'
 
 /**
- * @param url
+ * @param {any} url -
  */
 export default async function(url) {
 	const response = await axios({

apps/files_external/js/statusmanager.js

@@ -383,8 +383,8 @@ OCA.Files_External.StatusManager = {
 
 	/**
 	 * Function to display custom dialog to enter credentials
-	 * @param mountPoint
-	 * @param mountData
+	 * @param {any} mountPoint -
+	 * @param {any} mountData -
 	 */
 	showCredentialsDialog: function (mountPoint, mountData) {
 		var dialog = $(OCA.Files_External.Templates.credentialsDialog({

apps/files_sharing/js/public.js

@@ -377,12 +377,12 @@ OCA.Sharing.PublicApp = {
 	 * fall back to old behaviour where we redirect the user to his server to mount
 	 * the public link instead of creating a dedicated federated share
 	 *
-	 * @param remote
-	 * @param token
-	 * @param owner
-	 * @param ownerDisplayName
-	 * @param name
-	 * @param isProtected
+	 * @param {any} remote -
+	 * @param {any} token -
+	 * @param {any} owner -
+	 * @param {any} ownerDisplayName -
+	 * @param {any} name -
+	 * @param {any} isProtected -
 	 * @private
 	 */
 	_legacyCreateFederatedShare: function (remote, token, owner, ownerDisplayName, name, isProtected) {

apps/settings/js/settings/personalInfo.js

@@ -16,7 +16,7 @@ OC.Settings = OC.Settings || {};
  * The callback will be fired as soon as enter is pressed by the
  * user or 1 second after the last data entry
  *
- * @param callback
+ * @param {any} callback -
  * @param allowEmptyValue if this is set to true the callback is also called when the value is empty
  */
 jQuery.fn.keyUpDelayedOrEnter = function (callback, allowEmptyValue) {

apps/settings/src/service/WebAuthnRegistrationSerice.js

@@ -34,8 +34,8 @@ export async function startRegistration() {
 }
 
 /**
- * @param name
- * @param data
+ * @param {any} name -
+ * @param {any} data -
  */
 export async function finishRegistration(name, data) {
 	const url = generateUrl('/settings/api/personal/webauthn/registration')
@@ -45,7 +45,7 @@ export async function finishRegistration(name, data) {
 }
 
 /**
- * @param id
+ * @param {any} id -
  */
 export async function removeRegistration(id) {
 	const url = generateUrl(`/settings/api/personal/webauthn/registration/${id}`)

apps/settings/src/store/users.js

@@ -278,7 +278,7 @@ const actions = {
 	 * @param {object} options destructuring object
 	 * @param {number} options.offset List offset to request
 	 * @param {number} options.limit List number to return from offset
-	 * @param options.search
+	 * @param {string} options.search -
 	 * @return {Promise}
 	 */
 	getUsersFromList(context, { offset, limit, search }) {
@@ -301,7 +301,7 @@ const actions = {
 	 * @param {object} options destructuring object
 	 * @param {number} options.offset List offset to request
 	 * @param {number} options.limit List number to return from offset
-	 * @param options.groupid
+	 * @param {string} options.groupid -
 	 * @return {Promise}
 	 */
 	getUsersFromGroup(context, { groupid, offset, limit }) {
@@ -462,8 +462,8 @@ const actions = {
 	 * Add a user
 	 *
 	 * @param {object} context store context
-	 * @param {Function} context.commit
-	 * @param {Function} context.dispatch
+	 * @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
@@ -473,7 +473,6 @@ const actions = {
 	 * @param {string} options.subadmin User subadmin groups
 	 * @param {string} options.quota User email
 	 * @param {string} options.language User language
-	 * @param options.language
 	 * @return {Promise}
 	 */
 	addUser({ commit, dispatch }, { userid, password, displayName, email, groups, subadmin, quota, language }) {

apps/systemtags/src/systemtagsfilelist.js

@@ -48,7 +48,7 @@
 			/**
 			 * Array of system tag ids to filter by
 			 *
-			 * @type Array.<string>
+			 * @type {Array.<string>}
 			 */
 			_systemTagIds: [],
 			_lastUsedTags: [],

apps/systemtags/src/systemtagsinfoview.js

@@ -27,7 +27,7 @@
 (function(OCA) {
 
 	/**
-	 * @param model
+	 * @param {any} model -
 	 */
 	function modelToSelection(model) {
 		const data = model.toJSON()

apps/twofactor_backupcodes/src/service/PrintService.js

@@ -22,7 +22,7 @@
  */
 
 /**
- * @param data
+ * @param {any} data -
  */
 export function print(data) {
 	const name = OC.theme.name || 'Nextcloud'

apps/user_ldap/js/wizard/wizardTabAdvanced.js

@@ -17,8 +17,8 @@ OCA = OCA || {};
 		/**
 		 * initializes the instance. Always call it after initialization.
 		 *
-		 * @param tabIndex
-		 * @param tabID
+		 * @param {any} tabIndex -
+		 * @param {any} tabID -
 		 */
 		init: function (tabIndex, tabID) {
 			this._super(tabIndex, tabID);
@@ -281,7 +281,7 @@ OCA = OCA || {};
 		setDynamicGroupMemberURL: function(attribute) {
 			this.setElementValue(this.managedItems.ldap_dynamic_group_member_url.$element, attribute);
 		},
-                
+
 		/**
 		 * enabled or disables the use of nested groups (groups in groups in
 		 * groups…)

apps/user_ldap/js/wizard/wizardTabElementary.js

@@ -22,8 +22,8 @@ OCA = OCA || {};
 		/**
 		 * initializes the instance. Always call it after initialization.
 		 *
-		 * @param tabIndex
-		 * @param tabID
+		 * @param {any} tabIndex -
+		 * @param {any} tabID -
 		 */
 		init: function (tabIndex, tabID) {
 			tabIndex = 0;
@@ -245,8 +245,8 @@ OCA = OCA || {};
 		 * updates the configuration chooser upon the deletion of a
 		 * configuration and, if necessary, loads an existing one.
 		 *
-		 * @param view
-		 * @param result
+		 * @param {any} view -
+		 * @param {any} result -
 		 */
 		onDeleteConfiguration: function(view, result) {
 			if(result.isSuccess === true) {

apps/user_ldap/js/wizard/wizardTabExpert.js

@@ -17,8 +17,8 @@ OCA = OCA || {};
 		/**
 		 * initializes the instance. Always call it after initialization.
 		 *
-		 * @param tabIndex
-		 * @param tabID
+		 * @param {any} tabIndex -
+		 * @param {any} tabID -
 		 */
 		init: function (tabIndex, tabID) {
 			this._super(tabIndex, tabID);

apps/user_ldap/js/wizard/wizardTabGeneric.js

@@ -147,8 +147,8 @@ OCA = OCA || {};
 		/**
 		 * displays server error messages.
 		 *
-		 * @param view
-		 * @param payload
+		 * @param {any} view -
+		 * @param {any} payload -
 		 */
 		onServerError: function(view, payload) {
 			if (   !_.isUndefined(view.managedItems[payload.relatedKey])) {
@@ -456,7 +456,7 @@ OCA = OCA || {};
 		 * sets the filter mode initially and resets the "isToggling" marker.
 		 * This method is called after a save operation against the mode key.
 		 *
-		 * @param mode
+		 * @param {any} mode -
 		 */
 		setFilterModeOnce: function(mode) {
 			this.isToggling = false;

apps/user_ldap/js/wizard/wizardTabLoginFilter.js

@@ -16,8 +16,8 @@ OCA = OCA || {};
 		/**
 		 * initializes the instance. Always call it after initialization.
 		 *
-		 * @param tabIndex
-		 * @param tabID
+		 * @param {any} tabIndex -
+		 * @param {any} tabID -
 		 */
 		init: function (tabIndex, tabID) {
 			this._super(tabIndex, tabID);
@@ -134,7 +134,7 @@ OCA = OCA || {};
 		/**
 		 * presents the result of the login name test
 		 *
-		 * @param result
+		 * @param {any} result -
 		 */
 		handleLoginTestResult: function(result) {
 			var message;

apps/user_status/src/store/predefinedStatuses.js

@@ -48,7 +48,7 @@ const actions = {
 	 *
 	 * @param {object} vuex The Vuex components
 	 * @param {Function} vuex.commit The Vuex commit function
-	 * @param vuex.state
+	 * @param {object} vuex.state -
 	 */
 	async loadAllPredefinedStatuses({ state, commit }) {
 		if (state.predefinedStatuses.length > 0) {

core/src/OC/util.js

@@ -28,7 +28,7 @@ import OC from './index'
 import { formatFileSize as humanFileSize } from '@nextcloud/files'
 
 /**
- * @param t
+ * @param {any} t -
  */
 function chunkify(t) {
 	// Adapted from http://my.opera.com/GreyWyvern/blog/show.dml/1671288

core/src/OCP/comments.js

@@ -35,21 +35,21 @@ import $ from 'jquery'
 const urlRegex = /(\s|^)(https?:\/\/)?((?:[-A-Z0-9+_]+\.)+[-A-Z]+(?:\/[-A-Z0-9+&@#%?=~_|!:,.;()]*)*)(\s|$)/ig
 
 /**
- * @param content
+ * @param {any} content -
  */
 export function plainToRich(content) {
 	return this.formatLinksRich(content)
 }
 
 /**
- * @param content
+ * @param {any} content -
  */
 export function richToPlain(content) {
 	return this.formatLinksPlain(content)
 }
 
 /**
- * @param content
+ * @param {any} content -
  */
 export function formatLinksRich(content) {
 	return content.replace(urlRegex, function(_, leadingSpace, protocol, url, trailingSpace) {
@@ -65,7 +65,7 @@ export function formatLinksRich(content) {
 }
 
 /**
- * @param content
+ * @param {any} content -
  */
 export function formatLinksPlain(content) {
 	const $content = $('<div></div>').html(content)

core/src/OCP/whatsnew.js

@@ -26,7 +26,7 @@ import $ from 'jquery'
 import { generateOcsUrl } from '@nextcloud/router'
 
 /**
- * @param options
+ * @param {any} options -
  */
 export function query(options) {
 	options = options || {}
@@ -42,8 +42,8 @@ export function query(options) {
 }
 
 /**
- * @param version
- * @param options
+ * @param {any} version -
+ * @param {any} options -
  */
 export function dismiss(version, options) {
 	options = options || {}
@@ -59,10 +59,10 @@ export function dismiss(version, options) {
 }
 
 /**
- * @param data
- * @param statusText
- * @param xhr
- * @param dismissOptions
+ * @param {any} data -
+ * @param {any} statusText -
+ * @param {any} xhr -
+ * @param {any} dismissOptions -
  */
 function onQuerySuccess(data, statusText, xhr, dismissOptions) {
 	console.debug('querying Whats New data was successful: ' + statusText)
@@ -145,9 +145,9 @@ function onQuerySuccess(data, statusText, xhr, dismissOptions) {
 }
 
 /**
- * @param x
- * @param t
- * @param e
+ * @param {any} x -
+ * @param {any} t -
+ * @param {any} e -
  */
 function onQueryError(x, t, e) {
 	console.debug('querying Whats New Data resulted in an error: ' + t + e)
@@ -155,14 +155,14 @@ function onQueryError(x, t, e) {
 }
 
 /**
- * @param data
+ * @param {any} data -
  */
 function onDismissSuccess(data) {
 	// noop
 }
 
 /**
- * @param data
+ * @param {any} data -
  */
 function onDismissError(data) {
 	console.debug('dismissing Whats New data resulted in an error: ' + data)

core/src/Util/get-url-parameter.js

@@ -22,7 +22,7 @@
  */
 
 /**
- * @param name
+ * @param {any} name -
  */
 export default function getURLParameter(name) {
 	return decodeURIComponent(

core/src/jquery/selectrange.js

@@ -26,7 +26,7 @@ import $ from 'jquery'
 /**
  * select a range in an input field
  *
- * @link http://stackoverflow.com/questions/499126/jquery-set-cursor-position-in-text-area
+ * @see {@link http://stackoverflow.com/questions/499126/jquery-set-cursor-position-in-text-area}
  * @param {number} start start selection from
  * @param {number} end number of char from start
  * @return {void}

core/src/services/WebAuthnAuthenticationService.js

@@ -24,7 +24,7 @@ import Axios from '@nextcloud/axios'
 import { generateUrl } from '@nextcloud/router'
 
 /**
- * @param loginName
+ * @param {any} loginName -
  */
 export function startAuthentication(loginName) {
 	const url = generateUrl('/login/webauthn/start')
@@ -34,7 +34,7 @@ export function startAuthentication(loginName) {
 }
 
 /**
- * @param data
+ * @param {any} data -
  */
 export function finishAuthentication(data) {
 	const url = generateUrl('/login/webauthn/finish')

core/src/systemtags/systemtagscollection.js

@@ -61,7 +61,7 @@
 		 * Lazy fetch.
 		 * Only fetches once, subsequent calls will directly call the success handler.
 		 *
-		 * @param options
+		 * @param {any} options -
 		 * @param [options.force] true to force fetch even if cached entries exist
 		 *
 		 * @see Backbone.Collection#fetch