-
Notifications
You must be signed in to change notification settings - Fork 71
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
composer-require-checker check not picking up phpmailer #73
Comments
Reproduction should be a test case that can be integrated with this test suite. In https://github.com/Roave/BackwardCompatibilityCheck/blob/1.0.3/src/SourceLocator/StaticClassMapSourceLocator.php I implemented a source locator capable of working with any |
not sure how to integrate it with the test suite while at work- was just about able to smoosh together a bare-minimum reproduction repo- reproducing the problem with |
@WDD-Marv we all do work as well, heh 😛 I think we might need a better abstraction for detecting files overall though: the current one is very much a hack. |
@Ocramius I'm wondering if it's something specific to the symbol
|
@WDD-Marv no assumptions - everything should be based on AST. The best way to understand why it's not being picked up is with an isolated reproduction scenario that you reduce more and more until you get to the point where a change makes the test pass/break on a very tiny modification. |
additional note: swiftmailer classes aren't being picked up either. |
Stumbled upon the same problem. Might this be related to the fact that swiftmailer registers it's own autoloader? |
fix: Argument::that allows params
With & without
optimize-autoloader
being set totrue
incomposer.json
,composer-require-checker check ./composer.json
listsPHPMailer\PHPMailer\PHPMailer
as an "unknown symbol", which is a little odd as I've got other packages loaded with psr-4 that drop off the list when I pull them in withcomposer require
:sThe text was updated successfully, but these errors were encountered: