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

Introduce Spring's AliasFor declaration in MapperScan #462

Merged
merged 1 commit into from
May 11, 2022

Conversation

chenqimiao
Copy link
Contributor

@chenqimiao chenqimiao commented Mar 19, 2020

Introduce Spring's AliasFor declaration in MapperScan, We only need to get all the attributes through basePackages , without checking value, even if the user only fills in value.
Also this annotation AliasFor adds readability to some extent, I think this is a nice feature and hope you can consider introducing, looking forward to your reply.

@chenqimiao chenqimiao changed the title Introduce Spring's AliasFor semantics in MapperScan Introduce Spring's AliasFor declaration in MapperScan Mar 29, 2020
@chenqimiao
Copy link
Contributor Author

Can someone review this PR ?

@kazuki43zoo kazuki43zoo added the enhancement Improve a feature or add a new feature label Apr 1, 2020
@kazuki43zoo kazuki43zoo added this to the 2.1.x milestone Apr 1, 2020
@kazuki43zoo
Copy link
Member

@chenqimiao

Thanks for your contribution! Your suggestion is reasonable and correct!! But I have no plan that apply this PR at maintenance version up (e.g. 2.0.5, 2.0.6, ...) because this changes broke a current behavior. I will plan to merge this PR at next minor or major version up(e.g. 2.1.0, 3.0.0).

P.S.
Behavior is following when specify both attribute.

  • Before merge: Merge attribute values.
  • After merge: Error

@kazuki43zoo kazuki43zoo added the no backward compatibility Includes change no backward compatibility for previous version label Apr 1, 2020
@kazuki43zoo kazuki43zoo self-assigned this Apr 1, 2020
@gjyu63
Copy link

gjyu63 commented Jun 13, 2020

LGTM

@kazuki43zoo kazuki43zoo merged commit 49c1e3c into mybatis:master May 11, 2022
@kazuki43zoo
Copy link
Member

@chenqimiao I've merged! thanks!!

@kazuki43zoo kazuki43zoo added polishing Improve a implementation code or doc without change in current behavior/content and removed enhancement Improve a feature or add a new feature labels May 11, 2022
@chenqimiao chenqimiao deleted the polishing-@MapperScan branch May 14, 2022 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no backward compatibility Includes change no backward compatibility for previous version polishing Improve a implementation code or doc without change in current behavior/content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants