Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

Wrap errors for setting fields with useful information #60

Merged
merged 2 commits into from
Dec 27, 2017

Conversation

bobvanderlinden
Copy link
Contributor

Currently it is very hard to determine why java.lang.IllegalArgumentException: argument type mismatch errors are thrown when inserting new compositions. This PR makes it easier to determine why these errors are thrown.

@pieterbos
Copy link
Member

Useful :)

@bobvanderlinden
Copy link
Contributor Author

bobvanderlinden commented Dec 5, 2017

@pieterbos useful enough to merge? ;) Let me know if this requires a more specific exception type.

EDIT: I do not have permission to merge. Let me know if I should do something about that.

@pieterbos
Copy link
Member

illegalargumentexception is probably better. But I just haven't merged because I'm not at work this week, and since archie is moving to openehr/archie, i have to do more than just merge it.

@pieterbos pieterbos merged commit afddf8c into master Dec 27, 2017
@pieterbos pieterbos deleted the bobvanderlinden-wrap-field-error branch December 27, 2017 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants