[ticket/13887] Use correct JSDoc syntax

PHPBB3-13887
This commit is contained in:
Callum Macrae 2015-05-29 12:09:19 +02:00
parent 518902ef51
commit 2a83290e7c
3 changed files with 128 additions and 190 deletions

View file

@ -21,7 +21,7 @@ phpbb.isTouch = (window && typeof window.ontouchstart !== 'undefined');
/** /**
* Display a loading screen * Display a loading screen
* *
* @returns object Returns loadingIndicator. * @returns {object} Returns loadingIndicator.
*/ */
phpbb.loadingIndicator = function() { phpbb.loadingIndicator = function() {
if (!$loadingIndicator.is(':visible')) { if (!$loadingIndicator.is(':visible')) {
@ -54,7 +54,7 @@ phpbb.clearLoadingTimeout = function() {
/** /**
* Close popup alert after a specified delay * Close popup alert after a specified delay
* *
* @param int Delay in ms until darkenwrapper's click event is triggered * @param {int} delay Delay in ms until darkenwrapper's click event is triggered
*/ */
phpbb.closeDarkenWrapper = function(delay) { phpbb.closeDarkenWrapper = function(delay) {
phpbbAlertTimer = setTimeout(function() { phpbbAlertTimer = setTimeout(function() {
@ -67,14 +67,12 @@ phpbb.closeDarkenWrapper = function(delay) {
* *
* You can only call one alert or confirm box at any one time. * You can only call one alert or confirm box at any one time.
* *
* @param string title Title of the message, eg "Information" (HTML). * @param {string} title Title of the message, eg "Information" (HTML).
* @param string msg Message to display (HTML). * @param {string} msg Message to display (HTML).
* @param bool fadedark Remove the dark background when done? Defaults
* to yes.
* *
* @returns object Returns the div created. * @returns {object} Returns the div created.
*/ */
phpbb.alert = function(title, msg, fadedark) { phpbb.alert = function(title, msg) {
var $alert = $('#phpbb_alert'); var $alert = $('#phpbb_alert');
$alert.find('.alert_title').html(title); $alert.find('.alert_title').html(title);
$alert.find('.alert_text').html(msg); $alert.find('.alert_text').html(msg);
@ -94,7 +92,7 @@ phpbb.alert = function(title, msg, fadedark) {
/** /**
* Handler for opening an alert box. * Handler for opening an alert box.
* *
* @param jQuery $alert jQuery object. * @param {jQuery} $alert jQuery object.
*/ */
phpbb.alert.open = function($alert) { phpbb.alert.open = function($alert) {
if (!$dark.is(':visible')) { if (!$dark.is(':visible')) {
@ -134,8 +132,8 @@ phpbb.alert.open = function($alert) {
/** /**
* Handler for closing an alert box. * Handler for closing an alert box.
* *
* @param jQuery $alert jQuery object. * @param {jQuery} $alert jQuery object.
* @param bool fadedark Whether to remove dark background. * @param {bool} fadedark Whether to remove dark background.
*/ */
phpbb.alert.close = function($alert, fadedark) { phpbb.alert.close = function($alert, fadedark) {
var $fade = (fadedark) ? $dark : $alert; var $fade = (fadedark) ? $dark : $alert;
@ -153,13 +151,13 @@ phpbb.alert.close = function($alert, fadedark) {
* *
* You can only call one alert or confirm box at any one time. * You can only call one alert or confirm box at any one time.
* *
* @param string msg Message to display (HTML). * @param {string} msg Message to display (HTML).
* @param function callback Callback. Bool param, whether the user pressed * @param {function} callback Callback. Bool param, whether the user pressed
* yes or no (or whatever their language is). * yes or no (or whatever their language is).
* @param bool fadedark Remove the dark background when done? Defaults * @param {bool} fadedark Remove the dark background when done? Defaults
* to yes. * to yes.
* *
* @returns object Returns the div created. * @returns {object} Returns the div created.
*/ */
phpbb.confirm = function(msg, callback, fadedark) { phpbb.confirm = function(msg, callback, fadedark) {
var $confirmDiv = $('#phpbb_confirm'); var $confirmDiv = $('#phpbb_confirm');
@ -197,9 +195,9 @@ phpbb.confirm = function(msg, callback, fadedark) {
/** /**
* Turn a querystring into an array. * Turn a querystring into an array.
* *
* @argument string string The querystring to parse. * @argument {string} string The querystring to parse.
* @returns object The object created. * @returns {object} The object created.
*/ */{}
phpbb.parseQuerystring = function(string) { phpbb.parseQuerystring = function(string) {
var params = {}, i, split; var params = {}, i, split;
@ -223,12 +221,7 @@ phpbb.parseQuerystring = function(string) {
* For more info, view the following page on the phpBB wiki: * For more info, view the following page on the phpBB wiki:
* http://wiki.phpbb.com/JavaScript_Function.phpbb.ajaxify * http://wiki.phpbb.com/JavaScript_Function.phpbb.ajaxify
* *
* @param object options Options. * @param {object} options Options.
* @param bool/function refresh If we are sent back a refresh, should it be
* acted upon? This can either be true / false / a function.
* @param function callback Callback to call on completion of event. Has
* three parameters: the element that the event was evoked from, the JSON
* that was returned and (if it is a form) the form action.
*/ */
phpbb.ajaxify = function(options) { phpbb.ajaxify = function(options) {
var $elements = $(options.selector), var $elements = $(options.selector),
@ -283,7 +276,7 @@ phpbb.ajaxify = function(options) {
* It cannot be called from outside this function, and is purely here to * It cannot be called from outside this function, and is purely here to
* avoid repetition of code. * avoid repetition of code.
* *
* @param object res The object sent back by the server. * @param {object} res The object sent back by the server.
*/ */
function returnHandler(res) { function returnHandler(res) {
var alert; var alert;
@ -429,8 +422,8 @@ phpbb.search = {
/** /**
* Get cached search data. * Get cached search data.
* *
* @param string id Search ID. * @param {string} id Search ID.
* @return bool|object. Cached data object. Returns false if no data exists. * @returns {bool|object} Cached data object. Returns false if no data exists.
*/ */
phpbb.search.cache.get = function(id) { phpbb.search.cache.get = function(id) {
if (this.data[id]) { if (this.data[id]) {
@ -442,11 +435,9 @@ phpbb.search.cache.get = function(id) {
/** /**
* Set search cache data value. * Set search cache data value.
* *
* @param string id Search ID. * @param {string} id Search ID.
* @param string key Data key. * @param {string} key Data key.
* @param string value Data value. * @param {string} value Data value.
*
* @return undefined
*/ */
phpbb.search.cache.set = function(id, key, value) { phpbb.search.cache.set = function(id, key, value) {
if (!this.data[id]) { if (!this.data[id]) {
@ -458,21 +449,19 @@ phpbb.search.cache.set = function(id, key, value) {
/** /**
* Cache search result. * Cache search result.
* *
* @param string id Search ID. * @param {string} id Search ID.
* @param string keyword Keyword. * @param {string} keyword Keyword.
* @param array results Search results. * @param {Array} results Search results.
*
* @return undefined
*/ */
phpbb.search.cache.setResults = function(id, keyword, value) { phpbb.search.cache.setResults = function(id, keyword, results) {
this.data[id].results[keyword] = value; this.data[id].results[keyword] = results;
}; };
/** /**
* Trim spaces from keyword and lower its case. * Trim spaces from keyword and lower its case.
* *
* @param string keyword Search keyword to clean. * @param {string} keyword Search keyword to clean.
* @return string Cleaned string. * @returns {string} Cleaned string.
*/ */
phpbb.search.cleanKeyword = function(keyword) { phpbb.search.cleanKeyword = function(keyword) {
return $.trim(keyword).toLowerCase(); return $.trim(keyword).toLowerCase();
@ -482,11 +471,11 @@ phpbb.search.cleanKeyword = function(keyword) {
* Get clean version of search keyword. If textarea supports several keywords * Get clean version of search keyword. If textarea supports several keywords
* (one per line), it fetches the current keyword based on the caret position. * (one per line), it fetches the current keyword based on the caret position.
* *
* @param jQuery $input Search input|textarea. * @param {jQuery} $input Search input|textarea.
* @param string keyword Input|textarea value. * @param {string} keyword Input|textarea value.
* @param bool multiline Whether textarea supports multiple search keywords. * @param {bool} multiline Whether textarea supports multiple search keywords.
* *
* @return string Clean string. * @returns string Clean string.
*/ */
phpbb.search.getKeyword = function($input, keyword, multiline) { phpbb.search.getKeyword = function($input, keyword, multiline) {
if (multiline) { if (multiline) {
@ -500,8 +489,8 @@ phpbb.search.getKeyword = function($input, keyword, multiline) {
* Get the textarea line number on which the keyword resides - for textareas * Get the textarea line number on which the keyword resides - for textareas
* that support multiple keywords (one per line). * that support multiple keywords (one per line).
* *
* @param jQuery $textarea Search textarea. * @param {jQuery} $textarea Search textarea.
* @return int * @returns {int} The line number.
*/ */
phpbb.search.getKeywordLine = function ($textarea) { phpbb.search.getKeywordLine = function ($textarea) {
var selectionStart = $textarea.get(0).selectionStart; var selectionStart = $textarea.get(0).selectionStart;
@ -512,11 +501,9 @@ phpbb.search.getKeywordLine = function ($textarea) {
* Set the value on the input|textarea. If textarea supports multiple * Set the value on the input|textarea. If textarea supports multiple
* keywords, only the active keyword is replaced. * keywords, only the active keyword is replaced.
* *
* @param jQuery $input Search input|textarea. * @param {jQuery} $input Search input|textarea.
* @param string value Value to set. * @param {string} value Value to set.
* @param bool multiline Whether textarea supports multiple search keywords. * @param {bool} multiline Whether textarea supports multiple search keywords.
*
* @return undefined
*/ */
phpbb.search.setValue = function($input, value, multiline) { phpbb.search.setValue = function($input, value, multiline) {
if (multiline) { if (multiline) {
@ -531,12 +518,10 @@ phpbb.search.setValue = function($input, value, multiline) {
/** /**
* Sets the onclick event to set the value on the input|textarea to the selected search result. * Sets the onclick event to set the value on the input|textarea to the selected search result.
* *
* @param jQuery $input Search input|textarea. * @param {jQuery} $input Search input|textarea.
* @param object value Result object. * @param {object} value Result object.
* @param jQuery $row Result element. * @param {jQuery} $row Result element.
* @param jQuery $container jQuery object for the search container. * @param {jQuery} $container jQuery object for the search container.
*
* @return undefined
*/ */
phpbb.search.setValueOnClick = function($input, value, $row, $container) { phpbb.search.setValueOnClick = function($input, value, $row, $container) {
$row.click(function() { $row.click(function() {
@ -551,11 +536,11 @@ phpbb.search.setValueOnClick = function($input, value, $row, $container) {
* already, those are displayed instead. Executes the AJAX request function * already, those are displayed instead. Executes the AJAX request function
* itself due to the need to use a timeout to limit the number of requests. * itself due to the need to use a timeout to limit the number of requests.
* *
* @param array data Data to be sent to the server. * @param {Array} data Data to be sent to the server.
* @param object event Onkeyup event object. * @param {object} event Onkeyup event object.
* @param function sendRequest Function to execute AJAX request. * @param {function} sendRequest Function to execute AJAX request.
* *
* @return bool Returns false. * @returns {bool} Returns false.
*/ */
phpbb.search.filter = function(data, event, sendRequest) { phpbb.search.filter = function(data, event, sendRequest) {
var $this = $(this), var $this = $(this),
@ -609,12 +594,10 @@ phpbb.search.filter = function(data, event, sendRequest) {
/** /**
* Handle search result response. * Handle search result response.
* *
* @param object res Data received from server. * @param {object} res Data received from server.
* @param jQuery $input Search input|textarea. * @param {jQuery} $input Search input|textarea.
* @param bool fromCache Whether the results are from the cache. * @param {bool} fromCache Whether the results are from the cache.
* @param function callback Optional callback to run when assigning each search result. * @param {function} callback Optional callback to run when assigning each search result.
*
* @return undefined
*/ */
phpbb.search.handleResponse = function(res, $input, fromCache, callback) { phpbb.search.handleResponse = function(res, $input, fromCache, callback) {
if (typeof res !== 'object') { if (typeof res !== 'object') {
@ -641,12 +624,10 @@ phpbb.search.handleResponse = function(res, $input, fromCache, callback) {
/** /**
* Show search results. * Show search results.
* *
* @param array results Search results. * @param {Array} results Search results.
* @param jQuery $input Search input|textarea. * @param {jQuery} $input Search input|textarea.
* @param jQuery $container Search results container element. * @param {jQuery} $container Search results container element.
* @param function callback Optional callback to run when assigning each search result. * @param {function} callback Optional callback to run when assigning each search result.
*
* @return undefined
*/ */
phpbb.search.showResults = function(results, $input, $container, callback) { phpbb.search.showResults = function(results, $input, $container, callback) {
var $resultContainer = $('.search-results', $container); var $resultContainer = $('.search-results', $container);
@ -683,8 +664,7 @@ phpbb.search.showResults = function(results, $input, $container, callback) {
/** /**
* Clear search results. * Clear search results.
* *
* @param jQuery $container Search results container. * @param {jQuery} $container Search results container.
* @return undefined
*/ */
phpbb.search.clearResults = function($container) { phpbb.search.clearResults = function($container) {
$container.children(':not(.search-result-tpl)').remove(); $container.children(':not(.search-result-tpl)').remove();
@ -703,8 +683,8 @@ phpbb.history = {};
/** /**
* Check whether a method in the native history object is supported. * Check whether a method in the native history object is supported.
* *
* @param string fn Method name. * @param {string} fn Method name.
* @return bool Returns true if the method is supported. * @returns {bool} Returns true if the method is supported.
*/ */
phpbb.history.isSupported = function(fn) { phpbb.history.isSupported = function(fn) {
return !(typeof history === 'undefined' || typeof history[fn] === 'undefined'); return !(typeof history === 'undefined' || typeof history[fn] === 'undefined');
@ -714,12 +694,10 @@ phpbb.history.isSupported = function(fn) {
* Wrapper for the pushState and replaceState methods of the * Wrapper for the pushState and replaceState methods of the
* native history object. * native history object.
* *
* @param string mode Mode. Either push or replace. * @param {string} mode Mode. Either push or replace.
* @param string url New URL. * @param {string} url New URL.
* @param string title Optional page title. * @param {string} [title] Optional page title.
* @patam object obj Optional state object. * @param {object} [obj] Optional state object.
*
* @return undefined
*/ */
phpbb.history.alterUrl = function(mode, url, title, obj) { phpbb.history.alterUrl = function(mode, url, title, obj) {
var fn = mode + 'State'; var fn = mode + 'State';
@ -740,11 +718,9 @@ phpbb.history.alterUrl = function(mode, url, title, obj) {
/** /**
* Wrapper for the native history.replaceState method. * Wrapper for the native history.replaceState method.
* *
* @param string url New URL. * @param {string} url New URL.
* @param string title Optional page title. * @param {string} [title] Optional page title.
* @patam object obj Optional state object. * @param {object} [obj] Optional state object.
*
* @return undefined
*/ */
phpbb.history.replaceUrl = function(url, title, obj) { phpbb.history.replaceUrl = function(url, title, obj) {
phpbb.history.alterUrl('replace', url, title, obj); phpbb.history.alterUrl('replace', url, title, obj);
@ -753,11 +729,9 @@ phpbb.history.replaceUrl = function(url, title, obj) {
/** /**
* Wrapper for the native history.pushState method. * Wrapper for the native history.pushState method.
* *
* @param string url New URL. * @param {string} url New URL.
* @param string title Optional page title. * @param {string} [title] Optional page title.
* @patam object obj Optional state object. * @param {object} [obj] Optional state object.
*
* @return undefined
*/ */
phpbb.history.pushUrl = function(url, title, obj) { phpbb.history.pushUrl = function(url, title, obj) {
phpbb.history.alterUrl('push', url, title, obj); phpbb.history.alterUrl('push', url, title, obj);
@ -766,7 +740,8 @@ phpbb.history.pushUrl = function(url, title, obj) {
/** /**
* Hide the optgroups that are not the selected timezone * Hide the optgroups that are not the selected timezone
* *
* @param bool keepSelection Shall we keep the value selected, or shall the user be forced to repick one. * @param {bool} keepSelection Shall we keep the value selected, or shall the
* user be forced to repick one.
*/ */
phpbb.timezoneSwitchDate = function(keepSelection) { phpbb.timezoneSwitchDate = function(keepSelection) {
var $timezoneCopy = $('#timezone_copy'); var $timezoneCopy = $('#timezone_copy');
@ -824,7 +799,7 @@ phpbb.timezoneEnableDateSelection = function() {
/** /**
* Preselect a date/time or suggest one, if it is not picked. * Preselect a date/time or suggest one, if it is not picked.
* *
* @param bool forceSelector Shall we select the suggestion? * @param {bool} forceSelector Shall we select the suggestion?
*/ */
phpbb.timezonePreselectSelect = function(forceSelector) { phpbb.timezonePreselectSelect = function(forceSelector) {
@ -893,8 +868,8 @@ phpbb.ajaxCallbacks = {};
* *
* See the phpbb.ajaxify comments for information on stuff like parameters. * See the phpbb.ajaxify comments for information on stuff like parameters.
* *
* @param string id The name of the callback. * @param {string} id The name of the callback.
* @param function callback The callback to be called. * @param {function} callback The callback to be called.
*/ */
phpbb.addAjaxCallback = function(id, callback) { phpbb.addAjaxCallback = function(id, callback) {
if (typeof callback === 'function') { if (typeof callback === 'function') {
@ -985,7 +960,7 @@ phpbb.addAjaxCallback('toggle_link', function() {
* types text. * types text.
* *
* @param {jQuery} $items jQuery object(s) to resize * @param {jQuery} $items jQuery object(s) to resize
* @param {object} options Optional parameter that adjusts default * @param {object} [options] Optional parameter that adjusts default
* configuration. See configuration variable * configuration. See configuration variable
* *
* Optional parameters: * Optional parameters:
@ -1085,7 +1060,7 @@ phpbb.resizeTextArea = function($items, options) {
* @param {Array} endTags List of end tags to look for * @param {Array} endTags List of end tags to look for
* For example, Array('[/code]') * For example, Array('[/code]')
* *
* @return {boolean} True if cursor is in bbcode tag * @returns {boolean} True if cursor is in bbcode tag
*/ */
phpbb.inBBCodeTag = function(textarea, startTags, endTags) { phpbb.inBBCodeTag = function(textarea, startTags, endTags) {
var start = textarea.selectionStart, var start = textarea.selectionStart,
@ -1157,7 +1132,7 @@ phpbb.applyCodeEditor = function(textarea) {
* @param {boolean} stripCodeStart If true, only part of line * @param {boolean} stripCodeStart If true, only part of line
* after [code] tag will be returned. * after [code] tag will be returned.
* *
* @return {string} Line of text * @returns {string} Line of text
*/ */
function getLastLine(stripCodeStart) { function getLastLine(stripCodeStart) {
var start = textarea.selectionStart, var start = textarea.selectionStart,
@ -1190,7 +1165,7 @@ phpbb.applyCodeEditor = function(textarea) {
/** /**
* Append text at cursor position * Append text at cursor position
* *
* @param {string} Text Text to append * @param {string} text Text to append
*/ */
function appendText(text) { function appendText(text) {
var start = textarea.selectionStart, var start = textarea.selectionStart,
@ -1416,9 +1391,9 @@ phpbb.registerDropdown = function(toggle, dropdown, options) {
/** /**
* Get the HTML for a color palette table. * Get the HTML for a color palette table.
* *
* @param string dir Palette direction - either v or h * @param {string} dir Palette direction - either v or h
* @param int width Palette cell width. * @param {int} width Palette cell width.
* @param int height Palette cell height. * @param {int} height Palette cell height.
*/ */
phpbb.colorPalette = function(dir, width, height) { phpbb.colorPalette = function(dir, width, height) {
var r = 0, var r = 0,
@ -1470,7 +1445,7 @@ phpbb.colorPalette = function(dir, width, height) {
/** /**
* Register a color palette. * Register a color palette.
* *
* @param object el jQuery object for the palette container. * @param {jQuery} el jQuery object for the palette container.
*/ */
phpbb.registerPalette = function(el) { phpbb.registerPalette = function(el) {
var orientation = el.attr('data-orientation'), var orientation = el.attr('data-orientation'),
@ -1504,10 +1479,10 @@ phpbb.registerPalette = function(el) {
/** /**
* Set display of page element * Set display of page element
* *
* @param string id The ID of the element to change * @param {string} id The ID of the element to change
* @param int action Set to 0 if element display should be toggled, -1 for * @param {int} action Set to 0 if element display should be toggled, -1 for
* hiding the element, and 1 for showing it. * hiding the element, and 1 for showing it.
* @param string type Display type that should be used, e.g. inline, block or * @param {string} type Display type that should be used, e.g. inline, block or
* other CSS "display" types * other CSS "display" types
*/ */
phpbb.toggleDisplay = function(id, action, type) { phpbb.toggleDisplay = function(id, action, type) {
@ -1528,8 +1503,7 @@ phpbb.toggleDisplay = function(id, action, type) {
* Toggle additional settings based on the selected * Toggle additional settings based on the selected
* option of select element. * option of select element.
* *
* @param jQuery el jQuery select element object. * @param {jQuery} el jQuery select element object.
* @return undefined
*/ */
phpbb.toggleSelectSettings = function(el) { phpbb.toggleSelectSettings = function(el) {
el.children().each(function() { el.children().each(function() {
@ -1543,8 +1517,8 @@ phpbb.toggleSelectSettings = function(el) {
* Get function from name. * Get function from name.
* Based on http://stackoverflow.com/a/359910 * Based on http://stackoverflow.com/a/359910
* *
* @param string functionName Function to get. * @param {string} functionName Function to get.
* @return function * @returns function
*/ */
phpbb.getFunctionByName = function (functionName) { phpbb.getFunctionByName = function (functionName) {
var namespaces = functionName.split('.'), var namespaces = functionName.split('.'),

View file

@ -7,8 +7,6 @@ phpbb.plupload.ids = [];
/** /**
* Set up the uploader. * Set up the uploader.
*
* @return undefined
*/ */
phpbb.plupload.initialize = function() { phpbb.plupload.initialize = function() {
// Initialize the Plupload uploader. // Initialize the Plupload uploader.
@ -48,8 +46,6 @@ phpbb.plupload.initialize = function() {
/** /**
* Unsets all elements in the object uploader.settings.multipart_params whose keys * Unsets all elements in the object uploader.settings.multipart_params whose keys
* begin with 'attachment_data[' * begin with 'attachment_data['
*
* @return undefined
*/ */
phpbb.plupload.clearParams = function() { phpbb.plupload.clearParams = function() {
var obj = phpbb.plupload.uploader.settings.multipart_params; var obj = phpbb.plupload.uploader.settings.multipart_params;
@ -65,8 +61,7 @@ phpbb.plupload.clearParams = function() {
/** /**
* Update uploader.settings.multipart_params object with new data. * Update uploader.settings.multipart_params object with new data.
* *
* @param object obj * @param {object} obj
* @return undefined
*/ */
phpbb.plupload.updateMultipartParams = function(obj) { phpbb.plupload.updateMultipartParams = function(obj) {
phpbb.plupload.uploader.settings.multipart_params = $.extend( phpbb.plupload.uploader.settings.multipart_params = $.extend(
@ -78,7 +73,7 @@ phpbb.plupload.updateMultipartParams = function(obj) {
/** /**
* Convert the array of attachment objects into an object that PHP would expect as POST data. * Convert the array of attachment objects into an object that PHP would expect as POST data.
* *
* @return object An object in the form 'attachment_data[i][key]': value as * @returns {object} An object in the form 'attachment_data[i][key]': value as
* expected by the server * expected by the server
*/ */
phpbb.plupload.getSerializedData = function() { phpbb.plupload.getSerializedData = function() {
@ -100,9 +95,8 @@ phpbb.plupload.getSerializedData = function() {
* Get the index from the phpbb.plupload.data array where the given * Get the index from the phpbb.plupload.data array where the given
* attachment id appears. * attachment id appears.
* *
* @param int attach_id The attachment id of the file. * @param {int} attach_id The attachment id of the file.
* @return bool Returns false if the id cannot be found. * @returns {bool|int} Index of the file if exists, otherwise false.
* @return int Returns the index of the file if it exists.
*/ */
phpbb.plupload.getIndex = function(attach_id) { phpbb.plupload.getIndex = function(attach_id) {
var index = $.inArray(Number(attach_id), phpbb.plupload.ids); var index = $.inArray(Number(attach_id), phpbb.plupload.ids);
@ -112,10 +106,8 @@ phpbb.plupload.getIndex = function(attach_id) {
/** /**
* Set the data in phpbb.plupload.data and phpbb.plupload.ids arrays. * Set the data in phpbb.plupload.data and phpbb.plupload.ids arrays.
* *
* @param array data Array containing the new data to use. In the form of * @param {Array} data Array containing the new data to use. In the form of
* array(index => object(property: value). Requires attach_id to be one of the object properties. * array(index => object(property: value). Requires attach_id to be one of the object properties.
*
* @return undefined
*/ */
phpbb.plupload.setData = function(data) { phpbb.plupload.setData = function(data) {
// Make sure that the array keys are reset. // Make sure that the array keys are reset.
@ -130,11 +122,10 @@ phpbb.plupload.setData = function(data) {
/** /**
* Update the attachment data in the HTML and the phpbb & phpbb.plupload objects. * Update the attachment data in the HTML and the phpbb & phpbb.plupload objects.
* *
* @param array data Array containing the new data to use. * @param {Array} data Array containing the new data to use.
* @param string action The action that required the update. Used to update the inline attachment bbcodes. * @param {string} action The action that required the update. Used to update the inline attachment bbcodes.
* @param int index The index from phpbb.plupload_ids that was affected by the action. * @param {int} index The index from phpbb.plupload_ids that was affected by the action.
* @param array downloadUrl Optional array of download urls to update. * @param {Array} downloadUrl Optional array of download urls to update.
* @return undefined
*/ */
phpbb.plupload.update = function(data, action, index, downloadUrl) { phpbb.plupload.update = function(data, action, index, downloadUrl) {
@ -148,8 +139,7 @@ phpbb.plupload.update = function(data, action, index, downloadUrl) {
/** /**
* Update the relevant elements and hidden data for all attachments. * Update the relevant elements and hidden data for all attachments.
* *
* @param array downloadUrl Optional array of download urls to update. * @param {Array} downloadUrl Optional array of download urls to update.
* @return undefined
*/ */
phpbb.plupload.updateRows = function(downloadUrl) { phpbb.plupload.updateRows = function(downloadUrl) {
for (var i = 0; i < phpbb.plupload.ids.length; i++) { for (var i = 0; i < phpbb.plupload.ids.length; i++) {
@ -163,9 +153,8 @@ phpbb.plupload.updateRows = function(downloadUrl) {
* data for the file inserted into it. The row is then appended or prepended to * data for the file inserted into it. The row is then appended or prepended to
* #file-list based on the attach_order setting. * #file-list based on the attach_order setting.
* *
* @param object file Plupload file object for the new attachment. * @param {object} file Plupload file object for the new attachment.
* @return undefined */{}
*/
phpbb.plupload.insertRow = function(file) { phpbb.plupload.insertRow = function(file) {
var row = $(phpbb.plupload.rowTpl); var row = $(phpbb.plupload.rowTpl);
@ -183,9 +172,8 @@ phpbb.plupload.insertRow = function(file) {
/** /**
* Update the relevant elements and hidden data for an attachment. * Update the relevant elements and hidden data for an attachment.
* *
* @param int index The index from phpbb.plupload.ids of the attachment to edit. * @param {int} index The index from phpbb.plupload.ids of the attachment to edit.
* @param array downloadUrl Optional array of download urls to update. * @param {Array} downloadUrl Optional array of download urls to update.
* @return undefined
*/ */
phpbb.plupload.updateRow = function(index, downloadUrl) { phpbb.plupload.updateRow = function(index, downloadUrl) {
var attach = phpbb.plupload.data[index], var attach = phpbb.plupload.data[index],
@ -194,7 +182,7 @@ phpbb.plupload.updateRow = function(index, downloadUrl) {
// Add the link to the file // Add the link to the file
if (typeof downloadUrl !== 'undefined' && typeof downloadUrl[index] !== 'undefined') { if (typeof downloadUrl !== 'undefined' && typeof downloadUrl[index] !== 'undefined') {
var url = downloadUrl[index].replace('&amp;', '&'), var url = downloadUrl[index].replace('&amp;', '&'),
link = $('<a></a>'); link = ${('}<a></a>');
link.attr('href', url).html(attach.real_filename); link.attr('href', url).html(attach.real_filename);
row.find('.file-name').html(link) row.find('.file-name').html(link)
@ -207,10 +195,9 @@ phpbb.plupload.updateRow = function(index, downloadUrl) {
/** /**
* Update hidden input data for an attachment. * Update hidden input data for an attachment.
* *
* @param object row jQuery object for the attachment row. * @param {object} row jQuery object for the attachment row.
* @param object attach Attachment data object from phpbb.plupload.data * @param {object} attach Attachment data object from phpbb.plupload.data
* @param int index Attachment index from phpbb.plupload.ids * @param {int} index Attachment index from phpbb.plupload.ids
* @return undefined
*/ */
phpbb.plupload.updateHiddenData = function(row, attach, index) { phpbb.plupload.updateHiddenData = function(row, attach, index) {
row.find('input[type="hidden"]').remove(); row.find('input[type="hidden"]').remove();
@ -230,10 +217,8 @@ phpbb.plupload.updateHiddenData = function(row, attach, index) {
* responds with the updated attachment data list so that any future * responds with the updated attachment data list so that any future
* uploads can maintain state with the server * uploads can maintain state with the server
* *
* @param object row jQuery object for the attachment row. * @param {object} row jQuery object for the attachment row.
* @param int attachId Attachment id of the file to be removed. * @param {int} attachId Attachment id of the file to be removed.
*
* @return undefined
*/ */
phpbb.plupload.deleteFile = function(row, attachId) { phpbb.plupload.deleteFile = function(row, attachId) {
// If there's no attach id, then the file hasn't been uploaded. Simply delete the row. // If there's no attach id, then the file hasn't been uploaded. Simply delete the row.
@ -300,8 +285,6 @@ phpbb.plupload.deleteFile = function(row, attachId) {
/** /**
* Check the attachment list and hide its container if it's empty. * Check the attachment list and hide its container if it's empty.
*
* @return undefined
*/ */
phpbb.plupload.hideEmptyList = function() { phpbb.plupload.hideEmptyList = function() {
if (!$('#file-list').children().length) { if (!$('#file-list').children().length) {
@ -315,10 +298,8 @@ phpbb.plupload.hideEmptyList = function() {
* before the phpbb.plupload,data and phpbb.plupload.ids arrays are updated, otherwise it will * before the phpbb.plupload,data and phpbb.plupload.ids arrays are updated, otherwise it will
* not work correctly. * not work correctly.
* *
* @param string action The action that occurred -- either "addition" or "removal" * @param {string} action The action that occurred -- either "addition" or "removal"
* @param int index The index of the attachment from phpbb.plupload.ids that was affected. * @param {int} index The index of the attachment from phpbb.plupload.ids that was affected.
*
* @return undefined
*/ */
phpbb.plupload.updateBbcode = function(action, index) { phpbb.plupload.updateBbcode = function(action, index) {
var textarea = $('#message', phpbb.plupload.form), var textarea = $('#message', phpbb.plupload.form),
@ -366,10 +347,10 @@ phpbb.plupload.updateBbcode = function(action, index) {
/** /**
* Get Plupload file objects based on their upload status. * Get Plupload file objects based on their upload status.
* *
* @param int status Plupload status - plupload.DONE, plupload.FAILED, plupload.QUEUED, * @param {int} status Plupload status - plupload.DONE, plupload.FAILED,
* plupload.STARTED, plupload.STOPPED * plupload.QUEUED, plupload.STARTED, plupload.STOPPED
* *
* @return Returns an array of the Plupload file objects matching the status. * @returns {Array} The Plupload file objects matching the status.
*/ */
phpbb.plupload.getFilesByStatus = function(status) { phpbb.plupload.getFilesByStatus = function(status) {
var files = []; var files = [];
@ -387,7 +368,7 @@ phpbb.plupload.getFilesByStatus = function(status) {
* to upload. If so, disables the uploader and marks the queued files as failed. Otherwise * to upload. If so, disables the uploader and marks the queued files as failed. Otherwise
* makes sure that the uploader is enabled. * makes sure that the uploader is enabled.
* *
* @return bool Returns true if the limit has been reached. False if otherwise. * @returns {bool} True if the limit has been reached. False if otherwise.
*/ */
phpbb.plupload.handleMaxFilesReached = function() { phpbb.plupload.handleMaxFilesReached = function() {
// If there is no limit, the user is an admin or moderator. // If there is no limit, the user is an admin or moderator.
@ -412,8 +393,6 @@ phpbb.plupload.handleMaxFilesReached = function() {
/** /**
* Disable the uploader * Disable the uploader
*
* @return undefined
*/ */
phpbb.plupload.disableUploader = function() { phpbb.plupload.disableUploader = function() {
$('#add_files').addClass('disabled'); $('#add_files').addClass('disabled');
@ -422,8 +401,6 @@ phpbb.plupload.disableUploader = function() {
/** /**
* Enable the uploader * Enable the uploader
*
* @return undefined
*/ */
phpbb.plupload.enableUploader = function() { phpbb.plupload.enableUploader = function() {
$('#add_files').removeClass('disabled'); $('#add_files').removeClass('disabled');
@ -433,8 +410,7 @@ phpbb.plupload.enableUploader = function() {
/** /**
* Mark all queued files as failed. * Mark all queued files as failed.
* *
* @param string error Error message to present to the user. * @param {string} error Error message to present to the user.
* @return undefined
*/ */
phpbb.plupload.markQueuedFailed = function(error) { phpbb.plupload.markQueuedFailed = function(error) {
var files = phpbb.plupload.getFilesByStatus(plupload.QUEUED); var files = phpbb.plupload.getFilesByStatus(plupload.QUEUED);
@ -448,9 +424,8 @@ phpbb.plupload.markQueuedFailed = function(error) {
/** /**
* Marks a file as failed and sets the error message for it. * Marks a file as failed and sets the error message for it.
* *
* @param object file Plupload file object that failed. * @param {object} file Plupload file object that failed.
* @param string error Error message to present to the user. * @param {string} error Error message to present to the user.
* @return undefined
*/ */
phpbb.plupload.fileError = function(file, error) { phpbb.plupload.fileError = function(file, error) {
file.status = plupload.FAILED; file.status = plupload.FAILED;
@ -520,11 +495,8 @@ phpbb.plupload.uploader.bind('Error', function(up, error) {
* send the real filename along with the chunk. This is necessary because * send the real filename along with the chunk. This is necessary because
* for some reason the filename is set to 'blob' whenever a file is chunked * for some reason the filename is set to 'blob' whenever a file is chunked
* *
* @param object up The plupload.Uploader object * @param {object} up The plupload.Uploader object
* @param object file The plupload.File object that is about to be * @param {object} file The plupload.File object that is about to be uploaded
* uploaded
*
* @return undefined
*/ */
phpbb.plupload.uploader.bind('BeforeUpload', function(up, file) { phpbb.plupload.uploader.bind('BeforeUpload', function(up, file) {
if (phpbb.plupload.handleMaxFilesReached()) { if (phpbb.plupload.handleMaxFilesReached()) {
@ -539,12 +511,10 @@ phpbb.plupload.uploader.bind('BeforeUpload', function(up, file) {
* response from the server and checks for an error. If an error occurs it * response from the server and checks for an error. If an error occurs it
* is reported to the user and the upload of this particular file is halted * is reported to the user and the upload of this particular file is halted
* *
* @param object up The plupload.Uploader object * @param {object} up The plupload.Uploader object
* @param object file The plupload.File object whose chunk has just * @param {object} file The plupload.File object whose chunk has just
* been uploaded * been uploaded
* @param object response The response object from the server * @param {object} response The response object from the server
*
* @return undefined
*/ */
phpbb.plupload.uploader.bind('ChunkUploaded', function(up, file, response) { phpbb.plupload.uploader.bind('ChunkUploaded', function(up, file, response) {
if (response.chunk >= response.chunks - 1) { if (response.chunk >= response.chunks - 1) {
@ -584,8 +554,6 @@ phpbb.plupload.uploader.bind('ChunkUploaded', function(up, file, response) {
/** /**
* Fires when files are added to the queue. * Fires when files are added to the queue.
*
* @return undefined
*/ */
phpbb.plupload.uploader.bind('FilesAdded', function(up, files) { phpbb.plupload.uploader.bind('FilesAdded', function(up, files) {
// Prevent unnecessary requests to the server if the user already uploaded // Prevent unnecessary requests to the server if the user already uploaded
@ -627,12 +595,10 @@ phpbb.plupload.uploader.bind('FilesAdded', function(up, files) {
* appends it to the next file upload so that the server can maintain state * appends it to the next file upload so that the server can maintain state
* with regards to the attachments in a given post * with regards to the attachments in a given post
* *
* @param object up The plupload.Uploader object * @param {object} up The plupload.Uploader object
* @param object file The plupload.File object that has just been * @param {object} file The plupload.File object that has just been
* uploaded * uploaded
* @param string response The response string from the server * @param {string} response The response string from the server
*
* @return undefined
*/ */
phpbb.plupload.uploader.bind('FileUploaded', function(up, file, response) { phpbb.plupload.uploader.bind('FileUploaded', function(up, file, response) {
var json = {}, var json = {},
@ -674,11 +640,9 @@ phpbb.plupload.uploader.bind('FileUploaded', function(up, file, response) {
/** /**
* Fires when the entire queue of files have been uploaded. * Fires when the entire queue of files have been uploaded.
* *
* @param object up The plupload.Uploader object * @param {object} up The plupload.Uploader object
* @param array files An array of plupload.File objects that have just * @param {Array} files An array of plupload.File objects that have just
* been uploaded as part of a queue * been uploaded as part of a queue
*
* @return undefined
*/ */
phpbb.plupload.uploader.bind('UploadComplete', function(up, files) { phpbb.plupload.uploader.bind('UploadComplete', function(up, files) {
// Hide the progress bar // Hide the progress bar

View file

@ -42,8 +42,8 @@ phpbb.addAjaxCallback('mark_forums_read', function(res) {
/** /**
* This callback will mark all topic icons read * This callback will mark all topic icons read
* *
* @param update_topic_links bool Whether "Mark topics read" links should be * @param {bool} [update_topic_links=true] Whether "Mark topics read" links
* updated. Defaults to true. * should be updated. Defaults to true.
*/ */
phpbb.addAjaxCallback('mark_topics_read', function(res, updateTopicLinks) { phpbb.addAjaxCallback('mark_topics_read', function(res, updateTopicLinks) {
var readTitle = res.NO_UNREAD_POSTS; var readTitle = res.NO_UNREAD_POSTS;