Skip to content

[Feature Request] - properties in @MappedSuperclass support rename by extended entity #1002

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

Open
Arkmusn opened this issue Apr 24, 2025 · 2 comments
Labels
enhancement New feature or request pending Postpone plans because of more important tasks

Comments

@Arkmusn
Copy link

Arkmusn commented Apr 24, 2025

Reason

I can't filter for because the colume name is different between super class and entity.

Description

Super class:

@MappedSuperclass
public interface UserInfoVer1xx {
    @Key
    @Nullable String middlewareUid();

    @Key
    @Nullable String mac();
}

Extented Entity:

@Entity
public interface PreRegistrationRecord extends UserInfoVer1 {
    // the column name in this table is 'mac_id'
    // but column name defined in super class is 'mac'
    @Nullable String macId()
}

Existing solutions

No response

@Arkmusn Arkmusn added the enhancement New feature or request label Apr 24, 2025
@babyfish-ct
Copy link
Owner

Here, macId should be decorated by @IdView

If it is still no OK, please move the @idview into super type.

This task (derived type overrides) is also planned in final version, there are many more important tasks now, So I will add pending flag to this issue

@babyfish-ct babyfish-ct added the pending Postpone plans because of more important tasks label Apr 24, 2025
@Arkmusn
Copy link
Author

Arkmusn commented Apr 25, 2025

Here, macId should be decorated by @IdView

If it is still no OK, please move the @idview into super type.

This task (derived type overrides) is also planned in final version, there are many more important tasks now, So I will add pending flag to this issue

Thanks for the reply. Looking forward to the final version.

There may be a misunderstanding in naming property 🤣. macId is not an id . It's just a normal property.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending Postpone plans because of more important tasks
Projects
None yet
Development

No branches or pull requests

2 participants