Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Installing latest 6.x dev, the Local time zone doesn't work.
This is critical not only because it is major functionality, but also because it is a release blocker and a new release is very much needed due to #758294: Time changes to incorrect time (now fixed).
I hope to get at this in the next few days.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | fix-local-time-with-tests_2.patch | 27.32 KB | tstoeckler |
| #6 | fix-local-time-with-tests.patch | 26.58 KB | tstoeckler |
| #4 | local-time-and-tests.patch | 6.16 KB | tstoeckler |
| #3 | local_time_working.patch | 3.87 KB | tstoeckler |
| #2 | local_time.patch | 2.64 KB | tstoeckler |
Comments
Comment #1
tstoecklerThe problem is that when JS is turned off, 'Local' is passed to date_format_date() directly, which it cannot handle. This is a regression introduced by #742028: Sync 6.x and 7.x / Code clean-up
With JS on, it must also be something related to the former issue. I am not sure yet, though, what it is. Will roll a patch for the first one shortly.
Comment #2
tstoecklerThis is preliminary patch. It fixes the local time issue.
Changing the date format, doesn't work for me. I don't know if it is related to this issue, though.
Comment #3
tstoecklerWhat a dumb error. I don't know how that snuck in there.
Anyway, with the code-clean-up I can only hope this won't happen again.
Everything is working for me now.
I really feel anxious to release a new version now, but I should probably wait.
Comment #4
tstoecklerI had committed the above patch before (http://drupal.org/cvs?commit=368094).
It wasn't 100% functional though. New patch attached which fixes everything and also cleans up the tests, so that they actually work. Therefore we only have to test the JavaScript related functionality, which I did and it works. I will wait before committing this, to see how the Testbot likes this.
I will release 1.2 after this commit though.
Comment #5
tstoecklerhttp://drupal.org/cvs?commit=371012
Comment #6
tstoecklerFor D7.
Comment #7
tstoecklerSlight failure in display tests. Committed this. Maybe this fixes the Simpletest-ing of the D7 branch.
http://drupal.org/cvs?commit=372340