From 6cff19358aba894d57eab5faa55882ba424bedf7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20B=C3=B6hmer?= Date: Sat, 25 Nov 2023 13:21:39 +0100 Subject: [PATCH] Dont show the permission reset warning during database migration anymore This only affects database which were older than the new permission system and hopefully everybody should have updated by now. The message could irritate new users, so it was removed --- migrations/Version20221114193325.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/migrations/Version20221114193325.php b/migrations/Version20221114193325.php index d80cc1d2..9075dc7a 100644 --- a/migrations/Version20221114193325.php +++ b/migrations/Version20221114193325.php @@ -83,9 +83,12 @@ final class Version20221114193325 extends AbstractMultiPlatformMigration impleme //Reset the permissions of the admin user, to allow admin permissions (like the admins group) $this->addSql("UPDATE `users` SET permissions_data = '$admin' WHERE id = 2;"); + //This warning should not be needed, anymore, as almost everybody should have updated to the new version by now, and this warning would just irritate new users of the software + /* $this->logger->warning('!!! All permissions were reset! Please change them to the desired state, immediately !!!'); $this->logger->warning('!!! For security reasons all users (except the admin user) were disabled. Login with admin user and reenable other users after checking their permissions !!!'); $this->logger->warning('!!! For more infos see: https://github.com/Part-DB/Part-DB-symfony/discussions/193 !!!'); + */ } public function mySQLUp(Schema $schema): void