ZacharyKlein ( Zachary Klein )

Fix to allow reading entities with JSON fields using constructor arguments (#1057) Without this fix, when reading entities with JSON fields through DefaultJdbcRepositoryOperations an exception is thrown reading the JSON field: `Cannot convert type [class java.lang.String] with value [{foo: bar}] to target type: interface java.util.Map. Consider defining a TypeConverter bean to handle this case.` This seems partially related to #387 Fixes #1058

view details

push time in an hour

PR merged micronaut-projects/micronaut-data

Fix to allow reading entities with JSON fields using constructor arguments

Without this fix, when reading entities with JSON fields through DefaultJdbcRepositoryOperations an exception is thrown reading the JSON field: Cannot convert type [class java.lang.String] with value [{foo: bar}] to target type: interface java.util.Map. Consider defining a TypeConverter bean to handle this case.

This seems partially related to #387

Fixes #1058

+529 -6

1 commit in safarmer/micronaut-data/tree/fix/relation-with-json micronaut-projects/micronaut-data/tree/master

2 comments

28 changed files

safarmer

pr closed time in an hour

issue closedmicronaut-projects/micronaut-data

data-jdbc cannot read an entity with JSON fields when using constructor arguments

Task List

[x] Steps to reproduce provided
[ ] Stacktrace (if present) provided
[ ] Example that reproduces the problem uploaded to Github
[x] Full description of the issue provided (see below)

Steps to Reproduce

Create an entity that has a JSON field and has constructor arguments (or a record)
Insert a record
Read the entity from a repository method.

Example Entity:

@MappedEntity
public record Foo(
    @Id @GeneratedValue @Nullable Integer id, 
    String uid,
    int version,
    @TypeDef(type = DataType.JSON) Map<String, Object> data) {
}

Expected Behaviour

The entity is read and the JSON data is populated.

Actual Behaviour

Exception with the following error: Cannot convert type [class java.lang.String] with value [{foo: bar}] to target type: interface java.util.Map. Consider defining a TypeConverter bean to handle this case.

Environment Information

Operating System: Linux and MacOS
Micronaut Version: 2.5.6
JDK Version: 15 and 16

closed time in an hour

safarmer

pull request commentmicronaut-projects/micronaut-data

Fix to allow reading entities with JSON fields using constructor arguments

Thanks for the fix @safarmer!

safarmer

comment created time in an hour

push eventmicronaut-projects/micronaut-security

Sergio del Amo

commit sha 025b8ba91750c261c47100d1c696d02b0807c5ea

fix: SupplierUtil package (#698)

view details

Sergio del Amo

commit sha feb9a23adf75625bc3c1e399dab44a6180854e9d

Merge branch '3.0.x' into replace-persist-token

view details

Sergio del Amo

commit sha 2e0e81ee07e749737645425085161e28535286ae

remove unused import

view details

Sergio del Amo

commit sha 51502f8cb33355e5f2fd650b1ed1d10f7c1aec19

test: annotate LoginForm with @Introspected (#699) io.micronaut.core.beans.exceptions.IntrospectionException: No bean introspection available for type [class io.micronaut.security.token.jwt.cookie.LoginForm]. Ensure the class is annotated with io.micronaut.core.annotation.Introspected

view details

Sergio del Amo

commit sha f999508d238271d1dcd588e3b6d0f64c83de8849

Merge branch '3.0.x' into replace-persist-token

view details

push time in an hour

Pull request review commentmicronaut-projects/micronaut-data

Fix to allow reading entities with JSON fields using constructor arguments

 private Object setChildrenAndTriggerPostLoad(Object instance, MappingContext<?>                                     }                                 }                             }++                            if (prop.getDataType() == DataType.JSON && jsonCodec != null) {

I think H2 r2dbc driver doesn't support it yet

safarmer

comment created time in an hour

PR opened micronaut-projects/micronaut-security

BREAK: onApplicationEvent RefreshTokenGeneratedEvent

+16 -18

1 commit in micronaut-projects/micronaut-security/tree/replace-persist-token micronaut-projects/micronaut-security/tree/3.0.x

0 comment

8 changed files

pr created time in an hour

delete branch micronaut-projects/micronaut-security

delete branch : annotate-login-form-with-introspected

delete time in an hour

push eventmicronaut-projects/micronaut-security

Sergio del Amo

commit sha 51502f8cb33355e5f2fd650b1ed1d10f7c1aec19

test: annotate LoginForm with @Introspected (#699) io.micronaut.core.beans.exceptions.IntrospectionException: No bean introspection available for type [class io.micronaut.security.token.jwt.cookie.LoginForm]. Ensure the class is annotated with io.micronaut.core.annotation.Introspected

view details

push time in an hour

PR merged micronaut-projects/micronaut-security

test: annotate LoginForm with @Introspected

io.micronaut.core.beans.exceptions.IntrospectionException: No bean introspection available for type [class io.micronaut.security.token.jwt.cookie.LoginForm]. Ensure the class is annotated with io.micronaut.core.annotation.Introspected

+3 -0

1 commit in micronaut-projects/micronaut-security/tree/annotate-login-form-with-introspected micronaut-projects/micronaut-security/tree/3.0.x

0 comment

1 changed file

sdelamo

pr closed time in an hour

PR opened micronaut-projects/micronaut-security

test: annotate LoginForm with @Introspected

io.micronaut.core.beans.exceptions.IntrospectionException: No bean introspection available for type [class io.micronaut.security.token.jwt.cookie.LoginForm]. Ensure the class is annotated with io.micronaut.core.annotation.Introspected

+3 -0

1 commit in micronaut-projects/micronaut-security/tree/annotate-login-form-with-introspected micronaut-projects/micronaut-security/tree/3.0.x

0 comment

1 changed file

pr created time in an hour

create barnchmicronaut-projects/micronaut-security

branch : annotate-login-form-with-introspected

created branch time in an hour

push eventmicronaut-projects/micronaut-security

Sergio del Amo

commit sha 025b8ba91750c261c47100d1c696d02b0807c5ea

fix: SupplierUtil package (#698)