操作
バグ #1358
完了
抽出条件「本文」がMessage Modelのto_sを使用している
プラグイン名:
extract
クラッシュする:
いいえ
説明
抽出プラグインの標準抽出条件「本文」では、以下のS式が使用されています。
(,compare (to_s message) ,(car args))
Message Modelの #to_s を使用しているように見えるのですが、これは現在のDiva Modelの規約とは合っていないのではないでしょうか？
たとえば Plugin::Twitter::Message では、互換実装が発動して140文字にトリムされた本文との比較になってしまう可能性があるかと思います。
(description message) のほうが良いのではないかと思いますが、いかがでしょうか。
ファイル
関連するチケット
Shibafu Midorino さんが5年以上前に更新
- ファイル 0001-description.patch 0001-description.patch を追加
- ステータス を 実装待ち から パッチ適用待ち に変更
言いだしっぺがやるべきだなーと思ったのでパッチを作りました。
toshi_a 初音 さんが5年以上前に更新
- ステータス を パッチ適用待ち から レビュー待ち に変更
- 担当者 を Shibafu Midorino にセット
- ブランチ を topic/1358-extract-description にセット
3.9.1にmergeしたので、一応動作確認お願いします
Shibafu Midorino さんが5年以上前に更新
自分の日常使用の範囲では、問題なく動作していることが確認できました。
マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...？
toshi_a 初音 さんが5年以上前に更新
マージ待ちにする前の心残りなのですが、抽出プラグインが扱うModelでdescriptionを実装していないことって、おそらく無いですよね...？
descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。
Shibafu Midorino さんが5年以上前に更新
- ステータス を レビュー待ち から マージ待ち に変更
toshi_a 初音 さんは書きました:
descriptionがなければMessage Modelの要件を満たしておらずScoreが動かないので、必ず実装されています。
認識が合っていることが確認できて良かったです。ありがとうございます。
それでは、お手すきの際にマージをお願いします。
操作