Zend\Validator\Db\AbstractDb.php and mysqli (issue: 3656) #3658

Closed
wants to merge 2 commits into
from

3 participants

@katalonec

No description provided.

@weierophinney
Zend Framework member

Please see the build status: https://travis-ci.org/zendframework/zf2/jobs/4548074/#L830

This clearly breaks functionality for other databases. I'm thinking you'll need to do some detection of the adapter platform, and vary the functionality accordingly.

@ralphschindler can you verify or disprove my theory?

@ralphschindler
Zend Framework member

The tests actually use mocks, so the mock Statement would also have to return a ParameterContainer (which is why key() doesn't work in tests), I've done this in my fix. See above ^

@weierophinney weierophinney added a commit that closed this pull request Feb 5, 2013
@weierophinney weierophinney Merge branch 'hotfix/3680'
Close #3680
Close #3658
Fixes #3656
9302826
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/3680'
Close #3680
Close #3658
Fixes #3656
2199f5d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment