]> git.lyx.org Git - lyx.git/blobdiff - src/frontends/qt4/GuiApplication.cpp
Fix the tab ordering of GuiDocument components.
[lyx.git] / src / frontends / qt4 / GuiApplication.cpp
index 15d9f58c16bcf997b36b8efea3d773b8b4cdc42d..e855119323ccab1ec9e84faa4f212a7e2334d23d 100644 (file)
@@ -139,7 +139,7 @@ static void initializeResources()
 {
        static bool initialized = false;
        if (!initialized) {
-               Q_INIT_RESOURCE(Resources); 
+               Q_INIT_RESOURCE(Resources);
                initialized = true;
        }
 }
@@ -367,18 +367,19 @@ QString iconName(FuncRequest const & f, bool unknown)
                }
        }
 
-       FileName fname = libFileSearch("images/" + path, name1, "png");
+       QString imagedir = "images/" + path;
+       FileName fname = imageLibFileSearch(imagedir, name1, "png");
        if (fname.exists())
                return toqstr(fname.absFileName());
 
-       fname = libFileSearch("images/" + path, name2, "png");
+       fname = imageLibFileSearch(imagedir, name2, "png");
        if (fname.exists())
                return toqstr(fname.absFileName());
 
        path = ":/images/" + path;
        QDir res(path);
        if (!res.exists()) {
-               LYXERR0("Directory " << path << " not found in resource!"); 
+               LYXERR0("Directory " << path << " not found in resource!");
                return QString();
        }
        name1 += ".png";
