]> git.lyx.org Git - lyx.git/blobdiff - src/lyxfunc.C
architectural changes to tex2lyx
[lyx.git] / src / lyxfunc.C
index f14984b7f1c134715f3d5c54a4632b1a0a13b1d3..750d2c210026a8141f171e64645863c7ff3902b0 100644 (file)
@@ -15,6 +15,8 @@
 #include "kbmap.h"
 #include "lyxrow.h"
 #include "bufferlist.h"
+#include "buffer.h"
+#include "buffer_funcs.h"
 #include "BufferView.h"
 #include "lyxserver.h"
 #include "intl.h"
@@ -43,8 +45,7 @@
 #include "ParagraphParameters.h"
 
 #include "insets/insetcommand.h"
-#include "insets/inseterror.h"
-#include "insets/insetert.h"
+#include "insets/insetexternal.h"
 #include "insets/insettabular.h"
 
 #include "mathed/formulamacro.h"
 #include "support/FileInfo.h"
 #include "support/forkedcontr.h"
 #include "support/lstrings.h"
+#include "support/tostr.h"
 #include "support/path.h"
+#include "support/path_defines.h"
 #include "support/lyxfunctional.h"
 
-#include "BoostFormat.h"
-
 #include <ctime>
 #include <clocale>
 #include <cstdlib>
@@ -78,6 +79,8 @@
 #include <utility>
 #include <algorithm>
 
+using namespace lyx::support;
+
 using std::pair;
 using std::make_pair;
 using std::endl;
@@ -85,6 +88,7 @@ using std::find_if;
 using std::vector;
 using std::transform;
 using std::back_inserter;
+using namespace bv_funcs;
 
 extern BufferList bufferlist;
 extern LyXServer * lyxserver;
@@ -108,26 +112,12 @@ LyXFunc::LyXFunc(LyXView * o)
 }
 
 
-inline
-LyXText * LyXFunc::TEXT(bool flag = true) const
+void LyXFunc::moveCursorUpdate()
 {
-       if (flag)
-               return view()->text;
-       return view()->getLyXText();
-}
-
-
-inline
-void LyXFunc::moveCursorUpdate(bool flag, bool selecting)
-{
-       if (selecting || TEXT(flag)->selection.mark()) {
-               TEXT(flag)->setSelection(view());
-               if (TEXT(flag)->isTopLevel())
-                   view()->toggleToggle();
-       }
-       view()->update(TEXT(flag), BufferView::SELECT|BufferView::FITCUR);
-       view()->showCursor();
-
+       LyXText * lt = view()->text;
+       if (lt->selection.mark())
+               lt->setSelection();
+       view()->update();
        view()->switchKeyMap();
 }
 
@@ -141,14 +131,13 @@ void LyXFunc::handleKeyFunc(kb_action action)
        }
 
        owner->getIntl().getTransManager()
-               .deadkey(c, get_accent(action).accent, TEXT(false));
+               .deadkey(c, get_accent(action).accent, view()->getLyXText());
        // Need to clear, in case the minibuffer calls these
        // actions
        keyseq.clear();
        // copied verbatim from do_accent_char
-       view()->update(TEXT(false),
-              BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-       TEXT(false)->selection.cursor = TEXT(false)->cursor;
+       view()->update();
+       view()->getLyXText()->selection.cursor = view()->getLyXText()->cursor;
 }
 
 
