]> git.lyx.org Git - lyx.git/blobdiff - src/LyXFunc.cpp
Cleanup the TEXT_TO_INSET_OFFSET mess. This correction is done now once in InsetText...
[lyx.git] / src / LyXFunc.cpp
index e724ed0c78f6a7c3223113cacf2b43a5bd49dda3..97462277b3a45ab776e455c842be93cb4b50ed87 100644 (file)
@@ -84,9 +84,8 @@
 #include "frontends/Gui.h"
 #include "frontends/KeySymbol.h"
 #include "frontends/LyXView.h"
-#include "frontends/Menubar.h"
-#include "frontends/Toolbars.h"
 #include "frontends/Selection.h"
+#include "frontends/WorkArea.h"
 
 #include "support/environment.h"
 #include "support/FileFilterList.h"
 
 #include <sstream>
 
+using std::endl;
+using std::make_pair;
+using std::pair;
+using std::string;
+using std::istringstream;
+using std::ostringstream;
+
+namespace fs = boost::filesystem;
 
 namespace lyx {
 
 using bv_funcs::freefont2string;
 
+using frontend::LyXView;
+
 using support::absolutePath;
 using support::addName;
 using support::addPath;
@@ -136,15 +145,7 @@ using support::token;
 using support::trim;
 using support::prefixIs;
 
-using std::endl;
-using std::make_pair;
-using std::pair;
-using std::string;
-using std::istringstream;
-using std::ostringstream;
-
 namespace Alert = frontend::Alert;
-namespace fs = boost::filesystem;
 
 
 namespace {
@@ -219,7 +220,7 @@ void LyXFunc::initKeySequences(KeyMap * kb)
 
 void LyXFunc::setLyXView(LyXView * lv)
 {
-       if (!quitting && lyx_view_ && lyx_view_ != lv)
+       if (!quitting && lyx_view_ && lyx_view_->view() && lyx_view_ != lv)
                // save current selection to the selection buffer to allow
                // middle-button paste in another window
                cap::saveSelection(lyx_view_->view()->cursor());
@@ -234,6 +235,7 @@ void LyXFunc::handleKeyFunc(kb_action action)
        if (keyseq->length())
                c = 0;
 
+       BOOST_ASSERT(lyx_view_ && lyx_view_->view());
        lyx_view_->view()->getIntl().getTransManager().deadkey(
                c, get_accent(action).accent, view()->cursor().innerText(), view()->cursor());
        // Need to clear, in case the minibuffer calls these
@@ -261,48 +263,58 @@ void LyXFunc::gotoBookmark(unsigned int idx, bool openFile, bool switchToBuffer)
                        return;
        }
        // open may fail, so we need to test it again
-       if (theBufferList().exists(file)) {
-               // if the current buffer is not that one, switch to it.
-               if (lyx_view_->buffer()->fileName() != file) {
-                       if (switchToBuffer)
-                               dispatch(FuncRequest(LFUN_BUFFER_SWITCH, file));
-                       else
-                               return;
-               }
-               // moveToPosition use par_id, and par_pit and return new par_id.
-               pit_type new_pit;
-               pos_type new_pos;
-               int new_id;
-               boost::tie(new_pit, new_pos, new_id) = view()->moveToPosition(bm.bottom_pit, bm.bottom_pos, bm.top_id, bm.top_pos);
-               // if bottom_pit, bottom_pos or top_id has been changed, update bookmark
-               // see http://bugzilla.lyx.org/show_bug.cgi?id=3092
-               if (bm.bottom_pit != new_pit || bm.bottom_pos != new_pos || bm.top_id != new_id )
-                       const_cast<BookmarksSection::Bookmark &>(bm).updatePos(new_pit, new_pos, new_id);
+       if (!theBufferList().exists(file))
+               return;
+
+       // if the current buffer is not that one, switch to it.
+       if (lyx_view_->buffer()->fileName() != file) {
+               if (!switchToBuffer)
+                       return;
+               dispatch(FuncRequest(LFUN_BUFFER_SWITCH, file));
+       }
+       // moveToPosition try paragraph id first and then paragraph (pit, pos).
+       if (!view()->moveToPosition(bm.bottom_pit, bm.bottom_pos,
+               bm.top_id, bm.top_pos))
+               return;
+
+       // Cursor jump succeeded!
+       Cursor const & cur = view()->cursor();
+       pit_type new_pit = cur.pit();
+       pos_type new_pos = cur.pos();
+       int new_id = cur.paragraph().id();
+
+       // if bottom_pit, bottom_pos or top_id has been changed, update bookmark
+       // see http://bugzilla.lyx.org/show_bug.cgi?id=3092
+       if (bm.bottom_pit != new_pit || bm.bottom_pos != new_pos 
+               || bm.top_id != new_id) {
+               const_cast<BookmarksSection::Bookmark &>(bm).updatePos(
+                       new_pit, new_pos, new_id);
        }
 }
 
 
-void LyXFunc::processKeySym(KeySymbolPtr keysym, key_modifier::state state)
+void LyXFunc::processKeySym(KeySymbol const & keysym,
+               key_modifier::state state)
 {
-       LYXERR(Debug::KEY) << "KeySym is " << keysym->getSymbolName() << endl;
+       LYXERR(Debug::KEY) << "KeySym is " << keysym.getSymbolName() << endl;
 
        // Do nothing if we have nothing (JMarc)
-       if (!keysym->isOK()) {
+       if (!keysym.isOK()) {
                LYXERR(Debug::KEY) << "Empty kbd action (probably composing)"
                                   << endl;
                return;
        }
 
-       if (keysym->isModifier()) {
+       if (keysym.isModifier()) {
                LYXERR(Debug::KEY) << "isModifier true" << endl;
                return;
        }
 
        //Encoding const * encoding = view()->cursor().getEncoding();
-       //encoded_last_key = keysym->getISOEncoded(encoding ? encoding->name() : "");
+       //encoded_last_key = keysym.getISOEncoded(encoding ? encoding->name() : "");
        // FIXME: encoded_last_key shadows the member variable of the same
        // name. Is that intended?
-       char_type encoded_last_key = keysym->getUCSEncoded();
+       char_type encoded_last_key = keysym.getUCSEncoded();
 
        // Do a one-deep top-level lookup for
        // cancel and meta-fake keys. RVDK_PATCH_5
@@ -360,7 +372,7 @@ void LyXFunc::processKeySym(KeySymbolPtr keysym, key_modifier::state state)
                // Hmm, we didn't match any of the keysequences. See
                // if it's normal insertable text not already covered
                // by a binding
-               if (keysym->isText() && keyseq->length() == 1) {
+               if (keysym.isText() && keyseq->length() == 1) {
                        LYXERR(Debug::KEY) << "isText() is true, inserting." << endl;
                        func = FuncRequest(LFUN_SELF_INSERT,
                                           FuncRequest::KEYBOARD);
@@ -382,6 +394,12 @@ void LyXFunc::processKeySym(KeySymbolPtr keysym, key_modifier::state state)
        } else {
                dispatch(func);
        }
+
+       /* When we move around, or type, it's nice to be able to see
+        * the cursor immediately after the keypress.
+        */
+       if (lyx_view_ && lyx_view_->currentWorkArea())
+               lyx_view_->currentWorkArea()->startBlinkingCursor();
 }
 
 
@@ -456,7 +474,7 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & cmd) const
        case LFUN_BUFFER_SWITCH:
                // toggle on the current buffer, but do not toggle off
                // the other ones (is that a good idea?)
-               if (to_utf8(cmd.argument()) == buf->fileName())
+               if (buf && to_utf8(cmd.argument()) == buf->fileName())
                        flag.setOnOff(true);
                break;
 
@@ -608,8 +626,8 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & cmd) const
        }
 
        case LFUN_BUFFER_WRITE: {
-               enable = view()->buffer()->isUnnamed()
-                       || !view()->buffer()->isClean();
+               enable = lyx_view_->buffer()->isUnnamed()
+                       || !lyx_view_->buffer()->isClean();
                break;
        }
 
@@ -647,7 +665,7 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & cmd) const
                break;
 
        case LFUN_TOOLBAR_TOGGLE: {
-               bool const current = lyx_view_->getToolbars().visible(cmd.getArg(0));
+               bool const current = lyx_view_->isToolbarVisible(cmd.getArg(0));
                flag.setOnOff(current);
                break;
        }
@@ -712,6 +730,9 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & cmd) const
        case LFUN_TEXTCLASS_LOAD:
        case LFUN_BUFFER_SAVE_AS_DEFAULT:
        case LFUN_BUFFER_PARAMS_APPLY:
