Apply fixes from StyleCI

Author: stevebauman

src/Auth/DatabaseUserProvider.php

@@ -108,7 +108,7 @@ public function validateCredentials(Authenticatable $model, array $credentials)
      *
      * @return bool
      */
-    protected function isFallingBack() : bool
+    protected function isFallingBack(): bool
     {
         return Config::get('ldap_auth.login_fallback', false);
     }

src/Commands/Console/Import.php

@@ -92,7 +92,7 @@ public function handle()
      *
      * @return int
      */
-    public function import(array $users = []) : int
+    public function import(array $users = []): int
     {
         $imported = 0;
 
@@ -164,7 +164,7 @@ public function display(array $users = [])
      *
      * @return bool
      */
-    public function isLogging() : bool
+    public function isLogging(): bool
     {
         return ! $this->option('no-log');
     }
@@ -175,7 +175,7 @@ public function isLogging() : bool
      *
      * @return bool
      */
-    public function isDeleting() : bool
+    public function isDeleting(): bool
     {
         return $this->option('delete') == 'true';
     }
@@ -186,7 +186,7 @@ public function isDeleting() : bool
      *
      * @return bool
      */
-    public function isRestoring() : bool
+    public function isRestoring(): bool
     {
         return $this->option('restore') == 'true';
     }
@@ -198,7 +198,7 @@ public function isRestoring() : bool
      *
      * @return array
      */
-    public function getUsers() : array
+    public function getUsers(): array
     {
         /** @var \Adldap\Query\Builder $query */
         $query = Resolver::query();
@@ -234,7 +234,7 @@ public function getUsers() : array
      *
      * @return bool
      */
-    protected function save(User $user, Model $model) : bool
+    protected function save(User $user, Model $model): bool
     {
         if ($model->save() && $model->wasRecentlyCreated) {
             Event::dispatch(new Imported($user, $model));
@@ -313,7 +313,7 @@ protected function delete(User $user, Model $model)
      *
      * @return Model
      */
-    protected function model() : Model
+    protected function model(): Model
     {
         if (! $this->model) {
             $this->model = $this->option('model') ?? Config::get('ldap_auth.model') ?: $this->determineModel();
@@ -355,7 +355,7 @@ protected function determineModel()
      *
      * @return bool
      */
-    protected function isUsingSoftDeletes(Model $model) : bool
+    protected function isUsingSoftDeletes(Model $model): bool
     {
         return method_exists($model, 'trashed');
     }

src/Commands/SyncPassword.php

@@ -41,7 +41,7 @@ public function __construct(Model $model, array $credentials = [])
      *
      * @return Model
      */
-    public function handle() : Model
+    public function handle(): Model
     {
         if ($this->hasPasswordColumn()) {
             $password = $this->canSync() ?
@@ -82,7 +82,7 @@ protected function applyPassword($password)
      *
      * @return bool
      */
-    protected function passwordNeedsUpdate($password = null) : bool
+    protected function passwordNeedsUpdate($password = null): bool
     {
         $current = $this->currentModelPassword();
 
@@ -98,7 +98,7 @@ protected function passwordNeedsUpdate($password = null) : bool
      *
      * @return bool
      */
-    protected function hasPasswordColumn() : bool
+    protected function hasPasswordColumn(): bool
     {
         return ! is_null($this->column());
     }
@@ -128,7 +128,7 @@ protected function currentModelPassword()
      *
      * @return bool
      */
-    protected function canSync() : bool
+    protected function canSync(): bool
     {
         return array_key_exists('password', $this->credentials) && $this->syncing();
     }
@@ -138,7 +138,7 @@ protected function canSync() : bool
      *
      * @return bool
      */
-    protected function syncing() : bool
+    protected function syncing(): bool
     {
         return Config::get('ldap_auth.passwords.sync', false);
     }

src/Listeners/BindsLdapUserModel.php

@@ -42,7 +42,7 @@ public function handle($event)
      *
      * @return bool
      */
-    protected function isUsingAdldapProvider($guard = null) : bool
+    protected function isUsingAdldapProvider($guard = null): bool
     {
         return Auth::guard($guard)->getProvider() instanceof DatabaseUserProvider;
     }
@@ -54,7 +54,7 @@ protected function isUsingAdldapProvider($guard = null) : bool
      *
      * @return bool
      */
-    protected function canBind(Authenticatable $user) : bool
+    protected function canBind(Authenticatable $user): bool
     {
         return array_key_exists(HasLdapUser::class, class_uses_recursive($user)) && is_null($user->ldap);
     }

src/Resolvers/ResolverInterface.php

@@ -67,33 +67,33 @@ public function authenticate(User $user, array $credentials = []);
      *
      * @return \Adldap\Query\Builder
      */
-    public function query() : Builder;
+    public function query(): Builder;
 
     /**
      * Retrieves the configured LDAP users username attribute.
      *
      * @return string
      */
-    public function getLdapDiscoveryAttribute() : string;
+    public function getLdapDiscoveryAttribute(): string;
 
     /**
      * Retrieves the configured LDAP users authenticatable username attribute.
      *
      * @return string
      */
-    public function getLdapAuthAttribute() : string;
+    public function getLdapAuthAttribute(): string;
 
     /**
      * Retrieves the configured database username attribute.
      *
      * @return string
      */
-    public function getDatabaseUsernameColumn() : string;
+    public function getDatabaseUsernameColumn(): string;
 
     /**
      * Retrieves the configured database identifier attribute.
      *
      * @return string
      */
-    public function getDatabaseIdColumn() : string;
+    public function getDatabaseIdColumn(): string;
 }

src/Resolvers/UserResolver.php

@@ -130,7 +130,7 @@ public function authenticate(User $user, array $credentials = [])
     /**
      * {@inheritdoc}
      */
-    public function query() : Builder
+    public function query(): Builder
     {
         $query = $this->getLdapAuthProvider()->search()->users();
 
@@ -151,31 +151,31 @@ public function query() : Builder
     /**
      * {@inheritdoc}
      */
-    public function getLdapDiscoveryAttribute() : string
+    public function getLdapDiscoveryAttribute(): string
     {
         return Config::get('ldap_auth.identifiers.ldap.locate_users_by', 'userprincipalname');
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getLdapAuthAttribute() : string
+    public function getLdapAuthAttribute(): string
     {
         return Config::get('ldap_auth.identifiers.ldap.bind_users_by', 'distinguishedname');
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getDatabaseUsernameColumn() : string
+    public function getDatabaseUsernameColumn(): string
     {
         return Config::get('ldap_auth.identifiers.database.username_column', 'email');
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getDatabaseIdColumn() : string
+    public function getDatabaseIdColumn(): string
     {
         return Config::get('ldap_auth.identifiers.database.guid_column', 'objectguid');
     }
@@ -199,7 +199,7 @@ protected function getPasswordFromCredentials($credentials)
      *
      * @return ProviderInterface
      */
-    protected function getLdapAuthProvider() : ProviderInterface
+    protected function getLdapAuthProvider(): ProviderInterface
     {
         return $this->ldap->getProvider($this->connection ?? $this->getLdapAuthConnectionName());
     }
@@ -209,7 +209,7 @@ protected function getLdapAuthProvider() : ProviderInterface
      *
      * @return UserProvider
      */
-    protected function getAppAuthProvider() : UserProvider
+    protected function getAppAuthProvider(): UserProvider
     {
         return Auth::guard()->getProvider();
     }