store verification status in accounts table and update lookup server
Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
This commit is contained in:
parent
c9ccdca1ec
commit
6f41a3edc0
|
@ -53,6 +53,10 @@ class AccountManager {
|
|||
const PROPERTY_ADDRESS = 'address';
|
||||
const PROPERTY_TWITTER = 'twitter';
|
||||
|
||||
const NOT_VERIFIED = '0';
|
||||
const VERIFICATION_IN_PROGRESS = '1';
|
||||
const VERIFIED = '2';
|
||||
|
||||
/** @var IDBConnection database connection */
|
||||
private $connection;
|
||||
|
||||
|
@ -85,6 +89,7 @@ class AccountManager {
|
|||
if (empty($userData)) {
|
||||
$this->insertNewUser($user, $data);
|
||||
} elseif ($userData !== $data) {
|
||||
$data = $this->updateVerifyStatus($userData, $data);
|
||||
$this->updateExistingUser($user, $data);
|
||||
} else {
|
||||
// nothing needs to be done if new and old data set are the same
|
||||
|
@ -123,6 +128,41 @@ class AccountManager {
|
|||
return json_decode($result[0]['data'], true);
|
||||
}
|
||||
|
||||
/**
|
||||
* reset verification status if personal data changed
|
||||
*
|
||||
* @param array $oldData
|
||||
* @param array $newData
|
||||
* @return array
|
||||
*/
|
||||
protected function updateVerifyStatus($oldData, $newData) {
|
||||
|
||||
$twitterVerified = isset($oldData[self::PROPERTY_TWITTER]['verified']) && $oldData[self::PROPERTY_TWITTER]['verified'] === self::VERIFIED;
|
||||
$websiteVerified = isset($oldData[self::PROPERTY_WEBSITE]['verified']) && $oldData[self::PROPERTY_WEBSITE]['verified'] === self::VERIFIED;
|
||||
$emailVerified = isset($oldData[self::PROPERTY_EMAIL]['verified']) && $oldData[self::PROPERTY_EMAIL]['verified'] === self::VERIFIED;
|
||||
|
||||
if($twitterVerified &&
|
||||
$oldData[self::PROPERTY_TWITTER]['value'] !== $newData[self::PROPERTY_TWITTER]['value']
|
||||
) {
|
||||
$newData[self::PROPERTY_TWITTER]['value']['verified'] = self::NOT_VERIFIED;
|
||||
}
|
||||
|
||||
if($websiteVerified &&
|
||||
$oldData[self::PROPERTY_WEBSITE]['value'] !== $newData[self::PROPERTY_WEBSITE]['value']
|
||||
) {
|
||||
$newData[self::PROPERTY_WEBSITE]['value']['verified'] = self::NOT_VERIFIED;
|
||||
}
|
||||
|
||||
if($emailVerified &&
|
||||
$oldData[self::PROPERTY_EMAIL]['value'] !== $newData[self::PROPERTY_EMAIL]['value']
|
||||
) {
|
||||
$newData[self::PROPERTY_EMAIL]['value']['verified'] = self::NOT_VERIFIED;
|
||||
}
|
||||
|
||||
return $newData;
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
* add new user to accounts table
|
||||
*
|
||||
|
@ -171,21 +211,25 @@ class AccountManager {
|
|||
[
|
||||
'value' => $user->getDisplayName(),
|
||||
'scope' => self::VISIBILITY_CONTACTS_ONLY,
|
||||
'verified' => '0',
|
||||
],
|
||||
self::PROPERTY_ADDRESS =>
|
||||
[
|
||||
'value' => '',
|
||||
'scope' => self::VISIBILITY_PRIVATE,
|
||||
'verified' => '0',
|
||||
],
|
||||
self::PROPERTY_WEBSITE =>
|
||||
[
|
||||
'value' => '',
|
||||
'scope' => self::VISIBILITY_PRIVATE,
|
||||
'verified' => '0',
|
||||
],
|
||||
self::PROPERTY_EMAIL =>
|
||||
[
|
||||
'value' => $user->getEMailAddress(),
|
||||
'scope' => self::VISIBILITY_CONTACTS_ONLY,
|
||||
'verified' => '0',
|
||||
],
|
||||
self::PROPERTY_AVATAR =>
|
||||
[
|
||||
|
@ -195,11 +239,13 @@ class AccountManager {
|
|||
[
|
||||
'value' => '',
|
||||
'scope' => self::VISIBILITY_PRIVATE,
|
||||
'verified' => '0',
|
||||
],
|
||||
self::PROPERTY_TWITTER =>
|
||||
[
|
||||
'value' => '',
|
||||
'scope' => self::VISIBILITY_PRIVATE,
|
||||
'verified' => '0',
|
||||
],
|
||||
];
|
||||
}
|
||||
|
|
|
@ -516,6 +516,12 @@ class UsersController extends Controller {
|
|||
public function getVerificationCode($account) {
|
||||
|
||||
$user = $this->userSession->getUser();
|
||||
|
||||
if ($user === null) {
|
||||
return new DataResponse([], Http::STATUS_BAD_REQUEST);
|
||||
}
|
||||
|
||||
$accountData = $this->accountManager->getUser($user);
|
||||
$cloudId = $user->getCloudId();
|
||||
$message = "Use my Federated Cloud ID to share with me: " . $cloudId;
|
||||
$privateKey = $this->keyManager->getKey($user)->getPrivate();
|
||||
|
@ -527,17 +533,20 @@ class UsersController extends Controller {
|
|||
|
||||
switch ($account) {
|
||||
case 'verify-twitter':
|
||||
$accountData[AccountManager::PROPERTY_TWITTER]['verified'] = AccountManager::VERIFICATION_IN_PROGRESS;
|
||||
$msg = $this->l10n->t('In order to verify your Twitter account post following tweet on Twitter:');
|
||||
$code = $codeMd5;
|
||||
break;
|
||||
case 'verify-website':
|
||||
$accountData[AccountManager::PROPERTY_WEBSITE]['verified'] = AccountManager::VERIFICATION_IN_PROGRESS;
|
||||
$msg = $this->l10n->t('In order to verify your Website store following content in your webroot at \'CloudIdVerificationCode.txt\':');
|
||||
break;
|
||||
default:
|
||||
return new DataResponse([], Http::STATUS_BAD_REQUEST);
|
||||
break;
|
||||
}
|
||||
|
||||
$this->accountManager->updateUser($user, $accountData);
|
||||
|
||||
return new DataResponse(['msg' => $msg, 'code' => $code]);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue