diff options
author | Joas Schilling <coding@schilljs.com> | 2023-05-02 08:58:48 +0200 |
---|---|---|
committer | Joas Schilling <coding@schilljs.com> | 2023-05-02 16:56:20 +0200 |
commit | 10b4e6ba237db4d18ab4718fb56dab0ae3327acd (patch) | |
tree | 7098fbab2b19062f9c1358f9dc61676bb5c84497 /lib | |
parent | 76eaaee883331b57d37c9cd4a09696c94054a8e0 (diff) |
fix(permissions): Reset custom permissions on promotion
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Migration/Version16000Date20230502145340.php | 59 | ||||
-rw-r--r-- | lib/Service/ParticipantService.php | 8 |
2 files changed, 66 insertions, 1 deletions
diff --git a/lib/Migration/Version16000Date20230502145340.php b/lib/Migration/Version16000Date20230502145340.php new file mode 100644 index 000000000..94ce08752 --- /dev/null +++ b/lib/Migration/Version16000Date20230502145340.php @@ -0,0 +1,59 @@ +<?php + +declare(strict_types=1); + +/** + * @copyright Copyright (c) 2023 Joas Schilling <coding@schilljs.com> + * + * @author Joas Schilling <coding@schilljs.com> + * + * @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 <http://www.gnu.org/licenses/>. + */ + +namespace OCA\Talk\Migration; + +use Closure; +use OCA\Talk\Model\Attendee; +use OCA\Talk\Participant; +use OCP\DB\ISchemaWrapper; +use OCP\DB\QueryBuilder\IQueryBuilder; +use OCP\IDBConnection; +use OCP\Migration\IOutput; +use OCP\Migration\SimpleMigrationStep; + +class Version16000Date20230502145340 extends SimpleMigrationStep { + public function __construct( + protected IDBConnection $connection, + ) { + } + + /** + * @param IOutput $output + * @param Closure(): ISchemaWrapper $schemaClosure + * @param array $options + */ + public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void { + $query = $this->connection->getQueryBuilder(); + + $query->update('talk_attendees') + ->set('permissions', $query->createNamedParameter(Attendee::PERMISSIONS_DEFAULT, IQueryBuilder::PARAM_INT)) + ->where($query->expr()->eq('participant_type', $query->createNamedParameter(Participant::MODERATOR))) + ->orWhere($query->expr()->eq('participant_type', $query->createNamedParameter(Participant::GUEST_MODERATOR))); + $fixed = $query->executeStatement(); + + $output->info('Fixed permissions of ' . $fixed . ' moderators'); + } +} diff --git a/lib/Service/ParticipantService.php b/lib/Service/ParticipantService.php index 18bf8c698..afb33ac07 100644 --- a/lib/Service/ParticipantService.php +++ b/lib/Service/ParticipantService.php @@ -144,7 +144,6 @@ class ParticipantService { $this->dispatcher->dispatch(Room::EVENT_BEFORE_PARTICIPANT_TYPE_SET, $event); $attendee->setParticipantType($participantType); - $this->attendeeMapper->update($attendee); $promotedToModerator = in_array($participantType, [ Participant::OWNER, @@ -155,6 +154,13 @@ class ParticipantService { Participant::MODERATOR, ], true); + if ($promotedToModerator) { + // Reset permissions on promotion + $attendee->setPermissions(Attendee::PERMISSIONS_DEFAULT); + } + + $this->attendeeMapper->update($attendee); + // XOR so we don't move the participant in and out when they are changed from moderator to owner or vice-versa if (($promotedToModerator xor $demotedFromModerator) && $room->getBreakoutRoomMode() !== BreakoutRoom::MODE_NOT_CONFIGURED) { /** @var Manager $manager */ |