Project

General

Profile

Actions

バグ #1190

open

タイムラインのアイコン上のボタンがSpellを利用していない

Added by cob odo over 3 years ago. Updated over 2 years ago.

Status:
レビュー待ち
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
portal
ブランチ:
topic/1190-main-icon-button-should-use-spell
クラッシュする:
No

Description

favorite spellではなくmessage.favoriteメソッドを使っているため。


再現手順

・portal worldを選択する。
・ふぁぼボタンを押す


Related issues

Related to 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる実装待ちtoshi_a 初音2019-06-02

Actions
Actions #1

Updated by toshi_a 初音 over 3 years ago

Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?

Actions #2

Updated by cob odo over 3 years ago

toshi_a 初音 さんは書きました:

Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?

問題ありません。

当初こういうタイトルにしたのは、コードを読まない(読めない)人が一見不具合に思える事象に遭遇するとしたら、こういう経路だろう、ということで、そういう人でも一覧から既知の報告として発見できる可能性を考慮したためです。
標準添付プラグインの範囲内では、ボタンがspellを使っていなくても、表面的な挙動に不都合は無いので、サードパーティプラグインを使って初めて出てくるかな、という意味で「バグ」トラッカーでportal関連にしました。

とはいえ、結局はとしぁさんが管理しやすいようにトリアージすべきだと思いますので、変えていただいた方がいいと思います。

Actions #3

Updated by toshi_a 初音 over 3 years ago

  • Subject changed from portalを使うとアイコンの上のふぁぼボタンを押してもふぁぼれない to タイムラインのアイコン上のボタンがSpellを利用していない
  • Status changed from 分類待ち to 実装待ち

チケットのタイトルは変えておきますが、対応されたあとのレビューでは、再現手順に書いてもらった方法で試してもらうので大丈夫です。

Actions #4

Updated by cob odo over 3 years ago

RTボタンの方も、解除時はdestroy_share spellではなくMessage#destroyを使っているようです。

Actions #5

Updated by cob odo over 2 years ago

  • Status changed from 実装待ち to レビュー待ち
  • ブランチ set to topic/1190-main-icon-button-should-use-spell

1年越しにパッチを書きました。ただし、あんふぁぼのテストには #1325 が必要です。

Actions #6

Updated by cob odo over 2 years ago

……と書きましたが、あんふぁぼはうまく動いてないかもしれないですね。どちらかというと #1325 側の修正が足りていない雰囲気がします。

Actions #7

Updated by toshi_a 初音 about 2 years ago

  • Related to 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる added
Actions

Also available in: Atom PDF