Feature #691
スキルの無効化
100%
Description
ポイントが0になったスキルは無効化する。
物理削除は行わずに論理的にenableプロパティをfalseにする。
PointdownControllerを書き換えるのと、
getSkillsも対応する
Subtasks
Updated by yusuke kokubo almost 10 years ago
- Status changed from 新規（New） to 解決（Resolved）
7bddf6019e8d36c5558e7993e834bae2000eed27
とりあえず実装してみた。
ArrayListをフィルタリングする処理がとてもかっこ悪い…。
Updated by 啓介 大橋 almost 10 years ago
フィルタリングするところ(if(rel.getPoint() == null) continue;)ってinMemoryFilterじゃまずいんでしたっけ？？
Updated by yusuke kokubo almost 10 years ago
サーバ側で処理するべきかクライアント側で処理するべきかで迷ってたんですけど、
サーバ側でやった方が良いかもしれませんね…。
Updated by yusuke kokubo almost 10 years ago
すみません。やっぱり後で直しておきます。クライアント側で色々難しいことをやらせない方が良いのかも。
元々は、
enable=trueなやつもfalseなやつも取得しておいて、
表示するときはtrueなやつだけで、「過去のスキル」みたいなリンクをクリックすると
falseなやつがグニュって表示されるイメージでした。
でもよくよく考えたら過去のスキルをクリックしたタイミングでサーバアクセスすればすむ話なんですよね。
Updated by 啓介 大橋 almost 10 years ago
なるほど！
過去のスキルはのちのちDatastoreから消すのかな？って思ってました。
確かに最初は取ってこないで、リンク押下時に取ってきたほうが優しいですね！
Updated by yusuke kokubo almost 10 years ago
とりあえずenableなスキルを取得するAPIとdisableなスキルを取得するAPIをわけるようにします。
勝手なお願いですが、過去のスキルを取得するUIを作ってもらってもよいですか？
Updated by 啓介 大橋 almost 10 years ago
勝手なお願いですが、過去のスキルを取得するUIを作ってもらってもよいですか？
了解です！
別チケットで管理させてくださいw
Updated by yusuke kokubo almost 10 years ago
- Status changed from 解決（Resolved） to 終了（Closed）
APIはわけました。
このチケットはとりあえず終了にしておきます。