ignoring corrected offset might be better solution

jigarparekh80 · Aug 10, 2018 · d6f93d1 · d6f93d1
1 parent cf00af2
commit d6f93d1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/main/java/org/opensextant/solrtexttagger/TaggerRequestHandler.java b/src/main/java/org/opensextant/solrtexttagger/TaggerRequestHandler.java
@@ -182,8 +182,10 @@ protected void tagCallback(int startOffset, int endOffset, Object docIdsKey) {
                         startOffset, endOffset);
                 return;
               }
-              startOffset = offsetPair[0];
-              endOffset = offsetPair[1]<0?inputString.length():offsetPair[1];
+              if(offsetPair[0]>=0 && offsetPair[1]>=0 ) {
+            	  startOffset = offsetPair[0];
+              	  endOffset = offsetPair[1];
+              }
             }
 
             NamedList tag = new NamedList();

diff --git a/src/test/java/org/opensextant/solrtexttagger/HtmlInterpolationTest.java b/src/test/java/org/opensextant/solrtexttagger/HtmlInterpolationTest.java
@@ -30,6 +30,7 @@ public class HtmlInterpolationTest extends XmlInterpolationTest {
   public void setUp() throws Exception {
     super.setUp();
     baseParams.set("htmlOffsetAdjust", "true");
+    baseParams.set("matchText", "true");
   }
 
   @Override
@@ -58,6 +59,7 @@ public void testHtml() throws Exception {
     assertXmlTag("start end <em>other text</em>", true);
     assertXmlTag("start end<em> other text</em>", true);
     assertXmlTag("<em>other text</em> start end", true);
+    assertXmlTag("start <td/> end", true);
   }
 
   @Test