Completed
Push — master ( 247b25...4111bd )
by Thomas
26:36 queued 10s
created
lib/public/ICertificateManager.php 1 patch
Indentation   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -32,43 +32,43 @@
 block discarded – undo
32 32
  * @since 8.0.0
33 33
  */
34 34
 interface ICertificateManager {
35
-	/**
36
-	 * Returns all certificates trusted by the system
37
-	 *
38
-	 * @return \OCP\ICertificate[]
39
-	 * @since 8.0.0
40
-	 */
41
-	public function listCertificates(): array;
35
+    /**
36
+     * Returns all certificates trusted by the system
37
+     *
38
+     * @return \OCP\ICertificate[]
39
+     * @since 8.0.0
40
+     */
41
+    public function listCertificates(): array;
42 42
 
43
-	/**
44
-	 * @param string $certificate the certificate data
45
-	 * @param string $name the filename for the certificate
46
-	 * @return \OCP\ICertificate
47
-	 * @throws \Exception If the certificate could not get added
48
-	 * @since 8.0.0 - since 8.1.0 throws exception instead of returning false
49
-	 */
50
-	public function addCertificate(string $certificate, string $name): \OCP\ICertificate;
43
+    /**
44
+     * @param string $certificate the certificate data
45
+     * @param string $name the filename for the certificate
46
+     * @return \OCP\ICertificate
47
+     * @throws \Exception If the certificate could not get added
48
+     * @since 8.0.0 - since 8.1.0 throws exception instead of returning false
49
+     */
50
+    public function addCertificate(string $certificate, string $name): \OCP\ICertificate;
51 51
 
52
-	/**
53
-	 * @param string $name
54
-	 * @return bool
55
-	 * @since 8.0.0
56
-	 */
57
-	public function removeCertificate(string $name): bool;
52
+    /**
53
+     * @param string $name
54
+     * @return bool
55
+     * @since 8.0.0
56
+     */
57
+    public function removeCertificate(string $name): bool;
58 58
 
59
-	/**
60
-	 * Get the path to the certificate bundle
61
-	 *
62
-	 * @return string
63
-	 * @since 8.0.0
64
-	 */
65
-	public function getCertificateBundle(): string;
59
+    /**
60
+     * Get the path to the certificate bundle
61
+     *
62
+     * @return string
63
+     * @since 8.0.0
64
+     */
65
+    public function getCertificateBundle(): string;
66 66
 
67
-	/**
68
-	 * Get the full local path to the certificate bundle
69
-	 *
70
-	 * @return string
71
-	 * @since 9.0.0
72
-	 */
73
-	public function getAbsoluteBundlePath(): string;
67
+    /**
68
+     * Get the full local path to the certificate bundle
69
+     *
70
+     * @return string
71
+     * @since 9.0.0
72
+     */
73
+    public function getAbsoluteBundlePath(): string;
74 74
 }
Please login to merge, or discard this patch.
core/Command/User/Report.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
 				'count-dirs',
62 62
 				null,
63 63
 				InputOption::VALUE_NONE,
64
-				'Also count the number of user directories in the database (could time out on huge installations, therefore defaults to no with ' . self::DEFAULT_COUNT_DIRS_MAX_USERS . '+ users)'
64
+				'Also count the number of user directories in the database (could time out on huge installations, therefore defaults to no with '.self::DEFAULT_COUNT_DIRS_MAX_USERS.'+ users)'
65 65
 			)
66 66
 		;
67 67
 	}
Please login to merge, or discard this patch.
Indentation   +57 added lines, -57 removed lines patch added patch discarded remove patch
@@ -19,71 +19,71 @@
 block discarded – undo
19 19
 use Symfony\Component\Console\Output\OutputInterface;
20 20
 
