At my place of work, we have a non-standard workflow for working on development sites. We developed it at the same time that Drush was being written because we didn't know about Drush at the time. Sadly, due to the high number of multi-site Drupal installations we maintain, it ended up working differently from Drush.

ASIDE: The basic idea is that all sites share the same Drupal core, but we symlink sites/default to a copy of the site we're working on (remember, this is our development server). We check sites/default into our version-control. Because all sites share the other directories (including sites/all), and because sites developed at different times occasionally use different versions of the same libraries, we have to install libraries into sites/default/libraries.

Having the Libraries API support looking for libraries in sites/default/libraries (similar to the WYSIWYG module) would be really useful for us.

CommentFileSizeAuthor
#4 add_sites_default_to_search_path-1244108-3.patch3.18 KBmparker17
#2 add_sites_default_to_search_path-1244108-2.patch431 bytesmparker17
#1 add_sites_default_to_search_path-1244108.patch431 bytesmparker17
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
FileSize
add_sites_default_to_search_path-1244108.patch431 bytes

I've prepared a patch that adds this functionality to version 7.x-1.x-dev.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
FileSize
add_sites_default_to_search_path-1244108-2.patch431 bytes

Whoops... changed the version number too soon.

Here's a patch for 6.x-1.x-dev too. I've tested this one on the project I'm working on. jquery_ui successfully detects and uses the library.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Active » Postponed (maintainer needs more info)

Not sure about this. $config (conf_path()) should already be set to 'sites/default' if your site lives in sites/default.

Why is that not the case?

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
Status: Postponed (maintainer needs more info) » Active
FileSize
add_sites_default_to_search_path-1244108-3.patch3.18 KB

And, here's a patch for 7.x-2.x. It's missing tests though...

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented

@sun: where is $config(conf_path()) set? I cannot find any mention of it in settings.php.

Update:

  print_r(conf_path());

Returns sites/default on the drupal-6.22 site I'm working on right now.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
Priority: Normal » Minor
Status: Active » Postponed

My sincere apologies: it looks like I may have jumped the gun here. I missed the fact that the Libraries API was designed to pick up on conf_path().

As far as I know, the Libraries API hasn't found anything in sites/default/libraries in any of the Drupal 6 sites I've been working on lately, but I could be totally wrong and/or I could be setting things up incorrectly.

I'll investigate this further. I'll close the issue if I'm the one at fault here. If there's a bug, I'll file it as another issue. If this is a valid request, I'll mark it as active again with a bit more explanation.

Sorry for taking up your time!

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
Status: Postponed » Closed (works as designed)

Turns out this is my problem... it appears that our script was not naming the folders inside sites/default/libraries correctly, so the module I was trying to use was unable to recognize them.

Again, my apologies for taking up your time.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 commented
Assigned: mparker17 » Unassigned

Unassigning