Merge pull request #670 from nextcloud/dependabot/composer/nextcloud/…

…coding-standard-1.2.3

lib/DatabaseStatistics.php

@@ -130,7 +130,7 @@ protected function databaseSize(): string {
 				$result->closeCursor();
 				break;
 		}
-		return ($database_size !== false) ? (string) $database_size : 'N/A';
+		return ($database_size !== false) ? (string)$database_size : 'N/A';
 	}
 
 	/**

lib/OperatingSystems/FreeBSD.php

@@ -107,7 +107,7 @@ public function getNetworkInfo(): array {
 			$netstat = $this->executeCommand('netstat -rn');
 			preg_match_all("/(?<=^default)\s*[0-9a-fA-f\.:]+/m", $netstat, $gw);
 			if (count($gw[0]) > 0) {
-				$result['gateway'] = implode(", ", array_map("trim", $gw[0]));
+				$result['gateway'] = implode(', ', array_map('trim', $gw[0]));
 			}
 		} catch (RuntimeException) {
 			// okay
@@ -148,7 +148,7 @@ public function getNetworkInterfaces(): array {
 				continue;
 			}
 
-			preg_match("/(?<=ether ).*/m", $details, $mac);
+			preg_match('/(?<=ether ).*/m', $details, $mac);
 			if (isset($mac[0])) {
 				$netInterface->setMAC($mac[0]);
 			}

lib/Settings/AdminSection.php

@@ -40,8 +40,8 @@ public function getName(): string {
 
 	/**
 	 * @return int whether the form should be rather on the top or bottom of
-	 * the settings navigation. The sections are arranged in ascending order of
-	 * the priority values. It is required to return a value between 0 and 99.
+	 *             the settings navigation. The sections are arranged in ascending order of
+	 *             the priority values. It is required to return a value between 0 and 99.
 	 *
 	 * keep the server setting at the top, right after "overview" and "basic settings"
 	 */

lib/Settings/AdminSettings.php

@@ -90,8 +90,8 @@ public function getSection(): string {
 
 	/**
 	 * @return int whether the form should be rather on the top or bottom of
-	 * the admin section. The forms are arranged in ascending order of the
-	 * priority values. It is required to return a value between 0 and 100.
+	 *             the admin section. The forms are arranged in ascending order of the
+	 *             priority values. It is required to return a value between 0 and 100.
 	 *
 	 * keep the server setting at the top, right after "server settings"
 	 */

lib/ShareStatistics.php

@@ -62,7 +62,7 @@ protected function countEntries(string $tableName): int {
 		$row = $result->fetch();
 		$result->closeCursor();
 
-		return (int) $row['num_entries'];
+		return (int)$row['num_entries'];
 	}
 
 	/**
@@ -84,6 +84,6 @@ protected function countShares(int $type, bool $noPassword = false): int {
 		$row = $result->fetch();
 		$result->closeCursor();
 
-		return (int) $row['num_entries'];
+		return (int)$row['num_entries'];
 	}
 }

lib/StorageStatistics.php

@@ -44,7 +44,7 @@ protected function countUserEntries(): int {
 		$result = $query->executeQuery();
 		$row = $result->fetch();
 		$result->closeCursor();
-		return (int) $row['num_entries'];
+		return (int)$row['num_entries'];
 	}
 
 	protected function getCountOf(string $table): int {
@@ -92,6 +92,6 @@ protected function countStorages(string $type): int {
 		$result = $query->executeQuery();
 		$row = $result->fetch();
 		$result->closeCursor();
-		return (int) $row['num_entries'];
+		return (int)$row['num_entries'];
 	}
 }

lib/SystemStatistics.php

@@ -76,7 +76,7 @@ public function getSystemStatistics(bool $skipApps = false, bool $skipUpdate = t
 	 */
 	protected function getServerUpdateInfo(): array {
 		$updateInfo = [
-			'lastupdatedat' => (int) $this->config->getAppValue('core', 'lastupdatedat'),
+			'lastupdatedat' => (int)$this->config->getAppValue('core', 'lastupdatedat'),
 			'available' => false,
 		];
 

tests/lib/ApiControllerTest.php

@@ -28,10 +28,10 @@ class ApiControllerTest extends \Test\TestCase {
 	/** @var Os|\PHPUnit\Framework\MockObject\MockObject */
 	private $os;
 
-	/** @var IRequest|\PHPUnit\Framework\MockObject\MockObject  */
+	/** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */
 	private $request;
 
-	/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject  */
+	/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */
 	private $config;
 
 	/** @var IGroupManager|\PHPUnit\Framework\MockObject\MockObject */
@@ -142,9 +142,9 @@ public function testAuthFailureWithToken() {
 			->willReturn('invalidtoken');
 
 		$this->config->expects($this->once())
-		   ->method('getAppValue')
-		   ->with('serverinfo', 'token', null)
-		   ->willReturn('megatoken');
+			->method('getAppValue')
+			->with('serverinfo', 'token', null)
+			->willReturn('megatoken');
 		$response = $this->getController(null)->info();
 
 		$this->assertEquals(['message' => 'Unauthorized'], $response->getData());
@@ -158,9 +158,9 @@ public function testAuthSuccessWithToken() {
 			->willReturn('megatoken');
 
 		$this->config->expects($this->once())
-		   ->method('getAppValue')
-		   ->with('serverinfo', 'token', null)
-		   ->willReturn('megatoken');
+			->method('getAppValue')
+			->with('serverinfo', 'token', null)
+			->willReturn('megatoken');
 		$response = $this->getController(null)->info();
 
 		$this->assertEquals(Http::STATUS_OK, $response->getStatus());
@@ -173,9 +173,9 @@ public function testIn() {
 			->willReturn('megatoken');
 
 		$this->config->expects($this->once())
-		   ->method('getAppValue')
-		   ->with('serverinfo', 'token', null)
-		   ->willReturn('megatoken');
+			->method('getAppValue')
+			->with('serverinfo', 'token', null)
+			->willReturn('megatoken');
 		$response = $this->getController(null)->info();
 
 		$this->assertEquals(Http::STATUS_OK, $response->getStatus());

tests/lib/SessionStatisticsTest.php

@@ -22,13 +22,13 @@
  * @package OCA\ServerInfo\Tests
  */
 class SessionStatisticsTest extends TestCase {
-	/** @var  ITimeFactory | \PHPUnit_Framework_MockObject_MockObject */
+	/** @var ITimeFactory | \PHPUnit_Framework_MockObject_MockObject */
 	private $timeFactory;
 
-	/** @var  IDBConnection */
+	/** @var IDBConnection */
 	private $connection;
 
-	/** @var  SessionStatistics */
+	/** @var SessionStatistics */
 	private $instance;
 
 	private $table = 'authtoken';