The reason this happens just for them is that the timezone is set based on the user configuration during session authentication.

That should probably be pulled out of there, which would affect the cases that still include it manually.

CommentFileSizeAuthor
#2 drupal-2054203-2.patch715 bytesdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

Note: There are other tests too that fail, but this IMHO indicates an actual bug: That user timezone settings aren't respected when a different authentication is used.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Issue tags: +Needs tests
FileSize
drupal-2054203-2.patch715 bytes
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review

I guess we want to avoid setting it twice :)

Still think whatever we have right now with session.inc is a crazy mess :)

alansaviolobo queued 2: drupal-2054203-2.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: drupal-2054203-2.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Issue summary: View changes

Asked @znerol what to do here ... he is more into authentication/session stuff ...

znerol’s picture

znerol CreditAttribution: znerol commented

I think this should be in AccountProxy::setAccount() because this also covers the account switcher service. Also the now suprious calls should be removed from SessionManager.

almaudoh’s picture

almaudoh CreditAttribution: almaudoh commented
Status: Needs work » Closed (duplicate)

This seems to be covered now in #2328645: Remove remaining global $user which is RTBC.