Skip to content
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.

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

added SetArrayType, SetArrayTypeDescriptor #495

Closed
wants to merge 1 commit into from

Conversation

Aleksey2093
Copy link

No description provided.

@vladmihalcea
Copy link
Owner

vladmihalcea commented Oct 17, 2022

I think it's easier to adapt the current ListArrayType to work with Set attributes on the entity side since there isn't anything List specific in the Type itself.

I created #496 for that

@vladmihalcea
Copy link
Owner

Fixed.

@khaledE
Copy link

khaledE commented Mar 5, 2024

somehow this completely disappeared, I can't find the SetArrayType in the latest version

hypersistence-utils-hibernate-55
3.7.3

@vladmihalcea
Copy link
Owner

@khaledE
Copy link

khaledE commented Mar 5, 2024

oh interesting for some reason I keep getting this error when I do the same as in the example

org.springframework.orm.jpa.JpaSystemException: Could not set field value [[SOME_KEY]] value by reflection
nested exception is org.hibernate.PropertyAccessException: Could not set field value [[SOME_KEY]] value by reflection

but it works fine when I change the field to list

@vladmihalcea
Copy link
Owner

Check out that example I sent you as it works like a charm. You can compare them in debug to see where they differ.

@khaledE
Copy link

khaledE commented Mar 6, 2024

yes but that only works in the newer versions like 60 or 62
the change was never merged to hypersistence-utils-hibernate-55
I'll keep using my workaround until we're ready to upgrade

thanks!

@vladmihalcea
Copy link
Owner

vladmihalcea commented Mar 6, 2024

@khaledE Then how do you explain that there's also a test case in the hypersistence-utils-hibernate-55 module that also works fine with Set<LocalDateTime>?

https://github.com/vladmihalcea/hypersistence-utils/blob/master/hypersistence-utils-hibernate-55/src/test/java/io/hypersistence/utils/hibernate/type/array/ListArrayTypeTest.java#L508

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.

None yet

3 participants