Trying to follow and implement your caching functions, but

One question how about $content how is this one set??

PHPit - Totally PHP &r…

For regular non user/secure pages:
Change the cached pages to .php not .html.
Change the data for the cache file to $data = ‘<?php echo" ‘.addslashes($content).’ ";?>’;

When getting the cached pages use mktime (as previously suggested) and simply include the file ie:
// Cache too old
if (mktime()-(cache_time*60*60) > filemtime($file)){
return false;
}
include($file);
die();

NOTE: I haven’t done this as my site is trivial but I believe it should work
For personalized pages I suggest a shorter cache time so define a second variable secure_cache_time. At the top of any pages that are secure create a flag to identify as such define(’secure’,'1′) to the “cache engine”.
For the filename use a combination of the url and a unique user identifier (username, userid, ip) or some combination.
For the data $data = ‘<?php if(!correct_user) die; else echo" ‘.addslashes($content).’ ";?>’;
IMPORTANT: where correct_user is the logic you would regularly check for user credentials. Finally for added security setup an htaccess file on the directory or put the user cached pages outside the root directory all together.

Hacking in a method to define a per page cache_time would be a good idea since we generally know how often certain pages update and I also added a refresh_cache function. At the moment it can’t determine which files it needs updating so it just removes all of them (more of a refresh_all function).

Does it really matter if you use time() instead of mktime()? Both return the same thing anyway.

I like your idea of using filemtime instead of using my method. It’s probably a bit faster, and easier.

I understand that chmod’ing to 777 is a bad thing, but at least it works in all cases. Better of course would be something like 776 or 664.

Thanks for your comments!

- NEVER chmod to 777!!!

- use time() instead of mktime() to get the current timestamp
- the logic in get_url() seems to be reversed: if (!isset($_SERVER[’REQUEST_URI’])) $url = $_SERVER[’REQUEST_URI’];
- why load the entire cache file into memory just to check the date? I would use filemtime instead…

;)

this is really good but I am having problem accesing to the next page.
Thank you.

Regards,

Ramy Tran.