Can I use MapperFeature.FIX_FIELD_NAME_UPPER_CASE_PREFIX with ParameterNamesModule?

[bananayong]

Recently I read the cowtowncoder blog
First, thank you for releasing the most wanted features!

I thought that it would be possible to use FIX_FIELD_NAME_UPPER_CASE_PREFIX feature with ParameterNamesModule.
However, I ran some tests, and it didn't work.

    static class Apple {
        private final String iPhone;

        public Apple(String iPhone) {
            this.iPhone = iPhone;
        }

        public String getIPhone() {
            return iPhone;
        }

        @Override
        public String toString() {
            return "Apple{" +
                    "iPhone='" + iPhone + '\'' +
                    '}';
        }
    }

    public static void main(String[] args) throws JsonProcessingException {
        ObjectMapper mapper = JsonMapper.builder()
                .addModule(new ParameterNamesModule())
                .enable(MapperFeature.FIX_FIELD_NAME_UPPER_CASE_PREFIX)
                .build();
        Apple apple = new Apple("iPhone 15");
        String json = mapper.writeValueAsString(apple);
        System.out.println(json); // {"iPhone":"iPhone 15"}

        Apple result = mapper.readValue(json, Apple.class); // Error thrown
        System.out.println(result);
    }

It seems like the property name is determined by the constructor parameter and FIX_FIELD_NAME_UPPER_CASE_PREFIX feature is not applied. Is that intended? Or will it be improved in the next release?

[cowtowncoder]

There should be nothing preventing use of ParameterNamesModule but it is possible that handling could be incomplete. If so, would definitely want to fix this.

Thank you for reporting the problem!

[bananayong]

Great!
If this issue is resolved, it would be very helpful because many of my Spring Boot applications use ParameterNamesModule.

[cowtowncoder]

Right, the issue is more with Constructor Parameter names instead of Field names, but PNM is what can provide names for constructor parameters.

[cowtowncoder]

Fixed for 2.20.1 (see #5292).

[bananayong]

Thank you! 👍