Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-18 16:51:18 +00:00
Code Issues Projects Releases Packages Wiki Activity

[RAPPS] Must apply settings changes when a new download folder is created (#7792)

Browse source
This commit is contained in:
Whindmar Saksit 2025-03-19 12:24:52 +01:00 committed by GitHub
parent dba4efbcae
commit d769f5675d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 29 additions and 23 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
base/applications/rapps/geninst.cpp
View file

@ -173,17 +173,10 @@ struct InstallInfo : CommonInfo
}
};
static UINT
static inline UINT
ErrorBox(UINT Error = GetLastError())
{
if (!Error)
Error = ERROR_INTERNAL_ERROR;
WCHAR buf[400];
UINT fmf = FORMAT_MESSAGE_IGNORE_INSERTS | FORMAT_MESSAGE_FROM_SYSTEM;
FormatMessageW(fmf, NULL, Error, 0, buf, _countof(buf), NULL);
MessageBoxW(g_pInfo->GetGuiOwner(), buf, 0, MB_OK | MB_ICONSTOP);
g_pInfo->Error = Error;
return Error;
return g_pInfo->Error = ErrorBox(g_pInfo->GetGuiOwner(), Error);
}
static LPCWSTR

3
base/applications/rapps/include/misc.h
View file

@ -26,6 +26,9 @@ ErrorFromHResult(HRESULT hr)
return hr >= 0 ? ERROR_SUCCESS : hr;
}
UINT
ErrorBox(HWND hOwner, UINT Error = GetLastError());
VOID
CopyTextToClipboard(LPCWSTR lpszText);
VOID

12
base/applications/rapps/misc.cpp
View file

@ -12,6 +12,18 @@
static HANDLE hLog = NULL;
UINT
ErrorBox(HWND hOwner, UINT Error)
{
if (!Error)
Error = ERROR_INTERNAL_ERROR; // Note: geninst.cpp depends on this
WCHAR buf[400];
UINT fmf = FORMAT_MESSAGE_IGNORE_INSERTS | FORMAT_MESSAGE_FROM_SYSTEM;
FormatMessageW(fmf, NULL, Error, 0, buf, _countof(buf), NULL);
MessageBoxW(hOwner, buf, 0, MB_OK | MB_ICONSTOP);
return Error;
}
VOID
CopyTextToClipboard(LPCWSTR lpszText)
{

26
base/applications/rapps/settingsdlg.cpp
View file

@ -256,11 +256,6 @@ SettingsDlgProc(HWND hDlg, UINT Msg, WPARAM wParam, LPARAM lParam)
case IDOK:
{
HandleGeneralListItems(GetDlgItem(hDlg, IDC_GENERALLIST), NULL, &NewSettingsInfo);
if (SettingsInfo.bSmallIcons != NewSettingsInfo.bSmallIcons)
{
SendMessageW(hMainWnd, WM_SETTINGCHANGE, SPI_SETICONMETRICS, 0); // Note: WM_SETTINGCHANGE cannot be posted
PostMessageW(hMainWnd, WM_COMMAND, ID_REFRESH, 0);
}
CStringW szDir;
CStringW szSource;
@ -288,23 +283,21 @@ SettingsDlgProc(HWND hDlg, UINT Msg, WPARAM wParam, LPARAM lParam)
NewSettingsInfo.szNoProxyFor, _countof(NewSettingsInfo.szNoProxyFor), szNoProxy,
szNoProxy.GetLength() + 1);
CStringW::CopyChars(
NewSettingsInfo.szDownloadDir, _countof(NewSettingsInfo.szDownloadDir), szDir,
szDir.GetLength() + 1);
dwAttr = GetFileAttributesW(szDir);
if (dwAttr != INVALID_FILE_ATTRIBUTES && (dwAttr & FILE_ATTRIBUTE_DIRECTORY))
{
CStringW::CopyChars(
NewSettingsInfo.szDownloadDir, _countof(NewSettingsInfo.szDownloadDir), szDir,
szDir.GetLength() + 1);
}
else
if (dwAttr == INVALID_FILE_ATTRIBUTES || !(dwAttr & FILE_ATTRIBUTE_DIRECTORY))
{
CStringW szMsgText;
szMsgText.LoadStringW(IDS_CHOOSE_FOLDER_ERROR);
if (MessageBoxW(hDlg, szMsgText, NULL, MB_YESNO) == IDYES)
{
if (CreateDirectoryW(szDir, NULL))
if (!CreateDirectoryW(szDir, NULL))
{
EndDialog(hDlg, LOWORD(wParam));
ErrorBox(hDlg);
break;
}
}
else
@ -330,6 +323,11 @@ SettingsDlgProc(HWND hDlg, UINT Msg, WPARAM wParam, LPARAM lParam)
szSource.GetLength() + 1);
}
if (SettingsInfo.bSmallIcons != NewSettingsInfo.bSmallIcons)
{
SendMessageW(hMainWnd, WM_SETTINGCHANGE, SPI_SETICONMETRICS, 0); // Note: WM_SETTINGCHANGE cannot be posted
PostMessageW(hMainWnd, WM_COMMAND, ID_REFRESH, 0);
}
SettingsInfo = NewSettingsInfo;
SaveSettings(GetParent(hDlg), &SettingsInfo);
EndDialog(hDlg, LOWORD(wParam));