janis91 / ocr

lib/Db/OcrStatus.php

Doc Comments +5 added lines, -5 removed lines

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

lib/Service/OcrService.php

Doc Comments +3 added lines, -2 removed lines

@@ -252,6 +252,7 @@ 
 	 * The PersonalSettingsController will have the opportunity to delete ocr status objects.
 	 *
 	 * @param $statusId
+	 * @param string $userId
 	 * @return OcrStatus
 	 */
 	public function deleteStatus($statusId, $userId) {
@@ -281,7 +282,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) {
@@ -429,7 +430,7 @@ 
 
 	/**
 	 * @param File $fileInfo
-	 * @return bool
+	 * @return boolean|null
 	 */
 	private function checkSharedWithInitiator($fileInfo) {
 		try {

Spacing +6 added lines, -6 removed lines

@@ -169,7 +169,7 @@ 
 					$source = $fInfo->getPath();
 					if ($this->checkSharedWithInitiator($fInfo)) {
 						// Shared Item
-						$source = str_replace('home::','',$fInfo->getStoragename()) . '/' . $source;
+						$source = str_replace('home::', '', $fInfo->getStoragename()) . '/' . $source;
 						$target = $this->buildTargetForShared($fInfo);
 					} else {
 						// Not Shared
@@ -313,7 +313,7 @@ 
 	 */
 	protected function buildTargetForShared(File $fileInfo) {
 		try {
-			$share = $this->shareMapper->find($fileInfo->getFileid(), $this->userId, str_replace('home::','',$fileInfo->getStoragename()));
+			$share = $this->shareMapper->find($fileInfo->getFileid(), $this->userId, str_replace('home::', '', $fileInfo->getStoragename()));
 
 			// get rid of the .png or .pdf and so on
 			$fileName = substr($share->getFileTarget(), 0, -4); // '/thedom.png' => '/thedom' || '/Test/thedom.png' => '/Test/thedom'
@@ -362,7 +362,7 @@ 
 			$filePath = str_replace('files', '', $filePath); // 'files/Test/' => '/Test/' || 'files/' => '/'
 
 			// remove the last slash
-			$filePath = substr_replace($filePath,'',strrpos($filePath,'/'),strlen('/')); // '/Test/' => '/Test' || '/' => ''
+			$filePath = substr_replace($filePath, '', strrpos($filePath, '/'), strlen('/')); // '/Test/' => '/Test' || '/' => ''
 
 			// replace the first slash
 			$pos = strpos($filePath, '/');
@@ -433,8 +433,8 @@ 
 	 */
 	private function checkSharedWithInitiator($fileInfo) {
 		try {
-			$owner = str_replace('home::','',$fileInfo->getStoragename());
-			if ($this->userId === $owner){
+			$owner = str_replace('home::', '', $fileInfo->getStoragename());
+			if ($this->userId === $owner) {
 				// user is owner (no shared file)
 				return false;
 			} else {
@@ -488,7 +488,7 @@ 
 	 */
 	private function removeFileExtension($status) {
 		try {
-			return substr($status->getTarget(), 0, strrpos($status->getTarget(),'_OCR'));
+			return substr($status->getTarget(), 0, strrpos($status->getTarget(), '_OCR'));
 		} catch (Exception $e) {
 			$this->handleException($e);
 		}