プロジェクト

全般

プロフィール

バグ #1024

リプライの可否の条件に system? が使われている

あひる 家鴨5ヶ月前に追加. 4ヶ月前に更新.

ステータス:
新規
優先度:
通常
担当者:
-
対象バージョン:
開始日:
2017-04-06
期日:
進捗率:

0%

プラグイン名:
再現手順:

説明

アイコンホバーのリプライ表示の条件に repliable? ではなく、 system? が使用されている。


関連するチケット

関連している 機能 #980: 対象のModelが機能を提供できないアイコンオーバーボタンは、そのことがわかるような表示にする 新規 2017-01-29

履歴

#1 Izumi Tsutsui5ヶ月前に更新

system? が使われているのはたぶん #670#note-5 が発端だと思います。

いきなりDR無効化状態で起動すると
「チュートリアル中のみくったーさんにマウスオーバーアイコンでリプライできてしまう(そして落ちる)」
という問題が発覚した時にテキトーに提案したものが採用されたのです。

今はもっと適切なものがあるのであれば変えてもよいのかも?

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

なるほど、mikutterコマンドの条件だとrepliable?を見ているが、アイコンオーバボタンは独自に条件を実装しているので、迂回できてしまうということですね。

#1024#note-1
これはアイコンオーバボタンの話ですね。

そもそも、システムメッセージをこういうふうに特別扱いするのは3.5となってはあまり良くないのですが、かといってアイコンオーバボタンをシステムメッセージに表示するのは無意味なので、これ自体は良くはないが害はなさそうです。丁寧にやるなら、不可能なアクションに対応するボタンは表示しないとかやったほうがいいと思います。そうした場合、多分システムメッセージにはまりもだけが表示されるようになります。

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

実は、アイコンオーバボタンにはmikutterコマンドを配置したいとずっと思っていたのですが、設定するUIが面倒くさそうなのと、自分でアイコンオーバボタンをあまりつかっていないので、ずっと保留していました。
しかしこれを根本的に解決するには、リプライの可否を単一の方法で取得できる必要があって、mikutterコマンドの条件を使えたら美しそうですね。

本件の解決は、アイコンオーバボタン側に修正を行って、repliable?メソッドがtrueを返さないとPostBoxを生成しないようにします。上に書いた根本的な解決については、Serviceとの絡みもあるので、ちょっと考えます。

#4 toshi_a 初音14日前に更新

  • 関連している 機能 #980: 対象のModelが機能を提供できないアイコンオーバーボタンは、そのことがわかるような表示にする を追加

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