Checking on includes/session.inc drupal_session_initialize() its last task is to set the default timezone with date_default_timezone_set(drupal_get_user_timezone());. This is completly missing from mongodb_session.inc meaning without extra code elsewhere to handle this case the page will always fall back to whatever is set in php.ini as default timezone and the variable 'date_default_timezone' will have no effect.

CommentFileSizeAuthor
#5 missing_timezone_handling-1454078-5.patch868 bytesmisc
#1 mongodb-session-timezone-setting-1454078-1.patch411 bytesnagba

Comments

nagba’s picture

nagba commented
Assigned: Unassigned » nagba
Status: Active » Needs review
StatusFileSize
new mongodb-session-timezone-setting-1454078-1.patch411 bytes

One line patch adding 'date_default_timezone_set(drupal_get_user_timezone());' to drupal_session_initialize().

misc’s picture

misc commented
Status: Needs review » Reviewed & tested by the community

Seems good to me.

misc’s picture

misc commented
Status: Reviewed & tested by the community » Fixed

Added to the latest dev.

nagba’s picture

nagba commented

thanks

misc’s picture

misc commented
StatusFileSize
new missing_timezone_handling-1454078-5.patch868 bytes

I have some issues when user->timezone is not set, so I added a check for this.

misc’s picture

misc commented
Status: Fixed » Needs review
misc’s picture

misc commented
Status: Needs review » Fixed

Commited to the latest dev.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.