Microsoft-WindowsAPICodePack-Shellを分離してほしい

[Freeesia]

利用しているプロジェクトの方針としてファイルのアクセスに対してより適切な方法を提供しているので、Microsoft-WindowsAPICodePack-Shellによるダイアログを利用できないようにしたいと考えています。

また、このライブラリについて個人的にライセンスに不安があります。

ファイルダイアログはフレームワークを利用した実装の1つであり、フレームワークそのものではないので、利用しない選択肢をユーザーが採用できると嬉しいです。
Kamishibai.FileDialogのような別ライブラリとして提供するのはいかがでしょうか？

こちらもぜひ実装に協力させていただければと思います。
ご検討よろしくお願いします。

[nuitsjp]

ご連絡ありがとうございます。

Microsoft-WindowsAPICodePack-Shellのライセンスについて不安があるということで、基本はこちらに依存しない形を目指したいと思います。

対応方法ですが、実はOpenWindowやOpenDialogの実装をアプリケーション固有の実装に差し替えられるようにしたいと考えていました。
それと合わせて次のように対処したいと思いますが、次のような対応で目的は達せられるでしょうか？

1. IWindowServiceからOpenFileとSaveFileは削除する
2. IFileServiceを作成し、OpenFileとSaveFileはそちらに移動する
3. IFileServiceの実装は、Microsoft-WindowsAPICodePack-Shellに依存しないように修正するか、Kamishibai.FileDialogのような別プロジェクトに分離する（ちょっともう少し検討します）
4. PresentationServiceから、IWindowServiceとIFileServiceを利用し分けるように修正する
5. KamishibaiApplicationBuilderのBuildに型引数を指定できるバージョンを用意し、IWindowServiceとIFileServiceの実装を自由に指定可能にする（デフォルトは残したい）

いかがでしょうか？
こちらはたぶんに私に趣味嗜好が含まれそうなので、問題なければ、私の方で1週間以内に対応したいと思います。

[nuitsjp]

👆の

IFileServiceの実装は、Microsoft-WindowsAPICodePack-Shellに依存しないように修正するか、Kamishibai.FileDialogのような別プロジェクトに分離する（ちょっともう少し検討します）

これですが、Microsoft-WindowsAPICodePack-Shellの実装をすべてサポートするのはさすがに困難（だから過去から引っ張り出してきて利用されているわけですが・・・）なので、標準側にはMicrosoft.Win32.OpenFileDialog / SaveFileDialogを利用した、最低限の実装だけしようと思います。
その上で、Kamishibai.Microsoft-WindowsAPICodePack-Shellプロジェクトを作成して、現在の実装はそっちに移します。

起動処理の部分は、破壊的変更がまぬがれませんが、逆に言えばそこだけであとは既存の利用者がそのままで利用できるので。

その方向性でいかがでしょうか？

[Freeesia]

はい、そちらの方針で問題ないと思います。

ちなみに私としては #28 と合わせることで、Kamishibai.Microsoft-WindowsAPICodePack-ShellでPresentationServiceからファイルダイアログを開くコードを生成することで、フレームワークからファイル関連の実装義務を分離できると考えていました。

以下のようなコードを生成するイメージです。

partial IPresentationService
{
    DialogResult OpenFile(OpenFileDialogContext context);
    DialogResult SaveFile(SaveFileDialogContext context);
}

partial PresentationService
{
    public DialogResult OpenFile(OpenFileDialogContext context)
    {
        // Microsoft-WindowsAPICodePack-Shellを利用したファイルダイアログの実装
    }
    public DialogResult SaveFile(SaveFileDialogContext context)
    {
        // Microsoft-WindowsAPICodePack-Shellを利用したファイルダイアログの実装
    }
}

とはいえ、こちらに関しては「フレームワークにファイルダイアログは必要である」という思想であれば、沿わない方針なので元の提案で問題ありません。

特に急いで分離してほしいというわけではないので、お手すきの際に対応いただけますと幸いです。

[nuitsjp]

@Freeesia
なるほど、確かにそういう方法もありますね。良いですね。それ。

フレームワークの全体としてはファイルダイアログは必要なんですが、その対応方法でもよい気がします。
時間も取れるということなので、少し考えさせてください。
対応の方は私の方で実施いたします。

ご相談のっていただき、ありがとうございました！