diff --git a/src/UFExporter.frm b/src/UFExporter.frm index ee32b1f..fc37162 100644 --- a/src/UFExporter.frm +++ b/src/UFExporter.frm @@ -43,6 +43,7 @@ Attribute appn.VB_VarHelpID = -1 Const NoFolderStr As String = "" Const InvalidSelStr As String = "" Const NoHeaderRngStr As String = "" +Const BadHeaderDefStr As String = "" ' ===== GLOBALS ===== @@ -442,13 +443,30 @@ Private Function getHeaderRangeAddress() As String ' ' Or, if header export is deselected, report accordingly + Dim headerRange As Range + + ' Store the return value from retrieving the header range + Set headerRange = getHeaderRange + If ChBxHeaderRows.Value Then - If checkHeaderRowValues Then + ' Header range has to be defined in order for there to be + ' an address to return. The validity of the header row + ' definition in the userform is already checked + ' within getHeaderRange, and so it doesn't need(?) to be + ' checked again here. + If Not headerRange Is Nothing Then getHeaderRangeAddress = getHeaderRange.Address( _ RowAbsolute:=False, ColumnAbsolute:=False _ ) Else - getHeaderRangeAddress = InvalidSelStr + ' Though, it's clearer to change the error message in the display box + ' depending on whether the header definition is invalid, + ' or if the actual range selection on ActiveSheet is bad + If Not checkHeaderRowValues Then + getHeaderRangeAddress = BadHeaderDefStr + Else + getHeaderRangeAddress = InvalidSelStr + End If End If Else getHeaderRangeAddress = NoHeaderRngStr diff --git a/src/UFExporter.frx b/src/UFExporter.frx index b392f93..fe2f963 100644 Binary files a/src/UFExporter.frx and b/src/UFExporter.frx differ