logic errors in clean_term_cache()

[Denis-de-Bernardy]

clean_term_cache() has a completely random behaviour.

the attached patch ensures that it behaves consistently.

[Denis-de-Bernardy]

see also #11531

[westi]

Have you got some example plugin code we can use to test this and reproduce the issue so that more people can test the fix?

[dd32]

Can you also explain which part is the Complelely random section? I'm guessing its the Action args?

Also, What was the point of this over the original, It doesnt add anything extra..

$taxonomies = array($taxonomy); 
foreach ( $taxonomies as $taxonomy ) { 
 	foreach ( $ids as $id ) { 
 	   wp_cache_delete($id, $taxonomy); 
 	} 
} 

[Denis-de-Bernardy]

@westi: no working example, I merely ran into it while looking into improving a flush() function for memcached. please look at the code as it currently is.

the clean_terms_cache() accepts term_ids or term_taxonomy_ids as the first argument, and a taxonomy or an array of taxonomies as the second argument. use cases:

1. one taxonomy provided, array of t_ids given. it does exactly what it should, i.e. deletes the cache properly, and ends up calling:

do_action('clean_term_cache', $t_ids, $taxonomy);

2. no taxonomy provided, array of tt_ids given. it deletes the cache properly, but ends up calling the clean_term_cache action with tt_ids:

do_action('clean_term_cache', $tt_ids, $last_taxonomy_looped);

3. multiple taxonomies provided, array of t_ids given. then this deletes nothing (it's an invalid bucket):

wp_cache_delete($id, $taxonomy);

and the concluding do_action amounts to:

do_action('clean_term_cache', $t_ids, $last_taxonomy_looped);

[ryan]

Patch looks good to me.

[ryan]

(In [12510]) Cleanup clean_term_cache(). Props Denis-de-Bernardy. see #11530

[westi]

Setting the milestone to 3.0 for now as that is where the changes so far have been committed.

[hakre]

In my eyes it looks like that the patch was comitted but the ticket is still open. How to continue? Close the ticket? Keep it open?