Closed (fixed)
Project:
Drupal core
Version:
5.0-beta1
Component:
base system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
9 Nov 2006 at 18:36 UTC
Updated:
30 Nov 2006 at 09:30 UTC
Jump to comment: Most recent file
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
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';
I believe it should be
$searchdir[] = 'sites/all/' . $directory;
this would put it in line with how config and profile are set up...
$searchdir[] = "profiles/$profile/$directory";
...
$searchdir[] = "$config/$directory";
I attached a patch (just a one line fix). (let me know if it is in the correct format)
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | common.inc_49.patch | 514 bytes | skeen |
| #2 | common.inc_48.patch | 483 bytes | skeen |
| common.inc_47.patch | 483 bytes | skeen |
Comments
Comment #1
skeen commentedComment #2
skeen commentedI read the handbook for creating patches and the coding standards for string concatenation so I think this is a patch in the correct format and using Drupal coding standards....
Comment #3
skeen commentedok, sorry, I swore i saved the file before uploading?? I guess not though, here is the correct patch
Comment #4
drummCommitted to HEAD.
Comment #5
(not verified) commented