nextcloud / polls

lib/Db/Model.php

Spacing +1 added lines, -1 removed lines

@@ -34,6 +34,6 @@
 	 * @param $value
 	 */
 	public function __set($name, $value) {
-		$this->setter($name, [$value]);
+		$this->setter($name, [ $value ]);
 	}
 }

tests/Unit/Factories/CommentFactory.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
  */
 $fm->define('OCA\Polls\Db\Comment')->setDefinitions([
 	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	},

tests/Unit/Factories/ParticipationFactory.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
  */
 $fm->define('OCA\Polls\Db\Participation')->setDefinitions([
 	'userId' => Faker::firstNameMale(),
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	},

tests/Unit/Factories/DateFactory.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
  * General factory for the date model.
  */
 $fm->define('OCA\Polls\Db\Date')->setDefinitions([
-	'dt' => function () {
+	'dt' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	}

templates/create.tmpl.php

Braces +68 added lines, -20 removed lines

@@ -85,8 +85,11 @@ 
 						<span>
 						<?php if ($isUpdate): ?>
 							<?php p($l->t('Edit poll') . ' ' . $poll->getTitle()); ?>
-						<?php else: ?>
-						  <?php p($l->t('Create new poll')); ?>
+						<?php else {
+    : ?>
+						  <?php p($l->t('Create new poll'));
+}
+?>
 						<?php endif; ?>
 						</span>
 					</div>
@@ -96,11 +99,20 @@ 
 		<?php if ($isUpdate): ?>
 			<form name="finish_poll" action="<?php p($urlGenerator->linkToRoute('polls.page.update_poll')); ?>" method="POST">
 				<input type="hidden" name="pollId" value="<?php p($poll->getId()); ?>" />
-		<?php else: ?>
-			<form name="finish_poll" action="<?php p($urlGenerator->linkToRoute('polls.page.insert_poll')); ?>" method="POST">
+		<?php else {
+    : ?>
+			<form name="finish_poll" action="<?php p($urlGenerator->linkToRoute('polls.page.insert_poll'));
+}
+?>" method="POST">
 		<?php endif; ?>
-				<input type="hidden" name="chosenDates" id="chosenDates" value="<?php if (isset($chosen)) p($chosen); ?>" />
-				<input type="hidden" name="expireTs" id="expireTs" value="<?php if (isset($expireTs)) p($expireTs); ?>" />
+				<input type="hidden" name="chosenDates" id="chosenDates" value="<?php if (isset($chosen)) {
+    p($chosen);
+}
+?>" />
+				<input type="hidden" name="expireTs" id="expireTs" value="<?php if (isset($expireTs)) {
+    p($expireTs);
+}
+?>" />
 				<input type="hidden" name="userId" id="userId" value="<?php p($userId); ?>" />
 
 				<header class="row">
@@ -109,22 +121,40 @@ 
 				<div class="new_poll row">
 					<div class="col-50">
 						<label for="pollTitle" class="input_title"><?php p($l->t('Title')); ?></label>
-						<input type="text" class="input_field" id="pollTitle" name="pollTitle" value="<?php if (isset($title)) p($title); ?>" />
+						<input type="text" class="input_field" id="pollTitle" name="pollTitle" value="<?php if (isset($title)) {
+    p($title);
+}
+?>" />
 						<label for="pollDesc" class="input_title"><?php p($l->t('Description')); ?></label>
-						<textarea class="input_field" id="pollDesc" name="pollDesc"><?php if (isset($desc)) p($desc); ?></textarea>
+						<textarea class="input_field" id="pollDesc" name="pollDesc"><?php if (isset($desc)) {
+    p($desc);
+}
+?></textarea>
 
 						<label class="input_title"><?php p($l->t('Access')); ?></label>
 
-						<input type="radio" name="accessType" id="private" value="registered" class="radio" <?php if (!$isUpdate || $access === 'registered') print_unescaped('checked'); ?> />
+						<input type="radio" name="accessType" id="private" value="registered" class="radio" <?php if (!$isUpdate || $access === 'registered') {
+    print_unescaped('checked');
+}
+?> />
 						<label for="private"><?php p($l->t('Registered users only')); ?></label>
 
-						<input type="radio" name="accessType" id="hidden" value="hidden" class="radio" <?php if ($isUpdate && $access === 'hidden') print_unescaped('checked'); ?> />
+						<input type="radio" name="accessType" id="hidden" value="hidden" class="radio" <?php if ($isUpdate && $access === 'hidden') {
+    print_unescaped('checked');
+}
+?> />
 						<label for="hidden"><?php p($l->t('hidden')); ?></label>
 
-						<input type="radio" name="accessType" id="public" value="public" class="radio" <?php if ($isUpdate && $access === 'public') print_unescaped('checked'); ?> />
+						<input type="radio" name="accessType" id="public" value="public" class="radio" <?php if ($isUpdate && $access === 'public') {
+    print_unescaped('checked');
+}
+?> />
 						<label for="public"><?php p($l->t('Public access')); ?></label>
 
-						<input type="radio" name="accessType" id="select" value="select" class="radio" <?php if ($isUpdate && $access === 'select') print_unescaped('checked'); ?>>
+						<input type="radio" name="accessType" id="select" value="select" class="radio" <?php if ($isUpdate && $access === 'select') {
+    print_unescaped('checked');
+}
+?>>
 						<label for="select"><?php p($l->t('Select')); ?></label>
 						<span id="id_label_select">...</span>
 
@@ -140,9 +170,12 @@ 
 							</div>
 						</div>
 
-						<input type="hidden" name="accessValues" id="accessValues" value="<?php if ($isUpdate && $access === 'select') p($accessTypes) ?>" />
+						<input type="hidden" name="accessValues" id="accessValues" value="<?php if ($isUpdate && $access === 'select') {
+    p($accessTypes) ?>" />
 
-						<input id="isAnonymous" name="isAnonymous" type="checkbox" class="checkbox" <?php $isAnonymous ? print_unescaped('value="true" checked') : print_unescaped('value="false"'); ?> />
+						<input id="isAnonymous" name="isAnonymous" type="checkbox" class="checkbox" <?php $isAnonymous ? print_unescaped('value="true" checked') : print_unescaped('value="false"');
+}
+?> />
 						<label for="isAnonymous" class="input_title"><?php p($l->t('Anonymous')) ?></label>
 
 						<div id="anonOptions" style="display:none;">
