From 82c9ec8652c236600674871e18807e72c7bf45df Mon Sep 17 00:00:00 2001 From: noobyu6 <1044510784@qq.com> Date: Sat, 10 Sep 2022 03:00:54 +0800 Subject: [PATCH] modify class field to local variable --- .../datadeposit/SWORDv2ContainerServlet.java | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/src/main/java/edu/harvard/iq/dataverse/api/datadeposit/SWORDv2ContainerServlet.java b/src/main/java/edu/harvard/iq/dataverse/api/datadeposit/SWORDv2ContainerServlet.java index d8ba8eec4ca..441186cc63f 100644 --- a/src/main/java/edu/harvard/iq/dataverse/api/datadeposit/SWORDv2ContainerServlet.java +++ b/src/main/java/edu/harvard/iq/dataverse/api/datadeposit/SWORDv2ContainerServlet.java @@ -17,9 +17,11 @@ public class SWORDv2ContainerServlet extends SwordServlet { ContainerManagerImpl containerManagerImpl; @Inject StatementManagerImpl statementManagerImpl; - private ContainerManager cm; + // this field can be replaced by local variable +// private ContainerManager cm; private ContainerAPI api; - private StatementManager sm; + // this field can be replaced by local variable +// private StatementManager sm; private final ReentrantLock lock = new ReentrantLock(); @@ -28,13 +30,15 @@ public void init() throws ServletException { super.init(); // load the container manager implementation - this.cm = containerManagerImpl; - - // load the statement manager implementation - this.sm = statementManagerImpl; +// this.cm = containerManagerImpl; + ContainerManager cm = containerManagerImpl; + // load the statement manager implementation +// this.sm = statementManagerImpl; + StatementManager sm = statementManagerImpl; // initialise the underlying servlet processor - this.api = new ContainerAPI(this.cm, this.sm, this.config); +// this.api = new ContainerAPI(this.cm, this.sm, this.config); + this.api = new ContainerAPI(cm, sm, this.config); } @Override