Merging release v0.8

Author: timparsons

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

@@ -543,8 +543,10 @@ private void updateEntity(ObservationUnitPositionEntity entity, ObservationUnitP
 					relationshipEntity.setLevelCode(level.getLevelCode());
 					relationshipEntity.setLevelName(level.getLevelName());
 					relationshipEntity.setLevelOrder(level.getLevelOrder());
-					ObservationUnitEntity parentEntity = getObservationUnitEntity(level.getObservationUnitDbId());
-					relationshipEntity.setObservationUnit(parentEntity);
+					if(level.getObservationUnitDbId() != null) {
+						ObservationUnitEntity parentEntity = getObservationUnitEntity(level.getObservationUnitDbId());
+						relationshipEntity.setObservationUnit(parentEntity);
+					}
 					relationshipEntity.setPosition(entity);
 				} catch (BrAPIServerException e) {
 					e.printStackTrace();

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

@@ -119,50 +119,46 @@ public ScaleEntity updateEntity(ScaleEntity entity, ScaleBaseClass scale) throws
 
 		ontologyService.updateOntologyReference(entity, scale.getOntologyReference());
 
-		if (scale.getValidValues() != null) {
-			if (scale.getValidValues().isPresent()) {
-				ScaleBaseClassValidValues validValues = scale.getValidValues().get();
-
-				Integer maxInt = NumberUtils.isCreatable(entity.getValidValueMax())
-						? Integer.parseInt(entity.getValidValueMax())
-						: null;
-				Optional.ofNullable(UpdateUtility.replaceField(validValues.getMax(), maxInt))
-						.ifPresent(t -> entity.setValidValueMax(t.toString()));
-				entity.setValidValueMax(UpdateUtility.replaceField(validValues.getMaximumValue(), entity.getValidValueMax()));
-
-				Integer minInt = NumberUtils.isCreatable(entity.getValidValueMin())
-						? Integer.parseInt(entity.getValidValueMin())
-						: null;
-				Optional.ofNullable(UpdateUtility.replaceField(validValues.getMin(), minInt))
-						.ifPresent(t -> entity.setValidValueMin(t.toString()));
-				entity.setValidValueMin(UpdateUtility.replaceField(validValues.getMinimumValue(), entity.getValidValueMin()));
-
-				if (validValues.getCategories() != null) {
-					if (validValues.getCategories().isPresent()) {
-						List<ScaleBaseClassValidValuesCategories> categories = validValues.getCategories().get();
-
-						if (entity.getValidValueCategories() != null) {
-							for (ScaleValidValueCategoryEntity catEntity : entity.getValidValueCategories()) {
-								catEntity.setScale(null);
-							}
-						}
-						entity.setValidValueCategories(new ArrayList<>());
-						entity.setValidValueCategories(categories.stream().map(cat -> {
-							ScaleValidValueCategoryEntity catEntity = new ScaleValidValueCategoryEntity();
-							catEntity.setLabel(cat.getLabel());
-							catEntity.setValue(cat.getValue());
-							catEntity.setScale(entity);
-							return catEntity;
-						}).collect(Collectors.toList()));
-					} else {
-						entity.setValidValueCategories(null);
+		if (scale.getValidValues() != null && scale.getValidValues().isPresent()) {
+			ScaleBaseClassValidValues validValues = scale.getValidValues().get();
+
+			Integer maxInt = NumberUtils.isCreatable(entity.getValidValueMax())
+					? Integer.parseInt(entity.getValidValueMax())
+					: null;
+			Optional.ofNullable(UpdateUtility.replaceField(validValues.getMax(), maxInt))
+					.ifPresent(t -> entity.setValidValueMax(t.toString()));
+			entity.setValidValueMax(UpdateUtility.replaceField(validValues.getMaximumValue(), entity.getValidValueMax()));
+
+			Integer minInt = NumberUtils.isCreatable(entity.getValidValueMin())
+					? Integer.parseInt(entity.getValidValueMin())
+					: null;
+			Optional.ofNullable(UpdateUtility.replaceField(validValues.getMin(), minInt))
+					.ifPresent(t -> entity.setValidValueMin(t.toString()));
+			entity.setValidValueMin(UpdateUtility.replaceField(validValues.getMinimumValue(), entity.getValidValueMin()));
+
+			if (validValues.getCategories() != null && validValues.getCategories().isPresent()) {
+				List<ScaleBaseClassValidValuesCategories> categories = validValues.getCategories().get();
+
+				if (entity.getValidValueCategories() != null) {
+					for (ScaleValidValueCategoryEntity catEntity : entity.getValidValueCategories()) {
+						catEntity.setScale(null);
 					}
 				}
+				entity.setValidValueCategories(new ArrayList<>());
+				entity.setValidValueCategories(categories.stream().map(cat -> {
+					ScaleValidValueCategoryEntity catEntity = new ScaleValidValueCategoryEntity();
+					catEntity.setLabel(cat.getLabel());
+					catEntity.setValue(cat.getValue());
+					catEntity.setScale(entity);
+					return catEntity;
+				}).collect(Collectors.toList()));
 			} else {
 				entity.setValidValueCategories(null);
-				entity.setValidValueMax(null);
-				entity.setValidValueMin(null);
 			}
+		} else {
+			entity.setValidValueCategories(null);
+			entity.setValidValueMax(null);
+			entity.setValidValueMin(null);
 		}
 
 		return entity;

src/main/resources/.gitignore

@@ -1,2 +1,2 @@
-application.properties
+properties/application.properties
 /properties