clue/graph-composer

Invalid handling of not lowercase package names.

h4cc opened this issue · 1 comments

h4cc commented

There seems to be a issue with packages with non lowercase package names.
Example:
http://hhvm.h4cc.de/graphs/laravel_4_1.png

The package jeremeamia/SuperClosure has two nodes, one in mixed case and the other in lowercase.

clue commented

Thanks for reporting!

The problem actually stems from the fact that laravel/framework depends on jeremeamia/superclosure (lowercase) while its actual package name is jeremeamia/SuperClosure (mixed case).

Composer searches for packages in a case insensitive manner (composer/packagist#186), but still uses the original (mixed case) name for its directories and references.

We should probably use a similar approach to always show the original name, but still accept names in case insensitive manner. The dependency resolution in this library is entirely handled by jms/composer-deps-analyzer, so we'll likely have to forward this bug report. I'll look into this.