21 21
 class Report extends Command {
22
-	public const DEFAULT_COUNT_DIRS_MAX_USERS = 500;
22
+    public const DEFAULT_COUNT_DIRS_MAX_USERS = 500;
23 23
 
24
-	public function __construct(
25
-		protected IUserManager $userManager,
26
-		private IConfig $config,
27
-	) {
28
-		parent::__construct();
29
-	}
24
+    public function __construct(
25
+        protected IUserManager $userManager,
26
+        private IConfig $config,
27
+    ) {
28
+        parent::__construct();
29
+    }
30 30
 
31
-	protected function configure(): void {
32
-		$this
33
-			->setName('user:report')
34
-			->setDescription('shows how many users have access')
35
-			->addOption(
36
-				'count-dirs',
37
-				null,
38
-				InputOption::VALUE_NONE,
39
-				'Also count the number of user directories in the database (could time out on huge installations, therefore defaults to no with ' . self::DEFAULT_COUNT_DIRS_MAX_USERS . '+ users)'
40
-			)
41
-		;
42
-	}
31
+    protected function configure(): void {
32
+        $this
33
+            ->setName('user:report')
34
+            ->setDescription('shows how many users have access')
35
+            ->addOption(
36
+                'count-dirs',
37
+                null,
38
+                InputOption::VALUE_NONE,
39
+                'Also count the number of user directories in the database (could time out on huge installations, therefore defaults to no with ' . self::DEFAULT_COUNT_DIRS_MAX_USERS . '+ users)'
40
+            )
41
+        ;
42
+    }
43 43
 
44
-	protected function execute(InputInterface $input, OutputInterface $output): int {
45
-		$table = new Table($output);
46
-		$table->setHeaders(['Account Report', '']);
47
-		$userCountArray = $this->countUsers();
48
-		$total = 0;
49
-		if (!empty($userCountArray)) {
50
-			$rows = [];
51
-			foreach ($userCountArray as $classname => $users) {
52
-				$total += $users;
53
-				$rows[] = [$classname, $users];
54
-			}
44
+    protected function execute(InputInterface $input, OutputInterface $output): int {
45
+        $table = new Table($output);
46
+        $table->setHeaders(['Account Report', '']);
47
+        $userCountArray = $this->countUsers();
48
+        $total = 0;
49
+        if (!empty($userCountArray)) {
50
+            $rows = [];
51
+            foreach ($userCountArray as $classname => $users) {
52
+                $total += $users;
53
+                $rows[] = [$classname, $users];
54
+            }
55 55
 
56
-			$rows[] = [' '];
57
-			$rows[] = ['total users', $total];
58
-		} else {
59
-			$rows[] = ['No backend enabled that supports user counting', ''];
60
-		}
61
-		$rows[] = [' '];
56
+            $rows[] = [' '];
57
+            $rows[] = ['total users', $total];
58
+        } else {
59
+            $rows[] = ['No backend enabled that supports user counting', ''];
60
+        }
61
+        $rows[] = [' '];
62 62
 
63
-		if ($total <= self::DEFAULT_COUNT_DIRS_MAX_USERS || $input->getOption('count-dirs')) {
64
-			$userDirectoryCount = $this->countUserDirectories();
65
-			$rows[] = ['user directories', $userDirectoryCount];
66
-		}
63
+        if ($total <= self::DEFAULT_COUNT_DIRS_MAX_USERS || $input->getOption('count-dirs')) {
64
+            $userDirectoryCount = $this->countUserDirectories();
65
+            $rows[] = ['user directories', $userDirectoryCount];
66
+        }
67 67
 
68
-		$activeUsers = $this->userManager->countSeenUsers();
69
-		$rows[] = ['active users', $activeUsers];
68
+        $activeUsers = $this->userManager->countSeenUsers();
69
+        $rows[] = ['active users', $activeUsers];
70 70
 
71
-		$disabledUsers = $this->config->getUsersForUserValue('core', 'enabled', 'false');
72
-		$disabledUsersCount = count($disabledUsers);
73
-		$rows[] = ['disabled users', $disabledUsersCount];
71
+        $disabledUsers = $this->config->getUsersForUserValue('core', 'enabled', 'false');
72
+        $disabledUsersCount = count($disabledUsers);
73
+        $rows[] = ['disabled users', $disabledUsersCount];
74 74
 
75
-		$table->setRows($rows);
76
-		$table->render();
77
-		return 0;
78
-	}
75
+        $table->setRows($rows);
76
+        $table->render();
77
+        return 0;
78
+    }
79 79
 
80
-	private function countUsers(): array {
81
-		return $this->userManager->countUsers();
82
-	}
80
+    private function countUsers(): array {
81
+        return $this->userManager->countUsers();
82
+    }
83 83
 
84
-	private function countUserDirectories(): int {
85
-		$dataview = new View('/');
86
-		$userDirectories = $dataview->getDirectoryContent('/', 'httpd/unix-directory');
87
-		return count($userDirectories);
88
-	}
84
+    private function countUserDirectories(): int {
85
+        $dataview = new View('/');
86
+        $userDirectories = $dataview->getDirectoryContent('/', 'httpd/unix-directory');
87
+        return count($userDirectories);
88
+    }
89 89
 }