@@ -320,10 +309,6 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
        // encode this in the function itself.
        bool disable = false;
        switch (ev.action) {
-       case LFUN_MENUPRINT:
-               disable = !Exporter::IsExportable(buf, "dvi")
-                       || lyxrc.print_command == "none";
-               break;
        case LFUN_EXPORT:
                disable = ev.argument != "custom"
                        && !Exporter::IsExportable(buf, ev.argument);
@@ -338,8 +323,8 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
        case LFUN_COPY:
                if (tli) {
                        UpdatableInset * in = tli;
-                       if (in->lyxCode() != Inset::TABULAR_CODE) {
-                               in = tli->getFirstLockingInsetOfType(Inset::TABULAR_CODE);
+                       if (in->lyxCode() != InsetOld::TABULAR_CODE) {
+                               in = tli->getFirstLockingInsetOfType(InsetOld::TABULAR_CODE);
                        }
                        if (in && static_cast<InsetTabular*>(in)->hasSelection()) {
                                disable = false;
@@ -349,31 +334,35 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                disable = !mathcursor && !view()->getLyXText()->selection.set();
                break;
        case LFUN_RUNCHKTEX:
-               disable = lyxrc.chktex_command == "none";
+               disable = !buf->isLatex() || lyxrc.chktex_command == "none";
                break;
        case LFUN_BUILDPROG:
                disable = !Exporter::IsExportable(buf, "program");
                break;
 
-       case LFUN_LAYOUT_CHARACTER:
-               disable = tli && tli->lyxCode() == Inset::ERT_CODE;
-               break;
-
        case LFUN_LAYOUT_TABULAR:
                disable = !tli
-                       || (tli->lyxCode() != Inset::TABULAR_CODE
-                           && !tli->getFirstLockingInsetOfType(Inset::TABULAR_CODE));
+                       || (tli->lyxCode() != InsetOld::TABULAR_CODE
+                           && !tli->getFirstLockingInsetOfType(InsetOld::TABULAR_CODE));
+               break;
+
+       case LFUN_DEPTH_MIN:
+               disable = !changeDepth(view(), view()->getLyXText(), DEC_DEPTH, true);
+               break;
+
+       case LFUN_DEPTH_PLUS:
+               disable = !changeDepth(view(), view()->getLyXText(), INC_DEPTH, true);
                break;
 
        case LFUN_LAYOUT:
        case LFUN_LAYOUT_PARAGRAPH: {
-               Inset * inset = TEXT(false)->cursor.par()->inInset();
+               InsetOld * inset = view()->getLyXText()->cursor.par()->inInset();
                disable = inset && inset->forceDefaultParagraphs(inset);
                break;
        }
 
        case LFUN_INSET_OPTARG:
-               disable = (TEXT(false)->cursor.par()->layout()->optionalargs == 0);
+               disable = (view()->getLyXText()->cursor.par()->layout()->optionalargs == 0);
                break;
 
        case LFUN_TABULAR_FEATURE:
@@ -421,16 +410,20 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                if (tli) {
                        FuncStatus ret;
                        //ret.disabled(true);
-                       if (tli->lyxCode() == Inset::TABULAR_CODE) {
+                       if (tli->lyxCode() == InsetOld::TABULAR_CODE) {
                                ret = static_cast<InsetTabular *>(tli)
                                        ->getStatus(ev.argument);
-                       } else if (tli->getFirstLockingInsetOfType(Inset::TABULAR_CODE)) {
+                               flag |= ret;
+                               disable = false;
+                       } else if (tli->getFirstLockingInsetOfType(InsetOld::TABULAR_CODE)) {
                                ret = static_cast<InsetTabular *>
-                                       (tli->getFirstLockingInsetOfType(Inset::TABULAR_CODE))
+                                       (tli->getFirstLockingInsetOfType(InsetOld::TABULAR_CODE))
                                        ->getStatus(ev.argument);
+                               flag |= ret;
+                               disable = false;
+                       } else {
+                               disable = true;
                        }
-                       flag |= ret;
-                       disable = false;
                } else {
                        static InsetTabular inset(*owner->buffer(), 1, 1);
                        FuncStatus ret;
@@ -453,7 +446,6 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                break;
        case LFUN_VC_REVERT:
        case LFUN_VC_UNDO:
-       case LFUN_VC_HISTORY:
                disable = !buf->lyxvc.inUse();
                break;
        case LFUN_MENURELOAD:
@@ -478,9 +470,47 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                                && lt->inset_owner->owner()->isOpen()));
                break;
        }
-       case LFUN_LATEX_LOG:
-               disable = !IsFileReadable(buf->getLogName().second);
+
+       case LFUN_INSET_SETTINGS: {
+               disable = true;
+               UpdatableInset * inset = view()->theLockingInset();
+
+               if (!inset)
+                       break;
+
+               // get the innermost inset
+               inset = inset->getLockingInset();
+
+               // jump back to owner if an InsetText, so
+               // we get back to the InsetTabular or whatever
+               if (inset->lyxCode() == InsetOld::TEXT_CODE)
+                       inset = inset->owner();
+
+               InsetOld::Code code = inset->lyxCode();
+               switch (code) {
+                       case InsetOld::TABULAR_CODE:
+                               disable = ev.argument != "tabular";
+                               break;
+                       case InsetOld::ERT_CODE:
+                               disable = ev.argument != "ert";
+                               break;
+                       case InsetOld::FLOAT_CODE:
+                               disable = ev.argument != "float";
+                               break;
+                       case InsetOld::MINIPAGE_CODE:
+                               disable = ev.argument != "minipage";
+                               break;
+                       case InsetOld::WRAP_CODE:
+                               disable = ev.argument != "wrap";
+                               break;
+                       case InsetOld::NOTE_CODE:
+                               disable = ev.argument != "note";
+                               break;
+                       default:
+                               break;
+               }
                break;
+       }
 
        case LFUN_MATH_MUTATE:
                if (mathcursor)
@@ -500,86 +530,131 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                disable = !mathcursor;
                break;
 
+       case LFUN_DIALOG_SHOW: {
+               string const name = ev.getArg(0);
+               if (!buf) {
+                       disable = !(name == "aboutlyx" ||
+                                   name == "file" ||
+                                   name == "forks" ||
+                                   name == "preferences" ||
+                                   name == "texinfo");
+                       break;
+               }
+
+               if (name == "print") {
+                       disable = !Exporter::IsExportable(buf, "dvi") ||
+                               lyxrc.print_command == "none";
+               } else if (name == "character") {
+                       UpdatableInset * tli = view()->theLockingInset();
+                       disable = tli && tli->lyxCode() == InsetOld::ERT_CODE;
+               } else if (name == "vclog") {
+                       disable = !buf->lyxvc.inUse();
+               } else if (name == "latexlog") {
+                       disable = !IsFileReadable(buf->getLogName().second);
+               }
+               break;
+       }
+
        default:
                break;
        }
 
        // the functions which insert insets
-       Inset::Code code = Inset::NO_CODE;
+       InsetOld::Code code = InsetOld::NO_CODE;
        switch (ev.action) {
-       case LFUN_INSET_ERT:
-               code = Inset::ERT_CODE;
+       case LFUN_DIALOG_SHOW_NEW_INSET:
+               if (ev.argument == "bibitem")
+                       code = InsetOld::BIBITEM_CODE;
+               else if (ev.argument == "bibtex")
+                       code = InsetOld::BIBTEX_CODE;
+               else if (ev.argument == "citation")
+                       code = InsetOld::CITE_CODE;
+               else if (ev.argument == "ert")
+                       code = InsetOld::ERT_CODE;
+               else if (ev.argument == "external")
+                       code = InsetOld::EXTERNAL_CODE;
+               else if (ev.argument == "float")
+                       code = InsetOld::FLOAT_CODE;
+               else if (ev.argument == "graphics")
+                       code = InsetOld::GRAPHICS_CODE;
+               else if (ev.argument == "include")
+                       code = InsetOld::INCLUDE_CODE;
+               else if (ev.argument == "index")
+                       code = InsetOld::INDEX_CODE;
+               else if (ev.argument == "label")
+                       code = InsetOld::LABEL_CODE;
+               else if (ev.argument == "minipage")
+                       code = InsetOld::MINIPAGE_CODE;
+               else if (ev.argument == "ref")
+                       code = InsetOld::REF_CODE;
+               else if (ev.argument == "toc")
+                       code = InsetOld::TOC_CODE;
+               else if (ev.argument == "url")
+                       code = InsetOld::URL_CODE;
+               else if (ev.argument == "wrap")
+                       code = InsetOld::WRAP_CODE;
                break;
-       case LFUN_INSET_GRAPHICS:
-               code = Inset::GRAPHICS_CODE;
+
+       case LFUN_INSET_ERT:
+               code = InsetOld::ERT_CODE;
                break;
        case LFUN_INSET_FOOTNOTE:
-               code = Inset::FOOT_CODE;
+               code = InsetOld::FOOT_CODE;
                break;
        case LFUN_TABULAR_INSERT:
-               code = Inset::TABULAR_CODE;
-               break;
-       case LFUN_INSET_EXTERNAL:
-               code = Inset::EXTERNAL_CODE;
+               code = InsetOld::TABULAR_CODE;
                break;
        case LFUN_INSET_MARGINAL:
-               code = Inset::MARGIN_CODE;
+               code = InsetOld::MARGIN_CODE;
                break;
        case LFUN_INSET_MINIPAGE:
-               code = Inset::MINIPAGE_CODE;
+               code = InsetOld::MINIPAGE_CODE;
                break;
        case LFUN_INSET_FLOAT:
        case LFUN_INSET_WIDE_FLOAT:
-               code = Inset::FLOAT_CODE;
+               code = InsetOld::FLOAT_CODE;
                break;
        case LFUN_INSET_WRAP:
-               code = Inset::WRAP_CODE;
+               code = InsetOld::WRAP_CODE;
                break;
        case LFUN_FLOAT_LIST:
-               code = Inset::FLOAT_LIST_CODE;
+               code = InsetOld::FLOAT_LIST_CODE;
                break;
 #if 0
        case LFUN_INSET_LIST:
-               code = Inset::LIST_CODE;
+               code = InsetOld::LIST_CODE;
                break;
        case LFUN_INSET_THEOREM:
-               code = Inset::THEOREM_CODE;
+               code = InsetOld::THEOREM_CODE;
                break;
 #endif
        case LFUN_INSET_CAPTION:
-               code = Inset::CAPTION_CODE;
+               code = InsetOld::CAPTION_CODE;
                break;
        case LFUN_INSERT_NOTE:
-               code = Inset::NOTE_CODE;
+               code = InsetOld::NOTE_CODE;
                break;
        case LFUN_INSERT_LABEL:
-               code = Inset::LABEL_CODE;
+               code = InsetOld::LABEL_CODE;
                break;
        case LFUN_INSET_OPTARG:
-               code = Inset::OPTARG_CODE;
+               code = InsetOld::OPTARG_CODE;
                break;
-       case LFUN_REF_INSERT:
-               code = Inset::REF_CODE;
+       case LFUN_ENVIRONMENT_INSERT:
+               code = InsetOld::MINIPAGE_CODE;
                break;
        case LFUN_INDEX_INSERT:
-               code = Inset::INDEX_CODE;
+               code = InsetOld::INDEX_CODE;
                break;
        case LFUN_INDEX_PRINT:
-               code = Inset::INDEX_PRINT_CODE;
-               break;
-       case LFUN_CHILD_INSERT:
-               code = Inset::INCLUDE_CODE;
+               code = InsetOld::INDEX_PRINT_CODE;
                break;
        case LFUN_TOC_INSERT:
-               code = Inset::TOC_CODE;
-               break;
-       case LFUN_PARENTINSERT:
-               code = Inset::PARENT_CODE;
+               code = InsetOld::TOC_CODE;
                break;
        case LFUN_HTMLURL:
        case LFUN_URL:
-       case LFUN_INSERT_URL:
-               code = Inset::URL_CODE;
+               code = InsetOld::URL_CODE;
                break;
        case LFUN_QUOTE:
                // always allow this, since we will inset a raw quote
@@ -591,17 +666,17 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
        case LFUN_MENU_SEPARATOR:
        case LFUN_LDOTS:
        case LFUN_END_OF_SENTENCE:
-               code = Inset::SPECIALCHAR_CODE;
+               code = InsetOld::SPECIALCHAR_CODE;
                break;
-       case LFUN_PROTECTEDSPACE:
+       case LFUN_SPACE_INSERT:
                // slight hack: we know this is allowed in math mode
                if (!mathcursor)
-                       code = Inset::SPECIALCHAR_CODE;
+                       code = InsetOld::SPACE_CODE;
                break;
        default:
                break;
        }
-       if (code != Inset::NO_CODE && tli && !tli->insetAllowed(code))
+       if (code != InsetOld::NO_CODE && tli && !tli->insetAllowed(code))
                disable = true;
 
        if (disable)
@@ -617,7 +692,7 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
                flag.setOnOff(buf->isReadonly());
                break;
        case LFUN_APPENDIX:
-               flag.setOnOff(TEXT(false)->cursor.par()->params().startOfAppendix());
+               flag.setOnOff(view()->getLyXText()->cursor.par()->params().startOfAppendix());
                break;
        case LFUN_SWITCHBUFFER:
                // toggle on the current buffer, but do not toggle off
@@ -634,7 +709,7 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & ev) const
 
        // the font related toggles
        if (!mathcursor) {
-               LyXFont const & font = TEXT(false)->real_current_font;
+               LyXFont const & font = view()->getLyXText()->real_current_font;
                switch (ev.action) {
                case LFUN_EMPH:
                        flag.setOnOff(font.emph() == LyXFont::ON);
@@ -702,15 +777,7 @@ void LyXFunc::dispatch(string const & s, bool verbose)
        int const action = lyxaction.LookupFunc(s);
 
        if (action == LFUN_UNKNOWN_ACTION) {
-#if USE_BOOST_FORMAT
-boost::format fmt(_("Unknown function (%1$s)"));
-fmt % s;
-owner->message(fmt.str());
-#else
-               string const msg = string(_("Unknown function ("))
-                       + s + ')';
-               owner->message(msg);
-#endif
+               owner->message(bformat(_("Unknown function (%1$s)"), s));
                return;
        }
 
@@ -723,6 +790,28 @@ void LyXFunc::dispatch(int ac, bool verbose)
        dispatch(lyxaction.retrieveActionArg(ac), verbose);
 }
 
+namespace {
+       bool ensureBufferClean(BufferView * bv) {
+
+               Buffer & buf = *bv->buffer();
+               if (buf.isClean())
+                       return true;
+
+               string const file = MakeDisplayPath(buf.fileName(), 30);
+               string text = bformat(_("The document %1$s has unsaved "
+                                       "changes.\n\nDo you want to save "
+                                       "the document?"), file);
+               int const ret = Alert::prompt(_("Save changed document?"),
+                                             text, 0, 1, _("&Save"),
+                                             _("&Cancel"));
+
+               if (ret == 0)
+                       bv->owner()->dispatch(FuncRequest(LFUN_MENUWRITE));
+
+               return buf.isClean();
+       }
+
+} //namespace anon
 
 
 void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
@@ -743,9 +832,6 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 
        selection_possible = false;
 
-       if (view()->available())
-               view()->hideCursor();
-
        string argument = ev.argument;
        kb_action action = ev.action;
 
@@ -759,10 +845,13 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                goto exit_with_message;
        }
 
+       if (view()->available())
+               view()->hideCursor();
+
        if (view()->available() && view()->theLockingInset()) {
-               Inset::RESULT result;
-               if ((action > 1) || ((action == LFUN_UNKNOWN_ACTION) &&
-                                    (!keyseq.deleted())))
+               InsetOld::RESULT result;
+               if (action > 1 || (action == LFUN_UNKNOWN_ACTION &&
+                                    !keyseq.deleted()))
                {
                        UpdatableInset * inset = view()->theLockingInset();
 #if 1
@@ -774,6 +863,19 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                            && argument.empty()) {
                                argument = encoded_last_key;
                        }
+
+                       // the insets can't try to handle this,
+                       // a table cell in the dummy position will
+                       // lock its insettext, the insettext will
+                       // pass it the bufferview, and succeed,
+                       // so it will stay not locked. Not good
+                       // if we've just done LFUN_ESCAPE (which
+                       // injects an LFUN_PARAGRAPH_UPDATE)
+                       if (action == LFUN_PARAGRAPH_UPDATE) {
+                               view()->dispatch(ev);
+                               goto exit_with_message;
+                       }
+
                        // Undo/Redo is a bit tricky for insets.
                        if (action == LFUN_UNDO) {
                                view()->undo();
@@ -789,49 +891,52 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                                goto exit_with_message;
                                        // If UNDISPATCHED, just soldier on
                        else if (result == FINISHED) {
+                               owner->clearMessage();
                                goto exit_with_message;
                                // We do not need special RTL handling here:
                                // FINISHED means that the cursor should be
                                // one position after the inset.
                        } else if (result == FINISHED_RIGHT) {
-                               TEXT()->cursorRight(view());
-                               moveCursorUpdate(true, false);
-                               owner->view_state_changed();
+                               view()->text->cursorRight(view());
+                               moveCursorUpdate();
+                               owner->clearMessage();
                                goto exit_with_message;
                        } else if (result == FINISHED_UP) {
-                               if (TEXT()->cursor.irow()->previous()) {
+                               RowList::iterator const irow = view()->text->cursorIRow();
+                               if (irow != view()->text->rows().begin()) {
 #if 1
-                                       TEXT()->setCursorFromCoordinates(
-                                               view(), TEXT()->cursor.ix() + inset_x,
-                                               TEXT()->cursor.iy() -
-                                               TEXT()->cursor.irow()->baseline() - 1);
-                                       TEXT()->cursor.x_fix(TEXT()->cursor.x());
+                                       view()->text->setCursorFromCoordinates(
+                                               view()->text->cursor.ix() + inset_x,
+                                               view()->text->cursor.iy() -
+                                               irow->baseline() - 1);
+                                       view()->text->cursor.x_fix(view()->text->cursor.x());
 #else
-                                       TEXT()->cursorUp(view());
+                                       view()->text->cursorUp(view());
 #endif
-                                       moveCursorUpdate(true, false);
-                                       owner->view_state_changed();
+                                       moveCursorUpdate();
                                } else {
-                                       view()->update(TEXT(), BufferView::SELECT|BufferView::FITCUR);
+                                       view()->update();
                                }
+                               owner->clearMessage();
                                goto exit_with_message;
                        } else if (result == FINISHED_DOWN) {
-                               if (TEXT()->cursor.irow()->next()) {
+                               RowList::iterator const irow = view()->text->cursorIRow();
+                               if (boost::next(irow) != view()->text->rows().end()) {
 #if 1
-                                       TEXT()->setCursorFromCoordinates(
-                                               view(), TEXT()->cursor.ix() + inset_x,
-                                               TEXT()->cursor.iy() -
-                                               TEXT()->cursor.irow()->baseline() +
-                                               TEXT()->cursor.irow()->height() + 1);
-                                       TEXT()->cursor.x_fix(TEXT()->cursor.x());
+                                       view()->text->setCursorFromCoordinates(
+                                               view()->text->cursor.ix() + inset_x,
+                                               view()->text->cursor.iy() -
+                                               irow->baseline() +
+                                               irow->height() + 1);
+                                       view()->text->cursor.x_fix(view()->text->cursor.x());
 #else
-                                       TEXT()->cursorDown(view());
+                                       view()->text->cursorDown(view());
 #endif
                                } else {
-                                       TEXT()->cursorRight(view());
+                                       view()->text->cursorRight(view());
                                }
-                               moveCursorUpdate(true, false);
-                               owner->view_state_changed();
+                               moveCursorUpdate();
+                               owner->clearMessage();
                                goto exit_with_message;
                        }
 #warning I am not sure this is still right, please have a look! (Jug 20020417)
@@ -841,30 +946,30 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                                case LFUN_UNKNOWN_ACTION:
                                case LFUN_BREAKPARAGRAPH:
                                case LFUN_BREAKLINE:
-                                       TEXT()->cursorRight(view());
+                                       view()->text->cursorRight(view());
                                        view()->switchKeyMap();
                                        owner->view_state_changed();
                                        break;
                                case LFUN_RIGHT:
-                                       if (!TEXT()->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
-                                               TEXT()->cursorRight(view());
-                                               moveCursorUpdate(true, false);
+                                       if (!view()->text->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
+                                               view()->text->cursorRight(view());
+                                               moveCursorUpdate();
                                                owner->view_state_changed();
                                        }
                                        goto exit_with_message;
                                case LFUN_LEFT:
-                                       if (TEXT()->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
-                                               TEXT()->cursorRight(view());
-                                               moveCursorUpdate(true, false);
+                                       if (view()->text->cursor.par()->isRightToLeftPar(owner->buffer()->params)) {
+                                               view()->text->cursorRight(view());
+                                               moveCursorUpdate();
                                                owner->view_state_changed();
                                        }
                                        goto exit_with_message;
                                case LFUN_DOWN:
-                                       if (TEXT()->cursor.row()->next())
-                                               TEXT()->cursorDown(view());
+                                       if (boost::next(view()->text->cursorRow()) != view()->text->rows().end())
+                                               view()->text->cursorDown(view());
                                        else
-                                               TEXT()->cursorRight(view());
-                                       moveCursorUpdate(true, false);
+                                               view()->text->cursorRight(view());
+                                       moveCursorUpdate();
                                        owner->view_state_changed();
                                        goto exit_with_message;
                                default:
@@ -876,35 +981,31 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 
        switch (action) {
 
-       case LFUN_ESCAPE:
-       {
-               if (!view()->available()) break;
+       case LFUN_ESCAPE: {
+               if (!view()->available())
+                       break;
                // this function should be used always [asierra060396]
-               UpdatableInset * tli =
-                       view()->theLockingInset();
+               UpdatableInset * tli = view()->theLockingInset();
                if (tli) {
                        UpdatableInset * lock = tli->getLockingInset();
 
                        if (tli == lock) {
                                view()->unlockInset(tli);
-                               TEXT()->cursorRight(view());
-                               moveCursorUpdate(true, false);
+                               view()->text->cursorRight(view());
+                               moveCursorUpdate();
                                owner->view_state_changed();
                        } else {
-                               tli->unlockInsetInInset(view(),
-                                                       lock,
-                                                       true);
+                               tli->unlockInsetInInset(view(), lock, true);
                        }
                        finishUndo();
                        // Tell the paragraph dialog that we changed paragraph
-                       owner->getDialogs().updateParagraph();
+                       dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
                }
+               break;
        }
-       break;
 
-               // --- Misc -------------------------------------------
-       case LFUN_WORDFINDFORWARD  :
-       case LFUN_WORDFINDBACKWARD : {
+       case LFUN_WORDFINDFORWARD:
+       case LFUN_WORDFINDBACKWARD: {
                static string last_search;
                string searched_string;
 
@@ -915,29 +1016,23 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                        searched_string = last_search;
                }
                bool fw = (action == LFUN_WORDFINDFORWARD);
-               if (!searched_string.empty()) {
-                       lyxfind::LyXFind(view(), searched_string, fw);
-               }
-//             view()->showCursor();
+               if (!searched_string.empty())
+                       lyx::find::find(view(), searched_string, fw);
+               break;
        }
-       break;
 
        case LFUN_PREFIX:
-       {
-               if (view()->available() && !view()->theLockingInset()) {
-                       view()->update(TEXT(),
-                                             BufferView::SELECT|BufferView::FITCUR);
-               }
+               if (view()->available() && !view()->theLockingInset())
+                       view()->update();
                owner->message(keyseq.printOptions());
-       }
-       break;
+               break;
 
        // --- Misc -------------------------------------------
        case LFUN_EXEC_COMMAND:
                owner->focus_command_buffer();
                break;
 
-       case LFUN_CANCEL:                   // RVDK_PATCH_5
+       case LFUN_CANCEL:
                keyseq.reset();
                meta_fake_bit = key_modifier::none;
                if (view()->available())
@@ -946,20 +1041,17 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                setMessage(N_("Cancel"));
                break;
 
-       case LFUN_META_FAKE:                                 // RVDK_PATCH_5
-       {
+       case LFUN_META_FAKE:
                meta_fake_bit = key_modifier::alt;
                setMessage(keyseq.print());
-       }
-       break;
+               break;
 
        case LFUN_READ_ONLY_TOGGLE:
-               if (owner->buffer()->lyxvc.inUse()) {
+               if (owner->buffer()->lyxvc.inUse())
                        owner->buffer()->lyxvc.toggleReadOnly();
-               } else {
+               else
                        owner->buffer()->setReadonly(
                                !owner->buffer()->isReadonly());
-               }
                break;
 
        case LFUN_CENTER: // this is center and redraw.
@@ -981,56 +1073,58 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 
        case LFUN_MENUWRITE:
                if (!owner->buffer()->isUnnamed()) {
-                       ostringstream s1;
-#if USE_BOOST_FORMAT
-                       s1 << boost::format(_("Saving document %1$s..."))
-                          % MakeDisplayPath(owner->buffer()->fileName());
-#else
-                       s1 << _("Saving document ")
-                          << MakeDisplayPath(owner->buffer()->fileName())
-                          << _("...");
-#endif
-                       owner->message(STRCONV(s1.str()));
-                       MenuWrite(view(), owner->buffer());
-                       s1 << _(" done.");
-                       owner->message(STRCONV(s1.str()));
+                       string const str = bformat(_("Saving document %1$s..."),
+                          MakeDisplayPath(owner->buffer()->fileName()));
+                       owner->message(str);
+                       MenuWrite(owner->buffer());
+                       owner->message(str + _(" done."));
                } else
-                       WriteAs(view(), owner->buffer());
+                       WriteAs(owner->buffer());
                break;
 
        case LFUN_WRITEAS:
-               WriteAs(view(), owner->buffer(), argument);
+               WriteAs(owner->buffer(), argument);
                break;
 
-       case LFUN_MENURELOAD:
-               reloadBuffer();
+       case LFUN_MENURELOAD: {
+               string const file = MakeDisplayPath(view()->buffer()->fileName(), 20);
+               string text = bformat(_("Any changes will be lost. Are you sure "
+                       "you want to revert to the saved version of the document %1$s?"), file);
+               int const ret = Alert::prompt(_("Revert to saved document?"),
+                       text, 0, 1, _("&Revert"), _("&Cancel"));
+
+               if (ret == 0)
+                       view()->reload();
                break;
+       }
 
        case LFUN_UPDATE:
                Exporter::Export(owner->buffer(), argument, true);
+               view()->showErrorList(BufferFormat(*owner->buffer()));
                break;
 
        case LFUN_PREVIEW:
                Exporter::Preview(owner->buffer(), argument);
+               view()->showErrorList(BufferFormat(*owner->buffer()));
                break;
 
        case LFUN_BUILDPROG:
                Exporter::Export(owner->buffer(), "program", true);
+               view()->showErrorList(_("Build"));
                break;
 
        case LFUN_RUNCHKTEX:
-               MenuRunChktex(owner->buffer());
-               break;
-
-       case LFUN_MENUPRINT:
-               owner->getDialogs().showPrint();
+               owner->buffer()->runChktex();
+               view()->showErrorList(_("ChkTeX"));
                break;
 
        case LFUN_EXPORT:
                if (argument == "custom")
                        owner->getDialogs().showSendto();
-               else
+               else {
                        Exporter::Export(owner->buffer(), argument, false);
+                       view()->showErrorList(BufferFormat(*owner->buffer()));
+               }
                break;
 
        case LFUN_IMPORT:
@@ -1061,28 +1155,22 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                view()->redo();
                break;
 
-       case LFUN_MENUSEARCH:
-               owner->getDialogs().showSearch();
-               break;
-
-       case LFUN_REMOVEERRORS:
-               if (view()->removeAutoInsets()) {
-#warning repaint() or update() or nothing ?
-                       view()->repaint();
-                       view()->fitCursor();
-               }
-               break;
-
        case LFUN_DEPTH_MIN:
-               changeDepth(view(), TEXT(false), -1);
+               changeDepth(view(), view()->getLyXText(), DEC_DEPTH, false);
+               owner->view_state_changed();
                break;
 
        case LFUN_DEPTH_PLUS:
-               changeDepth(view(), TEXT(false), 1);
+               changeDepth(view(), view()->getLyXText(), INC_DEPTH, false);
+               owner->view_state_changed();
                break;
 
-       case LFUN_FREE:
-               owner->getDialogs().setUserFreeFont();
+       case LFUN_FREEFONT_APPLY:
+               apply_freefont(view());
+               break;
+
+       case LFUN_FREEFONT_UPDATE:
+               update_and_apply_freefont(view(), argument);
                break;
 
        case LFUN_RECONFIGURE:
@@ -1105,14 +1193,6 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 //#warning Find another implementation here (or another lyxfunc)!
 #endif
 #endif
-       case LFUN_HELP_ABOUTLYX:
-               owner->getDialogs().showAboutlyx();
-               break;
-
-       case LFUN_HELP_TEXINFO:
-               owner->getDialogs().showTexinfo();
-               break;
-
        case LFUN_HELP_OPEN:
        {
                string const arg = argument;
@@ -1126,61 +1206,62 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                               << arg << "'. Bad installation?" << endl;
                        break;
                }
-               ostringstream str;
-#if USE_BOOST_FORMAT
-               str << boost::format(_("Opening help file %1$s..."))
-                   % MakeDisplayPath(fname);
-#else
-               str << _("Opening help file ")
-                   << MakeDisplayPath(fname) << _("...");
-#endif
-               owner->message(STRCONV(str.str()));
-               view()->buffer(bufferlist.loadLyXFile(fname, false));
+               owner->message(bformat(_("Opening help file %1$s..."),
+                       MakeDisplayPath(fname)));
+               view()->loadLyXFile(fname, false);
                break;
        }
 
                // --- version control -------------------------------
        case LFUN_VC_REGISTER:
        {
-               if (!owner->buffer()->lyxvc.inUse())
+               if (!ensureBufferClean(view()))
+                       break;
+               if (!owner->buffer()->lyxvc.inUse()) {
                        owner->buffer()->lyxvc.registrer();
+                       view()->reload();
+               }
        }
        break;
 
        case LFUN_VC_CHECKIN:
        {
+               if (!ensureBufferClean(view()))
+                       break;
                if (owner->buffer()->lyxvc.inUse()
-                   && !owner->buffer()->isReadonly())
+                   && !owner->buffer()->isReadonly()) {
                        owner->buffer()->lyxvc.checkIn();
+                       view()->reload();
+               }
        }
        break;
 
        case LFUN_VC_CHECKOUT:
        {
+               if (!ensureBufferClean(view()))
+                       break;
                if (owner->buffer()->lyxvc.inUse()
-                   && owner->buffer()->isReadonly())
+                   && owner->buffer()->isReadonly()) {
                        owner->buffer()->lyxvc.checkOut();
+                       view()->reload();
+               }
        }
        break;
 
        case LFUN_VC_REVERT:
        {
                owner->buffer()->lyxvc.revert();
+               view()->reload();
        }
        break;
 
        case LFUN_VC_UNDO:
        {
                owner->buffer()->lyxvc.undoLast();
+               view()->reload();
        }
        break;
 
-       case LFUN_VC_HISTORY:
-       {
-               owner->getDialogs().showVCLogFile();
-               break;
-       }
-
        // --- buffers ----------------------------------------
 
        case LFUN_SWITCHBUFFER:
@@ -1188,53 +1269,28 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                break;
 
        case LFUN_FILE_NEW:
-       {
-               // servercmd: argument must be <file>:<template>
-               Buffer * tmpbuf = NewFile(argument);
-               if (tmpbuf)
-                       view()->buffer(tmpbuf);
-       }
-       break;
+               NewFile(view(), argument);
+               break;
 
        case LFUN_FILE_OPEN:
                open(argument);
                break;
 
-       case LFUN_LATEX_LOG:
-               owner->getDialogs().showLogFile();
-               break;
-
-       case LFUN_LAYOUT_DOCUMENT:
-               owner->getDialogs().showDocument();
-               break;
-
-       case LFUN_LAYOUT_PARAGRAPH:
-               owner->getDialogs().showParagraph();
-               break;
-
-       case LFUN_LAYOUT_CHARACTER:
-               owner->getDialogs().showCharacter();
-               break;
-
        case LFUN_LAYOUT_TABULAR:
            if (view()->theLockingInset()) {
-               if (view()->theLockingInset()->lyxCode()==Inset::TABULAR_CODE) {
+               if (view()->theLockingInset()->lyxCode()== InsetOld::TABULAR_CODE) {
                    InsetTabular * inset = static_cast<InsetTabular *>
                        (view()->theLockingInset());
                    inset->openLayoutDialog(view());
                } else if (view()->theLockingInset()->
-                          getFirstLockingInsetOfType(Inset::TABULAR_CODE)!=0) {
+                          getFirstLockingInsetOfType(InsetOld::TABULAR_CODE)!=0) {
                    InsetTabular * inset = static_cast<InsetTabular *>(
-                       view()->theLockingInset()->getFirstLockingInsetOfType(Inset::TABULAR_CODE));
+                       view()->theLockingInset()->getFirstLockingInsetOfType(InsetOld::TABULAR_CODE));
                    inset->openLayoutDialog(view());
                }
            }
            break;
 
-       case LFUN_LAYOUT_PREAMBLE:
-               owner->getDialogs().showPreamble();
-               break;
-
        case LFUN_DROP_LAYOUTS_CHOICE:
                owner->getToolbar().openLayoutList();
                break;
@@ -1243,10 +1299,6 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                owner->getMenubar().openByName(argument);
                break; // RVDK_PATCH_5
 
-       case LFUN_SPELLCHECK:
-               owner->getDialogs().showSpellchecker();
-               break;
-
        // --- lyxserver commands ----------------------------
 
 
@@ -1277,7 +1329,7 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                if (bufferlist.exists(s)) {
                        view()->buffer(bufferlist.getBuffer(s));
                } else {
-                       view()->buffer(bufferlist.loadLyXFile(s));
+                       view()->loadLyXFile(s);
                }
 
                view()->setCursorFromRow(row);
@@ -1294,8 +1346,8 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 
                int id;
                istr >> id;
-               Paragraph * par = owner->buffer()->getParFromID(id);
-               if (par == 0) {
+               ParIterator par = owner->buffer()->getParFromID(id);
+               if (par == owner->buffer()->par_iterator_end()) {
                        lyxerr[Debug::INFO] << "No matching paragraph found! ["
                                            << id << ']' << endl;
                        break;
@@ -1307,10 +1359,11 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                if (view()->theLockingInset())
                        view()->unlockInset(view()->theLockingInset());
                if (par->inInset()) {
-                       par->inInset()->edit(view());
+                       FuncRequest cmd(view(), LFUN_INSET_EDIT, "left");
+                       par->inInset()->localDispatch(cmd);
                }
                // Set the cursor
-               view()->getLyXText()->setCursor(view(), par, 0);
+               view()->getLyXText()->setCursor(par.pit(), 0);
                view()->switchKeyMap();
                owner->view_state_changed();
 
@@ -1320,14 +1373,6 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
        }
        break;
 
-       case LFUN_APROPOS:
-       case LFUN_GETTIP:
-       {
-               int const qa = lyxaction.LookupFunc(argument);
-               setMessage(lyxaction.helpText(static_cast<kb_action>(qa)));
-       }
-       break;
-
        // --- insert characters ----------------------------------------
 
        // ---  Mathed stuff. If we are here, there is no locked inset yet.
@@ -1348,15 +1393,39 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                dispatch(FuncRequest(view(), LFUN_SELFINSERT, "^"));
                break;
 
-       case LFUN_MATH_PANEL:
-               owner->getDialogs().showMathPanel();
+       case LFUN_DIALOG_SHOW: {
+               string const name = ev.getArg(0);
+               string data = trim(ev.argument.substr(name.size()));
+
+               if (name == "character") {
+                       data = freefont2string();
+                       if (!data.empty())
+                               owner->getDialogs().show("character", data);
+               } else if (name == "document")
+                       owner->getDialogs().showDocument();
+               else if (name == "findreplace")
+                       owner->getDialogs().showSearch();
+               else if (name == "forks")
+                       owner->getDialogs().showForks();
+               else if (name == "preamble")
+                       owner->getDialogs().showPreamble();
+               else if (name == "preferences")
+                       owner->getDialogs().showPreferences();
+               else if (name == "print")
+                       owner->getDialogs().showPrint();
+               else if (name == "spellchecker")
+                       owner->getDialogs().showSpellchecker();
+               else
+                       owner->getDialogs().show(name, data);
                break;
+       }
 
        case LFUN_DIALOG_SHOW_NEW_INSET: {
                string const & name = argument;
                string data;
                if (name == "bibitem" ||
                    name == "bibtex" ||
+                   name == "include" ||
                    name == "index" ||
                    name == "ref" ||
                    name == "toc" ||
@@ -1366,11 +1435,8 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                } else if (name == "citation") {
                        InsetCommandParams p("cite");
                        data = InsetCommandMailer::params2string(name, p);
-               } else if (name == "ert") {
-                       data = InsetERTMailer::params2string(name,
-                                                            InsetERT::Open);
                }
-                owner->getDialogs().show(name, data, 0);
+               owner->getDialogs().show(name, data, 0);
        }
        break;
 
@@ -1381,49 +1447,23 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
        case LFUN_DIALOG_UPDATE: {
                string const & name = argument;
                // Can only update a dialog connected to an existing inset
-               InsetBase * i = owner->getDialogs().getOpenInset(name);
-               if (!i)
-                       break;
-
-               if (name == "bibitem" ||
-                   name == "bibtex" ||
-                   name == "citation" ||
-                   name == "index" ||
-                   name == "ref" ||
-                   name == "toc" ||
-                   name == "url") {
-                       InsetCommand * inset = dynamic_cast<InsetCommand *>(i);
-                       if (!inset)
-                               break;
-
-                       InsetCommandMailer mailer(name, *inset);
-                       mailer.updateDialog();
-
-               } else if (name == "error") {
-                       InsetError * inset = dynamic_cast<InsetError *>(i);
-                       if (!inset)
-                               break;
-
-                       owner->getDialogs().update("error",
-                                                  inset->getContents());
-
-               } else if (name == "ert") {
-                       InsetERT * inset = dynamic_cast<InsetERT *>(i);
-                       if (!inset)
-                               break;
-
-                       InsetERTMailer mailer(*inset);
-                       mailer.updateDialog();
+               InsetBase * inset = owner->getDialogs().getOpenInset(name);
+               if (inset) {
+                       FuncRequest fr(view(), LFUN_INSET_DIALOG_UPDATE,
+                                      ev.argument);
+                       inset->localDispatch(fr);
+               } else if (name == "paragraph") {
+                       dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
                }
        }
        break;
 
        case LFUN_DIALOG_HIDE:
-               owner ->getDialogs().hide(argument);
+               Dialogs::hide(argument, 0);
                break;
 
        case LFUN_DIALOG_DISCONNECT_INSET:
-                owner->getDialogs().disconnect(argument);
+               owner->getDialogs().disconnect(argument);
                break;
 
        case LFUN_CHILDOPEN:
@@ -1437,7 +1477,7 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                if (bufferlist.exists(filename))
                        view()->buffer(bufferlist.getBuffer(filename));
                else
-                       view()->buffer(bufferlist.loadLyXFile(filename));
+                       view()->loadLyXFile(filename);
        }
        break;
 
@@ -1472,13 +1512,9 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
        }
        break;
 
-       case LFUN_DIALOG_PREFERENCES:
-               owner->getDialogs().showPreferences();
-               break;
-
        case LFUN_SAVEPREFERENCES:
        {
-               Path p(user_lyxdir);
+               Path p(user_lyxdir());
                lyxrc.write("preferences");
        }
        break;
@@ -1496,7 +1532,7 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                // if values really changed...but not very important right now. (Lgb)
                // All visible buffers will need resize
                view()->resize();
-               view()->repaint();
+               view()->update();
        }
        break;
 
@@ -1515,20 +1551,10 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                         x11_name != lcolor.getX11Name(LColor::graphicsbg));
 
                if (!lcolor.setColor(lyx_name, x11_name)) {
-#if USE_BOOST_FORMAT
                        setErrorMessage(
-                               boost::io::str(
-                                       boost::format(
-                                               _("Set-color \"%1$s\" failed "
+                               bformat(_("Set-color \"%1$s\" failed "
                                                  "- color is undefined or "
-                                                 "may not be redefined"))
-                                       % lyx_name));
-#else
-                       setErrorMessage(_("Set-color ") + lyx_name
-                                       + _(" failed - color is undefined"
-                                           " or may not be redefined"));
-#endif
-
+                                                 "may not be redefined"), lyx_name));
                        break;
                }
 
@@ -1539,12 +1565,13 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
 #warning FIXME!! The graphics cache no longer has a changeDisplay method.
 #endif
 #if 0
-                       grfx::GCache & gc = grfx::GCache::get();
+                       lyx::graphics::GCache & gc =
+                               lyx::graphics::GCache::get();
                        gc.changeDisplay(true);
 #endif
                }
 
-               view()->repaint();
+               view()->update();
                break;
        }
 
@@ -1552,10 +1579,6 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                owner->message(argument);
                break;
 
-       case LFUN_FORKS_SHOW:
-               owner->getDialogs().showForks();
-               break;
-
        case LFUN_FORKS_KILL:
        {
                if (!isStrInt(argument))
@@ -1571,6 +1594,12 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
                owner->getDialogs().toggleTooltips();
                break;
 
+       case LFUN_EXTERNAL_EDIT: {
+               InsetExternal()
+                       .localDispatch(FuncRequest(view(), action, argument));
+               break;
+       }
+
        default:
                // Then if it was none of the above
                // Trying the BufferView::pimpl dispatch:
@@ -1582,6 +1611,19 @@ void LyXFunc::dispatch(FuncRequest const & ev, bool verbose)
        } // end of switch
 
 exit_with_message:
+
+       view()->owner()->updateLayoutChoice();
+
+       if (view()->available()) {
+               view()->fitCursor();
+
+               // If we executed a mutating lfun, mark the buffer as dirty
+               if (!getStatus(ev).disabled()
+                   && !lyxaction.funcHasFlag(ev.action, LyXAction::NoBuffer)
+                   && !lyxaction.funcHasFlag(ev.action, LyXAction::ReadOnly))
+                       view()->buffer()->markDirty();
+       }
+
        sendDispatchMessage(getMessage(), ev, verbose);
 }
 
@@ -1696,7 +1738,7 @@ void LyXFunc::menuNew(string const & name, bool fromTemplate)
                templname = fname;
        }
 
-       view()->buffer(bufferlist.newFile(filename, templname, !name.empty()));
+       view()->newFile(filename, templname, !name.empty());
 }
 
 
@@ -1719,7 +1761,7 @@ void LyXFunc::open(string const & fname)
                        make_pair(string(_("Documents|#o#O")),
                                  string(lyxrc.document_path)),
                        make_pair(string(_("Examples|#E#e")),
-                                 string(AddPath(system_lyxdir, "examples"))));
+                                 string(AddPath(system_lyxdir(), "examples"))));
 
                FileDialog::Result result =
                        fileDlg.open(initpath,
@@ -1750,44 +1792,20 @@ void LyXFunc::open(string const & fname)
        // if the file doesn't exist, let the user create one
        FileInfo const f(filename, true);
        if (!f.exist()) {
-               if (!Alert::askQuestion(_("No such file"), disp_fn,
-                       _("Start a new document with this filename ?"))) {
-                       owner->message(_("Canceled."));
-                       return;
-               }
                // the user specifically chose this name. Believe them.
-               Buffer * buffer =  bufferlist.newFile(filename, "", true);
-               view()->buffer(buffer);
+               view()->newFile(filename, "", true);
                return;
        }
 
-       ostringstream str;
-#if USE_BOOST_FORMAT
-       str << boost::format(_("Opening document %1$s...")) % disp_fn;
-#else
-       str << _("Opening document ") << disp_fn << _("...");
-#endif
+       owner->message(bformat(_("Opening document %1$s..."), disp_fn));
 
-       owner->message(STRCONV(str.str()));
-
-       Buffer * openbuf = bufferlist.loadLyXFile(filename);
-       ostringstream str2;
-       if (openbuf) {
-               view()->buffer(openbuf);
-#if USE_BOOST_FORMAT
-               str2 << boost::format(_("Document %1$s opened.")) % disp_fn;
-#else
-               str2 << _("Document ") << disp_fn << _(" opened.");
-#endif
+       string str2;
+       if (view()->loadLyXFile(filename)) {
+               str2 = bformat(_("Document %1$s opened."), disp_fn);
        } else {
-#if USE_BOOST_FORMAT
-               str2 << boost::format(_("Could not open document %1$s"))
-                       % disp_fn;
-#else
-               str2 << _("Could not open document ") << disp_fn;
-#endif
+               str2 = bformat(_("Could not open document %1$s"), disp_fn);
        }
-       owner->message(STRCONV(str2.str()));
+       owner->message(str2);
 }
 
 
@@ -1810,21 +1828,15 @@ void LyXFunc::doImport(string const & argument)
                                initpath = trypath;
                }
 
