Don't need the Map method in SqlColumn

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/SqlColumn.java

@@ -18,7 +18,6 @@
 import java.sql.JDBCType;
 import java.util.Objects;
 import java.util.Optional;
-import java.util.function.Function;
 
 import org.jetbrains.annotations.NotNull;
 import org.mybatis.dynamic.sql.render.RenderingContext;
@@ -174,10 +173,6 @@ public <S> SqlColumn<S> withJavaType(Class<S> javaType) {
         return b.withJavaType(javaType).build();
     }
 
-    public <R> R map(Function<SqlColumn<?>, R> mapper) {
-        return mapper.apply(this);
-    }
-
     /**
      * This method helps us tell a bit of fiction to the Java compiler. Java, for better or worse,
      * does not carry generic type information through chained methods. We want to enable method

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

@@ -85,7 +85,7 @@ private Optional<FieldAndValueAndParameters> buildNullFragment(AbstractColumnMap
     }
 
     private Optional<FieldAndValueAndParameters> buildFragment(AbstractColumnMapping mapping, Object value) {
-        RenderedParameterInfo parameterInfo = mapping.column().map(renderingContext::calculateParameterInfo);
+        RenderedParameterInfo parameterInfo = renderingContext.calculateParameterInfo(mapping.column());
 
         return FieldAndValueAndParameters.withFieldName(mapping.columnName())
                 .withValuePhrase(parameterInfo.renderedPlaceHolder())

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

@@ -58,14 +58,14 @@ public FieldAndValueAndParameters visit(StringConstantMapping mapping) {
     @Override
     public FieldAndValueAndParameters visit(PropertyMapping mapping) {
         return FieldAndValueAndParameters.withFieldName(mapping.columnName())
-                .withValuePhrase(mapping.column().map(c -> calculateJdbcPlaceholder(c, mapping.property())))
+                .withValuePhrase(calculateJdbcPlaceholder(mapping.column(), mapping.property()))
                 .build();
     }
 
     @Override
     public FieldAndValueAndParameters visit(RowMapping mapping) {
         return FieldAndValueAndParameters.withFieldName(mapping.columnName())
-                .withValuePhrase(mapping.column().map(this::calculateJdbcPlaceholder))
+                .withValuePhrase(calculateJdbcPlaceholder(mapping.column()))
                 .build();
     }
 

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

@@ -60,7 +60,7 @@ public Optional<FieldAndValueAndParameters> visit(StringConstantMapping mapping)
     @Override
     public Optional<FieldAndValueAndParameters> visit(PropertyMapping mapping) {
         return FieldAndValueAndParameters.withFieldName(mapping.columnName())
-                .withValuePhrase(mapping.column().map(c -> calculateJdbcPlaceholder(c, mapping.property())))
+                .withValuePhrase(calculateJdbcPlaceholder(mapping.column(), mapping.property()))
                 .buildOptional();
     }
 
@@ -76,7 +76,7 @@ public Optional<FieldAndValueAndParameters> visit(PropertyWhenPresentMapping map
     @Override
     public Optional<FieldAndValueAndParameters> visit(RowMapping mapping) {
         return FieldAndValueAndParameters.withFieldName(mapping.columnName())
-                .withValuePhrase(mapping.column().map(this::calculateJdbcPlaceholder))
+                .withValuePhrase(calculateJdbcPlaceholder(mapping.column()))
                 .buildOptional();
     }
 

src/main/java/org/mybatis/dynamic/sql/update/render/SetPhraseVisitor.java

@@ -49,15 +49,15 @@ public Optional<FragmentAndParameters> visit(NullMapping mapping) {
 
     @Override
     public Optional<FragmentAndParameters> visit(ConstantMapping mapping) {
-        String fragment = mapping.column().map(renderingContext::aliasedColumnName)
+        String fragment = renderingContext.aliasedColumnName(mapping.column())
                 + " = " + mapping.constant(); //$NON-NLS-1$
         return FragmentAndParameters.withFragment(fragment)
                 .buildOptional();
     }
 
     @Override
     public Optional<FragmentAndParameters> visit(StringConstantMapping mapping) {
-        String fragment = mapping.column().map(renderingContext::aliasedColumnName)
+        String fragment = renderingContext.aliasedColumnName(mapping.column())
                 + " = " //$NON-NLS-1$
                 + StringUtilities.formatConstantForSQL(mapping.constant());
 
@@ -85,7 +85,7 @@ public <T> Optional<FragmentAndParameters> visit(ValueWhenPresentMapping<T> mapp
     @Override
     public Optional<FragmentAndParameters> visit(SelectMapping mapping) {
         SelectStatementProvider selectStatement = mapping.selectModel().render(renderingContext);
-        String fragment = mapping.column().map(renderingContext::aliasedColumnName)
+        String fragment = renderingContext.aliasedColumnName(mapping.column())
                 + " = (" //$NON-NLS-1$
                 + selectStatement.getSelectStatement()
                 + ")"; //$NON-NLS-1$
@@ -99,7 +99,7 @@ public Optional<FragmentAndParameters> visit(SelectMapping mapping) {
     public Optional<FragmentAndParameters> visit(ColumnToColumnMapping mapping) {
         FragmentAndParameters renderedColumn = mapping.rightColumn().render(renderingContext);
 
-        String setPhrase = mapping.column().map(renderingContext::aliasedColumnName)
+        String setPhrase = renderingContext.aliasedColumnName(mapping.column())
                 + " = "  //$NON-NLS-1$
                 + renderedColumn.fragment();
 
@@ -109,8 +109,8 @@ public Optional<FragmentAndParameters> visit(ColumnToColumnMapping mapping) {
     }
 
     private <T> Optional<FragmentAndParameters> buildValueFragment(AbstractColumnMapping mapping, T value) {
-        RenderedParameterInfo parameterInfo = mapping.column().map(renderingContext::calculateParameterInfo);
-        String setPhrase = mapping.column().map(renderingContext::aliasedColumnName)
+        RenderedParameterInfo parameterInfo = renderingContext.calculateParameterInfo(mapping.column());
+        String setPhrase = renderingContext.aliasedColumnName(mapping.column())
                 + " = "  //$NON-NLS-1$
                 + parameterInfo.renderedPlaceHolder();
 
@@ -121,7 +121,7 @@ private <T> Optional<FragmentAndParameters> buildValueFragment(AbstractColumnMap
 
     private Optional<FragmentAndParameters> buildNullFragment(AbstractColumnMapping mapping) {
         return FragmentAndParameters
-                .withFragment(mapping.column().map(renderingContext::aliasedColumnName) + " = null") //$NON-NLS-1$
+                .withFragment(renderingContext.aliasedColumnName(mapping.column()) + " = null") //$NON-NLS-1$
                 .buildOptional();
     }
 }