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

added setting to disable automatic strigify of pdo mysql #378

Conversation

roshanbudhathoki
Copy link

Fixes #362 as prescribed by

hungtrinh

@develart-projects
Copy link
Collaborator

@roshanbudhathoki seems like some Unit tests are failing after the change. Could you pls review?

@roshanbudhathoki
Copy link
Author

@develart-projects I am getting following error.

Testing fetchLatersVersion only works when TESTS_ZEND_VERSION_ONLINE_ENABLED is set.

Am I missing to setup something. Even unsure where do I setup this.

@develart-projects
Copy link
Collaborator

@roshanbudhathoki here is my local testing command:

php /usr/share/NetBeansProjects/ZF1-Future/bin/phpunit --colors --log-junit /var/tmp/nb-phpunit-log.xml --configuration /usr/share/NetBeansProjects/ZF1-Future/tests/phpunit.xml /usr/share/NetBeansProjects/ZF1-Future/tests/AllTests.php

@develart-projects develart-projects added the requested change some changes are needed before the merge label Sep 13, 2023
@roshanbudhathoki
Copy link
Author

@develart-projects I have fixed and updated the code, should work now. Please approve so that CI can run and give us the test results.

@develart-projects develart-projects added this to the 1.24.0 milestone Sep 19, 2023
@develart-projects develart-projects merged commit d9ef42a into Shardj:master Sep 19, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requested change some changes are needed before the merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zend db returning everything as string
2 participants