Volian/SourceCode
8
0
Fork 0
Code Issues Pull Requests 1 Projects Releases Wiki Activity

Fixed logic that allows the user to select an existing RO database. If the user presses Cancel, the code would crash. The code was changed to cancel the Import. Fixes B2015-216

Browse Source
This commit is contained in:
Rich 2015-11-30 22:36:18 +00:00
parent 5cc71b9e10
commit 8e13fc7179
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
PROMS/VEPROMS User Interface/dlgExportImport.cs
View File

@ -236,7 +236,7 @@ namespace VEPROMS
dlg.ImportedROFolder = rofolderpath;
dlg.LocalROFolders = localROPaths;
dlg.ShowDialog(this);
if (dlg.SelectedROFolder != string.Empty)
if ((dlg.SelectedROFolder ?? string.Empty) != string.Empty) // B2015-216 If the return value is null treat it like an empty string
{
MyRODb = RODb.GetByFolderPath(dlg.SelectedROFolder);
oldRODbID = newRODbID = MyRODb.RODbID;
@ -245,6 +245,7 @@ namespace VEPROMS
else
{
MessageBox.Show("Since you did not pick an existing RO folder defined for this database, the import process will terminate.");
this.Close();// Close the Import Window
return;
}
}