So I don’t know why the return $cached_result; in the else, but if that’s intentional, then you should probably do: return json_decode( $cached_result, true ); Because you’re caching the response body (which is a JSON string) and you’re reading the non-cached result like so: $array = json_decode($result, true);. But if that return was a mistake, … Read more
$ExpiryInterval = “24 * HOUR_IN_SECONDS”; $ExpiryInterval is being assigned a string, but you need a number. Consider this example: $foo = 5 * 10; $bar = “5 * 10”; The value of $foo is 50. The value of $bar is “5 * 10”. set_transient expects a number, not a string, “24 * HOUR_IN_SECONDS” is text/string, … Read more
get_site_transient() uses the older nomenclature for multisite which referred to a multisite network as a “site” and individual sites on the network as “blogs”. So get_site_transient() is getting the value of a transient for the whole network, while get_transient() gets a transient for an individual site/blog. If you look at the source of the function … Read more
I would do the following: Create WP-Cron task (or just use daily wp_scheduled_delete one to tag along) and hook your function to it. In that function: fetch XML file; fetch all posts with meta_weight set, using get_posts(); loop through posts and save comparison result in another meta field for each.
You have quite a bit of markup for that shortcode. Trying to pass all of that as a shortcode is going to be trouble. I pulled it apart to look at it. id=”map” z=”11″ w=”100%” h=”300″ scrollwheel=”false” maptype=”ROADMAP” address=”Southampton, United Kingdom” marker=”true” markerimage=”http://anachronistic.local:9102/wp-content/plugins/_TomRush/inc/js/TinyMCE_additions/modal/../img/icons/music_folk_map.png“ infowindow=”<div class=”marker inside”><img class=”marker icon” alt=”” src=”http://maps.gstatic.com/mapfiles/place_api/icons/geocode-71.png” /><strong>Southampton</strong><br>Southampton, UK<br><span class=”marker extras”>Some User … Read more
The rule of thumb is not to do any caching, except for object caching, for logged in users. Actually IIRC object caching will give you exactly what you want. But if you have to do it your way, you should make your cache keys to be based on the query and have different cache for … Read more
The technical side of it depends on the storage used. For default database storage the transient entries can be queried and deleted, since they have specific naming format. For enabled Object Cache storage cache can be flushed, which will get got cache and transient. The practical side of it — this is Bad Idea. Transients … Read more
Unless you do something weird with your settings, transients will just make your code harder to read without adding any measurable performance value. People that want performance should use object caching and page caching and your current code is probably already the best there is to utilize them. Transients will add almost nothing for them. … Read more
To my understanding, *_transient() is basically *_option() wrapped in a cache-timer. What that means in this context, is that set_transient( ‘…’, true, * MINUTE_IN_SECONDS ); will not run/re-evaluate every minute. Instead it will run/re-evaluate when the page has loaded by a visitor/user, and the data therein is older than 60 seconds. This is not ideal … Read more
There is no way to do that. You need to design a push solution instead of a pull one, and if you can’t and must have to use pull…. well than it sucks to be you :(. Web servers and web software is not designed to be used for “heavy lifting” in such frequent interval. … Read more