nextcloud / polls

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/EventFactory.php

Spacing +2 added lines, -2 removed lines

@@ -31,12 +31,12 @@
 	'title' => Faker::sentence(10),
 	'description' => Faker::text(255),
 	'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');
 	},

templates/main.tmpl.php

Braces +6 added lines, -4 removed lines

@@ -60,14 +60,17 @@ 
 			<div class="icon-polls"></div>
 			<h2><?php p($l->t('No existing polls.')); ?></h2>
 		</div>
-	<?php else : ?>
+	<?php else {
+	: ?>
 			<div class="table main-container has-controls">
 				<div class ="table-row table-header">
 
 					<div class="wrapper group-master">
 						<div class="wrapper group-1">
 							<div class="wrapper group-1-1">
-								<div class="flex-column name">		<?php p($l->t('Title')); ?></div>
+								<div class="flex-column name">		<?php p($l->t('Title'));
+}
+?></div>
 							</div>
 							<div class="wrapper group-1-2">
 								<div class="flex-column actions"></div>
@@ -265,8 +268,7 @@ 
 					return true;
 				}
 			}
-		}
-		else if (strpos($item, 'user_') === 0) {
+		} else if (strpos($item, 'user_') === 0) {
 			$usr = substr($item, 5);
 			if ($usr === $userId) {
 				return true;

Spacing +4 added lines, -4 removed lines

@@ -151,7 +151,7 @@ 
 					<div class="table-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 the status or type of poll -->
+								<div class="thumbnail <?php p($expiry_style . ' ' . $commented_class . ' ' . $participated_class); ?>"></div><!-- Image to display the status or type of poll -->
 								<a href="<?php p($pollUrl); ?>" class="wrapper group-1-1">
 									<div class="flex-column name">						  <?php p($poll->getTitle()); ?></div>
 									<div class="flex-column description">				   <?php p($poll->getDescription()); ?></div>
@@ -193,10 +193,10 @@ 
 								</div>
 								<div class="wrapper group-2-1">
 									<div class="flex-column access"><?php p($l->t($poll->getAccess())); ?></div>
-									<div class="flex-column created has-tooltip live-relative-timestamp" data-timestamp="<?php p(strtotime($poll->getCreated())*1000); ?>" data-value="<?php p($poll->getCreated()); ?>"><?php p(\OCP\Template::relative_modified_date(strtotime($poll->getCreated()))); ?></div>
+									<div class="flex-column created has-tooltip live-relative-timestamp" data-timestamp="<?php p(strtotime($poll->getCreated()) * 1000); ?>" data-value="<?php p($poll->getCreated()); ?>"><?php p(\OCP\Template::relative_modified_date(strtotime($poll->getCreated()))); ?></div>
 								</div>
 								<div class="wrapper group-2-2">
-									<div class="flex-column has-tooltip expiry<?php p($expiry_style . $timestamp_style); ?>" data-timestamp="<?php p(strtotime($poll->getExpire())*1000); ?>" data-value="<?php p($poll->getExpire()); ?>"> <?php p($expiry_date); ?></div>
+									<div class="flex-column has-tooltip expiry<?php p($expiry_style . $timestamp_style); ?>" data-timestamp="<?php p(strtotime($poll->getExpire()) * 1000); ?>" data-value="<?php p($poll->getExpire()); ?>"> <?php p($expiry_date); ?></div>
 									<div class="flex-column participants">
 										<div class="symbol alt-tooltip partic_voted icon-<?php p($participated_class); ?>" title="<?php p($participated_title); ?>"></div>
 										<div class="symbol alt-tooltip partic_commented icon-<?php p($commented_class); ?>" title="<?php p($commented_title); ?>"></div>
@@ -228,7 +228,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);
 }

templates/goto.tmpl.php

Braces +19 added lines, -7 removed lines

@@ -118,9 +118,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>
 
@@ -333,7 +336,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; ?>
@@ -431,8 +437,11 @@ 
 						<div class="author"><?php p($userMgr->get($userId)->getDisplayName()) ?></div>
 					</div>
 
-				<?php else: ?>
-					<a href="<?php p($urlGenerator->linkToRouteAbsolute('core.login.showLoginForm')); ?>"><?php p($l->t('Login or ...')); ?></a>
+				<?php else {
+	: ?>
+					<a href="<?php p($urlGenerator->linkToRouteAbsolute('core.login.showLoginForm'));
+}
+?>"><?php p($l->t('Login or ...')); ?></a>
 					<div class="authorRow user-cell flex-row">
 						<div class="avatar has-tooltip" title="?"></div>
 						<div id="id_ac_detected" class="author  flex-column external">
@@ -453,9 +462,12 @@ 
 
 				<?php if ($comments == null) : ?>
 					<li id="no-comments" class="emptycontent">
-				<?php else : ?>
+				<?php else {
+	: ?>
 					<li id="no-comments" class="emptycontent hidden">
-				<?php endif; ?>
+				<?php endif;
+}
+?>
 
 						<div class="icon-comment"></div>
 						<p><?php p($l->t('No comments yet. Be the first.')); ?></p>

Spacing +2 added lines, -2 removed lines

@@ -271,7 +271,7 @@ 
 							
 							if ($class === 'unvoted') {
 								$dataUnvoted = $l->t('New option!');
-								$updatedPoll=true;
+								$updatedPoll = true;
 							}
 							
 							print_unescaped('<li id="voteid_' . $optionElement->getId() . '" class="flex-column active poll-cell ' . $class . '" data-value="' . $optionElement->getPollOptionText() . '" data-unvoted="' . $dataUnvoted . '"></li>');
