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

omega8cc’s picture

Status: Active » Needs review
chrisschaub’s picture

Is this going to be an official patch?

omega8cc’s picture

Status: Needs review » Patch (to be ported)

The 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.

omega8cc’s picture

steven jones’s picture

Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Formatting

  • Commit 6e4fdca on 6.x-2.x, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x authored by omega8cc, committed by Steven Jones:
    Issue #1421356 by omega8cc: Fixed Wrong variables for files paths...
  • Commit ce6c2e4 on 6.x-1.x, dev-drupal-8, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by Steven Jones:
    Issue #1421356 by omega8cc: Fixed Wrong variables for files paths...