Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

同じIDのトゥートを2つ登録してからダブルクリックで削除すると、画面上では1つしか消えないが2つ削除されている #92

Closed
sasanquaneuf opened this issue Jan 13, 2020 · 7 comments · Fixed by #93

Comments

@sasanquaneuf
Copy link
Contributor

内容は表題の通り。下記画像参照。
<再現手順>
1.index.htmlでトゥートをいくつか登録し、同じIDを複数含む状態にする。
画像1
2.複数含まれるIDのトゥートを一つダブルクリックすると、画面上は一つだけ削除されるが、URLは複数削除されている。
※とりあえずURLずれてるけど、他にも影響あるかも?
画像2

@yumetodo
Copy link
Contributor

これって #72 での話ですか・・・?

@yume-yu
Copy link
Collaborator

yume-yu commented Jan 13, 2020

見えてるアドレス的には #72 っぽいですね

@yumetodo
Copy link
Contributor

image
image
つまりdelete card_list[index];が悪い。

@yumetodo
Copy link
Contributor

d50e36a#diff-42f69825340f0b5fa628d80900a46daaL65-R85

#62 で私がやらかしてた

@sasanquaneuf
Copy link
Contributor Author

いちおう、再現したのは#72のやつでしたが、 https://qithub-bot.github.io/mastogetter/ でもおかしくなりました。

@yumetodo
Copy link
Contributor

直します。

@yumetodo
Copy link
Contributor

修正投げました。

KEINOS added a commit that referenced this issue Jan 13, 2020
fix: カードを削除したときにcard_listが二重削除されていた問題を修正
Close #92
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants