From 7dc814ffa9269b907bdf0f864bec81b29f697663 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Thu, 11 Feb 2021 20:29:00 +0100 Subject: Add a repair step that updates the display name for existing users Signed-off-by: Joas Schilling --- appinfo/info.xml | 3 +- lib/Migration/CacheUserDisplayNames.php | 78 +++++++++++++++++++++++++++++++++ 2 files changed, 80 insertions(+), 1 deletion(-) create mode 100644 lib/Migration/CacheUserDisplayNames.php diff --git a/appinfo/info.xml b/appinfo/info.xml index 9bb262650..2f5788bf2 100644 --- a/appinfo/info.xml +++ b/appinfo/info.xml @@ -16,7 +16,7 @@ And in the works for the [coming versions](https://github.com/nextcloud/spreed/m ]]> - 12.0.0-dev.3 + 12.0.0-dev.4 agpl Daniel Calviño Sánchez @@ -68,6 +68,7 @@ And in the works for the [coming versions](https://github.com/nextcloud/spreed/m OCA\Talk\Migration\CreateHelpCommand OCA\Talk\Migration\ClearResourceAccessCache + OCA\Talk\Migration\CacheUserDisplayNames OCA\Talk\Migration\CreateHelpCommand diff --git a/lib/Migration/CacheUserDisplayNames.php b/lib/Migration/CacheUserDisplayNames.php new file mode 100644 index 000000000..bae450b35 --- /dev/null +++ b/lib/Migration/CacheUserDisplayNames.php @@ -0,0 +1,78 @@ + + * + * @license GNU AGPL version 3 or any later version + * + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +namespace OCA\Talk\Migration; + +use OCA\Talk\Model\Attendee; +use OCP\IDBConnection; +use OCP\IUser; +use OCP\IUserManager; +use OCP\Migration\IOutput; +use OCP\Migration\IRepairStep; + +class CacheUserDisplayNames implements IRepairStep { + + /** @var IDBConnection */ + protected $connection; + /** @var IUserManager */ + protected $userManager; + + public function __construct(IDBConnection $connection, + IUserManager $userManager) { + $this->connection = $connection; + $this->userManager = $userManager; + } + + public function getName(): string { + return 'Cache the user display names'; + } + + public function run(IOutput $output): void { + $update = $this->connection->getQueryBuilder(); + $update->update('talk_attendees') + ->set('display_name', $update->createParameter('displayName')) + ->where($update->expr()->eq('actor_type', $update->createParameter('actorType'))) + ->andWhere($update->expr()->eq('actor_id', $update->createParameter('actorId'))); + + $query = $this->connection->getQueryBuilder(); + $query->select('actor_id') + ->from('talk_attendees') + ->where($query->expr()->eq('actor_type', $query->createNamedParameter(Attendee::ACTOR_USERS))) + ->andWhere($query->expr()->eq('display_name', $query->createNamedParameter(''))) + ->groupBy('actor_id'); + + $result = $query->execute(); + while ($row = $result->fetch()) { + $user = $this->userManager->get($row['actor_id']); + if (!$user instanceof IUser) { + continue; + } + + $update->setParameter('displayName', $user->getDisplayName()) + ->setParameter('actorType', Attendee::ACTOR_USERS) + ->setParameter('actorId', $row['actor_id']); + $update->execute(); + } + $result->closeCursor(); + } +} -- cgit v1.2.3