Skip to content

Remove the "Optional" property from the CourtLocationInfo class#366

Merged
BryceStevenWilley merged 1 commit intomainfrom
nooptional_on_responses
Apr 29, 2026
Merged

Remove the "Optional" property from the CourtLocationInfo class#366
BryceStevenWilley merged 1 commit intomainfrom
nooptional_on_responses

Conversation

@BryceStevenWilley
Copy link
Copy Markdown
Collaborator

Jackson makes the ridiculous decision to stop parsing an object halfway through when this happens, printing out the error in the serialized string. Used to be parsed as {"empty": false, "present": true}, which is admittedly not a good serialization either (would be broken if it was used). Error below:

Fixed by storing the object specifically as an enum, TRUE, FALSE or DEFAULT (not used anywhere else, so okay for us to do this).

Java 8 optional type java.util.Optional<java.lang.Boolean> not supported by default: add Module "com.fasterxml.jackson.datatype:jackson-datatype-jdk8" to enable handling (or disable MapperFeature.REQUIRE_HANDLERS_FOR_JAVA8_OPTIONALS) (through reference chain: edu.suffolk.litlab.efsp.ecfcodes.tyler.CourtLocationInfo["allowserviceoninitial"])

Jackson makes the ridiculous decision to stop parsing an object halfway through when this happens, printing
out the error in the serialized string. Used to be parsed as `{"empty": false, "present": true}`, which is
admittedly not a good serialization either (would be broken if it was used). Error below:

Fixed by storing the object specifically as an enum, TRUE, FALSE or DEFAULT (not used anywhere else, so okay for us to do this). 

> Java 8 optional type `java.util.Optional<java.lang.Boolean>` not supported by default: add Module "com.fasterxml.jackson.datatype:jackson-datatype-jdk8" to enable handling (or disable `MapperFeature.REQUIRE_HANDLERS_FOR_JAVA8_OPTIONALS`) (through reference chain: edu.suffolk.litlab.efsp.ecfcodes.tyler.CourtLocationInfo["allowserviceoninitial"])
@BryceStevenWilley BryceStevenWilley merged commit 09c34c0 into main Apr 29, 2026
4 checks passed
@BryceStevenWilley BryceStevenWilley deleted the nooptional_on_responses branch April 29, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant