Icybee / module-users

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/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']
 
 		]);
 	}

lib/Block/ManageBlock/UserColumn.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
 
 		if ($value)
 		{
-			$query->and([ $this->id => $value ]);
+			$query->and([$this->id => $value]);
 		}
 
 		return $query;

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

@@ -175,7 +175,7 @@
 		{
 			return <<<EOT
 <div class="alert alert-error undismissable">Undefined user: {$uid}</div>
-EOT;
+eot;
 		}
 
 		return new FilterDecorator($record, $this->id, $this->is_filtering, $user ? $user->name : '');

lib/Element/PickUser.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	{
 		parent::__construct('select', $attributes + [
 
-			Element::OPTIONS => [ null => '' ] + $this->create_options($attributes)
+			Element::OPTIONS => [null => ''] + $this->create_options($attributes)
 
 		]);
 	}