Actions
バグ #1109
closedツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう
プラグイン名:
message_detail_view
クラッシュする:
No
Description
このツイート
https://twitter.com/PkgsrcChanges/status/939584643149516800
を検索もしくは https://github.com/osak/show_tweet 等のプラグインを使って表示すると
↓こんな感じで複数のリンクが貼られているのですが
これを右クリックメニューの「詳細」で表示すると
のように URL が途中で切れてしまうようです。
文字数制限もしくは extended_tweet の扱いに問題があるんでしょうか。
Files
Related issues
Updated by Shibafu Midorino almost 6 years ago
- File 0001-message_detail_view-description.patch 0001-message_detail_view-description.patch added
- Status changed from 新規 to パッチ適用待ち
- プラグイン名 set to message_detail_view
- クラッシュする set to No
該当の機能を提供するプラグインである message_detail_view の実装を見ると、本文の表示のために Model の #to_s
を参照していました。
Plugin::Twitter::Message#to_s
は (おそらく互換性維持のため?) 本文を、140文字で収まるよう末尾を捨てて返す作りになっています。
そのため、URLの途中で切り捨てられた本文が表示されていたと考えられます。
今日では、本文情報のようなデータは description
フィールドで扱うという規約が存在するため、ひとまずはそちらを参照するのが妥当だと考えます。
mikutter 3.8.7向けのパッチを添付しました。
Updated by toshi_a 初音 over 5 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee set to Izumi Tsutsui
- Target version changed from 3.5 to 3.8
- ブランチ set to topic/1109-message-detail-view-description
pushしました。確認してみてください。
Updated by Izumi Tsutsui over 5 years ago
- Status changed from レビュー待ち to マージ待ち
master だとテストがややしんどいので develop に同様のパッチを当ててテストしました。
問題ありません。
Updated by Izumi Tsutsui over 5 years ago
- Related to バグ #1358: 抽出条件「本文」がMessage Modelのto_sを使用している added
Actions