janis91 / ocr

lib/Db/OcrStatus.php

Doc Comments +4 added lines, -4 removed lines

@@ -73,13 +73,13 @@
 	/**
 	 * OcrStatus constructor.
 	 *
-	 * @param null $status
+	 * @param string $status
 	 * @param null $fileId
-	 * @param null $newName
+	 * @param string $newName
 	 * @param null $tempFile
-	 * @param null $type
+	 * @param string $type
 	 * @param null $userId
-	 * @param null $errorDisplayed
+	 * @param boolean $errorDisplayed
 	 */
 	public function __construct($status = null, $fileId = null, $newName = null, $tempFile = null, $type = null, $userId = null, $errorDisplayed = null) {
 		$this->setStatus($status);

Indentation +20 added lines, -20 removed lines

@@ -91,24 +91,24 @@
 		$this->setErrorDisplayed($errorDisplayed);
 	}
 
-    /**
-     * Specify data which should be serialized to JSON
-     * @link http://php.net/manual/en/jsonserializable.jsonserialize.php
-     * @return mixed data which can be serialized by <b>json_encode</b>,
-     * which is a value of any type other than a resource.
-     * @since 5.4.0
-     */
-    function jsonSerialize()
-    {
-        return [
-            'id' => $this->id,
-            'status' => $this->status,
-            'fileId' => $this->fileId,
-            'newName' => $this->newName,
-            'tempFile' => $this->tempFile,
-            'type' => $this->type,
-            'userId' => $this->userId,
-            'errorDisplayed' => $this->errorDisplayed
-        ];
-    }
+	/**
+	 * Specify data which should be serialized to JSON
+	 * @link http://php.net/manual/en/jsonserializable.jsonserialize.php
+	 * @return mixed data which can be serialized by <b>json_encode</b>,
+	 * which is a value of any type other than a resource.
+	 * @since 5.4.0
+	 */
+	function jsonSerialize()
+	{
+		return [
+			'id' => $this->id,
+			'status' => $this->status,
+			'fileId' => $this->fileId,
+			'newName' => $this->newName,
+			'tempFile' => $this->tempFile,
+			'type' => $this->type,
+			'userId' => $this->userId,
+			'errorDisplayed' => $this->errorDisplayed
+		];
+	}
 }
\ No newline at end of file

lib/Service/OcrService.php

Doc Comments +2 added lines, -1 removed lines

@@ -240,6 +240,7 @@ 
 	 * The PersonalSettingsController will have the opportunity to delete ocr status objects.
 	 *
 	 * @param $statusId
+	 * @param string $userId
 	 * @return OcrStatus
 	 */
 	public function deleteStatus($statusId, $userId) {
@@ -269,7 +270,7 @@ 
 	/**
 	 * Gets all status objects for a specific user in order to list them on the personal settings page.
 	 *
-	 * @param $userId
+	 * @param string $userId
 	 * @return array
 	 */
 	public function getAllForPersonal($userId) {

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use OCA\Ocr\Db\OcrStatus;
 use OCA\Ocr\Db\OcrStatusMapper;
 use OCP\AppFramework\Db\DoesNotExistException;
-use OCP\AppFramework\Db\Entity;
 use OCP\Files;
 use OCP\Files\FileInfo;
 use OCP\IConfig;

Indentation +19 added lines, -19 removed lines

@@ -200,7 +200,7 @@ 
 		try {
 			// TODO: release lock
 
-            // returns user specific processed files
+			// returns user specific processed files
 			$processed = $this->handleProcessed();
 
 			$pending = $this->statusMapper->findAllPending($this->userId);
@@ -251,18 +251,18 @@ 
 				$status = $this->statusMapper->delete($status);
 			}
 			$status->setNewName($this->removeFileExtension($status));
-            $status->setFileId(null);
-            $status->setTempFile(null);
-            $status->setType(null);
-            $status->setErrorDisplayed(null);
+			$status->setFileId(null);
+			$status->setTempFile(null);
+			$status->setType(null);
+			$status->setErrorDisplayed(null);
 			return $status;
 		} catch (Exception $e) {
-		    if ($e instanceof DoesNotExistException) {
-		        $ex = new NotFoundException($this->l10n->t('Cannot delete. Wrong id.'));
-                $this->handleException($ex);
-            } else {
-		        $this->handleException($e);
-		    }
+			if ($e instanceof DoesNotExistException) {
+				$ex = new NotFoundException($this->l10n->t('Cannot delete. Wrong id.'));
+				$this->handleException($ex);
+			} else {
+				$this->handleException($e);
+			}
 		}
 	}
 
