Caching issue in get_calendar() function

[alexVauch]

Now the result of get_calendar() execution is cached. Including checking of user agent:

if (strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== false || stripos($_SERVER['HTTP_USER_AGENT'], 'camino') !== false || stripos($_SERVER['HTTP_USER_AGENT'], 'safari') !== false)

I suggest to cache only db queries.

[alexVauch]

Caching only db queries.

[johnbillion]

Yuck! The user agent is being detected here in order to insert line breaks into the title attributes, but only in browsers that support it. We should kill that code instead of altering the caching.

[alexVauch]

Ok. But what about:
​http://core.trac.wordpress.org/browser/tags/3.7.1/src/wp-includes/general-template.php#L1294
I think that cache will be cause of wrong "today" too.

[SergeyBiryukov]

Caching was introduced in [4522]. Browser detection was added in [521], altered in [752].

[peterwilsoncc]

Browser sniffing has since being removed from core.