]> git.lyx.org Git - lyx.git/blobdiff - src/BufferView2.C
J�rgen Spitzm�ller's tweak of the Paragraph dialog.
[lyx.git] / src / BufferView2.C
index 4ca27927576084355d6650d55beb32451003a0fa..b58b1c3ca69174f352dac9b8b5d30db93e1f9121 100644 (file)
@@ -1,26 +1,23 @@
-// -*- C++ -*-
 /* This file is part of
  * ====================================================== 
  * 
  *           LyX, The Document Processor
  *        
  *           Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2000 The LyX Team.
+ *           Copyright 1995-2001 The LyX Team.
  *
  * ====================================================== */
 
-#include <config.h>
-
 #include <fstream>
 #include <algorithm>
 
+#include <config.h>
+
 #include "BufferView.h"
 #include "buffer.h"
 #include "lyxcursor.h"
 #include "lyxtext.h"
 #include "insets/inseterror.h"
-#include "insets/insetinfo.h"
-#include "insets/insetspecialchar.h"
 #include "LyXView.h"
 #include "bufferlist.h"
 #include "support/FileInfo.h"
@@ -33,6 +30,9 @@
 #include "support/lyxfunctional.h" //equal_1st_in_pair
 #include "language.h"
 #include "gettext.h"
+#include "undo_funcs.h"
+#include "debug.h"
+#include "iterators.h"
 
 extern BufferList bufferlist;
 
@@ -44,6 +44,7 @@ using std::find;
 using std::count;
 using std::count_if;
 
+
 // Inserts a file into current document
 bool BufferView::insertLyXFile(string const & filen)
        //
@@ -86,10 +87,11 @@ bool BufferView::insertLyXFile(string const & filen)
        bool res = true;
 
        if (c == '#') {
-               lyxerr.debug() << "Will insert file with header" << endl;
+               lyxerr[Debug::INFO] << "Will insert file with header" << endl;
                res = buffer()->readFile(lex, text->cursor.par());
        } else {
-               lyxerr.debug() << "Will insert file without header" << endl;
+               lyxerr[Debug::INFO] << "Will insert file without header" 
+                                   << endl;
                res = buffer()->readLyXformat2(lex, text->cursor.par());
        }
 
