Skip to content
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

Fatal error: Class PHPStan\Type\MixedType contains 2 abstract methods #93

Closed
1 task done
goba opened this issue Jul 4, 2019 · 1 comment · Fixed by #96
Closed
1 task done

Fatal error: Class PHPStan\Type\MixedType contains 2 abstract methods #93

goba opened this issue Jul 4, 2019 · 1 comment · Fixed by #96

Comments

@goba
Copy link
Sponsor

goba commented Jul 4, 2019

How is drupal-check installed?

  • drupal-check is installed using the phar

Environment:

  • OS: MacOS
  • PHP Version: PHP 7.1.23
  • Drupal core: 8.7.3

Describe the bug

Tried to scan redirect 1.4 straight with drupal-check. I tried looking at the phpstan queue if this issue existed and also the history of MixedType. Looks like 0.11.9 is out with both of the fixes drupal-check needed, so I think you should now return to bundling releases? :)

Console output

$ php drupal-check.phar redirect/ -vvv
Current working directory: [...]drupal/web/modules/contrib
Using Drupal root: [...]drupal/web
Using vendor root: [...]drupal/vendor
Using autoloader: [...]drupal/vendor/autoload.php
PHPStan crashed in the previous run probably because of excessive memory consumption.
It consumed around 38 MB of memory.

To avoid this issue, allow to use more memory with the --memory-limit option.
[...]drupal/web/modules/contrib/redirect/redirect.drush.inc

Fatal error: Class PHPStan\Type\MixedType contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (PHPStan\Type\Type::inferTemplateTypes, PHPStan\Type\Type::traverse) in [...]drupal/vendor/phpstan/phpstan/src/Type/MixedType.php on line 18

@goba
Copy link
Sponsor Author

goba commented Jul 4, 2019

Opened #94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant