With private downloads enabled, the css path ends up as mysite.com//var/www/drupal/files/ not mysite.com/system/files, is there a way if configuring this not to happen, or shall I just use my own patch?

CommentFileSizeAuthor
#6 private_css-787246-5.patch2.22 KBmorbiD
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StephenRobinson’s picture

Title: private download folder path is used instad of system/files » private download folder path is used instead of system/files
StephenRobinson’s picture

nikemen’s picture

Same problem here !

fajo’s picture

This is a bug in Drupals 6.x core function drupal_get_css(). I do not think it will be possible to work around this in the module code.

/Falk

mghopkins’s picture

Is it possible that the solution applied in js_injector to handle this issue be used in CSS Injector, or are the methods for getting the file fundamentally different?

The 1.3 version of js_injector module that overcomes the private file setting: http://drupalcode.org/viewvc/drupal/contributions/modules/js_injector/js...

Mike

morbiD’s picture

Status: Active » Needs review
FileSize
2.22 KB

How's this? Copied implementations from CTools and js_injector.

morbiD’s picture

Version: 6.x-1.4 » 6.x-1.x-dev

Patch was against dev.

aCCa’s picture

Patch in #6 is working on my side. Suggest to commit in devel. Thanks.

rfay’s picture

If somebody has reviewed and tested this and can RTBC it, it will get committed sooner or later. But hasn't had a review in more than a year.

aCCa’s picture

  • Patch solves the problem in site with private file download method
  • Not sure it fully comply with coding standards
  • Not sure all tests are necessary and efficients