トレイアイコンから「履歴とお気に入りの管理」ダイアログを開けるようにする

[berryzplus]

PR の目的

タイトル通りです。

カテゴリ

• 不具合修正
  過去に追加した新規メニュー項目が機能していませんでした。

PR の背景

年明けくらいから WinAppDriver による UIテスト の導入を検討しています。

対応を進める中で、過去に追加したメニュー項目が機能していないことに気付きました。

• ドロップダウンメニューに履歴の管理を追加 #551

4年も前の話で、何故この機能が漏れたのかについては記憶がないのですが、
修正内容から見て「コミット漏れ」だったのではないかと思います。

PR のメリット

PR のデメリット (トレードオフとかあれば)

仕様・動作説明

#551 で追加したメニュー項目「履歴の管理」が機能していなかったので、メニュー項目選択時に「履歴とお気に入りの管理」ダイアログを開くようコードを追加します。

PR の影響範囲

• トレイアイコンの左クリックメニュー「履歴の管理」が機能するようになります。

テスト内容

効果確認として以下を行いました。

テスト1

手順

1. サクラエディタを起動する
2. タスクバーの「通知のシェブロン」を押下して「通知のオーバーフロー」ウインドウを開く（「Windowsキー＋B、Enter」で操作しても良いです。）
3. 「通知のオーバーフロー」ウインドウに表示されている「サクラエディタのアイコン（＝トレイアイコン）」を「左クリック」する
4. メニュー項目「履歴の管理」を選択し、「履歴とお気に入りの管理」ダイアログが表示されることを確認する

関連 issue, PR

• ドロップダウンメニューに履歴の管理を追加 #551
• スモークテストを導入したい #1301

参考資料

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

0.0% Coverage
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.4109 completed (commit a9fb81dda8 by @berryzplus)

[beru]

cDlgFavorite.GetHwnd() == nullptr ですが本来必要な判定でしょうか？

判定せずに下記のように書くと問題が起きる事があるのでしょうか？

					{
						CDlgFavorite cDlgFavorite;
						cDlgFavorite.DoModal(m_hInstance, GetTrayHwnd(), (LPARAM)NULL);
					}

[berryzplus]

判定を抜くと静的解析で余分な警告が出ると思います。
「中括弧で括られたブロックを関数にしましょう」的なやつです。

要るか要らないかでいうと「要らない判定」です。
パフォーマンス的に害があると言えるほどの負荷はないのでスルーしてしまって欲しいです。
（まぁ、自分も突っ込まない自信はありませんが。。。）

本来はCEditWndとCControlTrayに共通するクラスを作って、共通コマンドを個別に関数化してあげると分かりやすくなると思いますがPRの趣旨と離れますし、周辺事情を説明しきるのが大変そうなので断念しました。

[beru]

静的解析の警告を避けるために変な書き方をするのは本末転倒な気がしますが、まぁあまり細かい事を気にするとモアナルア・ガーデンの保守費用のキックバ

[berryzplus]

レビューありがとうございます。
とりあえずマージしちゃいます。