SOLR-17617: Add missing Inject to CollectionProperty and InstallCoreData

apache · Jan 10, 2025 · ab49b04 · ab49b04
1 parent 1425e0e
commit ab49b04
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
@@ -169,6 +169,8 @@ Bug Fixes
 * SOLR-17587: Metrics: wt=prometheus fix for non-compliant exposition format containing duplicate TYPE lines.
   (Matthew Biscocho via David Smiley)
 
+* SOLR-17617: Fix v2 CollectionPropertyApi and InstallCoreDataApi. (Colvin Cowie)
+
 Dependency Upgrades
 ---------------------
 * SOLR-17471: Upgrade Lucene to 9.12.1. (Pierre Salagnac, Christine Poerschke)

diff --git a/solr/core/src/java/org/apache/solr/handler/admin/api/CollectionProperty.java b/solr/core/src/java/org/apache/solr/handler/admin/api/CollectionProperty.java
@@ -20,6 +20,8 @@
 import static org.apache.solr.security.PermissionNameProvider.Name.COLL_EDIT_PERM;
 
 import java.io.IOException;
+
+import jakarta.inject.Inject;
 import org.apache.solr.client.api.endpoint.CollectionPropertyApi;
 import org.apache.solr.client.api.model.SolrJerseyResponse;
 import org.apache.solr.client.api.model.UpdateCollectionPropertyRequestBody;
@@ -38,6 +40,7 @@
  */
 public class CollectionProperty extends AdminAPIBase implements CollectionPropertyApi {
 
+  @Inject
   public CollectionProperty(
       CoreContainer coreContainer,
       SolrQueryRequest solrQueryRequest,

diff --git a/solr/core/src/java/org/apache/solr/handler/admin/api/InstallCoreData.java b/solr/core/src/java/org/apache/solr/handler/admin/api/InstallCoreData.java
@@ -20,6 +20,8 @@
 import static org.apache.solr.security.PermissionNameProvider.Name.CORE_EDIT_PERM;
 
 import java.net.URI;
+
+import jakarta.inject.Inject;
 import org.apache.solr.client.api.endpoint.InstallCoreDataApi;
 import org.apache.solr.client.api.model.InstallCoreDataRequestBody;
 import org.apache.solr.client.api.model.SolrJerseyResponse;
@@ -43,6 +45,7 @@
  */
 public class InstallCoreData extends CoreAdminAPIBase implements InstallCoreDataApi {
 
+  @Inject
   public InstallCoreData(
       CoreContainer coreContainer,
       CoreAdminHandler.CoreAdminAsyncTracker coreAdminAsyncTracker,