From 3a94ca1a6c00e516ed36d95fb2a16764b22eb5e1 Mon Sep 17 00:00:00 2001
From: Lukas Eder <lukas.eder@gmail.com>
Date: Sat, 27 Oct 2012 20:28:26 +0200
Subject: [PATCH] [#1887] Remove all deprecated code - Removed
 QueryPartInternal API

---
 .../main/java/org/jooq/QueryPartInternal.java | 25 -----------
 .../jooq/impl/AbstractDelegatingQuery.java    | 19 ++++++++
 .../java/org/jooq/impl/AbstractQuery.java     | 27 +++++++++++-
 .../java/org/jooq/impl/AbstractQueryPart.java | 44 -------------------
 4 files changed, 45 insertions(+), 70 deletions(-)

diff --git a/jOOQ/src/main/java/org/jooq/QueryPartInternal.java b/jOOQ/src/main/java/org/jooq/QueryPartInternal.java
index b8cf8941c6..bb85b51cb7 100644
--- a/jOOQ/src/main/java/org/jooq/QueryPartInternal.java
+++ b/jOOQ/src/main/java/org/jooq/QueryPartInternal.java
@@ -36,9 +36,6 @@
 
 package org.jooq;
 
-import java.util.List;
-import java.util.Map;
-
 import org.jooq.exception.DataAccessException;
 
 /**
@@ -60,28 +57,6 @@ public interface QueryPartInternal extends QueryPart {
      */
     void toSQL(RenderContext context);
 
-    /**
-     * Retrieve the bind values that will be bound by this {@link QueryPart}
-     * <p>
-     * This method is exposed publicly in {@link Query#getBindValues()}
-     */
-    List<Object> getBindValues();
-
-    /**
-     * Retrieve the named parameters that will be bound by this
-     * {@link QueryPart}
-     * <p>
-     * This method is exposed publicly in {@link Query#getParams()}
-     */
-    Map<String, Param<?>> getParams();
-
-    /**
-     * Retrieve a named parameter that will be bound by this {@link QueryPart}
-     * <p>
-     * This method is exposed publicly in {@link Query#getParam(String)}
-     */
-    Param<?> getParam(String name);
-
     /**
      * Bind all parameters of this {@link QueryPart} to a PreparedStatement
      * <p>
diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractDelegatingQuery.java b/jOOQ/src/main/java/org/jooq/impl/AbstractDelegatingQuery.java
index 4214864e71..658efb9f9c 100644
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractDelegatingQuery.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractDelegatingQuery.java
@@ -35,9 +35,13 @@
  */
 package org.jooq.impl;
 
+import java.util.List;
+import java.util.Map;
+
 import org.jooq.AttachableInternal;
 import org.jooq.BindContext;
 import org.jooq.Configuration;
+import org.jooq.Param;
 import org.jooq.Query;
 import org.jooq.RenderContext;
 
@@ -65,6 +69,21 @@ abstract class AbstractDelegatingQuery<Q extends Query> extends AbstractQueryPar
         return super.getConfiguration();
     }
 
+    @Override
+    public final List<Object> getBindValues() {
+        return delegate.getBindValues();
+    }
+
+    @Override
+    public final Map<String, Param<?>> getParams() {
+        return delegate.getParams();
+    }
+
+    @Override
+    public final Param<?> getParam(String name) {
+        return delegate.getParam(name);
+    }
+
     @Override
     public final void toSQL(RenderContext context) {
         context.sql(delegate);
diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractQuery.java b/jOOQ/src/main/java/org/jooq/impl/AbstractQuery.java
index 28fc02d78e..e0cf119622 100644
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractQuery.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractQuery.java
@@ -42,8 +42,10 @@ import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
+import java.util.Map;
 
 import org.jooq.Attachable;
 import org.jooq.AttachableInternal;
@@ -93,9 +95,32 @@ abstract class AbstractQuery extends AbstractQueryPart implements Query, Attacha
     }
 
     // -------------------------------------------------------------------------
-    // The QueryPart and QueryPart internal API
+    // The Query API
     // -------------------------------------------------------------------------
 
+    @Override
+    public final List<Object> getBindValues() {
+        List<Object> result = new ArrayList<Object>();
+
+        for (Param<?> param : getParams().values()) {
+            result.add(param.getValue());
+        }
+
+        return Collections.unmodifiableList(result);
+    }
+
+    @Override
+    public final Map<String, Param<?>> getParams() {
+        ParamCollector collector = new ParamCollector(getConfiguration());
+        collector.bind(this);
+        return Collections.unmodifiableMap(collector.result);
+    }
+
+    @Override
+    public final Param<?> getParam(String name) {
+        return getParams().get(name);
+    }
+
     /**
      * Subclasses may override this for covariant result types
      * <p>
diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractQueryPart.java b/jOOQ/src/main/java/org/jooq/impl/AbstractQueryPart.java
index e182dbaf09..0444bcb714 100644
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractQueryPart.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractQueryPart.java
@@ -37,14 +37,8 @@
 package org.jooq.impl;
 
 import java.sql.SQLException;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.List;
-import java.util.Map;
 
 import org.jooq.Configuration;
-import org.jooq.Param;
-import org.jooq.Query;
 import org.jooq.QueryPart;
 import org.jooq.QueryPartInternal;
 import org.jooq.exception.DataAccessException;
@@ -69,44 +63,6 @@ abstract class AbstractQueryPart implements QueryPartInternal {
     // The QueryPart and QueryPart internal API
     // -------------------------------------------------------------------------
 
-    /**
-     * This method is also declared as {@link Query#getBindValues()}
-     * <p>
-     * {@inheritDoc}
-     */
-    @Override
-    public final List<Object> getBindValues() {
-        List<Object> result = new ArrayList<Object>();
-
-        for (Param<?> param : getParams().values()) {
-            result.add(param.getValue());
-        }
-
-        return Collections.unmodifiableList(result);
-    }
-
-    /**
-     * This method is also declared as {@link Query#getParams()}
-     * <p>
-     * {@inheritDoc}
-     */
-    @Override
-    public final Map<String, Param<?>> getParams() {
-        ParamCollector collector = new ParamCollector(getConfiguration());
-        collector.bind(this);
-        return Collections.unmodifiableMap(collector.result);
-    }
-
-    /**
-     * This method is also declared as {@link Query#getParam(String)}
-     * <p>
-     * {@inheritDoc}
-     */
-    @Override
-    public final Param<?> getParam(String name) {
-        return getParams().get(name);
-    }
-
     /**
      * Subclasses may override this
      */