Created
September 4, 2014 14:59
-
-
Save isuvorov/67690ce0631559383285 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
public function registration() | |
{ | |
$users = new User(); | |
$users->username = $this->service->getAttribute('name'); | |
$users->password = md5($this->service->getId()); | |
$users->email = $this->service->getServiceName() . '@services.ru'; | |
$users->save(); | |
$userlink = new Userlink(); | |
$userlink->userId = $users->id; | |
$userlink->serviceUsername = $this->service->id; | |
$userlink->serviceName = $this->service->getServiceName(); | |
$userlink->params = ['attributes' => $this->service->attributes]; | |
$userlink->save(); | |
$this->_id = $users->id; | |
$this->username = $this->service->getAttribute('name'); | |
return 0; | |
} |
$users->username = $this->service->getAttribute('name');
Об уникальности никто и не говорил. Для этого необходимо пересматривать саму структуру регистрации через соц сети. Чтобы люди вводили сразу себе username и email.
Или же делать эти поля не обязательными для заполнения. Тогда данные будут вытаскиваться из соц.сети, через которую зашёл человек.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
$users = new User();
множественное число - неверно, user тут один
$users->username = $this->service->getAttribute('name');
нет вообще никакой гарантии уникальности, может быть один и тотже юзернейм в разных соцсетях у разных людей
$users->email = $this->service->getServiceName() . '@services.ru';
бессмысленно, уж лучше $users->email = null; сделать
return 0;
тоже не хорошо, уж лучше или ничего не возвращать
или вернуть чтото полезное например созданного юзера
return $user;