mirror of
https://github.com/fadden/ciderpress.git
synced 2024-11-26 17:49:21 +00:00
d42b9c6dc0
The static analyzer was annoyed that the return value from calls to CString::LoadString() was being ignored. This adds a wrapper function that checks the value and logs a failure message if the string can't be found.
103 lines
2.8 KiB
C++
103 lines
2.8 KiB
C++
/*
|
|
* CiderPress
|
|
* Copyright (C) 2007 by faddenSoft, LLC. All Rights Reserved.
|
|
* See the file LICENSE for distribution terms.
|
|
*/
|
|
#include "stdafx.h"
|
|
#include "RenameEntryDialog.h"
|
|
|
|
BEGIN_MESSAGE_MAP(RenameEntryDialog, CDialog)
|
|
ON_WM_HELPINFO()
|
|
ON_COMMAND(IDHELP, OnHelp)
|
|
ON_BN_CLICKED(IDC_RENAME_SKIP, OnSkip)
|
|
END_MESSAGE_MAP()
|
|
|
|
|
|
BOOL RenameEntryDialog::OnInitDialog(void)
|
|
{
|
|
ASSERT(fBasePath.IsEmpty());
|
|
fOldFile = fOldName;
|
|
fFssepStr = fFssep;
|
|
|
|
CEdit* pEdit = (CEdit*) GetDlgItem(IDC_RENAME_PATHSEP);
|
|
pEdit->SetReadOnly(!fCanChangeFssep);
|
|
pEdit->LimitText(1);
|
|
|
|
/* if they can't rename the full path, only give them the file name */
|
|
if (fCanRenameFullPath || fFssep == '\0') {
|
|
fNewName = fOldName;
|
|
// fBasePath is empty
|
|
} else {
|
|
int offset;
|
|
|
|
offset = fOldName.ReverseFind(fFssep);
|
|
if (offset < fOldName.GetLength()) {
|
|
fBasePath = fOldName.Left(offset);
|
|
fNewName = fOldName.Right(fOldName.GetLength() - (offset+1));
|
|
} else {
|
|
/* weird -- filename ended with an fssep? */
|
|
ASSERT(false); // debugbreak
|
|
// fBasePath is empty
|
|
fNewName = fOldName;
|
|
}
|
|
}
|
|
|
|
/* do the DoDataExchange stuff */
|
|
CDialog::OnInitDialog();
|
|
|
|
/* select the editable text and set the focus */
|
|
pEdit = (CEdit*) GetDlgItem(IDC_RENAME_NEW);
|
|
ASSERT(pEdit != NULL);
|
|
pEdit->SetSel(0, -1);
|
|
pEdit->SetFocus();
|
|
|
|
return FALSE; // we set the focus
|
|
}
|
|
|
|
void RenameEntryDialog::DoDataExchange(CDataExchange* pDX)
|
|
{
|
|
CString msg, failed;
|
|
|
|
CheckedLoadString(&failed, IDS_MB_APP_NAME);
|
|
|
|
/* fNewName must come last, or the focus will be set on the wrong field
|
|
when we return after failure */
|
|
DDX_Text(pDX, IDC_RENAME_OLD, fOldFile);
|
|
DDX_Text(pDX, IDC_RENAME_PATHSEP, fFssepStr);
|
|
DDX_Text(pDX, IDC_RENAME_NEW, fNewName);
|
|
|
|
/* validate the path field */
|
|
if (pDX->m_bSaveAndValidate) {
|
|
if (fNewName.IsEmpty()) {
|
|
msg = "You must specify a new name.";
|
|
goto fail;
|
|
}
|
|
|
|
msg = fpArchive->TestPathName(fpEntry, fBasePath, fNewName, fFssep);
|
|
if (!msg.IsEmpty())
|
|
goto fail;
|
|
|
|
if (fFssepStr.IsEmpty())
|
|
fFssep = '\0';
|
|
else
|
|
fFssep = (char) fFssepStr.GetAt(0); // could be '\0', that's okay
|
|
}
|
|
|
|
return;
|
|
|
|
fail:
|
|
ASSERT(!msg.IsEmpty());
|
|
MessageBox(msg, failed, MB_OK);
|
|
pDX->Fail();
|
|
return;
|
|
}
|
|
|
|
void RenameEntryDialog::OnSkip(void)
|
|
{
|
|
/*
|
|
* User pressed the "skip" button, which causes us to bail with a result that
|
|
* skips the rename but continues with the series.
|
|
*/
|
|
EndDialog(IDIGNORE);
|
|
}
|