Passed
Push — master ( 6ce3c5...504e18 )
by John
14:34 queued 13s
created
apps/comments/lib/Search/Result.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -105,7 +105,7 @@
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
lib/public/AppFramework/Http/Template/ExternalShareMenuAction.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -66,16 +66,16 @@
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
apps/workflowengine/lib/Check/FileMimeType.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -155,7 +155,7 @@
 block discarded – undo
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 {
Please login to merge, or discard this patch.
apps/settings/lib/Settings/Personal/Security/TwoFactor.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -106,12 +106,12 @@
 block discarded – undo
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
 		];
Please login to merge, or discard this patch.
lib/private/BackgroundJob/Job.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -57,17 +57,17 @@
 block discarded – undo
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
 		}
Please login to merge, or discard this patch.
apps/files/lib/Controller/TransferOwnershipController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
apps/files_sharing/lib/Controller/ExternalSharesController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -135,15 +135,15 @@
 block discarded – undo
135 135
 		}
136 136
 
137 137
 		if (
138
-			$this->testUrl('https://' . $remote . '/ocs-provider/') ||
139
-			$this->testUrl('https://' . $remote . '/ocs-provider/index.php') ||
140
-			$this->testUrl('https://' . $remote . '/status.php', true)
138
+			$this->testUrl('https://'.$remote.'/ocs-provider/') ||
139
+			$this->testUrl('https://'.$remote.'/ocs-provider/index.php') ||
140
+			$this->testUrl('https://'.$remote.'/status.php', true)
141 141
 		) {
142 142
 			return new DataResponse('https');
143 143
 		} elseif (
144
-			$this->testUrl('http://' . $remote . '/ocs-provider/') ||
145
-			$this->testUrl('http://' . $remote . '/ocs-provider/index.php') ||
146
-			$this->testUrl('http://' . $remote . '/status.php', true)
144
+			$this->testUrl('http://'.$remote.'/ocs-provider/') ||
145
+			$this->testUrl('http://'.$remote.'/ocs-provider/index.php') ||
146
+			$this->testUrl('http://'.$remote.'/status.php', true)
147 147
 		) {
148 148
 			return new DataResponse('http');
149 149
 		} else {
Please login to merge, or discard this patch.
apps/contactsinteraction/lib/Migration/Version010000Date20200304152605.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -75,17 +75,17 @@
 block discarded – undo
75 75
 		]);
76 76
 		$table->setPrimaryKey(['id']);
77 77
 		// To find all recent entries
78
-		$table->addIndex(['actor_uid'], RecentContactMapper::TABLE_NAME . '_actor_uid');
78
+		$table->addIndex(['actor_uid'], RecentContactMapper::TABLE_NAME.'_actor_uid');
79 79
 		// To find a specific entry
80
-		$table->addIndex(['id', 'actor_uid'], RecentContactMapper::TABLE_NAME . '_id_uid');
80
+		$table->addIndex(['id', 'actor_uid'], RecentContactMapper::TABLE_NAME.'_id_uid');
81 81
 		// To find all recent entries with a given UID
82
-		$table->addIndex(['uid'], RecentContactMapper::TABLE_NAME . '_uid');
82
+		$table->addIndex(['uid'], RecentContactMapper::TABLE_NAME.'_uid');
83 83
 		// To find all recent entries with a given email address
84
-		$table->addIndex(['email'], RecentContactMapper::TABLE_NAME . '_email');
84
+		$table->addIndex(['email'], RecentContactMapper::TABLE_NAME.'_email');
85 85
 		// To find all recent entries with a give federated cloud id
86
-		$table->addIndex(['federated_cloud_id'], RecentContactMapper::TABLE_NAME . '_fed_id');
86
+		$table->addIndex(['federated_cloud_id'], RecentContactMapper::TABLE_NAME.'_fed_id');
87 87
 		// For the cleanup
88
-		$table->addIndex(['last_contact'], RecentContactMapper::TABLE_NAME . '_last_contact');
88
+		$table->addIndex(['last_contact'], RecentContactMapper::TABLE_NAME.'_last_contact');
89 89
 
90 90
 		return $schema;
91 91
 	}
Please login to merge, or discard this patch.
apps/contactsinteraction/lib/Db/CardSearchDao.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
 		}
68 68
 		$addressbooksQuery->selectDistinct('id')
69 69
 			->from('addressbooks')
70
-			->where($addressbooksQuery->expr()->eq('principaluri', $cardQuery->createNamedParameter("principals/users/" . $user->getUID())));
70
+			->where($addressbooksQuery->expr()->eq('principaluri', $cardQuery->createNamedParameter("principals/users/".$user->getUID())));
71 71
 		$propQuery->selectDistinct('cardid')
72 72
 			->from('cards_properties')
73 73
 			->where($propQuery->expr()->in('addressbookid', $propQuery->createFunction($addressbooksQuery->getSQL()), IQueryBuilder::PARAM_INT_ARRAY))
Please login to merge, or discard this patch.