@@ -100,40 +102,21 @@ bool BufferView::insertLyXFile(string const & filen)
 
 bool BufferView::removeAutoInsets()
 {
-       LyXParagraph * par = buffer()->paragraph;
+       Paragraph * par = buffer()->paragraph;
 
        LyXCursor tmpcursor = text->cursor;
        LyXCursor cursor;
 
        bool a = false;
-#ifndef NEW_INSETS
-       while (par) {
-               // this has to be done before the delete
-               if (par->footnoteflag != LyXParagraph::CLOSED_FOOTNOTE)
-                       text->SetCursor(this, cursor, par, 0);
-               if (par->AutoDeleteInsets()){
-                       a = true;
-                       if (par->footnoteflag != LyXParagraph::CLOSED_FOOTNOTE){
-                               text->RedoParagraphs(this, cursor,
-                                                    cursor.par()->next());
-                               text->FullRebreak(this);
-                       }
-               }
-               par = par->next_;
-       }
 
-       // avoid forbidden cursor positions caused by error removing
-       if (tmpcursor.pos() > tmpcursor.par()->Last())
-               tmpcursor.pos(tmpcursor.par()->Last());
-#else
        while (par) {
                // this has to be done before the delete
-               text->SetCursor(this, cursor, par, 0);
-               if (par->AutoDeleteInsets()){
+               text->setCursor(this, cursor, par, 0);
+               if (par->autoDeleteInsets()){
                        a = true;
-                       text->RedoParagraphs(this, cursor,
+                       text->redoParagraphs(this, cursor,
                                             cursor.par()->next());
-                       text->FullRebreak(this);
+                       text->fullRebreak(this);
                }
                par = par->next();
        }
@@ -141,8 +124,8 @@ bool BufferView::removeAutoInsets()
        // avoid forbidden cursor positions caused by error removing
        if (tmpcursor.pos() > tmpcursor.par()->size())
                tmpcursor.pos(tmpcursor.par()->size());
-#endif
-       text->SetCursorIntern(this, tmpcursor.par(), tmpcursor.pos());
+
+       text->setCursorIntern(this, tmpcursor.par(), tmpcursor.pos());
 
        return a;
 }
@@ -153,19 +136,13 @@ void BufferView::insertErrors(TeXErrors & terr)
        // Save the cursor position
        LyXCursor cursor = text->cursor;
 
-#ifndef NEW_INSETS
-       // This is drastic, but it's the only fix, I could find. (Asger)
-       allFloats(1, 0);
-       allFloats(1, 1);
-#endif
-
        for (TeXErrors::Errors::const_iterator cit = terr.begin();
             cit != terr.end();
             ++cit) {
-               string const desctext((*cit).error_desc);
-               string const errortext((*cit).error_text);
+               string const desctext(cit->error_desc);
+               string const errortext(cit->error_text);
                string const msgtxt = desctext + '\n' + errortext;
-               int const errorrow = (*cit).error_in_line;
+               int const errorrow = cit->error_in_line;
 
                // Insert error string for row number
                int tmpid = -1; 
@@ -175,25 +152,25 @@ void BufferView::insertErrors(TeXErrors & terr)
                        buffer()->texrow.increasePos(tmpid, tmppos);
                }
                
-               LyXParagraph * texrowpar = 0;
+               Paragraph * texrowpar = 0;
 
                if (tmpid == -1) {
-                       texrowpar = text->FirstParagraph();
+                       texrowpar = text->firstParagraph();
                        tmppos = 0;
                } else {
-                       texrowpar = text->GetParFromID(tmpid);
+                       texrowpar = buffer()->getParFromID(tmpid);
                }
 
                if (texrowpar == 0)
                        continue;
 
                InsetError * new_inset = new InsetError(msgtxt);
-               text->SetCursorIntern(this, texrowpar, tmppos);
-               text->InsertInset(this, new_inset);
-               text->FullRebreak(this);
+               text->setCursorIntern(this, texrowpar, tmppos);
+               text->insertInset(this, new_inset);
+               text->fullRebreak(this);
        }
        // Restore the cursor position
-       text->SetCursorIntern(this, cursor.par(), cursor.pos());
+       text->setCursorIntern(this, cursor.par(), cursor.pos());
 }
 
 
@@ -204,100 +181,24 @@ void BufferView::setCursorFromRow(int row)
 
        buffer()->texrow.getIdFromRow(row, tmpid, tmppos);
 
-       LyXParagraph * texrowpar;
+       Paragraph * texrowpar;
 
        if (tmpid == -1) {
-               texrowpar = text->FirstParagraph();
+               texrowpar = text->firstParagraph();
                tmppos = 0;
        } else {
-               texrowpar = text->GetParFromID(tmpid);
+               texrowpar = buffer()->getParFromID(tmpid);
        }
-       text->SetCursor(this, texrowpar, tmppos);
+       text->setCursor(this, texrowpar, tmppos);
 }
 
 
