diff --git a/lib/SimpleSAML/SessionHandler.php b/lib/SimpleSAML/SessionHandler.php index 6beb5c9476a0d3d75afb7112f77f7c99630571e5..debfba0b5fbf4a92b5fbe1a53b5b96fb066d442d 100644 --- a/lib/SimpleSAML/SessionHandler.php +++ b/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); } } diff --git a/lib/SimpleSAML/Store.php b/lib/SimpleSAML/Store.php index 0b3b8bd3be7406a852b75dee2acc1b813d5caf15..3d638d88a57c09a2e41c121210b6b7c27ece689c 100644 --- a/lib/SimpleSAML/Store.php +++ b/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() { diff --git a/modules/saml/lib/SP/LogoutStore.php b/modules/saml/lib/SP/LogoutStore.php index fa78cdfac16aac7e8c401bbd475c3fadc45398e1..b471f60e022f63b837aec324bbc426b4e4919a0f 100644 --- a/modules/saml/lib/SP/LogoutStore.php +++ b/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); }