Koodimonni / Composer-Dropin-Installer

Use composer for installing dropin dependencies
Do What The F*ck You Want To Public License
45 stars 12 forks source link

Only supports moving one file per package #11

Open ryanboswell opened 7 years ago

ryanboswell commented 7 years ago

It doesn't look like the plugin supports moving more than one drop in file per package, it's probably a relatively rare case, but WordPress plugins like https://github.com/stuttter/ludicrousdb have "drop ins" that go to multiple places. When we ran it though, only one file was moved and the others were left untouched.

I'm pretty sure that I've narrowed it down to a few lines in the getPaths method, where the files value is overwritten, which causes problems for multiple dropin files going to the same place from the same source, only the last one in the list is actually moved.

onnimonni commented 7 years ago

Cool! Can you submit a PR?