diff options
author | Marc Alexander <admin@m-a-styles.de> | 2013-05-13 12:17:31 +0200 |
---|---|---|
committer | Marc Alexander <admin@m-a-styles.de> | 2013-05-13 12:17:31 +0200 |
commit | 6890bf9f8d97473d55f2d6819c8ec155f9d52658 (patch) | |
tree | 4537a0662be81c68581f644517550c3816f556c4 /phpBB/includes/ucp/ucp_groups.php | |
parent | bcf7ea33106ba99aee20d672eda3d58c95251f11 (diff) | |
download | forums-6890bf9f8d97473d55f2d6819c8ec155f9d52658.tar forums-6890bf9f8d97473d55f2d6819c8ec155f9d52658.tar.gz forums-6890bf9f8d97473d55f2d6819c8ec155f9d52658.tar.bz2 forums-6890bf9f8d97473d55f2d6819c8ec155f9d52658.tar.xz forums-6890bf9f8d97473d55f2d6819c8ec155f9d52658.zip |
[ticket/11535] Correctly merge avatar_errors array into primary error array
The $avatar_errors array needs to be merged into the primary $error array
before the group settings get applied. This is currently not the case.
Functional tests for this will be provided by PR #1401.
PHPBB3-11535
Diffstat (limited to 'phpBB/includes/ucp/ucp_groups.php')
-rw-r--r-- | phpBB/includes/ucp/ucp_groups.php | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/phpBB/includes/ucp/ucp_groups.php b/phpBB/includes/ucp/ucp_groups.php index 50d13e00b1..a633ce448c 100644 --- a/phpBB/includes/ucp/ucp_groups.php +++ b/phpBB/includes/ucp/ucp_groups.php @@ -547,6 +547,9 @@ class ucp_groups $submit_ary['avatar_width'] = 0; $submit_ary['avatar_height'] = 0; } + + // Merge any avatars errors into the primary error array + $error = array_merge($error, $phpbb_avatar_manager->localize_errors($user, $avatar_error)); } if (!check_form_key('ucp_groups')) @@ -672,8 +675,11 @@ class ucp_groups } } - // Merge any avatars errors into the primary error array - $error = array_merge($error, $phpbb_avatar_manager->localize_errors($user, $avatar_error)); + if (!$update) + { + // Merge any avatars errors into the primary error array + $error = array_merge($error, $phpbb_avatar_manager->localize_errors($user, $avatar_error)); + } $template->assign_vars(array( 'S_EDIT' => true, |