* fix spelling in comments to please John.

[lyx.git] / src / frontends / qt4 / FileDialog.cpp

diff --git a/src/frontends/qt4/FileDialog.cpp b/src/frontends/qt4/FileDialog.cpp
index dc4e10ad307a1a435215802700004feb1f929639..51e49917cfa136191b709c7eedac2ebbb212d21a 100644 (file)
--- a/src/frontends/qt4/FileDialog.cpp
+++ b/src/frontends/qt4/FileDialog.cpp
@@ -18,9 +18,12 @@
 
 #include "support/debug.h"
 #include "support/FileName.h"
+#include "support/filetools.h"
 #include "support/gettext.h"
 #include "support/os.h"
 
+#include <string>
+
 /** when this is defined, the code will use
  * QFileDialog::getOpenFileName and friends to create filedialogs.
  * Effects:
@@ -28,11 +31,11 @@
  *   parameters);
  * - with Qt/Mac or Qt/Win, the dialogs native to the environment are used.
  * - with Qt/Win and Qt <= 4.3.0, there was a number of bugs with our own
- *   file dialog (http://bugzilla.lyx.org/show_bug.cgi?id=3907).
+ *   file dialog (http://www.lyx.org/trac/ticket/3907).
  *
  * Therefore there is a tradeoff in enabling or disabling this (JMarc)
  */
-#if defined(Q_WS_MACX) || (defined(Q_WS_WIN) && !defined(Q_CYGWIN_WIN))
+#if defined(Q_WS_MACX) || (defined(Q_WS_WIN) && !defined(Q_OS_CYGWIN))
 #define USE_NATIVE_FILEDIALOG 1
 #endif
 
@@ -42,7 +45,7 @@
 
 namespace lyx {
 
-using support::os::internal_path;
+using namespace support;
 
 
 class FileDialog::Private {
@@ -91,8 +94,12 @@ FileDialog::Result FileDialog::save(QString const & path,
        QString const startsWith = makeAbsPath(suggested, path);
        QString const name = 
                QFileDialog::getSaveFileName(qApp->focusWidget(),
-            title_, startsWith, filters, 0, QFileDialog::DontConfirmOverwrite);
-       result.second = toqstr(internal_path(fromqstr(name)));
+            title_, startsWith, filters.join(";;"),
+                        0, QFileDialog::DontConfirmOverwrite);
+       if (name.isNull())
+               result.first = FileDialog::Later;
+       else
+               result.second = toqstr(os::internal_path(fromqstr(name)));
 #else
        LyXFileDialog dlg(title_, path, filters, private_->b1, private_->b2);
 #if QT_VERSION != 0x040203
@@ -109,6 +116,8 @@ FileDialog::Result FileDialog::save(QString const & path,
        LYXERR(Debug::GUI, "result " << res);
        if (res == QDialog::Accepted)
                result.second = internalPath(dlg.selectedFiles()[0]);
+       else
+               result.first = FileDialog::Later;
        dlg.hide();
 #endif
        return result;
@@ -126,9 +135,12 @@ FileDialog::Result FileDialog::open(QString const & path,
 
 #ifdef USE_NATIVE_FILEDIALOG
        QString const startsWith = makeAbsPath(suggested, path);
-       result.second = internalPath(
-               QFileDialog::getOpenFileName(qApp->focusWidget(),
-               title_, startsWith, filters));
+       QString const file = QFileDialog::getOpenFileName(qApp->focusWidget(),
+               title_, startsWith, filters.join(";;"));
+       if (file.isNull())
+               result.first = FileDialog::Later;
+       else
+               result.second = internalPath(file);
 #else
        LyXFileDialog dlg(title_, path, filters, private_->b1, private_->b2);
 
@@ -140,6 +152,8 @@ FileDialog::Result FileDialog::open(QString const & path,
        LYXERR(Debug::GUI, "result " << res);
        if (res == QDialog::Accepted)
                result.second = internalPath(dlg.selectedFiles()[0]);
+       else
+               result.first = FileDialog::Later;
        dlg.hide();
 #endif
        return result;
@@ -155,10 +169,14 @@ FileDialog::Result FileDialog::opendir(QString const & path,
        result.first = FileDialog::Chosen;
 
 #ifdef USE_NATIVE_FILEDIALOG
-       QString const startsWith = makeAbsPath(suggested, path);
-       result.second = toqstr(internal_path(fromqstr(
-               QFileDialog::getExistingDirectory(qApp->focusWidget(),
-               title_, startsWith))));
+       QString const startsWith = toqstr(makeAbsPath(fromqstr(suggested),
+               fromqstr(path)).absFilename());
+       QString const dir = QFileDialog::getExistingDirectory(qApp->focusWidget(),
+               title_, startsWith);
+       if (dir.isNull())
+               result.first = FileDialog::Later;
+       else
+               result.second = toqstr(os::internal_path(fromqstr(dir)));
 #else
        LyXFileDialog dlg(title_, path, QStringList(qt_("Directories")),
                private_->b1, private_->b2);
@@ -173,6 +191,8 @@ FileDialog::Result FileDialog::opendir(QString const & path,
        LYXERR(Debug::GUI, "result " << res);
        if (res == QDialog::Accepted)
                result.second = internalPath(dlg.selectedFiles()[0]);
+       else
+               result.first = FileDialog::Later;
        dlg.hide();
 #endif
        return result;