Completed
Pull Request — master (#189)
by Maxence
29s
created
lib/Service/FilesService.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
 		} catch (NotFoundException $e) {
233 233
 			return [];
234 234
 		} catch (Throwable $e) {
235
-			$this->log(2, 'Issue while retrieving rootFolder for ' . $userId);
235
+			$this->log(2, 'Issue while retrieving rootFolder for '.$userId);
236 236
 
237 237
 			return [];
238 238
 		}
@@ -482,7 +482,7 @@  discard block
 block discarded – undo
482 482
 		$tagIds = $this->systemTagObjectMapper->getTagIdsForObjects([$file->getId()], 'files');
483 483
 		if (array_key_exists($file->getId(), $tagIds)) {
484 484
 			$tags = array_values(
485
-				array_map(function (ISystemTag $tag): string {
485
+				array_map(function(ISystemTag $tag): string {
486 486
 					return $tag->getName();
487 487
 				}, $this->systemTagManager->getTagsByIds($tagIds[$file->getId()]))
488 488
 			);
@@ -500,7 +500,7 @@  discard block
 block discarded – undo
500 500
 				]
501 501
 			);
502 502
 		} else {
503
-			$this->log(2, 'stat() on File #' . $file->getId() . ' is not an array: ' . json_encode($stat));
503
+			$this->log(2, 'stat() on File #'.$file->getId().' is not an array: '.json_encode($stat));
504 504
 		}
505 505
 
506 506
 		return $document;
@@ -617,8 +617,8 @@  discard block
 block discarded – undo
617 617
 			$document->getIndex()
618 618
 					 ->setStatus(IIndex::INDEX_IGNORE);
619 619
 			$this->miscService->log(
620
-				'Exception while generateDocument: ' . $e->getMessage() . ' (' . get_class($e) . ') at '
621
-				. $e->getFile() . ' line ' . $e->getLine()
620
+				'Exception while generateDocument: '.$e->getMessage().' ('.get_class($e).') at '
621
+				. $e->getFile().' line '.$e->getLine()
622 622
 			);
623 623
 		}
624 624
 	}
@@ -821,7 +821,7 @@  discard block
 block discarded – undo
821 821
 
822 822
 		$part = [];
823 823
 		foreach ($comments as $comment) {
824
-			$part[] = '<' . $comment->getActorId() . '> ' . $comment->getMessage();
824
+			$part[] = '<'.$comment->getActorId().'> '.$comment->getMessage();
825 825
 		}
826 826
 
827 827
 		$document->addPart('comments', implode(" \n ", $part));
@@ -856,7 +856,7 @@  discard block
 block discarded – undo
856 856
 					(!is_string($path)) ? $path = '' : $path;
857 857
 			} catch (Throwable $e) {
858 858
 				$this->miscService->log(
859
-					'Issue while getting information on documentId:' . $document->getId(), 0
859
+					'Issue while getting information on documentId:'.$document->getId(), 0
860 860
 				);
861 861
 			}
862 862
 		}
@@ -907,7 +907,7 @@  discard block
 block discarded – undo
907 907
 		}
908 908
 
909 909
 		// 20220219 Parse .drawio file
910
-		if ($extension  === 'drawio') {
910
+		if ($extension === 'drawio') {
911 911
 			$parsed = self::MIMETYPE_TEXT;
912 912
 			throw new KnownFileMimeTypeException();
913 913
 		}
@@ -1054,7 +1054,7 @@  discard block
 block discarded – undo
1054 1054
 		}
1055 1055
 
1056 1056
 		// 20220219 Inflate drawio file
1057
-		if ( $file->getExtension() === 'drawio') {
1057
+		if ($file->getExtension() === 'drawio') {
1058 1058
 			$content = $file->getContent();
1059 1059
 
1060 1060
 			try {
@@ -1074,7 +1074,7 @@  discard block
 block discarded – undo
1074 1074
 
1075 1075
 				    // Construct XML
1076 1076
 				    $diagram_xml = simplexml_load_string($diagram_str);
1077
-				    $content = $content . ' ' . $this->readDrawioXmlValue($diagram_xml);
1077
+				    $content = $content.' '.$this->readDrawioXmlValue($diagram_xml);
1078 1078
 				}
1079 1079
 
1080 1080
 			} catch (\Throwable $t) {
@@ -1106,16 +1106,16 @@  discard block
 block discarded – undo
1106 1106
 	 */
1107 1107
 	private function readDrawioXmlValue(\SimpleXMLElement $element) {
1108 1108
 		$str = '';
1109
-		if( $element['value'] != null && trim(strval($element['value'])) !== '') {
1110
-			$str = $str . " " .  trim(strval($element['value']));
1109
+		if ($element['value'] != null && trim(strval($element['value'])) !== '') {
1110
+			$str = $str." ".trim(strval($element['value']));
1111 1111
 		}
1112
-		if( $element != null && trim(strval($element)) !== '') {
1113
-			$str = $str . " " . trim(strval($element));
1112
+		if ($element != null && trim(strval($element)) !== '') {
1113
+			$str = $str." ".trim(strval($element));
1114 1114
 		}
1115 1115
 
1116 1116
 		try {
1117 1117
 			foreach ($element->children() as $child) {
1118
-				$str = $str . " " . $this->readDrawioXmlValue($child);
1118
+				$str = $str." ".$this->readDrawioXmlValue($child);
1119 1119
 			}
1120 1120
 		} finally {
1121 1121
 		}
@@ -1369,7 +1369,7 @@  discard block
 block discarded – undo
1369 1369
 			$path = '';
1370 1370
 		}
1371 1371
 
1372
-		$result = (($entrySlash) ? '/' : '') . $path;
1372
+		$result = (($entrySlash) ? '/' : '').$path;
1373 1373
 		$this->debug(
1374 1374
 			'getPathFromRoot', [
1375 1375
 				'path' => $path,
Please login to merge, or discard this patch.