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

Fix TypeError thrown during install #51

Merged
merged 1 commit into from
Sep 3, 2021

Conversation

alexstewartja
Copy link
Contributor

This change solves the problem of a fatal error thrown during the execution of the post-install/update script. Specifically:

Fatal error: Uncaught TypeError: Composer\Downloader\DownloadManager::download(): Argument #3 ($prevPackage) must be of type ?Composer\Package\PackageInterface, bool given, called in ...\jakoch\phantomjs-installer\src\PhantomInstaller\Installer.php on line 192 and defined in .../Composer/Downloader/DownloadManager.php:182

This change solves the problem of a fatal error thrown during the execution of the post-install/update script. Specifically:

```
Fatal error: Uncaught TypeError: Composer\Downloader\DownloadManager::download(): Argument jakoch#3 ($prevPackage) must be of type ?Composer\Package\PackageInterface, bool given, called in ...\jakoch\phantomjs-installer\src\PhantomInstaller\Installer.php on line 192 and defined in .../Composer/Downloader/DownloadManager.php:182
```
@jakoch
Copy link
Owner

jakoch commented Sep 3, 2021

Hey Alex, thank you for contributing! 👍🏻

How can i trigger the error itself?
I need to use Composer v1, right? And which version of jakoch/phantomjs-installer did you require?

@jakoch jakoch merged commit ccc6bab into jakoch:master Sep 3, 2021
jakoch added a commit that referenced this pull request Sep 3, 2021
Merge pull request #51 from alexstewartja/composer-fix
Fix TypeError thrown during install
@jakoch
Copy link
Owner

jakoch commented Sep 3, 2021

PR was merged.
Tagged/released new version 3.0.1.
Thank you.

@alexstewartja
Copy link
Contributor Author

No problem Jens. Thanks for the quick merge!

This PR fixes jonnnnyw/php-phantomjs#288, referenced by composer/composer#9698 and discussed in #50

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 this pull request may close these issues.

2 participants