newicon / neon

neon/user/grid/UserGrid.php

Spacing +6 added lines, -6 removed lines

@@ -45,10 +45,10 @@ 
 			$this->addTextColumn('impersonate')->setFilter(false);
 		// scopes
 		// $this->addScope('all','All');
-		$this->addScope('active','Active');
-		$this->addScope('pending','Pending');
-		$this->addScope('suspended','Suspended');
-		$this->addScope('banned','Banned');
+		$this->addScope('active', 'Active');
+		$this->addScope('pending', 'Pending');
+		$this->addScope('suspended', 'Suspended');
+		$this->addScope('banned', 'Banned');
 		$this->addScope('super', 'Super');
 
 		$this->setPageSize(100);
@@ -196,7 +196,7 @@ 
 	 */
 	private function getRoles()
 	{
-		static $roles=null;
+		static $roles = null;
 		if (is_null($roles)) {
 			$roles = neon('user')->getRoles(true);
 		}
@@ -210,7 +210,7 @@ 
 	 */
 	private function getRoleLabels()
 	{
-		static $labels=null;
+		static $labels = null;
 		if (is_null($labels)) {
 			$roles = $this->getRoles();
 			foreach ($roles as $role => $details)

Braces +12 added lines, -8 removed lines

@@ -41,8 +41,9 @@ 
 		$this->addTextColumn('created_at');
 		$this->addTextColumn('updated_at', ['title'=>'Updated At']);
 		// action
-		if (neon()->user->isSuper())
-			$this->addTextColumn('impersonate')->setFilter(false);
+		if (neon()->user->isSuper()) {
+					$this->addTextColumn('impersonate')->setFilter(false);
+		}
 		// scopes
 		// $this->addScope('all','All');
 		$this->addScope('active','Active');
@@ -104,8 +105,9 @@ 
 		$userRoles = explode('|', $column->getCellData($model));
 		$displayRoles = [];
 		foreach ($userRoles as $ur) {
-			if (isset($roles[$ur]))
-				$displayRoles[] = $column->highlight($roles[$ur]['label'], $search);
+			if (isset($roles[$ur])) {
+							$displayRoles[] = $column->highlight($roles[$ur]['label'], $search);
+			}
 		}
 		return implode(', ', $displayRoles);
 	}
@@ -213,8 +215,9 @@ 
 		static $labels=null;
 		if (is_null($labels)) {
 			$roles = $this->getRoles();
-			foreach ($roles as $role => $details)
-				$labels[$details['label']] = $role;
+			foreach ($roles as $role => $details) {
+							$labels[$details['label']] = $role;
+			}
 		}
 		return $labels;
 	}
@@ -230,8 +233,9 @@ 
 		$labels = $this->getRoleLabels();
 		$roles = [];
 		foreach ($labels as $label => $role) {
-			if (stripos($label, $filter) !== false)
-				$roles[] = $role;
+			if (stripos($label, $filter) !== false) {
+							$roles[] = $role;
+			}
 		}
 		return $roles;
 	}

neon/user/services/apiTokenManager/JwtTokenAuth.php

Spacing +4 added lines, -4 removed lines

@@ -113,13 +113,13 @@ 
 		try {
 			$jwt = (array) JWT::decode($cookieString, static::secret(), ['HS256']);
 		} catch (Exception $e) {
-			throw new UnauthorizedHttpException('Your request was made with invalid credentials.' . neon()->debug ? ' Can not decode JWT token' : '');
+			throw new UnauthorizedHttpException('Your request was made with invalid credentials.'.neon()->debug ? ' Can not decode JWT token' : '');
 		}
 		// We will compare the CSRF token in the decoded API token against the CSRF header
 		// sent with the request. If the two don't match then this request is sent from
 		// an invalid source and we won't authenticate the request for further handling.
 		if (!$this->validCsrf($jwt, $request)) {
-			throw new UnauthorizedHttpException('Your request was made with invalid credentials.' . neon()->debug ? ' CSRF validation failed' : '');
+			throw new UnauthorizedHttpException('Your request was made with invalid credentials.'.neon()->debug ? ' CSRF validation failed' : '');
 		}
 
 		return $jwt;
@@ -215,7 +215,7 @@ 
 	 */
 	protected static function requestShouldReceiveFreshToken(Request $request)
 	{
-		return $request->isMethod('GET') && neon()->user->isLoggedIn() && ! $request->getIsAjax();
+		return $request->isMethod('GET') && neon()->user->isLoggedIn() && !$request->getIsAjax();
 	}
 
 	/**
@@ -227,6 +227,6 @@ 
 	protected static function responseShouldReceiveFreshToken(Response $response)
 	{
 		// if the cookie already exists do not create another one
-		return ! isset($response->cookies[static::$cookieName]);
+		return !isset($response->cookies[static::$cookieName]);
 	}
 }
\ No newline at end of file

Braces +9 added lines, -6 removed lines

@@ -56,8 +56,9 @@ 
 		// As there is no cookie or cookie string we return null allowing other authentication methods
 		// to attempt authentication
 		$tokenString = $this->getJwtStringFromCookie($request);
-		if ($tokenString === null)
-			return null;
+		if ($tokenString === null) {
+					return null;
+		}
 		// decode the cookie and perform csrf validation
 		// this only checks that the cookie data is good
 		$jwt = $this->decodeAndValidateJwtToken($tokenString, $request);
@@ -69,8 +70,9 @@ 
 		// @see self::findAndValidateUserIdentity()
 		// this is similar to the stock yii classes yii\filters\auth\HttpBearerAuth and yii\filters\auth\QueryParamAuth;
 		$identity = $user->loginByAccessToken($jwt, get_class($this));
-		if ($identity === null)
-			$this->handleFailure($response);
+		if ($identity === null) {
+					$this->handleFailure($response);
+		}
 
 		return $identity;
 	}
@@ -132,8 +134,9 @@ 
 	 */
 	public function getJwtStringFromCookie(Request $request)
 	{
-		if (!$request->cookies->has(static::$cookieName))
-			return null;
+		if (!$request->cookies->has(static::$cookieName)) {
+					return null;
+		}
 		$cookie = $request->cookies->get(static::$cookieName)->value;
 		return $cookie;
 	}

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');