aboutsummaryrefslogtreecommitdiffstats
path: root/phpBB/includes/ucp/ucp_groups.php
diff options
context:
space:
mode:
authorMarc Alexander <admin@m-a-styles.de>2013-05-13 12:17:31 +0200
committerMarc Alexander <admin@m-a-styles.de>2013-05-13 12:17:31 +0200
commit6890bf9f8d97473d55f2d6819c8ec155f9d52658 (patch)
tree4537a0662be81c68581f644517550c3816f556c4 /phpBB/includes/ucp/ucp_groups.php
parentbcf7ea33106ba99aee20d672eda3d58c95251f11 (diff)
downloadforums-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.php10
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,