機能 #1205
closed返信時にGtk::PostBoxがIntelligentTextviewを付加するための要件を緩和してほしい
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
Related issues
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で良いんだけど、それは今回はどっちでも良いかな)
Updated by cob odo over 6 years ago
ただ、このパッチでも不十分だと思っていて、最低限icon(model.user.iconではなくmodel.icon)とdescriptionが取得できたら表示できるはずなので、そのようにしたいと思います。
それが理想的だと思います。
(本当はアイコンはoptionalで良いんだけど、それは今回はどっちでも良いかな)
とりあえずworldonというか、大抵のアイコンを持つカスタムModelには関係ないので、どっちでもいいかなーと。
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しておいてください。
Updated by cob odo over 6 years ago
- Status changed from レビュー待ち to 終了
ありがとうございます。問題なさそうでしたのでdevelopにmergeしました。
このチケットは終了にしておきます。
Updated by toshi_a 初音 over 6 years ago
- Related to バグ #1229: 特定のMessage Modelが表示される時クラッシュすることがある added