-
-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apply PHP 7.4 syntax and typed property #54
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @samsonasik!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, loads of failures:
There were 5 errors:
1) MezzioTest\Authentication\OAuth2\Repository\Pdo\AccessTokenRepositoryFactoryTest::testFactory
TypeError: Typed property MezzioTest\Authentication\OAuth2\Repository\Pdo\AccessTokenRepositoryFactoryTest::$container must be an instance of Psr\Container\ContainerInterface, Prophecy\Prophecy\ObjectProphecy used
/github/workspace/test/Repository/Pdo/AccessTokenRepositoryFactoryTest.php:22
phpvfscomposer:///github/workspace/vendor/phpunit/phpunit/phpunit:97
2) MezzioTest\Authentication\OAuth2\Repository\Pdo\AuthCodeRepositoryFactoryTest::testFactory
TypeError: Typed property MezzioTest\Authentication\OAuth2\Repository\Pdo\AuthCodeRepositoryFactoryTest::$container must be an instance of Psr\Container\ContainerInterface, Prophecy\Prophecy\ObjectProphecy used
/github/workspace/test/Repository/Pdo/AuthCodeRepositoryFactoryTest.php:22
phpvfscomposer:///github/workspace/vendor/phpunit/phpunit/phpunit:97
3) MezzioTest\Authentication\OAuth2\Repository\Pdo\ClientRepositoryFactoryTest::testFactory
TypeError: Typed property MezzioTest\Authentication\OAuth2\Repository\Pdo\ClientRepositoryFactoryTest::$container must be an instance of Psr\Container\ContainerInterface, Prophecy\Prophecy\ObjectProphecy used
/github/workspace/test/Repository/Pdo/ClientRepositoryFactoryTest.php:22
phpvfscomposer:///github/workspace/vendor/phpunit/phpunit/phpunit:97
4) MezzioTest\Authentication\OAuth2\Repository\Pdo\RefreshTokenRepositoryFactoryTest::testFactory
TypeError: Typed property MezzioTest\Authentication\OAuth2\Repository\Pdo\RefreshTokenRepositoryFactoryTest::$container must be an instance of Psr\Container\ContainerInterface, Prophecy\Prophecy\ObjectProphecy used
/github/workspace/test/Repository/Pdo/RefreshTokenRepositoryFactoryTest.php:22
phpvfscomposer:///github/workspace/vendor/phpunit/phpunit/phpunit:97
5) MezzioTest\Authentication\OAuth2\Repository\Pdo\ScopeRepositoryFactoryTest::testFactory
TypeError: Typed property MezzioTest\Authentication\OAuth2\Repository\Pdo\ScopeRepositoryFactoryTest::$container must be an instance of Psr\Container\ContainerInterface, Prophecy\Prophecy\ObjectProphecy used
/github/workspace/test/Repository/Pdo/ScopeRepositoryFactoryTest.php:22
phpvfscomposer:///github/workspace/vendor/phpunit/phpunit/phpunit:97
I think we may need to remove prophecy as part of our refactoring work
e9c198b
to
346fc71
Compare
@Ocramius it should be fixed now. |
4be9da1
to
8d4125b
Compare
Nope 🤔 |
Signed-off-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
8d4125b
to
c06922c
Compare
Let's try again |
@Ocramius All green 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @samsonasik!
Signed-off-by: Abdul Malik Ikhsan samsonasik@gmail.com
Description
Since composer.json require php 7.4, php 7.4 syntax can be applied with typed properties.
final class
, typed properties for protected is allowed if no inherit of parent or used by its final class parent.final class
, typed properties for protected can be changed toprivate
if not used in parent or exists in parent class.