Merge pull request #25653 from nextcloud/fix/psalm/offlineuser

Some psalm OfflineUser fixes
This commit is contained in:
kesselb 2021-02-15 17:04:16 +01:00 committed by GitHub
commit a33803a31b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions

View File

@ -134,7 +134,7 @@ class OfflineUser {
* @return string * @return string
*/ */
public function getUID() { public function getUID() {
if (!isset($this->uid)) { if ($this->uid === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->uid; return $this->uid;
@ -145,7 +145,7 @@ class OfflineUser {
* @return string * @return string
*/ */
public function getDN() { public function getDN() {
if (!isset($this->dn)) { if ($this->dn === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->dn; return $this->dn;
@ -156,7 +156,7 @@ class OfflineUser {
* @return string * @return string
*/ */
public function getDisplayName() { public function getDisplayName() {
if (!isset($this->displayName)) { if ($this->displayName === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->displayName; return $this->displayName;
@ -167,7 +167,7 @@ class OfflineUser {
* @return string * @return string
*/ */
public function getEmail() { public function getEmail() {
if (!isset($this->email)) { if ($this->email === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->email; return $this->email;
@ -178,7 +178,7 @@ class OfflineUser {
* @return string * @return string
*/ */
public function getHomePath() { public function getHomePath() {
if (!isset($this->homePath)) { if ($this->homePath === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->homePath; return $this->homePath;
@ -189,7 +189,7 @@ class OfflineUser {
* @return int * @return int
*/ */
public function getLastLogin() { public function getLastLogin() {
if (!isset($this->lastLogin)) { if ($this->lastLogin === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return (int)$this->lastLogin; return (int)$this->lastLogin;
@ -200,7 +200,7 @@ class OfflineUser {
* @return int * @return int
*/ */
public function getDetectedOn() { public function getDetectedOn() {
if (!isset($this->foundDeleted)) { if ($this->foundDeleted === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return (int)$this->foundDeleted; return (int)$this->foundDeleted;
@ -211,7 +211,7 @@ class OfflineUser {
* @return bool * @return bool
*/ */
public function getHasActiveShares() { public function getHasActiveShares() {
if (!isset($this->hasActiveShares)) { if ($this->hasActiveShares === null) {
$this->fetchDetails(); $this->fetchDetails();
} }
return $this->hasActiveShares; return $this->hasActiveShares;