Please login to merge, or discard this patch.
apps/dav/lib/Listener/CardListener.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 				);
64 64
 			} catch (Throwable $e) {
65 65
 				// Any error with activities shouldn't abort the addressbook creation, so we just log it
66
-				$this->logger->error('Error generating activities for a new card in addressbook: ' . $e->getMessage(), [
66
+				$this->logger->error('Error generating activities for a new card in addressbook: '.$e->getMessage(), [
67 67
 					'exception' => $e,
68 68
 				]);
69 69
 			}
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 				);
82 82
 			} catch (Throwable $e) {
83 83
 				// Any error with activities shouldn't abort the addressbook update, so we just log it
84
-				$this->logger->error('Error generating activities for a changed card in addressbook: ' . $e->getMessage(), [
84
+				$this->logger->error('Error generating activities for a changed card in addressbook: '.$e->getMessage(), [
85 85
 					'exception' => $e,
86 86
 				]);
87 87
 			}
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 				);
100 100
 			} catch (Throwable $e) {
101 101
 				// Any error with activities shouldn't abort the addressbook deletion, so we just log it
102
-				$this->logger->error('Error generating activities for a deleted card in addressbook: ' . $e->getMessage(), [
102
+				$this->logger->error('Error generating activities for a deleted card in addressbook: '.$e->getMessage(), [
103 103
 					'exception' => $e,
104 104
 				]);
105 105
 			}
Please login to merge, or discard this patch.
Indentation   +59 added lines, -59 removed lines patch added patch discarded remove patch
@@ -21,67 +21,67 @@
 block discarded – undo
21 21
 
22 22
 /** @template-implements IEventListener<CardCreatedEvent|CardUpdatedEvent|CardDeletedEvent> */
23 23
 class CardListener implements IEventListener {
24
-	public function __construct(
25
-		private ActivityBackend $activityBackend,
26
-		private LoggerInterface $logger,
27
-	) {
28
-	}
24
+    public function __construct(
25
+        private ActivityBackend $activityBackend,
26
+        private LoggerInterface $logger,
27
+    ) {
28
+    }
29 29
 
30
-	public function handle(Event $event): void {
31
-		if ($event instanceof CardCreatedEvent) {
32
-			try {
33
-				$this->activityBackend->triggerCardActivity(
34
-					Card::SUBJECT_ADD,
35
-					$event->getAddressBookData(),
36
-					$event->getShares(),
37
-					$event->getCardData()
38
-				);
30
+    public function handle(Event $event): void {
31
+        if ($event instanceof CardCreatedEvent) {
32
+            try {
33
+                $this->activityBackend->triggerCardActivity(
34
+                    Card::SUBJECT_ADD,
35
+                    $event->getAddressBookData(),
36
+                    $event->getShares(),
37
+                    $event->getCardData()
38
+                );
39 39
 
40
-				$this->logger->debug(
41
-					sprintf('Activity generated for a new card in addressbook %d', $event->getAddressBookId())
42
-				);
43
-			} catch (Throwable $e) {
44
-				// Any error with activities shouldn't abort the addressbook creation, so we just log it
45
-				$this->logger->error('Error generating activities for a new card in addressbook: ' . $e->getMessage(), [
46
-					'exception' => $e,
47
-				]);
48
-			}
49
-		} elseif ($event instanceof CardUpdatedEvent) {
50
-			try {
51
-				$this->activityBackend->triggerCardActivity(
52
-					Card::SUBJECT_UPDATE,
53
-					$event->getAddressBookData(),
54
-					$event->getShares(),
55
-					$event->getCardData()
56
-				);
40
+                $this->logger->debug(
41
+                    sprintf('Activity generated for a new card in addressbook %d', $event->getAddressBookId())
42
+                );
43
+            } catch (Throwable $e) {
44
+                // Any error with activities shouldn't abort the addressbook creation, so we just log it
45
+                $this->logger->error('Error generating activities for a new card in addressbook: ' . $e->getMessage(), [
46
+                    'exception' => $e,
47
+                ]);
48
+            }
49
+        } elseif ($event instanceof CardUpdatedEvent) {
50
+            try {
51
+                $this->activityBackend->triggerCardActivity(
52
+                    Card::SUBJECT_UPDATE,
53
+                    $event->getAddressBookData(),
54
+                    $event->getShares(),
55
+                    $event->getCardData()
56
+                );
57 57
 
58
-				$this->logger->debug(
59
-					sprintf('Activity generated for a changed card in addressbook %d', $event->getAddressBookId())
60
-				);
61
-			} catch (Throwable $e) {
62
-				// Any error with activities shouldn't abort the addressbook update, so we just log it
63
-				$this->logger->error('Error generating activities for a changed card in addressbook: ' . $e->getMessage(), [
64
-					'exception' => $e,
65
-				]);
66
-			}
67
-		} elseif ($event instanceof CardDeletedEvent) {
68
-			try {
69
-				$this->activityBackend->triggerCardActivity(
70
-					Card::SUBJECT_DELETE,
71
-					$event->getAddressBookData(),
72
-					$event->getShares(),
73
-					$event->getCardData()
74
-				);
58
+                $this->logger->debug(
59
+                    sprintf('Activity generated for a changed card in addressbook %d', $event->getAddressBookId())
60
+                );
61
+            } catch (Throwable $e) {
62
+                // Any error with activities shouldn't abort the addressbook update, so we just log it
63
+                $this->logger->error('Error generating activities for a changed card in addressbook: ' . $e->getMessage(), [
64
+                    'exception' => $e,
65
+                ]);
66
+            }
67
+        } elseif ($event instanceof CardDeletedEvent) {
68
+            try {
69
+                $this->activityBackend->triggerCardActivity(
70
+                    Card::SUBJECT_DELETE,
71
+                    $event->getAddressBookData(),
72
+                    $event->getShares(),
73
+                    $event->getCardData()
74
+                );
75 75
 
76
-				$this->logger->debug(
77
-					sprintf('Activity generated for a deleted card in addressbook %d', $event->getAddressBookId())
78
-				);
79
-			} catch (Throwable $e) {
80
-				// Any error with activities shouldn't abort the addressbook deletion, so we just log it
81
-				$this->logger->error('Error generating activities for a deleted card in addressbook: ' . $e->getMessage(), [
82
-					'exception' => $e,
83
-				]);
84
-			}
85
-		}
86
-	}
76
+                $this->logger->debug(
77
+                    sprintf('Activity generated for a deleted card in addressbook %d', $event->getAddressBookId())
78
+                );
79
+            } catch (Throwable $e) {
80
+                // Any error with activities shouldn't abort the addressbook deletion, so we just log it
81
+                $this->logger->error('Error generating activities for a deleted card in addressbook: ' . $e->getMessage(), [
82
+                    'exception' => $e,
83
+                ]);
84
+            }
85
+        }
86
+    }
87 87
 }
Please login to merge, or discard this patch.
apps/dav/lib/Listener/AddressbookListener.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 				);
61 61
 			} catch (Throwable $e) {
62 62
 				// Any error with activities shouldn't abort the addressbook creation, so we just log it
63
-				$this->logger->error('Error generating activities for a new addressbook: ' . $e->getMessage(), [
63
+				$this->logger->error('Error generating activities for a new addressbook: '.$e->getMessage(), [
64 64
 					'exception' => $e,
65 65
 				]);
66 66
 			}
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 				);
78 78
 			} catch (Throwable $e) {
79 79
 				// Any error with activities shouldn't abort the addressbook update, so we just log it
80
-				$this->logger->error('Error generating activities for a changed addressbook: ' . $e->getMessage(), [
80
+				$this->logger->error('Error generating activities for a changed addressbook: '.$e->getMessage(), [
81 81
 					'exception' => $e,
82 82
 				]);
83 83
 			}
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 				);
94 94
 			} catch (Throwable $e) {
95 95
 				// Any error with activities shouldn't abort the addressbook deletion, so we just log it
96
-				$this->logger->error('Error generating activities for a deleted addressbook: ' . $e->getMessage(), [
96
+				$this->logger->error('Error generating activities for a deleted addressbook: '.$e->getMessage(), [
97 97
 					'exception' => $e,
98 98
 				]);
99 99
 			}
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
 				);
112 112
 			} catch (Throwable $e) {
113 113
 				// Any error with activities shouldn't abort the addressbook creation, so we just log it
114
-				$this->logger->error('Error generating activities for (un)sharing addressbook: ' . $e->getMessage(), [
114
+				$this->logger->error('Error generating activities for (un)sharing addressbook: '.$e->getMessage(), [
115 115
 					'exception' => $e,
116 116
 				]);
117 117
 			}
Please login to merge, or discard this patch.
Indentation   +70 added lines, -70 removed lines patch added patch discarded remove patch
@@ -21,79 +21,79 @@
 block discarded – undo
21 21
 
22 22
 /** @template-implements IEventListener<AddressBookCreatedEvent|AddressBookUpdatedEvent|AddressBookDeletedEvent|AddressBookShareUpdatedEvent> */
23 23
 class AddressbookListener implements IEventListener {
24
-	public function __construct(
25
-		private ActivityBackend $activityBackend,
26
-		private LoggerInterface $logger,
27
-	) {
28
-	}
24
+    public function __construct(
25
+        private ActivityBackend $activityBackend,
26
+        private LoggerInterface $logger,
27
+    ) {
28
+    }
29 29
 
30
-	public function handle(Event $event): void {
31
-		if ($event instanceof AddressBookCreatedEvent) {
32
-			try {
33
-				$this->activityBackend->onAddressbookCreate(
34
-					$event->getAddressBookData()
35
-				);
30
+    public function handle(Event $event): void {
31
+        if ($event instanceof AddressBookCreatedEvent) {
32
+            try {
33
+                $this->activityBackend->onAddressbookCreate(
34
+                    $event->getAddressBookData()
35
+                );
36 36
 
37
-				$this->logger->debug(
38
-					sprintf('Activity generated for new addressbook %d', $event->getAddressBookId())
39
-				);
40
-			} catch (Throwable $e) {
41
-				// Any error with activities shouldn't abort the addressbook creation, so we just log it
42
-				$this->logger->error('Error generating activities for a new addressbook: ' . $e->getMessage(), [
43
-					'exception' => $e,
44
-				]);
45
-			}
46
-		} elseif ($event instanceof AddressBookUpdatedEvent) {
47
-			try {
48
-				$this->activityBackend->onAddressbookUpdate(
49
-					$event->getAddressBookData(),
50
-					$event->getShares(),
51
-					$event->getMutations()
52
-				);
37
+                $this->logger->debug(
38
+                    sprintf('Activity generated for new addressbook %d', $event->getAddressBookId())
39
+                );
40
+            } catch (Throwable $e) {
41
+                // Any error with activities shouldn't abort the addressbook creation, so we just log it
42
+                $this->logger->error('Error generating activities for a new addressbook: ' . $e->getMessage(), [
43
+                    'exception' => $e,
44
+                ]);
45
+            }
46
+        } elseif ($event instanceof AddressBookUpdatedEvent) {
47
+            try {
48
+                $this->activityBackend->onAddressbookUpdate(
49
+                    $event->getAddressBookData(),
50
+                    $event->getShares(),
51
+                    $event->getMutations()
52
+                );
53 53
 
54
-				$this->logger->debug(
55
-					sprintf('Activity generated for changed addressbook %d', $event->getAddressBookId())
56
-				);
57
-			} catch (Throwable $e) {
58
-				// Any error with activities shouldn't abort the addressbook update, so we just log it
59
-				$this->logger->error('Error generating activities for a changed addressbook: ' . $e->getMessage(), [
60
-					'exception' => $e,
61
-				]);
62
-			}
63
-		} elseif ($event instanceof AddressBookDeletedEvent) {
64
-			try {
65
-				$this->activityBackend->onAddressbookDelete(
66
-					$event->getAddressBookData(),
67
-					$event->getShares()
68
-				);
54
+                $this->logger->debug(
55
+                    sprintf('Activity generated for changed addressbook %d', $event->getAddressBookId())
56
+                );
57
+            } catch (Throwable $e) {
58
+                // Any error with activities shouldn't abort the addressbook update, so we just log it
59
+                $this->logger->error('Error generating activities for a changed addressbook: ' . $e->getMessage(), [
60
+                    'exception' => $e,
61
+                ]);
62
+            }
63
+        } elseif ($event instanceof AddressBookDeletedEvent) {
64
+            try {
65
+                $this->activityBackend->onAddressbookDelete(
66
+                    $event->getAddressBookData(),
67
+                    $event->getShares()
68
+                );
69 69
 
70
-				$this->logger->debug(
71
-					sprintf('Activity generated for deleted addressbook %d', $event->getAddressBookId())
72
-				);
73
-			} catch (Throwable $e) {
74
-				// Any error with activities shouldn't abort the addressbook deletion, so we just log it
75
-				$this->logger->error('Error generating activities for a deleted addressbook: ' . $e->getMessage(), [
76
-					'exception' => $e,
77
-				]);
78
-			}
79
-		} elseif ($event instanceof AddressBookShareUpdatedEvent) {
80
-			try {
81
-				$this->activityBackend->onAddressbookUpdateShares(
82
-					$event->getAddressBookData(),
83
-					$event->getOldShares(),
84
-					$event->getAdded(),
85
-					$event->getRemoved()
86
-				);
70
+                $this->logger->debug(
71
+                    sprintf('Activity generated for deleted addressbook %d', $event->getAddressBookId())
72
+                );
73
+            } catch (Throwable $e) {
74
+                // Any error with activities shouldn't abort the addressbook deletion, so we just log it
75
+                $this->logger->error('Error generating activities for a deleted addressbook: ' . $e->getMessage(), [
76
+                    'exception' => $e,
77
+                ]);
78
+            }
79
+        } elseif ($event instanceof AddressBookShareUpdatedEvent) {
80
+            try {
81
+                $this->activityBackend->onAddressbookUpdateShares(
82
+                    $event->getAddressBookData(),
83
+                    $event->getOldShares(),
84
+                    $event->getAdded(),
85
+                    $event->getRemoved()
86
+                );
87 87
 
88
-				$this->logger->debug(
89
-					sprintf('Activity generated for (un)sharing addressbook %d', $event->getAddressBookId())
90
-				);
91
-			} catch (Throwable $e) {
92
-				// Any error with activities shouldn't abort the addressbook creation, so we just log it
93
-				$this->logger->error('Error generating activities for (un)sharing addressbook: ' . $e->getMessage(), [
94
-					'exception' => $e,
95
-				]);
96
-			}
97
-		}
98
-	}
88
+                $this->logger->debug(
89
+                    sprintf('Activity generated for (un)sharing addressbook %d', $event->getAddressBookId())
90
+                );
91
+            } catch (Throwable $e) {
92
+                // Any error with activities shouldn't abort the addressbook creation, so we just log it
93
+                $this->logger->error('Error generating activities for (un)sharing addressbook: ' . $e->getMessage(), [
94
+                    'exception' => $e,
95
+                ]);
96
+            }
97
+        }
98
+    }
99 99
 }
Please login to merge, or discard this patch.
apps/dav/lib/Listener/CalendarObjectReminderUpdaterListener.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 				);
79 79
 			} catch (Throwable $e) {
80 80
 				// Any error with reminders shouldn't abort the calendar move, so we just log it
81
-				$this->logger->error('Error cleaning up reminders of a calendar moved into trashbin: ' . $e->getMessage(), [
81
+				$this->logger->error('Error cleaning up reminders of a calendar moved into trashbin: '.$e->getMessage(), [
82 82
 					'exception' => $e,
83 83
 				]);
84 84
 			}
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 				);
94 94
 			} catch (Throwable $e) {
95 95
 				// Any error with activities shouldn't abort the calendar deletion, so we just log it
96
-				$this->logger->error('Error cleaning up reminders of a deleted calendar: ' . $e->getMessage(), [
96
+				$this->logger->error('Error cleaning up reminders of a deleted calendar: '.$e->getMessage(), [
97 97
 					'exception' => $e,
98 98
 				]);
99 99
 			}
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 				);
115 115
 			} catch (Throwable $e) {
116 116
 				// Any error with reminders shouldn't abort the calendar deletion, so we just log it
117
-				$this->logger->error('Error restoring reminders of a calendar: ' . $e->getMessage(), [
117
+				$this->logger->error('Error restoring reminders of a calendar: '.$e->getMessage(), [
118 118
 					'exception' => $e,
119 119
 				]);
120 120
 			}
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 				);
130 130
 			} catch (Throwable $e) {
131 131
 				// Any error with reminders shouldn't abort the calendar object creation, so we just log it
132
-				$this->logger->error('Error creating reminders of a calendar object: ' . $e->getMessage(), [
132
+				$this->logger->error('Error creating reminders of a calendar object: '.$e->getMessage(), [
133 133
 					'exception' => $e,
134 134
 				]);
135 135
 			}
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 				);
145 145
 			} catch (Throwable $e) {
146 146
 				// Any error with activities shouldn't abort the calendar object deletion, so we just log it
147
-				$this->logger->error('Error cleaning up reminders of a calendar object: ' . $e->getMessage(), [
147
+				$this->logger->error('Error cleaning up reminders of a calendar object: '.$e->getMessage(), [
148 148
 					'exception' => $e,
149 149
 				]);
150 150
 			}
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
 				);
160 160
 			} catch (Throwable $e) {
161 161
 				// Any error with reminders shouldn't abort the calendar object deletion, so we just log it
162
-				$this->logger->error('Error deleting reminders of a calendar object: ' . $e->getMessage(), [
162
+				$this->logger->error('Error deleting reminders of a calendar object: '.$e->getMessage(), [
163 163
 					'exception' => $e,
164 164
 				]);
165 165
 			}
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
 				);
175 175
 			} catch (Throwable $e) {
176 176
 				// Any error with reminders shouldn't abort the calendar object deletion, so we just log it
177
-				$this->logger->error('Error restoring reminders of a calendar object: ' . $e->getMessage(), [
177
+				$this->logger->error('Error restoring reminders of a calendar object: '.$e->getMessage(), [
178 178
 					'exception' => $e,
179 179
 				]);
180 180
 			}
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
 				);
190 190
 			} catch (Throwable $e) {
191 191
 				// Any error with activities shouldn't abort the calendar object deletion, so we just log it
192
-				$this->logger->error('Error cleaning up reminders of a deleted calendar object: ' . $e->getMessage(), [
192
+				$this->logger->error('Error cleaning up reminders of a deleted calendar object: '.$e->getMessage(), [
193 193
 					'exception' => $e,
194 194
 				]);
195 195
 			}
Please login to merge, or discard this patch.
Indentation   +128 added lines, -128 removed lines patch added patch discarded remove patch
@@ -28,141 +28,141 @@
 block discarded – undo
28 28
 /** @template-implements IEventListener<CalendarMovedToTrashEvent|CalendarDeletedEvent|CalendarRestoredEvent|CalendarObjectCreatedEvent|CalendarObjectUpdatedEvent|CalendarObjectMovedToTrashEvent|CalendarObjectRestoredEvent|CalendarObjectDeletedEvent> */
29 29
 class CalendarObjectReminderUpdaterListener implements IEventListener {
30 30
 
31
-	public function __construct(
32
-		private ReminderBackend $reminderBackend,
33
-		private ReminderService $reminderService,
34
-		private CalDavBackend $calDavBackend,
35
-		private LoggerInterface $logger,
36
-	) {
37
-	}
31
+    public function __construct(
32
+        private ReminderBackend $reminderBackend,
33
+        private ReminderService $reminderService,
34
+        private CalDavBackend $calDavBackend,
35
+        private LoggerInterface $logger,
36
+    ) {
37
+    }
38 38
 
39
-	public function handle(Event $event): void {
40
-		if ($event instanceof CalendarMovedToTrashEvent) {
41
-			try {
42
-				$this->reminderBackend->cleanRemindersForCalendar(
43
-					$event->getCalendarId()
44
-				);
39
+    public function handle(Event $event): void {
40
+        if ($event instanceof CalendarMovedToTrashEvent) {
41
+            try {
42
+                $this->reminderBackend->cleanRemindersForCalendar(
43
+                    $event->getCalendarId()
44
+                );
45 45
 
46
-				$this->logger->debug(
47
-					sprintf('Reminders of calendar %d cleaned up after move into trashbin', $event->getCalendarId())
48
-				);
49
-			} catch (Throwable $e) {
50
-				// Any error with reminders shouldn't abort the calendar move, so we just log it
51
-				$this->logger->error('Error cleaning up reminders of a calendar moved into trashbin: ' . $e->getMessage(), [
52
-					'exception' => $e,
53
-				]);
54
-			}
55
-		} elseif ($event instanceof CalendarDeletedEvent) {
56
-			try {
57
-				$this->reminderBackend->cleanRemindersForCalendar(
58
-					$event->getCalendarId()
59
-				);
46
+                $this->logger->debug(
47
+                    sprintf('Reminders of calendar %d cleaned up after move into trashbin', $event->getCalendarId())
48
+                );
49
+            } catch (Throwable $e) {
50
+                // Any error with reminders shouldn't abort the calendar move, so we just log it
51
+                $this->logger->error('Error cleaning up reminders of a calendar moved into trashbin: ' . $e->getMessage(), [
52
+                    'exception' => $e,
53
+                ]);
54
+            }
55
+        } elseif ($event instanceof CalendarDeletedEvent) {
56
+            try {
57
+                $this->reminderBackend->cleanRemindersForCalendar(
58
+                    $event->getCalendarId()
59
+                );
60 60
 
61
-				$this->logger->debug(
62
-					sprintf('Reminders of calendar %d cleaned up', $event->getCalendarId())
63
-				);
64
-			} catch (Throwable $e) {
65
-				// Any error with activities shouldn't abort the calendar deletion, so we just log it
66
-				$this->logger->error('Error cleaning up reminders of a deleted calendar: ' . $e->getMessage(), [
67
-					'exception' => $e,
68
-				]);
69
-			}
70
-		} elseif ($event instanceof CalendarRestoredEvent) {
71
-			try {
72
-				$objects = $this->calDavBackend->getCalendarObjects($event->getCalendarId());
73
-				$this->logger->debug(sprintf('Restoring calendar reminder objects for %d items', count($objects)));
74
-				foreach ($objects as $object) {
75
-					$fullObject = $this->calDavBackend->getCalendarObject(
76
-						$event->getCalendarId(),
77
-						$object['uri']
78
-					);
79
-					$this->reminderService->onCalendarObjectCreate($fullObject);
80
-				}
61
+                $this->logger->debug(
62
+                    sprintf('Reminders of calendar %d cleaned up', $event->getCalendarId())
63
+                );
64
+            } catch (Throwable $e) {
65
+                // Any error with activities shouldn't abort the calendar deletion, so we just log it
66
+                $this->logger->error('Error cleaning up reminders of a deleted calendar: ' . $e->getMessage(), [
67
+                    'exception' => $e,
68
+                ]);
69
+            }
70
+        } elseif ($event instanceof CalendarRestoredEvent) {
71
+            try {
72
+                $objects = $this->calDavBackend->getCalendarObjects($event->getCalendarId());
73
+                $this->logger->debug(sprintf('Restoring calendar reminder objects for %d items', count($objects)));
74
+                foreach ($objects as $object) {
75
+                    $fullObject = $this->calDavBackend->getCalendarObject(
76
+                        $event->getCalendarId(),
77
+                        $object['uri']
78
+                    );
79
+                    $this->reminderService->onCalendarObjectCreate($fullObject);
80
+                }
81 81
 
82
-				$this->logger->debug(
83
-					sprintf('Reminders of calendar %d restored', $event->getCalendarId())
84
-				);
85
-			} catch (Throwable $e) {
86
-				// Any error with reminders shouldn't abort the calendar deletion, so we just log it
87
-				$this->logger->error('Error restoring reminders of a calendar: ' . $e->getMessage(), [
88
-					'exception' => $e,
89
-				]);
90
-			}
91
-		} elseif ($event instanceof CalendarObjectCreatedEvent) {
92
-			try {
93
-				$this->reminderService->onCalendarObjectCreate(
94
-					$event->getObjectData()
95
-				);
82
+                $this->logger->debug(
83
+                    sprintf('Reminders of calendar %d restored', $event->getCalendarId())
84
+                );
85
+            } catch (Throwable $e) {
86
+                // Any error with reminders shouldn't abort the calendar deletion, so we just log it
87
+                $this->logger->error('Error restoring reminders of a calendar: ' . $e->getMessage(), [
88
+                    'exception' => $e,
89
+                ]);
90
+            }
91
+        } elseif ($event instanceof CalendarObjectCreatedEvent) {
92
+            try {
93
+                $this->reminderService->onCalendarObjectCreate(
94
+                    $event->getObjectData()
95
+                );
96 96
 
97
-				$this->logger->debug(
98
-					sprintf('Reminders of calendar object of calendar %d created', $event->getCalendarId())
99
-				);
100
-			} catch (Throwable $e) {
101
-				// Any error with reminders shouldn't abort the calendar object creation, so we just log it
102
-				$this->logger->error('Error creating reminders of a calendar object: ' . $e->getMessage(), [
103
-					'exception' => $e,
104
-				]);
105
-			}
106
-		} elseif ($event instanceof CalendarObjectUpdatedEvent) {
107
-			try {
108
-				$this->reminderService->onCalendarObjectEdit(
109
-					$event->getObjectData()
110
-				);
97
+                $this->logger->debug(
98
+                    sprintf('Reminders of calendar object of calendar %d created', $event->getCalendarId())
99
+                );
100
+            } catch (Throwable $e) {
101
+                // Any error with reminders shouldn't abort the calendar object creation, so we just log it
102
+                $this->logger->error('Error creating reminders of a calendar object: ' . $e->getMessage(), [
103
+                    'exception' => $e,
104
+                ]);
105
+            }
106
+        } elseif ($event instanceof CalendarObjectUpdatedEvent) {
107
+            try {
108
+                $this->reminderService->onCalendarObjectEdit(
109
+                    $event->getObjectData()
110
+                );
111 111
 
112
-				$this->logger->debug(
113
-					sprintf('Reminders of calendar object of calendar %d cleaned up', $event->getCalendarId())
114
-				);
115
-			} catch (Throwable $e) {
116
-				// Any error with activities shouldn't abort the calendar object deletion, so we just log it
117
-				$this->logger->error('Error cleaning up reminders of a calendar object: ' . $e->getMessage(), [
118
-					'exception' => $e,
119
-				]);
120
-			}
121
-		} elseif ($event instanceof CalendarObjectMovedToTrashEvent) {
122
-			try {
123
-				$this->reminderService->onCalendarObjectDelete(
124
-					$event->getObjectData()
125
-				);
112
+                $this->logger->debug(
113
+                    sprintf('Reminders of calendar object of calendar %d cleaned up', $event->getCalendarId())
114
+                );
115
+            } catch (Throwable $e) {
116
+                // Any error with activities shouldn't abort the calendar object deletion, so we just log it
117
+                $this->logger->error('Error cleaning up reminders of a calendar object: ' . $e->getMessage(), [
118
+                    'exception' => $e,
119
+                ]);
120
+            }
121
+        } elseif ($event instanceof CalendarObjectMovedToTrashEvent) {
122
+            try {
123
+                $this->reminderService->onCalendarObjectDelete(
124
+                    $event->getObjectData()
125
+                );
126 126
 
127
-				$this->logger->debug(
128
-					sprintf('Reminders of restored calendar object of calendar %d deleted', $event->getCalendarId())
129
-				);
130
-			} catch (Throwable $e) {
131
-				// Any error with reminders shouldn't abort the calendar object deletion, so we just log it
132
-				$this->logger->error('Error deleting reminders of a calendar object: ' . $e->getMessage(), [
133
-					'exception' => $e,
134
-				]);
135
-			}
136
-		} elseif ($event instanceof CalendarObjectRestoredEvent) {
137
-			try {
138
-				$this->reminderService->onCalendarObjectCreate(
139
-					$event->getObjectData()
140
-				);
127
+                $this->logger->debug(
128
+                    sprintf('Reminders of restored calendar object of calendar %d deleted', $event->getCalendarId())
129
+                );
130
+            } catch (Throwable $e) {
131
+                // Any error with reminders shouldn't abort the calendar object deletion, so we just log it
132
+                $this->logger->error('Error deleting reminders of a calendar object: ' . $e->getMessage(), [
133
+                    'exception' => $e,
134
+                ]);
135
+            }
136
+        } elseif ($event instanceof CalendarObjectRestoredEvent) {
137
+            try {
138
+                $this->reminderService->onCalendarObjectCreate(
139
+                    $event->getObjectData()
140
+                );
141 141
 
142
-				$this->logger->debug(
143
-					sprintf('Reminders of restored calendar object of calendar %d restored', $event->getCalendarId())
144
-				);
145
-			} catch (Throwable $e) {
146
-				// Any error with reminders shouldn't abort the calendar object deletion, so we just log it
147
-				$this->logger->error('Error restoring reminders of a calendar object: ' . $e->getMessage(), [
148
-					'exception' => $e,
149
-				]);
150
-			}
151
-		} elseif ($event instanceof CalendarObjectDeletedEvent) {
152
-			try {
153
-				$this->reminderService->onCalendarObjectDelete(
154
-					$event->getObjectData()
155
-				);
142
+                $this->logger->debug(
143
+                    sprintf('Reminders of restored calendar object of calendar %d restored', $event->getCalendarId())
144
+                );
145
+            } catch (Throwable $e) {
146
+                // Any error with reminders shouldn't abort the calendar object deletion, so we just log it
147
+                $this->logger->error('Error restoring reminders of a calendar object: ' . $e->getMessage(), [
148
+                    'exception' => $e,
149
+                ]);
150
+            }
151
+        } elseif ($event instanceof CalendarObjectDeletedEvent) {
152
+            try {
153
+                $this->reminderService->onCalendarObjectDelete(
154
+                    $event->getObjectData()
155
+                );
156 156
 
157
-				$this->logger->debug(
158
-					sprintf('Reminders of calendar object of calendar %d cleaned up', $event->getCalendarId())
159
-				);
160
-			} catch (Throwable $e) {
161
-				// Any error with activities shouldn't abort the calendar object deletion, so we just log it
162
-				$this->logger->error('Error cleaning up reminders of a deleted calendar object: ' . $e->getMessage(), [
163
-					'exception' => $e,
164
-				]);
165
-			}
166
-		}
167
-	}
157
+                $this->logger->debug(
158
+                    sprintf('Reminders of calendar object of calendar %d cleaned up', $event->getCalendarId())
159
+                );
160
+            } catch (Throwable $e) {
161
+                // Any error with activities shouldn't abort the calendar object deletion, so we just log it
162
+                $this->logger->error('Error cleaning up reminders of a deleted calendar object: ' . $e->getMessage(), [
163
+                    'exception' => $e,
164
+                ]);
165
+            }
166
+        }
167
+    }
168 168
 }
Please login to merge, or discard this patch.
apps/dav/lib/Migration/Version1018Date20210312100735.php 1 patch
Indentation   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -12,34 +12,34 @@
 block discarded – undo
12 12
 
13 13
 class Version1018Date20210312100735 extends SimpleMigrationStep {
14 14
 
15
-	/**
16
-	 * @param IOutput $output
17
-	 * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
18
-	 * @param array $options
19
-	 * @return ISchemaWrapper
20
-	 */
21
-	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
22
-		/** @var ISchemaWrapper $schema */
23
-		$schema = $schemaClosure();
24
-
25
-		$calendarsTable = $schema->getTable('calendars');
26
-		$calendarsTable->addColumn('deleted_at', Types::INTEGER, [
27
-			'notnull' => false,
28
-			'length' => 4,
29
-			'unsigned' => true,
30
-		]);
31
-		$calendarsTable->addIndex([
32
-			'principaluri',
33
-			'deleted_at',
34
-		], 'cals_princ_del_idx');
35
-
36
-		$calendarObjectsTable = $schema->getTable('calendarobjects');
37
-		$calendarObjectsTable->addColumn('deleted_at', Types::INTEGER, [
38
-			'notnull' => false,
39
-			'length' => 4,
40
-			'unsigned' => true,
41
-		]);
42
-
43
-		return $schema;
44
-	}
15
+    /**
16
+     * @param IOutput $output
17
+     * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
18
+     * @param array $options
19
+     * @return ISchemaWrapper
20
+     */
21
+    public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
22
+        /** @var ISchemaWrapper $schema */
23
+        $schema = $schemaClosure();
24
+
25
+        $calendarsTable = $schema->getTable('calendars');
26
+        $calendarsTable->addColumn('deleted_at', Types::INTEGER, [
27
+            'notnull' => false,
28
+            'length' => 4,
29
+            'unsigned' => true,
30
+        ]);
31
+        $calendarsTable->addIndex([
32
+            'principaluri',
33
+            'deleted_at',
34
+        ], 'cals_princ_del_idx');
35
+
36
+        $calendarObjectsTable = $schema->getTable('calendarobjects');
37
+        $calendarObjectsTable->addColumn('deleted_at', Types::INTEGER, [
38
+            'notnull' => false,
39
+            'length' => 4,
40
+            'unsigned' => true,
41
+        ]);
42
+
43
+        return $schema;
44
+    }
45 45
 }
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/IRestorable.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -32,10 +32,10 @@
 block discarded – undo
