wxTinderbox build fix.

[wxWidgets.git] / src / motif / filedlg.cpp

diff --git a/src/motif/filedlg.cpp b/src/motif/filedlg.cpp
index 04966afc992f5dbbe859b500b273a7bdef143d26..0d3c9abdbe28fedf4facd0e30680d4ee8d64b98a 100644 (file)
--- a/src/motif/filedlg.cpp
+++ b/src/motif/filedlg.cpp
@@ -9,307 +9,331 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "filedlg.h"
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
+#ifdef __VMS
+#define XtDisplay XTDISPLAY
+#define XtParent XTPARENT
+#define XtWindow XTWINDOW

 #endif
 
 #include "wx/defs.h"
 #include "wx/utils.h"
 #endif
 
 #include "wx/defs.h"
 #include "wx/utils.h"

-#include "wx/dialog.h"

 #include "wx/filedlg.h"
 #include "wx/intl.h"
 #include "wx/filedlg.h"
 #include "wx/intl.h"

+#include "wx/app.h"
+#include "wx/settings.h"
+#include "wx/tokenzr.h"
+#include "wx/stockitem.h"

 
 

+#ifdef __VMS__
+#pragma message disable nosimpint
+#endif

 #include <Xm/Xm.h>
 #include <Xm/Xm.h>

-
-#if !USE_SHARED_LIBRARY
-IMPLEMENT_CLASS(wxFileDialog, wxDialog)
+#include <Xm/MwmUtil.h>
+#include <Xm/Label.h>
+#include <Xm/BulletinB.h>
+#include <Xm/Frame.h>
+#include <Xm/Text.h>
+#include <Xm/DialogS.h>
+#include <Xm/FileSB.h>
+#include <Xm/RowColumn.h>
+#include <Xm/LabelG.h>
+#ifdef __VMS__
+#pragma message enable nosimpint

 #endif
 
 #endif
 

-char *wxFileSelector(const char *title,
-                     const char *defaultDir, const char *defaultFileName,
-                     const char *defaultExtension, const char *filter, int flags,
-                     wxWindow *parent, int x, int y)
-{
-    // If there's a default extension specified but no filter, we create a suitable
-    // filter.
-
-    wxString filter2("");
-    if ( defaultExtension && !filter )
-        filter2 = wxString("*.") + wxString(defaultExtension) ;
-    else if ( filter )
-        filter2 = filter;
-
-    wxString defaultDirString;
-    if (defaultDir)
-        defaultDirString = defaultDir;
-    else
-        defaultDirString = "";
+#include "wx/motif/private.h"

 
 

-    wxString defaultFilenameString;
-    if (defaultFileName)
-        defaultFilenameString = defaultFileName;
-    else
-        defaultFilenameString = "";
+IMPLEMENT_CLASS(wxFileDialog, wxFileDialogBase)

 
 

-    wxFileDialog fileDialog(parent, title, defaultDirString, defaultFilenameString, filter2, flags, wxPoint(x, y));
+#define DEFAULT_FILE_SELECTOR_SIZE 0
+// Let Motif defines the size of File
+// Selector Box (if 1), or fix it to
+// wxFSB_WIDTH x wxFSB_HEIGHT (if 0)
+#define wxFSB_WIDTH                600
+#define wxFSB_HEIGHT               500

 
 

-    if ( fileDialog.ShowModal() == wxID_OK )
-    {
-        strcpy(wxBuffer, (const char *)fileDialog.GetPath());
-        return wxBuffer;
-    }
-    else
-        return NULL;
-}

 
 

-char *wxFileSelectorEx(const char *title,
-                       const char *defaultDir,
-                       const char *defaultFileName,
-                       int* defaultFilterIndex,
-                       const char *filter,
-                       int       flags,
-                       wxWindow* parent,
-                       int       x,
-                       int       y)
+wxString wxFileDialog::m_fileSelectorAnswer = wxEmptyString;
+bool wxFileDialog::m_fileSelectorReturned = false;

 
 

+static void wxFileSelClose(Widget WXUNUSED(w),
+                           void* WXUNUSED(client_data),
+                           XmAnyCallbackStruct *WXUNUSED(call_data))

 {
 {

-    wxFileDialog fileDialog(parent, title ? title : "", defaultDir ? defaultDir : "",
-        defaultFileName ? defaultFileName : "", filter ? filter : "", flags, wxPoint(x, y));
-
-    if ( fileDialog.ShowModal() == wxID_OK )
-    {
-        *defaultFilterIndex = fileDialog.GetFilterIndex();
-        strcpy(wxBuffer, (const char *)fileDialog.GetPath());
-        return wxBuffer;
-    }
-    else
-        return NULL;
+    wxFileDialog::m_fileSelectorAnswer = wxEmptyString;
+    wxFileDialog::m_fileSelectorReturned = true;

 }
 
 }
 

-// TODO: Motif file selector code
-#if 0
-char *wxFileSelectorAnswer = NULL;
-Bool wxFileSelectorReturned = FALSE;
-
-void wxFileSelCancel(Widget fs, XtPointer client_data, XmFileSelectionBoxCallbackStruct *cbs)
+void wxFileSelCancel( Widget WXUNUSED(fs), XtPointer WXUNUSED(client_data),
+                     XmFileSelectionBoxCallbackStruct *WXUNUSED(cbs) )

 {
 {

-  wxFileSelectorAnswer = NULL;
-  wxFileSelectorReturned = TRUE;
+    wxFileDialog::m_fileSelectorAnswer = wxEmptyString;
+    wxFileDialog::m_fileSelectorReturned = true;

 }
 
 }
 

-void wxFileSelOk(Widget fs, XtPointer client_data, XmFileSelectionBoxCallbackStruct *cbs)
+void wxFileSelOk(Widget WXUNUSED(fs), XtPointer WXUNUSED(client_data), XmFileSelectionBoxCallbackStruct *cbs)

 {
 {

-  char *filename = NULL;
-  if (!XmStringGetLtoR(cbs->value, XmSTRING_DEFAULT_CHARSET, &filename)) {
-    wxFileSelectorAnswer = NULL;
-    wxFileSelectorReturned = TRUE;
-  } else {
-    if (filename) {
-      if (wxFileSelectorAnswer) delete[] wxFileSelectorAnswer;
-      wxFileSelectorAnswer = copystring(filename);
-      XtFree(filename);
+    char *filename = NULL;
+    if (!XmStringGetLtoR(cbs->value, XmSTRING_DEFAULT_CHARSET, &filename)) {
+        wxFileDialog::m_fileSelectorAnswer = wxEmptyString;
+        wxFileDialog::m_fileSelectorReturned = true;
+    } else {
+        if (filename) {
+            wxFileDialog::m_fileSelectorAnswer = filename;
+            XtFree(filename);
+        }
+        wxFileDialog::m_fileSelectorReturned = true;

     }
     }

-    wxFileSelectorReturned = TRUE;
-  }

 }
 
 }
 

-
-char *wxMotifFileSelector(char *message,
-                     char *default_path, char *default_filename, 
-                     char *default_extension, char *wildcard, int flags,
-                     wxWindow *parent, int x, int y)
+static wxString ParseWildCard( const wxString& wild )

 {
 {

-  wxBeginBusyCursor();
-  static char fileBuf[512];
-  Widget parentWidget = 0;
-  if (parent)
-  {
-    if (parent->IsKindOf(CLASSINFO(wxFrame)))
-      parentWidget = ((wxFrame *)parent)->frameShell;
-    else if (parent->IsKindOf(CLASSINFO(wxDialogBox)))
-      parentWidget = ((wxDialogBox *)parent)->dialogShell;
-    else
-      parentWidget = (Widget)parent->handle;
-  }
-  else if (wxTheApp->wx_frame)
-    parentWidget = wxTheApp->wx_frame->frameShell;
-
-  Widget fileSel = XmCreateFileSelectionDialog(parentWidget, "file_selector", NULL, 0);
-  XtUnmanageChild(XmFileSelectionBoxGetChild(fileSel, XmDIALOG_HELP_BUTTON));
-
-  Widget shell = XtParent(fileSel);
-
-  if (message)
-    XtVaSetValues(shell, XmNtitle, message, NULL);
-
-  char *entirePath = NULL;
-
-  if (default_path && default_filename)
-  {
-    sprintf(wxBuffer, "%s/%s", default_path, default_filename);
-    entirePath = copystring(wxBuffer);
-  }
-  else if (default_path && !default_filename)
-  {
-    sprintf(wxBuffer, "%s/", default_path);
-    entirePath = copystring(wxBuffer);
-  }
-  else if ((!default_path) && default_filename)
-  {
-    sprintf(wxBuffer, "%s", default_filename);
-    entirePath = copystring(wxBuffer);
-  }
-
-  if (entirePath)
-  {
-    Widget selectionWidget = XmFileSelectionBoxGetChild(fileSel, XmDIALOG_TEXT);
-    XmTextSetString(selectionWidget, entirePath);
-    delete[] entirePath;
-  }
-
-  if (wildcard)
-  {
-    if (default_path)
-      sprintf(wxBuffer, "%s/%s", default_path, wildcard);
-    else
-      sprintf(wxBuffer, "%s", wildcard);
-
-    Widget filterWidget = XmFileSelectionBoxGetChild(fileSel, XmDIALOG_FILTER_TEXT);
-    XmTextSetString(filterWidget, wxBuffer);
-    XmFileSelectionDoSearch(fileSel, NULL);
-  }
-
-  // Suggested by Terry Gitnick, 16/9/97, because of change in Motif
-  // file selector on Solaris 1.5.1.
-  if ( default_path )
-  {
-      XmString thePath = XmStringCreateLtoR (default_path,
-                                           XmSTRING_DEFAULT_CHARSET);
- 
-      XtVaSetValues (fileSel,
-                   XmNdirectory, thePath,
-                   NULL);
-
-      XmStringFree(thePath);
-  }
-
-  XtAddCallback(fileSel, XmNcancelCallback, (XtCallbackProc)wxFileSelCancel, (XtPointer)NULL);
-  XtAddCallback(fileSel, XmNokCallback, (XtCallbackProc)wxFileSelOk, (XtPointer)NULL);
-
-//#if XmVersion > 1000
-// I'm not sure about what you mean with XmVersion.
-// If this is for Motif1.1/Motif1.2, then check XmVersion>=1200
-// (Motif1.1.4 ==> XmVersion 1100 )
-// Nevertheless, I put here a #define, so anyone can choose in (I)makefile...
-//
-#if !DEFAULT_FILE_SELECTOR_SIZE
-  int width = wxFSB_WIDTH;
-  int height = wxFSB_HEIGHT;
-  XtVaSetValues(fileSel,
-                 XmNwidth, width,
-                 XmNheight, height,
-                 XmNresizePolicy, XmRESIZE_NONE,
-                 NULL);
+#ifdef __WXDEBUG__
+    static const wxChar* msg =
+        _T("Motif file dialog does not understand this ")
+        _T("wildcard syntax");

 #endif
 
 #endif
 

-  XtManageChild(fileSel);
-
-  if (wxFileSelectorAnswer)
-    delete[] wxFileSelectorAnswer;
-
-  wxFileSelectorAnswer = NULL;
-  wxFileSelectorReturned = FALSE;
-
-  wxEndBusyCursor();
-
-  XtAddGrab(XtParent(fileSel), TRUE, FALSE);
-  XEvent event;
-  while (!wxFileSelectorReturned)
-  {
-    XtAppNextEvent(wxTheApp->appContext, &event);
-    XtDispatchEvent(&event);
-  }
-  XtRemoveGrab(XtParent(fileSel));
-
-  XmUpdateDisplay(wxTheApp->topLevel); // Experimental
-
-//  XtDestroyWidget(fileSel);
-  XtUnmapWidget(XtParent(fileSel));
-  XtDestroyWidget(XtParent(fileSel));
-
-  // Now process all events, because otherwise
-  // this might remain on the screen
-  XSync(XtDisplay(wxTheApp->topLevel), FALSE);
-  while (XtAppPending(wxTheApp->appContext))
-  {
-    XFlush(XtDisplay(wxTheApp->topLevel));
-    XtAppNextEvent(wxTheApp->appContext, &event);
-    XtDispatchEvent(&event);
-  }
-
-  if (wxFileSelectorAnswer)
-  {
-    strcpy(fileBuf, wxFileSelectorAnswer);
-    return fileBuf;
-  }
-  else return NULL;
+    wxArrayString wildDescriptions, wildFilters;
+    const size_t count = wxParseCommonDialogsFilter(wild,
+                                                    wildDescriptions,
+                                                    wildFilters);
+    wxCHECK_MSG( count, _T("*.*"), wxT("wxFileDialog: bad wildcard string") );
+    wxCHECK_MSG( count == 1, _T("*.*"), msg );
+
+    // check for *.txt;*.rtf
+    wxStringTokenizer tok2( wildFilters[0], _T(";") );
+    wxString wildcard = tok2.GetNextToken();
+
+    wxCHECK_MSG( tok2.CountTokens() <= 1, wildcard, msg );
+    return wildcard;

 }
 }