-#if USE_BOOST_FORMAT
-               boost::format fmt(_("Select %1$s file to import"));
-               fmt % formats.prettyName(format);
-               string const text = fmt.str();
-#else
-               string const text = _("Select ") + formats.prettyName(format)
-                       + _(" file to import");;
-#endif
+               string const text = bformat(_("Select %1$s file to import"),
+                       formats.prettyName(format));
 
                FileDialog fileDlg(text,
                        LFUN_IMPORT,
                        make_pair(string(_("Documents|#o#O")),
                                  string(lyxrc.document_path)),
                        make_pair(string(_("Examples|#E#e")),
-                                 string(AddPath(system_lyxdir, "examples"))));
+                                 string(AddPath(system_lyxdir(), "examples"))));
 
                string const extension = "*." + formats.extension(format)
                        + "| " + formats.prettyName(format)
@@ -1852,31 +1864,24 @@ void LyXFunc::doImport(string const & argument)
        string const lyxfile = ChangeExtension(filename, ".lyx");
 
        // Check if the document already is open
-       if (lyxrc.use_gui && bufferlist.exists(lyxfile)) {
-               switch (Alert::askConfirmation(_("Document is already open:"),
-                                       MakeDisplayPath(lyxfile, 50),
-                                       _("Do you want to close that document now?\n"
-                                         "('No' will just switch to the open version)")))
-                       {
-                       case 1:
-                               // If close is canceled, we cancel here too.
-                               if (!bufferlist.close(bufferlist.getBuffer(lyxfile)))
-                                       return;
-                               break;
-                       case 2:
-                               view()->buffer(bufferlist.getBuffer(lyxfile));
-                               return;
-                       case 3:
-                               owner->message(_("Canceled."));
-                               return;
-                       }
+       if (lyx_gui::use_gui && bufferlist.exists(lyxfile)) {
+               if (!bufferlist.close(bufferlist.getBuffer(lyxfile), true)) {
+                       owner->message(_("Canceled."));
+                       return;
+               }
        }
 
        // if the file exists already, and we didn't do
        // -i lyx thefile.lyx, warn
        if (FileInfo(lyxfile, true).exist() && filename != lyxfile) {
-               if (!Alert::askQuestion(_("A document by the name"),
-                       MakeDisplayPath(lyxfile), _("already exists. Overwrite?"))) {
+               string const file = MakeDisplayPath(lyxfile, 30);
+
+               string text = bformat(_("The document %1$s already exists.\n\n"
+                       "Do you want to over-write that document?"), file);
+               int const ret = Alert::prompt(_("Over-write document?"),
+                       text, 0, 1, _("&Over-write"), _("&Cancel"));
+
+               if (ret == 1) {
                        owner->message(_("Canceled."));
                        return;
                }
@@ -1886,20 +1891,12 @@ void LyXFunc::doImport(string const & argument)
 }
 
 
-void LyXFunc::reloadBuffer()
-{
-       string const fn = owner->buffer()->fileName();
-       if (bufferlist.close(owner->buffer()))
-               view()->buffer(bufferlist.loadLyXFile(fn));
-}
-
-
 void LyXFunc::closeBuffer()
 {
-       if (bufferlist.close(owner->buffer()) && !quitting) {
+       if (bufferlist.close(owner->buffer(), true) && !quitting) {
                if (bufferlist.empty()) {
-                       // need this otherwise SEGV may occur while trying to
-                       // set variables that don't exist
+                       // need this otherwise SEGV may occur while
+                       // trying to set variables that don't exist
                        // since there's no current buffer
                        owner->getDialogs().hideBufferDependent();
                } else {
@@ -1956,6 +1953,13 @@ string const LyXFunc::view_status_message()
 
 BufferView * LyXFunc::view() const
 {
-       lyx::Assert(owner);
+       Assert(owner);
        return owner->view().get();
 }
+
+
+bool LyXFunc::wasMetaKey() const
+{
+       return (meta_fake_bit != key_modifier::none);
+}
+