Icybee / module-users

lib/Block/EditBlock.php

Unused Use Statements -3 removed lines

@@ -12,14 +12,11 @@
 namespace Icybee\Modules\Users\Block;
 
 use Brickrouge\Group;
-
-
 use Brickrouge\Document;
 use Brickrouge\Element;
 use Brickrouge\Form;
 use Brickrouge\Text;
 use Brickrouge\Widget;
-
 use Icybee\Modules\Users\Module;
 use Icybee\Modules\Users\User;
 

Spacing +6 added lines, -6 removed lines

@@ -65,8 +65,8 @@ 
 
 			Element::GROUPS => [
 
-				'connection' => [ 'title' => 'Connection' ],
-				'advanced' => [ 'title' => 'Advanced' ]
+				'connection' => ['title' => 'Connection'],
+				'advanced' => ['title' => 'Advanced']
 
 			]
 
@@ -172,7 +172,7 @@ 
 				Group::LABEL => 'language',
 				Element::GROUP => 'advanced',
 				Element::DESCRIPTION => 'language',
-				Element::OPTIONS => [ null => '' ] + $languages
+				Element::OPTIONS => [null => ''] + $languages
 
 			]),
 
@@ -214,7 +214,7 @@ 
 			return null;
 		}
 
-		$rid = [ 2 => true ];
+		$rid = [2 => true];
 
 		if ($uid)
 		{
@@ -236,7 +236,7 @@ 
 			Form::LABEL => 'roles',
 			Element::GROUP => 'advanced',
 			Element::OPTIONS => $options,
-			Element::OPTIONS_DISABLED => [ 2 => true ],
+			Element::OPTIONS_DISABLED => [2 => true],
 			Element::REQUIRED => true,
 			Element::DESCRIPTION => 'roles',
 
@@ -255,7 +255,7 @@ 
 	{
 		$values = $this->values;
 
-		$options = [ '<username>' ];
+		$options = ['<username>'];
 
 		if ($values[User::USERNAME])
 		{

lib/Hooks.php

Doc Comments +4 added lines, -4 removed lines

@@ -216,7 +216,7 @@ 
 	 *
 	 * This is the getter for the `$app->user` property.
 	 *
-	 * @param Core|CoreBindings|\ICanBoogie\Binding\ActiveRecord\CoreBindings $app
+	 * @param Core $app
 	 *
 	 * @return User The user object, or guest user object.
 	 */
@@ -275,7 +275,7 @@ 
 	/**
 	 * Checks if a user has a given permission.
 	 *
-	 * @param Core|CoreBindings $app
+	 * @param Core $app
 	 * @param User $user
 	 * @param string $permission
 	 * @param string $target
@@ -292,7 +292,7 @@ 
 	/**
 	 * Checks if a user has the ownership of a record.
 	 *
-	 * @param Core|CoreBindings $app
+	 * @param Core $app
 	 * @param User $user
 	 * @param ActiveRecord $record
 	 *
@@ -358,7 +358,7 @@ 
 	 */
 
 	/**
-	 * @return Core|\Icybee\Binding\CoreBindings
+	 * @return Core
 	 */
 	static private function app()
 	{

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 			return;
 		}
 
-		$event->errors['rid'] = $event->errors->format('The role %name is used by :count users.', [ 'name' => $operation->record->name, ':count' => $count ]);
+		$event->errors['rid'] = $event->errors->format('The role %name is used by :count users.', ['name' => $operation->record->name, ':count' => $count]);
 	}
 
 	/**

Braces +3 added lines, -1 removed lines

@@ -233,7 +233,9 @@
 				$user = $model[$uid];
 			}
 		}
-		catch (\Exception $e) {}
+		catch (\Exception $e)
+		{
+}
 
 		if (!$user)
 		{

Unused Use Statements -3 removed lines

@@ -16,13 +16,10 @@
 use ICanBoogie\HTTP\AuthenticationRequired;
 use ICanBoogie\HTTP\PermissionRequired;
 use ICanBoogie\HTTP\Response;
-use ICanBoogie\HTTP\RequestDispatcher;
 use ICanBoogie\HTTP\SecurityError;
 use ICanBoogie\Operation;
 use ICanBoogie\PropertyNotDefined;
 use ICanBoogie\Routing\RouteDispatcher;
-use ICanBoogie\Session;
-
 use Icybee\Element\AdminDecorator;
 use Icybee\Element\DocumentDecorator;
 use Icybee\Modules\Members\Member;

lib/LoginForm.php

Unused Use Statements -3 removed lines

@@ -11,15 +11,12 @@
 
 namespace Icybee\Modules\Users;
 
-use ICanBoogie\Operation;
-
 use Brickrouge\A;
 use Brickrouge\Button;
 use Brickrouge\Document;
 use Brickrouge\Element;
 use Brickrouge\Form;
 use Brickrouge\Text;
-
 use Icybee\Binding\Core\PrototypedBindings;
 
 class LoginForm extends Form

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	{
 		$app = $this->app;
 
-		$this->lost_password = new A($this->t('lost_password', [], [ 'scope' => 'users.label', 'default' => 'I forgot my password' ]), "#lost-password", [
+		$this->lost_password = new A($this->t('lost_password', [], ['scope' => 'users.label', 'default' => 'I forgot my password']), "#lost-password", [
 
 			'rel' => 'nonce-request'
 

lib/Module.php

Unused Use Statements -2 removed lines

@@ -11,9 +11,7 @@
 
 namespace Icybee\Modules\Users;
 
-use ICanBoogie\I18n;
 use ICanBoogie\Operation;
-
 use Brickrouge\Button;
 use Brickrouge\Element;
 use Brickrouge\Form;

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
 			Element::CHILDREN => [
 
-				new Button('logout', [ 'type' => 'submit' ])
+				new Button('logout', ['type' => 'submit'])
 
 			]
 

lib/Operation/LoginOperation.php

Unused Use Statements -2 removed lines

@@ -12,11 +12,9 @@
 namespace Icybee\Modules\Users\Operation;
 
 use ICanBoogie\Errors;
-use ICanBoogie\HTTP\Request;
 use ICanBoogie\I18n;
 use ICanBoogie\Operation;
 use ICanBoogie\Module\ControllerBindings as ModuleBindings;
-
 use Icybee\Binding\Core\PrototypedBindings;
 use Icybee\Modules\Registry\MetaCollection;
 use Icybee\Modules\Users\LoginForm;

Spacing +2 added lines, -3 removed lines

@@ -85,8 +85,7 @@ 
 		{
 			if ($login_unlock_time > $now)
 			{
-				throw new \Exception
-				(
+				throw new \Exception(
 					\ICanBoogie\format("The user account has been locked after multiple failed login attempts.
 					An e-mail has been sent to unlock the account. Login attempts are locked until %time,
 					unless you unlock the account using the email sent.", [
@@ -159,7 +158,7 @@ 
 
 		if (!$user->is_admin && !$user->is_activated)
 		{
-			$errors->add(null, "User %username is not activated", [ '%username' => $username ]);
+			$errors->add(null, "User %username is not activated", ['%username' => $username]);
 
 			return false;
 		}

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
 attack attempt on the website. If you think this is the case, please contact its admin.
 
 The remote address of the request was: $request->ip.
-EOT
+eot
 				]);
 
 				unset($errors[User::PASSWORD]);

lib/Operation/UnlockLoginOperation.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use ICanBoogie\Errors;
 use ICanBoogie\Operation;
-
 use Icybee\Modules\Users\User;
 
 /**

lib/User.php

Unused Use Statements -2 removed lines

@@ -14,11 +14,9 @@
 use ICanBoogie\ActiveRecord;
 use ICanBoogie\ActiveRecord\CreatedAtProperty;
 use ICanBoogie\ActiveRecord\RecordNotFound;
-
 use Brickrouge\AlterCSSClassNamesEvent;
 use Brickrouge\CSSClassNames;
 use Brickrouge\CSSClassNamesProperty;
-
 use Icybee\Modules\Users\Roles\Role;
 use Icybee\Binding\Core\PrototypedBindings as IcybeeBindings;
 use Icybee\Modules\Registry\Binding\UserBindings as RegistryBindings;

Spacing +1 added lines, -1 removed lines

@@ -355,7 +355,7 @@
 		{
 			if (!$this->uid)
 			{
-				return [ $models['users.roles'][1] ];
+				return [$models['users.roles'][1]];
 			}
 		}
 		catch (\Exception $e)

lib/UserModel.php

Unused Use Statements -2 removed lines

@@ -12,8 +12,6 @@
 namespace Icybee\Modules\Users;
 
 use ICanBoogie\DateTime;
-use ICanBoogie\ActiveRecord;
-
 use Icybee\ConstructorModel;
 
 class UserModel extends ConstructorModel

Spacing +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
 				continue;
 			}
 
-			$has_many_roles->execute('INSERT {self} SET uid = ?, rid = ?', [ $rc, $rid ]);
+			$has_many_roles->execute('INSERT {self} SET uid = ?, rid = ?', [$rc, $rid]);
 		}
 	}
 
@@ -88,7 +88,7 @@ 
 
 		foreach ($restricted_sites as $site_id)
 		{
-			$has_many_sites->execute('INSERT {self} SET uid = ?, site_id = ?', [ $rc, $site_id ]);
+			$has_many_sites->execute('INSERT {self} SET uid = ?, site_id = ?', [$rc, $site_id]);
 		}
 	}
 }

config/event.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 return [
 
-	ICanBoogie\HTTP\AuthenticationRequired::class .'::rescue' => $hooks . 'on_security_exception_rescue',
+	ICanBoogie\HTTP\AuthenticationRequired::class . '::rescue' => $hooks . 'on_security_exception_rescue',
 	ICanBoogie\HTTP\PermissionRequired::class . '::rescue' => $hooks . 'on_security_exception_rescue',
 	ICanBoogie\Routing\RouteDispatcher::class . '::dispatch:before' => $hooks . 'before_routing_dispatcher_dispatch',
 	Icybee\Modules\Users\Roles\Operation\DeleteOperation::class . '::process:before' => $hooks . 'before_roles_delete',