操作
バグ #1109
完了
ツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう
プラグイン名:
message_detail_view
クラッシュする:
いいえ
説明
このツイート
https://twitter.com/PkgsrcChanges/status/939584643149516800
を検索もしくは https://github.com/osak/show_tweet 等のプラグインを使って表示すると
↓こんな感じで複数のリンクが貼られているのですが
これを右クリックメニューの「詳細」で表示すると
のように URL が途中で切れてしまうようです。
文字数制限もしくは extended_tweet の扱いに問題があるんでしょうか。
ファイル
関連するチケット
Shibafu Midorino さんが5年以上前に更新
- ファイル 0001-message_detail_view-description.patch 0001-message_detail_view-description.patch を追加
- ステータス を 新規 から パッチ適用待ち に変更
- プラグイン名 を message_detail_view にセット
- クラッシュする を いいえ にセット
該当の機能を提供するプラグインである message_detail_view の実装を見ると、本文の表示のために Model の #to_s を参照していました。
Plugin::Twitter::Message#to_s は (おそらく互換性維持のため？) 本文を、140文字で収まるよう末尾を捨てて返す作りになっています。
そのため、URLの途中で切り捨てられた本文が表示されていたと考えられます。
今日では、本文情報のようなデータは description フィールドで扱うという規約が存在するため、ひとまずはそちらを参照するのが妥当だと考えます。
mikutter 3.8.7向けのパッチを添付しました。
toshi_a 初音 さんが5年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を Izumi Tsutsui にセット
- 対象バージョン を 3.5 から 3.8 に変更
- ブランチ を topic/1109-message-detail-view-description にセット
pushしました。確認してみてください。
Izumi Tsutsui さんが5年以上前に更新
- ステータス を レビュー待ち から マージ待ち に変更
master だとテストがややしんどいので develop に同様のパッチを当ててテストしました。
問題ありません。
操作