* [FIX] Fixed issue when retrieveing Authenticator plugin data for th…

…e current user. Closes #442. Thanks to @fellsho, @st3reo and @HellChords for the feedback
nuxsmin · Mar 2, 2017 · 57416b6 · 57416b6
1 parent a408962
commit 57416b6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/inc/Plugins/Authenticator/ActionController.class.php b/inc/Plugins/Authenticator/ActionController.class.php
@@ -26,7 +26,6 @@
 
 use SP\Controller\ItemControllerInterface;
 use SP\Controller\RequestControllerTrait;
-use SP\Core\ActionsInterface;
 use SP\Core\Plugin\PluginDataStore;
 use SP\Core\Session as CoreSession;
 use SP\DataModel\PluginData;

diff --git a/inc/Plugins/Authenticator/PreferencesController.class.php b/inc/Plugins/Authenticator/PreferencesController.class.php
@@ -82,12 +82,12 @@ public function getSecurityTab()
             if (!$AuthenticatorData->isTwofaEnabled()) {
                 $IV = Util::generateRandomBytes();
                 $AuthenticatorData->setIV($IV);
-
-                Session::setUserData($AuthenticatorData);
             } else {
                 $IV = $AuthenticatorData->getIV();
             }
 
+            Session::setUserData($AuthenticatorData);
+
             $twoFa = new Authenticator($UserData->getUserId(), $UserData->getUserLogin(), $IV);
 
             $this->Controller->view->assign('qrCode', !$AuthenticatorData->isTwofaEnabled() ? $twoFa->getUserQRCode() : '');