newicon / neon

neon/user/services/apiTokenManager/models/UserApiToken.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
 	 */
 	public function getHttpBasicAuthorizationBase64()
 	{
-		return base64_encode($this->user_id . ':' . $this->token);
+		return base64_encode($this->user_id.':'.$this->token);
 	}
 
 	/**
@@ -110,7 +110,7 @@ 
 	 * @param string $name
 	 * @return UserApiToken|null
 	 */
-	public static function generateTokenFor($userId, $name='')
+	public static function generateTokenFor($userId, $name = '')
 	{
 		$token = UserApiToken::getTokenRecord($userId);
 		// extra security:

Braces +9 added lines, -6 removed lines

@@ -55,17 +55,20 @@
 	public static function getValidUserByToken($token)
 	{
 		$validToken = self::isValidToken($token);
-		if (!$validToken)
-			return null;
+		if (!$validToken) {
+					return null;
+		}
 
 		$apiToken = self::findOne(['active' => 1, 'token' => (string) $token]);
-		if ($apiToken == null)
-			return null;
+		if ($apiToken == null) {
+					return null;
+		}
 
 		// TODO: SO 27/10/2018 - update this to be User::findIndentity() - migrate to uuids.
 		$user = User::findOne(['id' => $apiToken->user_id, 'status' => User::STATUS_ACTIVE]);
-		if ($user == null)
-			return null;
+		if ($user == null) {
+					return null;
+		}
 
 		// Usage successful
 		$apiToken->touchTokenUsed();

neon/user/services/UserManager.php

Spacing +7 added lines, -7 removed lines

@@ -47,7 +47,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function listUsers(Iterator &$iterator=null)
+	public function listUsers(Iterator &$iterator = null)
 	{
 		return $this->findUsers(null, $iterator);
 	}
@@ -55,7 +55,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function searchUsers($searchString, Iterator &$iterator=null)
+	public function searchUsers($searchString, Iterator &$iterator = null)
 	{
 		return $this->findUsers(['like', 'username', $searchString], $iterator);
 	}
@@ -63,7 +63,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function addUser($username, $email, $password, $roles=[], $superuser=false)
+	public function addUser($username, $email, $password, $roles = [], $superuser = false)
 	{
 		$user = new UserModel;
 		$user->uuid = Hash::uuid64();
@@ -95,7 +95,7 @@ 
 				'uuid'=>$user->uuid,
 				'username'=>$user->username,
 				'email'=>$user->email,
-				'superuser'=>(boolean)$user->super,
+				'superuser'=>(boolean) $user->super,
 				'status' => $user->status,
 				'roles'=>$user->getRoles()
 			];
@@ -113,7 +113,7 @@ 
 	{
 		$user = $this->findUserModel($uuid);
 		if ($user) {
-			$newValues = array_intersect_key($changes, array_flip(['email','password','roles']));
+			$newValues = array_intersect_key($changes, array_flip(['email', 'password', 'roles']));
 			foreach ($newValues as $key=>$value) {
 				$user->$key = $value;
 			}
@@ -153,7 +153,7 @@ 
 	 *   for the query.
 	 * @param Iterator $iterator  an iterator to go over the query
 	 */
-	private function findUsers($where, Iterator &$iterator=null) 
+	private function findUsers($where, Iterator &$iterator = null) 
 	{
 		$iterator = $iterator ? $iterator : new Iterator;
 
@@ -163,7 +163,7 @@ 
 		}
 
 		// get the rows
-		$query = UserModel::find()->select(['uuid','username'])
+		$query = UserModel::find()->select(['uuid', 'username'])
 			->limit($iterator->length)
 			->offset($iterator->start)
 			->orderBy('username');

Braces +21 added lines, -14 removed lines

@@ -29,8 +29,9 @@ 
 	{
 		$auth = neon()->authManager;
 		$authRole = $auth->createRole($role);
-		if ($authRole)
-			$auth->add($authRole);
+		if ($authRole) {
+					$auth->add($authRole);
+		}
 	}
 
 	/**
@@ -40,8 +41,9 @@ 
 	{
 		$auth = neon()->authManager;
 		$authRole = $auth->getRole($role);
-		if ($authRole)
-			$auth->remove($authRole);
+		if ($authRole) {
+					$auth->remove($authRole);
+		}
 	}
 
 	/**
@@ -70,8 +72,9 @@ 
 		$user->username = $username;
 		$user->email = $email;
 		$user->setPassword($password);
-		if ($superuser && neon()->user->isSuper())
-			$user->super = $superuser;
+		if ($superuser && neon()->user->isSuper()) {
+					$user->super = $superuser;
+		}
 		$user->generateAuthKey();
 		$success = $user->save();
 		if ($success) {
@@ -117,8 +120,9 @@ 
 			foreach ($newValues as $key=>$value) {
 				$user->$key = $value;
 			}
-			if (isset($changes['superuser']))
-				$user->__set_super($changes['superuser']);
+			if (isset($changes['superuser'])) {
+							$user->__set_super($changes['superuser']);
+			}
 			$success = $user->save();
 			return $success ? true : $user->errors;
 		}
@@ -132,8 +136,9 @@ 
 	public function destroyUser($uuid)
 	{
 		$user = $this->findUserModel($uuid);
-		if ($user)
-			$user->delete();
+		if ($user) {
+					$user->delete();
+		}
 	}
 
 
@@ -168,15 +173,17 @@ 
 			->offset($iterator->start)
 			->orderBy('username');
 
-		if ($where)
-			$query->where($where);
+		if ($where) {
+					$query->where($where);
+		}
 		
 		$userRows = $query->all();
 
 		// and return in required format
 		$users = [];
-		foreach ($userRows as $ur)
-			$users[$ur['uuid']] = $ur['username'];
+		foreach ($userRows as $ur) {
+					$users[$ur['uuid']] = $ur['username'];
+		}
 		return $users;
 	}
 

neon/user/services/User.php

Spacing +5 added lines, -5 removed lines

@@ -226,7 +226,7 @@ 
 	 * @param int $size
 	 * @return string
 	 */
-	public function getGuestUserImageUrl($size=40)
+	public function getGuestUserImageUrl($size = 40)
 	{
 		return "//www.gravatar.com/avatar?d=mm&s=$size";
 	}
@@ -234,7 +234,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function getImageUrl($size=40)
+	public function getImageUrl($size = 40)
 	{
 		return $this->noIdentity() ? $this->getGuestUserImageUrl($size) : $this->identity->getImageUrl($size);
 	}
@@ -276,7 +276,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function getHomeUrl($role=null)
+	public function getHomeUrl($role = null)
 	{
 		return $this->noIdentity() ? null : $this->identity->getHomeUrl($role);
 	}
@@ -295,13 +295,13 @@ 
 		foreach ($userRoles as $ur) {
 			if (array_key_exists('homeUrl', $appRoles[$ur])) {
 				$url = $appRoles[$ur]['homeUrl'];
-				return parent::getReturnUrl( is_array($url) ? $url : [ $url ]);
+				return parent::getReturnUrl(is_array($url) ? $url : [$url]);
 			}
 		}
 
 		// use the default or application set one
 		if ($defaultUrl)
-			return parent::getReturnUrl( is_array($defaultUrl) ? $defaultUrl : [ $defaultUrl ] );
+			return parent::getReturnUrl(is_array($defaultUrl) ? $defaultUrl : [$defaultUrl]);
 
 		return parent::getReturnUrl();
 	}

Braces +6 added lines, -4 removed lines

@@ -107,8 +107,9 @@ 
 				$log = "User '$id' logged in from $ip. Session not enabled.";
 			}
 
-			if (neon()->getRequest()->enableCsrfValidation)
-				$this->regenerateCsrfToken();
+			if (neon()->getRequest()->enableCsrfValidation) {
+							$this->regenerateCsrfToken();
+			}
 
 			\Yii::info($log, __METHOD__);
 			$this->afterLogin($identity, false, $duration);
@@ -300,8 +301,9 @@ 
 		}
 
 		// use the default or application set one
-		if ($defaultUrl)
-			return parent::getReturnUrl( is_array($defaultUrl) ? $defaultUrl : [ $defaultUrl ] );
+		if ($defaultUrl) {
+					return parent::getReturnUrl( is_array($defaultUrl) ? $defaultUrl : [ $defaultUrl ] );
+		}
 
 		return parent::getReturnUrl();
 	}

