While resolving #1675964: Helper function for Drush command checks the wrong path I realized that the function only checks the directory that should contain the JS library instead of checking for the library itself. To reproduce:

$ drush dl modernizr libraries -y
$ drush en libraries modernizr -y
  [it will say it downloaded the library to /sites/all/libraries/modernizr]
$ rm -f /sites/all/libraries/modernizr/modernizr*
$ drush en modernizr
  [it will say the library is already present, but we just deleted it!]

This isn't critical because you'd have to go out of your way to cause this situation. Normal use of the module will never surface this bug.

CommentFileSizeAuthor
#1 1846166-1-drush-dir.patch3.83 KBrupl

Comments

rupl’s picture

Status: Active » Fixed
StatusFileSize
new3.83 KB

This patch was a little more involved than I thought it would be, but it's in dev now:

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.