opencorenmr / console-devel-2.0.1

2 stars 2 forks source link

Part of a file name between "." is ignored when saving data in the file widget (a file name containing period is not supported) #22

Open YIsoda opened 4 years ago

YIsoda commented 4 years ago

再現手順

環境:Windows 10 Pro 64bit,Ver. 1909

  1. データファイル data.opd を開く
  2. そのまま “Save” でファイル名を data-2.processed.opd とする

結果,data-2.opd が生成されます。

また,ピリオドで区切って ".processed." などをサフィックス代わりにくっつけたつもりで保存するとその部分が無視され,結果警告なくもとのデータファイルが上書きされてしまいます

YIsoda commented 4 years ago

(補足)通常,存在するファイル名を指定すると,画像のようにOSのダイアログボックスにもともとある上書き防止のダイアログが表示される(Windowsの場合)ため,そこを通り抜けているにもかかわらずサイレントにファイルが上書きされる動作は余計にわかりにくく感じます image

opencorenmr commented 4 years ago

Thanks for the report. To cover such cases, QFileInfo::baseName() needs to be replaced by QFileInfo::completeBaseName() in several places. I will soon check the codes and fix this problem!

なるほど、そういうファイルネーミングの流儀もあるのですね!この場合でも正しく働くようにするためにはQFileInfo::baseName()QFileInfo::completeBaseName()に直せばいいはず。該当箇所をチェックしてあとで直しておきます!

opencorenmr commented 4 years ago

Pull requested!

YIsoda commented 4 years ago

Thank you for the speedy fix!