Defect #588
完了賛同しましたダイアログが複数出る
100%
説明
UserUIにて「"自分も賛同する"」ボタンをクリックするたびに、
agreedSubmit(ボタン)にaddClickHandlerをしてしまっている為、
同じHandlerが複数動いてしまっている。
(多分同じ人が重複登録されていたのもそれのせい?)
addClickHandlerするときの戻り値HandlerRegistrationを保存しておいて、
登録前に毎回消してから登録する必要あり。
(それかagreedSubmitをnewしなおす)
yusuke kokubo さんが13年以上前に更新
ありがとうございます。
よくわかってなくてすみません。
この方法だと最後にaddClickHandlerしたハンドラしか有効にならなくないですか?
啓介 大橋 さんが13年以上前に更新
賛同ボタンクリックしたタイミングで、addClickHandlerされるので、
大丈夫かなと思ってます。
(キャンセルや、submitをしてダイアログとじても、
次に賛同ボタンが押されるときには、前のやつを消せばまだHandlerが積まれてない)
毎回賛同ボタン押すごとに最新のHandlerを更新するみたいな感じで。
完全にやるなら、submitや、cancelのタイミングでHandlerRegistration#removeHandlerを呼び出して
削除してもいいです。(削除後保存している、HandlerRegistrationをnull更新)
ちょっと試せてませんが。。。。
申し訳ないっす。。。
yusuke kokubo さんが13年以上前に更新
- ステータス を 新規(New) から 担当(Assigned) に変更
- 担当者 を yusuke kokubo にセット
とりあえずこんな感じに直してみました。どうでしょう?
http://github.com/YusukeKokubo/SkillMaps/commit/03314e1d06e0cd22654c4994ff7cf254a0c94b51
yusuke kokubo さんが13年以上前に更新
1回目
賛同ボタン作成→賛同ボタンクリック→submit click handler登録→(ホゲホゲ)2回目以降
賛同ボタン作成、前回のsubmit click handler削除→賛同ボタンクリック→submit click handler登録→(ホゲホゲ)
あー。なるほど!
すいません やってもらって。。。
こちらこそありがとうございます!
この問題はわかってる人に教えてもらわないとずっとわからないままだったと思います。
yusuke kokubo さんが13年以上前に更新
- ステータス を 解決(Resolved) から 終了(Closed) に変更
- 進捗率 を 0 から 100 に変更
最新のmasterではボタンを作るごとにインスタンスを別にしたので
重複するようなことはなくなったはず。