search

apache / cordova-plugman

Apache Cordova Plugman
https://cordova.apache.org/
Apache License 2.0
400 stars 151 forks source link

CB-6124 cordova plugin remove x fails when a plugin directory is missing #57

Closed jsoref closed 10 years ago

jsoref commented 10 years ago

https://issues.apache.org/jira/browse/CB-6124

jsoref commented 10 years ago

OK, here's a version (untested, should be squished) which does what I think we want.

It adds a few more verbose bits, it makes things use early returns, and it consistently uses a plugman variable (which avoids the risk of the wrong number of ../'s ...)

agrieve commented 10 years ago

Broken some (poorly written) tests and one of your early returns was missing a !.

I fixed the ! when i squished your commits, and I deleted the tests in a follow-up commit.