32 32
  */
33 33
 interface IRestorable {
34 34
 
35
-	/**
36
-	 * Restore this node
37
-	 *
38
-	 * @throws Exception
39
-	 */
40
-	public function restore(): void;
35
+    /**
36
+     * Restore this node
37
+     *
38
+     * @throws Exception
39
+     */
40
+    public function restore(): void;
41 41
 }
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/Trashbin/RestoreTarget.php 1 patch
Indentation   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -33,50 +33,50 @@
 block discarded – undo
33 33
 use Sabre\DAV\INode;
34 34
 
35 35
 class RestoreTarget implements ICollection, IMoveTarget {
36
-	public const NAME = 'restore';
36
+    public const NAME = 'restore';
37 37
 
38
-	public function createFile($name, $data = null) {
39
-		throw new Forbidden();
40
-	}
38
+    public function createFile($name, $data = null) {
39
+        throw new Forbidden();
40
+    }
41 41
 
42
-	public function createDirectory($name) {
43
-		throw new Forbidden();
44
-	}
42
+    public function createDirectory($name) {
43
+        throw new Forbidden();
44
+    }
45 45
 
46
-	public function getChild($name) {
47
-		throw new NotFound();
48
-	}
46
+    public function getChild($name) {
47
+        throw new NotFound();
48
+    }
49 49
 
50
-	public function getChildren(): array {
51
-		return [];
52
-	}
50
+    public function getChildren(): array {
51
+        return [];
52
+    }
53 53
 
54
-	public function childExists($name): bool {
55
-		return false;
56
-	}
54
+    public function childExists($name): bool {
55
+        return false;
56
+    }
57 57
 
58
-	public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
59
-		if ($sourceNode instanceof IRestorable) {
60
-			$sourceNode->restore();
61
-			return true;
62
-		}
58
+    public function moveInto($targetName, $sourcePath, INode $sourceNode): bool {
59
+        if ($sourceNode instanceof IRestorable) {
60
+            $sourceNode->restore();
61
+            return true;
62
+        }
63 63
 
64
-		return false;
65
-	}
64
+        return false;
65
+    }
66 66
 
