1
0
mirror of https://github.com/phpbb/phpbb.git synced 2025-05-12 10:35:20 +02:00

[feature/oauth] Login works on login_link now, still does not actually link

PHPBB3-11673
This commit is contained in:
Joseph Warner 2013-07-29 12:55:57 -04:00
parent 75206c74be
commit bcdeafedd7
2 changed files with 88 additions and 11 deletions

View File

@ -27,7 +27,7 @@ class ucp_login_link
function main($id, $mode)
{
global $config, $phpbb_container, $request, $template, $user;
global $auth, $config, $phpbb_container, $request, $template, $user;
$auth_provider = 'auth.provider.' . $request->variable('auth_provider', $config['auth_method']);
$auth_provider = $phpbb_container->get($auth_provider);
@ -35,25 +35,84 @@ class ucp_login_link
// Initialize necessary variables
$login_link_error = null;
// Ensure the person was sent here with login_link data
$data = $request->variable('login_link', array());
// Build the data array
$data = $this->get_login_link_data_array();
// Ensure the person was sent here with login_link data
if (empty($data))
{
$login_link_error = $user->lang['LOGIN_LINK_NO_DATA_PROVIDED'];
} else {
}
// Process POST and GET data
$login_error = false;
$login_username = '';
// Have the authentication provider check that all necessary data is available
// Perform link action if there is no error
if (!login_link_error)
{
if ($request->is_set_post('login'))
{
$login_username = $request->variable('login_username', '', false, phpbb_request_interface::POST);
$login_password = $request->untrimmed_variable('password', '', true, phpbb_request_interface::POST);
$result = $auth->login($login_username, $login_password);
if ($result['status'] != LOGIN_SUCCESS)
{
// Handle all errors first
if ($result['status'] == LOGIN_BREAK)
{
trigger_error($result['error_msg']);
}
switch ($result['status'])
{
case LOGIN_ERROR_ATTEMPTS:
$captcha = phpbb_captcha_factory::get_instance($config['captcha_plugin']);
$captcha->init(CONFIRM_LOGIN);
$template->assign_vars(array(
'CAPTCHA_TEMPLATE' => $captcha->get_template(),
));
$login_error = $user->lang[$result['error_msg']];
break;
case LOGIN_ERROR_PASSWORD_CONVERT:
$login_error = sprintf(
$user->lang[$result['error_msg']],
($config['email_enable']) ? '<a href="' . append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=sendpassword') . '">' : '',
($config['email_enable']) ? '</a>' : '',
($config['board_contact']) ? '<a href="mailto:' . htmlspecialchars($config['board_contact']) . '">' : '',
($config['board_contact']) ? '</a>' : ''
);
break;
// Username, password, etc...
default:
$login_error = $user->lang[$result['error_msg']];
// Assign admin contact to some error messages
if ($result['error_msg'] == 'LOGIN_ERROR_USERNAME' || $result['error_msg'] == 'LOGIN_ERROR_PASSWORD')
{
$login_error = (!$config['board_contact']) ? sprintf($user->lang[$result['error_msg']], '', '') : sprintf($user->lang[$result['error_msg']], '<a href="mailto:' . htmlspecialchars($config['board_contact']) . '">', '</a>');
}
break;
}
} else {
// The user is now logged in, attempt to link the user to the external account
$auth_provider->link_account($data);
}
}
}
// Common template elements
$template->assign_vars(array(
'LOGIN_LINK_ERROR' => $login_link_error,
'PASSWORD_CREDENTIAL' => 'password',
'USERNAME_CREDENTIAL' => 'username',
'PASSWORD_CREDENTIAL' => 'login_password',
'USERNAME_CREDENTIAL' => 'login_username',
));
// Registration template
@ -72,4 +131,22 @@ class ucp_login_link
$this->tpl_name = 'ucp_login_link';
$this->page_title = 'UCP_LOGIN_LINK';
}
protected function get_login_link_data_array()
{
global $request;
$var_names = $request->variable_names(phpbb_request_interface::GET);
$login_link_data = array();
foreach ($var_names as $var_name)
{
if (strpos($var_name, 'login_link_') === 0)
{
$login_link_data[$var_name] = $request->variable($var_name, '', false, phpbb_request_interface::GET);
}
}
return $login_link_data;
}
}

View File

@ -8,7 +8,7 @@
<p>{L_LOGIN_LINK_EXPLAIN}</p>
<!-- IF LOGIN_LINK_ERROR --><div class="content">
<div class="error">{LOGIN_ERROR}</div>
<div class="error">{LOGIN_LINK_ERROR}</div>
</div><!-- ENDIF -->
<div class="content">