@@ -273,9 +273,9 @@ 
 	 * @return array
 	 */
 	public function getAllForPersonal($userId) {
-	    try {
-            $status = $this->statusMapper->findAll($userId);
-            $statusNew = array();
+		try {
+			$status = $this->statusMapper->findAll($userId);
+			$statusNew = array();
 			for ($x = 0; $x < count($status); $x++) {
 				$newName = $this->removeFileExtension($status[$x]);
 				$status[$x]->setNewName($newName);
@@ -285,11 +285,11 @@ 
 				$status[$x]->setErrorDisplayed(null);
 				array_push($statusNew, $status[$x]);
 			}
-            return $statusNew;
-        } catch (Exception $e) {
-	        $this->handleException($e);
-        }
-    }
+			return $statusNew;
+		} catch (Exception $e) {
+			$this->handleException($e);
+		}
+	}
 
 	/**
 	 * Finishes all Processed files by copying them to the right path and deleteing the temp files.

lib/Controller/PersonalSettingsController.php

Indentation +6 added lines, -6 removed lines

@@ -46,11 +46,11 @@ 
 		$this->service = $service;
 	}
 
-    /**
-     * @NoAdminRequired
-     *
-     * @return Template
-     */
+	/**
+	 * @NoAdminRequired
+	 *
+	 * @return Template
+	 */
 	public function displayPanel() {
 		$tmpl = new Template('ocr', 'settings-personal');
 		return $tmpl;
@@ -63,7 +63,7 @@ 
 	 */
 	public function getAll() {
 		return $this->handleNotFound(function () {
-            return $this->service->getAllForPersonal($this->userId);
+			return $this->service->getAllForPersonal($this->userId);
 		});
 	}
 

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
 	 * @return \OCP\AppFramework\Http\DataResponse
 	 */
 	public function getAll() {
-		return $this->handleNotFound(function () {
+		return $this->handleNotFound(function() {
             return $this->service->getAllForPersonal($this->userId);
 		});
 	}
@@ -73,7 +73,7 @@ 
 	 * @return \OCP\AppFramework\Http\DataResponse
 	 */
 	public function deleteStatus($id) {
-		return $this->handleNotFound(function () use ($id) {
+		return $this->handleNotFound(function() use ($id) {
 			return $this->service->deleteStatus($id, $this->userId);
 		});
 	}

lib/Db/OcrStatusMapper.php

Indentation +10 added lines, -10 removed lines

@@ -41,16 +41,16 @@
 		return $this->findEntity($sql, [$id]);
 	}
 
-    /**
-     * Finds all user specific status entities
-     *
-     * @param $userId
-     * @return OcrStatus[]
-     */
-    public function findAll($userId) {
-        $sql = 'SELECT * FROM *PREFIX*ocr_status WHERE user_id = ?';
-        return $this->findEntities($sql, [$userId]);
-    }
+	/**
+	 * Finds all user specific status entities
+	 *
+	 * @param $userId
+	 * @return OcrStatus[]
+	 */
+	public function findAll($userId) {
+		$sql = 'SELECT * FROM *PREFIX*ocr_status WHERE user_id = ?';
+		return $this->findEntities($sql, [$userId]);
+	}
 
 	/**
 	 * Finds all status PROCESSED entities for a given userid

lib/Service/QueueService.php

Indentation +9 added lines, -9 removed lines

@@ -72,9 +72,9 @@ 
 	}
 
 	/**
-     * TODO: in the future this function could be used to give an admin information
+	 * TODO: in the future this function could be used to give an admin information
 	 * Counts the messages in the message queue.
-     *
+	 *
 	 * @return mixed
 	 */
 	public function countMessages() {
@@ -87,13 +87,13 @@ 
 		}
 	}
 
-    /**
-     * TODO: in the future this function could be used to give an admin information
-     * Counts the at this point processed files
-     *
-     * @return mixed
-     */
-    public function countActiveProcesses() {
+	/**
+	 * TODO: in the future this function could be used to give an admin information
+	 * Counts the at this point processed files
+	 *
+	 * @return mixed
+	 */
+	public function countActiveProcesses() {
 		try {
 			$stats = msg_stat_queue($this->statusqueue);
 			$this->logger->debug('Current active processing count: ' . json_encode($stats['msg_qnum']), ['app' => 'ocr']);

lib/Migration/DropOldTable.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 	 */
 	public function run(IOutput $output) {
 		$output->startProgress(1);
-		if ($this->connection->tableExists('ocr_status')){
+		if ($this->connection->tableExists('ocr_status')) {
 			$this->connection->dropTable('ocr_status');
 		}
 		$output->advance(1, "Drop old database table: ocr_status");

lib/AppInfo/Application.php

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 	 * Registers the Personal Settings Page for deletion of status objects and such things.
 	 * @codeCoverageIgnore
 	 */
-	public function registerPersonal(){
+	public function registerPersonal() {
 		\OCP\App::registerPersonal($this->getContainer()->getAppName(), 'personal');
 	}