diff options
Diffstat (limited to 'lib/private/Accounts/AccountManager.php')
| -rw-r--r-- | lib/private/Accounts/AccountManager.php | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/lib/private/Accounts/AccountManager.php b/lib/private/Accounts/AccountManager.php index f1607b1a1ef..225d076ca37 100644 --- a/lib/private/Accounts/AccountManager.php +++ b/lib/private/Accounts/AccountManager.php @@ -1,4 +1,5 @@ <?php + /** * @copyright Copyright (c) 2016, ownCloud, Inc. * @copyright Copyright (c) 2016, Björn Schießle @@ -33,9 +34,12 @@ use OCP\Accounts\IAccount; use OCP\Accounts\IAccountManager; use OCP\BackgroundJob\IJobList; use OCP\IDBConnection; +use OCP\ILogger; use OCP\IUser; use Symfony\Component\EventDispatcher\EventDispatcherInterface; use Symfony\Component\EventDispatcher\GenericEvent; +use function json_decode; +use function json_last_error; /** * Class AccountManager @@ -59,6 +63,9 @@ class AccountManager implements IAccountManager { /** @var IJobList */ private $jobList; + /** @var ILogger */ + private $logger; + /** * AccountManager constructor. * @@ -68,10 +75,12 @@ class AccountManager implements IAccountManager { */ public function __construct(IDBConnection $connection, EventDispatcherInterface $eventDispatcher, - IJobList $jobList) { + IJobList $jobList, + ILogger $logger) { $this->connection = $connection; $this->eventDispatcher = $eventDispatcher; $this->jobList = $jobList; + $this->logger = $logger; } /** @@ -137,6 +146,11 @@ class AccountManager implements IAccountManager { } $userDataArray = json_decode($result[0]['data'], true); + $jsonError = json_last_error(); + if ($userDataArray === null || $jsonError !== JSON_ERROR_NONE) { + $this->logger->critical("User data of $uid contained invalid JSON (error $jsonError), hence falling back to a default user record"); + return $this->buildDefaultUserRecord($user); + } $userDataArray = $this->addMissingDefaultValues($userDataArray); |