[ticket/12982] Refactoring: Cleaned up crazy insert users stuff

PHPBB3-12982
This commit is contained in:
Callum Macrae 2014-08-14 00:19:34 +01:00
parent 9e3db526d5
commit 9c87972385

View file

@ -279,11 +279,13 @@ jQuery(function($) {
/** /**
* Functions for user search popup * Functions for user search popup
*/ */
function insert_user(formId, value) function insertUser(formId, value)
{ {
var form = jQuery(formId), 'use strict';
formName = form.attr('data-form-name'),
fieldName = form.attr('data-field-name'), var $form = jQuery(formId),
formName = $form.attr('data-form-name'),
fieldName = $form.attr('data-field-name'),
item = opener.document.forms[formName][fieldName]; item = opener.document.forms[formName][fieldName];
if (item.value.length && item.type == 'textarea') { if (item.value.length && item.type == 'textarea') {
@ -293,33 +295,23 @@ function insert_user(formId, value)
item.value = value; item.value = value;
} }
function insert_marked_users(formId, users) function insert_marked_users(formId, users) {
{ 'use strict';
if (typeof(users.length) == "undefined")
{ for (var i = 0; i < users.length; i++) {
if (users.checked) if (users[i].checked) {
{ insertUser(formId, users[i].value);
insert_user(formId, users.value);
}
}
else if (users.length > 0)
{
for (i = 0; i < users.length; i++)
{
if (users[i].checked)
{
insert_user(formId, users[i].value);
}
} }
} }
self.close(); window.close();
} }
function insert_single_user(formId, user) function insert_single_user(formId, user) {
{ 'use strict';
insertUser(formId, user); insertUser(formId, user);
self.close(); window.close();
} }
/** /**