Fix forceInsertAll for single argument

slick · Aug 12, 2014 · b0b0252 · b0b0252
1 parent f0bc906
commit b0b0252
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/slick-testkit/src/main/scala/com/typesafe/slick/testkit/tests/InsertTest.scala b/slick-testkit/src/main/scala/com/typesafe/slick/testkit/tests/InsertTest.scala
@@ -112,6 +112,8 @@ class InsertTest extends TestkitTest[JdbcTestDB] {
       ts.forceInsert(104, "A")
       ts.map(_.ins).forceInsertAll((105, "B"), (106, "C"))
       assertEquals(3, ts.filter(_.id > 100).length.run)
+      ts.map(_.ins).forceInsertAll((111, "D"))
+      assertEquals(4, ts.filter(_.id > 100).length.run)
     }
   }
 

diff --git a/src/main/scala/scala/slick/driver/JdbcInsertInvokerComponent.scala b/src/main/scala/scala/slick/driver/JdbcInsertInvokerComponent.scala
@@ -189,7 +189,8 @@ trait JdbcInsertInvokerComponent extends BasicInsertInvokerComponent{ driver: Jd
     final def forceInsertAll(values: U*)(implicit session: Backend#Session): MultiInsertResult = internalInsertAll(compiled.forceInsert, values: _*)
 
     protected def internalInsertAll(a: compiled.Artifacts, values: U*)(implicit session: Backend#Session): MultiInsertResult = session.withTransaction {
-      if(!useBatchUpdates || (values.isInstanceOf[IndexedSeq[_]] && values.length < 2)) retMany(values, values.map(insert))
+      if(!useBatchUpdates || (values.isInstanceOf[IndexedSeq[_]] && values.length < 2))
+        retMany(values, values.map(v => internalInsert(a, v)))
       else preparedInsert(a.sql) { st =>
         st.clearParameters()
         for(value <- values) {