use correct variable and show error messages

This commit is contained in:
Christian Weiske 2011-05-11 07:55:01 +02:00
parent 1c415919cb
commit 708dc19122

View file

@ -103,7 +103,7 @@ if (POST_SUBMITTED!='' && $currentUser->getId() == $userid) {
} }
if (!$error) { if (!$error) {
if (!$userservice->updateUser($userid, $detPass, $detName, $detMail, $detPage, $detDesc)) { if (!$userservice->updateUser($userid, $detPass, $detName, $detMail, $detPage, $detDesc)) {
$tplvars['error'] = T_('An error occurred while saving your changes.'); $tplVars['error'] = T_('An error occurred while saving your changes.');
} else { } else {
$tplVars['msg'] = T_('Changes saved.'); $tplVars['msg'] = T_('Changes saved.');
} }
@ -118,11 +118,11 @@ if (!$userservice->isLoggedOn() || $currentUser->getId() != $userid) {
if (isset($_POST['action']) && $_POST['action'] == 'registerCurrentCert') { if (isset($_POST['action']) && $_POST['action'] == 'registerCurrentCert') {
if (!$scert->hasValidCert()) { if (!$scert->hasValidCert()) {
$tplvars['error'] = T_('You do not have a valid SSL client certificate'); $tplVars['error'] = T_('You do not have a valid SSL client certificate');
} else if (false !== $scert->getUserIdFromCert()) { } else if (false !== $scert->getUserIdFromCert()) {
$tplvars['error'] = T_('This certificate is already registered'); $tplVars['error'] = T_('This certificate is already registered');
} else if (false === $scert->registerCurrentCertificate($currentUser->getId())) { } else if (false === $scert->registerCurrentCertificate($currentUser->getId())) {
$tplvars['error'] = T_('Failed to register SSL client certificate.'); $tplVars['error'] = T_('Failed to register SSL client certificate.');
} else { } else {
$tplVars['msg'] = T_('SSL client certificate registered.'); $tplVars['msg'] = T_('SSL client certificate registered.');
} }
@ -131,18 +131,18 @@ if (!$userservice->isLoggedOn() || $currentUser->getId() != $userid) {
) { ) {
$certId = (int)$_POST['certId']; $certId = (int)$_POST['certId'];
$cert = $scert->getCert($certId); $cert = $scert->getCert($certId);
if ($cert === null) { if ($cert === null) {
$tplvars['error'] = T_('Certificate not found.'); $tplVars['error'] = T_('Certificate not found.');
} else if ($cert->uId != $currentUser->getId()) { } else if ($cert->uId != $currentUser->getId()) {
$tplvars['error'] = T_('The certificate does not belong to you.'); $tplVars['error'] = T_('The certificate does not belong to you.');
} else if (false === $scert->delete($certId)) { } else if (false === $scert->delete($certId)) {
$tplvars['error'] = T_('Failed to delete SSL client certificate.'); $tplVars['error'] = T_('Failed to delete SSL client certificate.');
} else { } else {
$tplVars['msg'] = T_('SSL client certificate deleted.'); $tplVars['msg'] = T_('SSL client certificate deleted.');
} }
} }
//Token Init //Token Init
$_SESSION['token'] = md5(uniqid(rand(), true)); $_SESSION['token'] = md5(uniqid(rand(), true));
$_SESSION['token_stamp'] = time(); $_SESSION['token_stamp'] = time();