[release-0.17] hotfix: プロジェクトファイル読み込みと声量・音域パラメータの初期化バグ修正 #1927
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Hiroshiba
commented
Mar 9, 2024
Comment on lines
-104
to
-105
| await store.dispatch("SET_SINGER", {}); | ||
|
|
There was a problem hiding this comment.
そもそも呼ぶ必要はないはず。
内部でSETUP_SINGERが行われるから呼んでるだけ？
とりあえずプロジェクトファイルが読み込まれてなかった時だけ実行するように移動しました。
Hiroshiba
commented
Mar 9, 2024
| @@ -134,7 +134,6 @@ | |||
| <script setup lang="ts"> | |||
| import { computed, watch, ref, onMounted, onUnmounted } from "vue"; | |||
| import { useStore } from "@/store"; | |||
| import { isProduction } from "@/type/preload"; | |||
Hiroshiba
commented
Mar 9, 2024
Comment on lines
-229
to
+242
| watch(keyRangeAdjustment, () => { | ||
| keyRangeAdjustmentInputBuffer.value = keyRangeAdjustment.value; | ||
| }); | ||
| watch( | ||
| keyRangeAdjustment, | ||
| () => { | ||
| keyRangeAdjustmentInputBuffer.value = keyRangeAdjustment.value; | ||
| }, | ||
| { immediate: true } | ||
| ); | ||
|
|
||
| watch(volumeRangeAdjustment, () => { | ||
| volumeRangeAdjustmentInputBuffer.value = volumeRangeAdjustment.value; | ||
| }); | ||
| watch( | ||
| volumeRangeAdjustment, | ||
| () => { | ||
| volumeRangeAdjustmentInputBuffer.value = volumeRangeAdjustment.value; | ||
| }, | ||
| { immediate: true } | ||
| ); |
There was a problem hiding this comment.
immediate: trueをつけました。
プロジェクトファイル読み込み起動だった場合、storeが更新された後にここのコンポーネントがマウントされます。
そのマウントされるタイミングでstoreから値を持ってきてなかったのでバグってた感じです。
そもそもたぶんInputBuffer用の変数は必要ないかもでした。
Hiroshiba
commented
Mar 9, 2024
Comment on lines
+117
to
+119
| await dispatch("SET_VOLUME_RANGE_ADJUSTMENT", { | ||
| volumeRangeAdjustment: tracks[0].volumeRangeAdjustment, | ||
| }); |
There was a problem hiding this comment.
プロジェクトファイル読み込んだ後に UI（Store）への反映を忘れていたのがバグの原因。
そもそも同じ状態を保持しているものが2つあるのはこういうことが起こるのでよくないはず。
（state.volumeRangeAdjustmentとstate.tracks[0].volumeRangeAdjustment）
多分片方をGETTERにすれば問題が起こらなくなるはず。
|
テストが通り次第マージしてアップデートをかけようと思ってます！ |
|
すみません、ちょっとワークアラウンドを足してしまいましたがマージします 🙇 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
の解決プルリクエストです。
解説はコード内で行おうと思います。
関連 Issue
#1926
その他
プロジェクトファイルには保存されてたので良かった。
マイグレーションも特に必要ないはず。