@@ -158,20 +191,32 @@ 
 					</div>
 					<div class="col-50">
 
-						<input type="radio" name="pollType" id="event" value="event" class="radio" <?php if (!$isUpdate || $poll->getType() === 0) print_unescaped('checked'); ?> />
+						<input type="radio" name="pollType" id="event" value="event" class="radio" <?php if (!$isUpdate || $poll->getType() === 0) {
+    print_unescaped('checked');
+}
+?> />
 						<label for="event"><?php p($l->t('Event schedule')); ?></label>
 
 						<!-- TODO texts to db -->
-						<input type="radio" name="pollType" id="text" value="text" class="radio" <?php if ($isUpdate && $poll->getType() === 1) print_unescaped('checked'); ?>>
+						<input type="radio" name="pollType" id="text" value="text" class="radio" <?php if ($isUpdate && $poll->getType() === 1) {
+    print_unescaped('checked');
+}
+?>>
 						<label for="text"><?php p($l->t('Text based')); ?></label>
 
-						<div id="date-select-container" <?php if ($isUpdate && $poll->getType() === 1) print_unescaped('style="display:none;"'); ?> >
+						<div id="date-select-container" <?php if ($isUpdate && $poll->getType() === 1) {
+    print_unescaped('style="display:none;"');
+}
+?> >
 							<label for="datetimepicker" class="input_title"><?php p($l->t('Dates')); ?></label>
 							<input id="datetimepicker" type="text" />
 							<table id="selected-dates-table" class="choices">
 							</table>
 						</div>
-						<div id="text-select-container" <?php if(!$isUpdate || $poll->getType() === 0) print_unescaped('style="display:none;"'); ?> >
+						<div id="text-select-container" <?php if(!$isUpdate || $poll->getType() === 0) {
+    print_unescaped('style="display:none;"');
+}
+?> >
 							<label for="text-title" class="input_title"><?php p($l->t('Text item')); ?></label>
 							<div class="input-group">
 								<input type="text" id="text-title" placeholder="<?php print_unescaped('Insert text...'); ?>" />
@@ -187,8 +232,11 @@ 
 				<div class="form-actions">
 					<?php if ($isUpdate): ?>
 						<input type="submit" id="submit_finish_poll" class="button btn primary" value="<?php p($l->t('Update poll')); ?>" />
-					<?php else: ?>
-						<input type="submit" id="submit_finish_poll" class="button btn primary" value="<?php p($l->t('Create poll')); ?>" />
+					<?php else {
+    : ?>
+						<input type="submit" id="submit_finish_poll" class="button btn primary" value="<?php p($l->t('Create poll'));
+}
+?>" />
 					<?php endif; ?>
 					<a href="<?php p($urlGenerator->linkToRoute('polls.page.index')); ?>" id="submit_cancel_poll" class="button"><?php p($l->t('Cancel')); ?></a>
 				</div>

Spacing +7 added lines, -7 removed lines

@@ -27,22 +27,22 @@ 
 	\OCP\Util::addScript('polls', 'create_edit');
 	\OCP\Util::addScript('polls', 'vendor/jquery.datetimepicker.full.min');
 
-	$userId = $_['userId'];
+	$userId = $_[ 'userId' ];
 	/** @var \OCP\IUserManager $userMgr */
-	$userMgr = $_['userMgr'];
+	$userMgr = $_[ 'userMgr' ];
 	/** @var \OCP\IURLGenerator $urlGenerator */
