diff options
| author | Tristan Darricau <tristan.darricau@sensiolabs.com> | 2015-10-08 15:42:33 +0200 |
|---|---|---|
| committer | Tristan Darricau <tristan.darricau@sensiolabs.com> | 2015-10-08 15:42:33 +0200 |
| commit | 542498e457c5e354a39c7cf3eb11f66f263acd59 (patch) | |
| tree | b44486de2557c166afd9c29db18dc51f576f4337 /phpBB/includes/acp | |
| parent | c38a128663aada4a6c9a9d4ce4159dc193e00513 (diff) | |
| parent | bd40f2f6198bc092ba20b2ec79648c425cec6c2e (diff) | |
| download | forums-542498e457c5e354a39c7cf3eb11f66f263acd59.tar forums-542498e457c5e354a39c7cf3eb11f66f263acd59.tar.gz forums-542498e457c5e354a39c7cf3eb11f66f263acd59.tar.bz2 forums-542498e457c5e354a39c7cf3eb11f66f263acd59.tar.xz forums-542498e457c5e354a39c7cf3eb11f66f263acd59.zip | |
Merge pull request #3724 from javiexin/ticket/13960
[ticket/13960] Profile field validation breaks ACP
* javiexin/ticket/13960:
[ticket/13960] Profile field validation breaks ACP
Diffstat (limited to 'phpBB/includes/acp')
| -rw-r--r-- | phpBB/includes/acp/acp_profile.php | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/phpBB/includes/acp/acp_profile.php b/phpBB/includes/acp/acp_profile.php index 43668b8ad5..abf84e84aa 100644 --- a/phpBB/includes/acp/acp_profile.php +++ b/phpBB/includes/acp/acp_profile.php @@ -537,13 +537,14 @@ class acp_profile } } - $step = (isset($_REQUEST['next'])) ? $step + 1 : ((isset($_REQUEST['prev'])) ? $step - 1 : $step); - if (sizeof($error)) { - $step--; $submit = false; } + else + { + $step = (isset($_REQUEST['next'])) ? $step + 1 : ((isset($_REQUEST['prev'])) ? $step - 1 : $step); + } // Build up the specific hidden fields foreach ($exclude as $num => $key_ary) @@ -561,7 +562,7 @@ class acp_profile $var = $profile_field->prepare_hidden_fields($step, $key, $action, $field_data); if ($var !== null) { - $_new_key_ary[$key] = $profile_field->prepare_hidden_fields($step, $key, $action, $field_data); + $_new_key_ary[$key] = $var; } } $cp->vars = $field_data; @@ -571,11 +572,7 @@ class acp_profile if (!sizeof($error)) { - if ($step == 3 && (sizeof($this->lang_defs['iso']) == 1 || $save)) - { - $this->save_profile_field($cp, $field_type, $action); - } - else if ($action == 'edit' && $save) + if (($step == 3 && (sizeof($this->lang_defs['iso']) == 1 || $save)) || ($action == 'edit' && $save)) { $this->save_profile_field($cp, $field_type, $action); } |