Skip to content

Commit deb6713

Browse files
committed
Add PHPStan
1 parent 66898c5 commit deb6713

File tree

7 files changed

+23
-5
lines changed

7 files changed

+23
-5
lines changed

.github/workflows/build.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,10 @@ jobs:
4545
name: Run CS
4646
run: vendor/bin/phpcs --standard=PSR2 src/
4747

48+
-
49+
name: Run PHPStan
50+
run: vendor/bin/phpstan analyse
51+
4852
-
4953
name: Run PHPSpec
5054
run: vendor/bin/phpspec run

composer.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@
1919
},
2020
"require-dev": {
2121
"squizlabs/php_codesniffer": "^3.7",
22-
"phpspec/phpspec": "^7.3"
22+
"phpspec/phpspec": "^7.3",
23+
"phpstan/phpstan": "^1.11"
2324
},
2425
"autoload": {
2526
"psr-4": {

phpstan.neon

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
parameters:
2+
level: 6
3+
paths:
4+
- src

src/Api/EcRequest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ private function calculateMac(): string
165165

166166
private function convertAmountToString(float $amount): string
167167
{
168-
return str_pad((string)round($amount, 2)*100, 9, '0', STR_PAD_LEFT);
168+
return str_pad((string) (round($amount, 2)*100), 9, '0', STR_PAD_LEFT);
169169
}
170170

171171
private function validate(): void

src/Notification/Request.php

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ class Request implements Signed
4141

4242
private ?string $scadenza_pan;
4343

44+
/**
45+
* @param array<string, string|int> $rawParams
46+
*/
4447
private function __construct(array $rawParams)
4548
{
4649
$this->checkForMissingParameters($rawParams);
@@ -68,15 +71,15 @@ private function __construct(array $rawParams)
6871
$this->session_id = $rawParams['session_id'] ?? null;
6972
}
7073

71-
public static function buildFromHttpRequest(ServerRequestInterface $request): static
74+
public static function buildFromHttpRequest(ServerRequestInterface $request): self
7275
{
7376
if (strtoupper($request->getMethod()) === 'POST') {
7477
$rawParams = $request->getParsedBody();
7578
} else {
7679
$rawParams = $request->getQueryParams();
7780
}
7881

79-
return new static($rawParams);
82+
return new self($rawParams);
8083
}
8184

8285
public function getAlias(): string
@@ -215,6 +218,9 @@ private function checkForMissingParameters(array $rawParams): void
215218
}
216219
}
217220

221+
/**
222+
* @param array<string, string|int> $rawParams
223+
*/
218224
private function validateParameters(array $rawParams): void
219225
{
220226
$rawAmount = $rawParams['importo'];

src/Signature/DefaultSignatureHashingManager.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public function hashSignatureString(string $string, string $method): string
2121
return $encodedString;
2222
}
2323

24-
private function mustEncodeHashResultAsUrlencodedBase64($method): bool
24+
private function mustEncodeHashResultAsUrlencodedBase64(string $method): bool
2525
{
2626
return $method === SignatureMethod::MD5_METHOD;
2727
}

src/Signature/Signable.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@
44

55
interface Signable
66
{
7+
/**
8+
* @return array<string, string|int>
9+
*/
710
public function getSignatureData(): array;
811

912
public function setSignature(string $signature): static;

0 commit comments

Comments
 (0)