@@ -392,13 +393,14 @@ QString iconName(FuncRequest const & f, bool unknown)
        LYXERR(Debug::GUI, "Cannot find icon with filename "
                           << "\"" << name1 << "\""
                           << " or filename "
-                          << "\"" << name2 << "\"" 
+                          << "\"" << name2 << "\""
                           << " for command \""
                           << lyxaction.getActionName(f.action())
                           << '(' << to_utf8(f.argument()) << ")\"");
 
        if (unknown) {
-               fname = libFileSearch(QString("images/"), "unknown", "png");
+               imagedir = "images/";
+               fname = imageLibFileSearch(imagedir, "unknown", "png");
                if (fname.exists())
                        return toqstr(fname.absFileName());
                return QString(":/images/unknown.png");
@@ -410,7 +412,8 @@ QString iconName(FuncRequest const & f, bool unknown)
 QPixmap getPixmap(QString const & path, QString const & name, QString const & ext)
 {
        QPixmap pixmap;
-       FileName fname = libFileSearch(path, name, ext);
+       QString imagedir = path;
+       FileName fname = imageLibFileSearch(imagedir, name, ext);
        QString path1 = toqstr(fname.absFileName());
        QString path2 = ":/" + path + name + "." + ext;
 
@@ -478,15 +481,15 @@ public:
                : QTranslator(parent)
        {}
 
-       QString translate(const char * /*context*/, 
-         const char * sourceText, 
-         const char * /*comment*/ = 0) 
+       QString translate(const char * /*context*/,
+         const char * sourceText,
+         const char * /*comment*/ = 0)
        {
                string const s = sourceText;
-               if (s == N_("About %1") || s == N_("Preferences") 
+               if (s == N_("About %1") || s == N_("Preferences")
                                || s == N_("Reconfigure") || s == N_("Quit %1"))
                        return qt_(s);
-               else 
+               else
                        return QString();
        }
 };
@@ -496,7 +499,7 @@ class GlobalMenuBar : public QMenuBar
 public:
        ///
        GlobalMenuBar() : QMenuBar(0) {}
-       
+
        ///
        bool event(QEvent * e)
        {
@@ -662,7 +665,7 @@ public:
        {
                switch (formatetc.cfFormat) {
                case CF_ENHMETAFILE:
-                       return emfMimeType(); 
+                       return emfMimeType();
                case CF_METAFILEPICT:
                        return wmfMimeType();
                }
@@ -791,7 +794,7 @@ GuiApplication::GuiApplication(int & argc, char ** argv)
        /// Only needed with Qt/Mac.
        installTranslator(new MenuTranslator(this));
 #endif
-       
+
 #ifdef Q_WS_X11
        // doubleClickInterval() is 400 ms on X11 which is just too long.
        // On Windows and Mac OS X, the operating system's value is used.
@@ -849,7 +852,7 @@ void GuiApplication::clearSession()
 }
 
 
-docstring GuiApplication::iconName(FuncRequest const & f, bool unknown)
+docstring Application::iconName(FuncRequest const & f, bool unknown)
 {
        return qstring_to_ucs4(lyx::frontend::iconName(f, unknown));
 }
@@ -861,7 +864,7 @@ FuncStatus GuiApplication::getStatus(FuncRequest const & cmd) const
 
        BufferView * bv = 0;
        BufferView * doc_bv = 0;
-       
+
        if (cmd.action() == LFUN_NOACTION) {
                status.message(from_utf8(N_("Nothing to do")));
                status.setEnabled(false);
@@ -879,7 +882,7 @@ FuncStatus GuiApplication::getStatus(FuncRequest const & cmd) const
        // If we do not have a GuiView, then other functions are disabled
        else if (!current_view_)
                status.setEnabled(false);
-               
+
        // Does the GuiView know something?
        else if (current_view_->getStatus(cmd, status)) { }
 
@@ -896,10 +899,10 @@ FuncStatus GuiApplication::getStatus(FuncRequest const & cmd) const
        // If we do not have a BufferView, then other functions are disabled
        else if (!(bv = current_view_->currentBufferView()))
                status.setEnabled(false);
-       
+
        // Does the current BufferView know something?
        else if (bv->getStatus(cmd, status)) { }
-               
+
        // Does the current Buffer know something?
        else if (bv->buffer().getStatus(cmd, status)) { }
 
@@ -911,12 +914,12 @@ FuncStatus GuiApplication::getStatus(FuncRequest const & cmd) const
        // Does the document Buffer know something?
        else if (doc_bv->buffer().getStatus(cmd, status)) { }
 
-       else { 
+       else {
                LYXERR(Debug::ACTION, "LFUN not handled in getStatus(): " << cmd);
                status.message(from_utf8(N_("Command not handled")));
                status.setEnabled(false);
        }
-       
+
        // the default error message if we disable the command
        if (!status.enabled() && status.message().empty())
                status.message(from_utf8(N_("Command disabled")));
@@ -1152,10 +1155,13 @@ void GuiApplication::gotoBookmark(unsigned int idx, bool openFile,
                dispatch(FuncRequest(LFUN_BOOKMARK_SAVE, "0"));
 
        // if the current buffer is not that one, switch to it.
-       BufferView * doc_bv = current_view_->documentBufferView();
+       BufferView * doc_bv = current_view_ ?
+               current_view_->documentBufferView() : 0;
        if (!doc_bv || doc_bv->buffer().fileName() != tmp.filename) {
                if (switchToBuffer) {
                        dispatch(FuncRequest(LFUN_BUFFER_SWITCH, file));
+                       if (!current_view_)
+                               return;
                        doc_bv = current_view_->documentBufferView();
                } else
                        return;
@@ -1203,7 +1209,7 @@ void GuiApplication::reconfigure(string const & option)
        // emit message signal.
        if (current_view_)
                current_view_->message(_("Reloading configuration..."));
-       lyxrc.read(libFileSearch(QString(), "lyxrc.defaults"));
+       lyxrc.read(libFileSearch(QString(), "lyxrc.defaults"), false);
        // Re-read packages.lst
        LaTeXFeatures::getAvailable();
 
@@ -1220,6 +1226,23 @@ void GuiApplication::reconfigure(string const & option)
                             "updated document class specifications."));
 }
 
+void GuiApplication::validateCurrentView()
+{
+       if (!d->views_.empty() && !current_view_) {
+               // currently at least one view exists but no view has the focus.
+               // choose the last view to make it current.
+               // a view without any open document is preferred.
+               GuiView * candidate = 0;
+               QHash<int, GuiView *>::const_iterator it = d->views_.begin();
+               QHash<int, GuiView *>::const_iterator end = d->views_.end();
+               for (; it != end; ++it) {
+                       candidate = *it;
+                       if (!candidate->documentBufferView())
+                               break;
+               }
+               setCurrentView(candidate);
+       }
+}
 
 void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
 {
@@ -1237,8 +1260,6 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                LYXERR(Debug::ACTION, "action "
                       << lyxaction.getActionName(action)
                       << " [" << action << "] is disabled at this location");
-               if (current_view_)
-                       current_view_->restartCursor();
                dr.setMessage(flag.message());
                dr.setError(true);
                dr.dispatched(false);
@@ -1263,7 +1284,10 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                // clear the last opened list, because
                // maybe this will end the session
                theSession().lastOpened().clear();
-               current_view_->closeScheduled();
+               // check for valid current_view_
+               validateCurrentView();
+               if (current_view_)
+                       current_view_->closeScheduled();
                break;
 
        case LFUN_LYX_QUIT:
@@ -1290,6 +1314,7 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
        }
 
        case LFUN_BUFFER_NEW:
+               validateCurrentView();
                if (d->views_.empty()
                   || (!lyxrc.open_buffers_in_tabs && current_view_->documentBufferView() != 0)) {
                        createView(QString(), false); // keep hidden
@@ -1302,6 +1327,7 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                break;
 
        case LFUN_BUFFER_NEW_TEMPLATE:
+               validateCurrentView();
                if (d->views_.empty()
                   || (!lyxrc.open_buffers_in_tabs && current_view_->documentBufferView() != 0)) {
                        createView();
@@ -1314,9 +1340,10 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                break;
 
        case LFUN_FILE_OPEN: {
+               validateCurrentView();
                // FIXME: create a new method shared with LFUN_HELP_OPEN.
                string const fname = to_utf8(cmd.argument());
-               if (d->views_.empty() || (!lyxrc.open_buffers_in_tabs 
+               if (d->views_.empty() || (!lyxrc.open_buffers_in_tabs
                          && current_view_->documentBufferView() != 0)) {
                        // We want the ui session to be saved per document and not per
                        // window number. The filename crc is a good enough identifier.
@@ -1352,9 +1379,12 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                current_view_->message(bformat(_("Opening help file %1$s..."),
                                               makeDisplayPath(fname.absFileName())));
                Buffer * buf = current_view_->loadDocument(fname, false);
+
 #ifndef DEVEL_VERSION
                if (buf)
                        buf->setReadonly(true);
+#else
+               (void) buf;
 #endif
                break;
        }
@@ -1369,19 +1399,19 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                        break;
                }
 
+#if 0
+               // FIXME: The graphics cache no longer has a changeDisplay method.
                string const graphicsbg = lcolor.getLyXName(Color_graphicsbg);
-               bool const graphicsbg_changed = 
+               bool const graphicsbg_changed =
                                lyx_name == graphicsbg && x11_name != graphicsbg;
-               if (graphicsbg_changed) {
-                       // FIXME: The graphics cache no longer has a changeDisplay method.
-#if 0
+               if (graphicsbg_changed)
                        graphics::GCache::get().changeDisplay(true);
 #endif
-               }
 
                if (!lcolor.setColor(lyx_name, x11_name)) {
-                       current_view_->message(
-                               bformat(_("Set-color \"%1$s\" failed "
+                       if (current_view_)
+                               current_view_->message(
+                                       bformat(_("Set-color \"%1$s\" failed "
                                        "- color is undefined or "
                                        "may not be redefined"),
                                        from_utf8(lyx_name)));
@@ -1399,7 +1429,7 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                LyXRC const lyxrc_orig = lyxrc;
 
                istringstream ss(to_utf8(cmd.argument()));
-               bool const success = lyxrc.read(ss) == 0;
+               bool const success = lyxrc.read(ss);
 
                if (!success) {
                        lyxerr << "Warning in LFUN_LYXRC_APPLY!\n"
@@ -1409,7 +1439,6 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                }
 
                actOnUpdatedPrefs(lyxrc_orig, lyxrc);
-               setSpellChecker();
                resetGui();
 
                break;
@@ -1442,13 +1471,18 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
 
        // --- lyxserver commands ----------------------------
        case LFUN_SERVER_GET_FILENAME: {
-               LASSERT(current_view_ && current_view_->documentBufferView(), return);
-               docstring const fname = from_utf8(
+               if (current_view_ && current_view_->documentBufferView()) {
+                       docstring const fname = from_utf8(
                                current_view_->documentBufferView()->buffer().absFileName());
-               dr.setMessage(fname);
-               LYXERR(Debug::INFO, "FNAME[" << fname << ']');
+                       dr.setMessage(fname);
+                       LYXERR(Debug::INFO, "FNAME[" << fname << ']');
+               } else {
+                       dr.setMessage(docstring());
+                       LYXERR(Debug::INFO, "No current file for LFUN_SERVER_GET_FILENAME");
+               }
                break;
        }
+
        case LFUN_SERVER_NOTIFY: {
                docstring const dispatch_buffer = d->keyseq.print(KeySequence::Portable);
                dr.setMessage(dispatch_buffer);
@@ -1580,80 +1614,12 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                break;
 
        default:
-               // Everything below is only for active window
-               if (current_view_ == 0)
-                       break;
-       
-               // Let the current GuiView dispatch its own actions.
-               current_view_->dispatch(cmd, dr);
-
-               if (dr.dispatched())
-                       break;
-
-               BufferView * bv = current_view_->currentBufferView();
-               LASSERT(bv, /**/);
-       
-               // Let the current BufferView dispatch its own actions.
-               bv->dispatch(cmd, dr);
-               if (dr.dispatched())
-                       break;
-       
-               BufferView * doc_bv = current_view_->documentBufferView();
-               // Try with the document BufferView dispatch if any.
-               if (doc_bv) {
-                       doc_bv->dispatch(cmd, dr);
-                       if (dr.dispatched())
-                               break;
-               }
-       
-               // OK, so try the current Buffer itself...
-               bv->buffer().dispatch(cmd, dr);
-               if (dr.dispatched())
-                       break;
-       
-               // and with the document Buffer.
-               if (doc_bv) {
-                       doc_bv->buffer().dispatch(cmd, dr);
-                       if (dr.dispatched())
-                               break;
-               }
-       
-               // Let the current Cursor dispatch its own actions.
-               Cursor old = bv->cursor();
-               bv->cursor().dispatch(cmd);
-       
-               // notify insets we just left
-               if (bv->cursor() != old) {
-                       old.fixIfBroken();
-                       bool badcursor = notifyCursorLeavesOrEnters(old, bv->cursor());
-                       if (badcursor)
-                               bv->cursor().fixIfBroken();
-               }
-       
-               // update completion. We do it here and not in
-               // processKeySym to avoid another redraw just for a
-               // changed inline completion
-               if (cmd.origin() == FuncRequest::KEYBOARD) {
-                       if (cmd.action() == LFUN_SELF_INSERT
-                                       || (cmd.action() == LFUN_ERT_INSERT && bv->cursor().inMathed()))
-                               current_view_->updateCompletion(bv->cursor(), true, true);
-                       else if (cmd.action() == LFUN_CHAR_DELETE_BACKWARD)
-                               current_view_->updateCompletion(bv->cursor(), false, true);
-                       else
-                               current_view_->updateCompletion(bv->cursor(), false, false);
-               }
-       
-               dr = bv->cursor().result();
+               // The LFUN must be for one of GuiView, BufferView, Buffer or Cursor;
+               // let's try that:
+               if (current_view_)
+                       current_view_->dispatch(cmd, dr);
+               break;
        }
-       
-       // if we executed a mutating lfun, mark the buffer as dirty
-       Buffer * doc_buffer = (current_view_ && current_view_->documentBufferView())
-                     ? &(current_view_->documentBufferView()->buffer()) : 0;
-       if (doc_buffer && theBufferList().isLoaded(doc_buffer)
-               && flag.enabled()
-               && !lyxaction.funcHasFlag(action, LyXAction::NoBuffer)
-               && !lyxaction.funcHasFlag(action, LyXAction::ReadOnly))
-               current_view_->currentBufferView()->buffer().markDirty();
 }
 
 
@@ -1705,7 +1671,6 @@ void GuiApplication::processKeySym(KeySymbol const & keysym, KeyModifier state)
        }
 
        if (keysym.isModifier()) {
-               LYXERR(Debug::KEY, "isModifier true");
                if (current_view_)
                        current_view_->restartCursor();
                return;
@@ -1902,13 +1867,13 @@ Selection & GuiApplication::selection()
 }
 
 
-FontLoader & GuiApplication::fontLoader() 
+FontLoader & GuiApplication::fontLoader()
 {
        return d->font_loader_;
 }
 
 
-Toolbars const & GuiApplication::toolbars() const 
+Toolbars const & GuiApplication::toolbars() const
 {
        return d->toolbars_;
 }
@@ -1916,11 +1881,11 @@ Toolbars const & GuiApplication::toolbars() const
 
 Toolbars & GuiApplication::toolbars()
 {
-       return d->toolbars_; 
+       return d->toolbars_;
 }
 
 
-Menus const & GuiApplication::menus() const 
+Menus const & GuiApplication::menus() const
 {
        return d->menus_;
 }
@@ -1928,7 +1893,7 @@ Menus const & GuiApplication::menus() const
 
 Menus & GuiApplication::menus()
 {
-       return d->menus_; 
+       return d->menus_;
 }
 
 
@@ -1974,7 +1939,7 @@ void GuiApplication::setGuiLanguage()
        // install translation file for Qt built-in dialogs
        QString const language_name = QString("qt_") + default_locale.name();
 
-       // language_name can be short (e.g. qt_zh) or long (e.g. qt_zh_CN). 
+       // language_name can be short (e.g. qt_zh) or long (e.g. qt_zh_CN).
        // Short-named translator can be loaded from a long name, but not the
        // opposite. Therefore, long name should be used without truncation.
        // c.f. http://doc.trolltech.com/4.1/qtranslator.html#load
@@ -2068,9 +2033,11 @@ void GuiApplication::restoreGuiSession()
                return;
 
        Session & session = theSession();
-       LastOpenedSection::LastOpened const & lastopened = 
+       LastOpenedSection::LastOpened const & lastopened =
                session.lastOpened().getfiles();
 
+       validateCurrentView();
+
        FileName active_file;
        // do not add to the lastfile list since these files are restored from
        // last session, and should be already there (regular files), or should
@@ -2092,7 +2059,7 @@ void GuiApplication::restoreGuiSession()
 
        // Restore last active buffer
        Buffer * buffer = theBufferList().getBuffer(active_file);
-       if (buffer)
+       if (buffer && current_view_)
                current_view_->setBuffer(buffer);
 
        // clear this list to save a few bytes of RAM
@@ -2166,7 +2133,7 @@ bool GuiApplication::notify(QObject * receiver, QEvent * event)
                return QApplication::notify(receiver, event);
        }
        catch (ExceptionMessage const & e) {
-               switch(e.type_) { 
+               switch(e.type_) {
                case ErrorException:
                        emergencyCleanup();
                        setQuitOnLastWindowClosed(false);
@@ -2180,7 +2147,7 @@ bool GuiApplication::notify(QObject * receiver, QEvent * event)
                        this->exit(1);
 
                case BufferException: {
-                       if (!current_view_->documentBufferView())
+                       if (!current_view_ || !current_view_->documentBufferView())
                                return false;
                        Buffer * buf = &current_view_->documentBufferView()->buffer();
                        docstring details = e.details_ + '\n';
@@ -2230,6 +2197,22 @@ bool GuiApplication::getRgbColor(ColorCode col, RGBColor & rgbcol)
 }
 
 
+bool Application::getRgbColorUncached(ColorCode col, RGBColor & rgbcol)
+{
+       QColor const qcol(lcolor.getX11Name(col).c_str());
+       if (!qcol.isValid()) {
+               rgbcol.r = 0;
+               rgbcol.g = 0;
+               rgbcol.b = 0;
+               return false;
+       }
+       rgbcol.r = qcol.red();
+       rgbcol.g = qcol.green();
+       rgbcol.b = qcol.blue();
+       return true;
+}
+
+
 string const GuiApplication::hexName(ColorCode col)
 {
        return ltrim(fromqstr(d->color_cache_.get(col).name()), "#");
@@ -2265,8 +2248,13 @@ void GuiApplication::commitData(QSessionManager & sm)
        /// interaction.
        /// We are changing that to close all wiew one by one.
        /// FIXME: verify if the default implementation is enough now.
-       if (sm.allowsInteraction() && !closeAllViews())
-               sm.cancel();
+       #ifdef QT_NO_SESSIONMANAGER
+               #warning Qt is compiled without session manager
+               (void) sm;
+       #else
+               if (sm.allowsInteraction() && !closeAllViews())
+                       sm.cancel();
+       #endif
 }
 
 
@@ -2333,6 +2321,104 @@ bool GuiApplication::searchMenu(FuncRequest const & func,
 }
 
 
+// Ensure that a file is read only once (prevents include loops)
+static QStringList uifiles;
+// store which ui files define Toolbars
+static QStringList toolbar_uifiles;
+
+
+GuiApplication::ReturnValues GuiApplication::readUIFile(FileName ui_path)
+{
+       enum {
+               ui_menuset = 1,
+               ui_toolbars,
+               ui_toolbarset,
+               ui_include,
+               ui_format,
+               ui_last
+       };
+
+       LexerKeyword uitags[] = {
+               { "format", ui_format },
+               { "include", ui_include },
+               { "menuset", ui_menuset },
+               { "toolbars", ui_toolbars },
+               { "toolbarset", ui_toolbarset }
+       };
+
+       Lexer lex(uitags);
+       lex.setFile(ui_path);
+       if (!lex.isOK()) {
+               lyxerr << "Unable to set LyXLeX for ui file: " << ui_path
+                                        << endl;
+       }
+
+       if (lyxerr.debugging(Debug::PARSER))
+               lex.printTable(lyxerr);
+
+       bool error = false;
+       // format before introduction of format tag
+       unsigned int format = 0;
+       while (lex.isOK()) {
+               int const status = lex.lex();
+
+               // we have to do this check here, outside the switch,
+               // because otherwise we would start reading include files,
+               // e.g., if the first tag we hit was an include tag.
+               if (status == ui_format)
+                       if (lex.next()) {
+                               format = lex.getInteger();
+                               continue;
+                       }
+
+               // this will trigger unless the first tag we hit is a format
+               // tag, with the right format.
+               if (format != LFUN_FORMAT)
+                       return FormatMismatch;
+
+               switch (status) {
+               case Lexer::LEX_FEOF:
+                       continue;
+
+               case ui_include: {
+                       lex.next(true);
+                       QString const file = toqstr(lex.getString());
+                       bool const success = readUIFile(file, true);
+                       if (!success) {
+                               LYXERR0("Failed to read inlcuded file: " << fromqstr(file));
+                               return ReadError;
+                       }
+                       break;
+               }
+
+               case ui_menuset:
+                       d->menus_.read(lex);
+                       break;
+
+               case ui_toolbarset:
+                       d->toolbars_.readToolbars(lex);
+                       break;
+
+               case ui_toolbars:
+                       d->toolbars_.readToolbarSettings(lex);
+                       toolbar_uifiles.push_back(toqstr(ui_path.absFileName()));
+                       break;
+
+               default:
+                       if (!rtrim(lex.getString()).empty())
+                               lex.printError("LyX::ReadUIFile: "
+                                              "Unknown menu tag: `$$Token'");
+                       else
+                               LYXERR0("Error with status: " << status);
+                       error = true;
+                       break;
+               }
+
+       }
+       return (error ? ReadError : ReadOK);
+}
+
+
 bool GuiApplication::readUIFile(QString const & name, bool include)
 {
        LYXERR(Debug::INIT, "About to read " << name << "...");
@@ -2370,8 +2456,6 @@ bool GuiApplication::readUIFile(QString const & name, bool include)
                return readUIFile(defaultUIFile, false);
        }
 
-       // Ensure that a file is read only once (prevents include loops)
-       static QStringList uifiles;
        QString const uifile = toqstr(ui_path.absFileName());
        if (uifiles.contains(uifile)) {
                if (!include) {
@@ -2389,62 +2473,24 @@ bool GuiApplication::readUIFile(QString const & name, bool include)
 
        LYXERR(Debug::INIT, "Found " << name << " in " << ui_path);
 
-       enum {
-               ui_menuset = 1,
-               ui_toolbars,
-               ui_toolbarset,
-               ui_include,
-               ui_last
-       };
+       ReturnValues retval = readUIFile(ui_path);
 
-       LexerKeyword uitags[] = {
-               { "include", ui_include },
-               { "menuset", ui_menuset },
-               { "toolbars", ui_toolbars },
-               { "toolbarset", ui_toolbarset }
-       };
-
-       Lexer lex(uitags);
-       lex.setFile(ui_path);
-       if (!lex.isOK()) {
-               lyxerr << "Unable to set LyXLeX for ui file: " << ui_path
-                      << endl;
-       }
-
-       if (lyxerr.debugging(Debug::PARSER))
-               lex.printTable(lyxerr);
-
-       // store which ui files define Toolbars
-       static QStringList toolbar_uifiles;
-
-       while (lex.isOK()) {
-               switch (lex.lex()) {
-               case ui_include: {
-                       lex.next(true);
-                       QString const file = toqstr(lex.getString());
-                       if (!readUIFile(file, true))
-                               return false;
-                       break;
+       if (retval == FormatMismatch) {
+               LYXERR(Debug::FILES, "Converting ui file to format " << LFUN_FORMAT);
+               FileName const tempfile = FileName::tempName("convert_ui");
+               bool const success = prefs2prefs(ui_path, tempfile, true);
+               if (!success) {
+                       LYXERR0("Unable to convert " << ui_path.absFileName() <<
+                               " to format " << LFUN_FORMAT << ".");
+               } else {
+                       retval = readUIFile(tempfile);
+                       tempfile.removeFile();
                }
-               case ui_menuset:
-                       d->menus_.read(lex);
-                       break;
-
-               case ui_toolbarset:
-                       d->toolbars_.readToolbars(lex);
-                       break;
-
-               case ui_toolbars:
-                       d->toolbars_.readToolbarSettings(lex);
-                       toolbar_uifiles.push_back(uifile);
-                       break;
+       }
 
-               default:
-                       if (!rtrim(lex.getString()).empty())
-                               lex.printError("LyX::ReadUIFile: "
-                                              "Unknown menu tag: `$$Token'");
-                       break;
-               }
+       if (retval != ReadOK) {
+               LYXERR0("Unable to read UI file: " << ui_path.absFileName());
+               return false;
        }
 
        if (include)