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

書き出し先を固定するをオンにした状態で書き出し先のフォルダが未指定のまま書き出しを行うとインストールディレクトリにファイルが保存されてしまう #1847

Closed
sabonerune opened this issue Feb 14, 2024 · 1 comment · Fixed by #1881
Labels
バグ 優先度:中 初心者歓迎タスク 初心者にも優しい簡単めなタスク

Comments

@sabonerune
Copy link
Contributor

不具合の内容

ref: #1844 (review)

書き出し先を固定するを初めてオンにすると書き出し先にのフォルダが未指定状態になります。
このままの状態で音声書き出しをするとインストールディレクトリにファイルが保存されてしまいます。

現象・ログ

fixedExportDirの初期値が空文字のため保存場所がカレントディレクトリからの相対パスとして扱われてしまうためカレントディレクトリであるインストールディレクトリに保存される。

再現手順

  1. 設定ファイルのfixedExportDirを空文字にする
  2. 書き出し先を固定するをオン
  3. 音声書き出しを行う

期待動作

少なくとも明示的に指定しない限りインストールディレクトリが書き出し先にならないようにする。

その他

Windows環境のVOICEVOXのインストールディレクトリはアンインストールが実行されると(更新・上書きも含む)中身が全て削除されます。

気付かないまま放置すると非常に悲しいことになるので修正するべきです。

どのような動作にすべきかは色々あるような気がします。

  • 保存時にfixedExportDirがファイルパスか確認する。
  • 初期値にapp.getPath("documents")が設定されるようにする。
  • 書き出し先を固定するをオンにしたときダイアログが表示されるようにする。
@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 14, 2024

issue作成ありがとうございます!!

取れる手のうち3つ目の

書き出し先を固定するをオンにしたときダイアログが表示されるようにする。

が一番良いと思います。空欄保存がないようにできるので。
ただまあ結構実装が大変な気がします。書き出しファイル名パターンと同じように実装すればいいと思うのですが・・・。

なので一旦

初期値にapp.getPath("documents")が設定されるようにする。

でも良いのかなと思っています。何よりも穴を防ぐことが大事そうかなと。

実装してくださる方募集中です 🙇
(方法が分からなければ何でも聞いてください!!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
バグ 優先度:中 初心者歓迎タスク 初心者にも優しい簡単めなタスク
Projects
None yet
2 participants