[BI-2040] removed all calls to fetchXXXXX(page) and the methods themsleves

Author: davedrp

src/main/java/org/brapi/test/BrAPITestServer/service/pheno/ObservationVariableService.java

@@ -143,13 +143,7 @@ public List<ObservationVariable> findObservationVariables(ObservationVariableSea
 		log.debug("Starting variable search");
 		Page<ObservationVariableEntity> page = observationVariableRepository.findAllBySearch(searchQuery, pageReq);
 		log.debug("Variable search complete");
-		if(!page.isEmpty()) {
-			observationVariableRepository.fetchXrefs(page, ObservationVariableEntity.class);
-			fetchSynonyms(page);
-			fetchMethodXrefs(page);
-			fetchScaleXrefs(page);
-			fetchTraitXrefs(page);
-		}
+
 
 		log.debug("converting "+page.getSize()+" entities");
 		List<ObservationVariable> observationVariables = page.map(this::convertFromEntity).getContent();
@@ -158,75 +152,6 @@ public List<ObservationVariable> findObservationVariables(ObservationVariableSea
 		return observationVariables;
 	}
 
-	public void fetchSynonyms(Page<ObservationVariableEntity> page) {
-		SearchQueryBuilder<ObservationVariableEntity> searchQuery = new SearchQueryBuilder<>(ObservationVariableEntity.class);
-		searchQuery.leftJoinFetch("synonyms", "synonyms")
-				   .appendList(page.stream().map(BrAPIBaseEntity::getId).collect(Collectors.toList()), "id");
-
-		Page<ObservationVariableEntity> synonyms = observationVariableRepository.findAllBySearch(searchQuery, PageRequest.of(0, page.getSize()));
-
-		Map<String, List<String>> synonymsByVar = new HashMap<>();
-		synonyms.forEach(entity -> synonymsByVar.put(entity.getId(), entity.getSynonyms()));
-
-		page.forEach(entity -> entity.setSynonyms(synonymsByVar.get(entity.getMethod().getId())));
-	}
-
-	public void fetchMethodXrefs(Page<ObservationVariableEntity> page) {
-		SearchQueryBuilder<MethodEntity> searchQuery = new SearchQueryBuilder<>(MethodEntity.class);
-		searchQuery.leftJoinFetch("externalReferences", "externalReferences")
-				   .join("variables", "variables")
-				   .appendList(page.stream().map(BrAPIBaseEntity::getId).collect(Collectors.toList()), "*variables.id");
-
-		Page<MethodEntity> xrefs = methodRepository.findAllBySearch(searchQuery, PageRequest.of(0, page.getSize()));
-
-		Map<String, List<ExternalReferenceEntity>> xrefByEntity = new HashMap<>();
-		xrefs.forEach(entity -> xrefByEntity.put(entity.getId(), entity.getExternalReferences()));
-
-		page.forEach(entity -> entity.getMethod().setExternalReferences(xrefByEntity.get(entity.getMethod().getId())));
-	}
-
-	public void fetchScaleXrefs(Page<ObservationVariableEntity> page) {
-		SearchQueryBuilder<ScaleEntity> searchQuery = new SearchQueryBuilder<>(ScaleEntity.class);
-		searchQuery.leftJoinFetch("externalReferences", "externalReferences")
-				   .join("variables", "variables")
-				   .appendList(page.stream().map(BrAPIBaseEntity::getId).collect(Collectors.toList()), "*variables.id");
-
-		Page<ScaleEntity> xrefs = scaleRepository.findAllBySearch(searchQuery, PageRequest.of(0, page.getSize()));
-
-		Map<String, List<ExternalReferenceEntity>> xrefByEntity = new HashMap<>();
-		xrefs.forEach(entity -> xrefByEntity.put(entity.getId(), entity.getExternalReferences()));
-
-		page.forEach(entity -> entity.getScale().setExternalReferences(xrefByEntity.get(entity.getMethod().getId())));
-	}
-
-	public void fetchScaleValidValueCategories(Page<ObservationVariableEntity> page) {
-		SearchQueryBuilder<ScaleEntity> searchQuery = new SearchQueryBuilder<>(ScaleEntity.class);
-		searchQuery.leftJoinFetch("validValueCategories", "validValueCategories")
-				   .join("variables", "variables")
-				   .appendList(page.stream().map(BrAPIBaseEntity::getId).collect(Collectors.toList()), "*variables.id");
-
-		Page<ScaleEntity> validValueCategories = scaleRepository.findAllBySearch(searchQuery, PageRequest.of(0, page.getSize()));
-
-		Map<String, List<ScaleValidValueCategoryEntity>> infoByEntity = new HashMap<>();
-		validValueCategories.forEach(entity -> infoByEntity.put(entity.getId(), entity.getValidValueCategories()));
-
-		page.forEach(entity -> entity.getScale().setValidValueCategories(infoByEntity.get(entity.getId())));
-	}
-
-	public void fetchTraitXrefs(Page<ObservationVariableEntity> page) {
-		SearchQueryBuilder<TraitEntity> searchQuery = new SearchQueryBuilder<>(TraitEntity.class);
-		searchQuery.leftJoinFetch("externalReferences", "externalReferences")
-				   .join("variables", "variables")
-				   .appendList(page.stream().map(BrAPIBaseEntity::getId).collect(Collectors.toList()), "*variables.id");
-
-		Page<TraitEntity> xrefs = traitRepository.findAllBySearch(searchQuery, PageRequest.of(0, page.getSize()));
-
-		Map<String, List<ExternalReferenceEntity>> xrefByEntity = new HashMap<>();
-		xrefs.forEach(entity -> xrefByEntity.put(entity.getId(), entity.getExternalReferences()));
-
-		page.forEach(entity -> entity.getTrait().setExternalReferences(xrefByEntity.get(entity.getMethod().getId())));
-	}
-
 	public List<ObservationVariable> saveObservationVariables(@Valid List<ObservationVariableNewRequest> body)
 			throws BrAPIServerException {
 		List<ObservationVariable> savedObservationVariables = new ArrayList<>();