nextcloud / polls

lib/Controller/PageController.php

Spacing +13 added lines, -13 removed lines

@@ -246,7 +246,7 @@ 
 			]);
 		} else {
 			User::checkLoggedIn();
-			return new TemplateResponse('polls', 'no.acc.tmpl', []);
+			return new TemplateResponse('polls', 'no.acc.tmpl', [ ]);
 		}
 	}
 
@@ -393,7 +393,7 @@ 
 	 */
 	public function createPoll() {
 		return new TemplateResponse('polls', 'create.tmpl',
-			['userId' => $this->userId, 'userMgr' => $this->manager, 'urlGenerator' => $this->urlGenerator]);
+			[ 'userId' => $this->userId, 'userMgr' => $this->manager, 'urlGenerator' => $this->urlGenerator ]);
 	}
 
 	/**
@@ -545,15 +545,15 @@ 
 					$part = new Participation();
 					$part->setPollId($pollId);
 					$part->setUserId($userId);
-					$part->setDt(date('Y-m-d H:i:s', $dates[$i]));
-					$part->setType($types[$i]);
+					$part->setDt(date('Y-m-d H:i:s', $dates[ $i ]));
+					$part->setType($types[ $i ]);
 					$this->participationMapper->insert($part);
 				} else {
 					$part = new ParticipationText();
 					$part->setPollId($pollId);
 					$part->setUserId($userId);
-					$part->setText($dates[$i]);
-					$part->setType($types[$i]);
+					$part->setText($dates[ $i ]);
+					$part->setType($types[ $i ]);
 					$this->participationTextMapper->insert($part);
 				}
 
@@ -561,7 +561,7 @@ 
 			$this->sendNotifications($pollId, $userId);
 		}
 		$hash = $poll->getHash();
-		$url = $this->urlGenerator->linkToRoute('polls.page.goto_poll', ['hash' => $hash]);
+		$url = $this->urlGenerator->linkToRoute('polls.page.goto_poll', [ 'hash' => $hash ]);
 		return new RedirectResponse($url);
 	}
 
@@ -619,15 +619,15 @@ 
 		$gids = array();
 		$sgids = array();
 		foreach ($selectedGroups as $sg) {
-			$sgids[] = str_replace('group_', '', $sg);
+			$sgids[ ] = str_replace('group_', '', $sg);
 		}
 		foreach ($groups as $g) {
-			$gids[] = $g->getGID();
+			$gids[ ] = $g->getGID();
 		}
 		$diffGids = array_diff($gids, $sgids);
 		$gids = array();
 		foreach ($diffGids as $g) {
-			$gids[] = ['gid' => $g, 'isGroup' => true];
+			$gids[ ] = [ 'gid' => $g, 'isGroup' => true ];
 		}
 		return $gids;
 	}
@@ -646,7 +646,7 @@ 
 		$users = array();
 		$sUsers = array();
 		foreach ($selectedUsers as $su) {
-			$sUsers[] = str_replace('user_', '', $su);
+			$sUsers[ ] = str_replace('user_', '', $su);
 		}
 		foreach ($userNames as $u) {
 			$alreadyAdded = false;
@@ -658,7 +658,7 @@ 
 				}
 			}
 			if (!$alreadyAdded) {
-				$users[] = array('uid' => $u->getUID(), 'displayName' => $u->getDisplayName(), 'isGroup' => false);
+				$users[ ] = array('uid' => $u->getUID(), 'displayName' => $u->getDisplayName(), 'isGroup' => false);
 			} else {
 				continue;
 			}
@@ -705,7 +705,7 @@ 
 		}
 		// Nextcloud >= 12
 		$groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
-		return array_map(function ($group) {
+		return array_map(function($group) {
 			return $group->getGID();
 		}, $groups);
 	}

Indentation +706 added lines, -706 removed lines

@@ -55,710 +55,710 @@
 
 class PageController extends Controller {
 
-	private $userId;
-	private $commentMapper;
-	private $dateMapper;
-	private $eventMapper;
-	private $notificationMapper;
-	private $participationMapper;
-	private $participationTextMapper;
-	private $textMapper;
-	private $urlGenerator;
-	private $manager;
-	private $avatarManager;
-	private $logger;
-	private $trans;
-	private $userMgr;
-	private $groupManager;
-
-	/**
-	 * PageController constructor.
-	 * @param $appName
-	 * @param IRequest $request
-	 * @param IUserManager $manager
-	 * @param IGroupManager $groupManager
-	 * @param IAvatarManager $avatarManager
-	 * @param ILogger $logger
-	 * @param IL10N $trans
-	 * @param IURLGenerator $urlGenerator
-	 * @param $userId
-	 * @param CommentMapper $commentMapper
-	 * @param DateMapper $dateMapper
-	 * @param EventMapper $eventMapper
-	 * @param NotificationMapper $notificationMapper
-	 * @param ParticipationMapper $ParticipationMapper
-	 * @param ParticipationTextMapper $ParticipationTextMapper
-	 * @param TextMapper $textMapper
-	 */
-	public function __construct(
-		$appName,
-		IRequest $request,
-		IUserManager $manager,
-		IGroupManager $groupManager,
-		IAvatarManager $avatarManager,
-		ILogger $logger,
-		IL10N $trans,
-		IURLGenerator $urlGenerator,
-		$userId,
-		CommentMapper $commentMapper,
-		DateMapper $dateMapper,
-		EventMapper $eventMapper,
-		NotificationMapper $notificationMapper,
-		ParticipationMapper $ParticipationMapper,
-		ParticipationTextMapper $ParticipationTextMapper,
-		TextMapper $textMapper
-	) {
-		parent::__construct($appName, $request);
-		$this->manager = $manager;
-		$this->groupManager = $groupManager;
-		$this->avatarManager = $avatarManager;
-		$this->logger = $logger;
-		$this->trans = $trans;
-		$this->urlGenerator = $urlGenerator;
-		$this->userId = $userId;
-		$this->commentMapper = $commentMapper;
-		$this->dateMapper = $dateMapper;
-		$this->eventMapper = $eventMapper;
-		$this->notificationMapper = $notificationMapper;
-		$this->participationMapper = $ParticipationMapper;
-		$this->participationTextMapper = $ParticipationTextMapper;
-		$this->textMapper = $textMapper;
-		$this->userMgr = \OC::$server->getUserManager();
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 */
-	public function index() {
-		$polls = $this->eventMapper->findAllForUserWithInfo($this->userId);
-		$comments = $this->commentMapper->findDistinctByUser($this->userId);
-		$partic = $this->participationMapper->findDistinctByUser($this->userId);
-		$particText = $this->participationTextMapper->findDistinctByUser($this->userId);
-		$response = new TemplateResponse('polls', 'main.tmpl', [
-			'polls' => $polls,
-			'comments' => $comments,
-			'participations' => $partic,
-			'participations_text' => $particText,
-			'userId' => $this->userId,
-			'userMgr' => $this->manager,
-			'urlGenerator' => $this->urlGenerator
-		]);
-		if (class_exists('OCP\AppFramework\Http\ContentSecurityPolicy')) {
-			$csp = new \OCP\AppFramework\Http\ContentSecurityPolicy();
-			$response->setContentSecurityPolicy($csp);
-		}
-		return $response;
-	}
-
-	/**
-	 * @param string $pollId
-	 * @param string $from
-	 */
-	private function sendNotifications($pollId, $from) {
-		$poll = $this->eventMapper->find($pollId);
-		$notifications = $this->notificationMapper->findAllByPoll($pollId);
-		foreach ($notifications as $notification) {
-			if ($from === $notification->getUserId()) {
-				continue;
-			}
-			$email = \OC::$server->getConfig()->getUserValue($notification->getUserId(), 'settings', 'email');
-			if (strlen($email) === 0 || !isset($email)) {
-				continue;
-			}
-			$url = \OC::$server->getURLGenerator()->getAbsoluteURL(\OC::$server->getURLGenerator()->linkToRoute('polls.page.goto_poll',
-				array('hash' => $poll->getHash())));
-
-			$recUser = $this->userMgr->get($notification->getUserId());
-			$sendUser = $this->userMgr->get($from);
-			$rec = "";
-			if ($recUser !== null) {
-				$rec = $recUser->getDisplayName();
-			}
-			if ($sendUser !== null) {
-				$sender = $sendUser->getDisplayName();
-			} else {
-				$sender = $from;
-			}
-			$msg = $this->trans->t('Hello %s,<br/><br/><strong>%s</strong> participated in the poll \'%s\'.<br/><br/>To go directly to the poll, you can use this <a href="%s">link</a>',
-				array(
-					$rec,
-					$sender,
-					$poll->getTitle(),
-					$url
-				));
-
-			$msg .= "<br/><br/>";
-
-			$toName = $this->userMgr->get($notification->getUserId())->getDisplayName();
-			$subject = $this->trans->t('Polls App - New Comment');
-			$fromAddress = Util::getDefaultEmailAddress('no-reply');
-			$fromName = $this->trans->t("Polls App") . ' (' . $from . ')';
-
-			try {
-				$mailer = \OC::$server->getMailer();
-				$message = $mailer->createMessage();
-				$message->setSubject($subject);
-				$message->setFrom(array($fromAddress => $fromName));
-				$message->setTo(array($email => $toName));
-				$message->setHtmlBody($msg);
-				$mailer->send($message);
-			} catch (\Exception $e) {
-				$message = 'Error sending mail to: ' . $toName . ' (' . $email . ')';
-				Util::writeLog("polls", $message, Util::ERROR);
-			}
-		}
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 * @param string $hash
-	 * @return TemplateResponse
-	 */
-	public function gotoPoll($hash) {
-		$poll = $this->eventMapper->findByHash($hash);
-		if ($poll->getType() == '0') {
-			$dates = $this->dateMapper->findByPoll($poll->getId());
-			$votes = $this->participationMapper->findByPoll($poll->getId());
-			$participants = $this->participationMapper->listParticipantsByPoll($poll->getId());
-		} else {
-			$dates = $this->textMapper->findByPoll($poll->getId());
-			$votes = $this->participationTextMapper->findByPoll($poll->getId());
-			$participants = $this->participationTextMapper->listParticipantsByPoll($poll->getId());
-		}
-		$comments = $this->commentMapper->findByPoll($poll->getId());
-		try {
-			$notification = $this->notificationMapper->findByUserAndPoll($poll->getId(), $this->userId);
-		} catch (DoesNotExistException $e) {
-			$notification = null;
-		}
-		if ($this->hasUserAccess($poll)) {
-			return new TemplateResponse('polls', 'goto.tmpl', [
-				'poll' => $poll,
-				'dates' => $dates,
-				'comments' => $comments,
-				'votes' => $votes,
-				'participants' => $participants,
-				'notification' => $notification,
-				'userId' => $this->userId,
-				'userMgr' => $this->manager,
-				'urlGenerator' => $this->urlGenerator,
-				'avatarManager' => $this->avatarManager
-			]);
-		} else {
-			User::checkLoggedIn();
-			return new TemplateResponse('polls', 'no.acc.tmpl', []);
-		}
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param string $pollId
-	 * @return RedirectResponse
-	 */
-	public function deletePoll($pollId) {
-		$pollToDelete = $this->eventMapper->find($pollId);
-		if ($this->userId !== $pollToDelete->getOwner()) {
-			return new TemplateResponse('polls', 'no.delete.tmpl');
-		}
-		$poll = new Event();
-		$poll->setId($pollId);
-		$this->eventMapper->delete($poll);
-		$this->textMapper->deleteByPoll($pollId);
-		$this->dateMapper->deleteByPoll($pollId);
-		$this->participationMapper->deleteByPoll($pollId);
-		$this->participationTextMapper->deleteByPoll($pollId);
-		$this->commentMapper->deleteByPoll($pollId);
-		$url = $this->urlGenerator->linkToRoute('polls.page.index');
-		return new RedirectResponse($url);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param string $hash
-	 * @return TemplateResponse
-	 */
-	public function editPoll($hash) {
-		$poll = $this->eventMapper->findByHash($hash);
-		if ($this->userId !== $poll->getOwner()) {
-			return new TemplateResponse('polls', 'no.create.tmpl');
-		}
-		if ($poll->getType() == '0') {
-			$dates = $this->dateMapper->findByPoll($poll->getId());
-		} else {
-			$dates = $this->textMapper->findByPoll($poll->getId());
-		}
-		return new TemplateResponse('polls', 'create.tmpl', [
-			'poll' => $poll,
-			'dates' => $dates,
-			'userId' => $this->userId,
-			'userMgr' => $this->manager,
-			'urlGenerator' => $this->urlGenerator
-		]);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $pollId
-	 * @param $pollType
-	 * @param $pollTitle
-	 * @param $pollDesc
-	 * @param $userId
-	 * @param $chosenDates
-	 * @param $expireTs
-	 * @param $accessType
-	 * @param $accessValues
-	 * @param $isAnonymous
-	 * @param $hideNames
-	 * @return RedirectResponse
-	 */
-	public function updatePoll(
-		$pollId,
-		$pollType,
-		$pollTitle,
-		$pollDesc,
-		$userId,
-		$chosenDates,
-		$expireTs,
-		$accessType,
-		$accessValues,
-		$isAnonymous,
-		$hideNames
-	) {
-		$event = $this->eventMapper->find($pollId);
-		$event->setTitle(htmlspecialchars($pollTitle));
-		$event->setDescription(htmlspecialchars($pollDesc));
-		$event->setIsAnonymous($isAnonymous ? 1 : 0);
-		$event->setFullAnonymous($isAnonymous && $hideNames ? 1 : 0);
-
-		if ($accessType === 'select') {
-			if (isset($accessValues)) {
-				$accessValues = json_decode($accessValues);
-				if ($accessValues !== null) {
-					$groups = array();
-					$users = array();
-					if ($accessValues->groups !== null) {
-						$groups = $accessValues->groups;
-					}
-					if ($accessValues->users !== null) {
-						$users = $accessValues->users;
-					}
-					$accessType = '';
-					foreach ($groups as $gid) {
-						$accessType .= $gid . ';';
-					}
-					foreach ($users as $uid) {
-						$accessType .= $uid . ';';
-					}
-				}
-			}
-		}
-		$event->setAccess($accessType);
-
-		$chosenDates = json_decode($chosenDates);
-
-		$expire = null;
-		if ($expireTs !== null && $expireTs !== '') {
-			$expire = date('Y-m-d H:i:s', $expireTs + 60 * 60 * 24); //add one day, so it expires at the end of a day
-		}
-		$event->setExpire($expire);
-
-		$this->dateMapper->deleteByPoll($pollId);
-		$this->textMapper->deleteByPoll($pollId);
-		if ($pollType === 'event') {
-			$event->setType(0);
-			$this->eventMapper->update($event);
-			sort($chosenDates);
-			foreach ($chosenDates as $el) {
-				$date = new Date();
-				$date->setPollId($pollId);
-				$date->setDt(date('Y-m-d H:i:s', $el));
-				$this->dateMapper->insert($date);
-			}
-		} else {
-			$event->setType(1);
-			$this->eventMapper->update($event);
-			foreach ($chosenDates as $el) {
-				$text = new Text();
-				$text->setText($el);
-				$text->setPollId($pollId);
-				$this->textMapper->insert($text);
-			}
-		}
-		$url = $this->urlGenerator->linkToRoute('polls.page.index');
-		return new RedirectResponse($url);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 */
-	public function createPoll() {
-		return new TemplateResponse('polls', 'create.tmpl',
-			['userId' => $this->userId, 'userMgr' => $this->manager, 'urlGenerator' => $this->urlGenerator]);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $pollType
-	 * @param $pollTitle
-	 * @param $pollDesc
-	 * @param $userId
-	 * @param $chosenDates
-	 * @param $expireTs
-	 * @param $accessType
-	 * @param $accessValues
-	 * @param $isAnonymous
-	 * @param $hideNames
-	 * @return RedirectResponse
-	 */
-	public function insertPoll(
-		$pollType,
-		$pollTitle,
-		$pollDesc,
-		$userId,
-		$chosenDates,
-		$expireTs,
-		$accessType,
-		$accessValues,
-		$isAnonymous,
-		$hideNames
-	) {
-		$event = new Event();
-		$event->setTitle(htmlspecialchars($pollTitle));
-		$event->setDescription(htmlspecialchars($pollDesc));
-		$event->setOwner($userId);
-		$event->setCreated(date('Y-m-d H:i:s'));
-		$event->setHash(\OC::$server->getSecureRandom()->getMediumStrengthGenerator()->generate(16,
-			ISecureRandom::CHAR_DIGITS .
-			ISecureRandom::CHAR_LOWER .
-			ISecureRandom::CHAR_UPPER));
-		$event->setIsAnonymous($isAnonymous ? 1 : 0);
-		$event->setFullAnonymous($isAnonymous && $hideNames ? 1 : 0);
-
-		if ($accessType === 'select') {
-			if (isset($accessValues)) {
-				$accessValues = json_decode($accessValues);
-				if ($accessValues !== null) {
-					$groups = array();
-					$users = array();
-					if ($accessValues->groups !== null) {
-						$groups = $accessValues->groups;
-					}
-					if ($accessValues->users !== null) {
-						$users = $accessValues->users;
-					}
-					$accessType = '';
-					foreach ($groups as $gid) {
-						$accessType .= $gid . ';';
-					}
-					foreach ($users as $uid) {
-						$accessType .= $uid . ';';
-					}
-				}
-			}
-		}
-		$event->setAccess($accessType);
-
-		$chosenDates = json_decode($chosenDates);
-
-		$expire = null;
-		if ($expireTs !== null && $expireTs !== '') {
-			$expire = date('Y-m-d H:i:s', $expireTs + 60 * 60 * 24); //add one day, so it expires at the end of a day
-		}
-		$event->setExpire($expire);
-
-		if ($pollType === 'event') {
-			$event->setType(0);
-			$ins = $this->eventMapper->insert($event);
-			$poll_id = $ins->getId();
-			sort($chosenDates);
-			foreach ($chosenDates as $el) {
-				$date = new Date();
-				$date->setPollId($poll_id);
-				$date->setDt(date('Y-m-d H:i:s', $el));
-				$this->dateMapper->insert($date);
-			}
-		} else {
-			$event->setType(1);
-			$ins = $this->eventMapper->insert($event);
-			$poll_id = $ins->getId();
-			$cnt = 1;
-			foreach ($chosenDates as $el) {
-				$text = new Text();
-				$text->setText($el . '_' . $cnt);
-				$text->setPollId($poll_id);
-				$this->textMapper->insert($text);
-				$cnt++;
-			}
-		}
-		$url = $this->urlGenerator->linkToRoute('polls.page.index');
-		return new RedirectResponse($url);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 * @param $pollId
-	 * @param $userId
-	 * @param $types
-	 * @param $dates
-	 * @param $receiveNotifications
-	 * @param $changed
-	 * @return RedirectResponse
-	 */
-	public function insertVote($pollId, $userId, $types, $dates, $receiveNotifications, $changed) {
-		if ($this->userId !== null) {
-			if ($receiveNotifications === 'true') {
-				try {
-					//check if user already set notification for this poll
-					$this->notificationMapper->findByUserAndPoll($pollId, $userId);
-				} catch (DoesNotExistException $e) {
-					//insert if not exist
-					$not = new Notification();
-					$not->setUserId($userId);
-					$not->setPollId($pollId);
-					$this->notificationMapper->insert($not);
-				}
-			} else {
-				try {
-					//delete if entry is in db
-					$not = $this->notificationMapper->findByUserAndPoll($pollId, $userId);
-					$this->notificationMapper->delete($not);
-				} catch (DoesNotExistException $e) {
-					//doesn't exist in db, nothing to do
-				}
-			}
-		}
-		$poll = $this->eventMapper->find($pollId);
-		if ($changed === 'true') {
-			$dates = json_decode($dates);
-			$types = json_decode($types);
-			$count_dates = count($dates);
-			if ($poll->getType() == '0') {
-				$this->participationMapper->deleteByPollAndUser($pollId, $userId);
-			} else {
-				$this->participationTextMapper->deleteByPollAndUser($pollId, $userId);
-			}
-			for ($i = 0; $i < $count_dates; $i++) {
-				if ($poll->getType() == '0') {
-					$part = new Participation();
-					$part->setPollId($pollId);
-					$part->setUserId($userId);
-					$part->setDt(date('Y-m-d H:i:s', $dates[$i]));
-					$part->setType($types[$i]);
-					$this->participationMapper->insert($part);
-				} else {
-					$part = new ParticipationText();
-					$part->setPollId($pollId);
-					$part->setUserId($userId);
-					$part->setText($dates[$i]);
-					$part->setType($types[$i]);
-					$this->participationTextMapper->insert($part);
-				}
-
-			}
-			$this->sendNotifications($pollId, $userId);
-		}
-		$hash = $poll->getHash();
-		$url = $this->urlGenerator->linkToRoute('polls.page.goto_poll', ['hash' => $hash]);
-		return new RedirectResponse($url);
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @PublicPage
-	 * @param $pollId
-	 * @param $userId
-	 * @param $commentBox
-	 * @return JSONResponse
-	 */
-	public function insertComment($pollId, $userId, $commentBox) {
-		$comment = new Comment();
-		$comment->setPollId($pollId);
-		$comment->setUserId($userId);
-		$comment->setComment($commentBox);
-		$comment->setDt(date('Y-m-d H:i:s'));
-		$this->commentMapper->insert($comment);
-		$this->sendNotifications($pollId, $userId);
-		if ($this->manager->get($userId) !== null) {
-			$newUserId = $this->manager->get($userId)->getDisplayName();
-		} else {
-			$newUserId = $userId;
-		}
-		return new JSONResponse(array(
-			'comment' => $commentBox,
-			'date' => date('Y-m-d H:i:s'),
-			'userName' => $newUserId
-		));
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $searchTerm
-	 * @param $groups
-	 * @param $users
-	 * @return array
-	 */
-	 public function search($searchTerm, $groups, $users) {
-		return array_merge($this->searchForGroups($searchTerm, $groups), $this->searchForUsers($searchTerm, $users));
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $searchTerm
-	 * @param $groups
-	 * @return array
-	 */
-	 public function searchForGroups($searchTerm, $groups) {
-		$selectedGroups = json_decode($groups);
-		$groups = $this->groupManager->search($searchTerm);
-		$gids = array();
-		$sgids = array();
-		foreach ($selectedGroups as $sg) {
-			$sgids[] = str_replace('group_', '', $sg);
-		}
-		foreach ($groups as $g) {
-			$gids[] = $g->getGID();
-		}
-		$diffGids = array_diff($gids, $sgids);
-		$gids = array();
-		foreach ($diffGids as $g) {
-			$gids[] = ['gid' => $g, 'isGroup' => true];
-		}
-		return $gids;
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $searchTerm
-	 * @param $users
-	 * @return array
-	 */
-	 public function searchForUsers($searchTerm, $users) {
-		$selectedUsers = json_decode($users);
-		Util::writeLog("polls", print_r($selectedUsers, true), Util::ERROR);
-		$userNames = $this->userMgr->searchDisplayName($searchTerm);
-		$users = array();
-		$sUsers = array();
-		foreach ($selectedUsers as $su) {
-			$sUsers[] = str_replace('user_', '', $su);
-		}
-		foreach ($userNames as $u) {
-			$alreadyAdded = false;
-			foreach ($sUsers as &$su) {
-				if ($su === $u->getUID()) {
-					unset($su);
-					$alreadyAdded = true;
-					break;
-				}
-			}
-			if (!$alreadyAdded) {
-				$users[] = array('uid' => $u->getUID(), 'displayName' => $u->getDisplayName(), 'isGroup' => false);
-			} else {
-				continue;
-			}
-		}
-		return $users;
-	}
-
-	/**
-	 * @NoAdminRequired
-	 * @NoCSRFRequired
-	 * @param $username
-	 * @return string
-	 */
-	 public function getDisplayName($username) {
-		return $this->manager->get($username)->getDisplayName();
-	}
-
-	/**
-	 * @return Event[]
-	 */
-	 public function getPollsForUser() {
-		return $this->eventMapper->findAllForUser($this->userId);
-	}
-
-	/**
-	 * @param $user
-	 * @return Event[]
-	 */
-	 public function getPollsForUserWithInfo($user = null) {
-		if ($user === null) {
-			return $this->eventMapper->findAllForUserWithInfo($this->userId);
-		} else {
-			return $this->eventMapper->findAllForUserWithInfo($user);
-		}
-	}
-	/**
-	 * @return array
-	 */
-	 public function getGroups() {
-		// $this->requireLogin();
-		if (class_exists('\OC_Group', true)) {
-			// Nextcloud <= 11, ownCloud
-			return \OC_Group::getUserGroups($this->userId);
-		}
-		// Nextcloud >= 12
-		$groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
-		return array_map(function ($group) {
-			return $group->getGID();
-		}, $groups);
-	}
-
-	/**
-	 * @param $poll
-	 * @return bool
-	 */
-	private function hasUserAccess($poll) {
-		$access = $poll->getAccess();
-		$owner = $poll->getOwner();
-		if ($access === 'public') {
-			return true;
-		}
-		if ($access === 'hidden') {
-			return true;
-		}
-		if ($this->userId === null) {
-			return false;
-		}
-		if ($access === 'registered') {
-			return true;
-		}
-		if ($owner === $this->userId) {
-			return true;
-		}
-		Util::writeLog("polls", $this->userId, Util::ERROR);
-		$user_groups = $this->getGroups();
-		$arr = explode(';', $access);
-		foreach ($arr as $item) {
-			if (strpos($item, 'group_') === 0) {
-				$grp = substr($item, 6);
-				foreach ($user_groups as $user_group) {
-					if ($user_group === $grp) {
-						return true;
-					}
-				}
-			} else {
-				if (strpos($item, 'user_') === 0) {
-					$usr = substr($item, 5);
-					if ($usr === User::getUser()) {
-						return true;
-					}
-				}
-			}
-		}
-		return false;
-	}
+    private $userId;
+    private $commentMapper;
+    private $dateMapper;
+    private $eventMapper;
+    private $notificationMapper;
+    private $participationMapper;
+    private $participationTextMapper;
+    private $textMapper;
+    private $urlGenerator;
+    private $manager;
+    private $avatarManager;
+    private $logger;
+    private $trans;
+    private $userMgr;
+    private $groupManager;
+
+    /**
+     * PageController constructor.
+     * @param $appName
+     * @param IRequest $request
+     * @param IUserManager $manager
+     * @param IGroupManager $groupManager
+     * @param IAvatarManager $avatarManager
+     * @param ILogger $logger
+     * @param IL10N $trans
+     * @param IURLGenerator $urlGenerator
+     * @param $userId
+     * @param CommentMapper $commentMapper
+     * @param DateMapper $dateMapper
+     * @param EventMapper $eventMapper
+     * @param NotificationMapper $notificationMapper
+     * @param ParticipationMapper $ParticipationMapper
+     * @param ParticipationTextMapper $ParticipationTextMapper
+     * @param TextMapper $textMapper
+     */
+    public function __construct(
+        $appName,
+        IRequest $request,
+        IUserManager $manager,
+        IGroupManager $groupManager,
+        IAvatarManager $avatarManager,
+        ILogger $logger,
+        IL10N $trans,
+        IURLGenerator $urlGenerator,
+        $userId,
+        CommentMapper $commentMapper,
+        DateMapper $dateMapper,
+        EventMapper $eventMapper,
+        NotificationMapper $notificationMapper,
+        ParticipationMapper $ParticipationMapper,
+        ParticipationTextMapper $ParticipationTextMapper,
+        TextMapper $textMapper
+    ) {
+        parent::__construct($appName, $request);
+        $this->manager = $manager;
+        $this->groupManager = $groupManager;
+        $this->avatarManager = $avatarManager;
+        $this->logger = $logger;
+        $this->trans = $trans;
+        $this->urlGenerator = $urlGenerator;
+        $this->userId = $userId;
+        $this->commentMapper = $commentMapper;
+        $this->dateMapper = $dateMapper;
+        $this->eventMapper = $eventMapper;
+        $this->notificationMapper = $notificationMapper;
+        $this->participationMapper = $ParticipationMapper;
+        $this->participationTextMapper = $ParticipationTextMapper;
+        $this->textMapper = $textMapper;
+        $this->userMgr = \OC::$server->getUserManager();
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     */
+    public function index() {
+        $polls = $this->eventMapper->findAllForUserWithInfo($this->userId);
+        $comments = $this->commentMapper->findDistinctByUser($this->userId);
+        $partic = $this->participationMapper->findDistinctByUser($this->userId);
+        $particText = $this->participationTextMapper->findDistinctByUser($this->userId);
+        $response = new TemplateResponse('polls', 'main.tmpl', [
+            'polls' => $polls,
+            'comments' => $comments,
+            'participations' => $partic,
+            'participations_text' => $particText,
+            'userId' => $this->userId,
+            'userMgr' => $this->manager,
+            'urlGenerator' => $this->urlGenerator
+        ]);
+        if (class_exists('OCP\AppFramework\Http\ContentSecurityPolicy')) {
+            $csp = new \OCP\AppFramework\Http\ContentSecurityPolicy();
+            $response->setContentSecurityPolicy($csp);
+        }
+        return $response;
+    }
+
+    /**
+     * @param string $pollId
+     * @param string $from
+     */
+    private function sendNotifications($pollId, $from) {
+        $poll = $this->eventMapper->find($pollId);
+        $notifications = $this->notificationMapper->findAllByPoll($pollId);
+        foreach ($notifications as $notification) {
+            if ($from === $notification->getUserId()) {
+                continue;
+            }
+            $email = \OC::$server->getConfig()->getUserValue($notification->getUserId(), 'settings', 'email');
+            if (strlen($email) === 0 || !isset($email)) {
+                continue;
+            }
+            $url = \OC::$server->getURLGenerator()->getAbsoluteURL(\OC::$server->getURLGenerator()->linkToRoute('polls.page.goto_poll',
+                array('hash' => $poll->getHash())));
+
+            $recUser = $this->userMgr->get($notification->getUserId());
+            $sendUser = $this->userMgr->get($from);
+            $rec = "";
+            if ($recUser !== null) {
+                $rec = $recUser->getDisplayName();
+            }
+            if ($sendUser !== null) {
+                $sender = $sendUser->getDisplayName();
+            } else {
+                $sender = $from;
+            }
+            $msg = $this->trans->t('Hello %s,<br/><br/><strong>%s</strong> participated in the poll \'%s\'.<br/><br/>To go directly to the poll, you can use this <a href="%s">link</a>',
+                array(
+                    $rec,
+                    $sender,
+                    $poll->getTitle(),
+                    $url
+                ));
+
+            $msg .= "<br/><br/>";
+
+            $toName = $this->userMgr->get($notification->getUserId())->getDisplayName();
+            $subject = $this->trans->t('Polls App - New Comment');
+            $fromAddress = Util::getDefaultEmailAddress('no-reply');
+            $fromName = $this->trans->t("Polls App") . ' (' . $from . ')';
+
+            try {
+                $mailer = \OC::$server->getMailer();
+                $message = $mailer->createMessage();
+                $message->setSubject($subject);
+                $message->setFrom(array($fromAddress => $fromName));
+                $message->setTo(array($email => $toName));
+                $message->setHtmlBody($msg);
+                $mailer->send($message);
+            } catch (\Exception $e) {
+                $message = 'Error sending mail to: ' . $toName . ' (' . $email . ')';
+                Util::writeLog("polls", $message, Util::ERROR);
+            }
+        }
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @PublicPage
+     * @param string $hash
+     * @return TemplateResponse
+     */
+    public function gotoPoll($hash) {
+        $poll = $this->eventMapper->findByHash($hash);
+        if ($poll->getType() == '0') {
+            $dates = $this->dateMapper->findByPoll($poll->getId());
+            $votes = $this->participationMapper->findByPoll($poll->getId());
+            $participants = $this->participationMapper->listParticipantsByPoll($poll->getId());
+        } else {
+            $dates = $this->textMapper->findByPoll($poll->getId());
+            $votes = $this->participationTextMapper->findByPoll($poll->getId());
+            $participants = $this->participationTextMapper->listParticipantsByPoll($poll->getId());
+        }
+        $comments = $this->commentMapper->findByPoll($poll->getId());
+        try {
+            $notification = $this->notificationMapper->findByUserAndPoll($poll->getId(), $this->userId);
+        } catch (DoesNotExistException $e) {
+            $notification = null;
+        }
+        if ($this->hasUserAccess($poll)) {
+            return new TemplateResponse('polls', 'goto.tmpl', [
+                'poll' => $poll,
+                'dates' => $dates,
+                'comments' => $comments,
+                'votes' => $votes,
+                'participants' => $participants,
+                'notification' => $notification,
+                'userId' => $this->userId,
+                'userMgr' => $this->manager,
+                'urlGenerator' => $this->urlGenerator,
+                'avatarManager' => $this->avatarManager
+            ]);
+        } else {
+            User::checkLoggedIn();
+            return new TemplateResponse('polls', 'no.acc.tmpl', []);
+        }
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param string $pollId
+     * @return RedirectResponse
+     */
+    public function deletePoll($pollId) {
+        $pollToDelete = $this->eventMapper->find($pollId);
+        if ($this->userId !== $pollToDelete->getOwner()) {
+            return new TemplateResponse('polls', 'no.delete.tmpl');
+        }
+        $poll = new Event();
+        $poll->setId($pollId);
+        $this->eventMapper->delete($poll);
+        $this->textMapper->deleteByPoll($pollId);
+        $this->dateMapper->deleteByPoll($pollId);
+        $this->participationMapper->deleteByPoll($pollId);
+        $this->participationTextMapper->deleteByPoll($pollId);
+        $this->commentMapper->deleteByPoll($pollId);
+        $url = $this->urlGenerator->linkToRoute('polls.page.index');
+        return new RedirectResponse($url);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param string $hash
+     * @return TemplateResponse
+     */
+    public function editPoll($hash) {
+        $poll = $this->eventMapper->findByHash($hash);
+        if ($this->userId !== $poll->getOwner()) {
+            return new TemplateResponse('polls', 'no.create.tmpl');
+        }
+        if ($poll->getType() == '0') {
+            $dates = $this->dateMapper->findByPoll($poll->getId());
+        } else {
+            $dates = $this->textMapper->findByPoll($poll->getId());
+        }
+        return new TemplateResponse('polls', 'create.tmpl', [
+            'poll' => $poll,
+            'dates' => $dates,
+            'userId' => $this->userId,
+            'userMgr' => $this->manager,
+            'urlGenerator' => $this->urlGenerator
+        ]);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $pollId
+     * @param $pollType
+     * @param $pollTitle
+     * @param $pollDesc
+     * @param $userId
+     * @param $chosenDates
+     * @param $expireTs
+     * @param $accessType
+     * @param $accessValues
+     * @param $isAnonymous
+     * @param $hideNames
+     * @return RedirectResponse
+     */
+    public function updatePoll(
+        $pollId,
+        $pollType,
+        $pollTitle,
+        $pollDesc,
+        $userId,
+        $chosenDates,
+        $expireTs,
+        $accessType,
+        $accessValues,
+        $isAnonymous,
+        $hideNames
+    ) {
+        $event = $this->eventMapper->find($pollId);
+        $event->setTitle(htmlspecialchars($pollTitle));
+        $event->setDescription(htmlspecialchars($pollDesc));
+        $event->setIsAnonymous($isAnonymous ? 1 : 0);
+        $event->setFullAnonymous($isAnonymous && $hideNames ? 1 : 0);
+
+        if ($accessType === 'select') {
+            if (isset($accessValues)) {
+                $accessValues = json_decode($accessValues);
+                if ($accessValues !== null) {
+                    $groups = array();
+                    $users = array();
+                    if ($accessValues->groups !== null) {
+                        $groups = $accessValues->groups;
+                    }
+                    if ($accessValues->users !== null) {
+                        $users = $accessValues->users;
+                    }
+                    $accessType = '';
+                    foreach ($groups as $gid) {
+                        $accessType .= $gid . ';';
+                    }
+                    foreach ($users as $uid) {
+                        $accessType .= $uid . ';';
+                    }
+                }
+            }
+        }
+        $event->setAccess($accessType);
+
+        $chosenDates = json_decode($chosenDates);
+
+        $expire = null;
+        if ($expireTs !== null && $expireTs !== '') {
+            $expire = date('Y-m-d H:i:s', $expireTs + 60 * 60 * 24); //add one day, so it expires at the end of a day
+        }
+        $event->setExpire($expire);
+
+        $this->dateMapper->deleteByPoll($pollId);
+        $this->textMapper->deleteByPoll($pollId);
+        if ($pollType === 'event') {
+            $event->setType(0);
+            $this->eventMapper->update($event);
+            sort($chosenDates);
+            foreach ($chosenDates as $el) {
+                $date = new Date();
+                $date->setPollId($pollId);
+                $date->setDt(date('Y-m-d H:i:s', $el));
+                $this->dateMapper->insert($date);
+            }
+        } else {
+            $event->setType(1);
+            $this->eventMapper->update($event);
+            foreach ($chosenDates as $el) {
+                $text = new Text();
+                $text->setText($el);
+                $text->setPollId($pollId);
+                $this->textMapper->insert($text);
+            }
+        }
+        $url = $this->urlGenerator->linkToRoute('polls.page.index');
+        return new RedirectResponse($url);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     */
+    public function createPoll() {
+        return new TemplateResponse('polls', 'create.tmpl',
+            ['userId' => $this->userId, 'userMgr' => $this->manager, 'urlGenerator' => $this->urlGenerator]);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $pollType
+     * @param $pollTitle
+     * @param $pollDesc
+     * @param $userId
+     * @param $chosenDates
+     * @param $expireTs
+     * @param $accessType
+     * @param $accessValues
+     * @param $isAnonymous
+     * @param $hideNames
+     * @return RedirectResponse
+     */
+    public function insertPoll(
+        $pollType,
+        $pollTitle,
+        $pollDesc,
+        $userId,
+        $chosenDates,
+        $expireTs,
+        $accessType,
+        $accessValues,
+        $isAnonymous,
+        $hideNames
+    ) {
+        $event = new Event();
+        $event->setTitle(htmlspecialchars($pollTitle));
+        $event->setDescription(htmlspecialchars($pollDesc));
+        $event->setOwner($userId);
+        $event->setCreated(date('Y-m-d H:i:s'));
+        $event->setHash(\OC::$server->getSecureRandom()->getMediumStrengthGenerator()->generate(16,
+            ISecureRandom::CHAR_DIGITS .
+            ISecureRandom::CHAR_LOWER .
+            ISecureRandom::CHAR_UPPER));
+        $event->setIsAnonymous($isAnonymous ? 1 : 0);
+        $event->setFullAnonymous($isAnonymous && $hideNames ? 1 : 0);
+
+        if ($accessType === 'select') {
+            if (isset($accessValues)) {
+                $accessValues = json_decode($accessValues);
+                if ($accessValues !== null) {
+                    $groups = array();
+                    $users = array();
+                    if ($accessValues->groups !== null) {
+                        $groups = $accessValues->groups;
+                    }
+                    if ($accessValues->users !== null) {
+                        $users = $accessValues->users;
+                    }
+                    $accessType = '';
+                    foreach ($groups as $gid) {
+                        $accessType .= $gid . ';';
+                    }
+                    foreach ($users as $uid) {
+                        $accessType .= $uid . ';';
+                    }
+                }
+            }
+        }
+        $event->setAccess($accessType);
+
+        $chosenDates = json_decode($chosenDates);
+
+        $expire = null;
+        if ($expireTs !== null && $expireTs !== '') {
+            $expire = date('Y-m-d H:i:s', $expireTs + 60 * 60 * 24); //add one day, so it expires at the end of a day
+        }
+        $event->setExpire($expire);
+
+        if ($pollType === 'event') {
+            $event->setType(0);
+            $ins = $this->eventMapper->insert($event);
+            $poll_id = $ins->getId();
+            sort($chosenDates);
+            foreach ($chosenDates as $el) {
+                $date = new Date();
+                $date->setPollId($poll_id);
+                $date->setDt(date('Y-m-d H:i:s', $el));
+                $this->dateMapper->insert($date);
+            }
+        } else {
+            $event->setType(1);
+            $ins = $this->eventMapper->insert($event);
+            $poll_id = $ins->getId();
+            $cnt = 1;
+            foreach ($chosenDates as $el) {
+                $text = new Text();
+                $text->setText($el . '_' . $cnt);
+                $text->setPollId($poll_id);
+                $this->textMapper->insert($text);
+                $cnt++;
+            }
+        }
+        $url = $this->urlGenerator->linkToRoute('polls.page.index');
+        return new RedirectResponse($url);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @PublicPage
+     * @param $pollId
+     * @param $userId
+     * @param $types
+     * @param $dates
+     * @param $receiveNotifications
+     * @param $changed
+     * @return RedirectResponse
+     */
+    public function insertVote($pollId, $userId, $types, $dates, $receiveNotifications, $changed) {
+        if ($this->userId !== null) {
+            if ($receiveNotifications === 'true') {
+                try {
+                    //check if user already set notification for this poll
+                    $this->notificationMapper->findByUserAndPoll($pollId, $userId);
+                } catch (DoesNotExistException $e) {
+                    //insert if not exist
+                    $not = new Notification();
+                    $not->setUserId($userId);
+                    $not->setPollId($pollId);
+                    $this->notificationMapper->insert($not);
+                }
+            } else {
+                try {
+                    //delete if entry is in db
+                    $not = $this->notificationMapper->findByUserAndPoll($pollId, $userId);
+                    $this->notificationMapper->delete($not);
+                } catch (DoesNotExistException $e) {
+                    //doesn't exist in db, nothing to do
+                }
+            }
+        }
+        $poll = $this->eventMapper->find($pollId);
+        if ($changed === 'true') {
+            $dates = json_decode($dates);
+            $types = json_decode($types);
+            $count_dates = count($dates);
+            if ($poll->getType() == '0') {
+                $this->participationMapper->deleteByPollAndUser($pollId, $userId);
+            } else {
+                $this->participationTextMapper->deleteByPollAndUser($pollId, $userId);
+            }
+            for ($i = 0; $i < $count_dates; $i++) {
+                if ($poll->getType() == '0') {
+                    $part = new Participation();
+                    $part->setPollId($pollId);
+                    $part->setUserId($userId);
+                    $part->setDt(date('Y-m-d H:i:s', $dates[$i]));
+                    $part->setType($types[$i]);
+                    $this->participationMapper->insert($part);
+                } else {
+                    $part = new ParticipationText();
+                    $part->setPollId($pollId);
+                    $part->setUserId($userId);
+                    $part->setText($dates[$i]);
+                    $part->setType($types[$i]);
+                    $this->participationTextMapper->insert($part);
+                }
+
+            }
+            $this->sendNotifications($pollId, $userId);
+        }
+        $hash = $poll->getHash();
+        $url = $this->urlGenerator->linkToRoute('polls.page.goto_poll', ['hash' => $hash]);
+        return new RedirectResponse($url);
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @PublicPage
+     * @param $pollId
+     * @param $userId
+     * @param $commentBox
+     * @return JSONResponse
+     */
+    public function insertComment($pollId, $userId, $commentBox) {
+        $comment = new Comment();
+        $comment->setPollId($pollId);
+        $comment->setUserId($userId);
+        $comment->setComment($commentBox);
+        $comment->setDt(date('Y-m-d H:i:s'));
+        $this->commentMapper->insert($comment);
+        $this->sendNotifications($pollId, $userId);
+        if ($this->manager->get($userId) !== null) {
+            $newUserId = $this->manager->get($userId)->getDisplayName();
+        } else {
+            $newUserId = $userId;
+        }
+        return new JSONResponse(array(
+            'comment' => $commentBox,
+            'date' => date('Y-m-d H:i:s'),
+            'userName' => $newUserId
+        ));
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $searchTerm
+     * @param $groups
+     * @param $users
+     * @return array
+     */
+        public function search($searchTerm, $groups, $users) {
+        return array_merge($this->searchForGroups($searchTerm, $groups), $this->searchForUsers($searchTerm, $users));
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $searchTerm
+     * @param $groups
+     * @return array
+     */
+        public function searchForGroups($searchTerm, $groups) {
+        $selectedGroups = json_decode($groups);
+        $groups = $this->groupManager->search($searchTerm);
+        $gids = array();
+        $sgids = array();
+        foreach ($selectedGroups as $sg) {
+            $sgids[] = str_replace('group_', '', $sg);
+        }
+        foreach ($groups as $g) {
+            $gids[] = $g->getGID();
+        }
+        $diffGids = array_diff($gids, $sgids);
+        $gids = array();
+        foreach ($diffGids as $g) {
+            $gids[] = ['gid' => $g, 'isGroup' => true];
+        }
+        return $gids;
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $searchTerm
+     * @param $users
+     * @return array
+     */
+        public function searchForUsers($searchTerm, $users) {
+        $selectedUsers = json_decode($users);
+        Util::writeLog("polls", print_r($selectedUsers, true), Util::ERROR);
+        $userNames = $this->userMgr->searchDisplayName($searchTerm);
+        $users = array();
+        $sUsers = array();
+        foreach ($selectedUsers as $su) {
+            $sUsers[] = str_replace('user_', '', $su);
+        }
+        foreach ($userNames as $u) {
+            $alreadyAdded = false;
+            foreach ($sUsers as &$su) {
+                if ($su === $u->getUID()) {
+                    unset($su);
+                    $alreadyAdded = true;
+                    break;
+                }
+            }
+            if (!$alreadyAdded) {
+                $users[] = array('uid' => $u->getUID(), 'displayName' => $u->getDisplayName(), 'isGroup' => false);
+            } else {
+                continue;
+            }
+        }
+        return $users;
+    }
+
+    /**
+     * @NoAdminRequired
+     * @NoCSRFRequired
+     * @param $username
+     * @return string
+     */
+        public function getDisplayName($username) {
+        return $this->manager->get($username)->getDisplayName();
+    }
+
+    /**
+     * @return Event[]
+     */
+        public function getPollsForUser() {
+        return $this->eventMapper->findAllForUser($this->userId);
+    }
+
+    /**
+     * @param $user
+     * @return Event[]
+     */
+        public function getPollsForUserWithInfo($user = null) {
+        if ($user === null) {
+            return $this->eventMapper->findAllForUserWithInfo($this->userId);
+        } else {
+            return $this->eventMapper->findAllForUserWithInfo($user);
+        }
+    }
+    /**
+     * @return array
+     */
+        public function getGroups() {
+        // $this->requireLogin();
+        if (class_exists('\OC_Group', true)) {
+            // Nextcloud <= 11, ownCloud
+            return \OC_Group::getUserGroups($this->userId);
+        }
+        // Nextcloud >= 12
+        $groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
+        return array_map(function ($group) {
+            return $group->getGID();
+        }, $groups);
+    }
+
+    /**
+     * @param $poll
+     * @return bool
+     */
+    private function hasUserAccess($poll) {
+        $access = $poll->getAccess();
+        $owner = $poll->getOwner();
+        if ($access === 'public') {
+            return true;
+        }
+        if ($access === 'hidden') {
+            return true;
+        }
+        if ($this->userId === null) {
+            return false;
+        }
+        if ($access === 'registered') {
+            return true;
+        }
+        if ($owner === $this->userId) {
+            return true;
+        }
+        Util::writeLog("polls", $this->userId, Util::ERROR);
+        $user_groups = $this->getGroups();
+        $arr = explode(';', $access);
+        foreach ($arr as $item) {
+            if (strpos($item, 'group_') === 0) {
+                $grp = substr($item, 6);
+                foreach ($user_groups as $user_group) {
+                    if ($user_group === $grp) {
+                        return true;
+                    }
+                }
+            } else {
+                if (strpos($item, 'user_') === 0) {
+                    $usr = substr($item, 5);
+                    if ($usr === User::getUser()) {
+                        return true;
+                    }
+                }
+            }
+        }
+        return false;
+    }
 }

templates/create.tmpl.php

Indentation +65 added lines, -65 removed lines

@@ -1,69 +1,69 @@
 <?php
-	/**
-	 * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @license GNU AGPL version 3 or any later version
-	 *
-	 *  This program is free software: you can redistribute it and/or modify
-	 *  it under the terms of the GNU Affero General Public License as
-	 *  published by the Free Software Foundation, either version 3 of the
-	 *  License, or (at your option) any later version.
-	 *
-	 *  This program is distributed in the hope that it will be useful,
-	 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
-	 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-	 *  GNU Affero General Public License for more details.
-	 *
-	 *  You should have received a copy of the GNU Affero General Public License
-	 *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-	 *
-	 */
-
-	\OCP\Util::addStyle('polls', 'main');
-	\OCP\Util::addStyle('polls', 'createpoll');
-	\OCP\Util::addStyle('polls', 'vendor/jquery.datetimepicker');
-	\OCP\Util::addScript('polls', 'create_edit');
-	\OCP\Util::addScript('polls', 'vendor/jquery.datetimepicker.full.min');
-
-	$userId = $_['userId'];
-	$userMgr = $_['userMgr'];
-	$urlGenerator = $_['urlGenerator'];
-	$isUpdate = isset($_['poll']) && $_['poll'] != null;
-	$isAnonymous = false;
-	$hideNames = false;
-
-	if ($isUpdate) {
-		$poll = $_['poll'];
-		$isAnonymous = $poll->getIsAnonymous();
-		$hideNames = $isAnonymous && $poll->getFullAnonymous();
-		$dates = $_['dates'];
-		$chosen = '[';
-		foreach ($dates as $d) {
-			if ($poll->getType() == '0') {
-				$chosen .= strtotime($d->getDt());
-			} else {
-				$chosen .= '"' . $d->getText() . '"';
-			}
-			$chosen .= ',';
-		}
-		$chosen = trim($chosen, ',');
-		$chosen .= ']';
-		$title = $poll->getTitle();
-		$desc = $poll->getDescription();
-		if ($poll->getExpire() != null) {
-			$expireTs = strtotime($poll->getExpire()) - 60*60*24; //remove one day, which has been added to expire at the end of a day
-			$expireStr = date('d.m.Y', $expireTs);
-		}
-		$access = $poll->getAccess();
-		$accessTypes = $access;
-		if (   $access != 'registered' 
-			&& $access != 'hidden' && $access != 'public'
-		) {
-			$access = 'select';
-		}
-	}
+    /**
+     * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @license GNU AGPL version 3 or any later version
+     *
+     *  This program is free software: you can redistribute it and/or modify
+     *  it under the terms of the GNU Affero General Public License as
+     *  published by the Free Software Foundation, either version 3 of the
+     *  License, or (at your option) any later version.
+     *
+     *  This program is distributed in the hope that it will be useful,
+     *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+     *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+     *  GNU Affero General Public License for more details.
+     *
+     *  You should have received a copy of the GNU Affero General Public License
+     *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+     *
+     */
+
+    \OCP\Util::addStyle('polls', 'main');
+    \OCP\Util::addStyle('polls', 'createpoll');
+    \OCP\Util::addStyle('polls', 'vendor/jquery.datetimepicker');
+    \OCP\Util::addScript('polls', 'create_edit');
+    \OCP\Util::addScript('polls', 'vendor/jquery.datetimepicker.full.min');
+
+    $userId = $_['userId'];
+    $userMgr = $_['userMgr'];
+    $urlGenerator = $_['urlGenerator'];
+    $isUpdate = isset($_['poll']) && $_['poll'] != null;
+    $isAnonymous = false;
+    $hideNames = false;
+
+    if ($isUpdate) {
+        $poll = $_['poll'];
+        $isAnonymous = $poll->getIsAnonymous();
+        $hideNames = $isAnonymous && $poll->getFullAnonymous();
+        $dates = $_['dates'];
+        $chosen = '[';
+        foreach ($dates as $d) {
+            if ($poll->getType() == '0') {
+                $chosen .= strtotime($d->getDt());
+            } else {
+                $chosen .= '"' . $d->getText() . '"';
+            }
+            $chosen .= ',';
+        }
+        $chosen = trim($chosen, ',');
+        $chosen .= ']';
+        $title = $poll->getTitle();
+        $desc = $poll->getDescription();
+        if ($poll->getExpire() != null) {
+            $expireTs = strtotime($poll->getExpire()) - 60*60*24; //remove one day, which has been added to expire at the end of a day
+            $expireStr = date('d.m.Y', $expireTs);
+        }
+        $access = $poll->getAccess();
+        $accessTypes = $access;
+        if (   $access != 'registered' 
+            && $access != 'hidden' && $access != 'public'
+        ) {
+            $access = 'select';
+        }
+    }
 ?>
 
 <div id="app">

lib/Db/ParticipationMapper.php

Indentation +51 added lines, -51 removed lines

@@ -28,60 +28,60 @@
 
 class ParticipationMapper extends Mapper {
 
-	/**
-	 * ParticipationMapper constructor.
-	 * @param IDBConnection $db
-	 */
-	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'polls_particip', '\OCA\Polls\Db\Participation');
-	}
+    /**
+     * ParticipationMapper constructor.
+     * @param IDBConnection $db
+     */
+    public function __construct(IDBConnection $db) {
+        parent::__construct($db, 'polls_particip', '\OCA\Polls\Db\Participation');
+    }
 
-	/**
-	 * @param string $userId
-	 * @param int $limit
-	 * @param int $offset
-	 * @return Participation[]
-	 */
-	public function findDistinctByUser($userId, $limit = null, $offset = null) {
-		$sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
-		return $this->findEntities($sql, [$userId], $limit, $offset);
-	}
+    /**
+     * @param string $userId
+     * @param int $limit
+     * @param int $offset
+     * @return Participation[]
+     */
+    public function findDistinctByUser($userId, $limit = null, $offset = null) {
+        $sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
+        return $this->findEntities($sql, [$userId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param int $limit
-	 * @param int $offset
-	 * @return Participation[]
-	 */
-	public function findByPoll($pollId, $limit = null, $offset = null) {
-		$sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
-	}
+    /**
+     * @param string $pollId
+     * @param int $limit
+     * @param int $offset
+     * @return Participation[]
+     */
+    public function findByPoll($pollId, $limit = null, $offset = null) {
+        $sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        return $this->findEntities($sql, [$pollId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param int $limit
-	 * @param int $offset
-	 */
-	public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
-		$sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
-	}
+    /**
+     * @param string $pollId
+     * @param int $limit
+     * @param int $offset
+     */
+    public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
+        $sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        return $this->findEntities($sql, [$pollId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $pollId
-	 */
-	public function deleteByPoll($pollId) {
-		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
-	}
+    /**
+     * @param string $pollId
+     */
+    public function deleteByPoll($pollId) {
+        $sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        $this->execute($sql, [$pollId]);
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param string $userId
-	 */
-	public function deleteByPollAndUser($pollId, $userId) {
-		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
-		$this->execute($sql, [$pollId, $userId]);
-	}
+    /**
+     * @param string $pollId
+     * @param string $userId
+     */
+    public function deleteByPollAndUser($pollId, $userId) {
+        $sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
+        $this->execute($sql, [$pollId, $userId]);
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -44,7 +44,7 @@ 
 	 */
 	public function findDistinctByUser($userId, $limit = null, $offset = null) {
 		$sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
-		return $this->findEntities($sql, [$userId], $limit, $offset);
+		return $this->findEntities($sql, [ $userId ], $limit, $offset);
 	}
 
 	/**
@@ -55,7 +55,7 @@ 
 	 */
 	public function findByPoll($pollId, $limit = null, $offset = null) {
 		$sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
+		return $this->findEntities($sql, [ $pollId ], $limit, $offset);
 	}
 
 	/**
@@ -65,7 +65,7 @@ 
 	 */
 	public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
 		$sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
+		return $this->findEntities($sql, [ $pollId ], $limit, $offset);
 	}
 
 	/**
@@ -73,7 +73,7 @@ 
 	 */
 	public function deleteByPoll($pollId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
+		$this->execute($sql, [ $pollId ]);
 	}
 
 	/**
@@ -82,6 +82,6 @@ 
 	 */
 	public function deleteByPollAndUser($pollId, $userId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
-		$this->execute($sql, [$pollId, $userId]);
+		$this->execute($sql, [ $pollId, $userId ]);
 	}
 }

lib/Db/ParticipationTextMapper.php

Indentation +51 added lines, -51 removed lines

@@ -28,60 +28,60 @@
 
 class ParticipationTextMapper extends Mapper {
 
-	/**
-	 * ParticipationTextMapper constructor.
-	 * @param IDBConnection $db
-	 */
-	public function __construct(IDBConnection $db) {
-		parent::__construct($db, 'polls_particip_text', '\OCA\Polls\Db\ParticipationText');
-	}
+    /**
+     * ParticipationTextMapper constructor.
+     * @param IDBConnection $db
+     */
+    public function __construct(IDBConnection $db) {
+        parent::__construct($db, 'polls_particip_text', '\OCA\Polls\Db\ParticipationText');
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param int $limit
-	 * @param int $offset
-	 * @return ParticipationText[]
-	 */
-	public function findByPoll($pollId, $limit = null, $offset = null) {
-		$sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
-	}
+    /**
+     * @param string $pollId
+     * @param int $limit
+     * @param int $offset
+     * @return ParticipationText[]
+     */
+    public function findByPoll($pollId, $limit = null, $offset = null) {
+        $sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        return $this->findEntities($sql, [$pollId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $userId
-	 * @param int $limit
-	 * @param int $offset
-	 * @return ParticipationText[]
-	 */
-	public function findDistinctByUser($userId, $limit = null, $offset = null) {
-		$sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
-		return $this->findEntities($sql, [$userId], $limit, $offset);
-	}
+    /**
+     * @param string $userId
+     * @param int $limit
+     * @param int $offset
+     * @return ParticipationText[]
+     */
+    public function findDistinctByUser($userId, $limit = null, $offset = null) {
+        $sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
+        return $this->findEntities($sql, [$userId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param int $limit
-	 * @param int $offset
-	 */
-	public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
-		$sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
-	}
+    /**
+     * @param string $pollId
+     * @param int $limit
+     * @param int $offset
+     */
+    public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
+        $sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        return $this->findEntities($sql, [$pollId], $limit, $offset);
+    }
 
-	/**
-	 * @param string $pollId
-	 */
-	public function deleteByPoll($pollId) {
-		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
-	}
+    /**
+     * @param string $pollId
+     */
+    public function deleteByPoll($pollId) {
+        $sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
+        $this->execute($sql, [$pollId]);
+    }
 
-	/**
-	 * @param string $pollId
-	 * @param string $userId
-	 */
-	public function deleteByPollAndUser($pollId, $userId) {
-		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
-		$this->execute($sql, [$pollId, $userId]);
-	}
+    /**
+     * @param string $pollId
+     * @param string $userId
+     */
+    public function deleteByPollAndUser($pollId, $userId) {
+        $sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
+        $this->execute($sql, [$pollId, $userId]);
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -44,7 +44,7 @@ 
 	 */
 	public function findByPoll($pollId, $limit = null, $offset = null) {
 		$sql = 'SELECT * FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
+		return $this->findEntities($sql, [ $pollId ], $limit, $offset);
 	}
 
 	/**
@@ -55,7 +55,7 @@ 
 	 */
 	public function findDistinctByUser($userId, $limit = null, $offset = null) {
 		$sql = 'SELECT DISTINCT * FROM ' . $this->getTableName() . ' WHERE user_id = ?';
-		return $this->findEntities($sql, [$userId], $limit, $offset);
+		return $this->findEntities($sql, [ $userId ], $limit, $offset);
 	}
 
 	/**
@@ -65,7 +65,7 @@ 
 	 */
 	public function listParticipantsByPoll($pollId, $limit = null, $offset = null) {
 		$sql = 'SELECT DISTINCT user_id FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		return $this->findEntities($sql, [$pollId], $limit, $offset);
+		return $this->findEntities($sql, [ $pollId ], $limit, $offset);
 	}
 
 	/**
@@ -73,7 +73,7 @@ 
 	 */
 	public function deleteByPoll($pollId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
+		$this->execute($sql, [ $pollId ]);
 	}
 
 	/**
@@ -82,6 +82,6 @@ 
 	 */
 	public function deleteByPollAndUser($pollId, $userId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ? AND user_id = ?';
-		$this->execute($sql, [$pollId, $userId]);
+		$this->execute($sql, [ $pollId, $userId ]);
 	}
 }

lib/AppInfo/Application.php

Indentation +126 added lines, -126 removed lines

@@ -37,130 +37,130 @@
 
 class Application extends App {
 
-	/**
-	 * Application constructor.
-	 * @param array $urlParams
-	 */
-	public function __construct(array $urlParams = array()) {
-		parent::__construct('polls', $urlParams);
-
-		$container = $this->getContainer();
-		$server = $container->getServer();
-
-		/**
-		 * Controllers
-		 */
-		$container->registerService('PageController', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new PageController(
-				$c->query('AppName'),
-				$c->query('Request'),
-				$c->query('UserManager'),
-				$c->query('GroupManager'),
-				$c->query('AvatarManager'),
-				$c->query('Logger'),
-				$c->query('L10N'),
-				$c->query('ServerContainer')->getURLGenerator(),
-				$c->query('UserId'),
-				$c->query('CommentMapper'),
-				$c->query('DateMapper'),
-				$c->query('EventMapper'),
-				$c->query('NotificationMapper'),
-				$c->query('ParticipationMapper'),
-				$c->query('ParticipationTextMapper'),
-				$c->query('TextMapper')
-			);
-		});
-
-		$container->registerService('UserManager', function ($c) {
-			/** @var SimpleContainer $c */
-			return $c->query('ServerContainer')->getUserManager();
-		});
-
-		$container->registerService('GroupManager', function ($c) {
-			/** @var SimpleContainer $c */
-			return $c->query('ServerContainer')->getGroupManager();
-		});
-
-		$container->registerService('AvatarManager', function ($c) {
-			/** @var SimpleContainer $c */
-			return $c->query('ServerContainer')->getAvatarManager();
-		});
-
-		$container->registerService('Logger', function ($c) {
-			/** @var SimpleContainer $c */
-			return $c->query('ServerContainer')->getLogger();
-		});
-
-		$container->registerService('L10N', function ($c) {
-			return $c->query('ServerContainer')->getL10N($c->query('AppName'));
-		});
-
-		$container->registerService('CommentMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new CommentMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('DateMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new DateMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('EventMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new EventMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('NotificationMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new NotificationMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('ParticipationMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new ParticipationMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('ParticipationTextMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new ParticipationTextMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-
-		$container->registerService('TextMapper', function ($c) use ($server) {
-			/** @var SimpleContainer $c */
-			return new TextMapper(
-				$server->getDatabaseConnection()
-			);
-		});
-	}
-
-	/**
-	 * Register navigation entry for main navigation.
-	 */
-	public function registerNavigationEntry() {
-		$container = $this->getContainer();
-		$container->query('OCP\INavigationManager')->add(function () use ($container) {
-			$urlGenerator = $container->query('OCP\IURLGenerator');
-			$l10n = $container->query('OCP\IL10N');
-			return [
-				'id' => 'polls',
-				'order' => 77,
-				'href' => $urlGenerator->linkToRoute('polls.page.index'),
-				'icon' => $urlGenerator->imagePath('polls', 'app.svg'),
-				'name' => $l10n->t('Polls')
-			];
-		});
-	}
+    /**
+     * Application constructor.
+     * @param array $urlParams
+     */
+    public function __construct(array $urlParams = array()) {
+        parent::__construct('polls', $urlParams);
+
+        $container = $this->getContainer();
+        $server = $container->getServer();
+
+        /**
+         * Controllers
+         */
+        $container->registerService('PageController', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new PageController(
+                $c->query('AppName'),
+                $c->query('Request'),
+                $c->query('UserManager'),
+                $c->query('GroupManager'),
+                $c->query('AvatarManager'),
+                $c->query('Logger'),
+                $c->query('L10N'),
+                $c->query('ServerContainer')->getURLGenerator(),
+                $c->query('UserId'),
+                $c->query('CommentMapper'),
+                $c->query('DateMapper'),
+                $c->query('EventMapper'),
+                $c->query('NotificationMapper'),
+                $c->query('ParticipationMapper'),
+                $c->query('ParticipationTextMapper'),
+                $c->query('TextMapper')
+            );
+        });
+
+        $container->registerService('UserManager', function ($c) {
+            /** @var SimpleContainer $c */
+            return $c->query('ServerContainer')->getUserManager();
+        });
+
+        $container->registerService('GroupManager', function ($c) {
+            /** @var SimpleContainer $c */
+            return $c->query('ServerContainer')->getGroupManager();
+        });
+
+        $container->registerService('AvatarManager', function ($c) {
+            /** @var SimpleContainer $c */
+            return $c->query('ServerContainer')->getAvatarManager();
+        });
+
+        $container->registerService('Logger', function ($c) {
+            /** @var SimpleContainer $c */
+            return $c->query('ServerContainer')->getLogger();
+        });
+
+        $container->registerService('L10N', function ($c) {
+            return $c->query('ServerContainer')->getL10N($c->query('AppName'));
+        });
+
+        $container->registerService('CommentMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new CommentMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('DateMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new DateMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('EventMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new EventMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('NotificationMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new NotificationMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('ParticipationMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new ParticipationMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('ParticipationTextMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new ParticipationTextMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+
+        $container->registerService('TextMapper', function ($c) use ($server) {
+            /** @var SimpleContainer $c */
+            return new TextMapper(
+                $server->getDatabaseConnection()
+            );
+        });
+    }
+
+    /**
+     * Register navigation entry for main navigation.
+     */
+    public function registerNavigationEntry() {
+        $container = $this->getContainer();
+        $container->query('OCP\INavigationManager')->add(function () use ($container) {
+            $urlGenerator = $container->query('OCP\IURLGenerator');
+            $l10n = $container->query('OCP\IL10N');
+            return [
+                'id' => 'polls',
+                'order' => 77,
+                'href' => $urlGenerator->linkToRoute('polls.page.index'),
+                'icon' => $urlGenerator->imagePath('polls', 'app.svg'),
+                'name' => $l10n->t('Polls')
+            ];
+        });
+    }
 }

templates/goto.tmpl.php

Indentation +261 added lines, -261 removed lines

@@ -1,91 +1,91 @@ 
 <?php
-	/**
-	 * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @license GNU AGPL version 3 or any later version
-	 *
-	 *  This program is free software: you can redistribute it and/or modify
-	 *  it under the terms of the GNU Affero General Public License as
-	 *  published by the Free Software Foundation, either version 3 of the
-	 *  License, or (at your option) any later version.
-	 *
-	 *  This program is distributed in the hope that it will be useful,
-	 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
-	 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-	 *  GNU Affero General Public License for more details.
-	 *
-	 *  You should have received a copy of the GNU Affero General Public License
-	 *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-	 *
-	 */
+    /**
+     * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @license GNU AGPL version 3 or any later version
+     *
+     *  This program is free software: you can redistribute it and/or modify
+     *  it under the terms of the GNU Affero General Public License as
+     *  published by the Free Software Foundation, either version 3 of the
+     *  License, or (at your option) any later version.
+     *
+     *  This program is distributed in the hope that it will be useful,
+     *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+     *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+     *  GNU Affero General Public License for more details.
+     *
+     *  You should have received a copy of the GNU Affero General Public License
+     *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+     *
+     */
 
-	use OCP\User;
+    use OCP\User;
 
-	\OCP\Util::addStyle('polls', 'main');
-	\OCP\Util::addStyle('polls', 'vote');
-	if (!User::isLoggedIn()) {
-		\OCP\Util::addStyle('polls', 'public');
-	}
+    \OCP\Util::addStyle('polls', 'main');
+    \OCP\Util::addStyle('polls', 'vote');
+    if (!User::isLoggedIn()) {
+        \OCP\Util::addStyle('polls', 'public');
+    }
 
-	\OCP\Util::addStyle('polls', 'app-navigation-simulation');
-	\OCP\Util::addScript('polls', 'app');
-	\OCP\Util::addScript('polls', 'vote');
+    \OCP\Util::addStyle('polls', 'app-navigation-simulation');
+    \OCP\Util::addScript('polls', 'app');
+    \OCP\Util::addScript('polls', 'vote');
 	
-	$userId = $_['userId'];
-	$userMgr = $_['userMgr'];
-	$urlGenerator = $_['urlGenerator'];
-	$avaMgr = $_['avatarManager'];
+    $userId = $_['userId'];
+    $userMgr = $_['userMgr'];
+    $urlGenerator = $_['urlGenerator'];
+    $avaMgr = $_['avatarManager'];
 
-	$poll = $_['poll'];
-	$dates = $_['dates'];
-	$votes = $_['votes'];
-	$comments = $_['comments'];
-	$isAnonymous = $poll->getIsAnonymous() && $userId != $poll->getOwner();
-	$hideNames = $poll->getIsAnonymous() && $poll->getFullAnonymous();
-	$notification = $_['notification'];
+    $poll = $_['poll'];
+    $dates = $_['dates'];
+    $votes = $_['votes'];
+    $comments = $_['comments'];
+    $isAnonymous = $poll->getIsAnonymous() && $userId != $poll->getOwner();
+    $hideNames = $poll->getIsAnonymous() && $poll->getFullAnonymous();
+    $notification = $_['notification'];
 
-	if ($poll->getExpire() === null) {
-		$expired = false;
-	} else {
-		$expired = time() > strtotime($poll->getExpire());
-	}
+    if ($poll->getExpire() === null) {
+        $expired = false;
+    } else {
+        $expired = time() > strtotime($poll->getExpire());
+    }
 	
-	if ($expired) {
-		$statusClass = "expired-vote";
-	} else {
-		$statusClass = "open-vote";
-		if (time() < strtotime($poll->getExpire())) {
-			$statusClass = $statusClass . ' endless';
-		}
-	}
+    if ($expired) {
+        $statusClass = "expired-vote";
+    } else {
+        $statusClass = "open-vote";
+        if (time() < strtotime($poll->getExpire())) {
+            $statusClass = $statusClass . ' endless';
+        }
+    }
 
- 	if ($poll->getType() == '0') {
-		$pollType = 'date-poll';
-		$pollTypeClass = 'date-poll';
-	} else if ($poll->getType() == '1') {
-		$pollType = 'option-poll';
-		$pollTypeClass = 'option-poll';
-	}
+        if ($poll->getType() == '0') {
+        $pollType = 'date-poll';
+        $pollTypeClass = 'date-poll';
+    } else if ($poll->getType() == '1') {
+        $pollType = 'option-poll';
+        $pollTypeClass = 'option-poll';
+    }
 
 
-	if (   $poll->getDescription() != null 
-		&& $poll->getDescription() != ''
-	) {
-		$description = nl2br($poll->getDescription());
-	} else {
-		$description = $l->t('No description provided.');
-	}
+    if (   $poll->getDescription() != null 
+        && $poll->getDescription() != ''
+    ) {
+        $description = nl2br($poll->getDescription());
+    } else {
+        $description = $l->t('No description provided.');
+    }
 
-	// init array for counting 'yes'-votes for each date
-	$total = array();
-	for ($i = 0 ; $i < count($dates) ; $i++) {
-		$total['yes'][$i] = 0;
-		$total['no'][$i] = 0;
-	}
-	$userVoted = array();
-	$pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', ['hash' => $poll->getHash()]);
+    // init array for counting 'yes'-votes for each date
+    $total = array();
+    for ($i = 0 ; $i < count($dates) ; $i++) {
+        $total['yes'][$i] = 0;
+        $total['no'][$i] = 0;
+    }
+    $userVoted = array();
+    $pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', ['hash' => $poll->getHash()]);
 ?>
 
 <div id="app-disabled">
@@ -119,82 +119,82 @@ 
 		<div id="votings" class="main-container">
 			<div class="wordwrap description"><span><?php p($description); ?></span>
 			<?php 
-				if ($expired) {
-					print_unescaped('<span class="' . $statusClass . '">' . $l->t('The poll expired on %s. Voting is disabled, but you can still comment.', array(date('d.m.Y H:i', strtotime($poll->getExpire())))) . '</span>');
-				}?>
+                if ($expired) {
+                    print_unescaped('<span class="' . $statusClass . '">' . $l->t('The poll expired on %s. Voting is disabled, but you can still comment.', array(date('d.m.Y H:i', strtotime($poll->getExpire())))) . '</span>');
+                }?>
 			</div>
 			<div class="table">
 					<ul class="row header" >
 						<?php
-						foreach ($dates as $el) {
-							if ($poll->getType() == '0') {
-								$timestamp = strtotime($el->getDt());
-								print_unescaped('<li id="slot_' . $el->getId() . '" title="' . $el->getDt() . ' ' . date_default_timezone_get() . '" class="column vote time" data-timestamp="' . $timestamp . '"data-value-utc="' . $el->getDt() . '">');
+                        foreach ($dates as $el) {
+                            if ($poll->getType() == '0') {
+                                $timestamp = strtotime($el->getDt());
+                                print_unescaped('<li id="slot_' . $el->getId() . '" title="' . $el->getDt() . ' ' . date_default_timezone_get() . '" class="column vote time" data-timestamp="' . $timestamp . '"data-value-utc="' . $el->getDt() . '">');
 								
-								print_unescaped('	<div class="date-box column">');
-								print_unescaped('		<div class="month">' . $l->t(date('M', $timestamp))  . '</div>');
-								print_unescaped('		<div class="day">'   .       date('j', $timestamp)   . '</div>');
-								print_unescaped('		<div class="dayow">' . $l->t(date('D', $timestamp))  . '</div>');
-								print_unescaped('		<div class="time">'  .       date('G:i', $timestamp) . ' UTC</div>');
-								print_unescaped('	</div>');
-							} else {
-								print_unescaped('<li id="slot_' . $el->getId() . '" title="' . preg_replace('/_\d+$/', '', $el->getText()) . '" class="column vote option">');
-								print_unescaped('	<div class="date-box column">' . preg_replace('/_\d+$/', '', $el->getText()).'</div>');
-							}
-							print_unescaped('<div class="counter row">');
-							print_unescaped('	<div class="yes row">');
-							print_unescaped('		<div class="svg"></div>');
-							print_unescaped('		<div id="counter_yes_voteid_' . $el->getId() . '" class ="result-cell yes" data-voteId="' . $el->getId() . '">0</div>');
-							print_unescaped('	</div>');
-							print_unescaped('	<div class="no row">');
-							print_unescaped('		<div class="svg"></div>');
-							print_unescaped('		<div id="counter_no_voteid_' . $el->getId() . '" class ="result-cell no" data-voteId="' . $el->getId() . '">0</div>');
-							print_unescaped('	</div>');
-							print_unescaped('</div>');
-						}
-						?>
+                                print_unescaped('	<div class="date-box column">');
+                                print_unescaped('		<div class="month">' . $l->t(date('M', $timestamp))  . '</div>');
+                                print_unescaped('		<div class="day">'   .       date('j', $timestamp)   . '</div>');
+                                print_unescaped('		<div class="dayow">' . $l->t(date('D', $timestamp))  . '</div>');
+                                print_unescaped('		<div class="time">'  .       date('G:i', $timestamp) . ' UTC</div>');
+                                print_unescaped('	</div>');
+                            } else {
+                                print_unescaped('<li id="slot_' . $el->getId() . '" title="' . preg_replace('/_\d+$/', '', $el->getText()) . '" class="column vote option">');
+                                print_unescaped('	<div class="date-box column">' . preg_replace('/_\d+$/', '', $el->getText()).'</div>');
+                            }
+                            print_unescaped('<div class="counter row">');
+                            print_unescaped('	<div class="yes row">');
+                            print_unescaped('		<div class="svg"></div>');
+                            print_unescaped('		<div id="counter_yes_voteid_' . $el->getId() . '" class ="result-cell yes" data-voteId="' . $el->getId() . '">0</div>');
+                            print_unescaped('	</div>');
+                            print_unescaped('	<div class="no row">');
+                            print_unescaped('		<div class="svg"></div>');
+                            print_unescaped('		<div id="counter_no_voteid_' . $el->getId() . '" class ="result-cell no" data-voteId="' . $el->getId() . '">0</div>');
+                            print_unescaped('	</div>');
+                            print_unescaped('</div>');
+                        }
+                        ?>
 						</li>
 					</ul>
 				<ul class="column table-body">
 					<?php
-					if ($votes != null) {
-						//group by user
-						$others = array();
-						$displayName = '';
-						$avatarName = '';
-						$activeClass = '';
-						foreach ($votes as $vote) {
-							if (!isset($others[$vote->getUserId()])) {
-								$others[$vote->getUserId()] = array();
-							}
-							array_push($others[$vote->getUserId()], $vote);
-						}
-						$userCnt = 0;
-						foreach (array_keys($others) as $usr) {
-							$userCnt++;
-							if ($usr == $userId) {
-								// if poll expired, just put current user among the others;
-								// otherwise skip here to add current user as last row (to vote)
-								if (!$expired) {
-									$userVoted = $others[$usr];
-									continue;
-								}
-							}
-							if (	$userMgr->get($usr) != null 
-								&& !$isAnonymous && !$hideNames
-							) {
-								$displayName = $userMgr->get($usr)->getDisplayName();
-								$avatarName = $usr;
-							} else {
-								if ($isAnonymous || $hideNames) {
-									$displayName = 'Anonymous';
-									$avatarName = $userCnt;
-								} else {
-									$displayName = $usr;
-									$avatarName = $usr;
-								}
-							}
-							?>
+                    if ($votes != null) {
+                        //group by user
+                        $others = array();
+                        $displayName = '';
+                        $avatarName = '';
+                        $activeClass = '';
+                        foreach ($votes as $vote) {
+                            if (!isset($others[$vote->getUserId()])) {
+                                $others[$vote->getUserId()] = array();
+                            }
+                            array_push($others[$vote->getUserId()], $vote);
+                        }
+                        $userCnt = 0;
+                        foreach (array_keys($others) as $usr) {
+                            $userCnt++;
+                            if ($usr == $userId) {
+                                // if poll expired, just put current user among the others;
+                                // otherwise skip here to add current user as last row (to vote)
+                                if (!$expired) {
+                                    $userVoted = $others[$usr];
+                                    continue;
+                                }
+                            }
+                            if (	$userMgr->get($usr) != null 
+                                && !$isAnonymous && !$hideNames
+                            ) {
+                                $displayName = $userMgr->get($usr)->getDisplayName();
+                                $avatarName = $usr;
+                            } else {
+                                if ($isAnonymous || $hideNames) {
+                                    $displayName = 'Anonymous';
+                                    $avatarName = $userCnt;
+                                } else {
+                                    $displayName = $usr;
+                                    $avatarName = $usr;
+                                }
+                            }
+                            ?>
 							<li class="row user">
 								<div class="first">
 									<div class="user-cell row">
@@ -206,115 +206,115 @@ 
 									</div>
 									<ul class="row">
 							<?php
-							// loop over dts
-							$i_tot = 0;
+                            // loop over dts
+                            $i_tot = 0;
 							
-							foreach ($dates as $dt) {
-								if ($poll->getType() == '0') {
-									$dateId = strtotime($dt->getDt());
-									$pollId = "voteid_" . $dt->getId();
-								} else {
-									$dateId = $dt->getText();
-									$pollId = "voteid_" . $dt->getId();
-								}
-								// look what user voted for this dts
-								$class = 'column poll-cell no';
-								foreach ($others[$usr] as $vote) {
-									$voteVal = null;
-									if ($poll->getType() == '0') {
-										$voteVal = strtotime($vote->getDt());
-									} else {
-										$voteVal = $vote->getText();
-									}
-									if ($dateId == $voteVal) {
-										if ($vote->getType() == '1') {
-											$class = 'column poll-cell yes';
-											$total['yes'][$i_tot]++;
-										} else if ($vote->getType() == '0') {
-											$class = 'column poll-cell no';
-											$total['no'][$i_tot]++;
-										} else if ($vote->getType() == '2') {
-											$class = 'column poll-cell maybe';
-										}
-										break;
-									}
-								}
-								print_unescaped('<li id="'. $pollId . '" class="' . $class . '"></li>');
-								$i_tot++;
-							}
+                            foreach ($dates as $dt) {
+                                if ($poll->getType() == '0') {
+                                    $dateId = strtotime($dt->getDt());
+                                    $pollId = "voteid_" . $dt->getId();
+                                } else {
+                                    $dateId = $dt->getText();
+                                    $pollId = "voteid_" . $dt->getId();
+                                }
+                                // look what user voted for this dts
+                                $class = 'column poll-cell no';
+                                foreach ($others[$usr] as $vote) {
+                                    $voteVal = null;
+                                    if ($poll->getType() == '0') {
+                                        $voteVal = strtotime($vote->getDt());
+                                    } else {
+                                        $voteVal = $vote->getText();
+                                    }
+                                    if ($dateId == $voteVal) {
+                                        if ($vote->getType() == '1') {
+                                            $class = 'column poll-cell yes';
+                                            $total['yes'][$i_tot]++;
+                                        } else if ($vote->getType() == '0') {
+                                            $class = 'column poll-cell no';
+                                            $total['no'][$i_tot]++;
+                                        } else if ($vote->getType() == '2') {
+                                            $class = 'column poll-cell maybe';
+                                        }
+                                        break;
+                                    }
+                                }
+                                print_unescaped('<li id="'. $pollId . '" class="' . $class . '"></li>');
+                                $i_tot++;
+                            }
 							
-							print_unescaped('</ul>');
-							print_unescaped('</li>');
-						}
-					}
-					$totalYesOthers = array_merge(array(), $total['yes']);
-					$totalNoOthers = array_merge(array(), $total['no']);
-					$toggleTooltip = $l->t('Switch all options at once');
-					if (!$expired) {
-						print_unescaped('<li class="row user current-user">');
-						print_unescaped('	<div class="row first">');
-						print_unescaped('		<div class="user-cell row">');
-						print_unescaped('			<div class="avatar-cell">');
-						if (User::isLoggedIn()) {
-							print_unescaped('			<div class="poll avatardiv" title="'.($userId).'"></div>');
-							print_unescaped('		</div>');
-							print_unescaped('		<div class="name">');
-							p($userMgr->get($userId)->getDisplayName());
-						} else {
-							print_unescaped('			<div class="poll avatardiv" title="?"></div>');
-							print_unescaped('		</div>');
-							print_unescaped('		<div id="id_ac_detected" class="external current-user"><input type="text" name="user_name" id="user_name" placeholder="' . $l->t('Your name here') . '" />');
-						}
-						print_unescaped('		</div>');
-						print_unescaped('	</div>');
-						print_unescaped('	<div id="toggle-cell" class="toggle-cell maybe" title="'. $toggleTooltip .'">');
-						print_unescaped('		<div class="toggle"></div>');
-						print_unescaped('	</div>');
-						print_unescaped('</div>');
-						print_unescaped('<ul class="row">');
+                            print_unescaped('</ul>');
+                            print_unescaped('</li>');
+                        }
+                    }
+                    $totalYesOthers = array_merge(array(), $total['yes']);
+                    $totalNoOthers = array_merge(array(), $total['no']);
+                    $toggleTooltip = $l->t('Switch all options at once');
+                    if (!$expired) {
+                        print_unescaped('<li class="row user current-user">');
+                        print_unescaped('	<div class="row first">');
+                        print_unescaped('		<div class="user-cell row">');
+                        print_unescaped('			<div class="avatar-cell">');
+                        if (User::isLoggedIn()) {
+                            print_unescaped('			<div class="poll avatardiv" title="'.($userId).'"></div>');
+                            print_unescaped('		</div>');
+                            print_unescaped('		<div class="name">');
+                            p($userMgr->get($userId)->getDisplayName());
+                        } else {
+                            print_unescaped('			<div class="poll avatardiv" title="?"></div>');
+                            print_unescaped('		</div>');
+                            print_unescaped('		<div id="id_ac_detected" class="external current-user"><input type="text" name="user_name" id="user_name" placeholder="' . $l->t('Your name here') . '" />');
+                        }
+                        print_unescaped('		</div>');
+                        print_unescaped('	</div>');
+                        print_unescaped('	<div id="toggle-cell" class="toggle-cell maybe" title="'. $toggleTooltip .'">');
+                        print_unescaped('		<div class="toggle"></div>');
+                        print_unescaped('	</div>');
+                        print_unescaped('</div>');
+                        print_unescaped('<ul class="row">');
 
-						$i_tot = 0;
-						foreach ($dates as $dt) {
-							if ($poll->getType() == '0') {
-								$dateId = strtotime($dt->getDt());
-								$pollId = "voteid_" . $dt->getId();
-							} else {
-								$dateId = $dt->getText();
-								$pollId = "voteid_" . $dt->getId();
-							}
-							// see if user already has data for this event
-							$class = 'no';
-							$activeClass = 'poll-cell active cl_click';
-							if (isset($userVoted)) {
-								foreach ($userVoted as $obj) {
-									$voteVal = null;
-									if($poll->getType() == '0') {
-										$voteVal = strtotime($obj->getDt());
-									} else {
-										$voteVal = $obj->getText();
-									}
-									if ($voteVal == $dateId) {
-										if ($obj->getType() == '1') {
-											$class = 'column poll-cell yes';
-											$total['yes'][$i_tot]++;
-										} else if ($obj->getType() == '0') {
-											$class = 'column poll-cell no';
-											$total['no'][$i_tot]++;
-										} else if($obj->getType() == '2') {
-											$class = 'column poll-cell maybe';
-										}
-										break;
-									}
-								}
-							}
-							print_unescaped('<li id="' . $pollId . '" class="' . $class . ' ' . $activeClass . '" data-value="' . $dateId . '"></li>');
+                        $i_tot = 0;
+                        foreach ($dates as $dt) {
+                            if ($poll->getType() == '0') {
+                                $dateId = strtotime($dt->getDt());
+                                $pollId = "voteid_" . $dt->getId();
+                            } else {
+                                $dateId = $dt->getText();
+                                $pollId = "voteid_" . $dt->getId();
+                            }
+                            // see if user already has data for this event
+                            $class = 'no';
+                            $activeClass = 'poll-cell active cl_click';
+                            if (isset($userVoted)) {
+                                foreach ($userVoted as $obj) {
+                                    $voteVal = null;
+                                    if($poll->getType() == '0') {
+                                        $voteVal = strtotime($obj->getDt());
+                                    } else {
+                                        $voteVal = $obj->getText();
+                                    }
+                                    if ($voteVal == $dateId) {
+                                        if ($obj->getType() == '1') {
+                                            $class = 'column poll-cell yes';
+                                            $total['yes'][$i_tot]++;
+                                        } else if ($obj->getType() == '0') {
+                                            $class = 'column poll-cell no';
+                                            $total['no'][$i_tot]++;
+                                        } else if($obj->getType() == '2') {
+                                            $class = 'column poll-cell maybe';
+                                        }
+                                        break;
+                                    }
+                                }
+                            }
+                            print_unescaped('<li id="' . $pollId . '" class="' . $class . ' ' . $activeClass . '" data-value="' . $dateId . '"></li>');
 
-							$i_tot++;
-						}
-						print_unescaped('</ul>');
-						print_unescaped('</li>');
-					}
-					?>
+                            $i_tot++;
+                        }
+                        print_unescaped('</ul>');
+                        print_unescaped('</li>');
+                    }
+                    ?>
 				</ul>
 			</div>
 			<div class="submit row">
@@ -405,19 +405,19 @@ 
 						<div class="comment">
 							<div class="comment-header">
 								<?php
-								print_unescaped('<span class="comment-date">' . date('d.m.Y H:i:s', strtotime($comment->getDt())) . '</span>');
-								if ($isAnonymous || $hideNames) {
-									p('Anonymous');
-								} else {
-									if ($userMgr->get($comment->getUserId()) != null) {
-										p($userMgr->get($comment->getUserId())->getDisplayName());
-									} else {
-										print_unescaped('<i>');
-										p($comment->getUserId());
-										print_unescaped('</i>');
-									}
-								}
-								?>
+                                print_unescaped('<span class="comment-date">' . date('d.m.Y H:i:s', strtotime($comment->getDt())) . '</span>');
+                                if ($isAnonymous || $hideNames) {
+                                    p('Anonymous');
+                                } else {
+                                    if ($userMgr->get($comment->getUserId()) != null) {
+                                        p($userMgr->get($comment->getUserId())->getDisplayName());
+                                    } else {
+                                        print_unescaped('<i>');
+                                        p($comment->getUserId());
+                                        print_unescaped('</i>');
+                                    }
+                                }
+                                ?>
 							</div>
 							<div class="wordwrap comment-content">
 								<?php p($comment->getComment()); ?>

Spacing +38 added lines, -38 removed lines

@@ -33,18 +33,18 @@ 
 	\OCP\Util::addScript('polls', 'app');
 	\OCP\Util::addScript('polls', 'vote');
 	
-	$userId = $_['userId'];
-	$userMgr = $_['userMgr'];
-	$urlGenerator = $_['urlGenerator'];
-	$avaMgr = $_['avatarManager'];
+	$userId = $_[ 'userId' ];
+	$userMgr = $_[ 'userMgr' ];
+	$urlGenerator = $_[ 'urlGenerator' ];
+	$avaMgr = $_[ 'avatarManager' ];
 
-	$poll = $_['poll'];
-	$dates = $_['dates'];
-	$votes = $_['votes'];
-	$comments = $_['comments'];
+	$poll = $_[ 'poll' ];
+	$dates = $_[ 'dates' ];
+	$votes = $_[ 'votes' ];
+	$comments = $_[ 'comments' ];
 	$isAnonymous = $poll->getIsAnonymous() && $userId != $poll->getOwner();
 	$hideNames = $poll->getIsAnonymous() && $poll->getFullAnonymous();
-	$notification = $_['notification'];
+	$notification = $_[ 'notification' ];
 
 	if ($poll->getExpire() === null) {
 		$expired = false;
@@ -70,7 +70,7 @@ 
 	}
 
 
-	if (   $poll->getDescription() != null 
+	if ($poll->getDescription() != null 
 		&& $poll->getDescription() != ''
 	) {
 		$description = nl2br($poll->getDescription());
@@ -80,12 +80,12 @@ 
 
 	// init array for counting 'yes'-votes for each date
 	$total = array();
-	for ($i = 0 ; $i < count($dates) ; $i++) {
-		$total['yes'][$i] = 0;
-		$total['no'][$i] = 0;
+	for ($i = 0; $i < count($dates); $i++) {
+		$total[ 'yes' ][ $i ] = 0;
+		$total[ 'no' ][ $i ] = 0;
 	}
 	$userVoted = array();
-	$pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', ['hash' => $poll->getHash()]);
+	$pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', [ 'hash' => $poll->getHash() ]);
 ?>
 
 <div id="app-disabled">
@@ -132,14 +132,14 @@ 
 								print_unescaped('<li id="slot_' . $el->getId() . '" title="' . $el->getDt() . ' ' . date_default_timezone_get() . '" class="column vote time" data-timestamp="' . $timestamp . '"data-value-utc="' . $el->getDt() . '">');
 								
 								print_unescaped('	<div class="date-box column">');
-								print_unescaped('		<div class="month">' . $l->t(date('M', $timestamp))  . '</div>');
-								print_unescaped('		<div class="day">'   .       date('j', $timestamp)   . '</div>');
-								print_unescaped('		<div class="dayow">' . $l->t(date('D', $timestamp))  . '</div>');
-								print_unescaped('		<div class="time">'  .       date('G:i', $timestamp) . ' UTC</div>');
+								print_unescaped('		<div class="month">' . $l->t(date('M', $timestamp)) . '</div>');
+								print_unescaped('		<div class="day">' . date('j', $timestamp) . '</div>');
+								print_unescaped('		<div class="dayow">' . $l->t(date('D', $timestamp)) . '</div>');
+								print_unescaped('		<div class="time">' . date('G:i', $timestamp) . ' UTC</div>');
 								print_unescaped('	</div>');
 							} else {
 								print_unescaped('<li id="slot_' . $el->getId() . '" title="' . preg_replace('/_\d+$/', '', $el->getText()) . '" class="column vote option">');
-								print_unescaped('	<div class="date-box column">' . preg_replace('/_\d+$/', '', $el->getText()).'</div>');
+								print_unescaped('	<div class="date-box column">' . preg_replace('/_\d+$/', '', $el->getText()) . '</div>');
 							}
 							print_unescaped('<div class="counter row">');
 							print_unescaped('	<div class="yes row">');
@@ -164,10 +164,10 @@ 
 						$avatarName = '';
 						$activeClass = '';
 						foreach ($votes as $vote) {
-							if (!isset($others[$vote->getUserId()])) {
-								$others[$vote->getUserId()] = array();
+							if (!isset($others[ $vote->getUserId() ])) {
+								$others[ $vote->getUserId() ] = array();
 							}
-							array_push($others[$vote->getUserId()], $vote);
+							array_push($others[ $vote->getUserId() ], $vote);
 						}
 						$userCnt = 0;
 						foreach (array_keys($others) as $usr) {
@@ -176,11 +176,11 @@ 
 								// if poll expired, just put current user among the others;
 								// otherwise skip here to add current user as last row (to vote)
 								if (!$expired) {
-									$userVoted = $others[$usr];
+									$userVoted = $others[ $usr ];
 									continue;
 								}
 							}
-							if (	$userMgr->get($usr) != null 
+							if ($userMgr->get($usr) != null 
 								&& !$isAnonymous && !$hideNames
 							) {
 								$displayName = $userMgr->get($usr)->getDisplayName();
@@ -219,7 +219,7 @@ 
 								}
 								// look what user voted for this dts
 								$class = 'column poll-cell no';
-								foreach ($others[$usr] as $vote) {
+								foreach ($others[ $usr ] as $vote) {
 									$voteVal = null;
 									if ($poll->getType() == '0') {
 										$voteVal = strtotime($vote->getDt());
@@ -229,17 +229,17 @@ 
 									if ($dateId == $voteVal) {
 										if ($vote->getType() == '1') {
 											$class = 'column poll-cell yes';
-											$total['yes'][$i_tot]++;
+											$total[ 'yes' ][ $i_tot ]++;
 										} else if ($vote->getType() == '0') {
 											$class = 'column poll-cell no';
-											$total['no'][$i_tot]++;
+											$total[ 'no' ][ $i_tot ]++;
 										} else if ($vote->getType() == '2') {
 											$class = 'column poll-cell maybe';
 										}
 										break;
 									}
 								}
-								print_unescaped('<li id="'. $pollId . '" class="' . $class . '"></li>');
+								print_unescaped('<li id="' . $pollId . '" class="' . $class . '"></li>');
 								$i_tot++;
 							}
 							
@@ -247,8 +247,8 @@ 
 							print_unescaped('</li>');
 						}
 					}
-					$totalYesOthers = array_merge(array(), $total['yes']);
-					$totalNoOthers = array_merge(array(), $total['no']);
+					$totalYesOthers = array_merge(array(), $total[ 'yes' ]);
+					$totalNoOthers = array_merge(array(), $total[ 'no' ]);
 					$toggleTooltip = $l->t('Switch all options at once');
 					if (!$expired) {
 						print_unescaped('<li class="row user current-user">');
@@ -256,7 +256,7 @@ 
 						print_unescaped('		<div class="user-cell row">');
 						print_unescaped('			<div class="avatar-cell">');
 						if (User::isLoggedIn()) {
-							print_unescaped('			<div class="poll avatardiv" title="'.($userId).'"></div>');
+							print_unescaped('			<div class="poll avatardiv" title="' . ($userId) . '"></div>');
 							print_unescaped('		</div>');
 							print_unescaped('		<div class="name">');
 							p($userMgr->get($userId)->getDisplayName());
@@ -267,7 +267,7 @@ 
 						}
 						print_unescaped('		</div>');
 						print_unescaped('	</div>');
-						print_unescaped('	<div id="toggle-cell" class="toggle-cell maybe" title="'. $toggleTooltip .'">');
+						print_unescaped('	<div id="toggle-cell" class="toggle-cell maybe" title="' . $toggleTooltip . '">');
 						print_unescaped('		<div class="toggle"></div>');
 						print_unescaped('	</div>');
 						print_unescaped('</div>');
@@ -288,7 +288,7 @@ 
 							if (isset($userVoted)) {
 								foreach ($userVoted as $obj) {
 									$voteVal = null;
-									if($poll->getType() == '0') {
+									if ($poll->getType() == '0') {
 										$voteVal = strtotime($obj->getDt());
 									} else {
 										$voteVal = $obj->getText();
@@ -296,11 +296,11 @@ 
 									if ($voteVal == $dateId) {
 										if ($obj->getType() == '1') {
 											$class = 'column poll-cell yes';
-											$total['yes'][$i_tot]++;
+											$total[ 'yes' ][ $i_tot ]++;
 										} else if ($obj->getType() == '0') {
 											$class = 'column poll-cell no';
-											$total['no'][$i_tot]++;
-										} else if($obj->getType() == '2') {
+											$total[ 'no' ][ $i_tot ]++;
+										} else if ($obj->getType() == '2') {
 											$class = 'column poll-cell maybe';
 										}
 										break;
@@ -339,7 +339,7 @@ 
 		</div>
 					
 		<div id="app-sidebar" class="detailsView scroll-container">
-			<a id="closeDetails" class="close icon-close" href="#" alt="<?php $l->t('Close');?>"></a>
+			<a id="closeDetails" class="close icon-close" href="#" alt="<?php $l->t('Close'); ?>"></a>
 
 
 			<div class="table">
@@ -359,7 +359,7 @@ 
 								</a>
 							</li>
 							<li>
-								<a id="id_edit_<?php p($poll->getId()); ?>" class="icon-rename svg" href="<?php p($urlGenerator->linkToRoute('polls.page.edit_poll', ['hash' => $poll->getHash()])); ?>">
+								<a id="id_edit_<?php p($poll->getId()); ?>" class="icon-rename svg" href="<?php p($urlGenerator->linkToRoute('polls.page.edit_poll', [ 'hash' => $poll->getHash() ])); ?>">
 									<?php p($l->t('Edit Poll')); ?>
 								</a>
 							</li>

Braces +19 added lines, -7 removed lines

@@ -110,9 +110,12 @@ 
 				<span class="symbol icon-settings"></span>
 				<?php if (count($comments)) : ?>
 					<div id="comment-counter" class="badge"><?php p(count($comments)) ?></div>
-				<?php else: ?>
+				<?php else {
+    : ?>
 					<div id="comment-counter" class="badge no-comments"><?php p(count($comments)) ?></div>
-				<?php endif; ?>
+				<?php endif;
+}
+?>
 			</a>			
 		</div>
 		
@@ -331,7 +334,10 @@ 
 				</div>
 			<?php if (User::isLoggedIn()) : ?>
 				<div class="notification">
-					<input type="checkbox" id="check_notif" class="checkbox" <?php if ($notification != null) print_unescaped(' checked'); ?> />
+					<input type="checkbox" id="check_notif" class="checkbox" <?php if ($notification != null) {
+    print_unescaped(' checked');
+}
+?> />
 					<label for="check_notif"><?php p($l->t('Receive notification email on activity')); ?></label>
 				</div>
 			<?php endif; ?>
@@ -389,8 +395,11 @@ 
 							<a href="<?php p($urlGenerator->linkToRouteAbsolute('core.login.showLoginForm')); ?>"><?php p($l->t('Login')); ?></a>
 							<?php p($l->t('or')); ?>
 							<?php print_unescaped('<div id="id_ac_detected" class="column external current-user"><input type="text" name="user_name_comm" id="user_name_comm" placeholder="' . $l->t('Your name here') . '" /></div>'); ?>
-						<?php else: ?>
-							<?php p($l->t('Logged in as') . ' ' . $userId); ?>
+						<?php else {
+    : ?>
+							<?php p($l->t('Logged in as') . ' ' . $userId);
+}
+?>
 						<?php endif; ?>
 							<textarea id="commentBox" name="commentBox"></textarea>
 							<p>
@@ -424,8 +433,11 @@ 
 							</div>
 						</div>
 					<?php endforeach; ?>
-				<?php else : ?>
-					<?php p($l->t('No comments yet. Be the first.')); ?>
+				<?php else {
+    : ?>
+					<?php p($l->t('No comments yet. Be the first.'));
+}
+?>
 				<?php endif; ?>
 			</div>
 		</div>

templates/main.tmpl.php

Indentation +123 added lines, -123 removed lines

@@ -1,36 +1,36 @@ 
 <?php
-	/**
-	 * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
-	 *
-	 * @license GNU AGPL version 3 or any later version
-	 *
-	 *  This program is free software: you can redistribute it and/or modify
-	 *  it under the terms of the GNU Affero General Public License as
-	 *  published by the Free Software Foundation, either version 3 of the
-	 *  License, or (at your option) any later version.
-	 *
-	 *  This program is distributed in the hope that it will be useful,
-	 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
-	 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-	 *  GNU Affero General Public License for more details.
-	 *
-	 *  You should have received a copy of the GNU Affero General Public License
-	 *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-	 *
-	 */
+    /**
+     * @copyright Copyright (c) 2017 Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
+     *
+     * @license GNU AGPL version 3 or any later version
+     *
+     *  This program is free software: you can redistribute it and/or modify
+     *  it under the terms of the GNU Affero General Public License as
+     *  published by the Free Software Foundation, either version 3 of the
+     *  License, or (at your option) any later version.
+     *
+     *  This program is distributed in the hope that it will be useful,
+     *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+     *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+     *  GNU Affero General Public License for more details.
+     *
+     *  You should have received a copy of the GNU Affero General Public License
+     *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+     *
+     */
 
-	use OCP\User;
+    use OCP\User;
 
-	\OCP\Util::addStyle('polls', 'main');
-	\OCP\Util::addStyle('polls', 'list');
-	\OCP\Util::addScript('polls', 'app');
-	\OCP\Util::addScript('polls', 'start');
+    \OCP\Util::addStyle('polls', 'main');
+    \OCP\Util::addStyle('polls', 'list');
+    \OCP\Util::addScript('polls', 'app');
+    \OCP\Util::addScript('polls', 'start');
 
-	$userId = $_['userId'];
-	$userMgr = $_['userMgr'];
-	$urlGenerator = $_['urlGenerator'];
+    $userId = $_['userId'];
+    $userMgr = $_['userMgr'];
+    $urlGenerator = $_['urlGenerator'];
 ?>
 
 <div id="app">
@@ -86,61 +86,61 @@ 
 
 				<?php foreach ($_['polls'] as $poll) : ?>
 					<?php
-						if (!userHasAccess($poll, $userId)) continue;
-						// direct url to poll
-						$pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', array('hash' => $poll->getHash()));
-						$owner = $poll->getOwner();
+                        if (!userHasAccess($poll, $userId)) continue;
+                        // direct url to poll
+                        $pollUrl = $urlGenerator->linkToRouteAbsolute('polls.page.goto_poll', array('hash' => $poll->getHash()));
+                        $owner = $poll->getOwner();
 
-						$expiry_style = '';
-						if ($poll->getType() == '0') {
-							$participated = $_['participations'];
-						} else {
-							$participated = $_['participations_text'];
-						}
-						$participated_class = 'partic_no';
-						$participated_title = 'You did not vote';
-						$participated_count = count($participated);
+                        $expiry_style = '';
+                        if ($poll->getType() == '0') {
+                            $participated = $_['participations'];
+                        } else {
+                            $participated = $_['participations_text'];
+                        }
+                        $participated_class = 'partic_no';
+                        $participated_title = 'You did not vote';
+                        $participated_count = count($participated);
 
-						$comments = $_['comments'];
-						$commented_class = 'commented_no';
-						$commented_title = 'You did not comment';
-						$commented_count = count($comments);
+                        $comments = $_['comments'];
+                        $commented_class = 'commented_no';
+                        $commented_title = 'You did not comment';
+                        $commented_count = count($comments);
 
-						if ($owner == $userId) {
-							$owner = $l->t('Yourself');
-						}
+                        if ($owner == $userId) {
+                            $owner = $l->t('Yourself');
+                        }
 
-						if ($poll->getExpire() != null) {
-							// $expiry_date = date('d.m.Y', strtotime($poll->getExpire()));
-							$expiry_date = OCP\relative_modified_date(strtotime($poll->getExpire())); // does not work, because relative_modified_date seems not to recognise future time diffs
-							$expiry_style = ' progress';
-							if (date('U') > strtotime($poll->getExpire())) {
-								$expiry_date = OCP\relative_modified_date(strtotime($poll->getExpire()));
-								$expiry_style = ' expired';
-							}
-						} else {
-							$expiry_style = ' endless';
-							$expiry_date = $l->t('Never');
-						}
+                        if ($poll->getExpire() != null) {
+                            // $expiry_date = date('d.m.Y', strtotime($poll->getExpire()));
+                            $expiry_date = OCP\relative_modified_date(strtotime($poll->getExpire())); // does not work, because relative_modified_date seems not to recognise future time diffs
+                            $expiry_style = ' progress';
+                            if (date('U') > strtotime($poll->getExpire())) {
+                                $expiry_date = OCP\relative_modified_date(strtotime($poll->getExpire()));
+                                $expiry_style = ' expired';
+                            }
+                        } else {
+                            $expiry_style = ' endless';
+                            $expiry_date = $l->t('Never');
+                        }
 
-						for ($i = 0; $i < count($participated); $i++) {
-							if ($poll->getId() == intval($participated[$i]->getPollId())) {
-								$participated_class = 'partic_yes';
-								$participated_title = 'You voted';
-								array_splice($participated, $i, 1);
-								break;
-							}
-						}
+                        for ($i = 0; $i < count($participated); $i++) {
+                            if ($poll->getId() == intval($participated[$i]->getPollId())) {
+                                $participated_class = 'partic_yes';
+                                $participated_title = 'You voted';
+                                array_splice($participated, $i, 1);
+                                break;
+                            }
+                        }
 
-						for ($i = 0; $i < count($comments); $i++) {
-							if ($poll->getId() == intval($comments[$i]->getPollId())) {
-								$commented_class = 'commented_yes';
-								$commented_title = 'You commented';
-								array_splice($comments, $i, 1);
-								break;
-							}
-						}
-					?>
+                        for ($i = 0; $i < count($comments); $i++) {
+                            if ($poll->getId() == intval($comments[$i]->getPollId())) {
+                                $commented_class = 'commented_yes';
+                                $commented_title = 'You commented';
+                                array_splice($comments, $i, 1);
+                                break;
+                            }
+                        }
+                    ?>
 
 
 					<div class="row table-body">
@@ -217,16 +217,16 @@ 
  * @return array with groups
  */
 function getGroups($userId) {
-	// $this->requireLogin();
-	if (class_exists('\OC_Group', true)) {
-		// Nextcloud <= 11, ownCloud
-		return \OC_Group::getUserGroups($userId);
-	}
-	// Nextcloud >= 12
-	$groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
-	return array_map(function ($group) {
-		return $group->getGID();
-	}, $groups);
+    // $this->requireLogin();
+    if (class_exists('\OC_Group', true)) {
+        // Nextcloud <= 11, ownCloud
+        return \OC_Group::getUserGroups($userId);
+    }
+    // Nextcloud >= 12
+    $groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
+    return array_map(function ($group) {
+        return $group->getGID();
+    }, $groups);
 }
 
 /**
@@ -235,42 +235,42 @@ 
  * @return boolean 
  */
 function userHasAccess($poll, $userId) {
-	if ($poll == null) {
-		return false;
-	}
-	$access = $poll->getAccess();
-	$owner = $poll->getOwner();
-	if (!User::isLoggedIn()) {
-		return false;
-	}
-	if ($access == 'public') {
-		return true;
-	}
-	if ($access == 'hidden') {
-		return true;
-	}
-	if ($access == 'registered') {
-		return true;
-	}
-	if ($owner == $userId) {
-		return true;
-	}
-	$user_groups = getGroups($userId);
+    if ($poll == null) {
+        return false;
+    }
+    $access = $poll->getAccess();
+    $owner = $poll->getOwner();
+    if (!User::isLoggedIn()) {
+        return false;
+    }
+    if ($access == 'public') {
+        return true;
+    }
+    if ($access == 'hidden') {
+        return true;
+    }
+    if ($access == 'registered') {
+        return true;
+    }
+    if ($owner == $userId) {
+        return true;
+    }
+    $user_groups = getGroups($userId);
 
-	$arr = explode(';', $access);
+    $arr = explode(';', $access);
 
-	foreach ($arr as $item) {
-		if (strpos($item, 'group_') == 0) {
-			$grp = substr($item, 6);
-			foreach ($user_groups as $user_group) {
-				if ($user_group == $grp) return true;
-			}
-		}
-		else if (strpos($item, 'user_') == 0) {
-			$usr = substr($item, 5);
-			if ($usr == $userId) return true;
-		}
-	}
-	return false;
+    foreach ($arr as $item) {
+        if (strpos($item, 'group_') == 0) {
+            $grp = substr($item, 6);
+            foreach ($user_groups as $user_group) {
+                if ($user_group == $grp) return true;
+            }
+        }
+        else if (strpos($item, 'user_') == 0) {
+            $usr = substr($item, 5);
+            if ($usr == $userId) return true;
+        }
+    }
+    return false;
 }
 ?>

Spacing +13 added lines, -13 removed lines

@@ -28,9 +28,9 @@ 
 	\OCP\Util::addScript('polls', 'app');
 	\OCP\Util::addScript('polls', 'start');
 
-	$userId = $_['userId'];
-	$userMgr = $_['userMgr'];
-	$urlGenerator = $_['urlGenerator'];
+	$userId = $_[ 'userId' ];
+	$userMgr = $_[ 'userMgr' ];
+	$urlGenerator = $_[ 'urlGenerator' ];
 ?>
 
 <div id="app">
@@ -51,7 +51,7 @@ 
 						</div>
 					</div>
 				</div>
-	<?php if (count($_['polls']) == 0) : ?>
+	<?php if (count($_[ 'polls' ]) == 0) : ?>
 		<div id="emptycontent" class="">
 			<div class="icon-polls"></div>
 			<h2><?php p($l->t('No existing polls.')); ?></h2>
@@ -84,7 +84,7 @@ 
 					 </div>
 				</div>
 
-				<?php foreach ($_['polls'] as $poll) : ?>
+				<?php foreach ($_[ 'polls' ] as $poll) : ?>
 					<?php
 						if (!userHasAccess($poll, $userId)) continue;
 						// direct url to poll
@@ -93,15 +93,15 @@ 
 
 						$expiry_style = '';
 						if ($poll->getType() == '0') {
-							$participated = $_['participations'];
+							$participated = $_[ 'participations' ];
 						} else {
-							$participated = $_['participations_text'];
+							$participated = $_[ 'participations_text' ];
 						}
 						$participated_class = 'partic_no';
 						$participated_title = 'You did not vote';
 						$participated_count = count($participated);
 
-						$comments = $_['comments'];
+						$comments = $_[ 'comments' ];
 						$commented_class = 'commented_no';
 						$commented_title = 'You did not comment';
 						$commented_count = count($comments);
@@ -124,7 +124,7 @@ 
 						}
 
 						for ($i = 0; $i < count($participated); $i++) {
-							if ($poll->getId() == intval($participated[$i]->getPollId())) {
+							if ($poll->getId() == intval($participated[ $i ]->getPollId())) {
 								$participated_class = 'partic_yes';
 								$participated_title = 'You voted';
 								array_splice($participated, $i, 1);
@@ -133,7 +133,7 @@ 
 						}
 
 						for ($i = 0; $i < count($comments); $i++) {
-							if ($poll->getId() == intval($comments[$i]->getPollId())) {
+							if ($poll->getId() == intval($comments[ $i ]->getPollId())) {
 								$commented_class = 'commented_yes';
 								$commented_title = 'You commented';
 								array_splice($comments, $i, 1);
@@ -146,7 +146,7 @@ 
 					<div class="row table-body">
 						<div class="wrapper group-master">
 							<div class="wrapper group-1">
-								<div class="thumbnail <?php p($expiry_style . " " . $commented_class. " " . $participated_class); ?>"></div>  <!-- Image to display status or type of poll */ -->
+								<div class="thumbnail <?php p($expiry_style . " " . $commented_class . " " . $participated_class); ?>"></div>  <!-- Image to display status or type of poll */ -->
 								<a href="<?php p($pollUrl); ?>" class="wrapper group-1-1">
 									<div class="column name">						  <?php p($poll->getTitle()); ?></div>
 									<div class="column description">				   <?php p($poll->getDescription()); ?></div>
@@ -169,7 +169,7 @@ 
 												</a>
 											</li>
 											<li>
-												<a id="id_edit_<?php p($poll->getId()); ?>" class="menuitem action permanent" href="<?php p($urlGenerator->linkToRoute('polls.page.edit_poll', ['hash' => $poll->getHash()])); ?>">
+												<a id="id_edit_<?php p($poll->getId()); ?>" class="menuitem action permanent" href="<?php p($urlGenerator->linkToRoute('polls.page.edit_poll', [ 'hash' => $poll->getHash() ])); ?>">
 													<span class="icon-rename"></span>
 													<span>Edit Poll</span>
 												</a>
@@ -224,7 +224,7 @@ 
 	}
 	// Nextcloud >= 12
 	$groups = \OC::$server->getGroupManager()->getUserGroups(\OC::$server->getUserSession()->getUser());
-	return array_map(function ($group) {
+	return array_map(function($group) {
 		return $group->getGID();
 	}, $groups);
 }