Hi Uncle Cheese,

For some reason the event_calendar module starting working really slow for a site on a site with not many events but several calendars. But I found that by changing;

$Y = date('Y', $ts); // year

to

$Y = date('y', $ts); // year

in sfDateTimeToolkit::breakdown() made alot of difference to performance. I could run some runtime results to prove this? But in this instance it was noticeably different. Can you see any potential problem in making this change?

I also noticed (a minor thing) that the switch should be using

default:
 Debug::message(sprintf('The unit of time provided is not valid: %s', $unit));

as opposed to outputting a symphony error message.

Happy coding,

I can't imagine that there's a causal link between changing "Y" to "y" and increased performance, but it's good to know. First I've heard of any performance issues on EC. If you find any resources that suggest performance issues with 4-digit years, send them over. If there's any substance to it, it would be one of the strangest PHP nuances I've ever encountered.