67
-	public function delete() {
68
-		throw new Forbidden();
69
-	}
67
+    public function delete() {
68
+        throw new Forbidden();
69
+    }
70 70
 
71
-	public function getName(): string {
72
-		return 'restore';
73
-	}
71
+    public function getName(): string {
72
+        return 'restore';
73
+    }
74 74
 
75
-	public function setName($name) {
76
-		throw new Forbidden();
77
-	}
75
+    public function setName($name) {
76
+        throw new Forbidden();
77
+    }
78 78
 
79
-	public function getLastModified() {
80
-		return 0;
81
-	}
79
+    public function getLastModified() {
80
+        return 0;
81
+    }
82 82
 }
Please login to merge, or discard this patch.
lib/public/AppFramework/Http/DownloadResponse.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
 
42 42
 		$filename = strtr($filename, ['"' => '\\"', '\\' => '\\\\']);
43 43
 
44
-		$this->addHeader('Content-Disposition', 'attachment; filename="' . $filename . '"');
44
+		$this->addHeader('Content-Disposition', 'attachment; filename="'.$filename.'"');
45 45
 		$this->addHeader('Content-Type', $contentType);
46 46
 	}
47 47
 }
Please login to merge, or discard this patch.
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -18,20 +18,20 @@
 block discarded – undo