@@ -286,7 +286,7 @@ 
 			</div>
 			<?php if ($updatedPoll) : ?>
 				<div class="updated-poll alert">
-				<p> <?php p($l->t('This poll was updated since your last visit. Please check your votes.'));?></p>
+				<p> <?php p($l->t('This poll was updated since your last visit. Please check your votes.')); ?></p>
 				</div>
 			<?php endif; ?>
 			

templates/create.tmpl.php

Spacing +1 added lines, -1 removed lines

@@ -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...'); ?>" />

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="chosenOptions" id="chosenOptions" 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="chosenOptions" id="chosenOptions" 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>

lib/Controller/PageController.php

Spacing +1 added lines, -1 removed lines

@@ -703,7 +703,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);
 	}

lib/Migration/Version009000Date20171202105141.php

Indentation +3 added lines, -3 removed lines

@@ -219,7 +219,7 @@ 
 	}
 
 	/**
-	* Copy date options 
+	 * Copy date options 
 	 */
 	protected function copyDateOptions() {
 		$insert = $this->connection->getQueryBuilder();
@@ -246,7 +246,7 @@ 
 	}
 
 	/**
-	* Copy text options 
+	 * Copy text options 
 	 */
 	protected function copyTextOptions() {
 		$insert = $this->connection->getQueryBuilder();
@@ -273,7 +273,7 @@ 
 	}
 
 	/**
-	* Copy date votes 
+	 * Copy date votes 
 	 */
 	protected function copyDateVotes() {
 		$insert = $this->connection->getQueryBuilder();

Spacing +3 added lines, -3 removed lines

@@ -121,7 +121,7 @@ 
 				'notnull' => false,
 			]);
 			$table->addColumn('poll_option_text', Type::STRING, [
-				'notnull' => false,  // maybe true?
+				'notnull' => false, // maybe true?
 				'length' => 256,
 			]);
 			$table->setPrimaryKey(['id']);
@@ -351,8 +351,8 @@ 
 			->from('polls_options')
 			// ->where($queryFind->expr()->eq('poll_id', $pollId))
 			// ->andWhere($queryFind->expr()->eq('poll_option', $text));
-			->where('poll_id = "'. $pollId .'"')
-			->andWhere('poll_option_text ="' .$text .'"');
+			->where('poll_id = "' . $pollId . '"')
+			->andWhere('poll_option_text ="' . $text . '"');
 
 		$resultFind = $queryFind->execute();
 		$row = $resultFind->fetch();

lib/AppInfo/Application.php

Spacing +12 added lines, -12 removed lines

@@ -47,7 +47,7 @@ 
 		/**
 		 * Controllers
 		 */
-		$container->registerService('PageController', function (IContainer $c) {
+		$container->registerService('PageController', function(IContainer $c) {
 			return new PageController(
 				$c->query('AppName'),
 				$c->query('Request'),
@@ -66,51 +66,51 @@ 
 			);
 		});
 
-		$container->registerService('UserManager', function (IContainer $c) {
+		$container->registerService('UserManager', function(IContainer $c) {
 			return $c->query('ServerContainer')->getUserManager();
 		});
 
-		$container->registerService('GroupManager', function (IContainer $c) {
+		$container->registerService('GroupManager', function(IContainer $c) {
 			return $c->query('ServerContainer')->getGroupManager();
 		});
 
-		$container->registerService('AvatarManager', function (IContainer $c) {
+		$container->registerService('AvatarManager', function(IContainer $c) {
 			return $c->query('ServerContainer')->getAvatarManager();
 		});
 
-		$container->registerService('Logger', function (IContainer $c) {
+		$container->registerService('Logger', function(IContainer $c) {
 			return $c->query('ServerContainer')->getLogger();
 		});
 
-		$container->registerService('L10N', function (IContainer $c) {
+		$container->registerService('L10N', function(IContainer $c) {
 			return $c->query('ServerContainer')->getL10N($c->query('AppName'));
 		});
 
-		$container->registerService('CommentMapper', function (IContainer $c) use ($server) {
+		$container->registerService('CommentMapper', function(IContainer $c) use ($server) {
 			return new CommentMapper(
 				$server->getDatabaseConnection()
 			);
 		});
 
-		$container->registerService('OptionsMapper', function (IContainer $c) use ($server) {
+		$container->registerService('OptionsMapper', function(IContainer $c) use ($server) {
 			return new OptionsMapper(
 				$server->getDatabaseConnection()
 			);
 		});
 
-		$container->registerService('EventMapper', function (IContainer $c) use ($server) {
+		$container->registerService('EventMapper', function(IContainer $c) use ($server) {
 			return new EventMapper(
 				$server->getDatabaseConnection()
 			);
 		});
 
-		$container->registerService('NotificationMapper', function (IContainer $c) use ($server) {
+		$container->registerService('NotificationMapper', function(IContainer $c) use ($server) {
 			return new NotificationMapper(
 				$server->getDatabaseConnection()
 			);
 		});
 
-		$container->registerService('VotesMapper', function (IContainer $c) use ($server) {
+		$container->registerService('VotesMapper', function(IContainer $c) use ($server) {
 			return new VotesMapper(
 				$server->getDatabaseConnection()
 			);
@@ -122,7 +122,7 @@ 
 	 */
 	public function registerNavigationEntry() {
 		$container = $this->getContainer();
-		$container->query('OCP\INavigationManager')->add(function () use ($container) {
+		$container->query('OCP\INavigationManager')->add(function() use ($container) {
 			$urlGenerator = $container->query('OCP\IURLGenerator');
 			$l10n = $container->query('OCP\IL10N');
 			return [