rugk / threema-msgapi-sdk-php

source/Threema/Console/Command/Base.php

Doc Comments +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
 	 * @param string $subject
 	 * @param string[] $requiredArguments
 	 * @param string $description
-	 * @param array $optionalArguments
+	 * @param string[] $optionalArguments
 	 */
 	public function __construct($subject, array $requiredArguments, $description, array $optionalArguments = array()) {
 		$this->subject = $subject;
@@ -115,7 +115,7 @@ 
 	 * return a valid file path
 	 *
 	 * @param string $pos
-	 * @return null|string
+	 * @return string
 	 * @throws \Threema\Core\Exception
 	 */
 	public function getArgumentFile($pos) {

Spacing +18 added lines, -18 removed lines

@@ -68,7 +68,7 @@ 
 	 * @return int
 	 */
 	public function getAllArgumentsCount() {
-		return $this->getRequiredArgumentCount() + (null !== $this->optionalArguments ? count($this->optionalArguments) : 0);
+		return $this->getRequiredArgumentCount()+(null !== $this->optionalArguments ? count($this->optionalArguments) : 0);
 	}
 
 	/**
@@ -76,15 +76,15 @@ 
 	 * @return int
 	 * @throws \Threema\Core\Exception
 	 */
-	private function getIndexByPos($pos){
+	private function getIndexByPos($pos) {
 		$i = array_search($pos, $this->requiredArguments);
-		if(false === $i) {
+		if (false === $i) {
 			$i = array_search($pos, $this->optionalArguments);
-			if(false !== $i) {
+			if (false !== $i) {
 				$i += count($this->requiredArguments);
 			}
 		}
-		if(false === $i) {
+		if (false === $i) {
 			throw new Exception('argument '.$pos.' not found');
 		}
 
@@ -100,8 +100,8 @@ 
 		$i = $this->getIndexByPos($pos);
 		$required = $i < count($this->requiredArguments);
 
-		if(false === array_key_exists($i, $this->arguments)) {
-			if(true === $required) {
+		if (false === array_key_exists($i, $this->arguments)) {
+			if (true === $required) {
 				throw new Exception('no argument at position '.$i.' ('.$pos.')');
 			}
 			else {
@@ -121,7 +121,7 @@ 
 	public function getArgumentFile($pos) {
 		$i = $this->getIndexByPos($pos);
 
-		if(false === is_file($this->arguments[$i])) {
+		if (false === is_file($this->arguments[$i])) {
 			throw new Exception('argument '.$i.' is not a file');
 		}
 
@@ -134,7 +134,7 @@ 
 	 */
 	public function getArgumentPrivateKey($pos) {
 		$content = Common::getPrivateKey($this->getArgumentStringOrFileContent($pos));
-		if(null !== $content) {
+		if (null !== $content) {
 			return hex2bin($content);
 		}
 		return null;
@@ -146,7 +146,7 @@ 
 	 */
 	public function getArgumentPublicKey($pos) {
 		$content = Common::getPublicKey($this->getArgumentStringOrFileContent($pos));
-		if(null !== $content) {
+		if (null !== $content) {
 			return hex2bin($content);
 		}
 		return null;
@@ -158,7 +158,7 @@ 
 	 */
 	private function getArgumentStringOrFileContent($pos) {
 		$content = $this->getArgument($pos);
-		if(file_exists($content)) {
+		if (file_exists($content)) {
 			$content = trim(file_get_contents($content));
 		}
 		return $content;
@@ -170,7 +170,7 @@ 
 	 */
 	public function getArgumentThreemaId($pos) {
 		$content = $this->getArgument($pos);
-		if(null !== $content && strlen($content) == 8) {
+		if (null !== $content && strlen($content) == 8) {
 			return strtoupper($content);
 		}
 		return null;
@@ -180,12 +180,12 @@ 
 	 * @return string
 	 */
 	protected function readStdIn() {
-		$f = fopen( 'php://stdin', 'r' );
+		$f = fopen('php://stdin', 'r');
 
 		$lines = array();
-		while( $line = trim(fgets($f)) ) {
+		while ($line = trim(fgets($f))) {
 
-			if(strlen($line) == 0 || $line == "\n") {
+			if (strlen($line) == 0 || $line == "\n") {
 				continue;
 			}
 
@@ -204,7 +204,7 @@ 
 
 		$argCount = null !== $this->arguments ? count($this->arguments) : 0;
 
-		if($argCount < count($this->requiredArguments)) {
+		if ($argCount < count($this->requiredArguments)) {
 			throw new Exception('invalid count of arguments');
 		}
 		$this->doRun();
@@ -217,10 +217,10 @@ 
 	final public function help($shellColors = true) {
 		$args = array_merge($this->requiredArguments, $this->optionalArguments);
 
-		if(count($args) > 0) {
+		if (count($args) > 0) {
 			$colorPrefix = '';
 			$colorPostfix = '';
-			if(true === $shellColors) {
+			if (true === $shellColors) {
 				$colorPrefix = "\033[0;36m\033[40m";
 				$colorPostfix = "\033[0m";
 			}

Braces +1 added lines, -2 removed lines

@@ -103,8 +103,7 @@
 		if(false === array_key_exists($i, $this->arguments)) {
 			if(true === $required) {
 				throw new Exception('no argument at position '.$i.' ('.$pos.')');
-			}
-			else {
+			} else {
 				return null;
 			}
 		}

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * @author Threema GmbH
- * @copyright Copyright (c) 2015-2016 Threema GmbH
- */
+	 * @author Threema GmbH
+	 * @copyright Copyright (c) 2015-2016 Threema GmbH
+	 */
 
 
 namespace Threema\Core;

source/Threema/Core/AssocArray.php

Doc Comments +2 added lines, -2 removed lines

@@ -36,8 +36,8 @@
 	}
 
 	/**
-	 * @param $string
-	 * @param array|null $requiredKeys
+	 * @param string $string
+	 * @param string[] $requiredKeys
 	 * @return AssocArray
 	 * @throws Exception
 	 */

Spacing +5 added lines, -5 removed lines

@@ -28,7 +28,7 @@ 
 	 * @return mixed|null return the key value or the default value
 	 */
 	public function getValue($key, $defaultValue = null) {
-		if(false === array_key_exists($key, $this->data)) {
+		if (false === array_key_exists($key, $this->data)) {
 			return $defaultValue;
 		}
 
@@ -43,15 +43,15 @@ 
 	 */
 	public static final function byJsonString($string, array $requiredKeys = null) {
 		$v = json_decode($string, true);
-		if(null === $v || false === $v) {
+		if (null === $v || false === $v) {
 			throw new Exception('invalid json string');
 		}
 
 		//validate first
-		if(null !== $requiredKeys) {
+		if (null !== $requiredKeys) {
 			//validate array first
-			foreach($requiredKeys as $requiredKey) {
-				if(false === array($v)) {
+			foreach ($requiredKeys as $requiredKey) {
+				if (false === array($v)) {
 					throw new Exception('required key '.$requiredKey.' failed');
 				}
 			}

source/Threema/MsgApi/Commands/DownloadFile.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 	/**
 	 * @param int $httpCode
 	 * @param object $res
-	 * @return UploadFileResult
+	 * @return DownloadFileResult
 	 */
 	function parseResult($httpCode, $res){
 		return new DownloadFileResult($httpCode, $res);

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 	 * @param object $res
 	 * @return UploadFileResult
 	 */
-	function parseResult($httpCode, $res){
+	function parseResult($httpCode, $res) {
 		return new UploadFileResult($httpCode, $res);
 	}
 }

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * @author Threema GmbH
- * @copyright Copyright (c) 2015-2016 Threema GmbH
- */
+	 * @author Threema GmbH
+	 * @copyright Copyright (c) 2015-2016 Threema GmbH
+	 */
 
 
 namespace Threema\Core;

source/Threema/MsgApi/Helpers/ReceiveMessageResult.php

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
 	}
 
 	/**
-	 * @return \string[]
+	 * @return string[]
 	 */
 	public function getErrors() {
 		return $this->errors;
@@ -87,7 +87,7 @@ 
 	}
 
 	/**
-	 * @return \string[]
+	 * @return string[]
 	 */
 	public function getFiles() {
 		return $this->files;

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * @author Threema GmbH
- * @copyright Copyright (c) 2015-2016 Threema GmbH
- */
+	 * @author Threema GmbH
+	 * @copyright Copyright (c) 2015-2016 Threema GmbH
+	 */
 
 
 namespace Threema\Core;

source/Threema/MsgApi/Messages/DeliveryReceipt.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 
 	/**
 	 * Get the acknowledged message ids
-	 * @return array
+	 * @return string[]
 	 */
 	public function getAckedMessageIds() {
 		return $this->ackedMessageIds;

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
 	 * @return string
 	 */
 	public function getReceiptTypeName() {
-		if(true === array_key_exists($this->receiptType, self::$receiptTypesToNames)) {
+		if (true === array_key_exists($this->receiptType, self::$receiptTypesToNames)) {
 			return self::$receiptTypesToNames[$this->receiptType];
 		}
 		return null;
@@ -77,7 +77,7 @@ 
 	 * @return string
 	 */
 	function __toString() {
-		$str = "Delivery receipt (" . $this->getReceiptTypeName() . "): ";
+		$str = "Delivery receipt (".$this->getReceiptTypeName()."): ";
 		$hexMessageIds = array();
 		foreach ($this->ackedMessageIds as $messageId) {
 			$hexMessageIds[] = bin2hex($messageId);

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * @author Threema GmbH
- * @copyright Copyright (c) 2015-2016 Threema GmbH
- */
+	 * @author Threema GmbH
+	 * @copyright Copyright (c) 2015-2016 Threema GmbH
+	 */
 
 
 namespace Threema\Core;

samples/testFetchPublicKey.php

Spacing +3 added lines, -3 removed lines

@@ -22,9 +22,9 @@
 
 
 $result = $connector->fetchPublicKey('ECHOECHO');
-if($result->isSuccess()) {
-	echo 'public key '.$result->getPublicKey() . "\n";
+if ($result->isSuccess()) {
+	echo 'public key '.$result->getPublicKey()."\n";
 }
 else {
-	echo 'error '.$result->getErrorMessage() . "\n";
+	echo 'error '.$result->getErrorMessage()."\n";
 }

Braces +1 added lines, -2 removed lines

@@ -29,7 +29,6 @@
 $result = $connector->sendSimple($receiver, "This is a Test Message");
 if($result->isSuccess()) {
 	echo 'Message ID: '.$result->getMessageId() . "\n";
-}
-else {
+} else {
 	echo 'Error: '.$result->getErrorMessage() . "\n";
 }

samples/testKeyLookup.php

Spacing +3 added lines, -3 removed lines

@@ -21,9 +21,9 @@
 $connector = new Connection($settings, $publicKeyStore);
 
 $result = $connector->keyLookupByPhoneNumber('123456789');
-if($result->isSuccess()) {
-	echo 'Threema ID found: '.$result->getId() . "\n";
+if ($result->isSuccess()) {
+	echo 'Threema ID found: '.$result->getId()."\n";
 }
 else {
-	echo 'Error: '.$result->getErrorMessage() . "\n";
+	echo 'Error: '.$result->getErrorMessage()."\n";
 }

Braces +1 added lines, -2 removed lines

@@ -29,7 +29,6 @@
 $result = $connector->sendSimple($receiver, "This is a Test Message");
 if($result->isSuccess()) {
 	echo 'Message ID: '.$result->getMessageId() . "\n";
-}
-else {
+} else {
 	echo 'Error: '.$result->getErrorMessage() . "\n";
 }

samples/testSendE2EFile.php

Spacing +4 added lines, -4 removed lines

@@ -23,12 +23,12 @@
 $senderPrivateKey = "MY_PUBLIC_KEY_IN_BIN";
 $filePath = "/path/to/my/file.pdf";
 
-$e2eHelper = new \Threema\MsgApi\Helpers\E2EHelper($senderPrivateKey,$connector);
+$e2eHelper = new \Threema\MsgApi\Helpers\E2EHelper($senderPrivateKey, $connector);
 $result = $e2eHelper->sendFileMessage("TEST1234", $filePath);
 
-if(true === $result->isSuccess()) {
-	echo 'File Message ID: '.$result->getMessageId() . "\n";
+if (true === $result->isSuccess()) {
+	echo 'File Message ID: '.$result->getMessageId()."\n";
 }
 else {
-	echo 'Error: '.$result->getErrorMessage() . "\n";
+	echo 'Error: '.$result->getErrorMessage()."\n";
 }

Braces +1 added lines, -2 removed lines

@@ -29,7 +29,6 @@
 $result = $connector->sendSimple($receiver, "This is a Test Message");
 if($result->isSuccess()) {
 	echo 'Message ID: '.$result->getMessageId() . "\n";
-}
-else {
+} else {
 	echo 'Error: '.$result->getErrorMessage() . "\n";
 }

samples/testSendE2EText.php

Spacing +4 added lines, -4 removed lines

@@ -22,12 +22,12 @@
 
 $senderPrivateKey = "MY_PUBLIC_KEY_IN_BIN";
 
-$e2eHelper = new \Threema\MsgApi\Helpers\E2EHelper($senderPrivateKey,$connector);
+$e2eHelper = new \Threema\MsgApi\Helpers\E2EHelper($senderPrivateKey, $connector);
 $result = $e2eHelper->sendTextMessage("TEST1234", "This is an end-to-end encrypted message");
 
-if(true === $result->isSuccess()) {
-	echo 'Message ID: '.$result->getMessageId() . "\n";
+if (true === $result->isSuccess()) {
+	echo 'Message ID: '.$result->getMessageId()."\n";
 }
 else {
-	echo 'Error: '.$result->getErrorMessage() . "\n";
+	echo 'Error: '.$result->getErrorMessage()."\n";
 }

Braces +1 added lines, -2 removed lines

@@ -29,7 +29,6 @@
 $result = $connector->sendSimple($receiver, "This is a Test Message");
 if($result->isSuccess()) {
 	echo 'Message ID: '.$result->getMessageId() . "\n";
-}
-else {
+} else {
 	echo 'Error: '.$result->getErrorMessage() . "\n";
 }