-#endif

 
 wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message,
 
 wxFileDialog::wxFileDialog(wxWindow *parent, const wxString& message,

-        const wxString& defaultDir, const wxString& defaultFileName, const wxString& wildCard,
-        long style, const wxPoint& pos)
+                           const wxString& defaultDir, const wxString& defaultFileName, const wxString& wildCard,
+                           long style, const wxPoint& pos)
+             :wxFileDialogBase(parent, message, defaultDir, defaultFileName, wildCard, style, pos)

 {
 {

-    m_message = message;
-    m_dialogStyle = style;
-    m_parent = parent;
-    m_path = "";
-    m_fileName = defaultFileName;
-    m_dir = defaultDir;
-    m_wildCard = wildCard;

     m_filterIndex = 1;
 }
 
     m_filterIndex = 1;
 }
 

-int wxFileDialog::ShowModal()
+static void wxChangeListBoxColours(wxWindow* WXUNUSED(win), Widget widget)

 {
 {

-    // TODO
-    return wxID_CANCEL;
-}
+    wxDoChangeBackgroundColour((WXWidget) widget, *wxWHITE);

 
 

-// Generic file load/save dialog
-static char *
-wxDefaultFileSelector(bool load, const char *what, const char *extension, const char *default_name, wxWindow *parent)
-{
-  char *ext = (char *)extension;
-  
-  char prompt[50];
-  wxString str;
-  if (load)
-    str = "Load %s file";
-  else
-    str = "Save %s file";
-  sprintf(prompt, wxGetTranslation(str), what);
-
-  if (*ext == '.') ext++;
-  char wild[60];
-  sprintf(wild, "*.%s", ext);
-
-  return wxFileSelector (prompt, NULL, default_name, ext, wild, 0, parent);
+    // Change colour of the scrolled areas of the listboxes
+    Widget listParent = XtParent (widget);
+#if 0
+    wxDoChangeBackgroundColour((WXWidget) listParent, *wxWHITE, true);
+#endif
+
+    Widget hsb = (Widget) 0;
+    Widget vsb = (Widget) 0;
+    XtVaGetValues (listParent,
+        XmNhorizontalScrollBar, &hsb,
+        XmNverticalScrollBar, &vsb,
+        NULL);
+
+   /* TODO: should scrollbars be affected? Should probably have separate
+    * function to change them (by default, taken from wxSystemSettings)
+    */
+    wxColour backgroundColour = wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE);
+    wxDoChangeBackgroundColour((WXWidget) hsb, backgroundColour, true);
+    wxDoChangeBackgroundColour((WXWidget) vsb, backgroundColour, true);
+
+    if (hsb)
+      XtVaSetValues (hsb,
+        XmNtroughColor, backgroundColour.AllocColour(XtDisplay(hsb)),
+        NULL);
+    if (vsb)
+      XtVaSetValues (vsb,
+        XmNtroughColor, backgroundColour.AllocColour(XtDisplay(vsb)),
+        NULL);

 }
 
 }
 

