The admin settings page says "Once you download the script, move it to sites/all/libraries/modernizr or equivalent and clear your cache." However this is only true if the libraries module is installed. If libraries is not installed, the path sites/all/libraries/modernizr is not scanned for the modernizr js, so the installation fails even though you have followed the instructions. The "or equivalent" part in the help text doesn't help at all.

I'm not sure what the right solution is. I suggest that sites/all/libraries/modernizr should always be scanned, but one could also choose to display a different help text if libraries modules is not enabled.

CommentFileSizeAuthor
#3 1887708-3-download-instructions.patch2.36 KBrupl

Comments

rupl’s picture

You're correct. This stems from the fact that the module opts to use Libraries API but can still operate if you do not have it enabled. You're right though, the instructions should be more specific and more in line with the Status Report page which correctly handles all situations.

This will be resolved before the final 3.0 release.

Next major release after 3.0 will require Libraries API. It's the best way to handle 3rd party libs, and it will reduce this module's codebase/documentation significantly.

marcvangend’s picture

OK, thanks for your response. Libraries API is a good choice afaic, it's used a lot and it would sure help to get rid of this confusing situation.

Until it's resolved, I hope that people who are as confused as I was, will find this issue and know what to do. Just install Libraries people!

rupl’s picture

Status: Active » Fixed
StatusFileSize
new2.36 KB

I committed this patch to dev. It re-words the instructions based on the current status of Libraries API. If Libraries is enabled then the instructions mention sites/all/libraries/modernizr, otherwise a location inside the path of the Modernizr module is used.

Status: Fixed » Closed (fixed)

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