From 06367d8b81407fc2948e0f666b38f4de2dffbd89 Mon Sep 17 00:00:00 2001
From: Bernhard Posselt <dev@bernhard-posselt.com>
Date: Wed, 25 Jun 2014 11:22:08 +0200
Subject: style fixes

---
 controller/itemapicontroller.php | 120 +++++++++++++++++++++------------------
 1 file changed, 65 insertions(+), 55 deletions(-)

(limited to 'controller/itemapicontroller.php')

diff --git a/controller/itemapicontroller.php b/controller/itemapicontroller.php
index 40366a936..ff66de951 100644
--- a/controller/itemapicontroller.php
+++ b/controller/itemapicontroller.php
@@ -39,18 +39,19 @@ class ItemApiController extends ApiController {
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $type
-	 * @param int $id
-	 * @param bool $getRead
-	 * @param int $batchSize
-	 * @param int $offset
-	 * @param int $oldestFirst
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $type
+     * @param int $id
+     * @param bool $getRead
+     * @param int $batchSize
+     * @param int $offset
+     * @param bool $oldestFirst
+     * @return array|mixed
+     */
 	public function index($type, $id, $getRead, $batchSize=20, $offset=0,
 	                      $oldestFirst=false) {
 		return $this->serializer->serialize(
@@ -62,15 +63,16 @@ class ItemApiController extends ApiController {
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $type
-	 * @param int $id
-	 * @param int $lastModified
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $type
+     * @param int $id
+     * @param int $lastModified
+     * @return array|mixed
+     */
 	public function updated($type, $id, $lastModified=0) {
 		return $this->serializer->serialize(
 			$this->itemService->findAllNew($id, $type, $lastModified,
@@ -85,28 +87,32 @@ class ItemApiController extends ApiController {
 		} catch(ServiceNotFoundException $ex){
 			return $this->error($ex, Http::STATUS_NOT_FOUND);
 		}
+
+        return [];
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $itemId
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $itemId
+     * @return array|\OCP\AppFramework\Http\JSONResponse
+     */
 	public function read($itemId) {
 		return $this->setRead(true, $itemId);
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $itemId
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $itemId
+     * @return array|\OCP\AppFramework\Http\JSONResponse
+     */
 	public function unread($itemId) {
 		return $this->setRead(false, $itemId);
 	}
@@ -118,30 +124,34 @@ class ItemApiController extends ApiController {
 		} catch(ServiceNotFoundException $ex){
 			return $this->error($ex, Http::STATUS_NOT_FOUND);
 		}
+
+        return [];
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $feedId
-	 * @param string $guidHash
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $feedId
+     * @param string $guidHash
+     * @return \OCP\AppFramework\Http\JSONResponse
+     */
 	public function star($feedId, $guidHash) {
 		return $this->setStarred(true, $feedId, $guidHash);
 	}
 
 
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @CORS
-	 *
-	 * @param int $feedId
-	 * @param string $guidHash
-	 */
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @CORS
+     *
+     * @param int $feedId
+     * @param string $guidHash
+     * @return array|\OCP\AppFramework\Http\JSONResponse
+     */
 	public function unstar($feedId, $guidHash) {
 		return $this->setStarred(false, $feedId, $guidHash);
 	}
@@ -178,7 +188,7 @@ class ItemApiController extends ApiController {
 	 * @param int[] item ids
 	 */
 	public function readMultiple($items) {
-		return $this->setMultipleRead(true, $items);
+		$this->setMultipleRead(true, $items);
 	}
 
 
@@ -190,7 +200,7 @@ class ItemApiController extends ApiController {
 	 * @param int[] item ids
 	 */
 	public function unreadMultiple($items) {
-		return $this->setMultipleRead(false, $items);
+		$this->setMultipleRead(false, $items);
 	}
 
 
@@ -214,7 +224,7 @@ class ItemApiController extends ApiController {
 	 * @param int[] item ids
 	 */
 	public function starMultiple($items) {
-		return $this->setMultipleStarred(true, $items);
+		$this->setMultipleStarred(true, $items);
 	}
 
 
@@ -226,7 +236,7 @@ class ItemApiController extends ApiController {
 	 * @param int[] item ids
 	 */
 	public function unstarMultiple($items) {
-		return $this->setMultipleStarred(false, $items);
+		$this->setMultipleStarred(false, $items);
 	}
 
 
-- 
cgit v1.2.3