Add a new visitor so we don't need to pass a weird string value

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/insert/render/BatchInsertRenderer.java

@@ -33,7 +33,7 @@ private BatchInsertRenderer(Builder<T> builder) {
     }
 
     public BatchInsert<T> render() {
-        ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy, "record"); //$NON-NLS-1$
+        ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy);
         FieldAndValueCollector collector = model.mapColumnMappings(MultiRowRenderingUtilities.toFieldAndValue(visitor))
                 .collect(FieldAndValueCollector.collect());
 

src/main/java/org/mybatis/dynamic/sql/insert/render/InsertRenderer.java

@@ -35,7 +35,7 @@ private InsertRenderer(Builder<T> builder) {
     }
 
     public InsertStatementProvider<T> render() {
-        ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy, "record"); //$NON-NLS-1$
+        ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy);
         FieldAndValueCollector collector = model.mapColumnMappings(toFieldAndValue(visitor))
                 .collect(FieldAndValueCollector.collect());
 

src/main/java/org/mybatis/dynamic/sql/insert/render/MultiRowInsertRenderer.java

@@ -33,7 +33,7 @@ private MultiRowInsertRenderer(Builder<T> builder) {
     }
 
     public MultiRowInsertStatementProvider<T> render() {
-        ValuePhraseVisitor visitor = new ValuePhraseVisitor(renderingStrategy, "records[%s]"); //$NON-NLS-1$
+        ValuePhraseVisitor visitor = new MultiRowValuePhraseVisitor(renderingStrategy);
         FieldAndValueCollector collector = model.mapColumnMappings(MultiRowRenderingUtilities.toFieldAndValue(visitor))
                 .collect(FieldAndValueCollector.collect());
 

src/main/java/org/mybatis/dynamic/sql/insert/render/MultiRowValuePhraseVisitor.java

@@ -0,0 +1,41 @@
+/**
+ *    Copyright 2016-2019 the original author or authors.
+ *
+ *    Licensed under the Apache License, Version 2.0 (the "License");
+ *    you may not use this file except in compliance with the License.
+ *    You may obtain a copy of the License at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *    Unless required by applicable law or agreed to in writing, software
+ *    distributed under the License is distributed on an "AS IS" BASIS,
+ *    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *    See the License for the specific language governing permissions and
+ *    limitations under the License.
+ */
+package org.mybatis.dynamic.sql.insert.render;
+
+import java.util.function.Function;
+
+import org.mybatis.dynamic.sql.SqlColumn;
+import org.mybatis.dynamic.sql.render.RenderingStrategy;
+import org.mybatis.dynamic.sql.util.PropertyMapping;
+
+public class MultiRowValuePhraseVisitor extends ValuePhraseVisitor {
+
+    public MultiRowValuePhraseVisitor(RenderingStrategy renderingStrategy) {
+        super(renderingStrategy);
+    }
+
+    @Override
+    public FieldAndValue visit(PropertyMapping mapping) {
+        return FieldAndValue.withFieldName(mapping.mapColumn(SqlColumn::name))
+                .withValuePhrase(mapping.mapColumn(toMultiRowJdbcPlaceholder(mapping.property())))
+                .build();
+    }
+
+    private Function<SqlColumn<?>, String> toMultiRowJdbcPlaceholder(String parameterName) {
+        return column -> renderingStrategy.getFormattedJdbcPlaceholder(column, "records[%s]", //$NON-NLS-1$
+                parameterName);
+    }
+}

src/main/java/org/mybatis/dynamic/sql/insert/render/ValuePhraseVisitor.java

@@ -27,12 +27,10 @@
 
 public class ValuePhraseVisitor implements InsertMappingVisitor<FieldAndValue> {
 
-    private RenderingStrategy renderingStrategy;
-    private String prefix;
+    protected RenderingStrategy renderingStrategy;
 
-    public ValuePhraseVisitor(RenderingStrategy renderingStrategy, String prefix) {
+    public ValuePhraseVisitor(RenderingStrategy renderingStrategy) {
         this.renderingStrategy = renderingStrategy;
-        this.prefix = prefix;
     }
 
     @Override
@@ -64,6 +62,6 @@ public FieldAndValue visit(PropertyMapping mapping) {
     }
 
     private Function<SqlColumn<?>, String> toJdbcPlaceholder(String parameterName) {
-        return column -> renderingStrategy.getFormattedJdbcPlaceholder(column, prefix, parameterName);
+        return column -> renderingStrategy.getFormattedJdbcPlaceholder(column, "record", parameterName); //$NON-NLS-1$
     }
 }