JAVA-151: Call _checkObject from Insert() as well as Save() to ensure…

… no keys with . are inserted. Test to *expect* an exception.

src/main/com/mongodb/DBApiLayer.java

@@ -193,13 +193,14 @@ protected WriteResult insert(DBObject[] arr, boolean shouldApply , com.mongodb.W
                 for ( int i=0; i<arr.length; i++ ){
                     DBObject o=arr[i];
                     apply( o );
+                    _checkObject( o , false , false );
                     Object id = o.get( "_id" );
                     if ( id instanceof ObjectId ){
                         ((ObjectId)id).notNew();
                     }
                 }
             }
-            
+
             WriteResult last = null;
 
             int cur = 0;

src/main/com/mongodb/DBCollection.java

@@ -817,7 +817,7 @@ protected DBCollection( DB base , String name ){
         _options = new Bytes.OptionHolder( _db._options );
     }
 
-    private  DBObject _checkObject( DBObject o , boolean canBeNull , boolean query ){
+    protected DBObject _checkObject( DBObject o , boolean canBeNull , boolean query ){
         if ( o == null ){
             if ( canBeNull )
                 return null;

src/test/com/mongodb/DBCollectionTest.java

@@ -184,6 +184,15 @@ public void testIndexExceptions(){
 
     }
 
+    @Test( expectedExceptions = IllegalArgumentException.class )
+    public void testDotKeysFail() {
+        DBCollection c = _db.getCollection("testdotkeysFail");
+        c.drop();
+
+        DBObject obj = BasicDBObjectBuilder.start().add("x",1).add("y",2).add("foo.bar","baz").get();
+        c.insert(obj);
+    }
+
 
     final DB _db;