From 1335a0e3cf11a0d4248e9eaaf748b89e6df741ef Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Tue, 4 Jan 2022 13:59:09 +0100 Subject: PHPStan level 5 (#4110) * Fix most PHPDocs errors Contributes to https://github.com/FreshRSS/FreshRSS/issues/4103 https://phpstan.org/writing-php-code/phpdoc-types * Avoid func_get_args Use variadic syntax instead https://php.net/manual/functions.arguments#functions.variable-arg-list And avoid dynamic functions names when possible to more easily identify calls and unused functions. Contributes to https://github.com/FreshRSS/FreshRSS/issues/4103 * PHPStan level 3 * PHPStand level 4 * Update default to PHPStan level 4 * Towards level 5 * Fix level 4 regression * Towards level 5 * Pass PHPStan level 5 * Towards level 6 * Remove erronenous regression from changelog https://github.com/FreshRSS/FreshRSS/pull/4116 --- p/api/greader.php | 25 ++++++++++++++++++++++--- 1 file changed, 22 insertions(+), 3 deletions(-) (limited to 'p/api/greader.php') diff --git a/p/api/greader.php b/p/api/greader.php index 33373293d..d233fd957 100644 --- a/p/api/greader.php +++ b/p/api/greader.php @@ -29,20 +29,36 @@ require(LIB_PATH . '/lib_rss.php'); //Includes class autoloader $ORIGINAL_INPUT = file_get_contents('php://input', false, null, 0, 1048576); if (PHP_INT_SIZE < 8) { //32-bit + /** + * @param string|int $dec + * @return string + */ function dec2hex($dec) { return str_pad(gmp_strval(gmp_init($dec, 10), 16), 16, '0', STR_PAD_LEFT); } + /** + * @param string $hex + * @return string + */ function hex2dec($hex) { - if (!ctype_xdigit($hex)) return 0; + if (!ctype_xdigit($hex)) return '0'; return gmp_strval(gmp_init($hex, 16), 10); } } else { //64-bit + /** + * @param string|int $dec + * @return string + */ function dec2hex($dec) { //http://code.google.com/p/google-reader-api/wiki/ItemId return str_pad(dechex($dec), 16, '0', STR_PAD_LEFT); } + /** + * @param string $hex + * @return string + */ function hex2dec($hex) { - if (!ctype_xdigit($hex)) return 0; - return hexdec($hex); + if (!ctype_xdigit($hex)) return '0'; + return '' . hexdec($hex); } } @@ -79,6 +95,9 @@ function multiplePosts($name) { //https://bugs.php.net/bug.php?id=51633 return $result; } +/** + * @return string + */ function debugInfo() { if (function_exists('getallheaders')) { $ALL_HEADERS = getallheaders(); -- cgit v1.2.3