-bool BufferView::insertInset(Inset * inset, string const & lout,
-                            bool /*no_table*/)
+bool BufferView::insertInset(Inset * inset, string const & lout)
 {
-       // if we are in a locking inset we should try to insert the
-       // inset there otherwise this is a illegal function now
-       if (theLockingInset()) {
-               if (theLockingInset()->InsertInsetAllowed(inset))
-                   return theLockingInset()->InsertInset(this, inset);
-               return false;
-       }
-
-       // not quite sure if we want this...
-       text->SetCursorParUndo(buffer());
-       text->FreezeUndo();
-       
-       beforeChange(text);
-       if (!lout.empty()) {
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->BreakParagraph(this);
-               update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-
-#ifndef NEW_INSETS
-               if (text->cursor.par()->Last()) {
-#else
-               if (text->cursor.par()->size()) {
-#endif
-                       text->CursorLeft(this);
-                       
-                       text->BreakParagraph(this);
-                       update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-               }
-
-               pair<bool, LyXTextClass::size_type> lres =
-                       textclasslist.NumberOfLayout(buffer()->params
-                                                    .textclass, lout);
-               LyXTextClass::size_type lay;
-               if (lres.first != false) {
-                       // layout found
-                       lay = lres.second;
-               } else {
-                       // layout not fount using default "Standard" (0)
-                       lay = 0;
-               }
-                
-               text->SetLayout(this, lay);
-               
-               text->SetParagraph(this, 0, 0,
-                                  0, 0,
-                                  VSpace(VSpace::NONE), VSpace(VSpace::NONE),
-                                  LYX_ALIGN_LAYOUT, 
-                                  string(),
-                                  0);
-               update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-               
-               text->current_font.setLatex(LyXFont::OFF);
-       }
-       
-       text->InsertInset(this, inset);
-       update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-
-       text->UnFreezeUndo();
-       return true;
+       return pimpl_->insertInset(inset, lout);
 }
 
 
-// Open and lock an updatable inset
-bool BufferView::open_new_inset(UpdatableInset * new_inset, bool behind)
-{
-       beforeChange(text);
-       text->FinishUndo();
-       if (!insertInset(new_inset)) {
-               delete new_inset;
-               return false;
-       }
-       if (behind) {
-               LyXFont & font = getLyXText()->real_current_font;
-               new_inset->Edit(this, new_inset->width(this, font), 0, 0);
-       } else
-               new_inset->Edit(this, 0, 0, 0);
-       return true;
-}
-
 /* This is also a buffer property (ale) */
 // Not so sure about that. a goto Label function can not be buffer local, just
 // think how this will work in a multiwindo/buffer environment, all the
@@ -312,8 +213,8 @@ bool BufferView::gotoLabel(string const & label)
                if (find(labels.begin(),labels.end(),label)
                     != labels.end()) {
                        beforeChange(text);
-                       text->SetCursor(this, it.getPar(), it.getPos());
-                       text->sel_cursor = text->cursor;
+                       text->setCursor(this, it.getPar(), it.getPos());
+                       text->selection.cursor = text->cursor;
                        update(text, BufferView::SELECT|BufferView::FITCUR);
                        return true;
                }
@@ -322,133 +223,6 @@ bool BufferView::gotoLabel(string const & label)
 }
 
 
