Merge pull request #26 from BaranekD/bugfix

Fixed the bug where isCesnetEligibleLastSeen was not set if user is empty

Author: vyskocilpavel

CHANGELOG.md

@@ -2,6 +2,9 @@
 All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
+#### Fixed
+- Fixed the bug where isCesnetEligibleLastSeen was not set if user is empty
+
 #### Changed
 - Warning in disco-tpl modified due to changes in module perun
 - Removed warning template - it is no longer needed here because it was moved to module perun

lib/Auth/Process/IsCesnetEligible.php

@@ -141,10 +141,12 @@ public function process(&$request)
                     );
                     $this->cesnetEligibleLastSeenValue = $this->cesnetEligibleLastSeenAttribute['value'];
                 }
+            }
 
-                if ($isHostelVerified || (!empty($this->eduPersonScopedAffiliation) && $this->isCesnetEligible())) {
-                    $this->cesnetEligibleLastSeenValue = date("Y-m-d H:i:s");
+            if ($isHostelVerified || (!empty($this->eduPersonScopedAffiliation) && $this->isCesnetEligible())) {
+                $this->cesnetEligibleLastSeenValue = date("Y-m-d H:i:s");
 
+                if (!empty($user)) {
                     if ($this->cesnetEligibleLastSeenAttribute === null) {
                         $this->cesnetEligibleLastSeenAttribute = $this->rpcConnector->get(
                             'attributesManager',