Passed
Push — master ( afb86e...0d3f9b )
by Joas
11:36 queued 10s
created
lib/private/Comments/Comment.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	 * 						the comments database scheme
53 53
 	 */
54 54
 	public function __construct(array $data = null) {
55
-		if(is_array($data)) {
55
+		if (is_array($data)) {
56 56
 			$this->fromArray($data);
57 57
 		}
58 58
 	}
@@ -85,12 +85,12 @@  discard block
 block discarded – undo
85 85
 	 * @since 9.0.0
86 86
 	 */
87 87
 	public function setId($id) {
88
-		if(!is_string($id)) {
88
+		if (!is_string($id)) {
89 89
 			throw new \InvalidArgumentException('String expected.');
90 90
 		}
91 91
 
92 92
 		$id = trim($id);
93
-		if($this->data['id'] === '' || ($this->data['id'] !== '' && $id === '')) {
93
+		if ($this->data['id'] === '' || ($this->data['id'] !== '' && $id === '')) {
94 94
 			$this->data['id'] = $id;
95 95
 			return $this;
96 96
 		}
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 	 * @since 9.0.0
117 117
 	 */
118 118
 	public function setParentId($parentId) {
119
-		if(!is_string($parentId)) {
119
+		if (!is_string($parentId)) {
120 120
 			throw new \InvalidArgumentException('String expected.');
121 121
 		}
122 122
 		$this->data['parentId'] = trim($parentId);
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
 	 * @since 9.0.0
143 143
 	 */
144 144
 	public function setTopmostParentId($id) {
145
-		if(!is_string($id)) {
145
+		if (!is_string($id)) {
146 146
 			throw new \InvalidArgumentException('String expected.');
147 147
 		}
148 148
 		$this->data['topmostParentId'] = trim($id);
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
 	 * @since 9.0.0
168 168
 	 */
169 169
 	public function setChildrenCount($count) {
170
-		if(!is_int($count)) {
170
+		if (!is_int($count)) {
171 171
 			throw new \InvalidArgumentException('Integer expected.');
172 172
 		}
173 173
 		$this->data['childrenCount'] = $count;
@@ -194,12 +194,12 @@  discard block
 block discarded – undo
194 194
 	 * @since 9.0.0
195 195
 	 */
196 196
 	public function setMessage($message, $maxLength = self::MAX_MESSAGE_LENGTH) {
197
-		if(!is_string($message)) {
197
+		if (!is_string($message)) {
198 198
 			throw new \InvalidArgumentException('String expected.');
199 199
 		}
200 200
 		$message = trim($message);
201 201
 		if ($maxLength && mb_strlen($message, 'UTF-8') > $maxLength) {
202
-			throw new MessageTooLongException('Comment message must not exceed ' . $maxLength. ' characters');
202
+			throw new MessageTooLongException('Comment message must not exceed '.$maxLength.' characters');
203 203
 		}
204 204
 		$this->data['message'] = $message;
205 205
 		return $this;
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 	 */
228 228
 	public function getMentions() {
229 229
 		$ok = preg_match_all("/\B(?<![^a-z0-9_\-@\.\'\s])@(\"[a-z0-9_\-@\.\' ]+\"|[a-z0-9_\-@\.\']+)/i", $this->getMessage(), $mentions);
230
-		if(!$ok || !isset($mentions[0]) || !is_array($mentions[0])) {
230
+		if (!$ok || !isset($mentions[0]) || !is_array($mentions[0])) {
231 231
 			return [];
232 232
 		}
233 233
 		$uids = array_unique($mentions[0]);
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
 	 * @since 9.0.0
257 257
 	 */
258 258
 	public function setVerb($verb) {
259
-		if(!is_string($verb) || !trim($verb)) {
259
+		if (!is_string($verb) || !trim($verb)) {
260 260
 			throw new \InvalidArgumentException('Non-empty String expected.');
261 261
 		}
262 262
 		$this->data['verb'] = trim($verb);
@@ -292,9 +292,9 @@  discard block
 block discarded – undo
292 292
 	 * @since 9.0.0
293 293
 	 */
294 294
 	public function setActor($actorType, $actorId) {
295
-		if(
295
+		if (
296 296
 		       !is_string($actorType) || !trim($actorType)
297
-		    || !is_string($actorId)   || !trim($actorId)
297
+		    || !is_string($actorId) || !trim($actorId)
298 298
 		) {
299 299
 			throw new \InvalidArgumentException('String expected.');
300 300
 		}
@@ -378,9 +378,9 @@  discard block
 block discarded – undo
378 378
 	 * @since 9.0.0
379 379
 	 */
380 380
 	public function setObject($objectType, $objectId) {
381
-		if(
381
+		if (
382 382
 		       !is_string($objectType) || !trim($objectType)
383
-		    || !is_string($objectId)   || !trim($objectId)
383
+		    || !is_string($objectId) || !trim($objectId)
384 384
 		) {
385 385
 			throw new \InvalidArgumentException('String expected.');
386 386
 		}
@@ -397,20 +397,20 @@  discard block
 block discarded – undo
397 397
 	 * @return IComment
398 398
 	 */
399 399
 	protected function fromArray($data) {
400
-		foreach(array_keys($data) as $key) {
400
+		foreach (array_keys($data) as $key) {
401 401
 			// translate DB keys to internal setter names
402
-			$setter = 'set' . implode('', array_map('ucfirst', explode('_', $key)));
402
+			$setter = 'set'.implode('', array_map('ucfirst', explode('_', $key)));
403 403
 			$setter = str_replace('Timestamp', 'DateTime', $setter);
404 404
 
405
-			if(method_exists($this, $setter)) {
405
+			if (method_exists($this, $setter)) {
406 406
 				$this->$setter($data[$key]);
407 407
 			}
408 408
 		}
409 409
 
410
-		foreach(['actor', 'object'] as $role) {
411
-			if(isset($data[$role . '_type']) && isset($data[$role . '_id'])) {
412
-				$setter = 'set' . ucfirst($role);
413
-				$this->$setter($data[$role . '_type'], $data[$role . '_id']);
410
+		foreach (['actor', 'object'] as $role) {
411
+			if (isset($data[$role.'_type']) && isset($data[$role.'_id'])) {
412
+				$setter = 'set'.ucfirst($role);
413
+				$this->$setter($data[$role.'_type'], $data[$role.'_id']);
414 414
 			}
415 415
 		}
416 416
 
Please login to merge, or discard this patch.