プロジェクト

全般

プロフィール

バグ #1523

完了

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

Izumi Tsutsui さんが2年以上前に追加. 2年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
mastodon_account_viewer
クラッシュする:
いいえ

説明

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


ファイル

Izumi Tsutsui さんが2年以上前に更新

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: の意図を把握できてませんが。

Shibafu Midorino さんが2年以上前に更新

  • ステータス分類待ち から 実装待ち に変更
  • 担当者Shibafu Midorino にセット

Shibafu Midorino さんが2年以上前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者Shibafu Midorino から Izumi Tsutsui に変更
  • ブランチtopic/1523-avoid-sprintf にセット

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

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

Shibafu Midorino さんが2年以上前に更新

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

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

Izumi Tsutsui さんが2年以上前に更新

  • ステータスレビュー待ち から マージ待ち に変更

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

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

Shibafu Midorino さんが2年以上前に更新

  • ステータスマージ待ち から 終了 に変更

merged

他の形式にエクスポート: Atom PDF