-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Turn OFF getMyProperty() method during DB save. #1646
Comments
I think we should do this cuz methods like getMyProperty() and setMyProperty() should change property value outside the database. Am I wrong? |
Already know it is more complicated even every test in travis were passed. But it works ok as far as we don't have to validation rules like this:
Or we just should assume that in my case this rule should looks like: |
I think a better solution might be to create a new method, Should take care of it, right? |
My entity:
when I set my $my_entity->qsm_year = 2017 I've got printed out 17 (what is correct), but when I am trying to save it in database I've got 2017 in DB. I think we should use similar approach as is in #1373 .
I will PR in a while cuz I'd like to see if it won't break any tests.
The text was updated successfully, but these errors were encountered: