drupal_realpath('public://') is the new file_directory_path(). See also http://api.drupal.org/api/drupal/includes--file.inc/function/file_direct...

    // '#description' => t('Subdirectory in the directory %dir where cache files will be stored.', array('%dir' => file_directory_path() . '/')),
    '#description' => t('Subdirectory in the directory %dir where cache files will be stored.', array('%dir' => drupal_realpath('public://') . '/')),
CommentFileSizeAuthor
#3 simplepie_d7_patch_2.zip1021 bytesagerson
#2 simplepie_d7_patch.zip1006 bytesagerson

Comments

agerson’s picture

agerson commented

http://drupal.org/update/modules/6/7#file_directory_path

agerson’s picture

agerson commented
Status: Active » Needs review
StatusFileSize
new simplepie_d7_patch.zip1006 bytes

EDIT: This patch does not work

agerson’s picture

agerson commented
StatusFileSize
new simplepie_d7_patch_2.zip1021 bytes

This one should. Please test.

zeahgan_white’s picture

zeahgan_white commented

the patch doesn't work... keeps saying that there aren't any .info files

zeahgan_white’s picture

zeahgan_white commented

the patch doesn't work... keeps saying that there aren't any .info files

digitalRoots’s picture

digitalRoots commented
  1. Change simplepie.admin.inc, line 24, from
    '#description' => t('Subdirectory in the directory %dir where cache files will be stored.', array('%dir' => file_directory_path() . '/')),
    to
    '#description' => t('Subdirectory in the directory %dir where cache files will be stored.', array('%dir' => variable_get('file_' . file_default_scheme() . '_path', conf_path() . '/'))),
  2. Change simplepie.module, line 89, from
    $simplepie->set_cache_location(file_directory_path() .'/'. $cache_location);
    to
    $simplepie->set_cache_location(variable_get('file_' . file_default_scheme() . '_path', conf_path() .'/'. $cache_location));
kenorb’s picture

kenorb commented
Issue summary: View changes
Related issues: +#2618516: Fatal error when restoring "Entire Site"