Closed (fixed)
Project:
Provision
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jan 2012 at 14:17 UTC
Updated:
12 Jun 2014 at 08:41 UTC
Jump to comment: Most recent
Aegir forces only D6-like variables even for D7 sites:
$conf['file_directory_path'] = 'sites/domain/files';
$conf['file_directory_temp'] = 'sites/domain/files/tmp';
It should force:
$conf['file_public_path'] = 'sites/domain/files';
$conf['file_temporary_path'] = 'sites/domain/files/tmp';
While even without correct value forced for file_public_path variable, it still works correctly with file_directory_path forced instead, the file_temporary_path will use Drupal default /tmp path, which is not what we want in the multisite environment.
Comments
Comment #1
omega8cc commentedThis should do the trick: http://drupalcode.org/sandbox/omega8cc/1074910.git/commit/3fd497a
Comment #2
chrisschaub commentedIs this going to be an official patch?
Comment #3
omega8cc commentedThe fix works just fine and is now a part of Provision 2.x used in BOA.
It should be committed to this official Provision 2.x branch, but I guess we need to port it also to 1.x.
Comment #4
omega8cc commentedHere is a better patch for 2.x: http://drupalcode.org/sandbox/omega8cc/1074910.git/commit/5748c0a
Comment #5
steven jones commentedThanks committed in: http://drupalcode.org/project/provision.git/commit/6e4fdca
Cleaned up in: http://drupalcode.org/project/provision.git/commit/286e29e
And ported to 6.x-1.x in: http://drupalcode.org/project/provision.git/commit/ce6c2e4
Hopefully that won't cause too many problems.
Comment #6.0
(not verified) commentedFormatting