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

same property converter names conflict #1198

Open
1 task
amoikevin opened this issue Nov 28, 2024 · 1 comment
Open
1 task

same property converter names conflict #1198

amoikevin opened this issue Nov 28, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@amoikevin
Copy link

Is there an existing issue?

Build info

  • ObjectBox version: [4.0.3]
  • OS: [Android ]
  • Device/ABI/architecture: [arm64-v8a ]

Steps to reproduce

file A.kt:

class A {
     @Convert(converter = B.Converter::class, dbType = String::class)
     var b:B?=null

     @Convert(converter = C.Converter::class, dbType = String::class)
     var c:C?=null
}

class B {
     class Converter...
}

class C {
     class Converter...
}

Expected behavior

generated ACursor.java:

...
import apple.ios.watches.model.B;
import apple.ios.watches.model.C;
...
private final B.Converter bConverter = new B.Converter();
private final C.Converter cConverter = new C.Converter();
...

Actual behavior

generated ACursor.java:

...
import apple.ios.watches.model.B.Converter;
import apple.ios.watches.model.C.Converter;
...
private final Converter bConverter = new Converter();
private final Converter cConverter = new Converter();
...

Code

Code

Logs, stack traces

Logs
@amoikevin amoikevin added the bug Something isn't working label Nov 28, 2024
@greenrobot-team
Copy link
Member

greenrobot-team commented Dec 2, 2024

Thanks for reporting! This likely boils down to fully supporting inner classes for converter classes as well, similar to #1006.

Internal issue objectbox-java#159

For anyone else affected, it helps us prioritize issues if you thumbs up the first post (GitHub can sort issues by thumbs ups)!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants