The use of file_directory_path() is deprecated in Drupal 7.

This patch addresses the issues encountered when enabling Use CSS classes and an automatically managed external CSS style sheet.

It introduces an "internal" function which centralises the location of the CSS file (which I have also tweaked to put the CSS file in a folder within the files folder).

CommentFileSizeAuthor
geshifilter.file_directory_path.patch2.57 KBnicholasthompson

Comments

soxofaan’s picture

soxofaan commented

Great, thanks for your work!

Did you try this functionality with the private file system as default file system? At the moment I can't try myself.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

No I didn't - I'm also using a public FS... But it's working fine for me (as can be seen here: http://www.thingy-ma-jig.co.uk/blog/30-09-2010/mysql-how-upper-case-words)

soxofaan’s picture

soxofaan commented

marked #1056988: Fatal error: Call to undefined function file_directory_path() as duplicate

mgifford’s picture

mgifford
he/him
Primary language English
commented

Patch applies nicely and eliminates the error. Hopefully it gets into the release.

mgifford’s picture

mgifford
he/him
Primary language English
commented

Patch applies nicely and eliminates the error. Hopefully it gets into the release.

boombatower’s picture

boombatower commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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

Vc Developer’s picture

Vc Developer commented

I apologize for re-opening this issue, but I need help with another module File Aliases http://drupal.org/node/1217142#comment-4727512 having the same problem. Need to know what function to use to solve it now while the maintainer is making the changes.

Here is the code:


      'form' => array(
        'file_alias' => array(
          '#type' => 'textfield',
          '#title' => t('File alias'),
          '#default_value' => file_directory_path() . '/[file:ffp:onlyname:original].[file:ffp:extension:original]',
          '#maxlength' => 512,
          '#size' => 128,
        ),

Thanks!

Vc Developer’s picture

Vc Developer commented

Cancel that!

I think I found the solution here with ctools http://drupal.org/node/906652 patch and so far it working fine! :)