[NOJIRA]

Updated beans to try to fix circular dep issue. Fixed a couple conversion issues.. because oops.
TIER · Aug 23, 2018 · 77b208b · 77b208b
1 parent b6105f8
commit 77b208b
Show file tree

Hide file tree

Showing 5 changed files with 66 additions and 45 deletions.
diff --git a/...in/java/edu/internet2/tier/shibboleth/admin/ui/configuration/CoreShibUiConfiguration.java b/...in/java/edu/internet2/tier/shibboleth/admin/ui/configuration/CoreShibUiConfiguration.java
@@ -1,6 +1,5 @@
 package edu.internet2.tier.shibboleth.admin.ui.configuration;
 
-import edu.internet2.tier.shibboleth.admin.ui.domain.frontend.EntityIdsSearchResultRepresentation;
 import edu.internet2.tier.shibboleth.admin.ui.opensaml.OpenSamlObjects;
 import edu.internet2.tier.shibboleth.admin.ui.repository.EntityDescriptorRepository;
 import edu.internet2.tier.shibboleth.admin.ui.repository.MetadataResolverRepository;
@@ -11,6 +10,7 @@
 import edu.internet2.tier.shibboleth.admin.ui.service.DirectoryServiceImpl;
 import edu.internet2.tier.shibboleth.admin.ui.service.EntityDescriptorService;
 import edu.internet2.tier.shibboleth.admin.ui.service.EntityIdsSearchService;
+import edu.internet2.tier.shibboleth.admin.ui.service.EntityIdsSearchServiceImpl;
 import edu.internet2.tier.shibboleth.admin.ui.service.EntityService;
 import edu.internet2.tier.shibboleth.admin.ui.service.FilterService;
 import edu.internet2.tier.shibboleth.admin.ui.service.FilterTargetService;
@@ -24,13 +24,6 @@
 import edu.internet2.tier.shibboleth.admin.util.AttributeUtility;
 import edu.internet2.tier.shibboleth.admin.util.LuceneUtility;
 import org.apache.lucene.analysis.Analyzer;
