Disable LFUN_INDEX_TAG_ALL if there is noting to tag (#12812)

src/BufferView.cpp

@@ -1759,6 +1759,10 @@ void BufferView::dispatch(FuncRequest const & cmd, DispatchResult & dr)
 	}
 
 	case LFUN_INDEX_TAG_ALL: {
+		if (cur.pos() == 0)
+			// nothing precedes
+			break;
+
 		Inset * ins = cur.nextInset();
 		if (!ins || ins->lyxCode() != INDEX_CODE)
 			// not at index inset
@@ -1795,6 +1799,8 @@ void BufferView::dispatch(FuncRequest const & cmd, DispatchResult & dr)
 		// Get word or selection
 		cur.text()->selectWord(cur, WHOLE_WORD);
 		docstring const searched_string = cur.selectionAsString(false);
+		if (searched_string.empty())
+			break;
 		// Start from the beginning
 		lyx::dispatch(FuncRequest(LFUN_BUFFER_BEGIN));
 		while (findOne(this, searched_string,

src/insets/InsetIndex.cpp

@@ -27,6 +27,7 @@
 #include "IndicesList.h"
 #include "InsetList.h"
 #include "Language.h"
+#include "Paragraph.h"
 #include "LaTeX.h"
 #include "LaTeXFeatures.h"
 #include "Lexer.h"
@@ -693,8 +694,31 @@ bool InsetIndex::getStatus(Cursor & cur, FuncRequest const & cmd,
 	case LFUN_INDEXMACRO_INSERT:
 		return macrosPossible(cmd.getArg(0));
 
-	case LFUN_INDEX_TAG_ALL:
-		return true;
+	case LFUN_INDEX_TAG_ALL: {
+		if (cur.pos() == 0)
+			// nothing to tag
+			return false;
+		// move backwards into preceding word
+		// skip over other index insets
+		DocIterator dit(cur);
+		dit.backwardPosIgnoreCollapsed();
+		while (true) {
+			if (dit.inset().lyxCode() == INDEX_CODE)
+				dit.pop_back();
+			else if (dit.prevInset() && dit.prevInset()->lyxCode() == INDEX_CODE)
+				dit.backwardPosIgnoreCollapsed();
+			else
+				break;
+		}
+		if (!dit.inTexted())
+			// action not possible
+			return false;
+		// Check if we actually have a word to tag
+		FontSpan tw = dit.locateWord(WHOLE_WORD);
+
+		// action possible if we have a word of at least one char
+		return (tw.size() > 0);
+	}
 
 	default:
 		return InsetCollapsible::getStatus(cur, cmd, flag);