-#ifndef NEW_INSETS
-void BufferView::allFloats(char flag, char figmar)
-{
-       if (!available()) return;
-
-       LyXCursor cursor = text->cursor;
-
-       if (!flag
-           && cursor.par()->footnoteflag != LyXParagraph::NO_FOOTNOTE
-           && ((figmar 
-                && cursor.par()->footnotekind != LyXParagraph::FOOTNOTE 
-                && cursor.par()->footnotekind != LyXParagraph::MARGIN
-                   )
-               || (!figmar
-                   && cursor.par()->footnotekind != LyXParagraph::FIG 
-                   && cursor.par()->footnotekind != LyXParagraph::TAB
-                   && cursor.par()->footnotekind != LyXParagraph::WIDE_FIG 
-                   && cursor.par()->footnotekind != LyXParagraph::WIDE_TAB
-                   && cursor.par()->footnotekind != LyXParagraph::ALGORITHM)))
-               toggleFloat();
-       else
-               beforeChange(text);
-
-       LyXCursor tmpcursor = cursor;
-       cursor.par(tmpcursor.par()->ParFromPos(tmpcursor.pos()));
-       cursor.pos(tmpcursor.par()->PositionInParFromPos(tmpcursor.pos()));
-
-       LyXParagraph *par = buffer()->paragraph;
-       while (par) {
-               if (flag) {
-                       if (par->footnoteflag == LyXParagraph::CLOSED_FOOTNOTE
-                           && ((figmar 
-                                && par->footnotekind != LyXParagraph::FOOTNOTE 
-                                && par->footnotekind !=  LyXParagraph::MARGIN)
-                               || (!figmar
-                                   && par->footnotekind != LyXParagraph::FIG 
-                                   && par->footnotekind != LyXParagraph::TAB
-                                   && par->footnotekind != LyXParagraph::WIDE_FIG 
-                                   && par->footnotekind != LyXParagraph::WIDE_TAB
-                                   && par->footnotekind != LyXParagraph::ALGORITHM
-                                       )
-                                   )
-                               ) {
-                               if (par->previous_
-                                   && par->previous_->footnoteflag != 
-                                   LyXParagraph::CLOSED_FOOTNOTE){ /* should be */ 
-                                       text->SetCursorIntern(this, 
-                                                             par->previous_,
-                                                             0);
-                                       text->OpenFootnote(this);
-                               }
-                       }
-               } else {
-                       if (par->footnoteflag == LyXParagraph::OPEN_FOOTNOTE
-                           && (
-                                   (figmar 
-                                    &&
-                                    par->footnotekind != LyXParagraph::FOOTNOTE 
-                                    &&
-                                    par->footnotekind !=  LyXParagraph::MARGIN
-                                           )
-                                   ||
-                                   (!figmar
-                                    &&
-                                    par->footnotekind != LyXParagraph::FIG 
-                                    &&
-                                    par->footnotekind != LyXParagraph::TAB
-                                    &&
-                                    par->footnotekind != LyXParagraph::WIDE_FIG 
-                                    &&
-                                    par->footnotekind != LyXParagraph::WIDE_TAB
-                                    &&
-                                    par->footnotekind != LyXParagraph::ALGORITHM
-                                           )
-                                   )
-                               ) {
-                               text->SetCursorIntern(this, par, 0);
-                               text->CloseFootnote(this);
-                       }
-               }
-               par = par->next_;
-       }
-
-       text->SetCursorIntern(this, cursor.par(), cursor.pos());
-       redraw();
-       fitCursor(text);
-}
-#endif
-
-
-void BufferView::insertNote()
-{
-       InsetInfo * new_inset = new InsetInfo();
-       insertInset(new_inset);
-       new_inset->Edit(this, 0, 0, 0);
-}
-
-
-#ifndef NEW_INSETS
-void BufferView::openStuff()
-{
-       if (available()) {
-               owner()->message(_("Open/Close..."));
-               hideCursor();
-               beforeChange(text);
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->OpenStuff(this);
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               setState();
-       }
-}
-
-
-void BufferView::toggleFloat()
-{
-       if (available()) {
-               owner()->message(_("Open/Close..."));
-               hideCursor();
-               beforeChange(text);
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->ToggleFootnote(this);
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               setState();
-       }
-}
-#endif
-
 void BufferView::menuUndo()
 {
        if (available()) {
@@ -456,7 +230,7 @@ void BufferView::menuUndo()
                hideCursor();
                beforeChange(text);
                update(text, BufferView::SELECT|BufferView::FITCUR);
-               if (!text->TextUndo(this))
+               if (!textUndo(this))
                        owner()->message(_("No forther undo information"));
                else
                        update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
@@ -477,7 +251,7 @@ void BufferView::menuRedo()
                hideCursor();
                beforeChange(text);
                update(text, BufferView::SELECT|BufferView::FITCUR);
-               if (!text->TextRedo(this))
+               if (!textRedo(this))
                        owner()->message(_("No further redo information"));
                else
                        update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
@@ -486,98 +260,13 @@ void BufferView::menuRedo()
 }
 
 
-void BufferView::hyphenationPoint()
-{
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               InsetSpecialChar * new_inset = 
-                       new InsetSpecialChar(InsetSpecialChar::HYPHENATION);
-               insertInset(new_inset);
-       }
-}
-
-
-void BufferView::ldots()
-{
-       if (available())  {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               InsetSpecialChar * new_inset = 
-                       new InsetSpecialChar(InsetSpecialChar::LDOTS);
-               insertInset(new_inset);
-       }
-}
-
-
-void BufferView::endOfSentenceDot()
-{
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               InsetSpecialChar * new_inset = 
-                       new InsetSpecialChar(InsetSpecialChar::END_OF_SENTENCE);
-               insertInset(new_inset);
-       }
-}
-
-
-void BufferView::menuSeparator()
-{
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               InsetSpecialChar * new_inset = 
-                       new InsetSpecialChar(InsetSpecialChar::MENU_SEPARATOR);
-               insertInset(new_inset);
-       }
-}
-
-
-void BufferView::newline()
-{
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->InsertChar(this, LyXParagraph::META_NEWLINE);
-               update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-       }
-}
-
-
-void BufferView::protectedBlank(LyXText * lt)
-{
-       if (available()) {
-               hideCursor();
-               update(lt, BufferView::SELECT|BufferView::FITCUR);
-               InsetSpecialChar * new_inset =
-                       new InsetSpecialChar(InsetSpecialChar::PROTECTED_SEPARATOR);
-               if (!insertInset(new_inset))
-                       delete new_inset;
-               else
-                       updateInset(new_inset, true);
-       }
-}
-
-
-void BufferView::hfill()
-{
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->InsertChar(this, LyXParagraph::META_HFILL);
-               update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-       }
-}
-
-
 void BufferView::copyEnvironment()
 {
        if (available()) {
                text->copyEnvironmentType();
                // clear the selection, even if mark_set
                toggleSelection();
-               text->ClearSelection(this);
+               text->clearSelection();
                update(text, BufferView::SELECT|BufferView::FITCUR);
                owner()->message(_("Paragraph environment type copied"));
        }
@@ -597,21 +286,22 @@ void BufferView::pasteEnvironment()
 void BufferView::copy()
 {
        if (available()) {
-               text->CopySelection(this);
+               text->copySelection(this);
                // clear the selection, even if mark_set
                toggleSelection();
-               text->ClearSelection(this);
+               text->clearSelection();
                update(text, BufferView::SELECT|BufferView::FITCUR);
                owner()->message(_("Copy"));
        }
 }
 
-void BufferView::cut()
+
+void BufferView::cut(bool realcut)
 {
        if (available()) {
                hideCursor();
                update(text, BufferView::SELECT|BufferView::FITCUR);
-               text->CutSelection(this);
+               text->cutSelection(this, true, realcut);
                update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
                owner()->message(_("Cut"));
        }
@@ -627,77 +317,20 @@ void BufferView::paste()
        hideCursor();
        // clear the selection
        toggleSelection();
-       text->ClearSelection(this);
+       text->clearSelection();
        update(text, BufferView::SELECT|BufferView::FITCUR);
        
        // paste
-       text->PasteSelection(this);
+       text->pasteSelection(this);
        update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
        
        // clear the selection 
        toggleSelection();
-       text->ClearSelection(this);
+       text->clearSelection();
        update(text, BufferView::SELECT|BufferView::FITCUR);
 }
 
 
-void BufferView::gotoInset(std::vector<Inset::Code> const & codes,
-                          bool same_content)
-{
-       if (!available()) return;
-   
-       hideCursor();
-       beforeChange(text);
-       update(text, BufferView::SELECT|BufferView::FITCUR);
-
-       string contents;
-       if (same_content &&
-           text->cursor.par()->GetChar(text->cursor.pos()) == LyXParagraph::META_INSET) {
-               Inset const * inset = text->cursor.par()->GetInset(text->cursor.pos());
-               if (find(codes.begin(), codes.end(), inset->LyxCode())
-                   != codes.end())
-                       contents =
-                               static_cast<InsetCommand const *>(inset)->getContents();
-       }
-       
-       if (!text->GotoNextInset(this, codes, contents)) {
-               if (text->cursor.pos() 
-                   || text->cursor.par() != text->FirstParagraph()) {
-                               LyXCursor tmp = text->cursor;
-                               text->cursor.par(text->FirstParagraph());
-                               text->cursor.pos(0);
-                               if (!text->GotoNextInset(this, codes, contents)) {
-                                       text->cursor = tmp;
-                                       owner()->message(_("No more insets"));
-                               }
-                       } else {
-                               owner()->message(_("No more insets"));
-                       }
-       }
-       update(text, BufferView::SELECT|BufferView::FITCUR);
-       text->sel_cursor = text->cursor;
-}
-
-
-void BufferView::gotoInset(Inset::Code code, bool same_content)
-{
-       gotoInset(vector<Inset::Code>(1, code), same_content);
-}
-
-
-void BufferView::insertCorrectQuote()
-{
-       char c;
-
-       if (text->cursor.pos())
-               c = text->cursor.par()->GetChar(text->cursor.pos() - 1);
-       else 
-               c = ' ';
-
-       insertInset(new InsetQuotes(c, buffer()->params));
-}
-
-
 /* these functions are for the spellchecker */ 
 string const BufferView::nextWord(float & value)
 {
@@ -706,7 +339,7 @@ string const BufferView::nextWord(float & value)
                return string();
        }
 
-       return text->SelectNextWord(this, value);
+       return text->selectNextWord(this, value);
 }
 
   
@@ -714,9 +347,11 @@ void BufferView::selectLastWord()
 {
        if (!available()) return;
    
+       LyXCursor cur = text->selection.cursor;
        hideCursor();
        beforeChange(text);
-       text->SelectSelectedWord(this);
+       text->selection.cursor = cur;
+       text->selectSelectedWord(this);
        toggleSelection(false);
        update(text, BufferView::SELECT|BufferView::FITCUR);
 }
@@ -728,8 +363,8 @@ void BufferView::endOfSpellCheck()
    
        hideCursor();
        beforeChange(text);
-       text->SelectSelectedWord(this);
-       text->ClearSelection(this);
+       text->selectSelectedWord(this);
+       text->clearSelection();
        update(text, BufferView::SELECT|BufferView::FITCUR);
 }
 
@@ -738,24 +373,25 @@ void BufferView::replaceWord(string const & replacestring)
 {
        if (!available()) return;
 
+       LyXText * tt = getLyXText();
        hideCursor();
-       update(text, BufferView::SELECT|BufferView::FITCUR);
+       update(tt, BufferView::SELECT|BufferView::FITCUR);
    
        /* clear the selection (if there is any) */ 
        toggleSelection(false);
-       update(text, BufferView::SELECT|BufferView::FITCUR);
+       update(tt, BufferView::SELECT|BufferView::FITCUR);
    
        /* clear the selection (if there is any) */ 
        toggleSelection(false);
-       text->ReplaceSelectionWithString(this, replacestring);
+       tt->replaceSelectionWithString(this, replacestring);
    
-       text->SetSelectionOverString(this, replacestring);
+       tt->setSelectionOverString(this, replacestring);
 
        // Go back so that replacement string is also spellchecked
        for (string::size_type i = 0; i < replacestring.length() + 1; ++i) {
-               text->CursorLeft(this);
+               tt->cursorLeft(this);
        }
-       update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
+       update(tt, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
 }
 // End of spellchecker stuff
 
@@ -766,7 +402,7 @@ bool BufferView::lockInset(UpdatableInset * inset)
                theLockingInset(inset);
                return true;
        } else if (inset) {
-           return theLockingInset()->LockInsetInInset(this, inset);
+           return theLockingInset()->lockInsetInInset(this, inset);
        }
        return false;
 }
