Missing CEST time zone in SimplePie

[Krstarica]

Central European Summer Time (CEST) is missing in SimplePie (wp-includes/SimplePie/Parse/Date.php).

This time zone is used by 30 European countries right now:
​https://en.wikipedia.org/?title=Central_European_Summer_Time

The fix is available:
​https://github.com/simplepie/simplepie/pull/380

It seems that the new version of SimplePie will not be released anytime soon (active v1.3.1 is released on 30 October 2012), so it would be nice to patch this file directly. Many thanks.

[swissspidy]

This is a simple patch that just adds the CEST timezone.

[wonderboymusic]

You need to make a PR here:
​https://github.com/simplepie/simplepie/blob/master/library/SimplePie/Parse/Date.php

it's a 3rd party library - once the fix is made upstream, we can fold in the changes

[dd32]

As above, there's a PR open.

When an issue in an external library affects core, we also often keep a tracking ticket open. For this, if it affects users majorly, we can merge it even before the next simplepie is released if need be.

[swissspidy]

For this, if it affects users majorly, we can merge it even before the next simplepie is released if need be.

Not sure how many have even noticed this bug. Either we fix this now or close it in favor of waiting for a SimplePie release.

It would be good to get a quick feedback from @rmccue on this.

[rmccue]

I'll merge upstream right now. I don't dedicate any time to SimplePie, so things like this slip under my radar.

[swissspidy]

If it affects users majorly, we can merge it even before the next simplepie is released if need be.

There won't be a new SimplePie release anytime soon and this affects enough sites/users to be considered for a merge.

@wonderboymusic Can you have another look and commit if it makes sense?

[wonderboymusic]

In 35283:

Feeds: add CEST to $timezone in SimplePie_Parse_Date.

Props swissspidy.
Fixes #32749.