We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
2で登録したものが反映されない
2で登録されたものが反映される
case2のデータがあることを前提とする
更新日時ではなく同一曲タイトルの一番スキル値が高いものが表示されてしまう。
仕様にするかバグとするかは要検討?
The text was updated successfully, but these errors were encountered:
ご指摘のとおり,「同一曲タイトルの一番スキル値が高いものを表示する」挙動になっています. これは一応自分の意図通りなのですが,クローン元と挙動が違うので,現状たしかにバグのようなものかなと思います.
これは,全難易度登録出来たほうがある意味便利なのではないかという見立ての元こういう作りになっているのですが,以下の一点が未実装なため,混乱を招く形になっています.Issue に書いておくべき問題だったと思います.すみません.
ご指摘いただいた各ケースについては,以下のように説明できます.
DBには下の2件が登録されます.
Agnus Dei 7.95 MAS(G) 99.00% (SS) 157.41 Agnus Dei 7.30 MAS(B) 100.00% (EXC) 146.00
このうち,ギター紫の方がスキルが高いのでスキル帳にはギター紫が反映されます. (ギター紫のスキルを削除するとベース紫が反映されます.)
0時20分のRoulette 7.30 EXT(G) 100.00% (EXC) 146.00 0時20分のRoulette 2.60 BSC(G) 100.00% (EXC) 52.00
このうち,ギター赤のほうがスキルが高いのでスキル帳にはギター赤が反映されます.
case2の状態で,DBには下の2件が登録されています.
ここで,ギター赤のスキルを編集すると,DBは以下の状態になります.
0時20分のRoulette 2.60 BSC(G) 100.00% (EXC) 52.00 0時20分のRoulette 7.30 EXT(G) 10.00% (C) 14.60
このうち,ギター青のほうがスキルが高いので,スキル帳にはギター青が反映されます.
Sorry, something went wrong.
現状たしかに問題なので,Openにしておいて上記点追加実装または修正するタイミングでCloseにします.
「同一曲タイトルの一番スキル値が高いものを表示する」 という不自然な仕様なため,ユーザ側からすると完全にバグに見える, という状況なので 一応 bug ラベルを設定しておきますが, バグ修正というよりかは,
*スキル対象難易度以外のデータはなんらかの形でアクセス可能にする(スキル帳の各行からスキルページへ飛ぶ)
という機能追加により修正予定です.
No branches or pull requests
再現手順
case1
Agnus Dei を
難易度:マスターギター
達成:99
で登録
Agnus Deiを
難易度:マスターベース
達成:100
で登録
2で登録したものが反映されない
case2
0時20分のRouletteを
難易度: ベーシックギター
達成:100
で登録
0時20分のRouletteを
難易度: エクストリームギター
達成:100
で登録
2で登録されたものが反映される
case3
case2のデータがあることを前提とする
0時20分のRouletteを押下して編集画面に遷移
難易度: エクストリームギター
達成:10
で編集ボタン押下しスキル閲覧画面に遷移
0時20分のRoulette
難易度: ベーシックギター
達成:100
が反映される
症状の推論
更新日時ではなく同一曲タイトルの一番スキル値が高いものが表示されてしまう。
仕様にするかバグとするかは要検討?
The text was updated successfully, but these errors were encountered: