installer checks date.timezone for no reason

[Atropos]

Changeset 2752 includes a patch to the installer that checks to see if date.timezone was set. This was an attempt to fix a problem with timestamps in RC logs not matching timezone of timestamps in Apache logs. ('Different Timezone offsets in my RoundCube log') did not mention any other reasons the patch may be needed. It did not fix the problem according to the person who experienced that issue. This patch does create a problem with users who want to install RC on web hosts who have a default installation of PHP that does not specify a timezone using date.timezone. I suggest backing out the patch.

[alec]

Roland has been

[till]

Roland has been ... ?

Btw, you could use .htaccess to set date.timezone, or you tell your webhost that they have a mis-configured PHP.

[rosali]

Till, it looked like the setting of the timezone in php fixed different timezone offsets between RC and apache logs. But since a few days the weird issue started again.

[Atropos]

Replying to rosali:

Till, it looked like the setting of the timezone in php fixed different timezone offsets between RC and apache logs. But since a few days the weird issue started again.

Why not have the installer determine a "server timezone" offset from GMT and store that as a config variable, then patch write_log to apply this offset? That is how the user date functions work after all. Apparently rosali can't trust PHP to use the same timezone for the duration that Roundcube is executing, or perhaps in a mod_php/FastCGI environment PHP's timezone is changed.

[rosali]

First complaint.

​http://www.roundcubeforum.net/news-announcements/5077-roundcube-news-release-candidate-0-3-a.html

[till]

Again, they should set it in .htaccess - instead of editing the installer. Regardless, they could also email their hosting company to fix this.

php_value date.timezone Europe/Berlin

[ Or whatever is applicable. ]

Re: your issue

I don't know enough about your environment. I'm pretty sure it's the same timezone issue as before. You can ping me via email and we'll try to resolve this.

[alec]

Again. This shouldn't be required setting. We should move this to some "Recommended" section.

[alec]

Fixed in [11e670da].