Skip to content

Fix #221 Support alternate radixes for numeric values #5317

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: 2.x
Choose a base branch
from
Open

Fix #221 Support alternate radixes for numeric values #5317

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
722182b
Fix #221 Support alternate radixes for numeric values
tiger9800 Sep 16, 2025
6096e27
Fix #221 - address PR comments
tiger9800 Sep 16, 2025
7a211db
Fix #221 - address PR comments
tiger9800 Sep 16, 2025
5db64fa
Fix #221 - address PR comments
tiger9800 Sep 16, 2025
229d691
Fix #221 - add test
tiger9800 Sep 16, 2025
7dabb4b
Fix #221 - address PR comments
tiger9800 Sep 16, 2025
49424e5
Merge branch '2.x' into 2.x
cowtowncoder Sep 18, 2025
5599e79
Fix #221 - wire default radix via ConfigOverrides
tiger9800 Sep 19, 2025
ac1c279
Merge branch '2.x' into 2.x
cowtowncoder Sep 26, 2025
0f42bc9
Merge branch '2.x' into 2.x
cowtowncoder Oct 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions src/main/java/com/fasterxml/jackson/databind/DeserializationConfig.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -373,6 +373,10 @@ public DeserializationConfig withRootName(PropertyName rootName) {
return new DeserializationConfig(this, rootName);
}

public DeserializationConfig withRadix(int radix) {
return _withBase(_base.withRadix(radix));
}

@Override
public DeserializationConfig withView(Class<?> view) {
return (_view == view) ? this : new DeserializationConfig(this, view);
Expand Down
16 changes: 15 additions & 1 deletion src/main/java/com/fasterxml/jackson/databind/ObjectMapper.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -402,6 +402,7 @@ public boolean useForType(JavaType t)
// 16-May-2009, tatu: Ditto ^^^
protected final static AnnotationIntrospector DEFAULT_ANNOTATION_INTROSPECTOR = new JacksonAnnotationIntrospector();

protected static final int DEFAULT_RADIX = 10;
/**
* Base settings contain defaults used for all {@link ObjectMapper}
* instances.
Expand All @@ -419,7 +420,7 @@ public boolean useForType(JavaType t)
// Since 2.12:
new DefaultAccessorNamingStrategy.Provider(),
// Since 2.16: [databind#2502] Add a way to configure Caches Jackson uses
DefaultCacheProvider.defaultInstance()
DefaultCacheProvider.defaultInstance(), DEFAULT_RADIX
);

/*
Expand Down Expand Up @@ -2545,6 +2546,19 @@ public ObjectMapper setDateFormat(DateFormat dateFormat)
return this;
}

/**
* Method for configuring the radix to use when serializing integral types
* as strings.
*
* @since 2.21
*/
public ObjectMapper setRadix(int radix)
{
_deserializationConfig = _deserializationConfig.withRadix(radix);
_serializationConfig = _serializationConfig.withRadix(radix);
return this;
}

/**
* @since 2.5
*/
Expand Down
4 changes: 4 additions & 0 deletions src/main/java/com/fasterxml/jackson/databind/SerializationConfig.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -371,6 +371,10 @@ public SerializationConfig with(SubtypeResolver str) {
return (str == _subtypeResolver)? this : new SerializationConfig(this, str);
}

public SerializationConfig withRadix(int radix) {
return _withBase(_base.withRadix(radix));
}

@Override
public SerializationConfig withView(Class<?> view) {
return (_view == view) ? this : new SerializationConfig(this, view);
Expand Down
120 changes: 90 additions & 30 deletions src/main/java/com/fasterxml/jackson/databind/cfg/BaseSettings.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ public final class BaseSettings
*/
TimeZone.getTimeZone("UTC");

private static final int DEFAULT_RADIX = 10;

/*
/**********************************************************
/* Configuration settings; introspection, related
Expand Down Expand Up @@ -155,21 +157,26 @@ public final class BaseSettings
*/
protected final CacheProvider _cacheProvider;

/**
* Default radix to use when serializing/deserializing integers to string.
*/
protected final int _defaultRadix;

/*
/**********************************************************
/* Construction
/**********************************************************
*/

/**
* @since 2.19
* @since 2.21
*/
public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
PropertyNamingStrategy pns, EnumNamingStrategy ens, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming,
CacheProvider cacheProvider)
PropertyNamingStrategy pns, EnumNamingStrategy ens, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming,
CacheProvider cacheProvider, int defaultRadix)
{
_classIntrospector = ci;
_annotationIntrospector = ai;
Expand All @@ -185,6 +192,35 @@ public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
_typeValidator = ptv;
_accessorNaming = accNaming;
_cacheProvider = cacheProvider;
_defaultRadix = defaultRadix;
}

/**
* @since 2.19
* @deprecated since 2.21, use variant that take defaultRadix
*/
public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
PropertyNamingStrategy pns, EnumNamingStrategy ens, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming,
CacheProvider cacheProvider)
{
this(ci,
ai,
pns,
ens,
tf,
typer,
dateFormat,
hi,
locale,
tz,
defaultBase64,
ptv,
accNaming,
cacheProvider,
DEFAULT_RADIX);
}

/**
Expand All @@ -193,11 +229,11 @@ public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
*/
@Deprecated
public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
PropertyNamingStrategy pns, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming,
CacheProvider cacheProvider)
PropertyNamingStrategy pns, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming,
CacheProvider cacheProvider)
{
this(ci, ai, pns, null, tf, typer, dateFormat, hi, locale, tz, defaultBase64, ptv, accNaming, cacheProvider);
}
Expand All @@ -208,10 +244,10 @@ public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
*/
@Deprecated
public BaseSettings(ClassIntrospector ci, AnnotationIntrospector ai,
PropertyNamingStrategy pns, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming)
PropertyNamingStrategy pns, TypeFactory tf,
TypeResolverBuilder<?> typer, DateFormat dateFormat, HandlerInstantiator hi,
Locale locale, TimeZone tz, Base64Variant defaultBase64,
PolymorphicTypeValidator ptv, AccessorNamingStrategy.Provider accNaming)
{
this(ci, ai, pns, tf, typer, dateFormat, hi, locale, tz, defaultBase64, ptv, accNaming,
DefaultCacheProvider.defaultInstance());
Expand All @@ -237,7 +273,8 @@ public BaseSettings copy() {
_defaultBase64,
_typeValidator,
_accessorNaming,
_cacheProvider);
_cacheProvider,
_defaultRadix);
}

/*
Expand All @@ -252,7 +289,7 @@ public BaseSettings withClassIntrospector(ClassIntrospector ci) {
}
return new BaseSettings(ci, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings withAnnotationIntrospector(AnnotationIntrospector ai) {
Expand All @@ -261,7 +298,7 @@ public BaseSettings withAnnotationIntrospector(AnnotationIntrospector ai) {
}
return new BaseSettings(_classIntrospector, ai, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings withInsertedAnnotationIntrospector(AnnotationIntrospector ai) {
Expand All @@ -288,7 +325,7 @@ public BaseSettings withPropertyNamingStrategy(PropertyNamingStrategy pns) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, pns, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

/**
Expand All @@ -300,7 +337,7 @@ public BaseSettings withEnumNamingStrategy(EnumNamingStrategy ens) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, ens,
_typeFactory, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

// @since 2.12
Expand All @@ -310,7 +347,7 @@ public BaseSettings withAccessorNaming(AccessorNamingStrategy.Provider p) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, p, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, p, _cacheProvider, _defaultRadix);
}

public BaseSettings withTypeFactory(TypeFactory tf) {
Expand All @@ -319,7 +356,7 @@ public BaseSettings withTypeFactory(TypeFactory tf) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
tf, _typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings withTypeResolverBuilder(TypeResolverBuilder<?> typer) {
Expand All @@ -328,7 +365,7 @@ public BaseSettings withTypeResolverBuilder(TypeResolverBuilder<?> typer) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, typer, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings withDateFormat(DateFormat df) {
Expand All @@ -342,7 +379,7 @@ public BaseSettings withDateFormat(DateFormat df) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, df, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings withHandlerInstantiator(HandlerInstantiator hi) {
Expand All @@ -351,7 +388,7 @@ public BaseSettings withHandlerInstantiator(HandlerInstantiator hi) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory, _typeResolverBuilder, _dateFormat, hi, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

public BaseSettings with(Locale l) {
Expand All @@ -360,7 +397,7 @@ public BaseSettings with(Locale l) {
}
return new BaseSettings(_classIntrospector, _annotationIntrospector, _propertyNamingStrategy, _enumNamingStrategy,
_typeFactory,_typeResolverBuilder, _dateFormat, _handlerInstantiator, l,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

/**
Expand All @@ -383,7 +420,7 @@ public BaseSettings with(TimeZone tz)
return new BaseSettings(_classIntrospector, _annotationIntrospector,
_propertyNamingStrategy, _enumNamingStrategy, _typeFactory,
_typeResolverBuilder, df, _handlerInstantiator, _locale,
tz, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider);
tz, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

/**
Expand All @@ -396,7 +433,7 @@ public BaseSettings with(Base64Variant base64) {
return new BaseSettings(_classIntrospector, _annotationIntrospector,
_propertyNamingStrategy, _enumNamingStrategy, _typeFactory,
_typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, base64, _typeValidator, _accessorNaming, _cacheProvider);
_timeZone, base64, _typeValidator, _accessorNaming, _cacheProvider, _defaultRadix);
}

/**
Expand All @@ -409,7 +446,7 @@ public BaseSettings with(PolymorphicTypeValidator v) {
return new BaseSettings(_classIntrospector, _annotationIntrospector,
_propertyNamingStrategy, _enumNamingStrategy, _typeFactory,
_typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, v, _accessorNaming, _cacheProvider);
_timeZone, _defaultBase64, v, _accessorNaming, _cacheProvider, _defaultRadix);
}

/**
Expand All @@ -425,7 +462,23 @@ public BaseSettings with(CacheProvider cacheProvider) {
return new BaseSettings(_classIntrospector, _annotationIntrospector,
_propertyNamingStrategy, _enumNamingStrategy, _typeFactory,
_typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, cacheProvider);
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, cacheProvider, _defaultRadix);
}

/**
* Fluent factory for constructing a new instance with provided default radix.
*
* @return a new instance with provided defaultRadix.
* @since 2.21
*/
public BaseSettings withRadix(int defaultRadix) {
if (defaultRadix == _defaultRadix) {
return this;
}
return new BaseSettings(_classIntrospector, _annotationIntrospector,
_propertyNamingStrategy, _enumNamingStrategy, _typeFactory,
_typeResolverBuilder, _dateFormat, _handlerInstantiator, _locale,
_timeZone, _defaultBase64, _typeValidator, _accessorNaming, _cacheProvider, defaultRadix);
}

/*
Expand Down Expand Up @@ -503,6 +556,13 @@ public boolean hasExplicitTimeZone() {
public Base64Variant getBase64Variant() {
return _defaultBase64;
}

/**
* @since 2.21
*/
public int getRadix() {
return _defaultRadix;
}

/**
* @since 2.16
Expand Down
10 changes: 10 additions & 0 deletions src/main/java/com/fasterxml/jackson/databind/cfg/MapperConfig.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -671,6 +671,16 @@ public Base64Variant getBase64Variant() {
return _base.getBase64Variant();
}

/**
* Method called during deserialization if a Number needs to be decoded from
* String. Default version just returns radix 10.
*
* @since 2.21
*/
public int getRadix() {
return _base.getRadix();
}

/**
* Method for accessing per-instance shared (baseline/default)
* attribute values; these are used as the basis for per-call
Expand Down
10 changes: 9 additions & 1 deletion src/main/java/com/fasterxml/jackson/databind/cfg/MapperConfigBase.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -682,6 +682,14 @@ public T withRootName(String rootName) {
*/
public abstract T with(SubtypeResolver str);

/**
* Method for constructing and returning a new instance with different
* radix to use.
*
* @since 2.21
*/
public abstract T withRadix(int radix);

/**
* Method for constructing and returning a new instance with different
* view to use.
Expand Down Expand Up @@ -933,8 +941,8 @@ public PropertyName findRootName(Class<?> rawRootType) {
public final Class<?> findMixInClassFor(Class<?> cls) {
return _mixIns.findMixInClassFor(cls);
}

// Not really relevant here (should not get called)

@Override
public MixInResolver copy() {
throw new UnsupportedOperationException();
Expand Down
Loading