Fix elasticBoost generation

The extra ()'s cause problems if using a single field for sort order
foursquare · Apr 30, 2012 · 3e1eb02 · 3e1eb02
1 parent 4630f5a
commit 3e1eb02
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/build.sbt b/build.sbt
@@ -1,6 +1,6 @@
 name := "slashem"
 
-version := "0.9.10a"
+version := "0.9.10b"
 
 organization := "com.foursquare"
 

diff --git a/src/main/scala/com/foursquare/slashem/Ast.scala b/src/main/scala/com/foursquare/slashem/Ast.scala
@@ -224,7 +224,7 @@ object Ast {
       fieldName
     }
     def elasticBoost(): Pair[List[String],String] = {
-      Pair(Nil,"(doc['" + fieldName + "'].value)")
+      Pair(Nil,"doc['" + fieldName + "'].value")
     }
   }
 

diff --git a/src/test/scala/com/foursquare/slashem/ElasticQueryTest.scala b/src/test/scala/com/foursquare/slashem/ElasticQueryTest.scala
@@ -71,6 +71,13 @@ class ElasticQueryTest extends SpecsMatchers with ScalaCheckMatchers {
   }
 
 
+  @Test
+  def simpleOrderTest {
+    val fullQuery = ESimplePanda.where(_.name contains "lol")
+                    .limit(5).orderDesc(_.followers)
+    val r = fullQuery fetch()
+  }
+
   @Test
   def testEmptySearch {
     try {