Hardcoded directory name in remote rss

Creating and modifying plugins.
Post Reply
Ralf

Hardcoded directory name in remote rss

Post by Ralf »

Hi,

Today I switched from the 404 error document archive handling to a PATH_INFO solution. That required removing or renaming the folder "archives" in order to create a file named "archives". No problems with that so far.

But I realized that the plugin "serendipity_plugin_remoterss.php" uses the directory "archives" as cache dir. Maybe it would be better to use another name (like "cache") for that folder. That should prevent conflicts if s9y comes with an own PATH_INFO URI feature in the future.

Ralf
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Hardcoded directory name in remote rss

Post by garvinhicking »

Upon Serendipity installation, it is made sure that the 'archives' folder exists and is world-writable, so apart from the 'upload' directory that is the only directory after installation where we are sure that we can write there. Creating a new directory can easily fail if the parent directory is not writable to the webserver user...

In 0.8 we could use the new 'templates_c' directory, though...but that does not really sound like a location for 'cache'.

BTW, the blogpdf uses the same directory to write its files to.

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Post Reply