]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView_pimpl.C
Remove the inset and view member functions from PreviewedInset.
[lyx.git] / src / BufferView_pimpl.C
index e1ccdc8b05c7e73281363937bfcdd6d8be598993..b7eafa9d9663223a7f89083ecc79dc97c681f35f 100644 (file)
@@ -1,20 +1,29 @@
 /**
  * \file BufferView_pimpl.C
- * Copyright 2002 the LyX Team
- * Read the file COPYING
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
+ * \author Asger Alstrup
+ * \author Alfredo Braustein
  * \author Lars Gullik Bjønnes
- * \author various
+ * \author Jean-Marc Lasgouttes
+ * \author Angus Leeming
+ * \author John Levon
+ * \author André Pönitz
+ * \author Dekel Tsur
+ * \author Jürgen Vigna
+ *
+ * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 
 #include "BufferView_pimpl.h"
-#include "bufferlist.h"
 #include "buffer.h"
 #include "buffer_funcs.h"
-#include "bufferview_funcs.h"
-#include "lfuns.h"
+#include "bufferlist.h"
+#include "bufferparams.h"
+#include "cursor.h"
 #include "debug.h"
 #include "factory.h"
 #include "FloatList.h"
 #include "gettext.h"
 #include "intl.h"
 #include "iterators.h"
-#include "Lsstream.h"
 #include "lyx_cb.h" // added for Dispatch functions
 #include "lyx_main.h"
 #include "lyxfind.h"
 #include "lyxfunc.h"
 #include "lyxtext.h"
 #include "lyxrc.h"
-#include "lyxrow.h"
 #include "lastfiles.h"
 #include "paragraph.h"
+#include "paragraph_funcs.h"
 #include "ParagraphParameters.h"
 #include "TextCache.h"
 #include "undo_funcs.h"
+#include "vspace.h"
 
 #include "insets/insetfloatlist.h"
-#include "insets/insetgraphics.h"
-#include "insets/insetinclude.h"
 #include "insets/insetref.h"
-#include "insets/insettext.h"
 
 #include "frontends/Alert.h"
 #include "frontends/Dialogs.h"
 #include "frontends/FileDialog.h"
 #include "frontends/LyXView.h"
 #include "frontends/LyXScreenFactory.h"
-#include "frontends/mouse_state.h"
 #include "frontends/screen.h"
 #include "frontends/WorkArea.h"
 #include "frontends/WorkAreaFactory.h"
 
-#include "mathed/formulabase.h"
-
 #include "graphics/Previews.h"
 
-#include "support/LAssert.h"
-#include "support/tostr.h"
+#include "mathed/formulabase.h"
+
 #include "support/filetools.h"
+#include "support/path_defines.h"
+#include "support/tostr.h"
 
 #include <boost/bind.hpp>
-#include <boost/signals/connection.hpp>
 
-#include <unistd.h>
-#include <sys/wait.h>
+using bv_funcs::bold;
+using bv_funcs::code;
+using bv_funcs::currentState;
+using bv_funcs::emph;
+using bv_funcs::fontSize;
+using bv_funcs::lang;
+using bv_funcs::noun;
+using bv_funcs::roman;
+using bv_funcs::sans;
+using bv_funcs::styleReset;
+using bv_funcs::underline;
 
+using lyx::pos_type;
+
+using lyx::support::AddPath;
+using lyx::support::bformat;
+using lyx::support::FileSearch;
+using lyx::support::IsDirWriteable;
+using lyx::support::MakeDisplayPath;
+using lyx::support::strToUnsignedInt;
+using lyx::support::system_lyxdir;
 
-using std::vector;
-using std::find_if;
-using std::find;
-using std::pair;
 using std::endl;
 using std::make_pair;
 using std::min;
+using std::string;
 
-using lyx::pos_type;
-using namespace lyx::support;
-using namespace bv_funcs;
 
 extern BufferList bufferlist;
 
@@ -187,9 +203,11 @@ bool BufferView::Pimpl::loadLyXFile(string const & filename, bool tolastfiles)
        // get absolute path of file and add ".lyx" to the filename if
        // necessary
        string s = FileSearch(string(), filename, "lyx");
-       if (s.empty()) {
+
+       bool const found = !s.empty();
+
+       if (!found)
                s = filename;
-       }
 
        // file already open?
        if (bufferlist.exists(s)) {
@@ -210,29 +228,35 @@ bool BufferView::Pimpl::loadLyXFile(string const & filename, bool tolastfiles)
                        // Fall through to new load. (Asger)
                }
        }
-       Buffer * b = bufferlist.newBuffer(s);
 
-       connectBuffer(*b);
+       Buffer * b;
 
-       if (! ::loadLyXFile(b, s)) {
-               bufferlist.release(b);
+       if (found) {
+               b = bufferlist.newBuffer(s);
+               connectBuffer(*b);
+               if (!::loadLyXFile(b, s)) {
+                       bufferlist.release(b);
+                       return false;
+               }
+       } else {
                string text = bformat(_("The document %1$s does not yet "
                                        "exist.\n\nDo you want to create "
                                        "a new document?"), s);
                int const ret = Alert::prompt(_("Create new document?"),
                         text, 0, 1, _("&Create"), _("Cancel"));
 
-               if (ret != 0)
+               if (ret == 0)
+                       b = ::newFile(s, string(), true);
+               else
                        return false;
        }
 
        buffer(b);
+       bv_->showErrorList(_("Parse"));
 
        if (tolastfiles)
                lastfiles->newFile(b->fileName());
 
-       bv_->showErrorList(_("Parse"));
-
        return true;
 }
 
@@ -255,6 +279,18 @@ Painter & BufferView::Pimpl::painter() const
 }
 
 
+void BufferView::Pimpl::top_y(int y)
+{
+       top_y_ = y;
+}
+
+
+int BufferView::Pimpl::top_y() const
+{
+       return top_y_;
+}
+
+
 void BufferView::Pimpl::buffer(Buffer * b)
 {
        lyxerr[Debug::INFO] << "Setting buffer in BufferView ("
@@ -275,26 +311,26 @@ void BufferView::Pimpl::buffer(Buffer * b)
        // set current buffer
        buffer_ = b;
 
+       top_y_ = 0;
+
        // if we're quitting lyx, don't bother updating stuff
        if (quitting)
                return;
 
        // if we are closing the buffer, use the first buffer as current
-       if (!buffer_) {
+       if (!buffer_)
                buffer_ = bufferlist.first();
-       }
 
        if (buffer_) {
                lyxerr[Debug::INFO] << "Buffer addr: " << buffer_ << endl;
                connectBuffer(*buffer_);
 
                // If we don't have a text object for this, we make one
-               if (bv_->text == 0) {
+               if (bv_->text == 0)
                        resizeCurrentBuffer();
-               }
 
                // FIXME: needed when ?
-               bv_->text->top_y(screen().topCursorVisible(bv_->text));
+               top_y(screen().topCursorVisible(bv_->text));
 
                // Buffer-dependent dialogs should be updated or
                // hidden. This should go here because some dialogs (eg ToC)
@@ -312,7 +348,7 @@ void BufferView::Pimpl::buffer(Buffer * b)
                textcache.clear();
        }
 
-       repaint();
+       update();
        updateScrollbar();
        owner_->updateMenubar();
        owner_->updateToolbar();
@@ -322,7 +358,7 @@ void BufferView::Pimpl::buffer(Buffer * b)
        if (buffer_) {
                // Don't forget to update the Layout
                string const layoutname =
-                       bv_->text->cursor.par()->layout()->name();
+                       bv_->text->cursorPar()->layout()->name();
                owner_->setLayout(layoutname);
        }
 
@@ -342,7 +378,7 @@ bool BufferView::Pimpl::fitCursor()
                ret = screen().fitCursor(bv_->text, bv_);
        }
 
-       dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
+       //dispatch(FuncRequest(LFUN_PARAGRAPH_UPDATE));
 
        // We need to always update, in case we did a
        // paste and we stayed anchored to a row, but
@@ -359,18 +395,18 @@ void BufferView::Pimpl::redoCurrentBuffer()
                resizeCurrentBuffer();
                updateScrollbar();
                owner_->updateLayoutChoice();
-               repaint();
+               update();
        }
 }
 
 
-int BufferView::Pimpl::resizeCurrentBuffer()
+void BufferView::Pimpl::resizeCurrentBuffer()
 {
        lyxerr[Debug::INFO] << "resizeCurrentBuffer" << endl;
 
-       ParagraphList::iterator par;
-       ParagraphList::iterator selstartpar;
-       ParagraphList::iterator selendpar;
+       int par = -1;
+       int selstartpar = -1;
+       int selendpar = -1;
        UpdatableInset * the_locking_inset = 0;
 
        pos_type pos = 0;
@@ -393,39 +429,35 @@ int BufferView::Pimpl::resizeCurrentBuffer()
                selection = bv_->text->selection.set();
                mark_set = bv_->text->selection.mark();
                the_locking_inset = bv_->theLockingInset();
-               resizeInsets(bv_);
                bv_->text->fullRebreak();
                update();
        } else {
-               lyxerr << "text not available!\n";
+               lyxerr << "text not available!" << endl;
                // See if we have a text in TextCache that fits
                // the new buffer_ with the correct width.
                bv_->text = textcache.findFit(buffer_, workarea().workWidth());
                if (bv_->text) {
-                       lyxerr << "text in cache!\n";
+                       lyxerr << "text in cache!" << endl;
                        if (lyxerr.debugging()) {
-                               lyxerr << "Found a LyXText that fits:\n";
+                               lyxerr << "Found a LyXText that fits:" << endl;
                                textcache.show(lyxerr, make_pair(buffer_, make_pair(workarea().workWidth(), bv_->text)));
                        }
                        // Set the owner of the newly found text
                        //      bv_->text->owner(bv_);
                        if (lyxerr.debugging())
                                textcache.show(lyxerr, "resizeCurrentBuffer");
-
-                       resizeInsets(bv_);
                } else {
-                       lyxerr << "no text in cache!\n";
-                       bv_->text = new LyXText(bv_);
-                       resizeInsets(bv_);
+                       lyxerr << "no text in cache!" << endl;
+                       bv_->text = new LyXText(bv_, 0, false, bv_->buffer()->paragraphs());
                        bv_->text->init(bv_);
                }
-
-               par = bv_->text->ownerParagraphs().end();
-               selstartpar = bv_->text->ownerParagraphs().end();
-               selendpar = bv_->text->ownerParagraphs().end();
        }
 
-       if (par != bv_->text->ownerParagraphs().end()) {
+#warning does not help much
+       //bv_->text->redoParagraphs(bv_->text->ownerParagraphs().begin(),
+       //      bv_->text->ownerParagraphs().end());
+
+       if (par != -1) {
                bv_->text->selection.set(true);
                // At this point just to avoid the Delete-Empty-Paragraph-
                // Mechanism when setting the cursor.
@@ -445,7 +477,7 @@ int BufferView::Pimpl::resizeCurrentBuffer()
                bv_->theLockingInset(the_locking_inset);
        }
 
-       bv_->text->top_y(screen().topCursorVisible(bv_->text));
+       top_y(screen().topCursorVisible(bv_->text));
 
        switchKeyMap();
        owner_->busy(false);
@@ -454,15 +486,6 @@ int BufferView::Pimpl::resizeCurrentBuffer()
        owner_->clearMessage();
 
        updateScrollbar();
-
-       return 0;
-}
-
-
-void BufferView::Pimpl::repaint()
-{
-       // Regenerate the screen.
-       screen().redraw(bv_->text, bv_);
 }
 
 
@@ -477,9 +500,9 @@ void BufferView::Pimpl::updateScrollbar()
        LyXText const & t = *bv_->text;
 
        lyxerr[Debug::GUI] << "Updating scrollbar: h " << t.height << ", top_y() "
-               << t.top_y() << ", default height " << defaultRowHeight() << endl;
+               << top_y() << ", default height " << defaultRowHeight() << endl;
 
-       workarea().setScrollbarParams(t.height, t.top_y(), defaultRowHeight());
+       workarea().setScrollbarParams(t.height, top_y(), defaultRowHeight());
 }
 
 
@@ -492,21 +515,21 @@ void BufferView::Pimpl::scrollDocView(int value)
 
        screen().hideCursor();
 
-       screen().draw(bv_->text, bv_, value);
+       top_y(value);
+       screen().redraw(*bv_);
 
        if (!lyxrc.cursor_follows_scrollbar)
                return;
 
-       LyXText * vbt = bv_->text;
-
        int const height = defaultRowHeight();
-       int const first = static_cast<int>((bv_->text->top_y() + height));
-       int const last = static_cast<int>((bv_->text->top_y() + workarea().workHeight() - height));
+       int const first = top_y() + height;
+       int const last = top_y() + workarea().workHeight() - height;
 
-       if (vbt->cursor.y() < first)
-               vbt->setCursorFromCoordinates(0, first);
-       else if (vbt->cursor.y() > last)
-               vbt->setCursorFromCoordinates(0, last);
+       LyXText * text = bv_->text;
+       if (text->cursor.y() < first)
+               text->setCursorFromCoordinates(0, first);
+       else if (text->cursor.y() > last)
+               text->setCursorFromCoordinates(0, last);
 
        owner_->updateLayoutChoice();
 }
@@ -522,7 +545,7 @@ void BufferView::Pimpl::scroll(int lines)
        int const line_height = defaultRowHeight();
 
        // The new absolute coordinate
-       int new_top_y = t->top_y() + lines * line_height;
+       int new_top_y = top_y() + lines * line_height;
 
        // Restrict to a valid value
        new_top_y = std::min(t->height - 4 * line_height, new_top_y);
@@ -531,7 +554,7 @@ void BufferView::Pimpl::scroll(int lines)
        scrollDocView(new_top_y);
 
        // Update the scrollbar.
-       workarea().setScrollbarParams(t->height, t->top_y(), defaultRowHeight());
+       workarea().setScrollbarParams(t->height, top_y(), defaultRowHeight());
 }
 
 
@@ -569,7 +592,7 @@ void BufferView::Pimpl::selectionRequested()
                 text->selection.end != bv_->text->xsel_cache.end))
        {
                bv_->text->xsel_cache = text->selection;
-               sel = text->selectionAsString(bv_->buffer(), false);
+               sel = text->selectionAsString(*bv_->buffer(), false);
        } else if (!text->selection.set()) {
                sel = string();
                bv_->text->xsel_cache.set(false);
@@ -584,7 +607,6 @@ void BufferView::Pimpl::selectionLost()
 {
        if (available()) {
                screen().hideCursor();
-               toggleSelection();
                bv_->getLyXText()->clearSelection();
                bv_->text->xsel_cache.set(false);
        }
@@ -617,69 +639,27 @@ void BufferView::Pimpl::workAreaResize()
                        if (lyxerr.debugging())
                                textcache.show(lyxerr, "Expose delete all");
                        textcache.clear();
-                       // FIXME: this is already done in resizeCurrentBuffer() ??
-                       resizeInsets(bv_);
-               } else if (heightChange) {
-                       // fitCursor() ensures we don't jump back
-                       // to the start of the document on vertical
-                       // resize
-                       fitCursor();
                }
        }
 
-       if (widthChange || heightChange) {
-               repaint();
-       }
+       if (widthChange || heightChange)
+               update();
 
        // always make sure that the scrollbar is sane.
        updateScrollbar();
        owner_->updateLayoutChoice();
-       return;
 }
 
 
 void BufferView::Pimpl::update()
 {
-       if (!bv_->theLockingInset() || !bv_->theLockingInset()->nodraw()) {
-               screen().update(*bv_);
-               bv_->text->clearPaint();
-       }
-}
-
-
-void BufferView::Pimpl::update(LyXText * text, BufferView::UpdateCodes f)
-{
-       if (!text->selection.set() && (f & SELECT)) {
-               text->selection.cursor = text->cursor;
-       }
-
-       text->partialRebreak();
-
-       if (text->inset_owner) {
-               text->inset_owner->setUpdateStatus(InsetText::NONE);
-               updateInset(text->inset_owner);
-       } else {
-               update();
-       }
-}
-
-
-void BufferView::Pimpl::update(BufferView::UpdateCodes f)
-{
-       LyXText * text = bv_->text;
-
-       if (!text->selection.set() && (f & SELECT)) {
-               text->selection.cursor = text->cursor;
-       }
-
-       text->partialRebreak();
-
-       if (text->inset_owner) {
-               text->inset_owner->setUpdateStatus(InsetText::NONE);
-               updateInset(text->inset_owner);
-       } else {
-               update();
+       //lyxerr << "BufferView::update()" << endl;
+       // fix cursor coordinate cache in case something went wrong
+       if (bv_->getLyXText()) {
+               // check needed to survive LyX startup
+               bv_->getLyXText()->redoCursor();
        }
+       screen().redraw(*bv_);
 }
 
 
@@ -707,22 +687,21 @@ bool BufferView::Pimpl::available() const
 
 Change const BufferView::Pimpl::getCurrentChange()
 {
-       if (!bv_->buffer()->params.tracking_changes)
+       if (!bv_->buffer()->params().tracking_changes)
                return Change(Change::UNCHANGED);
 
-       LyXText * t(bv_->getLyXText());
+       LyXText * text = bv_->getLyXText();
 
-       if (!t->selection.set())
+       if (!text->selection.set())
                return Change(Change::UNCHANGED);
 
-       LyXCursor const & cur(t->selection.start);
-       return cur.par()->lookupChangeFull(cur.pos());
+       return text->getPar(text->selection.start)
+               ->lookupChangeFull(text->selection.start.pos());
 }
 
 
 void BufferView::Pimpl::beforeChange(LyXText * text)
 {
-       toggleSelection();
        text->clearSelection();
 }
 
@@ -732,7 +711,7 @@ void BufferView::Pimpl::savePosition(unsigned int i)
        if (i >= saved_positions_num)
                return;
        saved_positions[i] = Position(buffer_->fileName(),
-                                     bv_->text->cursor.par()->id(),
+                                     bv_->text->cursorPar()->id(),
                                      bv_->text->cursor.pos());
        if (i > 0)
                owner_->message(bformat(_("Saved bookmark %1$s"), tostr(i)));
@@ -749,14 +728,14 @@ void BufferView::Pimpl::restorePosition(unsigned int i)
        beforeChange(bv_->text);
 
        if (fname != buffer_->fileName()) {
-               Buffer * b;
+               Buffer * b = 0;
                if (bufferlist.exists(fname))
                        b = bufferlist.getBuffer(fname);
                else {
                        b = bufferlist.newBuffer(fname);
                        ::loadLyXFile(b, fname); // don't ask, just load it
                }
-               if (b != 0)
+               if (b)
                        buffer(b);
        }
 
@@ -767,7 +746,7 @@ void BufferView::Pimpl::restorePosition(unsigned int i)
        bv_->text->setCursor(par.pit(),
                             min(par->size(), saved_positions[i].par_pos));
 
-       update(BufferView::SELECT);
+       update();
        if (i > 0)
                owner_->message(bformat(_("Moved to bookmark %1$s"), tostr(i)));
 }
@@ -811,31 +790,13 @@ void BufferView::Pimpl::insetUnlock()
 }
 
 
-void BufferView::Pimpl::toggleSelection(bool b)
-{
-       if (bv_->theLockingInset())
-               bv_->theLockingInset()->toggleSelection(bv_, b);
-       screen().toggleSelection(bv_->text, bv_, b);
-}
-
-
-void BufferView::Pimpl::toggleToggle()
-{
-       screen().toggleToggle(bv_->text, bv_);
-}
-
-
 void BufferView::Pimpl::center()
 {
-       LyXText * t = bv_->text;
+       LyXText * text = bv_->text;
 
-       beforeChange(t);
+       beforeChange(text);
        int const half_height = workarea().workHeight() / 2;
-       int new_y = 0;
-
-       if (t->cursor.y() > half_height) {
-               new_y = t->cursor.y() - half_height;
-       }
+       int new_y = std::max(0, text->cursor.y() - half_height);
 
        // FIXME: look at this comment again ...
 
@@ -848,9 +809,9 @@ void BufferView::Pimpl::center()
        // and also might have moved top_y() must make sure to call
        // updateScrollbar() currently. Never mind that this is a
        // pretty obfuscated way of updating t->top_y()
-       screen().draw(t, bv_, new_y);
-
-       update(BufferView::SELECT);
+       top_y(new_y);
+       //screen().draw();
+       update();
 }
 
 
@@ -872,31 +833,31 @@ InsetOld * BufferView::Pimpl::getInsetByCode(InsetOld::Code code)
        LyXCursor cursor = bv_->getLyXText()->cursor;
        Buffer::inset_iterator it =
                find_if(Buffer::inset_iterator(
-                       cursor.par(), cursor.pos()),
+                       cursorPar(), cursor.pos()),
                        buffer_->inset_iterator_end(),
                        lyx::compare_memfun(&Inset::lyxCode, code));
        return it != buffer_->inset_iterator_end() ? (*it) : 0;
 #else
        // Ok, this is a little bit too brute force but it
-       // should work for now. Better infrastructure is comming. (Lgb)
+       // should work for now. Better infrastructure is coming. (Lgb)
 
        Buffer * b = bv_->buffer();
-       LyXCursor cursor = bv_->getLyXText()->cursor;
+       LyXText * text =  bv_->getLyXText();
 
        Buffer::inset_iterator beg = b->inset_iterator_begin();
        Buffer::inset_iterator end = b->inset_iterator_end();
 
-       bool cursor_par_seen = false;
+       bool cursorPar_seen = false;
 
        for (; beg != end; ++beg) {
-               if (beg.getPar() == cursor.par()) {
-                       cursor_par_seen = true;
+               if (beg.getPar() == text->cursorPar()) {
+                       cursorPar_seen = true;
                }
-               if (cursor_par_seen) {
-                       if (beg.getPar() == cursor.par()
-                           && beg.getPos() >= cursor.pos()) {
+               if (cursorPar_seen) {
+                       if (beg.getPar() == text->cursorPar()
+                           && beg.getPos() >= text->cursor.pos()) {
                                break;
-                       } else if (beg.getPar() != cursor.par()) {
+                       } else if (beg.getPar() != text->cursorPar()) {
                                break;
                        }
                }
@@ -935,7 +896,7 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
                        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,
@@ -971,19 +932,19 @@ void BufferView::Pimpl::MenuInsertLyXFile(string const & filen)
 void BufferView::Pimpl::trackChanges()
 {
        Buffer * buf(bv_->buffer());
-       bool const tracking(buf->params.tracking_changes);
+       bool const tracking(buf->params().tracking_changes);
 
        if (!tracking) {
                ParIterator const end = buf->par_iterator_end();
                for (ParIterator it = buf->par_iterator_begin(); it != end; ++it)
                        it->trackChanges();
-               buf->params.tracking_changes = true;
+               buf->params().tracking_changes = true;
 
                // we cannot allow undos beyond the freeze point
-               buf->undostack.clear();
+               buf->undostack().clear();
        } else {
-               update(BufferView::SELECT);
-               bv_->text->setCursor(buf->paragraphs.begin(), 0);
+               update();
+               bv_->text->setCursor(0, 0);
 #warning changes FIXME
                //moveCursorUpdate(false);
 
@@ -996,44 +957,56 @@ void BufferView::Pimpl::trackChanges()
                ParIterator const end = buf->par_iterator_end();
                for (ParIterator it = buf->par_iterator_begin(); it != end; ++it)
                        it->untrackChanges();
-               buf->params.tracking_changes = false;
+               buf->params().tracking_changes = false;
        }
 
-       buf->redostack.clear();
+       buf->redostack().clear();
 }
 
 
-// Doesn't go through lyxfunc, so we need to update the
-// layout choice etc. ourselves
-bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & ev_in)
+bool BufferView::Pimpl::workAreaDispatch(FuncRequest const & ev)
 {
-       // e.g. Qt mouse press when no buffer
-       if (!available())
-               return false;
+       switch (ev.action) {
+       case LFUN_MOUSE_PRESS:
+       case LFUN_MOUSE_MOTION:
+       case LFUN_MOUSE_RELEASE:
+       case LFUN_MOUSE_DOUBLE:
+       case LFUN_MOUSE_TRIPLE:
+       {
+               // We pass those directly to the Bufferview, since
+               // otherwise selection handling breaks down
 
-       screen().hideCursor();
+               // Doesn't go through lyxfunc, so we need to update
+               // the layout choice etc. ourselves
 
-       // Make sure that the cached BufferView is correct.
-       FuncRequest ev = ev_in;
-       ev.setView(bv_);
+               // e.g. Qt mouse press when no buffer
+               if (!available())
+                       return false;
 
-       bool const res = dispatch(ev);
+               screen().hideCursor();
 
-       // see workAreaKeyPress
-       cursor_timeout.restart();
-       screen().showCursor(*bv_);
+               bool const res = dispatch(ev);
+
+               // see workAreaKeyPress
+               cursor_timeout.restart();
+               screen().showCursor(*bv_);
 
-       // FIXME: we should skip these when selecting
-       bv_->owner()->updateLayoutChoice();
-       bv_->owner()->updateToolbar();
-       bv_->fitCursor();
+               // FIXME: we should skip these when selecting
+               owner_->updateLayoutChoice();
+               owner_->updateToolbar();
+               fitCursor();
 
-       // slight hack: this is only called currently when
-       // we clicked somewhere, so we force through the display
-       // of the new status here.
-       bv_->owner()->clearMessage();
+               // slight hack: this is only called currently when we
+               // clicked somewhere, so we force through the display
+               // of the new status here.
+               owner_->clearMessage();
 
-       return res;
+               return res;
+       }
+       default:
+               owner_->dispatch(ev);
+               return true;
+       }
 }
 
 
@@ -1051,7 +1024,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                << " button[" << ev.button() << ']'
                << endl;
 
-       LyXTextClass const & tclass = buffer_->params.getLyXTextClass();
+       LyXTextClass const & tclass = buffer_->params().getLyXTextClass();
 
        switch (ev.action) {
 
@@ -1188,7 +1161,7 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                        owner_->getLyXFunc().handleKeyFunc(ev.action);
                        owner_->getIntl().getTransManager()
                                .TranslateAndInsert(ev.argument[0], bv_->getLyXText());
-                       update(bv_->getLyXText(), BufferView::SELECT);
+                       update();
                }
                break;
 
@@ -1228,10 +1201,8 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                        if (name == "bibitem") {
                                // We need to do a redraw because the maximum
                                // InsetBibitem width could have changed
-#warning please check you mean repaint() not update(),
-#warning and whether the repaint() is needed at all
-                               bv_->repaint();
-                               bv_->fitCursor();
+#warning check whether the update() is needed at all
+                               bv_->update();
                        }
                } else {
                        delete inset;
@@ -1251,12 +1222,8 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                break;
 
        case LFUN_LAYOUT_PARAGRAPH: {
-               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
-               if (!par)
-                       break;
-
                string data;
-               params2string(*par, data);
+               params2string(*bv_->getLyXText()->cursorPar(), data);
 
                data = "show\n" + data;
                bv_->owner()->getDialogs().show("paragraph", data);
@@ -1266,15 +1233,13 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
        case LFUN_PARAGRAPH_UPDATE: {
                if (!bv_->owner()->getDialogs().visible("paragraph"))
                        break;
-               Paragraph const * par = &*bv_->getLyXText()->cursor.par();
-               if (!par)
-                       break;
+               Paragraph const & par = *bv_->getLyXText()->cursorPar();
 
                string data;
-               params2string(*par, data);
+               params2string(par, data);
 
                // Will the paragraph accept changes from the dialog?
-               InsetOld * const inset = par->inInset();
+               InsetOld * const inset = par.inInset();
                bool const accept =
                        !(inset && inset->forceDefaultParagraphs(inset));
 
@@ -1292,14 +1257,14 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                string arg = ev.argument;
 
                if (arg.empty()) {
-                       arg = bv_->getLyXText()->selectionAsString(buffer_,
+                       arg = bv_->getLyXText()->selectionAsString(*buffer_,
                                                                   false);
 
                        // FIXME
                        if (arg.size() > 100 || arg.empty()) {
                                // Get word or selection
                                bv_->getLyXText()->selectWordWhenUnderCursor(lyx::WHOLE_WORD);
-                               arg = bv_->getLyXText()->selectionAsString(buffer_, false);
+                               arg = bv_->getLyXText()->selectionAsString(*buffer_, false);
                                // FIXME: where is getLyXText()->unselect(bv_) ?
                        }
                }
@@ -1317,40 +1282,38 @@ bool BufferView::Pimpl::dispatch(FuncRequest const & ev_in)
                break;
 
        case LFUN_ACCEPT_ALL_CHANGES: {
-               update(BufferView::SELECT);
-               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
+               bv_->text->setCursor(0, 0);
 #warning FIXME changes
                //moveCursorUpdate(false);
 
-               while (lyx::find::findNextChange(bv_)) {
+               while (lyx::find::findNextChange(bv_))
                        bv_->getLyXText()->acceptChange();
-               }
-               update(BufferView::SELECT);
+
+               update();
                break;
        }
 
        case LFUN_REJECT_ALL_CHANGES: {
-               update(BufferView::SELECT);
-               bv_->text->setCursor(bv_->buffer()->paragraphs.begin(), 0);
+               bv_->text->setCursor(0, 0);
 #warning FIXME changes
                //moveCursorUpdate(false);
 
-               while (lyx::find::findNextChange(bv_)) {
+               while (lyx::find::findNextChange(bv_))
                        bv_->getLyXText()->rejectChange();
-               }
-               update(BufferView::SELECT);
+
+               update();
                break;
        }
 
        case LFUN_ACCEPT_CHANGE: {
                bv_->getLyXText()->acceptChange();
-               update(BufferView::SELECT);
+               update();
                break;
        }
 
        case LFUN_REJECT_CHANGE: {
                bv_->getLyXText()->rejectChange();
-               update(BufferView::SELECT);
+               update();
                break;
        }
 
@@ -1382,20 +1345,15 @@ bool BufferView::Pimpl::insertInset(InsetOld * inset, string const & lout)
 
        beforeChange(bv_->text);
        if (!lout.empty()) {
-               update(BufferView::SELECT);
-               bv_->text->breakParagraph(bv_->buffer()->paragraphs);
-               update(BufferView::SELECT);
+               bv_->text->breakParagraph(bv_->buffer()->paragraphs());
 
-               if (!bv_->text->cursor.par()->empty()) {
+               if (!bv_->text->cursorPar()->empty()) {
                        bv_->text->cursorLeft(bv_);
-
-                       bv_->text->breakParagraph(bv_->buffer()->paragraphs);
-                       update(BufferView::SELECT);
+                       bv_->text->breakParagraph(bv_->buffer()->paragraphs());
                }
 
                string lres = lout;
-               LyXTextClass const & tclass =
-                       buffer_->params.getLyXTextClass();
+               LyXTextClass const & tclass = buffer_->params().getLyXTextClass();
                bool hasLayout = tclass.hasLayout(lres);
                string lay = tclass.defaultLayoutName();
 
@@ -1416,57 +1374,68 @@ bool BufferView::Pimpl::insertInset(InsetOld * inset, string const & lout)
                                   LYX_ALIGN_LAYOUT,
                                   string(),
                                   0);
-               update(BufferView::SELECT);
        }
 
        bv_->text->insertInset(inset);
-       update(BufferView::SELECT);
+       update();
 
        unFreezeUndo();
        return true;
 }
 
 
-void BufferView::Pimpl::updateInset(InsetOld * inset)
+void BufferView::Pimpl::updateInset(InsetOld const * inset)
 {
-       if (!inset || !available())
+       if (!available())
                return;
 
-       // first check for locking insets
-       if (bv_->theLockingInset()) {
-               if (bv_->theLockingInset() == inset) {
-                       if (bv_->text->updateInset(inset)) {
-                               update();
-                               updateScrollbar();
-                               return;
-                       }
-               } else if (bv_->theLockingInset()->updateInsetInInset(bv_, inset)) {
-                       if (bv_->text->updateInset(bv_->theLockingInset())) {
-                               update();
-                               updateScrollbar();
-                               return;
+       bv_->text->redoParagraph(outerPar(*bv_->buffer(), inset));
+
+       // this should not be needed, but it is...
+       // bv_->text->redoParagraph(bv_->text->cursorPar());
+       // bv_->text->fullRebreak();
+
+       update();
+       updateScrollbar();
+}
+
+
+bool BufferView::Pimpl::ChangeInsets(InsetOld::Code code,
+                                    string const & from, string const & to)
+{
+       bool need_update = false;
+       LyXCursor cursor = bv_->text->cursor;
+       LyXCursor tmpcursor = cursor;
+       cursor.par(tmpcursor.par());
+       cursor.pos(tmpcursor.pos());
+
+       ParIterator end = bv_->buffer()->par_iterator_end();
+       for (ParIterator it = bv_->buffer()->par_iterator_begin();
+            it != end; ++it) {
+               bool changed_inset = false;
+               for (InsetList::iterator it2 = it->insetlist.begin();
+                    it2 != it->insetlist.end(); ++it2) {
+                       if (it2->inset->lyxCode() == code) {
+                               InsetCommand * inset = static_cast<InsetCommand *>(it2->inset);
+                               if (inset->getContents() == from) {
+                                       inset->setContents(to);
+                                       changed_inset = true;
+                               }
                        }
                }
-       }
+               if (changed_inset) {
+                       need_update = true;
 
-       // then check if the inset is a top_level inset (has no owner)
-       // if yes do the update as always otherwise we have to update the
-       // toplevel inset where this inset is inside
-       InsetOld * tl_inset = inset;
-       while (tl_inset->owner())
-               tl_inset = tl_inset->owner();
-       if (tl_inset == inset) {
-               update(BufferView::UPDATE);
-               if (bv_->text->updateInset(inset)) {
-                       update(BufferView::SELECT);
-                       return;
-               }
-       } else if (static_cast<UpdatableInset *>(tl_inset)
-                          ->updateInsetInInset(bv_, inset))
-       {
-               if (bv_->text->updateInset(tl_inset)) {
-                       update();
-                       updateScrollbar();
+                       // FIXME
+
+                       // The test it.size()==1 was needed to prevent crashes.
+                       // How to set the cursor correctly when it.size()>1 ??
+                       if (it.size() == 1) {
+                               bv_->text->setCursorIntern(bv_->text->parOffset(it.pit()), 0);
+                               bv_->text->redoParagraph(bv_->text->cursorPar());
+                       }
                }
        }
+       bv_->text->setCursorIntern(cursor.par(), cursor.pos());
+       return need_update;
 }