-// Generic file load dialog
-char *
-wxLoadFileSelector(const char *what, const char *extension, const char *default_name, wxWindow *parent)
+int wxFileDialog::ShowModal()

 {
 {

-  return wxDefaultFileSelector(TRUE, what, extension, default_name, parent);
-}
+    wxBeginBusyCursor();

 
 

+    //  static char fileBuf[512];
+    Widget parentWidget = (Widget) 0;
+    if (m_parent)
+        parentWidget = (Widget) m_parent->GetTopWidget();
+    else
+        parentWidget = (Widget) wxTheApp->GetTopLevelWidget();
+    // prepare the arg list
+    Display* dpy = XtDisplay(parentWidget);
+    Arg args[10];
+    int ac = 0;
+
+    wxComputeColours (dpy, & m_backgroundColour, (wxColour*) NULL);
+
+    XtSetArg(args[ac], XmNbackground, g_itemColors[wxBACK_INDEX].pixel); ac++;
+    XtSetArg(args[ac], XmNtopShadowColor, g_itemColors[wxTOPS_INDEX].pixel); ac++;
+    XtSetArg(args[ac], XmNbottomShadowColor, g_itemColors[wxBOTS_INDEX].pixel); ac++;
+    XtSetArg(args[ac], XmNforeground, g_itemColors[wxFORE_INDEX].pixel); ac++;
+
+    wxFont font = wxSystemSettings::GetFont(wxSYS_DEFAULT_GUI_FONT);
+
+#if __WXMOTIF20__ && !__WXLESSTIF__
+    XtSetArg(args[ac], XmNbuttonRenderTable, font.GetFontTypeC(dpy)); ac++;
+    XtSetArg(args[ac], XmNlabelRenderTable, font.GetFontTypeC(dpy)); ac++;
+    XtSetArg(args[ac], XmNtextRenderTable, font.GetFontTypeC(dpy)); ac++;
+#else
+    XtSetArg(args[ac], XmNbuttonFontList, font.GetFontTypeC(dpy)); ac++;
+    XtSetArg(args[ac], XmNlabelFontList, font.GetFontTypeC(dpy)); ac++;
+    XtSetArg(args[ac], XmNtextFontList, font.GetFontTypeC(dpy)); ac++;
+#endif

 
 

-// Generic file save dialog
-char *
-wxSaveFileSelector(const char *what, const char *extension, const char *default_name, wxWindow *parent)
-{
-  return wxDefaultFileSelector(FALSE, what, extension, default_name, parent);
-}
+    Widget fileSel = XmCreateFileSelectionDialog(parentWidget,
+                                                 wxMOTIF_STR("file_selector"),
+                                                 args, ac);
+#define wxFSChild( name ) \
+    XmFileSelectionBoxGetChild(fileSel, name)
+
+    XtUnmanageChild(wxFSChild(XmDIALOG_HELP_BUTTON));
+
+    Widget filterWidget = wxFSChild(XmDIALOG_FILTER_TEXT);
+    Widget selectionWidget = wxFSChild(XmDIALOG_TEXT);
+    Widget dirListWidget = wxFSChild(XmDIALOG_DIR_LIST);
+    Widget fileListWidget = wxFSChild(XmDIALOG_LIST);
+
+    // for changing labels
+    Widget okWidget = wxFSChild(XmDIALOG_OK_BUTTON);
+    Widget applyWidget = wxFSChild(XmDIALOG_APPLY_BUTTON);
+    Widget cancelWidget = wxFSChild(XmDIALOG_CANCEL_BUTTON);
+    Widget dirlistLabel = wxFSChild(XmDIALOG_DIR_LIST_LABEL);
+    Widget filterLabel = wxFSChild(XmDIALOG_FILTER_LABEL);
+    Widget listLabel = wxFSChild(XmDIALOG_LIST_LABEL);
+    Widget selectionLabel = wxFSChild(XmDIALOG_SELECTION_LABEL);
+
+#undef wxFSChild
+
+    // change labels
+    wxXmString btnOK( wxGetStockLabel( wxID_OK, false ) ),
+               btnCancel( wxGetStockLabel( wxID_CANCEL, false ) ),
+               btnFilter( _("Filter") ), lblFilter( _("Filter") ),
+               lblDirectories( _("Directories") ),
+               lblFiles( _("Files" ) ), lblSelection( _("Selection") );
+
+    XtVaSetValues( okWidget, XmNlabelString, btnOK(), NULL );
+    XtVaSetValues( applyWidget, XmNlabelString, btnFilter(), NULL );
+    XtVaSetValues( cancelWidget, XmNlabelString, btnCancel(), NULL );
+    XtVaSetValues( dirlistLabel, XmNlabelString, lblDirectories(), NULL );
+    XtVaSetValues( filterLabel, XmNlabelString, lblFilter(), NULL );
+    XtVaSetValues( listLabel, XmNlabelString, lblFiles(), NULL );
+    XtVaSetValues( selectionLabel, XmNlabelString, lblSelection(), NULL );
+
+    Widget shell = XtParent(fileSel);
+
+    if (!m_message.IsNull())
+        XtVaSetValues(shell,
+                      XmNtitle, wxConstCast(m_message.c_str(), char),
+                      NULL);
+
+    if (!m_wildCard.empty())
+    {
+        // return something understandable by Motif
+        wxString wildCard = ParseWildCard( m_wildCard );
+        wxString filter;
+        if (!m_dir.empty())
+            filter = m_dir + wxString("/") + wildCard;
+        else
+            filter = wildCard;
+
+        XmTextSetString(filterWidget, wxConstCast(filter.c_str(), char));
+        XmFileSelectionDoSearch(fileSel, NULL);
+    }
+
+    // Suggested by Terry Gitnick, 16/9/97, because of change in Motif
+    // file selector on Solaris 1.5.1.
+    if ( !m_dir.empty() )
+    {
+        wxXmString thePath( m_dir );
+
+        XtVaSetValues (fileSel,
+            XmNdirectory, thePath(),
+            NULL);
+    }
+
+    wxString entirePath;
+
+    if (!m_dir.empty())
+    {
+        entirePath = m_dir + wxString("/") + m_fileName;
+    }
+    else
+    {
+        entirePath = m_fileName;
+    }
+
+    if (!entirePath.empty())
+    {
+        XmTextSetString(selectionWidget,
+                        wxConstCast(entirePath.c_str(), char));
+    }
+
+    XtAddCallback(fileSel, XmNcancelCallback,
+                  (XtCallbackProc)wxFileSelCancel, (XtPointer)NULL);
+    XtAddCallback(fileSel, XmNokCallback,
+                  (XtCallbackProc)wxFileSelOk, (XtPointer)NULL);
+    XtAddCallback(fileSel, XmNunmapCallback,
+                  (XtCallbackProc)wxFileSelClose, (XtPointer)this);
+
+    //#if XmVersion > 1000
+    // I'm not sure about what you mean with XmVersion.
+    // If this is for Motif1.1/Motif1.2, then check XmVersion>=1200
+    // (Motif1.1.4 ==> XmVersion 1100 )
+    // Nevertheless, I put here a #define, so anyone can choose in (I)makefile.
+    //
+#if !DEFAULT_FILE_SELECTOR_SIZE
+    int width = wxFSB_WIDTH;
+    int height = wxFSB_HEIGHT;
+    XtVaSetValues(fileSel,
+        XmNwidth, width,
+        XmNheight, height,
+        XmNresizePolicy, XmRESIZE_NONE,
+        NULL);
+#endif
+    wxDoChangeBackgroundColour((WXWidget) filterWidget, *wxWHITE);
+    wxDoChangeBackgroundColour((WXWidget) selectionWidget, *wxWHITE);
+
+    wxChangeListBoxColours(this, dirListWidget);
+    wxChangeListBoxColours(this, fileListWidget);
+
+    XtManageChild(fileSel);
+
+    m_fileSelectorAnswer = wxEmptyString;
+    m_fileSelectorReturned = false;

 
 

+    wxEndBusyCursor();

 
 

+    XtAddGrab(XtParent(fileSel), True, False);
+    XtAppContext context = (XtAppContext) wxTheApp->GetAppContext();
+    XEvent event;
+    while (!m_fileSelectorReturned)
+    {
+        XtAppNextEvent(context, &event);
+        XtDispatchEvent(&event);
+    }
+    XtRemoveGrab(XtParent(fileSel));
+
+    XtUnmapWidget(XtParent(fileSel));
+    XtDestroyWidget(XtParent(fileSel));
+
+    // Now process all events, because otherwise
+    // this might remain on the screen
+    wxFlushEvents(XtDisplay(fileSel));
+
+    m_path = m_fileSelectorAnswer;
+    m_fileName = wxFileNameFromPath(m_fileSelectorAnswer);
+    m_dir = wxPathOnly(m_path);
+
+    if (m_fileName.empty())
+        return wxID_CANCEL;
+    else
+        return wxID_OK;
+}