owncloud / chat

app/container.php

Doc Comments +3 added lines

@@ -314,6 +314,9 @@
 
 	}
 
+	/**
+	 * @param string $param
+	 */
 	public function query($param){
 		return $this->getContainer()->query($param);
 	}

Unused Use Statements -1 removed lines

@@ -36,7 +36,6 @@
 use OCA\Chat\OCH\OCH;
 use OCA\Chat\XMPP\XMPP;
 use OCA\Chat\BackendManager;
-use OCA\Chat\IBackend;
 use OCA\Chat\Middleware\ErrorMiddleware;
 
 /**

lib/och/db/attachmentmapper.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 namespace OCA\Chat\OCH\Db;
 
 use OCP\AppFramework\Db\DoesNotExistException;
-use OCA\Chat\OCH\Commands\AttachFile;
 use \OCP\AppFramework\Db\Mapper;
 use \OCP\IDb;
 

lib/och/db/usermapper.php

Doc Comments +6 added lines

@@ -43,6 +43,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $user
+	 */
 	public function findByUser($user){
 		$sql = <<<SQL
 			SELECT
@@ -56,6 +59,9 @@ 
 		return $result;
 	}
 	
+	 /**
+	  * @param string $user
+	  */
 	 public function findConvsIdByUser($user){
 		$sql = 'SELECT conversation_id FROM `' . $this->getTableName() . '` ' .
 				'WHERE `user` = ? ';

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 
 namespace OCA\Chat\OCH\Db;
 
-use Doctrine\DBAL\Sharding\SQLAzure\SQLAzureFederationsSynchronizer;
 use \OCP\AppFramework\Db\Mapper;
 use \OCP\IDb;
 use \OCP\AppFramework\Db\Entity;

tests/unit/lib/och/commands/Join.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use OCA\Chat\OCH\Db\InitConv;
 use OCA\Chat\OCH\Db\PushMessage;
 use \OCA\Chat\OCH\Db\UserOnline;
-use OCA\Chat\Db\DBException;
 use OCA\Chat\OCH\Exceptions\RequestDataInvalid;
 use OCA\Chat\OCH\Db\User;
 

tests/unit/lib/och/commands/SendChatMsg.php

Unused Use Statements -3 removed lines

@@ -14,11 +14,8 @@
 use OCA\Chat\Core\API;
 use OCA\Chat\OCH\Commands\SendChatMsg;
 use OCA\Chat\App\Chat;
-use OCA\Chat\Db\DBException;
 use OCA\Chat\OCH\Exceptions\RequestDataInvalid;
-use OCA\Chat\OCH\Db\UserOnline;
 use OCA\Chat\OCH\Db\User;
-use OCA\Chat\OCH\Db\PushMessage;
 
 // DONE
 class SendChatMsgTest extends \PHPUnit_Framework_TestCase {

tests/unit/lib/och/commands/StartConv.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 
 
 use OCA\Chat\Core\API;
-use OCA\Chat\OCH\Commands\StartConv;
 use OCA\Chat\App\Chat;
 use \OCA\Chat\OCH\Db\UserOnline;
 

tests/unit/lib/och/commands/SyncOnline.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use OCA\Chat\OCH\Commands\SyncOnline;
 use OCA\Chat\App\Chat;
 use OCA\Chat\Db\DBException;
-use OCA\Chat\OCH\Exceptions\RequestDataInvalid;
 use OCA\Chat\OCH\Db\UserOnline;
 
 // DONE

utility/controllertestutility.php

Doc Comments +3 added lines, -2 removed lines

@@ -25,9 +25,10 @@
 
 	/**
 	 * Checks if a controllermethod has the expected annotations
-	 * @param Controller/string $controller name or instance of the controller
-	 * @param array $expected an array containing the expected annotations
+	 * @param \OCA\Chat\Controller\OCH\ApiController $controller name or instance of the controller
+	 * @param string[] $expected an array containing the expected annotations
 	 * @param array $valid if you define your own annotations, pass them here
+	 * @param string $method
 	 */
 	protected function assertAnnotations($controller, $method, array $expected,
 										 array $valid=array()){