Check if the user exists before trying to set the quota

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

diff --git a/apps/user_ldap/lib/User/User.php b/apps/user_ldap/lib/User/User.php
index 7a840de87f64220676712e3d321bdc5db51c083f..392cb2e51ba96e763b6e30eedee57ed97854bb12 100644 (file)
--- a/apps/user_ldap/lib/User/User.php
+++ b/apps/user_ldap/lib/User/User.php
@@ -498,11 +498,16 @@ class User {
                        }
                }
 
-               if($quota !== false) {
-                       $this->userManager->get($this->uid)->setQuota($quota);
+               $targetUser = $this->userManager->get($this->uid);
+               if ($targetUser) {
+                       if($quota !== false) {
+                               $targetUser->setQuota($quota);
+                       } else {
+                               $this->log->log('not suitable default quota found for user ' . $this->uid . ': [' . $defaultQuota . ']', \OCP\Util::WARN);
+                               $targetUser->setQuota('default');
+                       }
                } else {
-                       $this->log->log('not suitable default quota found for user ' . $this->uid . ': [' . $defaultQuota . ']', \OCP\Util::WARN);
-                       $this->userManager->get($this->uid)->setQuota('default');
+                       $this->log->log('trying to set a quota for user ' . $this->uid . ' but the user is missing', \OCP\Util::ERROR);
                }
        }