Bump the all-actions group with 2 updates #89
php.yml
on: push
Annotations
16 errors and 6 warnings
|
Quality control
Process completed with exit code 2.
|
|
UnusedIssueHandlerSuppression:
psalm-dev.xml#L0
/home/runner/work/simplesamlphp-module-preprodwarning/simplesamlphp-module-preprodwarning/psalm-dev.xml:0:0: UnusedIssueHandlerSuppression: Suppressed issue type "UnresolvableInclude" for /tests/bootstrap.php was not thrown. (see https://psalm.dev/326)
|
|
MissingOverrideAttribute:
tests/src/Controller/PreProdWarningTest.php#L138
tests/src/Controller/PreProdWarningTest.php:138:13: MissingOverrideAttribute: Method SimpleSAML\Test\Module\preprodwarning\Controller\_home_runner_work_simplesamlphp_module_preprodwarning_simplesamlphp_module_preprodwarning_tests_src_Controller_PreProdWarningTest_php_132_3516::resumeprocessing should have the "Override" attribute (see https://psalm.dev/358)
|
|
MissingOverrideAttribute:
tests/src/Controller/PreProdWarningTest.php#L124
tests/src/Controller/PreProdWarningTest.php:124:13: MissingOverrideAttribute: Method SimpleSAML\Test\Module\preprodwarning\Controller\_home_runner_work_simplesamlphp_module_preprodwarning_simplesamlphp_module_preprodwarning_tests_src_Controller_PreProdWarningTest_php_123_3148::loadstate should have the "Override" attribute (see https://psalm.dev/358)
|
|
MissingOverrideAttribute:
tests/src/Controller/PreProdWarningTest.php#L97
tests/src/Controller/PreProdWarningTest.php:97:13: MissingOverrideAttribute: Method SimpleSAML\Test\Module\preprodwarning\Controller\_home_runner_work_simplesamlphp_module_preprodwarning_simplesamlphp_module_preprodwarning_tests_src_Controller_PreProdWarningTest_php_96_2393::loadstate should have the "Override" attribute (see https://psalm.dev/358)
|
|
MissingOverrideAttribute:
tests/src/Controller/PreProdWarningTest.php#L55
tests/src/Controller/PreProdWarningTest.php:55:13: MissingOverrideAttribute: Method SimpleSAML\Test\Module\preprodwarning\Controller\_home_runner_work_simplesamlphp_module_preprodwarning_simplesamlphp_module_preprodwarning_tests_src_Controller_PreProdWarningTest_php_54_1302::info should have the "Override" attribute (see https://psalm.dev/358)
|
|
MissingOverrideAttribute:
tests/src/Controller/PreProdWarningTest.php#L38
tests/src/Controller/PreProdWarningTest.php:38:5: MissingOverrideAttribute: Method SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest::setup should have the "Override" attribute (see https://psalm.dev/358)
|
|
UnusedClass:
tests/src/Controller/PreProdWarningTest.php#L24
tests/src/Controller/PreProdWarningTest.php:24:7: UnusedClass: Class SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest is never used (see https://psalm.dev/075)
|
|
Quality control
Process completed with exit code 2.
|
|
RiskyTruthyFalsyComparison:
src/Controller/PreProdWarning.php#L123
src/Controller/PreProdWarning.php:123:13: RiskyTruthyFalsyComparison: Operand of type null|scalar contains type scalar, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
|
ArgumentTypeCoercion:
src/Controller/PreProdWarning.php#L121
src/Controller/PreProdWarning.php:121:46: ArgumentTypeCoercion: Argument 1 of SimpleSAML\Auth\State::loadState expects string, but parent type scalar provided (see https://psalm.dev/193)
|
|
UnnecessaryVarAnnotation:
src/Controller/PreProdWarning.php#L120
src/Controller/PreProdWarning.php:120:24: UnnecessaryVarAnnotation: The @var array<array-key, mixed> annotation for $state is unnecessary (see https://psalm.dev/212)
|
|
ClassMustBeFinal:
src/Controller/PreProdWarning.php#L25
src/Controller/PreProdWarning.php:25:7: ClassMustBeFinal: Class SimpleSAML\Module\preprodwarning\Controller\PreProdWarning is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
|
|
MissingOverrideAttribute:
src/Auth/Process/Warning.php#L27
src/Auth/Process/Warning.php:27:5: MissingOverrideAttribute: Method SimpleSAML\Module\preprodwarning\Auth\Process\Warning::process should have the "Override" attribute (see https://psalm.dev/358)
|
|
ClassMustBeFinal:
src/Auth/Process/Warning.php#L17
src/Auth/Process/Warning.php:17:7: ClassMustBeFinal: Class SimpleSAML\Module\preprodwarning\Auth\Process\Warning is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
|
|
Linter
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Root composer.json requires simplesamlphp/simplesamlphp ^2.1.0 -> satisfiable by simplesamlphp/simplesamlphp[2.1.0, ..., v2.4.2].
- simplesamlphp/simplesamlphp[2.1.0, ..., v2.4.2] require ext-session * -> it is missing from your system. Install or enable PHP's session extension.
To enable extensions, verify that they are enabled in your .ini files:
- /etc/php84/php.ini
- /etc/php84/conf.d/00_ctype.ini
- /etc/php84/conf.d/00_curl.ini
- /etc/php84/conf.d/00_dom.ini
- /etc/php84/conf.d/00_iconv.ini
- /etc/php84/conf.d/00_intl.ini
- /etc/php84/conf.d/00_mbstring.ini
- /etc/php84/conf.d/00_openssl.ini
- /etc/php84/conf.d/00_simplexml.ini
- /etc/php84/conf.d/00_tokenizer.ini
- /etc/php84/conf.d/00_xmlwriter.ini
- /etc/php84/conf.d/01_phar.ini
- /etc/php84/conf.d/10_igbinary.ini
You can also run `php --ini` in a terminal to see which files are used by PHP in CLI mode.
Alternatively, you can run Composer with `--ignore-platform-req=ext-session` to temporarily ignore these required extensions.
|
|
PropertyNotSetInConstructor:
tests/src/Controller/PreProdWarningTest.php#L33
tests/src/Controller/PreProdWarningTest.php:33:23: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest::$session is not defined in constructor of SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest or in any methods called in the constructor (see https://psalm.dev/074)
|
|
PropertyNotSetInConstructor:
tests/src/Controller/PreProdWarningTest.php#L30
tests/src/Controller/PreProdWarningTest.php:30:22: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest::$logger is not defined in constructor of SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest or in any methods called in the constructor (see https://psalm.dev/074)
|
|
PropertyNotSetInConstructor:
tests/src/Controller/PreProdWarningTest.php#L27
tests/src/Controller/PreProdWarningTest.php:27:29: PropertyNotSetInConstructor: Property SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest::$config is not defined in constructor of SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest or in any methods called in the constructor (see https://psalm.dev/074)
|
|
ClassMustBeFinal:
tests/src/Controller/PreProdWarningTest.php#L24
tests/src/Controller/PreProdWarningTest.php:24:7: ClassMustBeFinal: Class SimpleSAML\Test\Module\preprodwarning\Controller\PreProdWarningTest is never extended and is not part of the public API, and thus must be made final. (see https://psalm.dev/361)
|
|
UnusedClass:
src/Controller/PreProdWarning.php#L25
src/Controller/PreProdWarning.php:25:7: UnusedClass: Class SimpleSAML\Module\preprodwarning\Controller\PreProdWarning is never used (see https://psalm.dev/075)
|
|
UnusedClass:
src/Auth/Process/Warning.php#L17
src/Auth/Process/Warning.php:17:7: UnusedClass: Class SimpleSAML\Module\preprodwarning\Auth\Process\Warning is never used (see https://psalm.dev/075)
|