Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

InsertDate ignores change of system timezone #33

Open
vovkkk opened this issue Jul 24, 2016 · 3 comments
Open

InsertDate ignores change of system timezone #33

vovkkk opened this issue Jul 24, 2016 · 3 comments
Labels
enhancement

Comments

@vovkkk
Copy link

vovkkk commented Jul 24, 2016

This happened unexpectedly: our local parliament decided to change time zone; they took decision just in July, and it became legit this night; I knew about it just hour ago.
Crazy stuff.

Steps to reproduce:

  1. Set in settings "tz_in": "Asia/Novosibirsk"
  2. Paste time
  3. Compare with Yandex.Time

Expected result
UTC+7, e.g. 14:50

Actual
UTC+6, 13:50
@vovkkk vovkkk changed the title Upstream pytz bug: Novosibirsk must be UTC+7 InsertDate ignores change of system timezone Jul 24, 2016
@vovkkk
Copy link
Author

vovkkk commented Jul 24, 2016

Actually this is not the point (original report).

The point is that if you change timezone of your OS, then you have to restart ST to make InsertDate show correct time.

@FichteFoll
Copy link
Owner

FichteFoll commented Jul 24, 2016 •
edited
Loading

What OS is this on?

I wonder if this is something I can actually fix.

@FichteFoll FichteFoll added the bug label Jul 24, 2016
@vovkkk
Copy link
Author

vovkkk commented Jul 24, 2016

Windows 8.1 Enterprise 32bit

vovkkk added a commit to vovkkk/InsertDate that referenced this issue Nov 26, 2016
@vovkkk
Fix FichteFoll#33 (GetLocalTime on NT instead of datetime.now)
7a8ced4 
the problem described in FichteFoll#33 is caused by datetime module itself;
thus for reliability and user convenience it can be monkey-patched
relatively simple
@vovkkk vovkkk mentioned this issue Nov 26, 2016
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@FichteFoll @vovkkk