Skip to content

Commit

Permalink
Fixing ordering of parameters for array_merge.
Browse files Browse the repository at this point in the history
  • Loading branch information
euantorano committed Jun 29, 2014
1 parent 1eec220 commit 34213ca
Show file tree
Hide file tree
Showing 6 changed files with 42 additions and 35 deletions.
2 changes: 1 addition & 1 deletion admin/inc/functions_view_manager.php
Expand Up @@ -117,7 +117,7 @@ function view_manager($base_url, $type, $fields, $sort_options=array(), $conditi
}
else
{
$mybb->input = array_merge(array('perpage' => 20, $mybb->input));
$mybb->input = array_merge($mybb->input, array('perpage' => 20));
}

// Write in our JS based field selector
Expand Down
13 changes: 7 additions & 6 deletions admin/modules/config/calendars.php
Expand Up @@ -76,12 +76,13 @@
}
else
{
$mybb->input = array_merge(array(
"allowhtml" => 0,
"eventlimit" => 4,
"disporder" => 1,
"moderation" => 0
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"allowhtml" => 0,
"eventlimit" => 4,
"disporder" => 1,
"moderation" => 0
)
);
}

$page->add_breadcrumb_item($lang->add_calendar);
Expand Down
31 changes: 17 additions & 14 deletions admin/modules/config/warning.php
Expand Up @@ -281,9 +281,10 @@ function checkAction(id)
}
else
{
$mybb->input = array_merge(array(
"percentage" => $level['percentage'],
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"percentage" => $level['percentage'],
)
);
$action = unserialize($level['action']);
if($action['type'] == 1)
{
Expand Down Expand Up @@ -469,11 +470,12 @@ function checkAction(id)
}
else
{
$mybb->input = array_merge(array(
"points" => "2",
"expire_time" => 1,
"expire_period" => "days"
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"points" => "2",
"expire_time" => 1,
"expire_period" => "days"
)
);
}

$page->add_breadcrumb_item($lang->add_warning_type);
Expand Down Expand Up @@ -557,12 +559,13 @@ function checkAction(id)
else
{
$expiration = fetch_friendly_expiration($type['expirationtime']);
$mybb->input = array_merge(array(
"title" => $type['title'],
"points" => $type['points'],
"expire_time" => $expiration['time'],
"expire_period" => $expiration['period']
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"title" => $type['title'],
"points" => $type['points'],
"expire_time" => $expiration['time'],
"expire_period" => $expiration['period']
)
);
}

$page->add_breadcrumb_item($lang->edit_warning_type);
Expand Down
18 changes: 10 additions & 8 deletions admin/modules/user/groups.php
Expand Up @@ -453,11 +453,12 @@
}
else
{
$mybb->input = array_merge(array(
"canmanagemembers" => 1,
"canmanagerequests" => 1,
"caninvitemembers" => 1
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"canmanagemembers" => 1,
"canmanagerequests" => 1,
"caninvitemembers" => 1
)
);
}

$form_container = new FormContainer($lang->add_group_leader." {$group['title']}");
Expand Down Expand Up @@ -694,9 +695,10 @@
}
else
{
$mybb->input = array_merge(array(
"namestyle" => "{username}"
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
"namestyle" => "{username}"
)
);
}

$form_container = new FormContainer($lang->add_user_group);
Expand Down
9 changes: 5 additions & 4 deletions admin/modules/user/titles.php
Expand Up @@ -78,10 +78,11 @@
}
else
{
$mybb->input = array_merge(array(
'stars' => '1',
'starimage' => '{theme}/star.png',
), $mybb->input);
$mybb->input = array_merge($mybb->input, array(
'stars' => '1',
'starimage' => '{theme}/star.png',
)
);
}

$page->add_breadcrumb_item($lang->add_new_user_title);
Expand Down
4 changes: 2 additions & 2 deletions admin/modules/user/users.php
Expand Up @@ -385,7 +385,7 @@
}
else
{
$mybb->input = array_merge(array('usergroup' => 2), $mybb->input);
$mybb->input = array_merge($mybb->input, array('usergroup' => 2));
}

$form_container = new FormContainer($lang->required_profile_info);
Expand Down Expand Up @@ -1300,7 +1300,7 @@
);
$form_container->output_row($lang->thread_view_options, "", "<div class=\"user_settings_bit\">".implode("</div><div class=\"user_settings_bit\">", $thread_options)."</div>");

$languages = array_merge(array('' => $lang->use_default), $lang->get_languages());
$languages = array_merge($lang->get_languages(), array('' => $lang->use_default));

$other_options = array(
$form->generate_check_box("showredirect", 1, $lang->show_redirect, array("checked" => $mybb->input['showredirect'])),
Expand Down

0 comments on commit 34213ca

Please sign in to comment.