5368-Lenient-error-handler-doesn't-always-include-location-information (hapifhir#6678)

* 5368---Lenient-error-handler-doesn't-always-include-location-information

* Credit for hapifhir#6678

---------

Co-authored-by: James Agnew <[email protected]>

Author: elavy-harris

hapi-fhir-base/src/main/java/ca/uhn/fhir/parser/LenientErrorHandler.java

@@ -73,7 +73,7 @@ public LenientErrorHandler(boolean theLogErrors) {
 	@Override
 	public void containedResourceWithNoId(IParseLocation theLocation) {
 		if (myLogErrors) {
-			ourLog.warn("Resource has contained child resource with no ID");
+			ourLog.warn("{}Resource has contained child resource with no ID", describeLocation(theLocation));
 		}
 	}
 
@@ -87,8 +87,9 @@ public void incorrectJsonType(
 			ScalarType theFoundScalarType) {
 		if (myLogErrors) {
 			if (ourLog.isWarnEnabled()) {
-				String message = createIncorrectJsonTypeMessage(
-						theElementName, theExpected, theExpectedScalarType, theFound, theFoundScalarType);
+				String message = describeLocation(theLocation) +
+						createIncorrectJsonTypeMessage(
+							theElementName, theExpected, theExpectedScalarType, theFound, theFoundScalarType);
 				ourLog.warn(message);
 			}
 		}
@@ -134,7 +135,7 @@ public boolean isErrorOnInvalidExtension() {
 	@Override
 	public void missingRequiredElement(IParseLocation theLocation, String theElementName) {
 		if (myLogErrors) {
-			ourLog.warn("Resource is missing required element: {}", theElementName);
+			ourLog.warn("{}Resource is missing required element: {}", describeLocation(theLocation), theElementName);
 		}
 	}
 

hapi-fhir-docs/src/main/resources/ca/uhn/hapi/fhir/changelog/8_2_0/5368-add-location-details-to-errorhandler.yaml

@@ -0,0 +1,5 @@
+---
+type: add
+issue: 5368
+title: "Several error messages reported by the LenientErrorHandler (the default parser error handler)
+  did not include location information in their message. Thanks to Elliot Lavy for the contribution!"

pom.xml

@@ -969,6 +969,11 @@
 			<name>Mangala Ekanayake</name>
 			<organization>Cambio</organization>
 		</developer>
+		<developer>
+			<id>elavy-harris</id>
+			<name>Eliott Lavy</name>
+			<organization>Harris</organization>
+		</developer>
 	</developers>
 
 	<licenses>