Merge pull request #680 from systemli/fix/qrcode

fix: migrate to new endroid/qr-code API
systemli · Dec 1, 2024 · c073f91 · c073f91
2 parents 049b3dd + b4690b4
commit c073f91
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/src/Controller/TwofactorController.php b/src/Controller/TwofactorController.php
@@ -312,16 +312,17 @@ public function displayTotpQrCode(TotpAuthenticatorInterface $totpAuthenticator)
             throw new NotFoundHttpException('Cannot display QR code');
         }
 
-        $result = Builder::create()
-            ->writer(new PngWriter())
-            ->writerOptions([])
-            ->data($totpAuthenticator->getQRContent($user))
-            ->encoding(new Encoding('UTF-8'))
-            ->errorCorrectionLevel(ErrorCorrectionLevel::High)
-            ->size(320)
-            ->margin(20)
-            ->roundBlockSizeMode(RoundBlockSizeMode::Margin)
-            ->build();
+        $builder = new Builder(
+            writer: new PngWriter(),
+            writerOptions: [],
+            data: $totpAuthenticator->getQRContent($user),
+            encoding: new Encoding('UTF-8'),
+            errorCorrectionLevel: ErrorCorrectionLevel::High,
+            size: 320,
+            margin: 20,
+            roundBlockSizeMode: RoundBlockSizeMode::Margin,
+        );
+        $result = $builder->build();
 
         return new Response($result->getString(), Response::HTTP_OK, ['Content-Type' => 'image/png']);
     }