Actions
バグ #1358
closed抽出条件「本文」がMessage Modelのto_sを使用している
プラグイン名:
extract
クラッシュする:
No
Description
抽出プラグインの標準抽出条件「本文」では、以下のS式が使用されています。
(,compare (to_s message) ,(car args))
Message Modelの #to_s
を使用しているように見えるのですが、これは現在のDiva Modelの規約とは合っていないのではないでしょうか?
たとえば Plugin::Twitter::Message
では、互換実装が発動して140文字にトリムされた本文との比較になってしまう可能性があるかと思います。
(description message)
のほうが良いのではないかと思いますが、いかがでしょうか。
Files
Related issues
Updated by Izumi Tsutsui about 5 years ago
- Related to バグ #1109: ツイートの詳細タブでURLリンクが多数ある場合に表示が切れてしまう added
Updated by toshi_a 初音 about 5 years ago
- Status changed from 分類待ち to 実装待ち
- Target version set to 3.9
これは確かに
Updated by Shibafu Midorino about 5 years ago
- File 0001-description.patch 0001-description.patch added
- Status changed from 実装待ち to パッチ適用待ち
言いだしっぺがやるべきだなーと思ったのでパッチを作りました。
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したので、一応動作確認お願いします
Updated by Shibafu Midorino about 5 years ago
自分の日常使用の範囲では、問題なく動作していることが確認できました。
マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...?
Updated by toshi_a 初音 about 5 years ago
マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...?
descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。
Updated by Shibafu Midorino about 5 years ago
- Status changed from レビュー待ち to マージ待ち
toshi_a 初音 さんは書きました:
descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。
認識が合っていることが確認できて良かったです。ありがとうございます。
それでは、お手すきの際にマージをお願いします。
Actions