From eb05766eed9fe6c880b5d73b1306d6bac9103250 Mon Sep 17 00:00:00 2001 From: Dmitriy Kopylenko Date: Mon, 19 Nov 2018 14:04:09 -0500 Subject: [PATCH] SHIBUI-997: Polishing --- .../OpenSamlMetadataResolverDelegate.java | 26 ++++++++----------- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/opensaml/OpenSamlMetadataResolverDelegate.java b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/opensaml/OpenSamlMetadataResolverDelegate.java index 0fb36acf8..45bca6e81 100644 --- a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/opensaml/OpenSamlMetadataResolverDelegate.java +++ b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/resolvers/opensaml/OpenSamlMetadataResolverDelegate.java @@ -31,23 +31,19 @@ public Iterable resolve(@Nullable CriteriaSet criteria) { void addIndexedDescriptorsFromBackingStore(AbstractMetadataResolver.EntityBackingStore backingStore, String resourceId, IndexWriter indexWriter) throws ComponentInitializationException { try { indexWriter.deleteAll(); - } catch (IOException e) { - logger.error(e.getMessage(), e); - } - for (String entityId : backingStore.getIndexedDescriptors().keySet()) { - Document document = new Document(); - document.add(new StringField("id", entityId, Field.Store.YES)); - document.add(new TextField("content", entityId, Field.Store.YES)); // TODO: change entityId to be content of entity descriptor block - document.add(new StringField("tag", resourceId, Field.Store.YES)); - try { - indexWriter.addDocument(document); - } catch (IOException e) { - logger.error(e.getMessage(), e); + for (String entityId : backingStore.getIndexedDescriptors().keySet()) { + Document document = new Document(); + document.add(new StringField("id", entityId, Field.Store.YES)); + document.add(new TextField("content", entityId, Field.Store.YES)); // TODO: change entityId to be content of entity descriptor block + document.add(new StringField("tag", resourceId, Field.Store.YES)); + try { + indexWriter.addDocument(document); + } catch (IOException e) { + logger.error(e.getMessage(), e); + } } - } - try { - indexWriter.commit(); } catch (IOException e) { + logger.error(e.getMessage(), e); throw new ComponentInitializationException(e); } }