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.

Sign up for GitHub

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

Jump to bottom

Allow to use getters/setters with protected/default visibility #441

Closed
beikov opened this issue Sep 5, 2017 · 1 comment
Closed

Allow to use getters/setters with protected/default visibility #441

beikov opened this issue Sep 5, 2017 · 1 comment
Labels
component: entity-view component: updatable-entity-view kind: enhancement
Milestone
1.2.0

Comments

@beikov
Copy link
Member

beikov commented Sep 5, 2017

A user might want to declare public action methods which mutate the state by calling setters in a more domain driven way. Since the setters would not have to be exposed to be able to call them within such a action method, we should allow to use the modifiers protected/default for setters.
Also see #127 for details.
@beikov beikov changed the title Allow to use setters with protected/default visibility Allow to use getters/setters with protected/default visibility Nov 14, 2017
@beikov
Copy link
Member Author

beikov commented Nov 14, 2017

Related to #474

@beikov beikov added this to the 1.2.0 milestone Feb 9, 2018
beikov added a commit to beikov/blaze-persistence that referenced this issue Feb 9, 2018
@beikov
[Blazebit#441] Reduce column sizes so that DB2 can join and allow non…
b54f6ca 
…-public getters and setters
@beikov beikov closed this as completed Feb 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: entity-view component: updatable-entity-view kind: enhancement
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@beikov