18 18
  * @template-extends Response<Http::STATUS_*, array<string, mixed>>
19 19
  */
20 20
 class DownloadResponse extends Response {
21
-	/**
22
-	 * Creates a response that prompts the user to download the file
23
-	 * @param string $filename the name that the downloaded file should have
24
-	 * @param C $contentType the mimetype that the downloaded file should have
25
-	 * @param S $status
26
-	 * @param H $headers
27
-	 * @since 7.0.0
28
-	 */
29
-	public function __construct(string $filename, string $contentType, int $status = Http::STATUS_OK, array $headers = []) {
30
-		parent::__construct($status, $headers);
21
+    /**
22
+     * Creates a response that prompts the user to download the file
23
+     * @param string $filename the name that the downloaded file should have
24
+     * @param C $contentType the mimetype that the downloaded file should have
25
+     * @param S $status
26
+     * @param H $headers
27
+     * @since 7.0.0
28
+     */
29
+    public function __construct(string $filename, string $contentType, int $status = Http::STATUS_OK, array $headers = []) {
30
+        parent::__construct($status, $headers);
31 31
 
32
-		$filename = strtr($filename, ['"' => '\\"', '\\' => '\\\\']);
32
+        $filename = strtr($filename, ['"' => '\\"', '\\' => '\\\\']);
33 33
 
34
-		$this->addHeader('Content-Disposition', 'attachment; filename="' . $filename . '"');
35
-		$this->addHeader('Content-Type', $contentType);
36
-	}
34
+        $this->addHeader('Content-Disposition', 'attachment; filename="' . $filename . '"');
35
+        $this->addHeader('Content-Type', $contentType);
36
+    }
37 37
 }
Please login to merge, or discard this patch.