@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | $gid = $input->getArgument('groupid'); |
66 | 66 | $group = $this->groupManager->get($gid); |
67 | 67 | if ($group) { |
68 | - $output->writeln('<error>Group "' . $gid . '" already exists.</error>'); |
|
68 | + $output->writeln('<error>Group "'.$gid.'" already exists.</error>'); |
|
69 | 69 | return 1; |
70 | 70 | } else { |
71 | 71 | $group = $this->groupManager->createGroup($gid); |
@@ -73,9 +73,9 @@ discard block |
||
73 | 73 | $output->writeln('<error>Could not create group</error>'); |
74 | 74 | return 2; |
75 | 75 | } |
76 | - $output->writeln('Created group "' . $group->getGID() . '"'); |
|
76 | + $output->writeln('Created group "'.$group->getGID().'"'); |
|
77 | 77 | |
78 | - $displayName = trim((string)$input->getOption('display-name')); |
|
78 | + $displayName = trim((string) $input->getOption('display-name')); |
|
79 | 79 | if ($displayName !== '') { |
80 | 80 | $group->setDisplayName($displayName); |
81 | 81 | } |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | $destinationUserObject = $this->userManager->get($destinationUser); |
99 | 99 | |
100 | 100 | if (!$sourceUserObject instanceof IUser) { |
101 | - $this->logger->alert('Could not transfer ownership: Unknown source user ' . $sourceUser); |
|
101 | + $this->logger->alert('Could not transfer ownership: Unknown source user '.$sourceUser); |
|
102 | 102 | $this->failedNotication($transfer); |
103 | 103 | return; |
104 | 104 | } |
@@ -136,7 +136,7 @@ discard block |
||
136 | 136 | 'targetUser' => $transfer->getTargetUser(), |
137 | 137 | 'nodeName' => $transfer->getNodeName(), |
138 | 138 | ]) |
139 | - ->setObject('transfer', (string)$transfer->getId()); |
|
139 | + ->setObject('transfer', (string) $transfer->getId()); |
|
140 | 140 | $this->notificationManager->notify($notification); |
141 | 141 | |
142 | 142 | // Send notification to source user |
@@ -149,7 +149,7 @@ discard block |
||
149 | 149 | 'targetUser' => $transfer->getTargetUser(), |
150 | 150 | 'nodeName' => $transfer->getNodeName(), |
151 | 151 | ]) |
152 | - ->setObject('transfer', (string)$transfer->getId()); |
|
152 | + ->setObject('transfer', (string) $transfer->getId()); |
|
153 | 153 | $this->notificationManager->notify($notification); |
154 | 154 | } |
155 | 155 | |
@@ -164,7 +164,7 @@ discard block |
||
164 | 164 | 'targetUser' => $transfer->getTargetUser(), |
165 | 165 | 'nodeName' => $transfer->getNodeName(), |
166 | 166 | ]) |
167 | - ->setObject('transfer', (string)$transfer->getId()); |
|
167 | + ->setObject('transfer', (string) $transfer->getId()); |
|
168 | 168 | $this->notificationManager->notify($notification); |
169 | 169 | |
170 | 170 | // Send notification to source user |
@@ -177,7 +177,7 @@ discard block |
||
177 | 177 | 'targetUser' => $transfer->getTargetUser(), |
178 | 178 | 'nodeName' => $transfer->getNodeName(), |
179 | 179 | ]) |
180 | - ->setObject('transfer', (string)$transfer->getId()); |
|
180 | + ->setObject('transfer', (string) $transfer->getId()); |
|
181 | 181 | $this->notificationManager->notify($notification); |
182 | 182 | } |
183 | 183 | } |
@@ -105,7 +105,7 @@ |
||
105 | 105 | $suffix = '…'; |
106 | 106 | } |
107 | 107 | |
108 | - return $prefix . mb_substr($message, $start, $end - $start) . $suffix; |
|
108 | + return $prefix.mb_substr($message, $start, $end - $start).$suffix; |
|
109 | 109 | } |
110 | 110 | |
111 | 111 | } |
@@ -66,16 +66,16 @@ |
||
66 | 66 | * @since 14.0.0 |
67 | 67 | */ |
68 | 68 | public function render(): string { |
69 | - return '<li>' . |
|
70 | - ' <button id="save-external-share" class="icon ' . Util::sanitizeHTML($this->getIcon()) . '" data-protected="false" data-owner-display-name="' . Util::sanitizeHTML($this->displayname) . '" data-owner="' . Util::sanitizeHTML($this->owner) . '" data-name="' . Util::sanitizeHTML($this->shareName) . '">' . Util::sanitizeHTML($this->getLabel()) . '</button>' . |
|
71 | - '</li>' . |
|
72 | - '<li id="external-share-menu-item" class="hidden">' . |
|
73 | - ' <span class="menuitem">' . |
|
74 | - ' <form class="save-form" action="#">' . |
|
75 | - ' <input type="text" id="remote_address" placeholder="[email protected]">' . |
|
76 | - ' <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>' . |
|
77 | - ' </form>' . |
|
78 | - ' </span>' . |
|
69 | + return '<li>'. |
|
70 | + ' <button id="save-external-share" class="icon '.Util::sanitizeHTML($this->getIcon()).'" data-protected="false" data-owner-display-name="'.Util::sanitizeHTML($this->displayname).'" data-owner="'.Util::sanitizeHTML($this->owner).'" data-name="'.Util::sanitizeHTML($this->shareName).'">'.Util::sanitizeHTML($this->getLabel()).'</button>'. |
|
71 | + '</li>'. |
|
72 | + '<li id="external-share-menu-item" class="hidden">'. |
|
73 | + ' <span class="menuitem">'. |
|
74 | + ' <form class="save-form" action="#">'. |
|
75 | + ' <input type="text" id="remote_address" placeholder="[email protected]">'. |
|
76 | + ' <input type="submit" value=" " id="save-button-confirm" class="icon-confirm" disabled="disabled"></button>'. |
|
77 | + ' </form>'. |
|
78 | + ' </span>'. |
|
79 | 79 | '</li>'; |
80 | 80 | } |
81 | 81 | } |
@@ -155,7 +155,7 @@ |
||
155 | 155 | } |
156 | 156 | |
157 | 157 | public function supportedEntities(): array { |
158 | - return [ File::class ]; |
|
158 | + return [File::class]; |
|
159 | 159 | } |
160 | 160 | |
161 | 161 | public function isAvailableForScope(int $scope): bool { |
@@ -106,12 +106,12 @@ |
||
106 | 106 | } |
107 | 107 | |
108 | 108 | return [ |
109 | - 'providers' => array_map(function (IProvidesPersonalSettings $provider) use ($user) { |
|
109 | + 'providers' => array_map(function(IProvidesPersonalSettings $provider) use ($user) { |
|
110 | 110 | return [ |
111 | 111 | 'provider' => $provider, |
112 | 112 | 'settings' => $provider->getPersonalSettings($user) |
113 | 113 | ]; |
114 | - }, array_filter($this->providerLoader->getProviders($user), function (IProvider $provider) { |
|
114 | + }, array_filter($this->providerLoader->getProviders($user), function(IProvider $provider) { |
|
115 | 115 | return $provider instanceof IProvidesPersonalSettings; |
116 | 116 | })) |
117 | 117 | ]; |
@@ -57,17 +57,17 @@ |
||
57 | 57 | |
58 | 58 | try { |
59 | 59 | $jobStartTime = time(); |
60 | - $logger->debug('Run ' . get_class($this) . ' job with ID ' . $this->getId(), ['app' => 'cron']); |
|
60 | + $logger->debug('Run '.get_class($this).' job with ID '.$this->getId(), ['app' => 'cron']); |
|
61 | 61 | $this->run($this->argument); |
62 | 62 | $timeTaken = time() - $jobStartTime; |
63 | 63 | |
64 | - $logger->debug('Finished ' . get_class($this) . ' job with ID ' . $this->getId() . ' in ' . $timeTaken . ' seconds', ['app' => 'cron']); |
|
64 | + $logger->debug('Finished '.get_class($this).' job with ID '.$this->getId().' in '.$timeTaken.' seconds', ['app' => 'cron']); |
|
65 | 65 | $jobList->setExecutionTime($this, $timeTaken); |
66 | 66 | } catch (\Throwable $e) { |
67 | 67 | if ($logger) { |
68 | 68 | $logger->logException($e, [ |
69 | 69 | 'app' => 'core', |
70 | - 'message' => 'Error while running background job (class: ' . get_class($this) . ', arguments: ' . print_r($this->argument, true) . ')' |
|
70 | + 'message' => 'Error while running background job (class: '.get_class($this).', arguments: '.print_r($this->argument, true).')' |
|
71 | 71 | ]); |
72 | 72 | } |
73 | 73 | } |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | 'targetUser' => $recipient, |
113 | 113 | 'nodeName' => $node->getName(), |
114 | 114 | ]) |
115 | - ->setObject('transfer', (string)$transferOwnership->getId()); |
|
115 | + ->setObject('transfer', (string) $transferOwnership->getId()); |
|
116 | 116 | |
117 | 117 | $this->notificationManager->notify($notification); |
118 | 118 | |
@@ -135,7 +135,7 @@ discard block |
||
135 | 135 | |
136 | 136 | $notification = $this->notificationManager->createNotification(); |
137 | 137 | $notification->setApp('files') |
138 | - ->setObject('transfer', (string)$id); |
|
138 | + ->setObject('transfer', (string) $id); |
|
139 | 139 | $this->notificationManager->markProcessed($notification); |
140 | 140 | |
141 | 141 | $newTransferOwnership = new TransferOwnershipEntity(); |
@@ -168,7 +168,7 @@ discard block |
||
168 | 168 | |
169 | 169 | $notification = $this->notificationManager->createNotification(); |
170 | 170 | $notification->setApp('files') |
171 | - ->setObject('transfer', (string)$id); |
|
171 | + ->setObject('transfer', (string) $id); |
|
172 | 172 | $this->notificationManager->markProcessed($notification); |
173 | 173 | |
174 | 174 | $notification = $this->notificationManager->createNotification(); |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | 'targetUser' => $transferOwnership->getTargetUser(), |
181 | 181 | 'nodeName' => $transferOwnership->getNodeName() |
182 | 182 | ]) |
183 | - ->setObject('transfer', (string)$transferOwnership->getId()); |
|
183 | + ->setObject('transfer', (string) $transferOwnership->getId()); |
|
184 | 184 | $this->notificationManager->notify($notification); |
185 | 185 | |
186 | 186 | $this->mapper->delete($transferOwnership); |
@@ -212,7 +212,7 @@ |
||
212 | 212 | return $this->file->fopen('w'); |
213 | 213 | } else { |
214 | 214 | $source = fopen('php://temp', 'w+'); |
215 | - return CallbackWrapper::wrap($source, null, null, null, null, function () use ($source) { |
|
215 | + return CallbackWrapper::wrap($source, null, null, null, null, function() use ($source) { |
|
216 | 216 | rewind($source); |
217 | 217 | $this->putContent($source); |
218 | 218 | }); |