]> git.lyx.org Git - lyx.git/blobdiff - src/CutAndPaste.C
bug 1987: Crash when switching to different document class
[lyx.git] / src / CutAndPaste.C
index 1d567c022fb5d50d9033cb9273a1e66c8029d96c..25193a4a07cf335c0d43d47c853b38d2bc3c957a 100644 (file)
@@ -23,6 +23,7 @@
 #include "errorlist.h"
 #include "funcrequest.h"
 #include "gettext.h"
+#include "insetiterator.h"
 #include "lfuns.h"
 #include "lyxrc.h"
 #include "lyxtext.h"
@@ -34,6 +35,7 @@
 #include "pariterator.h"
 #include "undo.h"
 
+#include "insets/insetcharstyle.h"
 #include "insets/insettabular.h"
 
 #include "mathed/math_data.h"
@@ -66,7 +68,13 @@ typedef limited_stack<pair<ParagraphList, textclass_type> > CutStack;
 
 CutStack theCuts(10);
 
-struct resetOwnerAndChanges : public std::unary_function<Paragraph, void> {
+// store whether the tabular stack is newer than the normal copy stack
+// FIXME: this is a workaround for bug 1919. Should be removed for 1.5, 
+// when we (hopefully) have a one-for-all paste mechanism.
+bool dirty_tabular_stack_;
+
+class resetOwnerAndChanges : public std::unary_function<Paragraph, void> {
+public:
        void operator()(Paragraph & p) const {
                p.cleanChanges();
                p.setInsetOwner(0);
@@ -113,9 +121,24 @@ pasteSelectionHelper(Buffer const & buffer, ParagraphList & pars,
        // Now remove all out of the pars which is NOT allowed in the
        // new environment and set also another font if that is required.
 
+       // Convert newline to paragraph break in ERT inset.
+       // This should not be here!
+       if (pars[pit].inInset() &&
+           pars[pit].inInset()->lyxCode() == InsetBase::ERT_CODE) {
+               for (ParagraphList::size_type i = 0; i < insertion.size(); ++i) {
+                       for (pos_type j = 0; j < insertion[i].size(); ++j) {
+                               if (insertion[i].isNewline(j)) {
+                                       insertion[i].erase(j);
+                                       breakParagraphConservative(
+                                                       buffer.params(),
+                                                       insertion, i, j);
+                               }
+                       }
+               }
+       }
+
        // Make sure there is no class difference.
-       lyx::cap::SwitchLayoutsBetweenClasses(textclass, tc, insertion,
-                                   errorlist);
+       lyx::cap::SwitchBetweenClasses(textclass, tc, insertion, errorlist);
 
        ParagraphList::iterator tmpbuf = insertion.begin();
        int depth_delta = pars[pit].params().depth() - tmpbuf->params().depth();
@@ -144,15 +167,18 @@ pasteSelectionHelper(Buffer const & buffer, ParagraphList & pars,
                // Set the inset owner of this paragraph.
                tmpbuf->setInsetOwner(pars[pit].inInset());
                for (pos_type i = 0; i < tmpbuf->size(); ++i) {
-                       if (tmpbuf->getChar(i) == Paragraph::META_INSET) {
-                               if (!pars[pit].insetAllowed(tmpbuf->getInset(i)->lyxCode()))
-                                       tmpbuf->erase(i--);
-                       }
+                       if (tmpbuf->getChar(i) == Paragraph::META_INSET &&
+                           !pars[pit].insetAllowed(tmpbuf->getInset(i)->lyxCode()))
+                               tmpbuf->erase(i--);
                }
        }
 
-       // Make the buf exactly the same layout as the cursor paragraph.
-       insertion.begin()->makeSameLayout(pars[pit]);
+       bool const empty = pars[pit].empty();
+       if (!empty) {
+               // Make the buf exactly the same layout as the cursor
+               // paragraph.
+               insertion.begin()->makeSameLayout(pars[pit]);
+       }
 
        // Prepare the paragraphs and insets for insertion.
        // A couple of insets store buffer references so need updating.
@@ -182,15 +208,24 @@ pasteSelectionHelper(Buffer const & buffer, ParagraphList & pars,
        std::swap(in.paragraphs(), insertion);
 
        // Split the paragraph for inserting the buf if necessary.
-       bool did_split = false;
-       if (pars[pit].size() || pit + 1 == pit_type(pars.size())) {
+       if (!empty)
                breakParagraphConservative(buffer.params(), pars, pit, pos);
-               did_split = true;
-       }
 
        // Paste it!
-       pars.insert(pars.begin() + pit + 1, insertion.begin(), insertion.end());
-       mergeParagraph(buffer.params(), pars, pit);
+       if (empty) {
+               pars.insert(pars.begin() + pit, insertion.begin(),
+                           insertion.end());
+
+               // merge the empty par with the last par of the insertion
+               mergeParagraph(buffer.params(), pars,
+                              pit + insertion.size() - 1);
+       } else {
+               pars.insert(pars.begin() + pit + 1, insertion.begin(),
+                           insertion.end());
+
+               // merge the first par of the insertion with the current par
+               mergeParagraph(buffer.params(), pars, pit);
+       }
 
        pit_type last_paste = pit + insertion.size() - 1;
 
@@ -199,7 +234,7 @@ pasteSelectionHelper(Buffer const & buffer, ParagraphList & pars,
        pos = pars[last_paste].size();
 
        // Maybe some pasting.
-       if (did_split && last_paste + 1 != pit_type(pars.size())) {
+       if (!empty && last_paste + 1 != pit_type(pars.size())) {
                if (pars[last_paste + 1].hasSameLayout(pars[last_paste])) {
                        mergeParagraph(buffer.params(), pars, last_paste);
                } else if (pars[last_paste + 1].empty()) {
@@ -223,41 +258,50 @@ PitPosPair eraseSelectionHelper(BufferParams const & params,
        pit_type startpit, pit_type endpit,
        int startpos, int endpos, bool doclear)
 {
+        // Start of selection is really invalid.
        if (startpit == pit_type(pars.size()) ||
            (startpos > pars[startpit].size()))
                return PitPosPair(endpit, endpos);
 
+        // Start and end is inside same paragraph
        if (endpit == pit_type(pars.size()) ||
            startpit == endpit) {
                endpos -= pars[startpit].erase(startpos, endpos);
                return PitPosPair(endpit, endpos);
        }
 
-       // clear end/begin fragments of the first/last par in selection
        bool all_erased = true;
 
+        // Clear fragments of the first par in selection
        pars[startpit].erase(startpos, pars[startpit].size());
        if (pars[startpit].size() != startpos)
                all_erased = false;
 
+        // Clear fragments of the last par in selection
        endpos -= pars[endpit].erase(0, endpos);
        if (endpos != 0)
                all_erased = false;
 
-       // Loop through the deleted pars if any, erasing as needed
-       for (pit_type pit = startpit + 1; pit != endpit;) {
-               // "erase" the contents of the par
-               pars[pit].erase(0, pars[pit].size());
-               if (!pars[pit].size()) {
-                       // remove the par if it's now empty
-                       pars.erase(pars.begin() + pit);
-                       --endpit;
-               } else {
-                       ++pit;
-                       all_erased = false;
+        // Erase all the "middle" paragraphs.
+       if (params.tracking_changes) {
+               // Look through the deleted pars if any, erasing as needed
+               for (pit_type pit = startpit + 1; pit != endpit;) {
+                       // "erase" the contents of the par
+                       pars[pit].erase(0, pars[pit].size());
+                       if (pars[pit].empty()) {
+                               // remove the par if it's now empty
+                               pars.erase(pars.begin() + pit);
+                               --endpit;
+                       } else {
+                               ++pit;
+                               all_erased = false;
+                       }
                }
+       } else {
+               pars.erase(pars.begin() + startpit + 1, pars.begin() + endpit);
+               endpit = startpit + 1;
        }
-
+       
 #if 0 // FIXME: why for cut but not copy ?
        // the cut selection should begin with standard layout
        if (realcut) {
@@ -274,18 +318,17 @@ PitPosPair eraseSelectionHelper(BufferParams const & params,
                pars[startpit + 1].stripLeadingSpaces();
        }
 
-       // paste the paragraphs again, if possible
+       // Merge first and last paragraph, if possible
        if (all_erased &&
            (pars[startpit].hasSameLayout(pars[startpit + 1]) ||
             pars[startpit + 1].empty())) {
                mergeParagraph(params, pars, startpit);
-               // this because endpar gets deleted here!
+               // This because endpar gets deleted here!
                endpit = startpit;
                endpos = startpos;
        }
 
        return PitPosPair(endpit, endpos);
-
 }
 
 
@@ -299,6 +342,7 @@ void copySelectionHelper(ParagraphList & pars,
 
        // Clone the paragraphs within the selection.
        ParagraphList paragraphs(pars.begin() + startpit, pars.begin() + endpit + 1);
+
        for_each(paragraphs.begin(), paragraphs.end(), resetOwnerAndChanges());
 
        // Cut out the end of the last paragraph.
@@ -312,18 +356,6 @@ void copySelectionHelper(ParagraphList & pars,
        theCuts.push(make_pair(paragraphs, tc));
 }
 
-
-
-PitPosPair cutSelectionHelper(BufferParams const & params,
-       ParagraphList & pars, pit_type startpit, pit_type endpit,
-       int startpos, int endpos, textclass_type tc, bool doclear)
-{
-       copySelectionHelper(pars, startpit, endpit, startpos, endpos, tc);
-       return eraseSelectionHelper(params, pars, startpit, endpit,
-               startpos, endpos, doclear);
-}
-
-
 } // namespace anon
 
 
@@ -343,13 +375,12 @@ string grabAndEraseSelection(LCursor & cur)
 }
 
 
-int SwitchLayoutsBetweenClasses(textclass_type c1, textclass_type c2,
+void SwitchBetweenClasses(textclass_type c1, textclass_type c2,
        ParagraphList & pars, ErrorList & errorlist)
 {
        BOOST_ASSERT(!pars.empty());
-       int ret = 0;
        if (c1 == c2)
-               return ret;
+               return;
 
        LyXTextClass const & tclass1 = textclasslist[c1];
        LyXTextClass const & tclass2 = textclasslist[c2];
@@ -357,6 +388,7 @@ int SwitchLayoutsBetweenClasses(textclass_type c1, textclass_type c2,
        InsetText in;
        std::swap(in.paragraphs(), pars);
 
+       // layouts
        ParIterator end = par_iterator_end(in);
        for (ParIterator it = par_iterator_begin(in); it != end; ++it) {
                string const name = it->layout()->name();
@@ -368,19 +400,48 @@ int SwitchLayoutsBetweenClasses(textclass_type c1, textclass_type c2,
                        it->layout(tclass2.defaultLayout());
 
                if (!hasLayout && name != tclass1.defaultLayoutName()) {
-                       ++ret;
                        string const s = bformat(
                                _("Layout had to be changed from\n%1$s to %2$s\n"
                                "because of class conversion from\n%3$s to %4$s"),
                         name, it->layout()->name(), tclass1.name(), tclass2.name());
                        // To warn the user that something had to be done.
-                       errorlist.push_back(ErrorItem("Changed Layout", s,
+                       errorlist.push_back(ErrorItem(_("Changed Layout"), s,
                                                      it->id(), 0,
                                                      it->size()));
                }
        }
+
+       // character styles
+       InsetIterator const i_end = inset_iterator_end(in);
+       for (InsetIterator it = inset_iterator_begin(in); it != i_end; ++it) {
+               if (it->lyxCode() == InsetBase::CHARSTYLE_CODE) {
+                       InsetCharStyle & inset =
+                               static_cast<InsetCharStyle &>(*it);
+                       string const name = inset.params().type;
+                       CharStyles::iterator const found_cs =
+                               tclass2.charstyle(name);
+                       if (found_cs == tclass2.charstyles().end()) {
+                               // The character style is undefined in tclass2
+                               inset.setUndefined();
+                               string const s = bformat(_(
+                                       "Character style %1$s is "
+                                       "undefined because of class "
+                                       "conversion from\n%2$s to %3$s"),
+                                        name, tclass1.name(), tclass2.name());
+                               // To warn the user that something had to be done.
+                               errorlist.push_back(ErrorItem(
+                                               _("Undefined character style"),
+                                               s, it.paragraph().id(),
+                                               it.pos(), it.pos() + 1));
+                       } else if (inset.undefined()) {
+                               // The character style is undefined in
+                               // tclass1 and is defined in tclass2
+                               inset.setDefined(found_cs);
+                       }
+               }
+       }
+
        std::swap(in.paragraphs(), pars);
-       return ret;
 }
 
 
@@ -431,7 +492,7 @@ void cutSelection(LCursor & cur, bool doclear, bool realcut)
                // finished. The solution used currently just works, to make it
                // faster we need to be more clever and probably also have more
                // calls to stuffClipboard. (Lgb)
-               cur.bv().stuffClipboard(cur.selectionAsString(true));
+//             cur.bv().stuffClipboard(cur.selectionAsString(true));
 
                // This doesn't make sense, if there is no selection
                if (!cur.selection())
@@ -474,7 +535,10 @@ void cutSelection(LCursor & cur, bool doclear, bool realcut)
 
                // need a valid cursor. (Lgb)
                cur.clearSelection();
-               text->updateCounters();
+               updateCounters(cur.buffer());
+
+               // tell tabular that a recent copy happened
+               dirtyTabularStack(false);
        }
 
        if (cur.inMathed()) {
@@ -525,6 +589,8 @@ void copySelection(LCursor & cur)
                pars.back().insert(0, grabSelection(cur), LyXFont());
                theCuts.push(make_pair(pars, bp.textclass));
        }
+       // tell tabular that a recent copy happened
+       dirtyTabularStack(false);
 }
 
 
@@ -556,10 +622,10 @@ void pasteSelection(LCursor & cur, size_t sel_index)
 
                boost::tie(ppp, endpit) =
                        pasteSelectionHelper(cur.buffer(),
-                                                               text->paragraphs(),
-                                                               cur.pit(), cur.pos(),
-                                                               cur.buffer().params().textclass,
-                                                               sel_index, el);
+                                             text->paragraphs(),
+                                             cur.pit(), cur.pos(),
+                                             cur.buffer().params().textclass,
+                                             sel_index, el);
                bufferErrors(cur.buffer(), el);
                cur.bv().showErrorList(_("Paste"));
 
@@ -567,7 +633,7 @@ void pasteSelection(LCursor & cur, size_t sel_index)
                cur.resetAnchor();
                text->setCursor(cur, ppp.first, ppp.second);
                cur.setSelection();
-               text->updateCounters();
+               updateCounters(cur.buffer());
        }
 
        if (cur.inMathed()) {
@@ -641,6 +707,7 @@ void eraseSelection(LCursor & cur)
        CursorSlice const & i1 = cur.selBegin();
        CursorSlice const & i2 = cur.selEnd();
        if (i1.inset().asMathInset()) {
+               cur.top() = i1;
                if (i1.idx() == i2.idx()) {
                        i1.cell().erase(i1.pos(), i2.pos());
                } else {
@@ -651,9 +718,9 @@ void eraseSelection(LCursor & cur)
                        for (InsetBase::row_type row = r1; row <= r2; ++row)
                                for (InsetBase::col_type col = c1; col <= c2; ++col)
                                        p->cell(p->index(row, col)).clear();
+                       // We've deleted the whole cell. Only pos 0 is valid.
+                       cur.pos() = 0;
                }
-               cur.back() = i1;
-               cur.pos() = 0; // We've deleted the whole cell. Only pos 0 is valid.
                cur.resetAnchor();
        } else {
                lyxerr << "can't erase this selection 1" << endl;
@@ -722,5 +789,17 @@ string grabSelection(LCursor & cur)
 }
 
 
+void dirtyTabularStack(bool b)
+{
+       dirty_tabular_stack_ = b;
+}
+
+
+bool tabularStackDirty()
+{
+       return dirty_tabular_stack_;
+}
+
+
 } // namespace cap
 } // namespace lyx