remove cglib dependency and comments

intuit · Jun 3, 2022 · 21c7c74 · 21c7c74
1 parent 9901742
commit 21c7c74
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 27 deletions.
diff --git a/ipp-v3-java-devkit/pom.xml b/ipp-v3-java-devkit/pom.xml
@@ -43,12 +43,7 @@
           <artifactId>jaxb2-commons-lang</artifactId>
         </exclusion>
       </exclusions> 
-    </dependency>  
-    <dependency> 
-      <groupId>cglib</groupId>  
-      <artifactId>cglib</artifactId>  
-      <version>3.2.12</version> 
-    </dependency>  
+    </dependency>
     <dependency> 
       <groupId>asm</groupId>  
       <artifactId>asm-commons</artifactId>  

diff --git a/ipp-v3-java-devkit/src/main/java/com/intuit/ipp/query/GenerateQuery.java b/ipp-v3-java-devkit/src/main/java/com/intuit/ipp/query/GenerateQuery.java
@@ -65,7 +65,11 @@ public final class GenerateQuery {
 	private GenerateQuery() {
 	}
 
-
+	/**
+	 *
+	 * @param cl the class
+	 * @return the proxified object
+	 */
 	@SuppressWarnings("unchecked")
 	public static <T> T createQueryEntity(Class<T> cl) {
 		Class<?> proxied = null;
@@ -77,7 +81,6 @@ public static <T> T createQueryEntity(Class<T> cl) {
 					.method(ElementMatchers.not(ElementMatchers.isClone().or(ElementMatchers.isFinalizer()).or(ElementMatchers.isEquals()).or(ElementMatchers.isHashCode()).or(ElementMatchers.isToString())))
 					.intercept(MethodDelegation.to(new MyMethodInterceptor()))
 					.make()
-//					.load(cl.getClassLoader(), ClassLoadingStrategy.Default.WRAPPER)
 					.load(ClassLoader.getSystemClassLoader(), ClassLoadingStrategy.Default.WRAPPER)
 					.getLoaded();
 		}
@@ -88,6 +91,11 @@ public static <T> T createQueryEntity(Class<T> cl) {
 		}
 	}
 
+	/**
+	 *
+	 * @param entity the entity
+	 * @return the proxified object
+	 */
 	@SuppressWarnings("unchecked")
 	public static <T> T createQueryEntity(T entity) {
 		Class<?> cl = entity.getClass();
@@ -97,14 +105,9 @@ public static <T> T createQueryEntity(T entity) {
 		} else {
 			proxied = new ByteBuddy()
 					.subclass(cl)
-//					.method(ElementMatchers.any())
-//					.intercept(MethodDelegation.to(new MyMethodInterceptor()))
-//					.method(ElementMatchers.isClone().or(ElementMatchers.isFinalizer()).or(ElementMatchers.isEquals()).or(ElementMatchers.isHashCode()).or(ElementMatchers.isToString()))
-//					.intercept(SuperMethodCall.INSTANCE)
 					.method(ElementMatchers.not(ElementMatchers.isClone().or(ElementMatchers.isFinalizer()).or(ElementMatchers.isEquals()).or(ElementMatchers.isHashCode()).or(ElementMatchers.isToString())))
 					.intercept(MethodDelegation.to(new MyMethodInterceptor()))
 					.make()
-//					.load(cl.getClassLoader(), ClassLoadingStrategy.Default.WRAPPER)
 					.load(ClassLoader.getSystemClassLoader(), ClassLoadingStrategy.Default.WRAPPER)
 					.getLoaded();
 		}
@@ -271,18 +274,4 @@ public static void setMessage(QueryMessage mess) {
 	public static void resetQueryMessage() {
 		setMessage(new QueryMessage());
 	}
-
-
-	public static void main(String[] args) {
-		Customer customer = createQueryEntity(Customer.class);
-		String query = select($(customer.getId()), $(customer.getDisplayName())).where($(customer.getId()).eq("10")).generate();
-		System.out.println(query);
-//		Class<?> proxied = new ByteBuddy()
-//				.subclass(Customer.class)
-//				.method(ElementMatchers.not(ElementMatchers.isClone().or(ElementMatchers.isFinalizer()).or(ElementMatchers.isEquals()).or(ElementMatchers.isHashCode()).or(ElementMatchers.isToString())))
-//				.intercept(MethodDelegation.to(new MyMethodInterceptor()))
-//				.make()
-//				.load(ClassLoader.getSystemClassLoader(), ClassLoadingStrategy.Default.WRAPPER)
-//				.getLoaded();
-	}
 }