Some exception handling. More output in verbose mode.
This commit is contained in:
parent
5652a5ca68
commit
20c44371c9
|
@ -39,16 +39,45 @@ class UpdateSubscriptionsCommand extends ContainerAwareCommand
|
|||
|
||||
if (!$serviceUser) {
|
||||
// @todo Retrieving user
|
||||
return false;
|
||||
}
|
||||
|
||||
if ($output->isVerbose()) {
|
||||
$output->writeln('Getting service subscribers');
|
||||
}
|
||||
|
||||
try {
|
||||
$serviceSubscribers = $api->getUserSubscribersByLogin($serviceUserName);
|
||||
} catch (\Exception $e) {
|
||||
$output->writeln('Error while getting service subscribers');
|
||||
return false;
|
||||
}
|
||||
|
||||
if ($output->isVerbose()) {
|
||||
$output->writeln('Updating service subscribers');
|
||||
}
|
||||
|
||||
// Updating service subscribers
|
||||
$subscriptionsManager->updateUserSubscribers($serviceUser, $serviceSubscribers);
|
||||
|
||||
if ($output->isVerbose()) {
|
||||
$output->writeln('Processing service subscribers');
|
||||
}
|
||||
|
||||
// Updating service users subscribers
|
||||
foreach ($serviceSubscribers as $user) {
|
||||
$output->writeln(' Processing @' . $user->getLogin());
|
||||
|
||||
try {
|
||||
$userCurrentSubscribers = $api->getUserSubscribersByLogin($user->getLogin());
|
||||
} catch (\Exception $e) {
|
||||
$output->writeln(' Error while getting subscribers. Skipping.');
|
||||
continue;
|
||||
}
|
||||
|
||||
if ($output->isVerbose()) {
|
||||
$output->writeln(' Updating service subscribers');
|
||||
}
|
||||
|
||||
$subscriptionsManager->updateUserSubscribers($user, $userCurrentSubscribers);
|
||||
|
||||
|
|
Loading…
Reference in a new issue