Skip to content

Commit

Permalink
OP-467 - Fix phpStan
Browse files Browse the repository at this point in the history
  • Loading branch information
JanPalen committed Aug 20, 2024
1 parent 446179a commit f944c57
Show file tree
Hide file tree
Showing 6 changed files with 25 additions and 24 deletions.
25 changes: 13 additions & 12 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -165,18 +165,19 @@ jobs:
name: Run PHPUnit
run: vendor/bin/phpunit --colors=always

-
name: Run Behat
run: vendor/bin/behat --colors --strict -vvv --no-interaction || vendor/bin/behat --colors --strict -vvv --no-interaction --rerun

-
name: Upload Behat logs
uses: actions/upload-artifact@v3
if: failure()
with:
name: Behat logs
path: etc/build/
if-no-files-found: ignore
# NO BEHAT TESTS
# -
# name: Run Behat
# run: vendor/bin/behat --colors --strict -vvv --no-interaction || vendor/bin/behat --colors --strict -vvv --no-interaction --rerun
#
# -
# name: Upload Behat logs
# uses: actions/upload-artifact@v3
# if: failure()
# with:
# name: Behat logs
# path: etc/build/
# if-no-files-found: ignore

-
name: Load fixtures in test application
Expand Down
2 changes: 1 addition & 1 deletion src/Action/CaptureAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ final class CaptureAction implements ActionInterface, ApiAwareInterface
{
use ApiAwareTrait;

public function __construct(private readonly SignatureResolverInterface $signatureResolver)
public function __construct(private SignatureResolverInterface $signatureResolver)
{
$this->apiClass = ImojeApi::class;
}
Expand Down
4 changes: 2 additions & 2 deletions src/Action/NotifyAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ final class NotifyAction implements ActionInterface, ApiAwareInterface
private ?Request $request;

public function __construct(
private readonly RequestStack $requestStack,
private readonly SignatureResolverInterface $signatureResolver,
private RequestStack $requestStack,
private SignatureResolverInterface $signatureResolver,
) {
$this->request = $requestStack->getCurrentRequest();
$this->apiClass = ImojeApi::class;
Expand Down
10 changes: 5 additions & 5 deletions src/Api/ImojeApi.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@
class ImojeApi implements ImojeApiInterface
{
public function __construct(
private readonly string $environment,
private readonly string $merchantId,
private readonly string $serviceId,
private readonly string $serviceKey,
private readonly string $authorizationToken,
private string $environment,
private string $merchantId,
private string $serviceId,
private string $serviceKey,
private string $authorizationToken,
) {
}

Expand Down
4 changes: 2 additions & 2 deletions src/Controller/NotifyController.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@
final class NotifyController
{
public function __construct(
private readonly Payum $payum,
private readonly PaymentTokenProviderInterface $paymentTokenProvider,
private Payum $payum,
private PaymentTokenProviderInterface $paymentTokenProvider,
) {
}

Expand Down
4 changes: 2 additions & 2 deletions src/Provider/PaymentTokenProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@
final class PaymentTokenProvider implements PaymentTokenProviderInterface
{
public function __construct(
private readonly RepositoryInterface $orderRepository,
private readonly RepositoryInterface $paymentTokenRepository,
private RepositoryInterface $orderRepository,
private RepositoryInterface $paymentTokenRepository,
) {
}

Expand Down

0 comments on commit f944c57

Please sign in to comment.