diff options
| author | Marc Alexander <admin@m-a-styles.de> | 2019-11-27 22:01:26 +0100 |
|---|---|---|
| committer | Marc Alexander <admin@m-a-styles.de> | 2019-11-27 22:01:26 +0100 |
| commit | 417271f5738b0a73bc3dba28f516675ca3d146ea (patch) | |
| tree | f47c3889d063f886158929fa85df8d49eb340ae8 /phpBB/includes | |
| parent | 7b428641f04766ea8711cb47e76bbe2b52638abe (diff) | |
| download | forums-417271f5738b0a73bc3dba28f516675ca3d146ea.tar forums-417271f5738b0a73bc3dba28f516675ca3d146ea.tar.gz forums-417271f5738b0a73bc3dba28f516675ca3d146ea.tar.bz2 forums-417271f5738b0a73bc3dba28f516675ca3d146ea.tar.xz forums-417271f5738b0a73bc3dba28f516675ca3d146ea.zip | |
[ticket/16211] Use form to ensure link data is passed on coppa registration
PHPBB3-16211
Diffstat (limited to 'phpBB/includes')
| -rw-r--r-- | phpBB/includes/ucp/ucp_register.php | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/phpBB/includes/ucp/ucp_register.php b/phpBB/includes/ucp/ucp_register.php index 29829c2e68..03ac63b12b 100644 --- a/phpBB/includes/ucp/ucp_register.php +++ b/phpBB/includes/ucp/ucp_register.php @@ -39,8 +39,8 @@ class ucp_register trigger_error('UCP_REGISTER_DISABLE'); } - $coppa = $request->is_set('coppa') ? (int) $request->variable('coppa', false) : false; - $token = $request->variable('hash', ''); + $coppa = $request->is_set('coppa_yes') ? 1 : ($request->is_set('coppa_no') ? 0 : false); + $coppa = $request->is_set('coppa') ? $request->variable('coppa', 0) : $coppa; $agreed = $request->variable('agreed', false); $submit = $request->is_set_post('submit'); $change_lang = $request->variable('change_lang', ''); @@ -51,7 +51,7 @@ class ucp_register $agreed = false; } - if ($coppa !== false && !check_link_hash($token, 'coppa') && !check_form_key('ucp_register')) + if ($coppa !== false && !check_form_key('ucp_register')) { $coppa = false; } @@ -170,15 +170,10 @@ class ucp_register ->format($user->lang['DATE_FORMAT'], true); unset($now); - $coppa_link_hash = '&hash=' . generate_link_hash('coppa'); - $template_vars = array( 'S_LANG_OPTIONS' => (count($lang_row) > 1) ? language_select($user_lang) : '', - 'L_COPPA_NO' => sprintf($user->lang['UCP_COPPA_BEFORE'], $coppa_birthday), - 'L_COPPA_YES' => sprintf($user->lang['UCP_COPPA_ON_AFTER'], $coppa_birthday), - - 'U_COPPA_NO' => append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=register&coppa=0' . $coppa_link_hash), - 'U_COPPA_YES' => append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=register&coppa=1' . $coppa_link_hash), + 'L_COPPA_NO' => $user->lang('UCP_COPPA_BEFORE', $coppa_birthday), + 'L_COPPA_YES' => $user->lang('UCP_COPPA_ON_AFTER', $coppa_birthday), 'S_SHOW_COPPA' => true, 'S_HIDDEN_FIELDS' => build_hidden_fields($s_hidden_fields), |