Project

General

Profile

バグ #1358

抽出条件「本文」がMessage Modelのto_sを使用している

Added by Shibafu Midorino 16 days ago. Updated 1 day ago.

Status:
パッチ適用待ち
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
extract
ブランチ:
クラッシュする:
No

Description

抽出プラグインの標準抽出条件「本文」では、以下のS式が使用されています。

(,compare (to_s message) ,(car args))

Message Modelの #to_s を使用しているように見えるのですが、これは現在のDiva Modelの規約とは合っていないのではないでしょうか?
たとえば Plugin::Twitter::Message では、互換実装が発動して140文字にトリムされた本文との比較になってしまう可能性があるかと思います。

(description message) のほうが良いのではないかと思いますが、いかがでしょうか。


Files

0001-description.patch (1.32 KB) 0001-description.patch Shibafu Midorino, 2019-07-15 19:35

Related issues

Related to バグ #1109: ツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう終了Actions

History

#1

Updated by Izumi Tsutsui 16 days ago

  • Related to バグ #1109: ツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう added
#2

Updated by toshi_a 初音 16 days ago

  • Status changed from 分類待ち to 実装待ち
  • Target version set to 3.9

これは確かに

#3

Updated by Shibafu Midorino 1 day ago

言いだしっぺがやるべきだなーと思ったのでパッチを作りました。

Also available in: Atom PDF