diff options
| author | jld3103 <jld3103yt@gmail.com> | 2023-03-31 08:53:56 +0200 |
|---|---|---|
| committer | jld3103 <jld3103yt@gmail.com> | 2023-04-05 09:08:56 +0200 |
| commit | d9f85220038b2421870a36f9b6a860d6444a988a (patch) | |
| tree | 10b8cbdb0c0ded7947b50654a4e1d4a30390768a /apps/user_ldap | |
| parent | 54140dd6bfda81276bd889a9a141d51e328100d1 (diff) | |
| download | nextcloud-server-d9f85220038b2421870a36f9b6a860d6444a988a.tar.gz nextcloud-server-d9f85220038b2421870a36f9b6a860d6444a988a.zip | |
Fix types for reading and writing config values
Signed-off-by: jld3103 <jld3103yt@gmail.com>
Diffstat (limited to 'apps/user_ldap')
| -rw-r--r-- | apps/user_ldap/lib/Jobs/Sync.php | 10 | ||||
| -rw-r--r-- | apps/user_ldap/lib/User/User.php | 4 |
2 files changed, 7 insertions, 7 deletions
diff --git a/apps/user_ldap/lib/Jobs/Sync.php b/apps/user_ldap/lib/Jobs/Sync.php index b231089b79b..1ba24af5399 100644 --- a/apps/user_ldap/lib/Jobs/Sync.php +++ b/apps/user_ldap/lib/Jobs/Sync.php @@ -73,10 +73,10 @@ class Sync extends TimedJob { parent::__construct($time); $this->userManager = $userManager; $this->setInterval( - \OC::$server->getConfig()->getAppValue( + (int)\OC::$server->getConfig()->getAppValue( 'user_ldap', 'background_sync_interval', - self::MIN_INTERVAL + (string)self::MIN_INTERVAL ) ); } @@ -97,7 +97,7 @@ class Sync extends TimedJob { $interval = floor(24 * 60 * 60 / $runsPerDay); $interval = min(max($interval, self::MIN_INTERVAL), self::MAX_INTERVAL); - $this->config->setAppValue('user_ldap', 'background_sync_interval', $interval); + $this->config->setAppValue('user_ldap', 'background_sync_interval', (string)$interval); } /** @@ -198,7 +198,7 @@ class Sync extends TimedJob { $cycleData = [ 'prefix' => $this->config->getAppValue('user_ldap', 'background_sync_prefix', null), - 'offset' => (int)$this->config->getAppValue('user_ldap', 'background_sync_offset', 0), + 'offset' => (int)$this->config->getAppValue('user_ldap', 'background_sync_offset', '0'), ]; if ( @@ -255,7 +255,7 @@ class Sync extends TimedJob { * @return bool */ public function qualifiesToRun($cycleData) { - $lastChange = $this->config->getAppValue('user_ldap', $cycleData['prefix'] . '_lastChange', 0); + $lastChange = (int)$this->config->getAppValue('user_ldap', $cycleData['prefix'] . '_lastChange', '0'); if ((time() - $lastChange) > 60 * 30) { return true; } diff --git a/apps/user_ldap/lib/User/User.php b/apps/user_ldap/lib/User/User.php index 15894ce04b7..edf43494777 100644 --- a/apps/user_ldap/lib/User/User.php +++ b/apps/user_ldap/lib/User/User.php @@ -303,7 +303,7 @@ class User { } if (!is_null($attr) - && $this->config->getAppValue('user_ldap', 'enforce_home_folder_naming_rule', true) + && $this->config->getAppValue('user_ldap', 'enforce_home_folder_naming_rule', 'true') ) { // a naming rule attribute is defined, but it doesn't exist for that LDAP user throw new \Exception('Home dir attribute can\'t be read from LDAP for uid: ' . $this->getUsername()); @@ -355,7 +355,7 @@ class User { */ public function markLogin() { $this->config->setUserValue( - $this->uid, 'user_ldap', self::USER_PREFKEY_FIRSTLOGIN, 1); + $this->uid, 'user_ldap', self::USER_PREFKEY_FIRSTLOGIN, '1'); } /** |