Skip to content

Commit e779422

Browse files
committed
Merge branch 'QA_5_2'
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
2 parents 9931600 + 4bc1ae5 commit e779422

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

ChangeLog

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ phpMyAdmin - ChangeLog
55

66
5.2.1 (not yet released)
77
- issue #17522 Fix case where the routes cache file is invalid
8+
- issue #17506 Fix error when configuring 2FA without XMLWriter or Imagick
89

910
5.2.0 (2022-05-10)
1011
- issue #16521 Upgrade Bootstrap to version 5

libraries/classes/TwoFactor.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,11 @@
1515
use PhpMyAdmin\Plugins\TwoFactor\Key;
1616
use PhpMyAdmin\Plugins\TwoFactorPlugin;
1717
use PragmaRX\Google2FAQRCode\Google2FA;
18+
use XMLWriter;
1819

1920
use function array_merge;
2021
use function class_exists;
22+
use function extension_loaded;
2123
use function in_array;
2224
use function ucfirst;
2325

@@ -123,7 +125,11 @@ public function getAvailableBackends()
123125
$result[] = 'simple';
124126
}
125127

126-
if (class_exists(Google2FA::class) && class_exists(ImageRenderer::class)) {
128+
if (
129+
class_exists(Google2FA::class)
130+
&& class_exists(ImageRenderer::class)
131+
&& (class_exists(XMLWriter::class) || extension_loaded('imagick'))
132+
) {
127133
$result[] = 'application';
128134
}
129135

0 commit comments

Comments
 (0)