Fixed bug that didn't allow the connection to the SQLite database to be closed #5887

Closed
wants to merge 1 commit into
from

3 participants

@brian978

See issue #5228 for more details

brian978 Fixed bug that didn't allow the connection to the SQLite database to …
…be closed when quoting a value using the Platform object
9d08d9f
@ralphschindler ralphschindler added the Db label Mar 1, 2014
@ralphschindler ralphschindler added this to the 2.2.6 milestone Mar 1, 2014
@ralphschindler ralphschindler self-assigned this Mar 1, 2014
@ralphschindler
Zend Framework member

Can you change this PR to master? Seeing as though this is a bugfix, I wanted to get it into 2.2

@ralphschindler
Zend Framework member

Also, there seems to be some rouge files coming in:

Adapter/Platform/_files/.gitkeep b/tests/ZendTest/Db/Adapter/Platform/_files/.gitkeep

I think you mean to keep one of those but not the other?

@weierophinney
Zend Framework member

some rouge files

"rogue", not "rouge". I'm pretty sure they're not red. :) (sorry, @ralphschindler, could not resist!)

@weierophinney weierophinney added a commit that closed this pull request Mar 4, 2014
@weierophinney weierophinney Merge branch 'hotfix/5887-master'
Close #5887
Fixes #5228
6c73f61
@weierophinney weierophinney added a commit that referenced this pull request Mar 4, 2014
@weierophinney weierophinney Merge branch 'hotfix/5887-master' into develop
Forward port #5887
b7d9934
@weierophinney
Zend Framework member

Cherry-picked to master, and merged to develop.

@brian978

@ralphschindler I wanted to keep the tests/ZendTest/Db/Adapter/Platform/_files/ folder so the unit test could write the database file in it

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