Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Fix and test case for: RowGateway primary key not null constraint do not fail with empty string #6335

Merged
merged 6 commits into from
Aug 7, 2014
Merged

Conversation

jeanCarloMachado
Copy link
Contributor

When passing quotes as a empty string on the primary key column the initialize fails to recognize the empty value because of the array casting on the constructor.

@jeanCarloMachado
Copy link
Contributor Author

There's 3 trash commits, but, since it does not alter nothing from the current tree, my current knowledge points no problem from that.

$refRowGatewayProp->setValue($this->rowGateway, $rgPropertyValue);
}
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EOF EOL

@weierophinney weierophinney added this to the 2.3.2 milestone Aug 6, 2014
weierophinney added a commit that referenced this pull request Aug 7, 2014
Fix and test case for: RowGateway primary key not null constraint do not fail with empty string
@weierophinney weierophinney merged commit 1f53530 into zendframework:master Aug 7, 2014
weierophinney added a commit that referenced this pull request Aug 7, 2014
- Removed unused/unnecessary method from test case
- Added import statements
- Added whitespace
- Modified test to use setExpectedException() instead of try/catch
weierophinney added a commit that referenced this pull request Aug 7, 2014
weierophinney added a commit that referenced this pull request Aug 7, 2014
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants