Skip to content

Commit

Permalink
bug #1172 Removed constructor property promotion to fix compatibility…
Browse files Browse the repository at this point in the history
… with PHP 7.x (webhdx)

This PR was merged into the 2.x branch.

Discussion
----------

Removed constructor property promotion to fix compatibility with PHP 7.x

Looks like #1041 introduced PHP 7.x incompatible change. The issue is constructor property promotion in `\Lexik\Bundle\JWTAuthenticationBundle\Command\EnableEncryptionConfigCommand` class.

Commits
-------

dd21e04 Removed constructor property promotion to fix compatibility with PHP 7.x
  • Loading branch information
chalasr committed Dec 1, 2023
2 parents a17afb7 + dd21e04 commit a3f124d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion Command/EnableEncryptionConfigCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,14 @@ final class EnableEncryptionConfigCommand extends AbstractConfigCommand
*/
protected static $defaultName = 'lexik:jwt:enable-encryption';

private ?AlgorithmManagerFactory $algorithmManagerFactory;

public function __construct(
private ?AlgorithmManagerFactory $algorithmManagerFactory,
?AlgorithmManagerFactory $algorithmManagerFactory = null
) {
parent::__construct();

$this->algorithmManagerFactory = $algorithmManagerFactory;
}

/**
Expand Down

0 comments on commit a3f124d

Please sign in to comment.