UserSession class [5] - Refactor src/Module/ files with DI

[friendica.git] / src / Module / Security / OpenID.php

diff --git a/src/Module/Security/OpenID.php b/src/Module/Security/OpenID.php
index 360c9c672f3c6e06c029b9426b9084f29cee498e..2675f1fec335af80af3fa9c6911f855c845f5753 100644 (file)
--- a/src/Module/Security/OpenID.php
+++ b/src/Module/Security/OpenID.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -31,7 +31,7 @@ use LightOpenID;
  */
 class OpenID extends BaseModule
 {
-       public function content(): string
+       protected function content(array $request = []): string
        {
                if (DI::config()->get('system', 'no_openid')) {
                        DI::baseUrl()->redirect();
@@ -73,9 +73,7 @@ class OpenID extends BaseModule
 
                                        DI::auth()->setForUser(DI::app(), $user, true, true);
 
-                                       // just in case there was no return url set
-                                       // and we fell through
-                                       DI::baseUrl()->redirect();
+                                       $this->baseUrl->redirect(DI::session()->pop('return_path', ''));
                                }
 
                                // Successful OpenID login - but we can't match it to an existing account.
@@ -84,14 +82,14 @@ class OpenID extends BaseModule
                                $session->set('openid_identity', $authId);
 
                                // Detect the server URL
-                               $open_id_obj = new LightOpenID(DI::baseUrl()->getHostName());
+                               $open_id_obj = new LightOpenID(DI::baseUrl()->getHostname());
                                $open_id_obj->identity = $authId;
                                $session->set('openid_server', $open_id_obj->discover($open_id_obj->identity));
 
                                if (intval(DI::config()->get('config', 'register_policy')) === \Friendica\Module\Register::CLOSED) {
-                                       notice($l10n->t('Account not found. Please login to your existing account to add the OpenID to it.'));
+                                       DI::sysmsg()->addNotice($l10n->t('Account not found. Please login to your existing account to add the OpenID to it.'));
                                } else {
-                                       notice($l10n->t('Account not found. Please register a new account or login to your existing account to add the OpenID to it.'));
+                                       DI::sysmsg()->addNotice($l10n->t('Account not found. Please register a new account or login to your existing account to add the OpenID to it.'));
                                }
 
                                DI::baseUrl()->redirect('login');