[BUG] ノートの投稿内容に承認されていない絵文字が含まれる #52
Comments
Sayamame-beans
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
{{ message }}
Sayamame-beans
changed the title
多分、ここは
acceptedを返したいはず…ですがemojiProcessListに入れたものと同じreconstructionを代入していて、acceptedがどこにも使われていないようです。https://github.com/niwaniwa/MisskeyEmojiBot/blob/2c55996b1c044514d2890d70b94dacd6dbb02baf/emoji.go#L114-L128
にもかかわらず、何故現状でも
IsFinish == trueなはずの絵文字が返せているのか…(軽く調べましたが、goは触れたことが無いため殆ど推測になるのですが、)恐らく値で渡されていることで参照がズレているのだと思います。
例えば
IsAccepted、https://github.com/niwaniwa/MisskeyEmojiBot/blob/2c55996b1c044514d2890d70b94dacd6dbb02baf/misskey.go#L16
この関数の中ではtrueだと思うのですが、他の場所から見ると(初期値の)falseのままなのではないかな?と…
IsFinishも同様ですね。(approveの例)https://github.com/niwaniwa/MisskeyEmojiBot/blob/2c55996b1c044514d2890d70b94dacd6dbb02baf/emoji.go#L67-L81
approve(IsFinishへの代入), uploadToMisskey(IsAcceptedへの代入), disapprove(両者への代入) で引数にポインタを使うようにすれば、多分大丈夫…?
abort, resetも危うそうに見えます。
推測が多分に含まれていて申し訳ないですが、調べていただけると助かります。
The text was updated successfully, but these errors were encountered: