Fixed bug -- use of undefined variable.

module/VuFind/src/VuFind/Auth/CAS.php

@@ -57,8 +57,6 @@ class CAS extends AbstractBase
      * @throws AuthException
      * @return void
      */
-
-
     protected function validateConfig()
     {
         $cas = $this->config->CAS;
@@ -128,7 +126,6 @@ class CAS extends AbstractBase
         $casauth = $this->setupCAS();
         $casauth->forceAuthentication();
 
-
         // Check if username is set.
         $username = $casauth->getAttribute($cas->username);
         if (empty($username)) {
@@ -138,7 +135,6 @@ class CAS extends AbstractBase
         // If we made it this far, we should log in the user!
         $user = $this->getUserTable()->getByUsername($username);
 
-
         // Has the user configured attributes to use for populating the user table?
         $attribsToCheck = array(
             "cat_username", "cat_password", "email", "lastname", "firstname",
@@ -201,7 +197,7 @@ class CAS extends AbstractBase
         ) {
             $casauth = $this->setupCAS();
             if ($casauth->checkAuthentication() !== true) {
-                return empty($username);
+                return true;
             }
         }
         return false;