]> git.lyx.org Git - features.git/blobdiff - src/insets/InsetIndex.cpp
Allow removing words from the personal dictionary, that weren't previously added.
[features.git] / src / insets / InsetIndex.cpp
index 5f4cb7e93fc49ab81d4738118603d9dfb15d46cb..f40ba031fcca8bf7b88f223e34eecc7f129a3c90 100644 (file)
@@ -18,6 +18,7 @@
 #include "BufferView.h"
 #include "ColorSet.h"
 #include "Cursor.h"
+#include "CutAndPaste.h"
 #include "DispatchResult.h"
 #include "Encoding.h"
 #include "ErrorList.h"
@@ -55,6 +56,9 @@
 using namespace std;
 using namespace lyx::support;
 
+// Uncomment to enable InsetIndex-specific debugging mode: the tree for the index will be printed to std::cout.
+// #define LYX_INSET_INDEX_DEBUG
+
 namespace lyx {
 
 namespace {
@@ -129,6 +133,9 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
 {
        OutputParams runparams(runparams_in);
        runparams.inIndexEntry = true;
+       if (runparams_in.postpone_fragile_stuff)
+               // This is not needed and would impact sorting
+               runparams.moving_arg = false;
 
        otexstringstream os;
 
@@ -157,7 +164,7 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
                getSortkey(os, runparams);
                os << "@";
                os << ourlatex.str();
-               getSubentries(os, runparams);
+               getSubentries(os, runparams, ourlatex.str());
                if (hasSeeRef()) {
                        os << "|";
                        os << insetindexpagerangetranslator_latex().find(params_.range);
@@ -166,6 +173,9 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
                        os << "|";
                        os << insetindexpagerangetranslator_latex().find(params_.range);
                        os << from_utf8(params_.pagefmt);
+               } else if (params_.range != InsetIndexParams::PageRange::None) {
+                       os << "|";
+                       os << insetindexpagerangetranslator_latex().find(params_.range);
                }
        } else {
                // We check whether we need a sort key.
@@ -211,7 +221,7 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
 
                odocstringstream subentries;
                otexstream otsub(subentries);
-               getSubentries(otsub, runparams);
+               getSubentries(otsub, runparams, ourlatex.str());
                if (subentries.str().empty()) {
                        // Separate the entries and subentries, i.e., split on "!".
                        // This goes wrong on an escaped "!", but as the escape
@@ -227,6 +237,12 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
                        vector<docstring>::const_iterator it2 = levels_plain.begin();
                        bool first = true;
                        for (; it != end; ++it) {
+                               if ((*it).empty()) {
+                                       emptySubentriesWarning(ourlatex.str());
+                                       if (it2 < levels_plain.end())
+                                               ++it2;
+                                       continue;
+                               }
                                // The separator needs to be put back when
                                // writing the levels, except for the first level
                                if (!first)
@@ -258,6 +274,9 @@ void InsetIndex::latex(otexstream & ios, OutputParams const & runparams_in) cons
                        os << "|"
                           << insetindexpagerangetranslator_latex().find(params_.range)
                           << cmd;
+               } else if (params_.range != InsetIndexParams::PageRange::None) {
+                       os << "|";
+                       os << insetindexpagerangetranslator_latex().find(params_.range);
                }
        }
        os << '}';
@@ -354,7 +373,8 @@ void InsetIndex::docbook(XMLStream & xs, OutputParams const & runparams) const
 
        // Handle primary, secondary, and tertiary terms (entries, subentries, and subsubentries, for LaTeX).
        vector<docstring> terms;
-       if (const vector<docstring> potential_terms = getSubentriesAsText(runparams); !potential_terms.empty()) {
+       const vector<docstring> potential_terms = getSubentriesAsText(runparams);
+       if (!potential_terms.empty()) {
                terms = potential_terms;
                // The main term is not present in the vector, as it's not a subentry. The main index term is inserted raw in
                // the index inset. Considering that the user either uses the new or the legacy mechanism, the main term is the
@@ -564,6 +584,21 @@ void InsetIndex::doDispatch(Cursor & cur, FuncRequest & cmd)
                        params_.index = from_utf8(cmd.getArg(1));
                        break;
                }
+               if (cmd.getArg(0) == "changeparam") {
+                       string const p = cmd.getArg(1);
+                       string const v = cmd.getArg(2);
+                       cur.recordUndoInset(this);
+                       if (p == "range")
+                               params_.range = insetindexpagerangetranslator().find(v);
+                       if (p == "pagefmt") {
+                               if (v == "default" || v == "textbf"
+                                   || v == "textit" || v == "emph")
+                                       params_.pagefmt = v;
+                               else
+                                       lyx::dispatch(FuncRequest(LFUN_INSET_SETTINGS, "index"));
+                       }
+                       break;
+               }
                InsetIndexParams params;
                InsetIndex::string2params(to_utf8(cmd.argument()), params);
                cur.recordUndoInset(this);
@@ -582,12 +617,22 @@ void InsetIndex::doDispatch(Cursor & cur, FuncRequest & cmd)
 
        case LFUN_PARAGRAPH_BREAK: {
                // Since this inset in single-par anyway, let's use
-               // return to enter subindexes
-               FuncRequest fr(LFUN_INDEXMACRO_INSERT, "subindex");
+               // return to enter subentries
+               FuncRequest fr(LFUN_INDEXMACRO_INSERT, "subentry");
                lyx::dispatch(fr);
                break;
        }
 
+       case LFUN_INSET_INSERT_COPY: {
+               Cursor & bvcur = cur.bv().cursor();
+               if (cmd.origin() == FuncRequest::TOC && bvcur.inTexted()) {
+                       cap::copyInsetToTemp(cur, clone());
+                       cap::pasteFromTemp(bvcur, bvcur.buffer()->errorList("Paste"));
+               } else
+                       cur.undispatched();
+               break;
+       }
+
        default:
                InsetCollapsible::doDispatch(cur, cmd);
                break;
@@ -611,6 +656,23 @@ bool InsetIndex::getStatus(Cursor & cur, FuncRequest const & cmd,
                                from_utf8(cmd.getArg(1)) == params_.index);
                        return true;
                }
+               if (cmd.getArg(0) == "changeparam") {
+                       string const p = cmd.getArg(1);
+                       string const v = cmd.getArg(2);
+                       if (p == "range") {
+                               flag.setEnabled(v == "none" || v == "start" || v == "end");
+                               flag.setOnOff(params_.range == insetindexpagerangetranslator().find(v));
+                       }
+                       if (p == "pagefmt") {
+                               flag.setEnabled(!v.empty());
+                               if (params_.pagefmt == "default" || params_.pagefmt == "textbf"
+                                   || params_.pagefmt == "textit" || params_.pagefmt == "emph")
+                                       flag.setOnOff(params_.pagefmt == v);
+                               else
+                                       flag.setOnOff(v == "custom");
+                       }
+                       return true;
+               }
                return InsetCollapsible::getStatus(cur, cmd, flag);
 
        case LFUN_INSET_DIALOG_UPDATE: {
@@ -618,13 +680,22 @@ bool InsetIndex::getStatus(Cursor & cur, FuncRequest const & cmd,
                flag.setEnabled(realbuffer.params().use_indices);
                return true;
        }
-       
+
+       case LFUN_INSET_INSERT_COPY:
+               // This can only be invoked by ToC widget
+               flag.setEnabled(cmd.origin() == FuncRequest::TOC
+                               && cur.bv().cursor().inset().insetAllowed(lyxCode()));
+               return true;
+
        case LFUN_PARAGRAPH_BREAK:
-               return macrosPossible("subindex");
-       
+               return macrosPossible("subentry");
+
        case LFUN_INDEXMACRO_INSERT:
                return macrosPossible(cmd.getArg(0));
 
+       case LFUN_INDEX_TAG_ALL:
+               return true;
+
        default:
                return InsetCollapsible::getStatus(cur, cmd, flag);
        }
@@ -665,7 +736,22 @@ docstring InsetIndex::getSortkeyAsText(OutputParams const & runparams) const
 }
 
 
-void InsetIndex::getSubentries(otexstream & os, OutputParams const & runparams) const
+void InsetIndex::emptySubentriesWarning(docstring const & mainentry) const
+{
+       // Empty subentries crash makeindex. So warn and ignore this.
+       TeXErrors terr;
+       ErrorList & errorList = buffer().errorList("Export");
+       docstring const s = bformat(_("There is an empty index subentry in the entry '%1$s'.\n"
+                                     "It will be ignored in the output."), mainentry);
+       Paragraph const & par = buffer().paragraphs().front();
+       errorList.push_back(ErrorItem(_("Empty index subentry!"), s,
+                                     {par.id(), 0}, {par.id(), -1}));
+       buffer().bufferErrors(terr, errorList);
+}
+
+
+void InsetIndex::getSubentries(otexstream & os, OutputParams const & runparams,
+                              docstring const & mainentry) const
 {
        Paragraph const & par = paragraphs().front();
        InsetList::const_iterator it = par.insetList().begin();
@@ -675,7 +761,11 @@ void InsetIndex::getSubentries(otexstream & os, OutputParams const & runparams)
                if (inset.lyxCode() == INDEXMACRO_CODE) {
                        InsetIndexMacro const & iim =
                                static_cast<InsetIndexMacro const &>(inset);
-                       if (iim.params().type == InsetIndexMacroParams::Subindex) {
+                       if (iim.params().type == InsetIndexMacroParams::Subentry) {
+                               if (iim.hasNoContent()) {
+                                       emptySubentriesWarning(mainentry);
+                                       continue;
+                               }
                                ++i;
                                if (i > 2)
                                        return;
@@ -700,7 +790,7 @@ std::vector<docstring> InsetIndex::getSubentriesAsText(OutputParams const & runp
                if (inset.lyxCode() == INDEXMACRO_CODE) {
                        InsetIndexMacro const & iim =
                                static_cast<InsetIndexMacro const &>(inset);
-                       if (iim.params().type == InsetIndexMacroParams::Subindex) {
+                       if (iim.params().type == InsetIndexMacroParams::Subentry) {
                                ++i;
                                if (i > 2)
                                        break;
@@ -748,7 +838,8 @@ void InsetIndex::getSeeRefs(otexstream & os, OutputParams const & runparams) con
 }
 
 
-docstring InsetIndex::getSeeAsText(OutputParams const & runparams) const
+docstring InsetIndex::getSeeAsText(OutputParams const & runparams,
+                                  bool const asLabel) const
 {
        Paragraph const & par = paragraphs().front();
        InsetList::const_iterator it = par.insetList().begin();
@@ -758,9 +849,14 @@ docstring InsetIndex::getSeeAsText(OutputParams const & runparams) const
                        InsetIndexMacro const & iim =
                                static_cast<InsetIndexMacro const &>(inset);
                        if (iim.params().type == InsetIndexMacroParams::See) {
-                               otexstringstream os;
-                               iim.getLatex(os, runparams);
-                               return os.str();
+                               if (asLabel) {
+                                       docstring const l;
+                                       return iim.getNewLabel(l);
+                               } else {
+                                       otexstringstream os;
+                                       iim.getLatex(os, runparams);
+                                       return os.str();
+                               }
                        }
                }
        }
@@ -768,7 +864,8 @@ docstring InsetIndex::getSeeAsText(OutputParams const & runparams) const
 }
 
 
-std::vector<docstring> InsetIndex::getSeeAlsoesAsText(OutputParams const & runparams) const
+std::vector<docstring> InsetIndex::getSeeAlsoesAsText(OutputParams const & runparams,
+                                                     bool const asLabel) const
 {
        std::vector<docstring> seeAlsoes;
 
@@ -780,9 +877,14 @@ std::vector<docstring> InsetIndex::getSeeAlsoesAsText(OutputParams const & runpa
                        InsetIndexMacro const & iim =
                                static_cast<InsetIndexMacro const &>(inset);
                        if (iim.params().type == InsetIndexMacroParams::Seealso) {
-                               otexstringstream os;
-                               iim.getLatex(os, runparams);
-                               seeAlsoes.emplace_back(os.str());
+                               if (asLabel) {
+                                       docstring const l;
+                                       seeAlsoes.emplace_back(iim.getNewLabel(l));
+                               } else {
+                                       otexstringstream os;
+                                       iim.getLatex(os, runparams);
+                                       seeAlsoes.emplace_back(os.str());
+                               }
                        }
                }
        }
@@ -819,7 +921,7 @@ bool hasInsetWithCode(const InsetIndex * const inset_index, const InsetCode code
 
 bool InsetIndex::hasSubentries() const
 {
-       return hasInsetWithCode(this, INDEXMACRO_CODE, {InsetIndexMacroParams::Subindex});
+       return hasInsetWithCode(this, INDEXMACRO_CODE, {InsetIndexMacroParams::Subentry});
 }
 
 
@@ -838,7 +940,7 @@ bool InsetIndex::hasSortKey() const
 bool InsetIndex::macrosPossible(string const type) const
 {
        if (type != "see" && type != "seealso"
-           && type != "sortkey" && type != "subindex")
+           && type != "sortkey" && type != "subentry")
                return false;
 
        Paragraph const & par = paragraphs().front();
@@ -854,8 +956,8 @@ bool InsetIndex::macrosPossible(string const type) const
                             && (iim.params().type == InsetIndexMacroParams::See
                                 || iim.params().type == InsetIndexMacroParams::Seealso))
                                return false;
-                       if (type == "subindex"
-                            && iim.params().type == InsetIndexMacroParams::Subindex) {
+                       if (type == "subentry"
+                            && iim.params().type == InsetIndexMacroParams::Subentry) {
                                ++subidxs;
                                if (subidxs > 1)
                                        return false;
@@ -940,6 +1042,13 @@ docstring const InsetIndex::buttonLabel(BufferView const & bv) const
                        res += " " + docstring(1, char_type(0x2023));// TRIANGULAR BULLET
                        res += " " + sublbl;
                }
+               docstring see = getSeeAsText(rp, true);
+               if (see.empty() && !getSeeAlsoesAsText(rp, true).empty())
+                       see = getSeeAlsoesAsText(rp, true).front();
+               if (!see.empty()) {
+                       res += " " + docstring(1, char_type(0x261e));// WHITE RIGHT POINTING INDEX
+                       res += " " + see;
+               }
        }
        if (!insetindexpagerangetranslator_latex().find(params_.range).empty())
                res += " " + from_ascii(insetindexpagerangetranslator_latex().find(params_.range));
@@ -1004,6 +1113,13 @@ void InsetIndex::addToToc(DocIterator const & cpit, bool output_active,
                        str += " " + docstring(1, char_type(0x2023));// TRIANGULAR BULLET
                        str += " " + sublbl;
                }
+               docstring see = getSeeAsText(rp, true);
+               if (see.empty() && !getSeeAlsoesAsText(rp, true).empty())
+                       see = getSeeAlsoesAsText(rp, true).front();
+               if (!see.empty()) {
+                       str += " " + docstring(1, char_type(0x261e));// WHITE RIGHT POINTING INDEX
+                       str += " " + see;
+               }
        }
        string type = "index";
        if (buffer().masterBuffer()->params().use_indices)
@@ -1333,7 +1449,9 @@ public:
 private:
        bool isModern()
        {
+#ifdef LYX_INSET_INDEX_DEBUG
                std::cout << to_utf8(entry_) << std::endl;
+#endif // LYX_INSET_INDEX_DEBUG
 
                // If a modern parameter is present, this is definitely a modern index inset. Similarly, if it contains the
                // usual LaTeX symbols (!|@), then it is definitely a legacy index inset. Otherwise, if it has features of
@@ -1561,12 +1679,17 @@ docstring termAtLevel(const IndexNode* node, unsigned depth)
 
 void insertIntoNode(const IndexEntry& entry, IndexNode* node, unsigned depth = 0)
 {
+       // Do not insert empty entries.
+       if (entry.terms().empty())
+               return;
+
        // depth == 0 is for the root, not yet the index, hence the increase when going to vector size.
        for (IndexNode* child : node->children) {
                if (entry.terms()[depth] == termAtLevel(child, depth)) {
                        if (depth + 1 == entry.terms().size()) { // == child.entries.begin()->terms().size()
                                // All term entries match: it's an entry.
-                               child->entries.emplace_back(entry);
+                               if (!entry.terms()[depth].empty())
+                                       child->entries.emplace_back(entry);
                                return;
                        } else {
                                insertIntoNode(entry, child, depth + 1);
@@ -1612,7 +1735,7 @@ void outputIndexPage(XMLStream & xs, const IndexNode* root_node, unsigned depth
 
        xs << xml::StartTag("li", "class='" + generateCssClassAtDepth(depth) + "'");
        xs << xml::CR();
-       xs << XMLStream::ESCAPE_NONE << termAtLevel(root_node, depth);
+       xs << termAtLevel(root_node, depth);
        // By tree assumption, all the entries at this node have the same set of terms.
 
        if (!root_node->entries.empty()) {
@@ -1665,6 +1788,7 @@ void outputIndexPage(XMLStream & xs, const IndexNode* root_node, unsigned depth
                        }
                        entry_number += 1;
                }
+               xs << xml::CR();
        }
 
        if (!root_node->entries.empty() && !root_node->children.empty()) {
@@ -1687,7 +1811,7 @@ void outputIndexPage(XMLStream & xs, const IndexNode* root_node, unsigned depth
        xs << xml::CR();
 }
 
-// Only useful for debugging.
+#ifdef LYX_INSET_INDEX_DEBUG
 void printTree(const IndexNode* root_node, unsigned depth = 0)
 {
        static const std::string pattern = "    ";
@@ -1722,6 +1846,7 @@ void printTree(const IndexNode* root_node, unsigned depth = 0)
                printTree(node, depth + 1);
        }
 }
+#endif // LYX_INSET_INDEX_DEBUG
 }
 
 
@@ -1747,7 +1872,7 @@ docstring InsetPrintIndex::xhtml(XMLStream &, OutputParams const & op) const
                return docstring();
 
        const IndexNode* index_root = buildIndexTree(entries);
-#if 0
+#ifdef LYX_INSET_INDEX_DEBUG
        printTree(index_root);
 #endif