Take the 2-minute tour ×
WordPress Development Stack Exchange is a question and answer site for WordPress developers and administrators. It's 100% free, no registration required.

Wanted some opinions on a solution I'm trying to come up with:

I have a fan/enthusiast plugin I'm working on for fans of TV shows. One of the custom post types attached to the plugin is a release type which stores data on dvd releases of a show. Some of the meta data that I weant stored is info from Amazon using their Product Advertising API. I wanted to cache some of the pricing data available there, such as the current price of the dvd.

I was thinking of using transients but from what I can tell their nothing more than a layer on WP options; their too global. I'd have to create numerous transients with a naming scheme that would point to a certain post.

The other option would be to use custom fields but use the CRON API to update it, I'd either have one Cron job update all the custom fields hourly or create one cron job per post that did so.

My question is which solution would be more practical performance and convenience wise?

share|improve this question

3 Answers 3

up vote 3 down vote accepted

I think I would write a wrapper function around custom fields, something like this:

function get_post_transient( $post_ID, $meta_key, $update_function ) {
    $current_value = get_post_meta( $post_ID, $meta_key, true );
    if ( is_array( $current_value ) && $current_value['expiration'] < date('U') )
        return $current_value['data'];

    $new_value = call_user_function( $update_function, $post_ID );
    update_post_meta( $post_ID, $meta_key, $new_value );

    return $new_value['data'];  

}

and save the meta as an array, of the form:

array(
    'expiration' => //timestamp that this field exires
    'data' => // all the custom data you want to save.
);

You would just need to have your update function return the same array structure. I don't know if there's any real benefit of this approach over using the options table through transients, but it would make more sense to me to have post meta stored in the postmeta table.

share|improve this answer
    
I think this or at least a variation of this is my best bet. –  Manny Fleurmond Dec 2 '11 at 5:19

When implementing caching with an API I've used the transients route. Mainly because they're set up for it and easy to use. There aren't a huge amount of problems in terms of table size or queries. MySQL is made for big datasets.

My approach was to take a hash of the url and/or arguments I was passing to the API and use that as my transient identifier. The function that does the communication with the API first checks for the transient and if it doesn't exist or has expired carries on with the regular API call. The advantage here over what you're suggesting with post meta is that it's a single table and key we're checking so it's quick, there's no looping or complex code needed and the API is only called when the transient has expired and when the data is requested which should reduce the number of API calls you make.

Works well for both my flickr and themoviedb.org plugins.

share|improve this answer

If it's connected to a specific post, it really should be in postmeta. I've violated this principle many times myself for speed and ease of use reasons, but I felt dirty each time. :)

The best way I think would be to store both the data and an expiration time in postmeta instead. Instead of checking all the types every hour, check them only when viewed. Major benefit: You can do this lazily!

Example:

On post view, the meta is pulled. If it's expired data, then the expired data is shown, but a setting is left somewhere in a global variable (storing just the post id, probably) telling another part of the code to refresh that data at the end of the page display.

Then you use the "shutdown" action hook. Here, you'll check that global variable, and if some posts' data needs refreshing, you go pull the data from amazon and update the postmeta, and bump the expiration by some amount of time.

This is the best of all worlds, really. The post is still displayed fast and your viewer doesn't have to wait for the new data to come from amazon. The amazon data gets refreshed after the page is already finished. And you're only refreshing data people are looking at, and only when they're looking at them, minimizing processing.

share|improve this answer
    
Though it waits till after the page is loaded to update expired data, I think I'd rather give updated data later rather than giving expired data now. Still a nice way of caching. I'll file it for a future project. Thanks, @Otto –  Manny Fleurmond Dec 2 '11 at 5:18
    
Your call. I'd prefer to display data fast than be totally 100% accurate. –  Otto Dec 2 '11 at 6:30

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.