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

Update commands now can fallback to API for point-tools user retrieval if it's not found in the database. Also small bug fixed.

Browse source
This commit is contained in:
Alexey Skobkin 2019-01-19 02:07:00 +03:00
parent 2618398d06
commit 6c8c0829ca
2 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/Skobkin/Bundle/PointToolsBundle/Command/UpdateSubscriptionsCommand.php
View file

@ -192,16 +192,19 @@ class UpdateSubscriptionsCommand extends Command
try { try {
$serviceUser = $this->userRepo->findActiveUserWithSubscribers($this->appUserId); $serviceUser = $this->userRepo->findActiveUserWithSubscribers($this->appUserId);
} catch (\Exception $e) { } catch (\Exception $e) {
$this->logger->error('Error while getting active user with subscribers', ['app_user_id' => $appUserId]); $this->logger->error('Error while getting active user with subscribers', ['app_user_id' => $this->appUserId]);
throw $e; throw $e;
} }
if (!$serviceUser) { if (!$serviceUser) {
$this->logger->critical('Service user not found or marked as removed'); $this->logger->warning('Service user not found or marked as removed. Falling back to API.');
// @todo Retrieving user
throw new \RuntimeException('Service user not found in the database'); try {
$serviceUser = $this->api->getUserById($this->appUserId);
} catch (UserNotFoundException $e) {
throw new \RuntimeException('Service user not found in the database and could not be retrieved from API.');
}
} }
$this->logger->info('Getting service subscribers'); $this->logger->info('Getting service subscribers');

8
src/Skobkin/Bundle/PointToolsBundle/Command/UpdateUsersPrivacyCommand.php
View file

@ -164,9 +164,13 @@ class UpdateUsersPrivacyCommand extends Command
} }
if (!$serviceUser) { if (!$serviceUser) {
$this->logger->critical('Service user not found or marked as removed'); $this->logger->warning('Service user not found or marked as removed. Falling back to API.');
throw new \RuntimeException('Service user not found in the database'); try {
$serviceUser = $this->api->getUserById($this->appUserId);
} catch (UserNotFoundException $e) {
throw new \RuntimeException('Service user not found in the database and could not be retrieved from API.');
}
} }
$this->logger->info('Getting service subscribers'); $this->logger->info('Getting service subscribers');