-import org.apache.lucene.document.Document;
-import org.apache.lucene.index.IndexReader;
-import org.apache.lucene.queryparser.classic.ParseException;
-import org.apache.lucene.queryparser.classic.QueryParser;
-import org.apache.lucene.search.IndexSearcher;
-import org.apache.lucene.search.ScoreDoc;
-import org.apache.lucene.search.TopDocs;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -46,9 +39,6 @@
 import org.springframework.web.util.UrlPathHelper;
 
 import javax.servlet.http.HttpServletRequest;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.List;
 
 @Configuration
 public class CoreShibUiConfiguration {
@@ -92,44 +82,20 @@ public AttributeUtility attributeUtility() {
         return new AttributeUtility(openSamlObjects());
     }
 
-    @Autowired
-    Analyzer fullTokenAnalyzer;
-
-    @Autowired
-    DirectoryService directoryService;
-
     @Autowired
     LocaleResolver localeResolver;
 
     @Autowired
     ResourceBundleMessageSource messageSource;
 
-    @Autowired
-    LuceneUtility luceneUtility;
-
     @Bean
     public EntityDescriptorFilesScheduledTasks entityDescriptorFilesScheduledTasks(EntityDescriptorRepository entityDescriptorRepository) {
         return new EntityDescriptorFilesScheduledTasks(this.metadataDir, entityDescriptorRepository, openSamlObjects());
     }
 
     @Bean
-    public EntityIdsSearchService entityIdsSearchService() {
-        return (resourceId, term, limit) -> {
-            List<String> entityIds = new ArrayList<>();
-            try {
-                IndexReader indexReader = luceneUtility.getIndexReader(resourceId);
-                IndexSearcher searcher = new IndexSearcher(indexReader);
-                QueryParser parser = new QueryParser("content", fullTokenAnalyzer);
-                TopDocs topDocs = searcher.search(parser.parse(term.trim()), limit);
-                for (ScoreDoc scoreDoc : topDocs.scoreDocs) {
-                    Document document = searcher.doc(scoreDoc.doc);
-                    entityIds.add(document.get("id"));
-                }
-            } catch (IOException | ParseException e) {
-                logger.error(e.getMessage(), e);
-            }
-            return new EntityIdsSearchResultRepresentation(entityIds);
-        };
+    public EntityIdsSearchService entityIdsSearchService(LuceneUtility luceneUtility, Analyzer fullTokenAnalyzer) {
+        return new EntityIdsSearchServiceImpl(luceneUtility, fullTokenAnalyzer);
     }
 
     @Bean
@@ -199,7 +165,7 @@ public DirectoryService directoryService() {
     }
 
     @Bean
-    public LuceneUtility luceneUtility() {
-        return new LuceneUtility();
+    public LuceneUtility luceneUtility(DirectoryService directoryService) {
+        return new LuceneUtility(directoryService);
     }
 }
diff --git a/...nd/src/main/java/edu/internet2/tier/shibboleth/admin/ui/service/DirectoryServiceImpl.java b/...nd/src/main/java/edu/internet2/tier/shibboleth/admin/ui/service/DirectoryServiceImpl.java
@@ -3,6 +3,7 @@
 import org.apache.lucene.store.Directory;
 import org.apache.lucene.store.RAMDirectory;
 
+import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -25,6 +26,6 @@ public Directory getDirectory(String resourceId) {
 
     @Override
     public List<Directory> getDirectories() {
-        return (List<Directory>) directoryMap.values();
+        return new ArrayList<>(directoryMap.values());
     }
 }
diff --git a/.../src/main/java/edu/internet2/tier/shibboleth/admin/ui/service/EntityIdsSearchService.java b/.../src/main/java/edu/internet2/tier/shibboleth/admin/ui/service/EntityIdsSearchService.java
@@ -1,6 +1,7 @@
 package edu.internet2.tier.shibboleth.admin.ui.service;
 
 import edu.internet2.tier.shibboleth.admin.ui.domain.frontend.EntityIdsSearchResultRepresentation;
+import edu.internet2.tier.shibboleth.admin.util.LuceneUtility;
 import net.andreinc.mockneat.MockNeat;
 
 import java.util.ArrayList;
@@ -10,7 +11,6 @@
 /**
  * API component responsible for entity ids search.
  */
-@FunctionalInterface
 public interface EntityIdsSearchService {
 
     /**

diff --git a/.../main/java/edu/internet2/tier/shibboleth/admin/ui/service/EntityIdsSearchServiceImpl.java b/.../main/java/edu/internet2/tier/shibboleth/admin/ui/service/EntityIdsSearchServiceImpl.java
@@ -0,0 +1,52 @@
+package edu.internet2.tier.shibboleth.admin.ui.service;
+
+import edu.internet2.tier.shibboleth.admin.ui.domain.frontend.EntityIdsSearchResultRepresentation;
+import edu.internet2.tier.shibboleth.admin.util.LuceneUtility;
+import org.apache.lucene.analysis.Analyzer;
+import org.apache.lucene.document.Document;
+import org.apache.lucene.index.IndexReader;
+import org.apache.lucene.queryparser.classic.ParseException;
+import org.apache.lucene.queryparser.classic.QueryParser;
+import org.apache.lucene.search.IndexSearcher;
+import org.apache.lucene.search.ScoreDoc;
+import org.apache.lucene.search.TopDocs;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+
+/**
+ * @author Bill Smith (wsmith@unicon.net)
+ */
+public class EntityIdsSearchServiceImpl implements EntityIdsSearchService {
+    private static final Logger logger = LoggerFactory.getLogger(EntityIdsSearchServiceImpl.class);
+    private Analyzer fullTokenAnalyzer;
+    private LuceneUtility luceneUtility;
+
+    public EntityIdsSearchServiceImpl(LuceneUtility luceneUtility, Analyzer fullTokenAnalyzer) {
+        this.luceneUtility = luceneUtility;
+        this.fullTokenAnalyzer = fullTokenAnalyzer;
+    }
+
+    @Override
+    public EntityIdsSearchResultRepresentation findBySearchTermAndOptionalLimit(String resourceId,
+                                                                                String searchTerm,
+                                                                                int limit) {
+        List<String> entityIds = new ArrayList<>();
+        try {
+            IndexReader indexReader = luceneUtility.getIndexReader(resourceId);
+            IndexSearcher searcher = new IndexSearcher(indexReader);
+            QueryParser parser = new QueryParser("content", fullTokenAnalyzer);
+            TopDocs topDocs = searcher.search(parser.parse(searchTerm.trim()), limit);
+            for (ScoreDoc scoreDoc : topDocs.scoreDocs) {
+                Document document = searcher.doc(scoreDoc.doc);
+                entityIds.add(document.get("id"));
+            }
+        } catch (IOException | ParseException e) {
+            logger.error(e.getMessage(), e);
+        }
+        return new EntityIdsSearchResultRepresentation(entityIds);
+    }
+}
diff --git a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/util/LuceneUtility.java b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/util/LuceneUtility.java
@@ -8,7 +8,6 @@
 import org.apache.lucene.store.Directory;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.springframework.beans.factory.annotation.Autowired;
 
 import java.io.IOException;
 import java.util.ArrayList;
@@ -19,10 +18,12 @@
  */
 public class LuceneUtility {
     private static final Logger logger = LoggerFactory.getLogger(LuceneUtility.class);
-
-    @Autowired
     private DirectoryService directoryService;
 
+    public LuceneUtility(DirectoryService directoryService) {
+        this.directoryService = directoryService;
+    }
+
     public IndexReader getIndexReader(String resourceId) throws IOException {
         IndexReader indexReader;
         if (StringUtils.isBlank(resourceId)) {
@@ -35,7 +36,8 @@ public IndexReader getIndexReader(String resourceId) throws IOException {
                     logger.error(e.getMessage(), e);
                 }
             });
-            IndexReader[] indexReaders = (IndexReader[]) indexReaderList.toArray();
+            IndexReader[] indexReaders = new IndexReader[indexReaderList.size()];
+            indexReaders = indexReaderList.toArray(indexReaders);
             indexReader = new MultiReader(indexReaders, true);
         } else {
             indexReader = DirectoryReader.open(directoryService.getDirectory(resourceId));