fixup syntax error in openid.php

darcs-hash:20080618150557-84dde-61120bcf12e9ecfaafda079d1a943c75f99f5930.gz
This commit is contained in:
Evan Prodromou 2008-06-18 11:05:57 -04:00
parent 23c0b1f482
commit 30c3cd552b
2 changed files with 76 additions and 75 deletions

View File

@ -28,7 +28,8 @@ class OpenidloginAction extends Action {
if (common_logged_in()) { if (common_logged_in()) {
common_user_error(_t('Already logged in.')); common_user_error(_t('Already logged in.'));
} else if ($_SERVER['REQUEST_METHOD'] == 'POST') { } else if ($_SERVER['REQUEST_METHOD'] == 'POST') {
$result = oid_authenticate($this->trimmed('openid_url'), 'finishopenidlogin'); $result = oid_authenticate($this->trimmed('openid_url'),
'finishopenidlogin');
if (is_string($result)) { # error message if (is_string($result)) { # error message
$this->show_form($result); $this->show_form($result);
} }
@ -45,9 +46,10 @@ class OpenidloginAction extends Action {
common_element('div', 'instructions', common_element('div', 'instructions',
_t('Login with an OpenID account.')); _t('Login with an OpenID account.'));
} }
$formaction = common_local_url('openidlogin');
common_element_start('form', array('method' => 'POST', common_element_start('form', array('method' => 'POST',
'id' => 'openidlogin', 'id' => 'openidlogin',
'action' => common_local_url('openidlogin'))); 'action' => $formaction));
common_input('openid_url', _t('OpenID URL')); common_input('openid_url', _t('OpenID URL'));
common_submit('submit', _t('Login')); common_submit('submit', _t('Login'));
common_element_end('form'); common_element_end('form');

View File

@ -62,79 +62,78 @@ function oid_link_user($id, $canonical, $display) {
function oid_authenticate($openid_url, $returnto) { function oid_authenticate($openid_url, $returnto) {
$consumer = oid_consumer(); $consumer = oid_consumer();
if (!$consumer) { if (!$consumer) {
common_server_error(_t('Cannot instantiate OpenID consumer object.')); common_server_error(_t('Cannot instantiate OpenID consumer object.'));
return false; return false;
} }
common_ensure_session(); common_ensure_session();
$auth_request = $consumer->begin($openid_url); $auth_request = $consumer->begin($openid_url);
// Handle failure status return values. // Handle failure status return values.
if (!$auth_request) { if (!$auth_request) {
return _t('Not a valid OpenID.'); return _t('Not a valid OpenID.');
} else if (Auth_OpenID::isFailure($auth_request)) { } else if (Auth_OpenID::isFailure($auth_request)) {
return _t('OpenID failure: ') . $auth_request->message; return _t('OpenID failure: ') . $auth_request->message;
} }
$sreg_request = Auth_OpenID_SRegRequest::build(// Required $sreg_request = Auth_OpenID_SRegRequest::build(// Required
array(), array(),
// Optional // Optional
array('nickname', array('nickname',
'email', 'email',
'fullname', 'fullname',
'language', 'language',
'timezone', 'timezone',
'postcode', 'postcode',
'country')); 'country'));
if ($sreg_request) { if ($sreg_request) {
$auth_request->addExtension($sreg_request); $auth_request->addExtension($sreg_request);
} }
$trust_root = common_root_url(); $trust_root = common_root_url();
$process_url = common_local_url($returnto); $process_url = common_local_url($returnto);
if ($auth_request->shouldSendRedirect()) { if ($auth_request->shouldSendRedirect()) {
$redirect_url = $auth_request->redirectURL($trust_root, $redirect_url = $auth_request->redirectURL($trust_root,
$process_url); $process_url);
if (!$redirect_url) { if (!$redirect_url) {
} else if (Auth_OpenID::isFailure($redirect_url)) { } else if (Auth_OpenID::isFailure($redirect_url)) {
return _t('Could not redirect to server: ') . $redirect_url->message; return _t('Could not redirect to server: ') . $redirect_url->message;
} else {
common_redirect($redirect_url);
}
} else { } else {
// Generate form markup and render it. common_redirect($redirect_url);
$form_id = 'openid_message'; }
$form_html = $auth_request->formMarkup($trust_root, $process_url, } else {
false, array('id' => $form_id)); // Generate form markup and render it.
$form_id = 'openid_message';
$form_html = $auth_request->formMarkup($trust_root, $process_url,
false, array('id' => $form_id));
# XXX: This is cheap, but things choke if we don't escape ampersands # XXX: This is cheap, but things choke if we don't escape ampersands
# in the HTML attributes # in the HTML attributes
$form_html = preg_replace('/&/', '&', $form_html); $form_html = preg_replace('/&/', '&', $form_html);
// Display an error if the form markup couldn't be generated; // Display an error if the form markup couldn't be generated;
// otherwise, render the HTML. // otherwise, render the HTML.
if (Auth_OpenID::isFailure($form_html)) { if (Auth_OpenID::isFailure($form_html)) {
$this->show_form(_t('Could not create OpenID form: ') . $form_html->message); $this->show_form(_t('Could not create OpenID form: ') . $form_html->message);
} else { } else {
common_show_header(_t('OpenID Auto-Submit')); common_show_header(_t('OpenID Auto-Submit'));
common_element('p', 'instructions', common_element('p', 'instructions',
_t('This form should automatically submit itself. '. _t('This form should automatically submit itself. '.
'If not, click the submit button to go to your '. 'If not, click the submit button to go to your '.
'OpenID provider.')); 'OpenID provider.'));
common_raw($form_html); common_raw($form_html);
common_element('script', NULL, common_element('script', NULL,
'$(document).ready(function() { ' . '$(document).ready(function() { ' .
' $("#'. $form_id .'").submit(); '. ' $("#'. $form_id .'").submit(); '.
'});'); '});');
common_show_footer(); common_show_footer();
}
} }
} }
} }