Project

General

Profile

Actions

機能 #1205

closed

返信時にGtk::PostBoxがIntelligentTextviewを付加するための要件を緩和してほしい

Added by cob odo over 6 years ago. Updated over 6 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
Start date:
2018-04-12
Due date:
% Done:

0%

プラグイン名:
topic/1205-intelligent-text-view-for-everything

Description

現在は

m.is_a?(Message)

で対象を絞っていますが、以下のようになっているとカスタムModelで対応しやすいです。
m.is_a?(Diva::Model::MessageMixin) && m.respond_to?(:user) && m.user.is_a?(Diva::Model::UserMixin)

試作したパッチを添付します。

worldonプラグインではこの対応のためにGtk::PostBox#generate_boxをモンキーパッチしているんですが、将来的な本体側の変更を考えると、できればモンキーパッチではなく、本体側の要件緩和で対応されてほしいです。


Files

postbox_replies.patch (772 Bytes) postbox_replies.patch cob odo, 2018-04-12 17:13

Related issues

Related to バグ #1229: 特定のMessage Modelが表示される時クラッシュすることがある終了Actions
Actions #1

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 新規 to 実装待ち
  • Target version changed from 3.6 to 3.7

これはダックタイピングを阻害していて良くないですね。
ただ、このパッチでも不十分だと思っていて、最低限icon(model.user.iconではなくmodel.icon)とdescriptionが取得できたら表示できるはずなので、そのようにしたいと思います。
(本当はアイコンはoptionalで良いんだけど、それは今回はどっちでも良いかな)

Actions #2

Updated by cob odo over 6 years ago

ただ、このパッチでも不十分だと思っていて、最低限icon(model.user.iconではなくmodel.icon)とdescriptionが取得できたら表示できるはずなので、そのようにしたいと思います。

それが理想的だと思います。

(本当はアイコンはoptionalで良いんだけど、それは今回はどっちでも良いかな)

とりあえずworldonというか、大抵のアイコンを持つカスタムModelには関係ないので、どっちでもいいかなーと。

Actions #3

Updated by toshi_a 初音 over 6 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to cob odo
  • プラグイン名 set to topic/1205-intelligent-text-view-for-everything

修正しました。これでほかのModelも表示されることが確認できました。

ただ、Worldonで検証した場合は、Message Modelのiconメソッドが定義されていないためアイコンを表示しません。全てのMessage ModelにUserに相当するものが存在するとも限らないのでMessage#iconを使っています。

これで問題なければ、developにmergeしておいてください。

Actions #4

Updated by cob odo over 6 years ago

  • Status changed from レビュー待ち to 終了

ありがとうございます。問題なさそうでしたのでdevelopにmergeしました。
このチケットは終了にしておきます。

Actions #5

Updated by toshi_a 初音 over 6 years ago

  • Related to バグ #1229: 特定のMessage Modelが表示される時クラッシュすることがある added
Actions

Also available in: Atom PDF