Skip to content
This repository has been archived by the owner on Jul 24, 2023. It is now read-only.

Commit

Permalink
Update SyncPassword.php
Browse files Browse the repository at this point in the history
The config states that `column` should be set to null if there is no password column. Since PHP 7.0 doesn't support nullable return types, we can't type hint the return from this method.

Config reference:
https://github.com/Adldap2/Adldap2-Laravel/blob/3e118ce82c7fb8d90d7fedfe331c825a5812e904/src/Config/auth.php#L214
  • Loading branch information
36864 authored Dec 22, 2017
1 parent 3e118ce commit da5deed
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Commands/SyncPassword.php
Original file line number Diff line number Diff line change
Expand Up @@ -145,9 +145,9 @@ protected function syncing() : bool
/**
* Retrieves the password column to use.
*
* @return string
* @return string|null
*/
protected function column() : string
protected function column()
{
return Config::get('adldap_auth.passwords.column', 'password');
}
Expand Down

0 comments on commit da5deed

Please sign in to comment.