Merge pull request #13746 from SORMAS-Foundation/bugfix-13742-igra_values_not_visible_lab_message

#13742 - Fixed missing disease in Sample/PathogenTest form

Author: raulbob

sormas-ui/src/main/java/de/symeda/sormas/ui/samples/AbstractSampleForm.java

@@ -111,13 +111,16 @@ public abstract class AbstractSampleForm extends AbstractEditForm<SampleDto> {
 					fluidRowLocs(CaseDataDto.OTHER_DELETION_REASON);
     //@formatter:on
 
+	private Disease disease;
+
 	protected AbstractSampleForm(Class<SampleDto> type, String propertyI18nPrefix, Disease disease, UiFieldAccessCheckers fieldAccessCheckers) {
 		super(
 			type,
 			propertyI18nPrefix,
 			true,
 			FieldVisibilityCheckers.withDisease(disease).andWithCountry(FacadeProvider.getConfigFacade().getCountryLocale()),
 			fieldAccessCheckers);
+		this.disease = disease;
 	}
 
 	protected void addCommonFields() {
@@ -456,4 +459,9 @@ private void updateRequestedTestFields() {
 			getContent().removeComponent(REQUESTED_ADDITIONAL_TESTS_READ_LOC);
 		}
 	}
+
+	
+	public Disease getDisease() {
+		return disease;
+	}
 }

sormas-ui/src/main/java/de/symeda/sormas/ui/samples/humansample/SampleController.java

@@ -100,6 +100,7 @@
 public class SampleController {
 
 	private Disease disease;
+
 	public SampleController() {
 	}
 
@@ -212,7 +213,10 @@ private CollapsiblePathogenTestForm addPathogenTestComponent(
 		separator.setVisible(addSeparator);
 		sampleComponent.addComponent(separator, sampleComponent.getComponentCount() - 1);
 
-		PathogenTestForm pathogenTestForm = new PathogenTestForm(sampleComponent.getWrappedComponent(), true, caseSampleCount, false, true, disease);  // Valid because jurisdiction doesn't matter for entities that are about to be created
+		final Disease pathogenTestFormDisease = disease != null ? disease : sampleComponent.getWrappedComponent().getDisease();
+
+		PathogenTestForm pathogenTestForm =
+			new PathogenTestForm(sampleComponent.getWrappedComponent(), true, caseSampleCount, false, true, pathogenTestFormDisease);  // Valid because jurisdiction doesn't matter for entities that are about to be created
 		// prefill fields
 		if (pathogenTest != null) {
 			pathogenTestForm.setValue(pathogenTest);
@@ -228,8 +232,7 @@ private CollapsiblePathogenTestForm addPathogenTestComponent(
 			pathogenTestResultField.setValue(PathogenTestResultType.PENDING);
 			ComboBox testDiseaseField = pathogenTestForm.getField(PathogenTestDto.TESTED_DISEASE);
 			// setting the disease field value is only necessary if the disease is not null
-			testDiseaseField.setValue(disease);
-
+			testDiseaseField.setValue(pathogenTestFormDisease);
 		}
 		// setup field updates
 		Field testLabField = pathogenTestForm.getField(PathogenTestDto.LAB);