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

3.2.x model hasupdated #12760

Merged
merged 1 commit into from
Apr 1, 2017
Merged

Conversation

Jurigag
Copy link
Contributor

@Jurigag Jurigag commented Mar 28, 2017

Hello!

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the Contributing Guidelines?
  • I have checked that another pull request for this purpose does not exist.
  • I wrote some tests for this PR.

Small description of change: In version 3.1.0 we fixed long sitting bug that snapshot is not updated after save/create/refresh operation being made

Fixed Phalcon\Mvc\Model to create/refresh snapshot after create/update/refresh operation #11007, #11818, #11424

Since 3.1.0 now it's happening, though in some cases people would specifically need to check if field was really changed(i call this updated in this case) after save/create, right now if someone has old code working on this bug it will no longer work. This PR is adding such ability by adding two new methods so by small change it can work as expected.

Thanks

@Jurigag Jurigag changed the base branch from master to 3.2.x March 28, 2017 12:38
@Jurigag Jurigag force-pushed the 3.2.x-model-hasupdated branch 5 times, most recently from 3ee7a0b to 542bcb5 Compare March 28, 2017 16:22
@sergeyklay sergeyklay added this to the 3.2.0 milestone Apr 1, 2017
@sergeyklay
Copy link
Contributor

@Jurigag Could you please rebase

@Jurigag Jurigag force-pushed the 3.2.x-model-hasupdated branch from 542bcb5 to 50fe927 Compare April 1, 2017 18:40
@sergeyklay sergeyklay merged commit 346259c into phalcon:3.2.x Apr 1, 2017
@sergeyklay
Copy link
Contributor

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants