Fix an issue with data types returned by SimpleSAML_Store.

lib/SimpleSAML/SessionHandler.php

@@ -110,6 +110,7 @@ abstract class SimpleSAML_SessionHandler
         if ($store === false) {
             self::$sessionHandler = new SimpleSAML_SessionHandlerPHP();
         } else {
+            /** @var SimpleSAML_Store $store At this point, $store can only be an object */
             self::$sessionHandler = new SimpleSAML_SessionHandlerStore($store);
         }
     }

lib/SimpleSAML/Store.php

@@ -14,7 +14,7 @@ abstract class SimpleSAML_Store
      *
      * This is false if the data store isn't enabled, and null if we haven't attempted to initialize it.
      *
-     * @var SimpleSAML_Store|boolean|null
+     * @var SimpleSAML_Store|false|null
      */
     private static $instance;
 
@@ -22,7 +22,7 @@ abstract class SimpleSAML_Store
     /**
      * Retrieve our singleton instance.
      *
-     * @return SimpleSAML_Store|boolean  The data store, or false if it isn't enabled.
+     * @return SimpleSAML_Store|false  The data store, or false if it isn't enabled.
      */
     public static function getInstance()
     {

modules/saml/lib/SP/LogoutStore.php

@@ -125,7 +125,7 @@ class sspmod_saml_SP_LogoutStore {
 	/**
 	 * Retrieve all session IDs from a key-value store.
 	 *
-	 * @param SimpleSAML_Store_SQL $store  The datastore.
+	 * @param SimpleSAML_Store $store  The datastore.
 	 * @param string $authId  The authsource ID.
 	 * @param string $nameId  The hash of the users NameID.
 	 * @param array $sessionIndexes  The session indexes.
@@ -234,6 +234,7 @@ class sspmod_saml_SP_LogoutStore {
 			/* We cannot fetch all sessions without a SQL store. */
 			return FALSE;
 		} else {
+			/** @var SimpleSAML_Store $sessions At this point the store cannot be false */
 			$sessions = self::getSessionsStore($store, $authId, $strNameId, $sessionIndexes);
 
 		}