neon/user/services/UserInvite.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 				return neon()->mailer->compose($templates, $templateData)
 					->setTo($user->email)
 					->setFrom([$fromAddress => $fromName])
-					->setSubject($siteName.' invitation for ' . $user->email)
+					->setSubject($siteName.' invitation for '.$user->email)
 					->send();
 			}
 		}

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
 				$fromName = setting('admin', 'fromEmailName', neon()->name);
 				$fromAddress = setting('admin', 'fromEmailAddress', 'system@'.neon()->request->getHostName());
 				
-				if (!$fromAddress)
-					throw new \Exception('The From Address needs to be set in the admin section for emails');
+				if (!$fromAddress) {
+									throw new \Exception('The From Address needs to be set in the admin section for emails');
+				}
 
 				$templates = [
 					'html' => '@neon/user/views/mail/userInviteToken-html',

neon/user/plugins/smarty/function.getUser.php

Spacing +1 added lines, -1 removed lines

@@ -10,6 +10,6 @@
  */
 function smarty_function_getUser($params, $template)
 {
-	$assign = isset($params['assign'])?$params['assign']:'user';
+	$assign = isset($params['assign']) ? $params['assign'] : 'user';
 	$template->assign($assign, neon()->user);
 }

neon/user/web/DbSession.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
  */
 class DbSession extends \yii\web\DbSession
 {
-	protected function composeFields($id=null, $data=null)
+	protected function composeFields($id = null, $data = null)
 	{
 		// NOTE: do not interact with the identity inside this call
 		// this will trigger a "session_regenerate_id(): Cannot call session save handler in a recursive manner" error

neon/user/controllers/IndexController.php

Braces +8 added lines, -6 removed lines

@@ -103,12 +103,14 @@
 			if ($loggedInUser->isSuper()) {
 				if ($loggedInUser->getId() != $id) {
 					$user = $this->findModel($id);
-					if ($user->delete())
-						neon()->session->setFlash('success', 'The user has been successfully deleted');
-					else
-						neon()->session->setFlash('error', 'Something went wrong while trying to delete the user');
-				} else
-					neon()->session->setFlash('error', 'Sorry, you cannot delete yourself from the system.');
+					if ($user->delete()) {
+											neon()->session->setFlash('success', 'The user has been successfully deleted');
+					} else {
+											neon()->session->setFlash('error', 'Something went wrong while trying to delete the user');
+					}
+				} else {
+									neon()->session->setFlash('error', 'Sorry, you cannot delete yourself from the system.');
+				}
 				return $this->redirect(['/user/index/index']);
 			}
 			neon()->session->setFlash('error', 'You do not have permissions to delete the user');

neon/user/controllers/AccountController.php

Spacing +3 added lines, -3 removed lines

@@ -26,18 +26,18 @@
 			[
 				'allow' => true,
 				'actions' => ['login', 'logout', 'request-password-reset', 'reset-password', 'impersonate-restore'],
-				'roles' => [ '?', '@' ] // any guest user or logged in user
+				'roles' => ['?', '@'] // any guest user or logged in user
 			],
 			[
 				'allow' => setting('user', 'allowUserInvitations', false), // Allow registration if allowed in settings
 				'actions' => ['register'],
-				'roles' => [ '?', '@' ]
+				'roles' => ['?', '@']
 			],
 			// allow users with neon-administrator access to all functions defined
 			[
 				'allow' => true,
 				'actions' => ['index', 'impersonate'],
-				'roles' => [ 'neon-administrator' ]
+				'roles' => ['neon-administrator']
 			],
 			// everything else is denied
 		];

neon/user/controllers/InviteController.php

Spacing +2 added lines, -2 removed lines

@@ -87,11 +87,11 @@
 			// allow authenticated users if invites are allowed
 			[
 				'allow' => setting('user', 'allowUserInvitations', false),
-				'roles' => [ 'neon-administrator' ]
+				'roles' => ['neon-administrator']
 			],
 			[
 				'allow' => setting('user', 'allowUserInvitations', false),
-				'matchCallback' => function ($rule, $action) {
+				'matchCallback' => function($rule, $action) {
 					return neon()->user->isSuper();
 				}
 			]