diff options
author | Marco Nassabain <marco.nassabain@hotmail.com> | 2021-03-19 18:13:13 +0100 |
---|---|---|
committer | Sean Molenaar <SMillerDev@users.noreply.github.com> | 2021-04-08 22:31:21 +0200 |
commit | bbdd95c30d545cc7dd9719836a94c7b21b381c68 (patch) | |
tree | 5ce0a0d8dacf98e9e13f9582cfc6299eeee56741 /lib | |
parent | 7f21574615f0ed9d5e2fa218f3ed018600b58a42 (diff) |
🔥 Remove UserManager code from ItemMapper & tests
Signed-off-by: Marco Nassabain <marco.nassabain@hotmail.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Db/ItemMapperV2.php | 38 |
1 files changed, 1 insertions, 37 deletions
diff --git a/lib/Db/ItemMapperV2.php b/lib/Db/ItemMapperV2.php index 3e0aa80b5..44d1740cd 100644 --- a/lib/Db/ItemMapperV2.php +++ b/lib/Db/ItemMapperV2.php @@ -22,7 +22,6 @@ use OCP\AppFramework\Db\MultipleObjectsReturnedException; use OCP\DB\Exception as DBException; use OCP\DB\QueryBuilder\IQueryBuilder; use OCP\IDBConnection; -use OCP\IUserManager; /** * Class ItemMapper @@ -34,49 +33,14 @@ class ItemMapperV2 extends NewsMapperV2 const TABLE_NAME = 'news_items'; /** - * @var IUserManager - */ - private $userManager; - - /** * ItemMapper constructor. * * @param IDBConnection $db * @param Time $time */ - public function __construct(IDBConnection $db, Time $time, IUserManager $userManager) + public function __construct(IDBConnection $db, Time $time) { parent::__construct($db, $time, Item::class); - $this->userManager = $userManager; - } - - /** - * Override parent constructor to insert sharer display names for shared items - * - * @param IQueryBuilder $query - * @return Entity[] all fetched entities - */ - public function findEntities(IQueryBuilder $query): array - { - $entities = parent::findEntities($query); - - foreach ($entities as $entity) { - $sharedBy = $entity->getSharedBy(); - $sharedByDisplayName = null; - - // Get user display name - if (!is_null($sharedBy)) { - $user = $this->userManager->get($sharedBy); - if (!is_null($user)) { - $sharedByDisplayName = $user->getDisplayName(); - } - } - - // Set sharer display name - $entity->setSharedByDisplayName($sharedByDisplayName); - } - - return $entities; } /** |