Fix part of #5 - Finder parameter must be an array - Githubissues

Halleck45 / DesignPatternDetector

detection of design patterns in PHP code

MIT License

114 stars 14 forks source link

Fix part of #5 - Finder parameter must be an array #6

Open vukanac opened 7 years ago

vukanac commented 7 years ago

With old composer.json latest phpmetrics was installed and it has the constructor with different parameters. We have to convert accepting argument and options in CLI command from string to array.

I have upgraded libs to the latest
converted CLI params to be array instead of string

Hal/Component/Finder has changed constructor and now accepts array - require options for include and exclude to be arrays
Finder has removed method and introduced instead - required CLI argument for the path to be changed in the array.

Small note:

This part is solved, but due to removed Tokenizer from phpmetrics a new bug show up. I hope that that new bug is not a constraint for this pull request.