「削除して編集」で引用として添付したノートを外せない

[esperecyan]

💡 Summary

「削除して編集」したときに、引用として添付していたノートを外せません。×ボタンが出ていて、押すと×ボタンが消えますが、実際には何も起こりません。

🥰 Expected Behavior

引用ノートなしでノートが投稿される。

🤬 Actual Behavior

削除して編集を押す前に引用していたノートが引用されたままになっている。

📝 Steps to Reproduce

1. 引用付きのノートを投稿
2. 投稿したノートを「削除して編集」
3. 編集画面ダイアログで、引用付きの×ボタンを押す
4. 投稿する

💻 Frontend Environment

* Model and OS of the device(s):Windows 10
* Browser:Firefox 129.0.2
* Server URL:https://misskey.io/ / https://misskeytsf.love/
* Misskey:2024.5.0-io.2d / 2024.8.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

[Sayamame-beans]

ふーむ…
related: #14285 (comment)

[Sayamame-beans]

#14285 (comment) から

revisit: quoteIdは引用対象のプレビューが見えないという問題がある(renoteIdの場合は見える) → #10598
プレビューで見えるようになってくれると、恐らくrenoteIdとquoteIdに分ける必要がなくなるし、renoteIdを空に出来るようにフロントを変更するのが最善になりそう…なのですが、それを実現するにはちょっと遠いので、さしあたって対応するならば、

• 引用ノートを削除して編集するときに引用を解除出来ない状態のまま、[ x ]を出ないようにする(元の挙動、引用元ノートが見える)
• 引用ノートを削除して編集するときに引用を解除出来るようにする(14476 Expected Behavior)代わりに、引用元ノートが見えなくなる

のどちらかを選ぶ必要がありますね…

[samunohito]

#10598 は少し実装規模が大きそうですね。

引用ノートを削除して編集するときに引用を解除出来るようにする(14476 Expected Behavior)代わりに、引用元ノートが見えなくなる

こちらですぐに対応できるのであれば、とりあえず直しちゃってよいと思います。
#10598 はひとまず引用を解除できないのを直してから考えても良いかと

[Sayamame-beans]

ではそのようにします

[Sayamame-beans]

MkPostFormの方からだとprops.renoteをundefinedに書き換えるのがどうも上手く出来なかったのですが、根本を断てば良い事に今更気付きました…

os.postの引数付き呼び出し一覧(ソース: https://github.com/search?q=repo%3Amisskey-dev%2Fmisskey%20os.post&type=code)

ドライブからのファイル付きノート: `initialFiles`
チャンネル投稿: `channel`
引用ノート: `renote`, (`channel`)
返信: `reply`, (`channel`)
ユーザーページからのダイレクトノート: `specified`, `initialText`(メンション文字列)
AiScriptからの投稿フォーム: `initialText`, `initialCw`, `initialVisibility`, `initialLocalOnly`, `instant`(True)
ページ/Play/ギャラリー共有, about-misskey, MisskeyGames(Reversi): `initialText`, `instant`(True)
MisskeyGames(drop-and-fusion): `initialText`, `initialFiles`, `instant`(True)
削除して編集: `initialNote`, `renote`, `reply`, `channel`

なので、削除して編集からrenoteを消します(initialNote内にもあるためquoteIdへのコピーには影響無し)

(props.renoteをundefinedに書き換える方法が分かれば、quoteIdとrenoteを両方残しておいて、quoteIdを消した際にrenoteも消すという手も使えそうですが、一旦分からないのでそのまま行きます)

[tai-cha]

props.renoteを消すのと対価な実装を書いてみました
#15249