Only display redundant preview data with preview picture

[friendica.git] / src / App / Authentication.php

diff --git a/src/App/Authentication.php b/src/App/Authentication.php
index 2ffa5814a260b98a22bcb976214a468f26c6aa3c..e3d273747029b452226a9a017737706984af64f8 100644 (file)
--- a/src/App/Authentication.php
+++ b/src/App/Authentication.php
@@ -1,15 +1,30 @@
 <?php
-
 /**
- * @file /src/Core/Authentication.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
 
 namespace Friendica\App;
 
 use Exception;
 use Friendica\App;
-use Friendica\Core\Config\IConfiguration;
-use Friendica\Core\Config\IPConfiguration;
+use Friendica\Core\Config\IConfig;
+use Friendica\Core\PConfig\IPConfig;
 use Friendica\Core\Hook;
 use Friendica\Core\Session;
 use Friendica\Core\System;
@@ -22,7 +37,7 @@ use Friendica\Util\DateTimeFormat;
 use Friendica\Util\Network;
 use Friendica\Util\Strings;
 use LightOpenID;
-use Friendica\Core\L10n\L10n;
+use Friendica\Core\L10n;
 use Psr\Log\LoggerInterface;
 
 /**
@@ -30,7 +45,7 @@ use Psr\Log\LoggerInterface;
  */
 class Authentication
 {
-       /** @var IConfiguration */
+       /** @var IConfig */
        private $config;
        /** @var App\Mode */
        private $mode;
@@ -46,23 +61,23 @@ class Authentication
        private $cookie;
        /** @var Session\ISession */
        private $session;
-       /** @var IPConfiguration */
+       /** @var IPConfig */
        private $pConfig;
 
        /**
         * Authentication constructor.
         *
-        * @param IConfiguration   $config
-        * @param App\Mode        $mode
-        * @param App\BaseURL     $baseUrl
-        * @param L10n            $l10n
-        * @param Database        $dba
-        * @param LoggerInterface $logger
-        * @param User\Cookie     $cookie
+        * @param IConfig          $config
+        * @param App\Mode         $mode
+        * @param App\BaseURL      $baseUrl
+        * @param L10n             $l10n
+        * @param Database         $dba
+        * @param LoggerInterface  $logger
+        * @param User\Cookie      $cookie
         * @param Session\ISession $session
-        * @param IPConfiguration  $pConfig
+        * @param IPConfig         $pConfig
         */
-       public function __construct(IConfiguration $config, App\Mode $mode, App\BaseURL $baseUrl, L10n $l10n, Database $dba, LoggerInterface $logger, User\Cookie $cookie, Session\ISession $session, IPConfiguration $pConfig)
+       public function __construct(IConfig $config, App\Mode $mode, App\BaseURL $baseUrl, L10n $l10n, Database $dba, LoggerInterface $logger, User\Cookie $cookie, Session\ISession $session, IPConfig $pConfig)
        {
                $this->config  = $config;
                $this->mode    = $mode;
@@ -76,7 +91,7 @@ class Authentication
        }
 
        /**
-        * @brief Tries to auth the user from the cookie or session
+        * Tries to auth the user from the cookie or session
         *
         * @param App   $a      The Friendica Application context
         *
@@ -192,7 +207,7 @@ class Authentication
 
                // if it's an email address or doesn't resolve to a URL, fail.
                if ($noid || strpos($openid_url, '@') || !Network::isUrlValid($openid_url)) {
-                       notice($this->l10n->t('Login failed.') . EOL);
+                       notice($this->l10n->t('Login failed.'));
                        $this->baseUrl->redirect();
                }
 
@@ -255,7 +270,7 @@ class Authentication
                        }
                } catch (Exception $e) {
                        $this->logger->warning('authenticate: failed login attempt', ['action' => 'login', 'username' => Strings::escapeTags($username), 'ip' => $_SERVER['REMOTE_ADDR']]);
-                       info($this->l10n->t('Login failed. Please check your credentials.' . EOL));
+                       notice($this->l10n->t('Login failed. Please check your credentials.'));
                        $this->baseUrl->redirect();
                }
 
@@ -283,7 +298,7 @@ class Authentication
        }
 
        /**
-        * @brief Sets the provided user's authenticated session
+        * Sets the provided user's authenticated session
         *
         * @param App   $a           The Friendica application context
         * @param array $user_record The current "user" record
@@ -373,9 +388,7 @@ class Authentication
                        if ($user_record['login_date'] <= DBA::NULL_DATETIME) {
                                info($this->l10n->t('Welcome %s', $user_record['username']));
                                info($this->l10n->t('Please upload a profile photo.'));
-                               $this->baseUrl->redirect('profile_photo/new');
-                       } else {
-                               info($this->l10n->t("Welcome back %s", $user_record['username']));
+                               $this->baseUrl->redirect('settings/profile/photo/new');
                        }
                }