Project

General

Profile

Actions

バグ #1523

closed

gtk3: mastodon の「user について」タブの toots/day の表示がおかしい

Added by Izumi Tsutsui 7 months ago. Updated 7 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
mastodon_account_viewer
ブランチ:
topic/1523-avoid-sprintf
クラッシュする:
No

Description

#1509 を見ていて気づいたのですが、
mastodon のユーザーについてタブの toots/day の表示が意図どおりでないようです。


Files

Actions #1

Updated by Izumi Tsutsui 7 months ago

708509590 のリファクタリングの意図をつかめてませんが

diff --git a/plugin/mastodon_account_viewer/mastodon_account_viewer.rb b/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
index 5d0b21e3..dab57b58 100644
--- a/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
+++ b/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
@@ -17,7 +17,7 @@ Plugin.create(:mastodon_account_viewer) do
     }
     param_toot_count = {
       count: user.statuses_count,
-      toots_per_day: since_day == 0 ? user.statuses_count : '%{avg}.2f' % { avg: Rational(user.statuses_count, since_day).to_f }
+      toots_per_day: since_day == 0 ? user.statuses_count : '%{avg}' % { avg: Rational(user.statuses_count, since_day).to_f.round(2) }
     }
     [
       [_('名前'), user.display_name],

もしくはリファクタリング前と同様に
diff --git a/plugin/mastodon_account_viewer/mastodon_account_viewer.rb b/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
index 5d0b21e3..cfe8d735 100644
--- a/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
+++ b/plugin/mastodon_account_viewer/mastodon_account_viewer.rb
@@ -17,7 +17,7 @@ Plugin.create(:mastodon_account_viewer) do
     }
     param_toot_count = {
       count: user.statuses_count,
-      toots_per_day: since_day == 0 ? user.statuses_count : '%{avg}.2f' % { avg: Rational(user.statuses_count, since_day).to_f }
+      toots_per_day: since_day == 0 ? user.statuses_count : "%.2f" % (Rational(user.statuses_count, since_day).to_f)
     }
     [
       [_('名前'), user.display_name],

でいいんですかね。 avg: の意図を把握できてませんが。

Actions #2

Updated by Shibafu Midorino 7 months ago

  • Status changed from 分類待ち to 実装待ち
  • Assignee set to Shibafu Midorino
Actions #3

Updated by Shibafu Midorino 7 months ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from Shibafu Midorino to Izumi Tsutsui
  • ブランチ set to topic/1523-avoid-sprintf

String#% において %<name>%{name} は違う意味を持つようですが、Rubocopでコードをフォーマットした時にうっかり後者に統一されてしまったのを見落としてただけなんじゃないかなと思います。(聞かないと分からんですが)

で、ここでは小数を表示用に丸めたいだけで sprintf 的なものを通すのは必須じゃないはずなので、 #note-1 のパッチからさらに書式文字列も捨てちゃう形で良い気がします。
そんな感じで修正を上げておきました。

Actions #4

Updated by Shibafu Midorino 7 months ago

リグレッションと判断したので、修正後の表示確認をお願いします。

(でも仕様確認だったらtoshi_aさん案件だったのかなあ)

Actions #5

Updated by Izumi Tsutsui 7 months ago

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

topic/1523-avoid-sprintf が pull できない謎にハマりつつ
適当に手動マージして動作はOKです。

としぁさんのこだわりがあるかどうかは、何かあればマージした後に適当に直してもらう、でいいような。
(見た目を直すのが先と思うので)

Actions #6

Updated by Shibafu Midorino 7 months ago

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

merged

Actions

Also available in: Atom PDF