From f3760f138dcbaf7a2190336a0378cf1b2190c9f5 Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Mon, 17 Apr 2023 08:30:21 +0200 Subject: Complete PHPStan Level 6 (#5305) * Complete PHPStan Level 6 Fix https://github.com/FreshRSS/FreshRSS/issues/4112 And initiate PHPStan Level 7 * PHPStan Level 6 for tests * Use phpstan/phpstan-phpunit * Update to PHPStan version 1.10 * Fix mixed bug * Fix mixed return bug * Fix paginator bug * Fix FreshRSS_UserConfiguration * A couple more Minz_Configuration bug fixes * A few trivial PHPStan Level 7 fixes * A few more simple PHPStan Level 7 * More files passing PHPStan Level 7 Add interface to replace removed class from https://github.com/FreshRSS/FreshRSS/pull/5251 * A few more PHPStan Level 7 preparations * A few last details --- app/Models/Context.php | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'app/Models/Context.php') diff --git a/app/Models/Context.php b/app/Models/Context.php index b2631291a..03006cbbf 100644 --- a/app/Models/Context.php +++ b/app/Models/Context.php @@ -110,10 +110,9 @@ final class FreshRSS_Context { /** * Initialize the context for the current user. - * @return FreshRSS_UserConfiguration|false * @throws Minz_ConfigurationParamException */ - public static function initUser(string $username = '', bool $userMustExist = true) { + public static function initUser(string $username = '', bool $userMustExist = true): ?FreshRSS_UserConfiguration { FreshRSS_Context::$user_conf = null; if (!isset($_SESSION)) { Minz_Session::init('FreshRSS'); @@ -145,7 +144,7 @@ final class FreshRSS_Context { Minz_Session::unlock(); if (FreshRSS_Context::$user_conf == null) { - return false; + return null; } FreshRSS_Context::$search = new FreshRSS_BooleanSearch(''); @@ -249,8 +248,8 @@ final class FreshRSS_Context { /** * Return the current get as a string or an array. * - * If $array is true, the first item of the returned value is 'f' or 'c' and - * the second is the id. + * If $array is true, the first item of the returned value is 'f' or 'c' or 't' and the second is the id. + * @phpstan-return ($asArray is true ? array{'c'|'f'|'t',bool|int} : string) * @return string|array{string,bool|int} */ public static function currentGet(bool $asArray = false) { -- cgit v1.2.3