Project

General

Profile

バグ #1150

Activityタブに表示されるTwitter dmの時刻がUTCになっている

Added by Izumi Tsutsui over 1 year ago. Updated about 1 year ago.

Status:
終了
Priority:
低め
Target version:
プラグイン名:
ブランチ:
クラッシュする:
No

Description

題名のとおりですが

Activityタブの右下に各アクティビティ(ツイート・ふぁぼ・システムメッセージ等)の
通知の時刻(?)が表示されるようになっていますが、
Twitter の dm の時刻だけは UTC で表示されているようです。

下記の例だと、2番めの dm の受信時刻について
Activity 欄での表示は 01:14:01 になっていますが
実際の受信時刻は JST 10:14:01 と思われます。

大した話ではないのですが、ふと Activity 欄を見ていて
「こんな時間に dm 来てたの?」→ 9時間ズレてた
などと思ってしまったので。


Files

activity-1.png (15.2 KB) activity-1.png Izumi Tsutsui, 2018-01-06 19:02
activity-2.png (21.6 KB) activity-2.png Izumi Tsutsui, 2018-01-06 19:02
activity-3.png (16.7 KB) activity-3.png Izumi Tsutsui, 2018-01-06 19:02

Associated revisions

Revision f1786297 (diff)
Added by Izumi Tsutsui over 1 year ago

Activityタブに表示されるTwitter dmの時刻がUTCになっている refs #1150

History

#1

Updated by toshi_a 初音 over 1 year ago

ここだけタイムゾーンを無視して表示してしまってそうですね。タイムラインでは強制的にローカルタイムゾーンに変換してから表示していたと思うので、これも同じようにすれば治りそうです

#2

Updated by Izumi Tsutsui over 1 year ago

activity.rb ではなぜか
source:core/plugin/activity/activity.rb@3ac4ed84#L299

      date = Time.parse(dm[:created_at])

で時刻を取っていますが、 retweet と同じセンス(?)で Twitter プラグイン側
source:core/plugin/twitter/mikutwitter/api_call_support.rb

        cnv[:created] = Time.parse(dm[:created_at]).localtime

と設定されている(?)らしい dm[:created] を参照するようにすると
Activity タブの時刻表示も localtime で出るようになりました。
--- core/plugin/activity/activity.rb.orig    2017-12-24 15:00:56.000000000 +0000
+++ core/plugin/activity/activity.rb
@@ -296,7 +296,7 @@ Plugin.create(:activity) do

   on_direct_messages do |service, dms|
     dms.each{ |dm|
-      date = Time.parse(dm[:created_at])
+      date = dm[:created]
       if date > BOOT_TIME
         activity(:dm, dm[:text],
                  description:

BOOT_TIME との比較において localtime にすべきなのかそうでないのか
というのがわかっていませんが。

#3

Updated by toshi_a 初音 over 1 year ago

  • Status changed from 新規 to パッチ適用待ち

これはcreatedを利用すべきですね。

#4

Updated by toshi_a 初音 over 1 year ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui

ブランチ topic/1150-invalid-timezone-twitter-dm-in-activity にpushしました。確認お願いします。

#5

Updated by Izumi Tsutsui over 1 year ago

  • Status changed from レビュー待ち to マージ待ち

f1786297 で JST で表示されることを確認しました。

「自分宛に dm を送る」という行為は公式でもできるのだろうか……

#6

Updated by Izumi Tsutsui over 1 year ago

  • Assignee changed from Izumi Tsutsui to toshi_a 初音
#7

Updated by toshi_a 初音 about 1 year ago

  • Status changed from マージ待ち to 終了

masterにmergeしました

Also available in: Atom PDF