Allow placing libraries in root directory

I work with Rob and asked him to write this patch, which is now in use with our internal base Drupal install. This issue was raised earlier at http://drupal.org/node/320562#comment-1626450. The install profile option was given does not work in our case. We have 3 install profiles so we'd have to duplicate code or use symbolic links in place of this patch.

It would be great to get this patch (or a variation) committed so we don't have to maintain it separately over the long term. We will be available to revise the patch and/or apply to the Libraries API if needed.

Please explain why sites/all/libraries does not work.

"sites/all/libraries" does not work for this case because the sites directory does not exist in our internal Drupal distribution (think of Acquia Drupal as a parallel). "/sites" is completely different for every site/install and is normally checked out from a separate code repository than the base Drupal distribution. Since we use WYSIWYG on most sites, it makes sense to include the module and necessary editor libraries directly in the distribution.

I understand that this is an edge case but I'd assume that we're not the only shop needing something like this. Also, this patch would make the libraries directory follow the override convention set in core for modules and themes. Example: /modules, /profiles/example/modules, sites/all/modules or sites/example.com/modules).

ok, let's do this - however, not for Wysiwyg, but the new Libraries API, Wysiwyg module requires in 3.x.

Tagging.

@sun - I missed your earlier comment. Thanks for accepting the change for Libraries API / WYSIWYG 3.x. We'll maintain this patch for the 2.x life-cycle (unless the patch magically gets committed of course).

Updated / rerolled patch for Libaries API.
I added an extensive code comment and changed the order to match the core module/theme lookup pattern.
Tested it to work as intended.

I just realized that a drupalroot/libraries folder would actually be the correct place for the core jquery / jqueryUI libraries instead of that crazy mess of a /misc folder. Since I guess the goal is for Libraries API to end up in core.
(Yeah, I know, it's really just a bump...)

+++ libraries.module	23 Feb 2010 07:42:27 -0000
@@ -71,6 +71,11 @@ function libraries_get_libraries() {
+  // Similarly to Drupal core providing modules and themes in the 'modules' and
+  // 'themes' directories in the Drupal root directory, certain distributions may
+  // want to place libraries in a 'libraries' directory in the Drupal root directory.
+  $searchdir[] = $directory; ¶

Note: Comment exceeds 80 chars and trailing white-space.

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Powered by Dreditor.