Completed
Pull Request — master (#445)
by Maxence
01:58
created
lib/Db/CircleProviderRequestBuilder.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
79 79
 		$qb->andWhere(
80 80
 			$expr->in(
81
-				$pf . 'share_with',
81
+				$pf.'share_with',
82 82
 				$qb->createNamedParameter($circleUniqueIds, IQueryBuilder::PARAM_STR_ARRAY)
83 83
 			)
84 84
 		);
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 		$expr = $qb->expr();
96 96
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
97 97
 
98
-		$qb->andWhere($expr->eq($pf . 'id', $qb->createNamedParameter($shareId)));
98
+		$qb->andWhere($expr->eq($pf.'id', $qb->createNamedParameter($shareId)));
99 99
 	}
100 100
 
101 101
 
@@ -144,8 +144,8 @@  discard block
 block discarded – undo
144 144
 		/** @noinspection PhpMethodParametersCountMismatchInspection */
145 145
 		$qb->andWhere(
146 146
 			$expr->orX(
147
-				$expr->eq($pf . 'parent', $qb->createNamedParameter($circleId)),
148
-				$expr->eq($pf . 'id', $qb->createNamedParameter($circleId))
147
+				$expr->eq($pf.'parent', $qb->createNamedParameter($circleId)),
148
+				$expr->eq($pf.'id', $qb->createNamedParameter($circleId))
149 149
 			)
150 150
 		);
151 151
 	}
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
168 168
 		$qb->andWhere(
169 169
 			$expr->in(
170
-				$pf . 'file_source',
170
+				$pf.'file_source',
171 171
 				$qb->createNamedParameter($files, IQueryBuilder::PARAM_INT_ARRAY)
172 172
 			)
173 173
 		);
@@ -200,13 +200,13 @@  discard block
 block discarded – undo
200 200
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
201 201
 
202 202
 		if ($reShares === false) {
203
-			$qb->andWhere($expr->eq($pf . 'uid_initiator', $qb->createNamedParameter($userId)));
203
+			$qb->andWhere($expr->eq($pf.'uid_initiator', $qb->createNamedParameter($userId)));
204 204
 		} else {
205 205
 			/** @noinspection PhpMethodParametersCountMismatchInspection */
206 206
 			$qb->andWhere(
207 207
 				$expr->orX(
208
-					$expr->eq($pf . 'uid_owner', $qb->createNamedParameter($userId)),
209
-					$expr->eq($pf . 'uid_initiator', $qb->createNamedParameter($userId))
208
+					$expr->eq($pf.'uid_owner', $qb->createNamedParameter($userId)),
209
+					$expr->eq($pf.'uid_initiator', $qb->createNamedParameter($userId))
210 210
 				)
211 211
 			);
212 212
 		}
@@ -304,11 +304,11 @@  discard block
 block discarded – undo
304 304
 		$expr = $qb->expr();
305 305
 		$andX = $expr->andX();
306 306
 
307
-		$andX->add($expr->eq($aliasM . '.user_id', $qb->createNamedParameter($userId)));
308
-		$andX->add($expr->eq($aliasM . '.circle_id', $aliasC . '.unique_id'));
309
-		$andX->add($expr->gte($aliasM . '.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
310
-		$andX->add($expr->eq($aliasM . '.instance', $qb->createNamedParameter('')));
311
-		$andX->add($expr->eq($aliasM . '.user_type', $qb->createNamedParameter(Member::TYPE_USER)));
307
+		$andX->add($expr->eq($aliasM.'.user_id', $qb->createNamedParameter($userId)));
308
+		$andX->add($expr->eq($aliasM.'.circle_id', $aliasC.'.unique_id'));
309
+		$andX->add($expr->gte($aliasM.'.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
310
+		$andX->add($expr->eq($aliasM.'.instance', $qb->createNamedParameter('')));
311
+		$andX->add($expr->eq($aliasM.'.user_type', $qb->createNamedParameter(Member::TYPE_USER)));
312 312
 
313 313
 		return $andX;
314 314
 	}
@@ -330,10 +330,10 @@  discard block
 block discarded – undo
330 330
 		$expr = $qb->expr();
331 331
 		$andX = $expr->andX();
332 332
 
333
-		$andX->add($expr->eq($aliasM . '.user_id', $qb->createNamedParameter($userId)));
334
-		$andX->add($expr->eq($aliasM . '.circle_id', $aliasC . '.unique_id'));
333
+		$andX->add($expr->eq($aliasM.'.user_id', $qb->createNamedParameter($userId)));
334
+		$andX->add($expr->eq($aliasM.'.circle_id', $aliasC.'.unique_id'));
335 335
 
336
-		$andX->add($expr->gte($aliasM . '.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
336
+		$andX->add($expr->gte($aliasM.'.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
337 337
 
338 338
 		return $andX;
339 339
 	}
Please login to merge, or discard this patch.
lib/Service/SharingFrameService.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 	public function createFrame($circleUniqueId, SharingFrame $frame, $broadcast = null) {
147 147
 
148 148
 		$this->miscService->log(
149
-			'Create frame with payload ' . json_encode($frame->getPayload()) . ' as ' . $this->userId, 0
149
+			'Create frame with payload '.json_encode($frame->getPayload()).' as '.$this->userId, 0
150 150
 		);
151 151
 		try {
152 152
 			$circle = $this->circlesRequest->getCircle($circleUniqueId, $this->userId);
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 			$this->initiateShare($circle->getUniqueId(), $frame->getUniqueId());
162 162
 		} catch (Exception $e) {
163 163
 			$this->miscService->log(
164
-				'fail to create frame for circle ' . $circleUniqueId . ' - ' . $e->getMessage()
164
+				'fail to create frame for circle '.$circleUniqueId.' - '.$e->getMessage()
165 165
 			);
166 166
 			throw $e;
167 167
 		}
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 		];
301 301
 
302 302
 		$client = $this->clientService->newClient();
303
-		$addr = $this->configService->getLocalAddress() . \OC::$WEBROOT;
303
+		$addr = $this->configService->getLocalAddress().\OC::$WEBROOT;
304 304
 		$opts = [
305 305
 			'body'            => $args,
306 306
 			'timeout'         => Application::CLIENT_TIMEOUT,
@@ -317,8 +317,8 @@  discard block
 block discarded – undo
317 317
 			return true;
318 318
 		} catch (Exception $e) {
319 319
 			$this->miscService->log(
320
-				'fail to initialise circle share to ' . $addr . ' for circle ' . $circleUniqueId . ' - '
321
-				. json_encode($opts) . ' - ' . $e->getMessage(), 3
320
+				'fail to initialise circle share to '.$addr.' for circle '.$circleUniqueId.' - '
321
+				. json_encode($opts).' - '.$e->getMessage(), 3
322 322
 			);
323 323
 			throw $e;
324 324
 		}
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
 	 * @return string
332 332
 	 */
333 333
 	private function generatePayloadDeliveryURL($remote) {
334
-		return $this->configService->generateRemoteHost($remote) .
334
+		return $this->configService->generateRemoteHost($remote).
335 335
 			   $this->urlGenerator->linkToRoute('circles.Shares.initShareDelivery');
336 336
 	}
337 337
 
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
 
407 407
 		} catch (Exception $e) {
408 408
 			$this->miscService->log(
409
-				'fail to send frame to ' . $link->getAddress() . ' - ' . $e->getMessage()
409
+				'fail to send frame to '.$link->getAddress().' - '.$e->getMessage()
410 410
 			);
411 411
 		}
412 412
 	}
Please login to merge, or discard this patch.