@@ -774,19 +410,21 @@ bool BufferView::lockInset(UpdatableInset * inset)
 
 void BufferView::showLockedInsetCursor(int x, int y, int asc, int desc)
 {
-       if (theLockingInset() && available()) {
+       if (available() && theLockingInset()) {
                LyXCursor cursor = text->cursor;
+               Inset * locking_inset = theLockingInset()->getLockingInset();
+
                if ((cursor.pos() - 1 >= 0) &&
-                   (cursor.par()->GetChar(cursor.pos() - 1) ==
-                    LyXParagraph::META_INSET) &&
-                   (cursor.par()->GetInset(cursor.pos() - 1) ==
-                    theLockingInset()->GetLockingInset()))
-                       text->SetCursor(this, cursor,
-                                       cursor.par(), cursor.pos() - 1);
+                   (cursor.par()->getChar(cursor.pos() - 1) ==
+                    Paragraph::META_INSET) &&
+                   (cursor.par()->getInset(cursor.pos() - 1) ==
+                    locking_inset))
+                       text->setCursor(this, cursor,
+                                       cursor.par(), cursor.pos() - 1);
                LyXScreen::Cursor_Shape shape = LyXScreen::BAR_SHAPE;
                LyXText * txt = getLyXText();
-               if (theLockingInset()->GetLockingInset()->LyxCode() ==
-                   Inset::TEXT_CODE &&
+               if (locking_inset->isTextInset() &&
+                   locking_inset->lyxCode() != Inset::ERT_CODE &&
                    (txt->real_current_font.language() !=
                     buffer()->params.language
                     || txt->real_current_font.isVisibleRightToLeft()
@@ -794,8 +432,8 @@ void BufferView::showLockedInsetCursor(int x, int y, int asc, int desc)
                        shape = (txt->real_current_font.isVisibleRightToLeft())
                                ? LyXScreen::REVERSED_L_SHAPE
                                : LyXScreen::L_SHAPE;
-               y += cursor.y() + theLockingInset()->InsetInInsetY();
-               pimpl_->screen_->ShowManualCursor(text, x, y, asc, desc,
+               y += cursor.y() + theLockingInset()->insetInInsetY();
+               pimpl_->screen_->showManualCursor(text, x, y, asc, desc,
                                                  shape);
        }
 }
@@ -804,7 +442,7 @@ void BufferView::showLockedInsetCursor(int x, int y, int asc, int desc)
 void BufferView::hideLockedInsetCursor()
 {
        if (theLockingInset() && available()) {
-               pimpl_->screen_->HideCursor();
+               pimpl_->screen_->hideCursor();
        }
 }
 
@@ -812,8 +450,8 @@ void BufferView::hideLockedInsetCursor()
 void BufferView::fitLockedInsetCursor(int x, int y, int asc, int desc)
 {
        if (theLockingInset() && available()) {
-               y += text->cursor.y() + theLockingInset()->InsetInInsetY();
-               if (pimpl_->screen_->FitManualCursor(text, this, x, y, asc, desc))
+               y += text->cursor.y() + theLockingInset()->insetInInsetY();
+               if (pimpl_->screen_->fitManualCursor(text, this, x, y, asc, desc))
                        updateScrollbar();
        }
 }
@@ -822,13 +460,13 @@ void BufferView::fitLockedInsetCursor(int x, int y, int asc, int desc)
 int BufferView::unlockInset(UpdatableInset * inset)
 {
        if (inset && theLockingInset() == inset) {
-               inset->InsetUnlock(this);
+               inset->insetUnlock(this);
                theLockingInset(0);
-               text->FinishUndo();
+               finishUndo();
                return 0;
        } else if (inset && theLockingInset() &&
-                  theLockingInset()->UnlockInsetInInset(this, inset)) {
-               text->FinishUndo();
+                  theLockingInset()->unlockInsetInInset(this, inset)) {
+               finishUndo();
                return 0;
        }
        return bufferlist.unlockInset(inset);
@@ -841,60 +479,15 @@ void BufferView::lockedInsetStoreUndo(Undo::undo_kind kind)
                return; // shouldn't happen
        if (kind == Undo::EDIT) // in this case insets would not be stored!
                kind = Undo::FINISH;
-#ifndef NEW_INSETS
-       text->SetUndo(buffer(), kind,
-                     text->cursor.par()->
-                     ParFromPos(text->cursor.pos())->previous_, 
-                     text->cursor.par()->
-                     ParFromPos(text->cursor.pos())->next_);
-#else
-       text->SetUndo(buffer(), kind,
-                     text->cursor.par()->previous(), 
-                     text->cursor.par()->next());
-#endif
+       setUndo(this, kind,
+               text->cursor.par(),
+               text->cursor.par()->next());
 }
 
 
 void BufferView::updateInset(Inset * inset, bool mark_dirty)
 {
-       if (!inset)
-               return;
-
-       // first check for locking insets
-       if (theLockingInset()) {
-               if (theLockingInset() == inset) {
-                       if (text->UpdateInset(this, inset)) {
-                               update();
-                               if (mark_dirty) {
-                                       buffer()->markDirty();
-                               }
-                               updateScrollbar();
-                               return;
-                       }
-               } else if (theLockingInset()->UpdateInsetInInset(this,inset)) {
-                       if (text->UpdateInset(this, theLockingInset())) {
-                               update();
-                               if (mark_dirty){
-                                       buffer()->markDirty();
-                               }
-                               updateScrollbar();
-                               return;
-                       }
-               }
-       }
-  
-       // then check the current buffer
-       if (available()) {
-               hideCursor();
-               update(text, BufferView::UPDATE);
-               if (text->UpdateInset(this, inset)) {
-                       if (mark_dirty)
-                               update(text, BufferView::SELECT|BufferView::FITCUR|BufferView::CHANGE);
-                       else 
-                               update(text, SELECT);
-                       return;
-               }
-       }
+       pimpl_->updateInset(inset, mark_dirty);
 }
 
 
@@ -902,56 +495,40 @@ bool BufferView::ChangeInsets(Inset::Code code,
                              string const & from, string const & to)
 {
        bool flag = false;
-       LyXParagraph * par = buffer()->paragraph;
        LyXCursor cursor = text->cursor;
        LyXCursor tmpcursor = cursor;
-#ifndef NEW_INSETS
-       cursor.par(tmpcursor.par()->ParFromPos(tmpcursor.pos()));
-       cursor.pos(tmpcursor.par()->PositionInParFromPos(tmpcursor.pos()));
-#else
        cursor.par(tmpcursor.par());
        cursor.pos(tmpcursor.pos());
-#endif
 
-       while (par) {
+       ParIterator end = buffer()->par_iterator_end();
+       for (ParIterator it = buffer()->par_iterator_begin();
+            it != end; ++it) {
+               Paragraph * par = *it;
                bool flag2 = false;
-               for (LyXParagraph::inset_iterator it = par->inset_iterator_begin();
-                    it != par->inset_iterator_end(); ++it) {
-                       if ((*it)->LyxCode() == code) {
-                               InsetCommand * inset = static_cast<InsetCommand *>(*it);
+               for (Paragraph::inset_iterator it2 = par->inset_iterator_begin();
+                    it2 != par->inset_iterator_end(); ++it2) {
+                       if ((*it2)->lyxCode() == code) {
+                               InsetCommand * inset = static_cast<InsetCommand *>(*it2);
                                if (inset->getContents() == from) {
                                        inset->setContents(to);
                                        flag2 = true;
                                }
                        }
                }
-#ifndef NEW_INSETS
                if (flag2) {
                        flag = true;
-                       if (par->footnoteflag != LyXParagraph::CLOSED_FOOTNOTE){
-                               // this is possible now, since SetCursor takes
-                               // care about footnotes
-                               text->SetCursorIntern(this, par, 0);
-                               text->RedoParagraphs(this, text->cursor,
+#warning Fix me
+                       // The test it.size()==1 was needed to prevent crashes.
+                       // How to set the cursor corretly when it.size()>1 ??
+                       if (it.size() == 1) {
+                               text->setCursorIntern(this, par, 0);
+                               text->redoParagraphs(this, text->cursor,
                                                     text->cursor.par()->next());
-                               text->FullRebreak(this);
+                               text->fullRebreak(this);
                        }
                }
-               par = par->next_;
-#else
-               if (flag2) {
-                       flag = true;
-                       // this is possible now, since SetCursor takes
-                       // care about footnotes
-                       text->SetCursorIntern(this, par, 0);
-                       text->RedoParagraphs(this, text->cursor,
-                                            text->cursor.par()->next());
-                       text->FullRebreak(this);
-               }
-               par = par->next();
-#endif
        }
-       text->SetCursorIntern(this, cursor.par(), cursor.pos());
+       text->setCursorIntern(this, cursor.par(), cursor.pos());
        return flag;
 }
 
@@ -979,6 +556,7 @@ bool BufferView::ChangeCitationsIfUnique(string const & from, string const & to)
        return ChangeInsets(Inset::CITE_CODE, from, to);
 }
 
+
 UpdatableInset * BufferView::theLockingInset() const
 {
        // If NULL is not allowed we should put an Assert here. (Lgb)
@@ -990,7 +568,7 @@ UpdatableInset * BufferView::theLockingInset() const
 
 void BufferView::theLockingInset(UpdatableInset * inset)
 {
-    text->the_locking_inset = inset;
+       text->the_locking_inset = inset;
 }
 
 
@@ -1022,6 +600,6 @@ LyXText * BufferView::getParentText(Inset * inset) const
 Language const * BufferView::getParentLanguage(Inset * inset) const
 {
        LyXText * text = getParentText(inset);
-       return text->cursor.par()->GetFontSettings(buffer()->params,
+       return text->cursor.par()->getFontSettings(buffer()->params,
                                                   text->cursor.pos()).language();
 }