Fixed bolt protocol disconnect packet not sent upon object destroy #616
static-analysis.yml
on: pull_request
Lint & Analyse
28s
Annotations
10 errors and 12 warnings
RiskyTruthyFalsyComparison:
src/Bolt/BoltConnection.php#L299
src/Bolt/BoltConnection.php:299:13: RiskyTruthyFalsyComparison: Operand of type null|string contains type string, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Bolt/BoltConnection.php#L302
src/Bolt/BoltConnection.php:302:13: RiskyTruthyFalsyComparison: Operand of type float|null contains type float, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Bolt/Session.php#L153
src/Bolt/Session.php:153:13: RiskyTruthyFalsyComparison: Operand of type float|null contains type float, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Common/GeneratorHelper.php#L37
src/Common/GeneratorHelper.php:37:17: RiskyTruthyFalsyComparison: Operand of type float|null contains type float, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Databags/TransactionConfiguration.php#L102
src/Databags/TransactionConfiguration.php:102:13: RiskyTruthyFalsyComparison: Operand of type iterable<string, array<array-key, mixed>|null|scalar>|null contains type iterable<string, array<array-key, mixed>|null|scalar>, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Databags/TransactionConfiguration.php#L106
src/Databags/TransactionConfiguration.php:106:13: RiskyTruthyFalsyComparison: Operand of type float|null contains type float, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Types/AbstractCypherSequence.php#L285
src/Types/AbstractCypherSequence.php:285:17: RiskyTruthyFalsyComparison: Operand of type callable(TValue:Laudis\Neo4j\Types\AbstractCypherSequence as mixed, TValue:Laudis\Neo4j\Types\AbstractCypherSequence as mixed):int|null contains type callable(TValue:Laudis\Neo4j\Types\AbstractCypherSequence as mixed, TValue:Laudis\Neo4j\Types\AbstractCypherSequence as mixed):int, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
RiskyTruthyFalsyComparison:
src/Types/Map.php#L193
src/Types/Map.php:193:21: RiskyTruthyFalsyComparison: Operand of type callable(string, string):int|null contains type callable(string, string):int, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
|
UnnecessaryVarAnnotation:
tests/Unit/CypherListTest.php#L447
tests/Unit/CypherListTest.php:447:18: UnnecessaryVarAnnotation: The @var int annotation for $sumAfter is unnecessary (see https://psalm.dev/212)
|
Lint & Analyse
Process completed with exit code 2.
|
Lint & Analyse
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/cache@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Lint & Analyse
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
PossiblyUndefinedIntArrayOffset:
src/Authentication/Authenticate.php#L85
src/Authentication/Authenticate.php:85:21: PossiblyUndefinedIntArrayOffset: Possibly undefined array key (see https://psalm.dev/215)
|
MoreSpecificReturnType:
src/Bolt/SslConfigurationFactory.php#L29
src/Bolt/SslConfigurationFactory.php:29:16: MoreSpecificReturnType: The declared return type 'array{0: ''|'s'|'ssc', 1: array{SNI_enabled?: bool, allow_self_signed?: bool, peer_name?: string, verify_peer?: bool}}' for Laudis\Neo4j\Bolt\SslConfigurationFactory::create is more specific than the inferred return type 'list{string, array{SNI_enabled?: bool, allow_self_signed?: bool, peer_name?: string, verify_peer?: bool}}' (see https://psalm.dev/070)
|
LessSpecificReturnStatement:
src/Bolt/SslConfigurationFactory.php#L46
src/Bolt/SslConfigurationFactory.php:46:20: LessSpecificReturnStatement: The type 'list{non-empty-string, array{SNI_enabled?: bool, allow_self_signed?: bool, peer_name?: string, verify_peer?: bool}}' is more general than the declared return type 'array{0: ''|'s'|'ssc', 1: array{SNI_enabled?: bool, allow_self_signed?: bool, peer_name?: string, verify_peer?: bool}}' for Laudis\Neo4j\Bolt\SslConfigurationFactory::create (see https://psalm.dev/129)
|
LessSpecificReturnStatement:
src/Bolt/SslConfigurationFactory.php#L49
src/Bolt/SslConfigurationFactory.php:49:16: LessSpecificReturnStatement: The type 'list{string, array<never, never>}' is more general than the declared return type 'array{0: ''|'s'|'ssc', 1: array{SNI_enabled?: bool, allow_self_signed?: bool, peer_name?: string, verify_peer?: bool}}' for Laudis\Neo4j\Bolt\SslConfigurationFactory::create (see https://psalm.dev/129)
|
InvalidArgument:
src/Common/SysVSemaphore.php#L44
src/Common/SysVSemaphore.php:44:25: InvalidArgument: Argument 1 of Laudis\Neo4j\Common\SysVSemaphore::__construct expects resource, but SysvSemaphore|false provided (see https://psalm.dev/004)
|
InvalidArgument:
src/Common/SysVSemaphore.php#L50
src/Common/SysVSemaphore.php:50:29: InvalidArgument: Argument 1 of sem_acquire expects SysvSemaphore, but resource provided (see https://psalm.dev/004)
|
InvalidArgument:
src/Common/SysVSemaphore.php#L61
src/Common/SysVSemaphore.php:61:26: InvalidArgument: Argument 1 of sem_release expects SysvSemaphore, but resource provided (see https://psalm.dev/004)
|
MissingTemplateParam:
src/Databags/DatabaseInfo.php#L23
src/Databags/DatabaseInfo.php:23:13: MissingTemplateParam: Laudis\Neo4j\Databags\DatabaseInfo has missing template params when extending Laudis\Neo4j\Types\AbstractCypherObject, expecting 2 (see https://psalm.dev/182)
|
MissingTemplateParam:
src/Databags/Plan.php#L27
src/Databags/Plan.php:27:13: MissingTemplateParam: Laudis\Neo4j\Databags\Plan has missing template params when extending Laudis\Neo4j\Types\AbstractCypherObject, expecting 2 (see https://psalm.dev/182)
|
MissingTemplateParam:
src/Databags/ProfiledPlan.php#L26
src/Databags/ProfiledPlan.php:26:13: MissingTemplateParam: Laudis\Neo4j\Databags\ProfiledPlan has missing template params when extending Laudis\Neo4j\Types\AbstractCypherObject, expecting 2 (see https://psalm.dev/182)
|