nextcloud / polls

lib/Model/Acl.php

Indentation +1 added lines, -1 removed lines

@@ -295,7 +295,7 @@
 	/**
 	 * @NoAdminRequired
 	 * @return string
-	*/
+	 */
 	public function getAccessLevel(): string {
 		if ($this->getIsOwner()) {
 			return 'owner';

Spacing +3 added lines, -3 removed lines

@@ -182,7 +182,7 @@ 
 	public function getGroupShare(): bool {
 		return count(
 			array_filter($this->shareMapper->findByPoll($this->getPollId()), function($item) {
-				if ($item->getType() === 'group' && $this->groupManager->isInGroup($this->getUserId(),$item->getUserId())) {
+				if ($item->getType() === 'group' && $this->groupManager->isInGroup($this->getUserId(), $item->getUserId())) {
 					return true;
 				}
 			})
@@ -252,7 +252,7 @@ 
 	 * @return bool
 	 */
 	public function getAllowSeeUsernames(): bool {
-		return !(($this->poll->getAnonymous() && !$this->getIsOwner()) || $this->poll->getFullAnonymous());;
+		return !(($this->poll->getAnonymous() && !$this->getIsOwner()) || $this->poll->getFullAnonymous()); ;
 	}
 
 	/**
@@ -296,7 +296,7 @@ 
 			$this->foundByToken = true;
 			$this->setPollId($share->getPollId());
 
-			if (($share->getType() === 'group' || $share->getType() === 'user')  && !\OC::$server->getUserSession()->isLoggedIn()) {
+			if (($share->getType() === 'group' || $share->getType() === 'user') && !\OC::$server->getUserSession()->isLoggedIn()) {
 				// User must be logged in for shareType user and group
 				$this->setPollId(0);
 				$this->setUserId(null);

lib/Service/AnonymizeService.php

Indentation +11 added lines, -11 removed lines

@@ -45,11 +45,11 @@ 
 	}
 
 	/**
-	* Anonymizes the participants of a poll
-	* @NoAdminRequired
-	* @param Array $array Input list which should be anonymized must be a collection of Vote or Comment
-	* @return array Returns the original array with anonymized user names
-	*/
+	 * Anonymizes the participants of a poll
+	 * @NoAdminRequired
+	 * @param Array $array Input list which should be anonymized must be a collection of Vote or Comment
+	 * @return array Returns the original array with anonymized user names
+	 */
 	private function anonymize($array) {
 		// get mapping for the complete poll
 		foreach ($array as &$element) {
@@ -70,12 +70,12 @@ 
 	}
 
 	/**
-	* Initialize anonymizer with pollId and userId
-	* Creates a mapping list with unique Anonymous strings based on the partcipants of a poll
-	* @NoAdminRequired
-	* @param integer $pollId
-	* @param string $userId - usernames, which will not be anonymized
-	*/
+	 * Initialize anonymizer with pollId and userId
+	 * Creates a mapping list with unique Anonymous strings based on the partcipants of a poll
+	 * @NoAdminRequired
+	 * @param integer $pollId
+	 * @param string $userId - usernames, which will not be anonymized
+	 */
 
 	public function set($pollId, $userId) {
 		$this->pollId = $pollId;

lib/Controller/SystemController.php

Spacing +2 added lines, -2 removed lines

@@ -170,7 +170,7 @@ 
 
 		$votes = $this->voteMapper->findParticipantsByPoll($pollId);
 		foreach ($votes as $vote) {
-			if ($vote->getUserId() !== '' && $vote->getUserId() !== null ) {
+			if ($vote->getUserId() !== '' && $vote->getUserId() !== null) {
 				$list[] = [
 					'id' => $vote->getUserId(),
 					'user' => $vote->getUserId(),
@@ -182,7 +182,7 @@ 
 
 		$shares = $this->shareMapper->findByPoll($pollId);
 		foreach ($shares as $share) {
-			if ($share->getUserId() !== '' && $share->getUserId() !== null ) {
+			if ($share->getUserId() !== '' && $share->getUserId() !== null) {
 				$list[] = [
 					'id' => $share->getUserId(),
 					'user' => $share->getUserId(),

lib/Controller/AclController.php

Indentation +5 added lines, -5 removed lines

@@ -52,11 +52,11 @@
 	}
 
 	/**
-	* Read acl with poll id for current user
-	* @NoAdminRequired
-	* @param integer $pollId
-	* @return array
-	*/
+	 * Read acl with poll id for current user
+	 * @NoAdminRequired
+	 * @param integer $pollId
+	 * @return array
+	 */
 	public function get($id) {
 		$acl = $this->acl->setPollId($id);
 		// $acl = $this->acl->setUserId('dartcafe');

tests/Unit/FactoryMuffin.php

Indentation +25 added lines, -25 removed lines

@@ -7,30 +7,30 @@
 
 class FactoryMuffin extends OriginalFactoryMuffin
 {
-    /**
-     * Generate and set the model attributes.
-     * NOTE: Patch the original method to support dynamic setter and getter
-    *        of the OCP\AppFramework\Db\Entity class
-     *
-     * @param object $model The model instance.
-     * @param array  $attr  The model attributes.
-     *
-     * @return void
-     */
-    protected function generate($model, array $attr = [])
-    {
-        foreach ($attr as $key => $kind) {
-            $value = $this->factory->generate($kind, $model, $this);
+	/**
+	 * Generate and set the model attributes.
+	 * NOTE: Patch the original method to support dynamic setter and getter
+	 *        of the OCP\AppFramework\Db\Entity class
+	 *
+	 * @param object $model The model instance.
+	 * @param array  $attr  The model attributes.
+	 *
+	 * @return void
+	 */
+	protected function generate($model, array $attr = [])
+	{
+		foreach ($attr as $key => $kind) {
+			$value = $this->factory->generate($kind, $model, $this);
 
-            $setter = 'set'.ucfirst(static::camelize($key));
-            // check if there is a setter and use it instead
-            if ($model instanceof Entity && is_callable([$model, $setter])) {
-                $model->$setter($value);
-            } elseif (method_exists($model, $setter) && is_callable([$model, $setter])) {
-                $model->$setter($value);
-            } else {
-                $model->$key = $value;
-            }
-        }
-    }
+			$setter = 'set'.ucfirst(static::camelize($key));
+			// check if there is a setter and use it instead
+			if ($model instanceof Entity && is_callable([$model, $setter])) {
+				$model->$setter($value);
+			} elseif (method_exists($model, $setter) && is_callable([$model, $setter])) {
+				$model->$setter($value);
+			} else {
+				$model->$key = $value;
+			}
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         foreach ($attr as $key => $kind) {
             $value = $this->factory->generate($kind, $model, $this);
 
-            $setter = 'set'.ucfirst(static::camelize($key));
+            $setter = 'set' . ucfirst(static::camelize($key));
             // check if there is a setter and use it instead
             if ($model instanceof Entity && is_callable([$model, $setter])) {
                 $model->$setter($value);

Braces +2 added lines, -4 removed lines

@@ -5,8 +5,7 @@ 
 use League\FactoryMuffin\FactoryMuffin as OriginalFactoryMuffin;
 use OCP\AppFramework\Db\Entity;
 
-class FactoryMuffin extends OriginalFactoryMuffin
-{
+class FactoryMuffin extends OriginalFactoryMuffin {
     /**
      * Generate and set the model attributes.
      * NOTE: Patch the original method to support dynamic setter and getter
@@ -17,8 +16,7 @@ 
      *
      * @return void
      */
-    protected function generate($model, array $attr = [])
-    {
+    protected function generate($model, array $attr = []) {
         foreach ($attr as $key => $kind) {
             $value = $this->factory->generate($kind, $model, $this);
 

lib/Controller/VoteController.php

Indentation +10 added lines, -10 removed lines

@@ -154,18 +154,18 @@
 
 	/**
 	 * Public functions
-	*/
+	 */
 
 	/**
-	* setByToken
-	* @NoAdminRequired
-	* @PublicPage
-	* @NoCSRFRequired
-	* @param Array $option
-	* @param string $setTo
-	* @param string $token
-	* @return DataResponse
-	*/
+	 * setByToken
+	 * @NoAdminRequired
+	 * @PublicPage
+	 * @NoCSRFRequired
+	 * @param Array $option
+	 * @param string $setTo
+	 * @param string $token
+	 * @return DataResponse
+	 */
 	public function setByToken($option, $setTo, $token) {
 		try {
 			$this->acl->setToken($token);

lib/Service/LogService.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 use OCA\Polls\Db\Log;
 use OCA\Polls\Db\LogMapper;
 
-class LogService  {
+class LogService {
 
 	private $mapper;
 	private $logItem;
@@ -57,7 +57,7 @@ 
 	public function isRepetition() {
 		try {
 			$lastRecord = $this->mapper->getLastRecord($this->logItem->getPollId());
-			return ( intval($lastRecord->getPollId()) === intval($this->logItem->getPollId())
+			return (intval($lastRecord->getPollId()) === intval($this->logItem->getPollId())
 				&& $lastRecord->getUserId() === $this->logItem->getUserId()
 				&& $lastRecord->getMessageId() === $this->logItem->getMessageId()
 				&& $lastRecord->getMessage() === $this->logItem->getMessage()

Braces +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 use OCA\Polls\Db\Log;
 use OCA\Polls\Db\LogMapper;
 
-class LogService  {
+class LogService {
 
 	private $mapper;
 	private $logItem;

Indentation +12 added lines, -12 removed lines

@@ -50,10 +50,10 @@ 
 
 
 	/**
-	* Prevent repetition of the same log event
-	* @NoAdminRequired
-	* @return Bool
-	*/
+	 * Prevent repetition of the same log event
+	 * @NoAdminRequired
+	 * @return Bool
+	 */
 	public function isRepetition() {
 		try {
 			$lastRecord = $this->mapper->getLastRecord($this->logItem->getPollId());
@@ -68,14 +68,14 @@ 
 	}
 
 	/**
-	* Log poll activity
-	* @NoAdminRequired
-	* @param $pollId
-	* @param $messageId
-	* @param $userId
-	* @param $message
-	* @return Log
-	*/
+	 * Log poll activity
+	 * @NoAdminRequired
+	 * @param $pollId
+	 * @param $messageId
+	 * @param $userId
+	 * @param $message
+	 * @return Log
+	 */
 	public function setLog($pollId, $messageId, $userId = null, $message = null) {
 		$this->logItem = new Log();
 		$this->logItem->setPollId($pollId);

lib/Service/MailService.php

Indentation +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
 		}
 
 		if (!$toEmail || !filter_var($toEmail, FILTER_VALIDATE_EMAIL)) {
-       		throw new Exception( 'Invalid email address (' . $toEmail .')' );
+	   		throw new Exception( 'Invalid email address (' . $toEmail .')' );
 		}
 
 		try {

Braces +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 use OCA\Polls\Db\Share;
 use OCA\Polls\Db\LogMapper;
 
-class MailService  {
+class MailService {
 
 	private $userManager;
 	private $groupManager;

Spacing +11 added lines, -11 removed lines

@@ -41,7 +41,7 @@ 
 use OCA\Polls\Db\Share;
 use OCA\Polls\Db\LogMapper;
 
-class MailService  {
+class MailService {
 
 	private $userManager;
 	private $groupManager;
@@ -120,7 +120,7 @@ 
 		}
 
 		if (!$toEmail || !filter_var($toEmail, FILTER_VALIDATE_EMAIL)) {
-       		throw new Exception( 'Invalid email address (' . $toEmail .')' );
+       		throw new Exception('Invalid email address (' . $toEmail . ')');
 		}
 
 		try {
@@ -252,7 +252,7 @@ 
 			$emailTemplate->addBodyText(str_replace(
 				['{owner}', '{title}'],
 				[$owner->getDisplayName(), $poll->getTitle()],
-				$trans->t('{owner} invited you to take part in the poll "{title}"' )
+				$trans->t('{owner} invited you to take part in the poll "{title}"')
 			));
 
 			$emailTemplate->addBodyButton(
@@ -260,7 +260,7 @@ 
 				$recipient['link']
 			);
 
-			$emailTemplate->addFooter($trans->t('This email is sent to you, because you are invited to vote in this poll by the poll owner.' ));
+			$emailTemplate->addFooter($trans->t('This email is sent to you, because you are invited to vote in this poll by the poll owner.'));
 
 			try {
 				$this->sendMail(
@@ -325,27 +325,27 @@ 
 					} elseif ($logItem->getMessageId() === 'setVote') {
 						$emailTemplate->addBodyText($trans->t(
 							'- %s voted.',
-							array( $this->userManager->get($logItem->getUserId())->getDisplayName()))
+							array($this->userManager->get($logItem->getUserId())->getDisplayName()))
 					);
 					} elseif ($logItem->getMessageId() === 'updatePoll') {
 						$emailTemplate->addBodyText($trans->t(
 							'- %s updated the poll configuration. Please check your votes.',
-							array( $this->userManager->get($logItem->getUserId())->getDisplayName()))
+							array($this->userManager->get($logItem->getUserId())->getDisplayName()))
 						);
 					} elseif ($logItem->getMessageId() === 'deletePoll') {
 						$emailTemplate->addBodyText($trans->t(
 							'- %s deleted the poll.',
-							array( $this->userManager->get($logItem->getUserId())->getDisplayName()))
+							array($this->userManager->get($logItem->getUserId())->getDisplayName()))
 						);
 					} elseif ($logItem->getMessageId() === 'restorePoll') {
 						$emailTemplate->addBodyText($trans->t(
 							'- %s restored the poll.',
-							array( $this->userManager->get($logItem->getUserId())->getDisplayName()))
+							array($this->userManager->get($logItem->getUserId())->getDisplayName()))
 						);
 					} elseif ($logItem->getMessageId() === 'expirePoll') {
 						$emailTemplate->addBodyText($trans->t(
 							'- The poll expired.',
-							array( $this->userManager->get($logItem->getUserId())->getDisplayName()))
+							array($this->userManager->get($logItem->getUserId())->getDisplayName()))
 						);
 					}
 
@@ -360,10 +360,10 @@ 
 				$url,
 				/** @scrutinizer ignore-type */ false
 			);
-			$emailTemplate->addFooter($trans->t('This email is sent to you, because you subscribed to notifications of this poll. To opt out, visit the poll and remove your subscription.' ));
+			$emailTemplate->addFooter($trans->t('This email is sent to you, because you subscribed to notifications of this poll. To opt out, visit the poll and remove your subscription.'));
 
 			try {
-				$this->sendMail( $emailTemplate, $subscription->getUserId());
+				$this->sendMail($emailTemplate, $subscription->getUserId());
 			} catch (Exeption $e) {
 				// todo alert Owner
 				// Notification to $subscription->getUserId() could not be sent

lib/Db/LogMapper.php

Indentation +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 			->from($this->getTableName())
 			->where($qb->expr()->eq('poll_id', $qb->createNamedParameter($pollId, IQueryBuilder::PARAM_INT)))
 			->setMaxResults( 1 )
-		    ->orderBy('id', 'DESC');
+			->orderBy('id', 'DESC');
 
 		return $this->findEntity($qb);
 

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 		$qb->select('*')
 			->from($this->getTableName())
 			->where($qb->expr()->eq('poll_id', $qb->createNamedParameter($pollId, IQueryBuilder::PARAM_INT)))
-			->setMaxResults( 1 )
+			->setMaxResults(1)
 		    ->orderBy('id', 'DESC');
 
 		return $this->findEntity($qb);