+       case LFUN_LAYOUT_MODULES_CLEAR:
+       case LFUN_LAYOUT_MODULE_ADD:
+       case LFUN_LAYOUT_RELOAD:
        case LFUN_LYXRC_APPLY:
        case LFUN_BUFFER_NEXT:
        case LFUN_BUFFER_PREVIOUS:
@@ -754,7 +775,7 @@ FuncStatus LyXFunc::getStatus(FuncRequest const & cmd) const
 
 bool LyXFunc::ensureBufferClean(BufferView * bv)
 {
-       Buffer & buf = *bv->buffer();
+       Buffer & buf = bv->buffer();
        if (buf.isClean())
                return true;
 
@@ -784,7 +805,7 @@ void showPrintError(string const & name)
 }
 
 
-void loadTextclass(string const & name)
+void loadTextClass(string const & name)
 {
        std::pair<bool, textclass_type> const tc_pair =
                textclasslist.numberOfClass(name);
@@ -799,10 +820,10 @@ void loadTextclass(string const & name)
        textclass_type const tc = tc_pair.second;
 
        if (!textclasslist[tc].load()) {
-               docstring s = bformat(_("The document could not be converted\n"
-                                                 "into the document class %1$s."),
+               docstring s = bformat(_("The document class %1$s."
+                                  "could not be loaded."),
                                   from_utf8(textclasslist[tc].name()));
-               Alert::error(_("Could not change class"), s);
+               Alert::error(_("Could not load class"), s);
        }
 }
 
@@ -869,15 +890,14 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
 
                case LFUN_COMMAND_EXECUTE:
                        BOOST_ASSERT(lyx_view_);
-                       lyx_view_->getToolbars().display("minibuffer", true);
-                       lyx_view_->focus_command_buffer();
+                       lyx_view_->showMiniBuffer(true);
                        break;
 
                case LFUN_CANCEL:
                        BOOST_ASSERT(lyx_view_ && lyx_view_->view());
                        keyseq->reset();
                        meta_fake_bit = key_modifier::none;
-                       if (view()->buffer())
+                       if (lyx_view_->buffer())
                                // cancel any selection
                                dispatch(FuncRequest(LFUN_MARK_OFF));
                        setMessage(from_ascii(N_("Cancel")));
@@ -900,15 +920,17 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                // --- Menus -----------------------------------------------
                case LFUN_BUFFER_NEW:
                        menuNew(argument, false);
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_BUFFER_NEW_TEMPLATE:
                        menuNew(argument, true);
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_BUFFER_CLOSE:
                        closeBuffer();
-                       view()->update();
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_BUFFER_WRITE:
@@ -957,7 +979,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
 
                case LFUN_BUFFER_RELOAD: {
                        BOOST_ASSERT(lyx_view_ && lyx_view_->buffer());
-                       docstring const file = makeDisplayPath(view()->buffer()->fileName(), 20);
+                       docstring const file = makeDisplayPath(lyx_view_->buffer()->fileName(), 20);
                        docstring 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?"),
@@ -1184,7 +1206,13 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        }
                        lyx_view_->message(bformat(_("Opening help file %1$s..."),
                                makeDisplayPath(fname.absFilename())));
-                       lyx_view_->loadLyXFile(fname, false);
+                       Buffer * buf = lyx_view_->loadLyXFile(fname, false);
+                       if (buf) {
+                               updateLabels(*buf);
+                               lyx_view_->setBuffer(buf);
+                               lyx_view_->showErrorList("Parse");
+                       }
+                       updateFlags = Update::None;
                        break;
                }
 
@@ -1238,39 +1266,41 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                case LFUN_BUFFER_SWITCH:
                        BOOST_ASSERT(lyx_view_);
                        lyx_view_->setBuffer(theBufferList().getBuffer(argument));
-                       updateFlags = Update::Force;
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_BUFFER_NEXT:
                        BOOST_ASSERT(lyx_view_);
-                       lyx_view_->setBuffer(theBufferList().next(view()->buffer()));
-                       updateFlags = Update::Force;
+                       lyx_view_->setBuffer(theBufferList().next(lyx_view_->buffer()));
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_BUFFER_PREVIOUS:
                        BOOST_ASSERT(lyx_view_);
-                       lyx_view_->setBuffer(theBufferList().previous(view()->buffer()));
-                       updateFlags = Update::Force;
+                       lyx_view_->setBuffer(theBufferList().previous(lyx_view_->buffer()));
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_FILE_NEW:
                        BOOST_ASSERT(lyx_view_);
-                       newFile(view(), argument);
+                       newFile(*lyx_view_, argument);
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_FILE_OPEN:
                        BOOST_ASSERT(lyx_view_);
                        open(argument);
+                       updateFlags = Update::None;
                        break;
 
                case LFUN_DROP_LAYOUTS_CHOICE:
                        BOOST_ASSERT(lyx_view_);
-                       lyx_view_->getToolbars().openLayoutList();
+                       lyx_view_->openLayoutList();
                        break;
 
                case LFUN_MENU_OPEN:
                        BOOST_ASSERT(lyx_view_);
-                       lyx_view_->getMenubar().openByName(from_utf8(argument));
+                       lyx_view_->openMenu(from_utf8(argument));
                        break;
 
                // --- lyxserver commands ----------------------------
@@ -1293,24 +1323,35 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        int row;
                        istringstream is(argument);
                        is >> file_name >> row;
-                       if (prefixIs(file_name, package().temp_dir().absFilename())) {
+                       Buffer * buf = 0;
+                       bool loaded = false;
+                       if (prefixIs(file_name, package().temp_dir().absFilename()))
                                // Needed by inverse dvi search. If it is a file
                                // in tmpdir, call the apropriated function
-                               lyx_view_->setBuffer(theBufferList().getBufferFromTmp(file_name));
-                       else {
+                               buf = theBufferList().getBufferFromTmp(file_name);
+                       else {
                                // Must replace extension of the file to be .lyx
                                // and get full path
                                FileName const s = fileSearch(string(), changeExtension(file_name, ".lyx"), "lyx");
                                // Either change buffer or load the file
-                               if (theBufferList().exists(s.absFilename())) {
-                                       lyx_view_->setBuffer(theBufferList().getBuffer(s.absFilename()));
-                               } else {
-                                       lyx_view_->loadLyXFile(s);
+                               if (theBufferList().exists(s.absFilename()))
+                                       buf = theBufferList().getBuffer(s.absFilename());
+                               else {
+                                       buf = lyx_view_->loadLyXFile(s);
+                                       loaded = true;
                                }
                        }
 
-                       view()->setCursorFromRow(row);
+                       if (!buf) {
+                               updateFlags = Update::None;
+                               break;
+                       }
 
+                       updateLabels(*buf);
+                       lyx_view_->setBuffer(buf);
+                       view()->setCursorFromRow(row);
+                       if (loaded)
+                               lyx_view_->showErrorList("Parse");
                        updateFlags = Update::FitCursor;
                        break;
                }
@@ -1469,36 +1510,33 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                }
 
                case LFUN_BUFFER_CHILD_OPEN: {
-                       // takes an optional argument, "|bool", at the end
-                       // indicating whether this file is being opened automatically
-                       // by LyX itself, in which case we will not want to switch
-                       // buffers after opening. The default is false, so in practice
-                       // it is used only when true.
-                       BOOST_ASSERT(lyx_view_);
-                       int const arglength = argument.length();
-                       FileName filename;
-                       bool autoOpen = false;
-                       if (argument.substr(arglength - 5, 5) == "|true") {
-                               autoOpen = true;
-                               filename = makeAbsPath(argument.substr(0, arglength - 5), 
-                                       lyx_view_->buffer()->filePath());
-                       } else if (argument.substr(arglength - 6, 6) == "|false") {
-                               filename = makeAbsPath(argument.substr(0, arglength - 6), 
-                                       lyx_view_->buffer()->filePath());
-                       } else filename = 
-                               makeAbsPath(argument, lyx_view_->buffer()->filePath());
+                       BOOST_ASSERT(lyx_view_ && lyx_view_->buffer());
+                       Buffer * parent = lyx_view_->buffer();
+                       FileName filename = makeAbsPath(argument, parent->filePath());
                        view()->saveBookmark(false);
+                       Buffer * child = 0;
+                       bool parsed = false;
                        if (theBufferList().exists(filename.absFilename())) {
-                               Buffer * buf = theBufferList().getBuffer(filename.absFilename());
-                               if (!autoOpen)
-                                       lyx_view_->setBuffer(buf, true);
-                               else
-                                       buf->setParentName(lyx_view_->buffer()->fileName());
-                       } else
-                               lyx_view_->loadLyXFile(filename, true, true, autoOpen);
+                               child = theBufferList().getBuffer(filename.absFilename());
+                       } else {
+                               setMessage(bformat(_("Opening child document %1$s..."),
+                                       makeDisplayPath(filename.absFilename())));
+                               child = lyx_view_->loadLyXFile(filename, true);
+                               parsed = true;
+                       }
+                       if (child) {
+                               // Set the parent name of the child document.
+                               // This makes insertion of citations and references in the child work,
+                               // when the target is in the parent or another child document.
+                               child->setParentName(parent->fileName());
+                               updateLabels(*child->getMasterBuffer());
+                               lyx_view_->setBuffer(child);
+                               if (parsed)
+                                       lyx_view_->showErrorList("Parse");
+                       }
 
                        // If a screen update is required (in case where auto_open is false), 
-                       // loadLyXFile() would have taken care of it already. Otherwise we shall 
+                       // setBuffer() would have taken care of it already. Otherwise we shall 
                        // reset the update flag because it can cause a circular problem.
                        // See bug 3970.
                        updateFlags = Update::None;
@@ -1511,22 +1549,22 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        break;
 
                case LFUN_KEYMAP_OFF:
-                       BOOST_ASSERT(lyx_view_);
+                       BOOST_ASSERT(lyx_view_ && lyx_view_->view());
                        lyx_view_->view()->getIntl().keyMapOn(false);
                        break;
 
                case LFUN_KEYMAP_PRIMARY:
-                       BOOST_ASSERT(lyx_view_);
+                       BOOST_ASSERT(lyx_view_ && lyx_view_->view());
                        lyx_view_->view()->getIntl().keyMapPrim();
                        break;
 
                case LFUN_KEYMAP_SECONDARY:
-                       BOOST_ASSERT(lyx_view_);
+                       BOOST_ASSERT(lyx_view_ && lyx_view_->view());
                        lyx_view_->view()->getIntl().keyMapSec();
                        break;
 
                case LFUN_KEYMAP_TOGGLE:
-                       BOOST_ASSERT(lyx_view_);
+                       BOOST_ASSERT(lyx_view_ && lyx_view_->view());
                        lyx_view_->view()->getIntl().toggleKeyMap();
                        break;
 
@@ -1598,9 +1636,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        theApp()->updateColor(lcolor.getFromLyXName(lyx_name));
 
                        if (graphicsbg_changed) {
-#ifdef WITH_WARNINGS
-#warning FIXME!! The graphics cache no longer has a changeDisplay method.
-#endif
+                               // FIXME: The graphics cache no longer has a changeDisplay method.
 #if 0
                                graphics::GCache::get().changeDisplay(true);
 #endif
@@ -1711,33 +1747,66 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
 
                case LFUN_BUFFER_PARAMS_APPLY: {
                        BOOST_ASSERT(lyx_view_);
-                       biblio::CiteEngine const engine =
-                               lyx_view_->buffer()->params().getEngine();
+                       biblio::CiteEngine const oldEngine =
+                                       lyx_view_->buffer()->params().getEngine();
+                       
+                       Buffer * buffer = lyx_view_->buffer();
 
+                       TextClass_ptr oldClass = buffer->params().getTextClass_ptr();
+                       recordUndoFullDocument(view());
+                       
                        istringstream ss(argument);
                        Lexer lex(0,0);
                        lex.setStream(ss);
-                       int const unknown_tokens =
-                               lyx_view_->buffer()->readHeader(lex);
+                       int const unknown_tokens = buffer->readHeader(lex);
 
                        if (unknown_tokens != 0) {
                                lyxerr << "Warning in LFUN_BUFFER_PARAMS_APPLY!\n"
-                                      << unknown_tokens << " unknown token"
-                                      << (unknown_tokens == 1 ? "" : "s")
-                                      << endl;
+                                               << unknown_tokens << " unknown token"
+                                               << (unknown_tokens == 1 ? "" : "s")
+                                               << endl;
                        }
-                       if (engine == lyx_view_->buffer()->params().getEngine())
-                               break;
-
-                       Cursor & cur = view()->cursor();
-                       FuncRequest fr(LFUN_INSET_REFRESH);
-
-                       Inset & inset = lyx_view_->buffer()->inset();
-                       InsetIterator it  = inset_iterator_begin(inset);
-                       InsetIterator const end = inset_iterator_end(inset);
-                       for (; it != end; ++it)
-                               if (it->lyxCode() == Inset::CITE_CODE)
-                                       it->dispatch(cur, fr);
+                       
+                       updateLayout(oldClass, buffer);
+                       
+                       biblio::CiteEngine const newEngine =
+                                       lyx_view_->buffer()->params().getEngine();
+                       
+                       if (oldEngine != newEngine) {
+                               Cursor & cur = view()->cursor();
+                               FuncRequest fr(LFUN_INSET_REFRESH);
+       
+                               Inset & inset = lyx_view_->buffer()->inset();
+                               InsetIterator it  = inset_iterator_begin(inset);
+                               InsetIterator const end = inset_iterator_end(inset);
+                               for (; it != end; ++it)
+                                       if (it->lyxCode() == Inset::CITE_CODE)
+                                               it->dispatch(cur, fr);
+                       }
+                       
+                       updateFlags = Update::Force | Update::FitCursor;
+                       break;
+               }
+               
+               case LFUN_LAYOUT_MODULES_CLEAR: {
+                       BOOST_ASSERT(lyx_view_);
+                       Buffer * buffer = lyx_view_->buffer();
+                       TextClass_ptr oldClass = buffer->params().getTextClass_ptr();
+                       recordUndoFullDocument(view());
+                       buffer->params().clearLayoutModules();
+                       updateLayout(oldClass, buffer);
+                       updateFlags = Update::Force | Update::FitCursor;
+                       break;
+               }
+               
+               case LFUN_LAYOUT_MODULE_ADD: {
+                       BOOST_ASSERT(lyx_view_);
+                       Buffer * buffer = lyx_view_->buffer();
+                       TextClass_ptr oldClass = buffer->params().getTextClass_ptr();
+                       recordUndoFullDocument(view());
+                       buffer->params().addLayoutModule(argument);
+                       updateLayout(oldClass, buffer);
+                       updateFlags = Update::Force | Update::FitCursor;
                        break;
                }
 
@@ -1745,10 +1814,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        BOOST_ASSERT(lyx_view_);
                        Buffer * buffer = lyx_view_->buffer();
 
-                       textclass_type const old_class =
-                               buffer->params().textclass;
-
-                       loadTextclass(argument);
+                       loadTextClass(argument);
 
                        std::pair<bool, textclass_type> const tc_pair =
                                textclasslist.numberOfClass(argument);
@@ -1756,30 +1822,36 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        if (!tc_pair.first)
                                break;
 
+                       textclass_type const old_class = buffer->params().getBaseClass();
                        textclass_type const new_class = tc_pair.second;
+
                        if (old_class == new_class)
                                // nothing to do
                                break;
 
-                       lyx_view_->message(_("Converting document to new document class..."));
+                       //Save the old, possibly modular, layout for use in conversion.
+                       TextClass_ptr oldClass = buffer->params().getTextClass_ptr();
                        recordUndoFullDocument(view());
-                       buffer->params().textclass = new_class;
-                       StableDocIterator backcur(view()->cursor());
-                       ErrorList & el = buffer->errorList("Class Switch");
-                       cap::switchBetweenClasses(
-                               old_class, new_class,
-                               static_cast<InsetText &>(buffer->inset()), el);
-
-                       view()->setCursor(backcur.asDocIterator(&(buffer->inset())));
-
-                       buffer->errors("Class Switch");
-                       updateLabels(*buffer);
+                       buffer->params().setBaseClass(new_class);
+                       updateLayout(oldClass, buffer);
+                       updateFlags = Update::Force | Update::FitCursor;
+                       break;
+               }
+               
+               case LFUN_LAYOUT_RELOAD: {
+                       BOOST_ASSERT(lyx_view_);
+                       Buffer * buffer = lyx_view_->buffer();
+                       TextClass_ptr oldClass = buffer->params().getTextClass_ptr();
+                       textclass_type const tc = buffer->params().getBaseClass();
+                       textclasslist.reset(tc);
+                       buffer->params().setBaseClass(tc);
+                       updateLayout(oldClass, buffer);
                        updateFlags = Update::Force | Update::FitCursor;
                        break;
                }
 
                case LFUN_TEXTCLASS_LOAD:
-                       loadTextclass(argument);
+                       loadTextClass(argument);
                        break;
 
                case LFUN_LYXRC_APPLY: {
@@ -1864,7 +1936,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                }
                }
 
-               if (lyx_view_ && view()->buffer()) {
+               if (lyx_view_ && lyx_view_->buffer()) {
                        // BufferView::update() updates the ViewMetricsInfo and
                        // also initializes the position cache for all insets in
                        // (at least partially) visible top-level paragraphs.
@@ -1873,7 +1945,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                                // Buffer::changed() signals that a repaint is needed.
                                // The frontend (WorkArea) knows which area to repaint
                                // thanks to the ViewMetricsInfo updated above.
-                               view()->buffer()->changed();
+                               lyx_view_->buffer()->changed();
                        }
 
                        lyx_view_->updateStatusBar();
@@ -1882,7 +1954,7 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                        if (flag.enabled()
                            && !lyxaction.funcHasFlag(action, LyXAction::NoBuffer)
                            && !lyxaction.funcHasFlag(action, LyXAction::ReadOnly))
-                               view()->buffer()->markDirty();
+                               lyx_view_->buffer()->markDirty();
 
                        //Do we have a selection?
                        theSelection().haveSelection(view()->cursor().selection());
@@ -1893,7 +1965,6 @@ void LyXFunc::dispatch(FuncRequest const & cmd)
                }
        }
        if (!quitting && lyx_view_) {
-               lyx_view_->updateMenubar();
                lyx_view_->updateToolbars();
                // Some messages may already be translated, so we cannot use _()
                sendDispatchMessage(translateIfPossible(getMessage()), cmd);
@@ -1954,7 +2025,7 @@ void LyXFunc::menuNew(string const & name, bool fromTemplate)
        string initpath = lyxrc.document_path;
        string filename(name);
 
-       if (view()->buffer()) {
+       if (lyx_view_->buffer()) {
                string const trypath = lyx_view_->buffer()->filePath();
                // If directory is writeable, use this as default.
                if (isDirWriteable(FileName(trypath)))
@@ -1996,10 +2067,8 @@ void LyXFunc::menuNew(string const & name, bool fromTemplate)
        }
 
        Buffer * const b = newFile(filename, templname, !name.empty());
-       if (b) {
-               updateLabels(*b);
+       if (b)
                lyx_view_->setBuffer(b);
-       }
 }
 
 
@@ -2007,7 +2076,7 @@ void LyXFunc::open(string const & fname)
 {
        string initpath = lyxrc.document_path;
 
-       if (view()->buffer()) {
+       if (lyx_view_->buffer()) {
                string const trypath = lyx_view_->buffer()->filePath();
                // If directory is writeable, use this as default.
                if (isDirWriteable(FileName(trypath)))
@@ -2059,7 +2128,11 @@ void LyXFunc::open(string const & fname)
        lyx_view_->message(bformat(_("Opening document %1$s..."), disp_fn));
 
        docstring str2;
-       if (lyx_view_->loadLyXFile(fullname)) {
+       Buffer * buf = lyx_view_->loadLyXFile(fullname);
+       if (buf) {
+               updateLabels(*buf);
+               lyx_view_->setBuffer(buf);
+               lyx_view_->showErrorList("Parse");
                str2 = bformat(_("Document %1$s opened."), disp_fn);
        } else {
                str2 = bformat(_("Could not open document %1$s"), disp_fn);
@@ -2080,7 +2153,7 @@ void LyXFunc::doImport(string const & argument)
        if (filename.empty()) {
                string initpath = lyxrc.document_path;
 
-               if (view()->buffer()) {
+               if (lyx_view_->buffer()) {
                        string const trypath = lyx_view_->buffer()->filePath();
                        // If directory is writeable, use this as default.
                        if (isDirWriteable(FileName(trypath)))
@@ -2168,8 +2241,19 @@ void LyXFunc::closeBuffer()
 void LyXFunc::reloadBuffer()
 {
        FileName filename(lyx_view_->buffer()->fileName());
+       docstring const disp_fn = makeDisplayPath(filename.absFilename());
+       docstring str;
        closeBuffer();
-       lyx_view_->loadLyXFile(filename);
+       Buffer * buf = lyx_view_->loadLyXFile(filename);
+       if (buf) {
+               updateLabels(*buf);
+               lyx_view_->setBuffer(buf);
+               lyx_view_->showErrorList("Parse");
+               str = bformat(_("Document %1$s reloaded."), disp_fn);
+       } else {
+               str = bformat(_("Could not reload document %1$s"), disp_fn);
+       }
+       lyx_view_->message(str);
 }
 
 // Each "lyx_view_" should have it's own message method. lyxview and
@@ -2202,7 +2286,8 @@ docstring const LyXFunc::viewStatusMessage()
        if (keyseq->length() > 0 && !keyseq->deleted())
                return keyseq->printOptions(true);
 
-       if (!view()->buffer())
+       BOOST_ASSERT(lyx_view_);
+       if (!lyx_view_->buffer())
                return _("Welcome to LyX!");
 
        return view()->cursor().currentState();
@@ -2222,6 +2307,24 @@ bool LyXFunc::wasMetaKey() const
 }
 
 
+void LyXFunc::updateLayout(TextClass_ptr const & oldlayout,
+                           Buffer * buffer)
+{
+       lyx_view_->message(_("Converting document to new document class..."));
+       
+       StableDocIterator backcur(view()->cursor());
+       ErrorList & el = buffer->errorList("Class Switch");
+       cap::switchBetweenClasses(
+                       oldlayout, buffer->params().getTextClass_ptr(),
+                       static_cast<InsetText &>(buffer->inset()), el);
+
+       view()->setCursor(backcur.asDocIterator(&(buffer->inset())));
+
+       buffer->errors("Class Switch");
+       updateLabels(*buffer);
+}
+
+
 namespace {
 
 void actOnUpdatedPrefs(LyXRC const & lyxrc_orig, LyXRC const & lyxrc_new)