Fixing a bug that causes fatal error when a RowGateway's primary key wer... #5322

Merged
merged 1 commit into from Nov 20, 2013

Conversation

Projects
None yet
2 participants
Contributor

lucascorbeaux commented Oct 22, 2013

...e changed directly : refresh the select's where clause after the update query. Attempting to solve the issue #5232.

@lucascorbeaux lucascorbeaux Fixing a bug that causes fatal error when a RowGateway's primary key …
…were changed directly : refresh the select's where clause after the update query.
40f98a3

@ralphschindler ralphschindler added a commit that referenced this pull request Nov 20, 2013

@ralphschindler ralphschindler PR #5322, Fixes #5232
Merge branch 'hotfix/5232' of git://github.com/lucascorbeaux/zf2 into lucascorbeaux-hotfix/5232

* 'hotfix/5232' of git://github.com/lucascorbeaux/zf2:
  Fixing a bug that causes fatal error when a RowGateway's primary key were changed directly : refresh the select's where clause after the update query.
fc24c43

@ralphschindler ralphschindler added a commit that referenced this pull request Nov 20, 2013

@ralphschindler ralphschindler Merging #5322
Merge branch 'lucascorbeaux-hotfix/5232'

* lucascorbeaux-hotfix/5232:
  Added a few comments to the test, change var name
  Fixing a bug that causes fatal error when a RowGateway's primary key were changed directly : refresh the select's where clause after the update query.
17779c3

@ralphschindler ralphschindler merged commit 40f98a3 into zendframework:master Nov 20, 2013

1 check failed

default The Travis CI build failed
Details

@ralphschindler ralphschindler added a commit that referenced this pull request Nov 20, 2013

@ralphschindler ralphschindler Forward #5322
Merge branch 'lucascorbeaux-hotfix/5232' into develop

* lucascorbeaux-hotfix/5232:
  Added a few comments to the test, change var name
  Fixing a bug that causes fatal error when a RowGateway's primary key were changed directly : refresh the select's where clause after the update query.
f976d54
Member

ralphschindler commented Nov 20, 2013

Nice fix. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment