Skip to content

Add JSONB config for useBigIntegerStringAdapter and useBigDecimalStri…#136

Closed
jgallimore wants to merge 1 commit intoapache:masterfrom
jgallimore:big-decimal-integer-adapter-config
Closed

Add JSONB config for useBigIntegerStringAdapter and useBigDecimalStri…#136
jgallimore wants to merge 1 commit intoapache:masterfrom
jgallimore:big-decimal-integer-adapter-config

Conversation

@jgallimore
Copy link
Copy Markdown
Contributor

…ngAdapter

Following on from #134, could this change be considered?

This should allow this:

final JsonbConfig config = new JsonbConfig().withPropertyNamingStrategy(PropertyNamingStrategy.IDENTITY)
          .setProperty("johnzon.useBigIntegerStringAdapter", "false")
          .setProperty("johnzon.useBigDecimalStringAdapter", "false")
          ;

@jgallimore
Copy link
Copy Markdown
Contributor Author

This is actually duplicating this: https://github.com/apache/johnzon/blob/master/johnzon-jsonb/src/main/java/org/apache/johnzon/jsonb/JohnzonBuilder.java#L238-L246, so I'll close this and backport this section to 1.2.x if needed.

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.

2 participants