Icybee / module-users

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/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',

config/hooks.php

Spacing +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
 
 	'patron.markups' => [
 
-		'user' => [ $hooks . 'markup_user' ],
-		'users:form:login' => [ $hooks . 'markup_form_login' ]
+		'user' => [$hooks . 'markup_user'],
+		'users:form:login' => [$hooks . 'markup_form_login']
 
 	]
 

config/routes.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 
 		'pattern' => '/api/logout',
 		'controller' => LogoutOperation::class,
-		'via' => [ 'GET', 'POST' ]
+		'via' => ['GET', 'POST']
 
 	],
 
@@ -58,7 +58,7 @@ 
 ] + Make::admin('users', Routing\UsersAdminController::class, [
 
 	'id_name' => 'uid',
-	'only' => [ 'index', 'new', 'edit', 'confirm-delete', 'profile', 'authenticate' ],
+	'only' => ['index', 'new', 'edit', 'confirm-delete', 'profile', 'authenticate'],
 	'actions' => [
 
 		'profile' => [

descriptor.php

Spacing +17 added lines, -17 removed lines

@@ -17,19 +17,19 @@ 
 			Model::SCHEMA => [
 
 				'uid' => 'serial',
-				'constructor' => [ 'varchar', 64, 'indexed' => true ],
-				'email' => [ 'varchar', 64, 'unique' => true ],
-				'password_hash' => [ 'varchar', 255, 'charset' => 'ascii/bin' ],
-				'username' => [ 'varchar', 32, 'unique' => true ],
-				'firstname' => [ 'varchar', 32 ],
-				'lastname' => [ 'varchar', 32 ],
-				'nickname' => [ 'varchar', 32 ],
-				'name_as' => [ 'integer', 'tiny' ],
-				'language' => [ 'varchar', 8 ],
-				'timezone' => [ 'varchar', 32 ],
+				'constructor' => ['varchar', 64, 'indexed' => true],
+				'email' => ['varchar', 64, 'unique' => true],
+				'password_hash' => ['varchar', 255, 'charset' => 'ascii/bin'],
+				'username' => ['varchar', 32, 'unique' => true],
+				'firstname' => ['varchar', 32],
+				'lastname' => ['varchar', 32],
+				'nickname' => ['varchar', 32],
+				'name_as' => ['integer', 'tiny'],
+				'language' => ['varchar', 8],
+				'timezone' => ['varchar', 32],
 				'logged_at' => 'datetime',
-				'created_at' => [ 'timestamp', 'default' => 'CURRENT_TIMESTAMP' ],
-				'is_activated' => [ 'boolean', 'indexed' => true ]
+				'created_at' => ['timestamp', 'default' => 'CURRENT_TIMESTAMP'],
+				'is_activated' => ['boolean', 'indexed' => true]
 
 			]
 		],
@@ -41,8 +41,8 @@ 
 			Model::CLASSNAME => 'ICanBoogie\ActiveRecord\Model',
 			Model::SCHEMA => [
 
-				'uid' => [ 'foreign', 'primary' => true ],
-				'rid' => [ 'foreign', 'primary' => true ]
+				'uid' => ['foreign', 'primary' => true],
+				'rid' => ['foreign', 'primary' => true]
 
 			]
 		],
@@ -54,8 +54,8 @@ 
 			Model::CLASSNAME => 'ICanBoogie\ActiveRecord\Model',
 			Model::SCHEMA => [
 
-				'uid' => [ 'foreign', 'primary' => true ],
-				'site_id' => [ 'foreign', 'primary' => true ]
+				'uid' => ['foreign', 'primary' => true],
+				'site_id' => ['foreign', 'primary' => true]
 
 			]
 		]
@@ -69,7 +69,7 @@ 
 	],
 
 	Descriptor::REQUIRED => true,
-	Descriptor::REQUIRES => [ 'users.roles' ],
+	Descriptor::REQUIRES => ['users.roles'],
 	Descriptor::TITLE => "Users"
 
 ];

lib/Block/AvailableSites.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
 		/* @var $site \Icybee\Modules\Sites\Site */
 		$site = $app->site;
-		$ws_title = \ICanBoogie\escape($site->admin_title ? $site->admin_title : $site->title .':' . $site->language);
+		$ws_title = \ICanBoogie\escape($site->admin_title ? $site->admin_title : $site->title . ':' . $site->language);
 
 		$available = $site->model
 		->where('site_id IN(' . implode(',', $app->user->restricted_sites_ids) . ')')

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

@@ -76,6 +76,6 @@
 please select another website to work with:</p>
 $form
 </div>
-EOT;
+eot;
 	}
 }

lib/Block/EditBlock.php

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])
 		{

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\Roles\Binding\UserBindings;
 use Icybee\Modules\Users\User;

lib/Block/ManageBlock.php

Spacing +2 added lines, -2 removed lines

@@ -30,8 +30,8 @@
 	{
 		parent::__construct($module, $attributes + [
 
-			self::T_COLUMNS_ORDER => [ User::USERNAME, User::IS_ACTIVATED, User::EMAIL, 'roles', User::CREATED_AT, User::LOGGED_AT ],
-			self::T_ORDER_BY => [ 'created_at', 'desc' ]
+			self::T_COLUMNS_ORDER => [User::USERNAME, User::IS_ACTIVATED, User::EMAIL, 'roles', User::CREATED_AT, User::LOGGED_AT],
+			self::T_ORDER_BY => ['created_at', 'desc']
 
 		]);
 	}