-	$urlGenerator = $_['urlGenerator'];
-	$isUpdate = isset($_['poll']) && $_['poll'] !== null;
+	$urlGenerator = $_[ 'urlGenerator' ];
+	$isUpdate = isset($_[ 'poll' ]) && $_[ 'poll' ] !== null;
 	$isAnonymous = false;
 	$hideNames = false;
 
 	if ($isUpdate) {
 		/** @var OCA\Polls\Db\Event $poll */
-		$poll = $_['poll'];
+		$poll = $_[ 'poll' ];
 		$isAnonymous = $poll->getIsAnonymous();
 		$hideNames = $isAnonymous && $poll->getFullAnonymous();
 		/** @var OCA\Polls\Db\Date[]|OCA\Polls\Db\Text[] $dates */
-		$dates = $_['dates'];
+		$dates = $_[ 'dates' ];
 		$chosen = '[';
 		foreach ($dates as $d) {
 			if ($poll->getType() === 0) {
@@ -171,7 +171,7 @@ 
 							<table id="selected-dates-table" class="choices">
 							</table>
 						</div>
-						<div id="text-select-container" <?php if(!$isUpdate || $poll->getType() === 0) print_unescaped('style="display:none;"'); ?> >
+						<div id="text-select-container" <?php if (!$isUpdate || $poll->getType() === 0) print_unescaped('style="display:none;"'); ?> >
 							<label for="text-title" class="input_title"><?php p($l->t('Text item')); ?></label>
 							<div class="input-group">
 								<input type="text" id="text-title" placeholder="<?php print_unescaped('Insert text...'); ?>" />

lib/Db/ParticipationTextMapper.php

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);
 	}
 
 	/**
@@ -66,7 +66,7 @@ 
 	 */
 	public function findParticipantsByPoll($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);
 	}
 
 	/**
@@ -74,7 +74,7 @@ 
 	 */
 	public function deleteByPoll($pollId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
+		$this->execute($sql, [ $pollId ]);
 	}
 
 	/**
@@ -83,6 +83,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/ParticipationMapper.php

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);
 	}
 
 	/**
@@ -66,7 +66,7 @@ 
 	 */
 	public function findParticipantsByPoll($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);
 	}
 
 	/**
@@ -74,7 +74,7 @@ 
 	 */
 	public function deleteByPoll($pollId) {
 		$sql = 'DELETE FROM ' . $this->getTableName() . ' WHERE poll_id = ?';
-		$this->execute($sql, [$pollId]);
+		$this->execute($sql, [ $pollId ]);
 	}
 
 	/**
@@ -83,6 +83,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/Controller/PageController.php

Spacing +14 added lines, -14 removed lines

@@ -223,7 +223,7 @@ 
 		try {
 			$poll = $this->eventMapper->findByHash($hash);
 		} catch (DoesNotExistException $e) {
-			return new TemplateResponse('polls', 'no.acc.tmpl', []);
+			return new TemplateResponse('polls', 'no.acc.tmpl', [ ]);
 		}
 		if ($poll->getType() === 0) {
 			$dates = $this->dateMapper->findByPoll($poll->getId());
@@ -255,7 +255,7 @@ 
 			]);
 		} else {
 			User::checkLoggedIn();
-			return new TemplateResponse('polls', 'no.acc.tmpl', []);
+			return new TemplateResponse('polls', 'no.acc.tmpl', [ ]);
 		}
 	}
 
@@ -406,7 +406,7 @@ 
 	 */
 	public function createPoll() {
 		return new TemplateResponse('polls', 'create.tmpl',
-			['userId' => $this->userId, 'userMgr' => $this->userMgr, 'urlGenerator' => $this->urlGenerator]);
+			[ 'userId' => $this->userId, 'userMgr' => $this->userMgr, 'urlGenerator' => $this->urlGenerator ]);
 	}
 
 	/**
@@ -560,15 +560,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);
 				}
 
@@ -576,7 +576,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);
 	}
 
@@ -633,15 +633,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;
 	}
@@ -660,7 +660,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;
@@ -672,7 +672,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;
 			}
@@ -700,7 +700,7 @@ 
 		}
 		// Nextcloud >= 12
 		$groups = $this->groupManager->getUserGroups(\OC::$server->getUserSession()->getUser());
-		return array_map(function ($group) {
+		return array_map(function($group) {
 			return $group->getGID();
 		}, $groups);
 	}

tests/Unit/Factories/EventFactory.php

Spacing +2 added lines, -2 removed lines

@@ -31,12 +31,12 @@
 	'title' => Faker::sentence(10),
 	'description' => Faker::text(256),
 	'owner' => Faker::firstNameMale(),
-	'created' => function () {
+	'created' => function() {
 		$date = new DateTime('today');
 		return $date->format('Y-m-d H:i:s');
 	},
 	'access' => 'registered',
-	'expire' => function () {
+	'expire' => function() {
 		$date = new DateTime('tomorrow');
 		return $date->format('Y-m-d H:i:s');
 	},