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

Comments

stephenrobinson’s picture

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

stephenrobinson commented
nikemen’s picture

nikemen commented

Same problem here !

fajo’s picture

fajo commented

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

mghopkins commented

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

morbiD commented
Status: Active » Needs review
StatusFileSize
new private_css-787246-5.patch2.22 KB

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

morbiD’s picture

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

Patch was against dev.

aCCa’s picture

aCCa commented

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

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

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

aCCa commented
  • 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