This is required to merge with Timeblock module.
See #679948: Duplicate modules.

I thought it would be easy, but it conflicts a bit with our own JS update function.
I started work on this, but couldn't get it to work just yet.

CommentFileSizeAuthor
#4 local_time.patch1.25 KBtstoeckler
#3 local_time_d7.patch9.15 KBtstoeckler
#2 local_time_d7.patch27 byteststoeckler
#1 local_time.patch15.04 KBtstoeckler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Active » Needs review
FileSize
local_time.patch15.04 KB

Stupid mistake...
Patch contains one really small fix with a variable default, that I think wasn't being used anyway.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
FileSize
local_time_d7.patch27 bytes

That was the D6 version. It contained a silly bug (which would have been caught by SimpleTest :( ) where custom time zones aren't saved.
Will commit the D7 version first and then back to the bugfix.
This is the D7 patch

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
FileSize
local_time_d7.patch9.15 KB

Ehhh...

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Needs review » Fixed
FileSize
local_time.patch1.25 KB

Here the follow-up D6 patch.
And the commit-links:
D6: http://drupal.org/cvs?commit=344372
D6-follow-up: http://drupal.org/cvs?commit=344386
D7: http://drupal.org/cvs?commit=344382

Status: Fixed » Closed (fixed)

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

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented

This was actually broken:
http://drupal.org/cvs?commit=351722