1 files changed, 11 insertions, 15 deletions

diff --git a/lib/Controller/SubscriptionController.php b/lib/Controller/SubscriptionController.php
index e90f4a66..f2517237 100644
--- a/lib/Controller/SubscriptionController.php
+++ b/lib/Controller/SubscriptionController.php
@@ -28,7 +28,6 @@ use OCP\AppFramework\Db\DoesNotExistException;
 use OCA\Polls\Exceptions\NotAuthorizedException;
 
 use OCP\IRequest;
-use OCP\ILogger;
 use OCP\AppFramework\Controller;
 use OCP\AppFramework\Http;
 use OCP\AppFramework\Http\DataResponse;
@@ -37,38 +36,32 @@ use OCA\Polls\Service\SubscriptionService;
 
 class SubscriptionController extends Controller {
 
-	private $userId;
+	/** @var SubscriptionService */
 	private $subscriptionService;
-	private $logger;
 
 	/**
 	 * SubscriptionController constructor.
 	 * @param string $appName
-	 * @param $UserId
 	 * @param SubscriptionService $subscriptionService
 	 * @param IRequest $request
-	 * @param ILogger $logger
 	 */
 
 	public function __construct(
 		string $appName,
-		$userId,
 		SubscriptionService $subscriptionService,
-		IRequest $request,
-		ILogger $logger
-
+		IRequest $request
 	) {
 		parent::__construct($appName, $request);
-		$this->userId = $userId;
 		$this->subscriptionService = $subscriptionService;
-		$this->logger = $logger;
 	}
 
 	/**
+	 * Get subscription status
 	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param integer $pollId
+	 * @param int $pollId
 	 * @return DataResponse
+	 * @throws DoesNotExistException
+	 * @throws NotAuthorizedException
 	 */
 	public function get($pollId) {
 		try {
@@ -81,9 +74,12 @@ class SubscriptionController extends Controller {
 	}
 
 	/**
+	 * Switch subscription status
 	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param integer $pollId
+	 * @param int $pollId
+	 * @param int $subscribed
+	 * @return DataResponse
+	 * @throws NotAuthorizedException
 	 */
 	public function set($pollId, $subscribed) {
 		try {