You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/05/22 13:53:53 UTC
(commons-dbcp) 02/05: Fix PMD UnnecessaryFullyQualifiedName in PoolableCallableStatement
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-dbcp.git
commit c6454a8aae5924f32e967728faf20103990b835c
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Wed May 22 09:48:46 2024 -0400
Fix PMD UnnecessaryFullyQualifiedName in PoolableCallableStatement
---
src/changes/changes.xml | 1 +
src/main/java/org/apache/commons/dbcp2/PoolableCallableStatement.java | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index f9a8e71c..fdf4e984 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -65,6 +65,7 @@ The <action> type attribute can be add,update,fix,remove.
<release version="2.12.1" date="YYYY-MM-DD" description="This is a minor release, including bug fixes and enhancements.">
<!-- FIX -->
<action type="update" dev="ggregory" due-to="Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in AbandonedTrace.</action>
+ <action type="update" dev="ggregory" due-to="Gary Gregory">Fix PMD UnnecessaryFullyQualifiedName in PoolableCallableStatement.</action>
<!-- ADD -->
<!-- UPDATE -->
<action type="update" dev="ggregory" due-to="Gary Gregory">Bump org.apache.commons:commons-parent from 66 to 69 #360, #371.</action>
diff --git a/src/main/java/org/apache/commons/dbcp2/PoolableCallableStatement.java b/src/main/java/org/apache/commons/dbcp2/PoolableCallableStatement.java
index 64691fd1..382dd371 100644
--- a/src/main/java/org/apache/commons/dbcp2/PoolableCallableStatement.java
+++ b/src/main/java/org/apache/commons/dbcp2/PoolableCallableStatement.java
@@ -76,7 +76,7 @@ public class PoolableCallableStatement extends DelegatingCallableStatement {
@Override
public void activate() throws SQLException {
setClosedInternal(false);
- AbandonedTrace.add(getConnectionInternal(), this);
+ add(getConnectionInternal(), this);
super.activate();
}