Skip to content

Commit 5949a29

Browse files
committed
use null coalescing operator instead of if checks
1 parent 8bf8800 commit 5949a29

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

lib/TwoFactorAuth.php

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -168,10 +168,7 @@ public function getQRText(string $label, string $secret): string
168168
public function getQrCodeProvider(): IQRCodeProvider
169169
{
170170
// Set default QR Code provider if none was specified
171-
if ($this->qrcodeprovider === null) {
172-
return $this->qrcodeprovider = new QRServerProvider();
173-
}
174-
return $this->qrcodeprovider;
171+
return $this->qrcodeprovider ?? ($this->qrcodeprovider = new QRServerProvider());
175172
}
176173

177174
/**
@@ -200,10 +197,7 @@ public function getRngProvider(): IRNGProvider
200197
public function getTimeProvider(): ITimeProvider
201198
{
202199
// Set default time provider if none was specified
203-
if ($this->timeprovider === null) {
204-
return $this->timeprovider = new LocalMachineTimeProvider();
205-
}
206-
return $this->timeprovider;
200+
return $this->timeprovider ?? ($this->timeprovider = new LocalMachineTimeProvider());
207201
}
208202

209203
/**
@@ -229,7 +223,7 @@ private function codeEquals(string $safe, string $user): bool
229223

230224
private function getTime(?int $time = null): int
231225
{
232-
return ($time === null) ? $this->getTimeProvider()->getTime() : $time;
226+
return $time ?? $this->getTimeProvider()->getTime();
233227
}
234228

235229
private function getTimeSlice(?int $time = null, int $offset = 0): int

0 commit comments

Comments
 (0)