Provide way to reset static cache in ctools_export_load_object()

Definitely useful, but it needs to be more granular. Resetting the entire cache, including tables that you're not concerned with, is probably wasteful, and potentially even damaging to other modules using the object cache. Would you mind rerolling with a more complicated $reset parameter that allows either a) a full reset of the cache or b) selective resets? Off the top of my head, it seems like using a simple array with the names of the caches to be reset ought to be sufficient.

It's much more fine-grained now.

Still not quite what I'm looking for. Really doing this properly may well entail a separate function where the static caches are contained that has more get/set style parameters. That way we can separate the granular cache clearing from the loader itself.

Meh, I may just write this myself real quick.

/**
 * Handles the static cache for the exports.
 *
 * @param $op
 *   'set' or 'get' - Together with $table.
 *   'set': Marked this table fully cached
 *   'get': Returns TRUE if the table is fully cached
 * @param $table
 *   Name of the database table to set a table ready or to adjust the scope of cache retting.
 * @return
 *   The reference of the current cache data.
 */
function &ctools_export_static_cache($op = NULL, $table = NULL, $reset = FALSE) {

Is this function signature something that you'd like to see?
In ctools_export_load_object(), i'd eliminate the two static variables and encapsulates the functionality in one function.

If you like this direction, i can post a patch, otherwise, if you give me more detailed instructions, i happily implement that direction as well.

/me smiles

Yup, that's exactly the kind of thing I had in mind. I just ran out of time. Roll me a patch with that sort of function signature, and I'll put it right in.

Here is the patch.

I'll have a look at this and respond in detail tomorrow. Thanks!

Is this modelled after the static system that's in D7? (I ask because I'm not familiar enough with it to just look at the code and see).

I'm happy. Almost completely happy.

Only problem is that we're still just globally clearing the export cache. Give me some changes to ctools_export_load_object_reset(), or a separate function for granular cache resets, and I'll commit.

I like this! Committed to both branches.