Thanks for this great module. It was easy to create a custom module which did exactly what I needed with very little code.

The problem I came across:
My site was reporting it's inability to create the session_cache folder.
Session cache directory /../private/session_cache could not be created.

The problem is in session_cache_file.module Line 43 in function session_cache_file_directory()

There is a / at the beginning of $path which shouldn't be there. I think.
Remove the / at the beginning and the problem goes away.

This problem may only occur with a private folder path like mine = ../private

function session_cache_file_directory($bin = NULL) {
  $path = variable_get('file_private_path');

  // Typically $path == 'sites/default/files/private'.
  if (empty($path)) {
    drupal_set_message(t('Session Cache File: the <strong>Private file system path</strong> is not set. Please configure it <a href="@url">here</a>.',
      array('@url' => url('admin/config/media/file-system'))), 'warning', FALSE);
    return FALSE;
  }
  //
  $path = "/$path/session_cache"; // <<<<<<<<<<<<  $path = "$path/session_cache"; 

My apologies for not creating a diff. My computer died last week and I am still setting up my dev tools.

Regards
Geoff
behindthepage

Comments

behindthepage’s picture

Issue summary: View changes
rdeboer’s picture

Hi Geoff,
Thanks for your feedback and for your patch.
As it happens another code change was made previously in this function.

Can you try if the existing Session Cache File module, version 7.x-1.x-dev works for you?
Rik

studiotwelve’s picture

Change line 41 in latest release from

  $path = DRUPAL_ROOT . "/$path/session_cache";

to

  $path = "$path/session_cache";
sgdev’s picture

Version: 7.x-1.3 » 7.x-1.x-dev
Status: Active » Needs review

@studiotwelve, I believe you need to use the 7.x-1.x-dev version as @RdeBoer mentioned in comment #2.

After switching to -dev, I'm not seeing the error any longer. I'd suggest for others to review 7.x-1.x-dev to check if it fixes the problem for them, and post here if it does.

rdeboer’s picture

Thanks @ron_s
I've created a new official release to avoid confusion and because 7.x-1.x-dev has had few complaints.
Rik

rdeboer’s picture

Assigned: Unassigned » rdeboer
Status: Needs review » Fixed

Marking as fixed based on the above.
If problems re-surface, please re-open.
Rik

Status: Fixed » Closed (fixed)

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