Vistaスタイルのファイルダイアログでファイルの作成確認が余計に行われる about sakura HOT 6 CLOSED

なんというか、既知のバグを報告しておきます。

既存のファイルダイアログには、上書き確認メッセージを表示するカスタムが施されているのですが、アイテムダイアログにはこれが移植されてないっぽいです。

文字列リソース STR_DLGOPNFL3の定義が（おそらく）誤っており、新旧比較すると差になります。

誤： "名前を付けて保存"
正： "名前を付けて保存の確認"

これ直しても誰も嬉しくなさそうなので黙ってました。
アイテムダイアログに関しての説明では、「既存機能の一部が移植されていないかも」という話になると思います。

from sakura.

以下がcppで定義されているために、単体テストが記述不能である点が気になっています。

• struct CDlgOpenFile_CommonFileDialog 旧来のOFNダイアログ実装。
• struct CDlgOpenFile_CommonItemDialog Vista以降のアイテムダイアログ実装。

このプロジェクトで「エンジニアリングしたい民」はどうも自分だけのようなので、他の人が気にならないなら放置で良いと思っています。

from sakura.

以下がcppで定義されているために、単体テストが記述不能である点が気になっています。

• struct CDlgOpenFile_CommonFileDialog 旧来のOFNダイアログ実装。
• struct CDlgOpenFile_CommonItemDialog Vista以降のアイテムダイアログ実装。

tests/unittests/test-cdlgopenfile.cpp のテストを増やしたいという事でしょうか？

Interface関数以外の内部実装の単体テストを細かく書いてしまうと、内部実装を調整した際に単体テストの保守も行う必要が出てくるので大変な気が個人的にします。ただ他の人が何かの理由でやりたいのであればあえて止めはしません。

struct CDlgOpenFile_CommonFileDialog と struct CDlgOpenFile_CommonItemDialog の定義をヘッダファイル (例えば sakura_core/dlg/CDlgOpenFile.h とか？)に引っ越しするとかですかね。

このプロジェクトで「エンジニアリングしたい民」はどうも自分だけのようなので、他の人が気にならないなら放置で良いと思っています。

テストも自動化しないとですね。https://github.com/appium/appium-windows-driver を今度使ってみます。

from sakura.

FOS_CREATEPROMPT
Value: 0x2000
Prompt for creation if the item returned in the open dialog does not exist. Note that this does not actually create the item.

この値が CDlgOpenFile_CommonItemDialog::DoModalOpenDlgImpl1 で使われているのが、ファイルの作成確認が余計に行われる原因である事を確認しました。

from sakura.

なんというか、既知のバグを報告しておきます。

既存のファイルダイアログには、上書き確認メッセージを表示するカスタムが施されているのですが、アイテムダイアログにはこれが移植されてないっぽいです。

把握していなかったので、上書き確認メッセージがどう異なるかを動作確認してみました。

既存のファイルダイアログ (実装 CDlgOpenFile_CommonItemDialog) の上書き確認メッセージ

Vistaスタイルのファイルダイアログ (実装 CDlgOpenFile_CommonItemDialog) の上書き確認メッセージ

メッセージボックスのタイトルが異なりますね。berryzplusさんが書いたように "名前を付けて保存の確認" が正しいと思います。

他に気づいた点として、ファイルパスが既存のファイルダイアログはフルパスで表示されていますが、Vistaスタイルのファイルダイアログではファイル名だけが表示されています。

どちらも些細な点だと思うので更新しなくても良いと思いますが、仮に変更するとしたら CDlgOpenFile_CommonItemDialog::OnOverwrite に実装を追加すれば良さそうです。

https://learn.microsoft.com/en-us/windows/win32/api/shobjidl_core/nf-shobjidl_core-ifiledialogevents-onoverwrite

from sakura.

#1932 がmergeされてこのissueの問題は解消されたと思うのでcloseします。

from sakura.