Project

General

Profile

Actions

バグ #1358

closed

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

Added by Shibafu Midorino about 5 years ago. Updated about 5 years ago.

Status:
終了
Priority:
通常
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リンクが多数ある場合に表示が切れてしまう終了Izumi TsutsuiActions
Actions #1

Updated by Izumi Tsutsui about 5 years ago

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

Updated by toshi_a 初音 about 5 years ago

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

これは確かに

Actions #3

Updated by Shibafu Midorino about 5 years ago

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

Actions #4

Updated by toshi_a 初音 about 5 years ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Shibafu Midorino
  • ブランチ set to topic/1358-extract-description

3.9.1にmergeしたので、一応動作確認お願いします

Actions #5

Updated by Shibafu Midorino about 5 years ago

自分の日常使用の範囲では、問題なく動作していることが確認できました。

マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...?

Actions #6

Updated by toshi_a 初音 about 5 years ago

マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...?

descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。

Actions #7

Updated by Shibafu Midorino about 5 years ago

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

toshi_a 初音 さんは書きました:

descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。

認識が合っていることが確認できて良かったです。ありがとうございます。
それでは、お手すきの際にマージをお願いします。

Actions #8

Updated by toshi_a 初音 about 5 years ago

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

merged

Actions

Also available in: Atom PDF