プロジェクト

全般

プロフィール

機能 #1205

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

cob odo4ヶ月前に追加. 4ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2018-04-12
期日:
進捗率:

0%

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

説明

現在は

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をモンキーパッチしているんですが、将来的な本体側の変更を考えると、できればモンキーパッチではなく、本体側の要件緩和で対応されてほしいです。

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

関連するチケット

関連している バグ #1229: 特定のMessage Modelが表示される時クラッシュすることがある終了

関係しているリビジョン

リビジョン 29a3762b (差分)
toshi_a 初音4ヶ月前に追加

PostBoxのリプライ先にTwitterのMessageしか表示されていなかった refs #1205

履歴

#1 toshi_a 初音4ヶ月前に更新

  • ステータス新規 から 実装待ち に変更
  • 対象バージョン3.6 から 3.7 に変更

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

#2 cob odo4ヶ月前に更新

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

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

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

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

#3 toshi_a 初音4ヶ月前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者cob odo にセット
  • プラグイン名topic/1205-intelligent-text-view-for-everything にセット

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

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

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

#4 cob odo4ヶ月前に更新

  • ステータスレビュー待ち から 終了 に変更

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

#5 toshi_a 初音3ヶ月前に更新

  • 関連している バグ #1229: 特定のMessage Modelが表示される時クラッシュすることがある を追加

他の形式にエクスポート: Atom PDF