skobkin/point-tools
1
1
Fork 0
Code Issues 4 Pull requests 1 Releases Activity

User without login flushing fix.

Browse source
This commit is contained in:
Alexey Skobkin 2015-10-01 22:14:30 +03:00
parent 0685e6d8f9
commit dfc7fb70e4
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/Skobkin/Bundle/PointToolsBundle/Service/UserApi.php
View file

@ -135,13 +135,16 @@ class UserApi extends AbstractApi
if (!$user) {
$user = new User();
$user->setId((int) $userData['id']);
$user
->setId((int) $userData['id'])
->setLogin($userData['login'])
;
$this->em->persist($user);
try {
$this->em->flush();
} catch (\Exception $e) {
throw new ApiException(sprintf('Error while flushing new user [%d] %s', $user->getId(), $user->getLogin()), 0, $e);
throw new ApiException(sprintf('Error while flushing new user [%d] %s: %s', $user->getId(), $user->getLogin(), $e->getMessage()), 0, $e);
}
}
@ -156,7 +159,7 @@ class UserApi extends AbstractApi
try {
$this->em->flush();
} catch (\Exception $e) {
throw new ApiException(sprintf('Error while flushing changes for [%d] %s', $user->getId(), $user->getLogin()), 0, $e);
throw new ApiException(sprintf('Error while flushing changes for [%d] %s: %s', $user->getId(), $user->getLogin(), $e->getMessage()), 0, $e);
}
$resultUsers[] = $user;