By skeen on

I've installed drupal 5, beta 1 and implemented the sites/all directory

I currently have a structure like this

sites
  |_all
     |_modules
     |     |_foo
     |        |_foo.module
     |
     |_themes
         |_bar_theme

This looks to be working (modules and themes in sites/all are recognized). In looking at some of the core code thought (i.e. common.inc - drupal_system_listing), it looks to be expecting only modules in the top level of sites/all like so

sites
  |_all
      |_foo
           |_foo.module
      |_bar
           |_bar.module

Does anyone have a link for a specific/official description for the use of sites/all?, I can jsut find general description so far.

thanks

⋅ Categories: Drupal 5.x

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

It's just another directory that will be searched for modules and themes. It was created because core modules and themes are in their own directories, and this is a useful way to keep track of any additions you make to your site. You could do the same in sites/default, but if you have a multisite install that wouldn't be global to all sites.

I usually structure mine like:

sites
  |_all
     |_modules
     |     |_contrib
     |     |  |_cck
     |     |  |  |_content.module
     |     |  |_views
     |     |     |_views.module
     |     |
     |     |_custom
     |        |_foo
     |           |_foo.module
     |
     |_themes
     |     |_contrib
     |     |  |_zen
     |     |    |_page.tpl.php
     |     |
     |     |_custom
     |        |_mysite
     |           |_page.tpl.php

(hopefully that makes it in translation :P)

skeen’s picture

skeen commented

but in that case I think common.inc : drupal_system_listing($mask, $directory, $key = 'name', $min_depth = 1) has a bug in the way is sets up site/all in the search.

In the comments for the function it states $directory for example being 'modules' but then later it sets up sites/all like so...

// Always search sites/all/* as well as the global directories
$searchdir[] = 'sites/all';

it should be
$searchdir[] = 'sites/all/' . $directory;

I'll look for a bug and or submit one...

thanks for the clarification webchick

skeen’s picture

skeen commented

http://drupal.org/node/93882