I found cache_get($cache) is wrong and did also other fixes in this function.

---
I wonder what's the intention for this function. It is only used to cache the result of autoadmin_schema_get_all() and $options argument is only used to clear the cache. If it is planned to support caching the result of other functions I suggest this signature:

function autoadmin_cache_function($function, $arguments = array(), $reset = FALSE)).. and also use call_user_func_array() instead.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

_rune’s picture

Hi

I no longer have plans to cache other functions with it and think it should be removed.

Cheers
Rune

_rune’s picture

Status: Needs review » Closed (fixed)

Removed dependency on the function completely in version 1.7. Thanks for the patch though!

_rune’s picture

Status: Closed (fixed) » Needs work

Reopening, I messed up the fix. Fix tomorrow.

_rune’s picture

Status: Needs work » Closed (fixed)

Finally fixed in v. 1.8. I _really_ hate this cache pattern and starts to remember why I made the blasted cache function in the first place :)