fix: Linker sometimes adds '...' when there are no alternative defini…

…tions (Closes #148)

lib/linker.js

@@ -123,10 +123,12 @@ function getLinkifyVisitor(context, vFile, indexEntriesMap) {
  */
 function linkifyAst(paragraphNode, headingNode, indexEntrySet, context, vFile) {
     const {linking} = context.conf;
+    const skipOnTooMuchAlternativeDefs = Math.sign(linking.limitByAlternatives) < 0;
+    const maxAlternativeDefs = Math.abs(linking.limitByAlternatives);
     const countDefinitions = indexEntrySet.length;
-    const maxAlternativeDefs = linking.limitByAlternatives;
-    const hasAlternativeDefs = countDefinitions > 1;
-    const hasMoreAlternativeDefs = countDefinitions > maxAlternativeDefs;
+    const countAlternativeDefs = countDefinitions - 1;
+    const hasAlternativeDefs = countAlternativeDefs > 0;
+    const hasMoreAlternativeDefs = countAlternativeDefs > maxAlternativeDefs;
     const termNodes = indexEntrySet.map(e => e.node);
     const termNode = termNodes[0];
     const maxReplacements = (linking.mentions === "first-in-paragraph") ? 1 : Infinity;
@@ -136,13 +138,12 @@ function linkifyAst(paragraphNode, headingNode, indexEntrySet, context, vFile) {
     if (linking.headingDepths[termNode.headingDepth] !== true) {
         return;
     }
-
-    if (maxAlternativeDefs < 0 && countDefinitions > Math.abs(maxAlternativeDefs)) {
+    if (skipOnTooMuchAlternativeDefs && countAlternativeDefs >= maxAlternativeDefs) {
         return;
     }
-    if (hasAlternativeDefs) {
+    if (hasAlternativeDefs && maxAlternativeDefs !== 0) {
         linkNodes = termNodes
-            .filter((e, idx) => idx < maxAlternativeDefs)
+            .filter((e, idx) => idx > 0 && idx <= maxAlternativeDefs)
             .map(getSuperscriptLinkMapper(context, vFile));
         if (hasMoreAlternativeDefs) {
             linkNodes.push(html("<sup>...</sup>"));
@@ -186,7 +187,7 @@ function getSuperscriptLinkMapper(context, vFile) {
         const shortDesc = tNode.getShortDescription();
         const glossUrl = resolveGlossaryUrl(context, vFile, tNode);
         return link(glossUrl, shortDesc,
-            html(j === 0 ? `<sup>${j+1})</sup>` : `<sup> ${j+1})</sup>`)
+            html(j === 0 ? `<sup>${j+2})</sup>` : `<sup> ${j+2})</sup>`)
         );
     };
 }