Clean up single forward slashes in search queries

See https://hyp.is/4fitdomhEeiNvvdYkHSG-g/
hbz · Aug 31, 2018 · 8e1bbeb · 8e1bbeb
1 parent 8ba3691
commit 8e1bbeb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/app/modules/IndexComponent.java b/app/modules/IndexComponent.java
@@ -266,6 +266,8 @@ public SearchResponse query(String q, String filter, int from, int size) {
 
 	@Override
 	public QueryStringQueryBuilder queryStringQuery(String q) {
+		// Clean up single forward slash, but keep regular /expressions/
+		q = q.indexOf('/') == q.lastIndexOf('/') ? q.replace("/", " ") : q;
 		return QueryBuilders.queryStringQuery(q).defaultOperator(Operator.AND);
 	}
 }
diff --git a/test/modules/IndexQueryTest.java b/test/modules/IndexQueryTest.java
@@ -57,6 +57,12 @@ public void testBooleanSearch() {
 		Assert.assertEquals(6, index.query("twain OR schriftsteller").getHits().getTotalHits());
 	}
 
+	@Test
+	public void testSlashInQuery() {
+		Assert.assertEquals(1, index.query("Hauptschule / Lehrer").getHits().getTotalHits());
+		Assert.assertEquals(1, index.query("/[Hh]auptschulen?/").getHits().getTotalHits());
+	}
+
 	@Test
 	public void testContextQuery() {
 		Assert.assertEquals(0, index.query("context.jsonld").getHits().getTotalHits());